Provide fake external constructor for Matcher

Classes which depend on identity for behaviours should not be exposed
because this inheritance can be modified at runtime. This commit
provides a fake Matcher constructor which simply calls the real one.

This has a side‐effect of reducing the amount of prototype modification
which this class setup entailed, at the cost of needing to manually
redefine all the properties from the internal Matcher constructor
prototype onto the external one.

diff --git a/string.js b/string.js
index 28412fc98b8854b9b26f248c5e8b8ae98526d033..3dc39acade9259c7c5120c28cde20ad398225426 100644 (file)
--- a/string.js
+++ b/string.js
@@ -10,6 +10,7 @@
 import { bind, call, identity, makeCallable } from "./function.js";
 import {
   defineOwnProperties,
 import { bind, call, identity, makeCallable } from "./function.js";
 import {
   defineOwnProperties,

+  getOwnPropertyDescriptors,

   getPrototype,
   objectCreate,
   setPrototype,
   getPrototype,
   objectCreate,
   setPrototype,


@@ -213,12 +214,25 @@ export const {
       return call(getUnicode, this.#regExp, []);
     }
   };
       return call(getUnicode, this.#regExp, []);
     }
   };

-  const matcherPrototype = setPrototype(
-    Matcher.prototype,
-    rePrototype,
+
+  const matcherConstructor = defineOwnProperties(
+    class extends RegExp {
+      constructor(...args) {
+        return new Matcher(...args);
+      }
+    },
+    {
+      name: { value: "Matcher" },
+      length: { value: 1 },
+    },
+  );
+  const matcherPrototype = defineOwnProperties(
+    matcherConstructor.prototype,
+    getOwnPropertyDescriptors(Matcher.prototype),
+    { constructor: { value: matcherConstructor } },

   );
 
   );
 

-  return { Matcher };
+  return { Matcher: matcherConstructor };

 })();
 
 export const {
 })();
 
 export const {