chore: ignore nested git worktree checkouts - #566
Conversation
.worktrees/ (git worktree add target) was untracked in the root .gitignore, so the primary checkout's git status collapsed every worktree's full content into one opaque untracked-directory line — a real risk for an accidental git add -A picking up nested checkouts.
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
ⓘ Qodo reviews are paused because your trial has ended. Ask your workspace admin to add credits to resume reviews. Manage billing |
🤖 CodeAnt AI — Review Status
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Thanks for using CodeAnt! 🎉We're free for open-source projects. if you're enjoying it, help us grow by sharing. Share on X · |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThe PR adds the root File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
🏁 CodeAnt Quality Gate ResultsCommit: ✅ Overall Status: PASSEDQuality Gate Details
|
There was a problem hiding this comment.
This PR correctly adds .worktrees/ to .gitignore to prevent accidental commits of nested git worktree checkouts. The change is straightforward and addresses the stated risk effectively. No issues found.
You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (1)
Included review availability: 0 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour. 📝 WalkthroughWalkthroughThe ChangesWorktree configuration
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This localized .gitignore change only excludes nested .worktrees/ directories from accidental tracking. No actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| Docker | Sep 1, 2026 11:13a.m. | Review ↗ | |
| JavaScript | Sep 1, 2026 11:13a.m. | Review ↗ | |
| Python | Sep 1, 2026 11:13a.m. | Review ↗ | |
| Rust | Sep 1, 2026 11:13a.m. | Review ↗ | |
| Shell | Sep 1, 2026 11:13a.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Summary
.worktrees/(thegit worktree addtarget directory used throughout local development) was not in the root.gitignore, so the primary checkout'sgit statuscollapsed every nested worktree's full content into one opaque untracked-directory line.git add -Apicking up nested worktree checkouts (each with their own.gitfile,node_modules, etc.).Test plan
git statusin the primary checkout no longer lists.worktrees/as untracked content..gitignore).Summary by Sourcery
Enhancements:
.worktrees/directories so nested Git worktree checkouts do not appear as untracked content or risk accidental inclusion.Summary by CodeRabbit