Add entire trace command to find slow performance hooks etc.

[pfleidi]

entire trace command

Adds a new entire trace CLI command that displays hook performance traces, and extends the perf/span package with loop iteration tracking.

New command: entire trace

Parses structured perf log entries from .entire/logs/entire.log and renders them as formatted timing tables with hierarchical sub-step breakdowns.

• entire trace — show the most recent hook trace
• entire trace --last 5 — show the last 5 traces
• entire trace --hook post-commit — filter by hook type

Traces require ENTIRE_LOG_LEVEL=DEBUG to be enabled.

Implementation (cmd/entire/cli/trace.go, trace_cmd.go):

• JSONL log parser that extracts steps.*_ms and steps.*_err keys into a hierarchical step tree
• Sub-step detection: keys like foo.0, foo.1 are nested under their parent foo when the parent exists
• ASCII tree rendering (├─/└─) with manual padding to avoid UTF-8 multi-byte alignment issues
• Input validation on --last flag (must be >= 1)

Example output:

❯ entire trace --hook post-commit --last 2
post-commit  123ms  2026-03-06T16:35:48-08:00

  STEP                        DURATION
  cleanup_shadow_branches         12ms
  find_sessions_for_worktree       0ms
  open_repository_and_head         0ms
  process_sessions               100ms
    ├─ process_sessions.0          3ms
    ├─ process_sessions.1          0ms
    ├─ process_sessions.2         95ms
    └─ process_sessions.3          0ms
  resolve_commit_trees             5ms

perf/span: LoopSpan and child deduplication

Extends the span package to support tracking loop iterations as grouped sub-spans.

• LoopSpan — wraps a grouping span; Iteration() creates child spans, End() auto-ends unfinished iterations
• childStepKey() — deduplicates child span names (.1, .2 suffixes) so repeated loop iterations don't overwrite each other in log output
• Grandchild emission — group spans emit their children as steps.<name>.<index>_ms entries with 0-based indexing

Hook integration

PostCommit in manual_commit_hooks.go now wraps the session processing loop with StartLoop/Iteration/End to emit per-session timing in traces.

[cursor]

PR Summary

Medium Risk
Changes perf trace serialization and adds new loop-based spans; incorrect keying or span lifecycles could make perf logs misleading or noisy, and the post-commit hook now does extra tracing work (though only emitted at DEBUG).

Overview
Adds a new entire perf subcommand that reads .entire/logs/entire.log, extracts msg="perf" JSONL entries (including steps.*_ms/*_err), and renders them as a readable timing table with optional --last and --hook filtering plus hierarchical sub-step display.

Extends perf/span logging to preserve duplicate child step names (suffixing .1, .2, …) and to emit grouped loop iteration timings as steps.<name>.<index>_*, then instruments the post-commit session-processing loop to record per-session iteration spans. Includes comprehensive unit tests for parsing/rendering, key de-duplication, and loop span behavior (plus minor test formatting cleanup).

^{Written by Cursor Bugbot for commit c1c901d. Configure here.}

[Copilot]

Pull request overview

This PR adds a new entire perf CLI command for viewing hook performance traces, extends the perf/span package with loop iteration tracking (LoopSpan), and integrates the new loop span into the PostCommit hook to emit per-session timing data.

Changes:

• New entire perf command (perf.go, perf_cmd.go) that parses JSONL log entries and renders hierarchical timing tables with sub-step breakdowns
• New LoopSpan type in perf/span.go with StartLoop/Iteration/End pattern, plus childStepKey deduplication for repeated child span names
• Integration of LoopSpan in PostCommit hook to track per-session processing times

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
cmd/entire/cli/perf.go	New file: JSONL log parser, sub-step detection, and ASCII table renderer
cmd/entire/cli/perf_cmd.go	New file: Cobra command definition for entire perf
cmd/entire/cli/perf_test.go	New file: Tests for parsing, collecting, rendering, and flag validation
perf/span.go	Adds childStepKey dedup, grandchild emission, LoopSpan type
perf/span_test.go	Tests for deduplication, loop span creation, auto-end, and errors
cmd/entire/cli/strategy/manual_commit_hooks.go	Wraps session loop with StartLoop/Iteration/End
cmd/entire/cli/root.go	Registers newPerfCmd()
cmd/entire/cli/strategy/common_test.go	Whitespace formatting cleanup

[pfleidi]

Bugbot run

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

[Copilot]

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated no new comments.

[khaong]

From Claude:

Review

Consolidated review covering 1 must-fix and 3 should-fix items. See inline comments.

Overall the PR is clean — follows project conventions, good test coverage, nice separation of concerns. These are the items worth addressing before merge.

[Copilot]

Pull request overview

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

---

You can also share your feedback on Copilot code review. Take the survey.