fix: 🐛 simply copy callable funtions' extra properties

packages/sandbox/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 1.1.4
+
+- [fix] simply copy callable funtions's extra properties.
+
 ## 1.1.2
 
 - [fix] hijacked eventListener were not been removed after sandbox unload. ([#295](https://github.com/ice-lab/icestark/issues/295))
@@ -9,6 +13,7 @@
 ## 1.1.1
 
 - [fix] falsy values except `undefined` would be trapped by proxy window. ([#156](https://github.com/ice-lab/icestark/issues/156))
+
 ## 1.1.0
 
 - [feat] mark access to all properties added to local window by using method `getAddedProperties`.

packages/sandbox/__tests__/index.spec.ts

@@ -129,3 +129,25 @@ describe('eval in sandbox', () => {
   });
 });
 
+describe('callable functions in sandbox', () => {
+  const sandbox = new Sandbox({ multiMode: true });
+
+  test('callable function with extra properties', () => {
+    // @ts-ignore
+    window.axios = function(){};
+    // @ts-ignore
+    axios.create = function(){};
+    let error = null;
+    try {
+      sandbox.execScriptInSandbox(
+        `
+          axios.create();
+        `,
+      );
+    } catch (e) {
+      error = e.message;
+    }
+
+    expect(error).toBe(null);
+  });
+});

packages/sandbox/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ice/sandbox",
-  "version": "1.1.3",
+  "version": "1.1.4",
   "description": "sandbox for execute scripts",
   "main": "lib/index.js",
   "scripts": {

packages/sandbox/src/index.ts

@@ -153,8 +153,17 @@ export default class Sandbox {
         }
 
         if (isWindowFunction(value)) {
-          // fix Illegal invocation
-          return value.bind(originalWindow);
+          // When run into some window's functions, such as `console.table`,
+          // an illegal invocation exception is thrown.
+          const boundValue = value.bind(originalWindow);
+
+          // Axios, Moment, and other callable functions may have additional properties.
+          // Simply copy them into boundValue.
+          for (const key in value) {
+            boundValue[key] = value[key];
+          }
+
+          return boundValue;
         } else {
           // case of window.clientWidth、new window.Object()
           return value;