Skip to content

fix(eve): skip nested git repos in dev snapshots#589

Closed
iroiro147 wants to merge 1 commit into
vercel:mainfrom
iroiro147:fix/dev-snapshot-skip-nested-git-526
Closed

fix(eve): skip nested git repos in dev snapshots#589
iroiro147 wants to merge 1 commit into
vercel:mainfrom
iroiro147:fix/dev-snapshot-skip-nested-git-526

Conversation

@iroiro147

Copy link
Copy Markdown
Contributor

Description

Closes #526.

eve dev source snapshots now stop recursing when a copied child directory contains its own .git file or directory. That treats nested repositories, submodules, and git worktrees as source-copy boundaries without special-casing .claude/worktrees.

This keeps ordinary sibling content copied as before while avoiding duplicate checkout copies inside dev-runtime snapshots.

How did you test your changes?

  • pnpm --filter eve exec vitest run --config vitest.integration.config.ts src/internal/nitro/dev-runtime-artifacts.integration.test.ts -t 'nested git|generated output'\n- pnpm --filter eve run typecheck\n- pnpm --filter eve exec oxlint src/internal/nitro/dev-runtime-source-snapshot-copy.ts src/internal/nitro/dev-runtime-artifacts.integration.test.ts\n- pnpm exec oxfmt --check packages/eve/src/internal/nitro/dev-runtime-source-snapshot-copy.ts packages/eve/src/internal/nitro/dev-runtime-artifacts.integration.test.ts .changeset/dev-snapshot-nested-git-boundaries.md\n- pnpm guard:invariants\n- pnpm changeset status --since origin/main\n- git diff --check\n\nNote: pnpm --filter eve run test:integration -- src/internal/nitro/dev-runtime-artifacts.integration.test.ts -t 'nested git|generated output' expands to the full integration suite in this repo and hit an unrelated local listen EPERM in src/client/client.integration.test.ts; the tier-configured touched-file command above passed.\n\n### PR Checklist\n\n- [x] I linked an issue with prior discussion confirming this change is wanted\n- [x] I ran the relevant checks from CONTRIBUTING.md\n- [x] I added tests and documentation where relevant\n- [x] I added a changeset if this touches the published eve package\n- [x] DCO sign-off passes for every commit (git commit --signoff)\n

Signed-off-by: iroiro147 <sarthak.singh@juspay.in>
@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

@iroiro147 is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@iroiro147

Copy link
Copy Markdown
Contributor Author

Thanks, this is now covered by #645, which has merged and closes #526. Closing this duplicate PR to keep the queue tidy.

@iroiro147 iroiro147 closed this Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

eve dev dev-runtime snapshot recurses into nested git repositories/worktrees (e.g. .claude/worktrees/*) and copies a second checkout of the app

1 participant