@jwiedeman/gtm-kit 1.2.0 → 1.2.2

package/README.md

@@ -227,6 +227,10 @@ While `@jwiedeman/gtm-kit` works standalone, we provide framework-specific adapt
 | Vue 3         | `@jwiedeman/gtm-kit-vue`          | `npm install @jwiedeman/gtm-kit @jwiedeman/gtm-kit-vue`          |
 | Nuxt 3        | `@jwiedeman/gtm-kit-nuxt`         | `npm install @jwiedeman/gtm-kit @jwiedeman/gtm-kit-nuxt`         |
 | Next.js       | `@jwiedeman/gtm-kit-next`         | `npm install @jwiedeman/gtm-kit @jwiedeman/gtm-kit-next`         |
+| Svelte        | `@jwiedeman/gtm-kit-svelte`       | `npm install @jwiedeman/gtm-kit @jwiedeman/gtm-kit-svelte`       |
+| SolidJS       | `@jwiedeman/gtm-kit-solid`        | `npm install @jwiedeman/gtm-kit @jwiedeman/gtm-kit-solid`        |
+| Remix         | `@jwiedeman/gtm-kit-remix`        | `npm install @jwiedeman/gtm-kit @jwiedeman/gtm-kit-remix`        |
+| Astro         | `@jwiedeman/gtm-kit-astro`        | `npm install @jwiedeman/gtm-kit @jwiedeman/gtm-kit-astro`        |
 
 ---
 

package/dist/index.cjs

@@ -670,9 +670,11 @@ var ScriptManager = class {
       delay
     });
     this.retryCounters.set(containerId, currentAttempt + 1);
-    setTimeout(() => {
+    const timerId = setTimeout(() => {
+      this.activeTimeouts.delete(containerId);
       this.loadScript(container, targetParent);
     }, delay);
+    this.activeTimeouts.set(containerId, timerId);
   }
   /**
    * Handle final failure after all retries exhausted.
@@ -731,6 +733,11 @@ var ScriptManager = class {
       const stringValue = String(value);
       if (key === "nonce") {
         script.nonce = stringValue;
+        continue;
+      }
+      const lowerKey = key.toLowerCase();
+      if (lowerKey === "src" || lowerKey.startsWith("on")) {
+        continue;
       }
       script.setAttribute(key, stringValue);
     }
@@ -810,11 +817,9 @@ var ScriptManager = class {
             this.insertedScripts.delete(script);
             this.scheduleRetry(container, targetParent);
           } else {
-            settle("failed");
-            const state = this.loadStates.get(containerId);
-            if (state) {
-              state.error = `Script load timeout (${this.scriptTimeout}ms)`;
-            }
+            settled = true;
+            this.clearContainerTimeout(containerId);
+            this.handleFinalFailure(containerId, url, `Script load timeout (${this.scriptTimeout}ms)`, true);
           }
         }
       }, this.scriptTimeout);
@@ -1193,6 +1198,7 @@ var GtmClientImpl = class {
       this.logger.debug("GTM client already initialized; skipping.");
       return;
     }
+    this.initialized = true;
     this.logger.info("Initializing GTM client.", {
       containers: this.containers.map((container) => container.id),
       dataLayerName: this.resolvedDataLayerName
@@ -1235,7 +1241,6 @@ var GtmClientImpl = class {
     this.pushStartEvent();
     this.flushQueue();
     this.scriptManager.ensure(this.containers);
-    this.initialized = true;
   }
   /**
    * Pushes a value to the GTM dataLayer.
@@ -1302,9 +1307,8 @@ var GtmClientImpl = class {
   }
   setConsentDefaults(state, options) {
     if (this.initialized) {
-      this.logger.warn(
-        "setConsentDefaults() called after init(). Consent defaults should be set BEFORE calling init() to ensure proper tag behavior. The defaults will still be pushed, but GTM may have already fired tags with implied consent.",
-        { state, options }
+      throw new Error(
+        "setConsentDefaults() must be called BEFORE init(). Google requires consent defaults to be set before GTM loads to ensure proper tag behavior. Move your setConsentDefaults() call before client.init()."
       );
     }
     if ((options == null ? void 0 : options.waitForUpdate) !== void 0) {
@@ -1693,7 +1697,7 @@ var GtmClientImpl = class {
         criticalIndices.add(i);
       }
     }
-    const targetTrimCount = Math.max(1, excess + 1);
+    const targetTrimCount = Math.max(1, excess);
     let trimmedCount = 0;
     let index = 0;
     while (trimmedCount < targetTrimCount && index < dataLayer.length) {