From 0c0e2e6c8b30d7a5cbd041e0f2d14c4074eefe56 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Fri, 8 May 2026 22:54:57 +0100
Subject: [PATCH] test: simplify acp env key normalization

---
 src/acp/client.test.ts | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/acp/client.test.ts b/src/acp/client.test.ts
index 13e99cc1ff8..729134bb98e 100644
--- a/src/acp/client.test.ts
+++ b/src/acp/client.test.ts
@@ -10,7 +10,13 @@ vi.mock("../secrets/provider-env-vars.js", () => ({
     baseEnv: NodeJS.ProcessEnv,
     keys: Iterable<string>,
   ): NodeJS.ProcessEnv => {
-    const denied = new Set([...keys].map((key) => key.trim().toUpperCase()).filter(Boolean));
+    const denied = new Set<string>();
+    for (const key of keys) {
+      const normalized = key.trim().toUpperCase();
+      if (normalized) {
+        denied.add(normalized);
+      }
+    }
     const env = { ...baseEnv };
     for (const key of Object.keys(env)) {
       if (denied.has(key.toUpperCase())) {