@mediar-ai/terminator 0.23.1 → 0.23.2

package/package.json

@@ -40,11 +40,11 @@
     }
   },
   "optionalDependencies": {
-    "@mediar-ai/terminator-darwin-arm64": "0.23.1",
-    "@mediar-ai/terminator-darwin-x64": "0.23.1",
-    "@mediar-ai/terminator-linux-x64-gnu": "0.23.1",
-    "@mediar-ai/terminator-win32-arm64-msvc": "0.23.1",
-    "@mediar-ai/terminator-win32-x64-msvc": "0.23.1"
+    "@mediar-ai/terminator-darwin-arm64": "0.23.2",
+    "@mediar-ai/terminator-darwin-x64": "0.23.2",
+    "@mediar-ai/terminator-linux-x64-gnu": "0.23.2",
+    "@mediar-ai/terminator-win32-arm64-msvc": "0.23.2",
+    "@mediar-ai/terminator-win32-x64-msvc": "0.23.2"
   },
   "repository": {
     "type": "git",
@@ -62,5 +62,5 @@
     "test-hook": "powershell.exe -ExecutionPolicy Bypass -File \"../../.git/hooks/pre-push.ps1\""
   },
   "types": "wrapper.d.ts",
-  "version": "0.23.1"
+  "version": "0.23.2"
 }

package/tests/execute-browser-script-wrapper.test.js

@@ -94,10 +94,14 @@ async function testFileInputWithEnvInjection() {
     count: 3,
   });
 
-  // Verify the env was injected into the script
+  // Verify the env was injected into the script as an env object
   assert.ok(
-    receivedScript.includes("const column_positions = "),
-    "column_positions should be injected into script",
+    receivedScript.includes("const env = "),
+    "env object should be injected into script",
+  );
+  assert.ok(
+    receivedScript.includes('"column_positions"'),
+    "column_positions key should be in env object",
   );
   assert.ok(
     receivedScript.includes(JSON.stringify(columnPositions)),

package/tests/fixtures/script-with-env.js

@@ -1,12 +1,16 @@
-// Browser script that uses injected env variables
+// Browser script that uses injected env object
 (() => {
-  // column_positions should be auto-injected by wrapper
-  if (typeof column_positions === 'undefined') {
-    throw new Error('column_positions not injected');
+  // env object should be auto-injected by wrapper
+  if (typeof env === "undefined") {
+    throw new Error("env not injected");
+  }
+
+  if (!env.column_positions) {
+    throw new Error("column_positions not in env");
   }
 
   return {
-    positions: column_positions,
-    count: column_positions.length
+    positions: env.column_positions,
+    count: env.column_positions.length,
   };
-})()
+})();

package/wrapper.js

@@ -270,8 +270,8 @@ async function enhancedExecuteBrowserScript(scriptOrFunction, envOrOptions) {
     );
   }
   if (shouldInjectEnv && env && Object.keys(env).length > 0) {
-    const envDeclarations = Object.entries(env).map(([key, value]) => `const ${key} = ${JSON.stringify(value)};`).join("\n");
-    script = `${envDeclarations}
+    const envObject = `const env = ${JSON.stringify(env)};`;
+    script = `${envObject}
 ${script}`;
   }
   const resultStr = await this._originalExecuteBrowserScript(script);

package/wrapper.ts

@@ -292,10 +292,9 @@ async function enhancedExecuteBrowserScript(
 
   // If env variables are provided and we should inject them (file-based scripts only)
   if (shouldInjectEnv && env && Object.keys(env).length > 0) {
-    const envDeclarations = Object.entries(env)
-      .map(([key, value]) => `const ${key} = ${JSON.stringify(value)};`)
-      .join("\n");
-    script = `${envDeclarations}\n${script}`;
+    // Inject as an env object that scripts can access
+    const envObject = `const env = ${JSON.stringify(env)};`;
+    script = `${envObject}\n${script}`;
   }
 
   // Call the original native method