fix: remove unnecessary isPlainObject check

if the renderable has `defaultProps` it is a ordinary function component which only accepts a props param

Author: sastan

.size-snapshot.json

@@ -1,31 +1,31 @@
 {
   "dist/react-render-callback.umd.js": {
-    "bundled": 4277,
-    "minified": 1687,
-    "gzipped": 735
+    "bundled": 3207,
+    "minified": 1340,
+    "gzipped": 617
   },
   "dist/react-render-callback.esm.js": {
-    "bundled": 2628,
-    "minified": 1511,
-    "gzipped": 631,
+    "bundled": 2490,
+    "minified": 1390,
+    "gzipped": 598,
     "treeshaked": {
       "rollup": {
-        "code": 289,
-        "import_statements": 121
+        "code": 265,
+        "import_statements": 97
       },
       "webpack": {
-        "code": 1369
+        "code": 1312
       }
     }
   },
   "dist/react-render-callback.cjs.js": {
-    "bundled": 2785,
-    "minified": 1625,
-    "gzipped": 667
+    "bundled": 2627,
+    "minified": 1489,
+    "gzipped": 631
   },
   "dist/react-render-callback.umd.min.js": {
-    "bundled": 4310,
-    "minified": 1673,
-    "gzipped": 727
+    "bundled": 3240,
+    "minified": 1326,
+    "gzipped": 608
   }
 }

other/misc-tests/__tests__/build.js

@@ -24,33 +24,41 @@ import * as umdImport from '../../../dist/react-render-callback.umd'
 const cjsRequire = require('../../../') // picks up the main from package.json
 const umdRequire = require('../../../dist/react-render-callback.umd')
 
-test(`import render from 'react-render-callback/dist/react-render-callback.esm'`, () => {
-  expect(esImport.default).toBeInstanceOf(Function)
-})
-test(`import {createRender} from 'react-render-callback/dist/react-render-callback.esm'`, () => {
-  expect(esImport.createRender).toBeInstanceOf(Function)
-})
-
 test(`import render from 'react-render-callback'`, () => {
   expect(cjsImport.default).toBeInstanceOf(Function)
-  expect(cjsImport.createRender).toBeInstanceOf(Function)
 })
 
 test(`import {createRender} from 'react-render-callback'`, () => {
   expect(cjsImport.createRender).toBeInstanceOf(Function)
 })
 
+test(`import render from 'react-render-callback/dist/react-render-callback.esm'`, () => {
+  expect(esImport.default).toBeInstanceOf(Function)
+})
+test(`import {createRender} from 'react-render-callback/dist/react-render-callback.esm'`, () => {
+  expect(esImport.createRender).toBeInstanceOf(Function)
+})
+
 test(`import render from 'react-render-callback/dist/react-render-callback.umd'`, () => {
   expect(umdImport.default).toBeInstanceOf(Function)
+})
+
+test(`import {createRender} from 'react-render-callback/dist/react-render-callback.umd'`, () => {
   expect(umdImport.createRender).toBeInstanceOf(Function)
 })
 
 test(`const render = require('react-render-callback')`, () => {
   expect(cjsRequire).toBeInstanceOf(Function)
+})
+
+test(`const {createRender} = require('react-render-callback')`, () => {
   expect(cjsRequire.createRender).toBeInstanceOf(Function)
 })
 
 test(`const render = require('react-render-callback/dist/react-render-callback.umd')`, () => {
   expect(umdRequire).toBeInstanceOf(Function)
+})
+
+test(`const {createRender} = require('react-render-callback/dist/react-render-callback.umd')`, () => {
   expect(umdRequire.createRender).toBeInstanceOf(Function)
 })

package.json

@@ -38,8 +38,7 @@
     "react": "^0.14.0 || ^15.0.0-0 || ^16.0.0-0"
   },
   "dependencies": {
-    "@babel/runtime": "^7.0.0",
-    "is-plain-object": "^2.0.4"
+    "@babel/runtime": "^7.0.0"
   },
   "devDependencies": {
     "@babel/plugin-transform-runtime": "^7.0.0",

src/createRender.js

@@ -1,7 +1,5 @@
 import {isValidElement, createElement, cloneElement} from 'react'
 
-import isPlainObject from 'is-plain-object'
-
 import constant from './internal/constant'
 import ignoredValuesToNull from './internal/ignoredValuesToNull'
 import isReactComponent from './internal/isReactComponent'
@@ -14,7 +12,7 @@ export default (renderable, options) => {
   if (typeof renderable === 'function') {
     return (...args) =>
       ignoredValuesToNull(
-        renderable.defaultProps && args.length === 1 && isPlainObject(args[0])
+        renderable.defaultProps
           ? renderable({...renderable.defaultProps, ...args[0]})
           : renderable(...args),
       )