[skarim]

Optimizes GitOps and worktree operations by reducing repeated lookups, and updates the regression tests to preserve existing safety checks.

Functionality and impact

• Reuses directory paths from the current validation instead of querying Git again.
• Reuses the current preflight executor instead of resolving the same worktree twice.
• Preserves identity, ownership, and recovery checks without adding a persistent cache.

Key areas to review

• internal/git/worktree.go: validated directory metadata and worktree selection.
• internal/git/gitops.go: use of the validated directory lookup.
• internal/worktree/context.go: scoped executor reuse and cleanliness checks within each preflight.

[Copilot]

🟢 Approval recommended

The narrowly scoped refactor preserves existing safety checks, adds focused regression coverage, and has no identified blocking issues.

No open issues 🎉

What changed in this PR

This PR reduces repeated Git lookups in scoped worktree operations while preserving per-call validation and avoiding persistent caching.

Changes:

• Reuses directory paths obtained during the current identity validation.
• Reuses the preflight executor when preparing a branch.
• Adds regression tests for lookup counts, per-call freshness, and cleanliness checks.

File	Description
internal/​worktree/​context.go	Extracts shared preflight logic and reuses its executor in Prepare.
internal/​worktree/​context_test.go	Tests executor reuse and cleanliness checks within each call.
internal/​git/​worktree.go	Reuses validated directory metadata for lookups and worktree selection.
internal/​git/​gitops.go	Routes GitDir through the validated directory helper.
internal/​git/​gitops_test.go	Tests reduced directory probes and revalidation after a worktree moves.

_{Review effort: Balanced}

[francisfuzz]

🚢 , left one non-blocking comment