perf: optimize stop hook performance

[khaong]

Summary

• Consolidate redundant git status calls into single ComputeFileChanges function
• Eliminate double transcript parsing in CalculateTotalTokenUsage

Performance Improvement

Optimization	Savings
Git status consolidation	~40-80ms
Transcript parsing	~0.3-0.6ms
Total	~40-80ms

Measured in repos with 400+ files. The main win comes from calling worktree.Status() once instead of twice.

Test plan

• All unit tests pass
• All integration tests pass
• Linting clean

🤖 Generated with Claude Code

---

Note

Medium Risk
Touches checkpoint tree construction and file-change detection around session start, which can affect what gets committed (especially deletions/renames/ignored files); behavior is covered by added tests but relies on invoking the git CLI and parsing porcelain output.

Overview
Reduces stop-hook overhead by consolidating new and deleted file detection into a single ComputeFileChanges call, updating Claude Code and Gemini hook handlers (and debug auto-commit) to use it while deprecating the old split helpers.

Optimizes temporary checkpoint creation for the first checkpoint by replacing a full working-directory walk with a git status --porcelain -z -uall based collector that respects repo + global gitignore, and by merging user pre-existing deletions into the checkpoint tree; adds extensive tests covering first-checkpoint capture of modified tracked files, untracked files, gitignored exclusions, deletions, renames, and filenames with spaces.

Avoids double transcript parsing in Claude Code token accounting by parsing once and reusing the parsed transcript for both token usage and spawned subagent detection (including handling empty transcript paths).

^{Written by Cursor Bugbot for commit 941f759. This will update automatically on new commits. Configure here.}

[Copilot]

Pull request overview

Optimizes stop-hook performance by consolidating git status queries and avoiding redundant transcript parsing when calculating Claude Code token usage.

Changes:

• Introduces ComputeFileChanges to compute new + deleted files from a single worktree.Status() call and updates hook/debug call sites to use it.
• Deprecates ComputeNewFiles / ComputeDeletedFiles in favor of the consolidated implementation.
• Updates CalculateTotalTokenUsage to parse the transcript once and reuse the parsed result for both token usage and spawned-agent detection.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
cmd/entire/cli/state.go	Adds ComputeFileChanges and marks older file-change helpers as deprecated
cmd/entire/cli/state_test.go	Adds a minimal test stub for ComputeFileChanges
cmd/entire/cli/hooks_geminicli_handlers.go	Switches Gemini hook to consolidated file-change computation
cmd/entire/cli/hooks_claudecode_handlers.go	Switches Claude Code hook to consolidated file-change computation
cmd/entire/cli/debug.go	Updates debug output to use consolidated file-change computation
cmd/entire/cli/agent/claudecode/transcript.go	Removes double transcript parsing in CalculateTotalTokenUsage

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.}

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.