fix: pass non-ASCII terminal env vars through

Author: asyrjasalo

README.md

@@ -51,7 +51,7 @@ Terminal and git env vars are forwarded into the sandbox:
 
 `TERM`, `COLORTERM`, `TERM_PROGRAM`, `TERM_PROGRAM_VERSION`, `NO_COLOR`, `FORCE_COLOR`, `LANG`, `LC_ALL`, `EMAIL`, `GIT_AUTHOR_NAME`, `GIT_AUTHOR_EMAIL`, `GIT_COMMITTER_NAME`, `GIT_COMMITTER_EMAIL`, `PI_RUN_CODE_UNSANDBOXED`
 
-Non-ASCII values are filtered out (microsandbox VMM limitation).
+Non-ASCII values are passed through as-is.
 
 ## API keys
 

src/index.ts

@@ -47,7 +47,7 @@ export function termEnv(
   }
   const result: Record<string, string> = {}
   for (const [k, v] of Object.entries(vars)) {
-    if (v && /^[\x20-\x7E]*$/.test(v)) result[k] = v
+    if (v) result[k] = v
   }
   return result
 }

test/index.test.ts

@@ -57,6 +57,23 @@ describe("termEnv", () => {
       expect(v).toBeDefined()
     }
   })
+
+  it("passes non-ASCII values through", () => {
+    const env = termEnv({
+      TERM: "xterm",
+      GIT_AUTHOR_NAME: "Jöhn Müller",
+      EMAIL: "user@exämple.com",
+      LANG: "et_EE.UTF-8",
+    })
+    expect(env.GIT_AUTHOR_NAME).toBe("Jöhn Müller")
+    expect(env.EMAIL).toBe("user@exämple.com")
+    expect(env.LANG).toBe("et_EE.UTF-8")
+  })
+
+  it("passes emoji values through", () => {
+    const env = termEnv({ TERM: "xterm", COLORTERM: "🎨 truecolor" })
+    expect(env.COLORTERM).toBe("🎨 truecolor")
+  })
 })
 
 describe("buildSecrets", () => {