Skip to content

fix(eve): prevent Vercel env pull setup hangs - #1495

Draft
OwenKephart wants to merge 1 commit into
mainfrom
fix/env-pull-hang
Draft

fix(eve): prevent Vercel env pull setup hangs#1495
OwenKephart wants to merge 1 commit into
mainfrom
fix/env-pull-hang

Conversation

@OwenKephart

@OwenKephart OwenKephart commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • run vercel env pull with stdin detached, including interactive setup flows
  • abort a Vercel CLI child that remains alive after the environment files are written
  • cover the bounded env-pull invocation in unit tests

Root cause

Vercel CLI runs an optional Claude Code Vercel-plugin installer after a successful env command. Its confirmation helper checked only whether stdin was a TTY, not Vercel CLI's --non-interactive state. Therefore vercel env pull --yes --non-interactive could successfully write .env.local and then wait at the plugin-install confirmation when stdin remained inherited from eve.

The Vercel CLI fix is in vercel/vercel#17345. This eve change is still required to detach stdin for automation and bound a misbehaving child. With stdin ignored, the problematic CLI prompt declines immediately; the 30-second timeout remains a safety net for any future stuck subprocess.

Validation

  • pnpm --filter eve build:compiled
  • pnpm --filter eve exec vitest run --config vitest.unit.config.ts src/setup/run-vercel-link.test.ts src/setup/boxes/deploy-project.test.ts src/setup/primitives/run-vercel.test.ts
  • pnpm typecheck --filter eve
  • pnpm lint
  • pnpm fmt

Note: the sandbox uses Node 22.22.2 while the repository requires Node >=24; commands completed with engine warnings.

Signed-off-by: owenkephart <owen.kephart@vercel.com>
@OwenKephart OwenKephart added bug Something isn't working cli setup labels Jul 31, 2026
@vercel

vercel Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
eve-docs Ready Ready Preview Jul 31, 2026 9:54pm
eve-docs-4759 Ready Ready Preview, v0 Jul 31, 2026 9:54pm

@OwenKephart

Copy link
Copy Markdown
Contributor Author

Confirmed root cause in Vercel CLI: after a successful env command, the optional Claude Code plugin installer prompts based only on stdin.isTTY, ignoring --non-interactive. The upstream fix is vercel/vercel#17345. This PR detaches stdin so the existing CLI version also declines that prompt immediately, plus retains a deadline as a fallback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cli setup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant