fix: strip GCL_VARIABLE_ (empty suffix) from env, add ignorePredefinedVars tests

Always delete GCL_VARIABLE_* entries from process.env before yargs
parses, including the bare GCL_VARIABLE_ key, so strictOptions() does
not reject it. Add tests for Argv.ignorePredefinedVars getter.

Author: firecow

src/argv.ts

@@ -27,8 +27,9 @@ export function stripGclVariableEnvVars (env: Record<string, string | undefined>
     const stripped: Record<string, string> = {};
     for (const key of Object.keys(env)) {
         if (!key.startsWith(prefix) || env[key] == null) continue;
-        if (key.length <= prefix.length) continue;
-        stripped[key] = env[key]!;
+        if (key.length > prefix.length) {
+            stripped[key] = env[key]!;
+        }
         delete env[key];
     }
     return stripped;

tests/test-cases/gcl-variable-env/integration.test.ts

@@ -1,6 +1,7 @@
-import {stripGclVariableEnvVars, injectGclVariableEnvVars} from "../../../src/argv.js";
+import {stripGclVariableEnvVars, injectGclVariableEnvVars, Argv} from "../../../src/argv.js";
 import {execFile} from "child_process";
 import {promisify} from "util";
+import {WriteStreamsMock} from "../../../src/write-streams.js";
 
 const execFileAsync = promisify(execFile);
 
@@ -23,11 +24,11 @@ describe("stripGclVariableEnvVars", () => {
         expect(env["GCL_CWD"]).toBe("/tmp");
     });
 
-    test("skips GCL_VARIABLE_ with empty name", () => {
+    test("strips GCL_VARIABLE_ with empty name from env but does not return it", () => {
         const env: Record<string, string | undefined> = {"GCL_VARIABLE_": "empty"};
         const stripped = stripGclVariableEnvVars(env);
         expect(stripped).toEqual({});
-        expect(env["GCL_VARIABLE_"]).toBe("empty");
+        expect(env["GCL_VARIABLE_"]).toBeUndefined();
     });
 
     test("skips null/undefined values", () => {
@@ -92,6 +93,28 @@ describe("injectGclVariableEnvVars", () => {
     });
 });
 
+describe("Argv.ignorePredefinedVars", () => {
+    test("returns empty array by default", async () => {
+        const argv = await Argv.build({cwd: "tests/test-cases/gcl-variable-env"}, new WriteStreamsMock());
+        expect(argv.ignorePredefinedVars).toEqual([]);
+    });
+
+    test("handles array input", async () => {
+        const argv = await Argv.build({cwd: "tests/test-cases/gcl-variable-env", ignorePredefinedVars: ["VAR1", "VAR2"]}, new WriteStreamsMock());
+        expect(argv.ignorePredefinedVars).toEqual(["VAR1", "VAR2"]);
+    });
+
+    test("splits comma-separated values in array elements", async () => {
+        const argv = await Argv.build({cwd: "tests/test-cases/gcl-variable-env", ignorePredefinedVars: ["VAR1,VAR2"]}, new WriteStreamsMock());
+        expect(argv.ignorePredefinedVars).toEqual(["VAR1", "VAR2"]);
+    });
+
+    test("handles string input for backwards compatibility", async () => {
+        const argv = await Argv.build({cwd: "tests/test-cases/gcl-variable-env", ignorePredefinedVars: "VAR1,VAR2"}, new WriteStreamsMock());
+        expect(argv.ignorePredefinedVars).toEqual(["VAR1", "VAR2"]);
+    });
+});
+
 test("GCL_VARIABLE_* env vars are injected into job output via CLI", async () => {
     const {stdout} = await execFileAsync("bun", ["src/index.ts", "test-job", "--cwd", "tests/test-cases/gcl-variable-env"], {
         env: {