enh: bind mount original repo when running in a git worktree

[yarikoptic]

When the workspace is a git worktree (detected via .git being a symlink or a file with gitdir: reference), add a bind mount for the original repository. This allows Claude to access the original repo's git objects and references, and perform git operations (commit, fetch, etc.).

🤖 Generated with Claude Code

note: bind mount 'rw' so we could commit etc

---

Updates (added by Claude Code)

Addressed review feedback

Per @asmacdo's feedback about security concerns with silent bind mounting, added --worktree option with four modes:

• ask (default): Prompts user with security warning before bind mounting
• bind: Automatically bind mounts original repo
• ok: Skip bind mounting and continue normally
• error: Exit if worktree detected

Contributors

• Initial implementation: Claude Code (Claude Opus 4.5) - worktree detection and bind mount logic
• --worktree option modes: GitHub Copilot (PR Add --worktree option to control original repo bind mounting #22 merged into this branch)
• Conflict resolution & rebase onto main: Claude Code + @asmacdo
• Review & direction: @asmacdo, @yarikoptic

Usage

# Prompt before mounting (default)
yolo

# Always bind mount
yolo --worktree=bind

# Never bind mount  
yolo --worktree=ok

# Disallow worktrees
yolo --worktree=error

Security note: Bind mounting the original repo exposes more files and allows modifications. The ask prompt helps prevent unintended access.

[Copilot]

Copilot wasn't able to review any files in this pull request.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[asmacdo]

I've encountered this also, so I'm very happy to see this! I wonder though if we should consider making it optional and off by default. My reasoning

• bindmounting anything other than cwd silently might expose something the user doesnt want exposed
• running in permissions bypass could result in unintended changes to the original repo rather than the worktree

wdyt?

[yarikoptic]

arguments do make sense! I guess we better make it also "explicit" as ask user unless specified... @copilot can improve this PR to add an option --worktree ask,bind,ok,error . So by default -- ask user to either bind or not ('ok') or error out.

[Copilot]

@yarikoptic I've opened a new pull request, #22, to work on those changes. Once the pull request is ready, I'll request review from you.

[yarikoptic]

@copilot please address conflicts

[Copilot]

@yarikoptic I've opened a new pull request, #24, to work on those changes. Once the pull request is ready, I'll request review from you.

[asmacdo]

@copilot address conflicts between PR #19 and the main branch

[Copilot]

Pull request overview

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[asmacdo]

@yarikoptic ready for review. (I couldnt request review, so I've also invited you to join project as admin)

[asmacdo]

Actually I think this should be "no", ok doesnt match the behavior IMO

[asmacdo]

went with skip instead