test: keep commitments client boundary

shakkernerd · shakkernerd · commit f0f5da0e3941 · 2026-06-24T20:52:00.000+01:00
diff --git a/scripts/e2e/commitments-safety-docker-client.ts b/scripts/e2e/commitments-safety-docker-client.ts
@@ -15,7 +15,6 @@ import {
   loadCommitmentStore,
   resolveCommitmentStorePath,
 } from "../../dist/commitments/store.js";
-import { deleteTestEnvValue, setTestEnvValue } from "../../src/test-utils/env.js";
 
 const DEFAULT_COMMITMENT_EXTRACTION_QUEUE_MAX_ITEMS = 64;
 
@@ -25,18 +24,26 @@ function assert(condition: unknown, message: string): asserts condition {
   }
 }
 
+function setEnvValue(key: string, value: string): void {
+  Reflect.set(process.env, key, value);
+}
+
+function deleteEnvValue(key: string): void {
+  Reflect.deleteProperty(process.env, key);
+}
+
 async function withStateDir<T>(name: string, fn: (stateDir: string) => Promise<T>): Promise<T> {
   const root = await fs.mkdtemp(path.join(os.tmpdir(), `openclaw-${name}-`));
   const previousStateDir = process.env.OPENCLAW_STATE_DIR;
   try {
-    setTestEnvValue("OPENCLAW_STATE_DIR", root);
+    setEnvValue("OPENCLAW_STATE_DIR", root);
     return await fn(root);
   } finally {
     resetCommitmentExtractionRuntimeForTests();
     if (previousStateDir === undefined) {
-      deleteTestEnvValue("OPENCLAW_STATE_DIR");
+      deleteEnvValue("OPENCLAW_STATE_DIR");
     } else {
-      setTestEnvValue("OPENCLAW_STATE_DIR", previousStateDir);
+      setEnvValue("OPENCLAW_STATE_DIR", previousStateDir);
     }
     await fs.rm(root, { recursive: true, force: true });
   }
