Remove nativeFunctions option

Author: MichaelXF

src/options.ts

@@ -406,20 +406,6 @@ export interface ObfuscateOptions {
      */
     context?: string[];
 
-    /**
-     * ### `lock.nativeFunctions`
-     *
-     * Set of global functions that are native. Such as `require`, `fetch`. If these variables are modified the program crashes.
-     * Set to `true` to use the default set of native functions. (`string[]/true/false`)
-     *
-     * - Potency Low
-     * - Resilience Medium
-     * - Cost Medium
-     *
-     * [See all settings here](https://github.com/MichaelXF/js-confuser/blob/master/README.md#options)
-     */
-    nativeFunctions?: string[] | Set<string> | boolean;
-
     /**
      * ### `lock.startDate`
      *

src/transforms/lock/lock.ts

@@ -219,9 +219,7 @@ export default class Lock extends Transform {
     if (this.options.lock.domainLock && this.options.lock.domainLock.length) {
       choices.push("domainLock");
     }
-    if (this.options.lock.nativeFunctions) {
-      choices.push("nativeFunction");
-    }
+
     if (this.options.lock.context && this.options.lock.context.length) {
       choices.push("context");
     }
@@ -312,45 +310,6 @@ export default class Lock extends Transform {
 
           break;
 
-        case "nativeFunction":
-          var set = this.options.lock.nativeFunctions;
-          if (set === true) {
-            if (this.options.target == "node") {
-              set = new Set(["Function", "String"]);
-            } else {
-              set = new Set(["Function", "String", "fetch"]);
-            }
-          }
-          if (Array.isArray(set)) {
-            set = new Set(set);
-          }
-          if (!set) {
-            set = new Set();
-          }
-
-          var fn = choice(Array.from(set));
-          if (fn) {
-            test = Template(
-              `(${fn}+"").indexOf("[native code]") == -1`
-            ).single().expression;
-
-            if (Math.random() > 0.5) {
-              test = Template(
-                `${fn}.toString().split("{ [native code] }").length <= 1`
-              ).single().expression;
-            }
-
-            nodes.push(
-              IfStatement(
-                test,
-                this.getCounterMeasuresCode(object, parents) || [],
-                null
-              )
-            );
-          }
-
-          break;
-
         case "startDate":
           test = BinaryExpression(
             "<",

test/transforms/lock/lock.test.ts

@@ -64,6 +64,7 @@ it("should work with endDate and call countermeasures function", async () => {
   expect(value).toStrictEqual(true);
 });
 
+// REMOVED FEATURE:
 // it("strings should be encoded when startDate and endDate are given", async () => {
 //   var startDate = await JsConfuser.obfuscate(` input("ENCODED_STRING") `, {
 //     target: "node",
@@ -82,54 +83,6 @@ it("should work with endDate and call countermeasures function", async () => {
 //   expect(value).toStrictEqual("ENCODED_STRING");
 // });
 
-it("should work with nativeFunctions and call countermeasures function", async () => {
-  var output = await JsConfuser.obfuscate(
-    ` function countermeasures(){ input(true) } `,
-    {
-      target: "node",
-      lock: {
-        nativeFunctions: ["fetch"],
-        countermeasures: "countermeasures",
-      },
-    }
-  );
-
-  // custom function, not "native"
-  var fetch = () => {};
-
-  var value = "never_called";
-  function input(valueIn) {
-    value = valueIn;
-  }
-
-  eval(output);
-  expect(value).toStrictEqual(true);
-});
-
-it("should work with nativeFunctions and not call countermeasures function when correct", async () => {
-  var output = await JsConfuser.obfuscate(
-    ` function countermeasures(){ input(true) } `,
-    {
-      target: "node",
-      lock: {
-        nativeFunctions: ["fetch"],
-        countermeasures: "countermeasures",
-      },
-    }
-  );
-
-  // bound functions return the "[native code]" string
-  var fetch = (() => {}).bind(this);
-
-  var value = "never_called";
-  function input(valueIn) {
-    value = valueIn;
-  }
-
-  eval(output);
-  expect(value).toStrictEqual("never_called");
-});
-
 it("countermeasures function should still work even with renameVariables enabled", async () => {
   var output = await JsConfuser.obfuscate(
     ` function countermeasures(){ input(true) } `,