clay / claycli / 26615618704

87%

master: 87%

Pull #252

github

🍕 Simplify Vite client env to a pure window.process.env redirect

Drop the build-time inlining layer (and the .env reader) that was added for the
Browserify-parity experiment. The Docker image build is bare, so inlining never
contributed in prod; the runtime redirect (process.env -> window.process.env,
hydrated from preloadData._envVars and gated by client-env.json) is what
actually fixes the Kiln pickers under Vite.

This matches the legacy Browserify transformEnv plugin (lib/cmd/compile/scripts.js)
exactly — it rewrites process.env to window.process.env and inlines nothing — and
removes the latent risk of baking non-allowlisted secrets (e.g. CORAL_GRAPHQL_TOKEN)
into the public bundle on any env-present build. process.env.NODE_ENV stays a
build-time literal so guards still tree-shake.

Co-authored-by: Cursor <cursoragent@cursor.com>

758 of 941 branches covered (80.55%)

Branch coverage included in aggregate %.

42 of 42 new or added lines in 2 files covered. (100.0%)

1622 of 1792 relevant lines covered (90.51%)

10.28 hits per line