assign-gingerly 0.0.20 → 0.0.22

package/assignGingerly.js

@@ -1,22 +1,3 @@
-// Polyfill for Map.prototype.getOrInsert and WeakMap.prototype.getOrInsert
-if (typeof Map.prototype.getOrInsertComputed !== 'function') {
-    Map.prototype.getOrInsertComputed = function (key, insert) {
-        if (this.has(key))
-            return this.get(key);
-        const value = insert();
-        this.set(key, value);
-        return value;
-    };
-}
-if (typeof WeakMap.prototype.getOrInsertComputed !== 'function') {
-    WeakMap.prototype.getOrInsertComputed = function (key, insert) {
-        if (this.has(key))
-            return this.get(key);
-        const value = insert();
-        this.set(key, value);
-        return value;
-    };
-}
 /**
  * GUID for global instance map storage to ensure uniqueness across package versions
  */
@@ -36,33 +17,40 @@ export function getInstanceMap() {
  * Base registry class for managing enhancement configurations
  */
 export class EnhancementRegistry {
-    #items = [];
+    #items = new Set();
     push(items) {
         if (Array.isArray(items)) {
-            this.#items.push(...items);
+            items.forEach(item => this.#items.add(item));
         }
         else {
-            this.#items.push(items);
+            this.#items.add(items);
         }
     }
     getItems() {
-        return this.#items;
+        return Array.from(this.#items);
     }
     findBySymbol(symbol) {
-        return this.#items.find(item => {
+        for (const item of this.#items) {
             const symlinks = item.symlinks;
             if (!symlinks)
-                return false;
-            return Object.keys(symlinks).some(key => {
+                continue;
+            const hasSymbol = Object.keys(symlinks).some(key => {
                 if (typeof key === 'symbol' || (typeof symlinks[key] === 'symbol')) {
                     return key === symbol || symlinks[key] === symbol;
                 }
                 return false;
             }) || Object.getOwnPropertySymbols(symlinks).some(sym => sym === symbol);
-        });
+            if (hasSymbol)
+                return item;
+        }
+        return undefined;
     }
     findByEnhKey(enhKey) {
-        return this.#items.find(item => item.enhKey === enhKey);
+        for (const item of this.#items) {
+            if (item.enhKey === enhKey)
+                return item;
+        }
+        return undefined;
     }
 }
 /**

package/assignGingerly.ts

@@ -2,28 +2,21 @@
 
 import { EnhancementConfig } from "./types/assign-gingerly/types";
 
-// Polyfill for Map.prototype.getOrInsert and WeakMap.prototype.getOrInsert
-if (typeof Map.prototype.getOrInsertComputed !== 'function') {
-  Map.prototype.getOrInsertComputed = function(key, insert) {
-    if (this.has(key)) return this.get(key);
-    const value = insert();
-    this.set(key, value);
-    return value;
-  };
-}
-if (typeof WeakMap.prototype.getOrInsertComputed !== 'function') {
-  WeakMap.prototype.getOrInsertComputed = function(key, insert) {
-    if (this.has(key)) return this.get(key);
-    const value = insert();
-    this.set(key, value);
-    return value;
-  };
-}
-
-/**
- * @deprecated Use EnhancementConfig instead
- */
-export type IBaseRegistryItem<T = any> = EnhancementConfig<T>;
+// Polyfill for WeakMap.prototype.getOrInsert
+
+// if (typeof WeakMap.prototype.getOrInsertComputed !== 'function') {
+//   WeakMap.prototype.getOrInsertComputed = function(key, insert) {
+//     if (this.has(key)) return this.get(key);
+//     const value = insert();
+//     this.set(key, value);
+//     return value;
+//   };
+// }
+
+// /**
+//  * @deprecated Use EnhancementConfig instead
+//  */
+// export type IBaseRegistryItem<T = any> = EnhancementConfig<T>;
 
 /**
  * Interface for the options passed to assignGingerly
@@ -53,38 +46,46 @@ export function getInstanceMap(): WeakMap<object, Map<EnhancementConfig, any>> {
  * Base registry class for managing enhancement configurations
  */
 export class EnhancementRegistry {
-  #items: EnhancementConfig[] = [];
+  #items: Set<EnhancementConfig> = new Set();
 
   push(items: EnhancementConfig | EnhancementConfig[]): void {
     if (Array.isArray(items)) {
-      this.#items.push(...items);
+      items.forEach(item => this.#items.add(item));
     } else {
-      this.#items.push(items);
+      this.#items.add(items);
     }
   }
 
   getItems(): EnhancementConfig[] {
-    return this.#items;
+    return Array.from(this.#items);
   }
 
   findBySymbol(symbol: symbol | string): EnhancementConfig | undefined {
-    return this.#items.find(item => {
+    for (const item of this.#items) {
       const symlinks = item.symlinks;
-      if (!symlinks) return false;
-      return Object.keys(symlinks).some(key => {
+      if (!symlinks) continue;
+
+      const hasSymbol = Object.keys(symlinks).some(key => {
         if (typeof key === 'symbol' || (typeof symlinks[key as any] === 'symbol')) {
           return key === symbol || symlinks[key as any] === symbol;
         }
         return false;
       }) || Object.getOwnPropertySymbols(symlinks).some(sym => sym === symbol);
-    });
+
+      if (hasSymbol) return item;
+    }
+    return undefined;
   }
 
   findByEnhKey(enhKey: string | symbol): EnhancementConfig | undefined {
-    return this.#items.find(item => item.enhKey === enhKey);
+    for (const item of this.#items) {
+      if (item.enhKey === enhKey) return item;
+    }
+    return undefined;
   }
 }
 
+
 /**
  * Helper function to check if a string key represents a Symbol.for expression
  */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "assign-gingerly",
-  "version": "0.0.20",
+  "version": "0.0.22",
   "description": "This package provides a utility function for carefully merging one object into another.",
   "homepage": "https://github.com/bahrus/assign-gingerly#readme",
   "bugs": {
@@ -58,12 +58,14 @@
   "scripts": {
     "serve": "node ./node_modules/spa-ssi/serve.js",
     "test": "playwright test",
-    "update": "ncu -u && npm install"
+    "update": "ncu -u && npm install",
+    "safari": "npx playwright wk http://localhost:8000",
+    "chrome": "npx playwright cr http://localhost:8000"
   },
   "devDependencies": {
-    "@playwright/test": "^1.58.2",
+    "@playwright/test": "1.59.0-alpha-2026-02-28",
     "spa-ssi": "0.0.27",
-    "@types/node": "^25.3.0",
+    "@types/node": "^25.3.3",
     "typescript": "^5.9.3"
   }
 }

package/playwright-browser.config.ts

@@ -0,0 +1,42 @@
+import { defineConfig, devices } from '@playwright/test';
+
+/**
+ * Playwright configuration for running tests in actual browser JavaScript engines.
+ * This configuration opens HTML test pages in real browsers instead of running
+ * tests in Node.js.
+ */
+export default defineConfig({
+  testDir: './tests-browser',
+  fullyParallel: true,
+  forbidOnly: !!process.env.CI,
+  retries: process.env.CI ? 2 : 0,
+  workers: process.env.CI ? 1 : undefined,
+  reporter: 'html',
+  
+  use: {
+    baseURL: 'http://localhost:8000',
+    trace: 'on-first-retry',
+  },
+
+  projects: [
+    {
+      name: 'chromium',
+      use: { ...devices['Desktop Chrome'] },
+    },
+    {
+      name: 'firefox',
+      use: { ...devices['Desktop Firefox'] },
+    },
+    {
+      name: 'webkit',
+      use: { ...devices['Desktop Safari'] },
+    },
+  ],
+
+  // Start local web server before running tests
+  webServer: {
+    command: 'npm run serve',
+    port: 8000,
+    reuseExistingServer: !process.env.CI,
+  },
+});

package/playwright.config.ts

@@ -24,7 +24,7 @@ export default defineConfig({
   /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
   use: {
     /* Base URL to use in actions like `await page.goto('/')`. */
-    baseURL: 'http://localhost:5173',
+    baseURL: 'http://localhost:8000',
   },
 
   /* Configure projects for major browsers */
@@ -46,9 +46,9 @@ export default defineConfig({
   ],
 
   /* Run your local dev server before starting the tests */
-  // webServer: {
-  //   command: 'npm run dev',
-  //   url: 'http://localhost:5173',
-  //   reuseExistingServer: !process.env.CI,
-  // },
+  webServer: {
+    command: 'npm run serve',
+    port: 8000,
+    reuseExistingServer: !process.env.CI,
+  },
 });

package/{global.d.ts → types/global.d.ts}

@@ -1,4 +1,4 @@
-// Type declarations for Map.prototype.getOrInsert and WeakMap.prototype.getOrInsert
+// Type declarations for Map.prototype.getOrInsertComputed and WeakMap.prototype.getOrInsertComputed
 // Feature is now supported in all modern browsers (Chrome 146+, Firefox 134+, Safari 18.2+)
 // See: https://web-platform-dx.github.io/web-features-explorer/features/getorinsert/