Skip to content

Optimize hooks-mode cherry-pick finalization performance#578

Open
svarlamov wants to merge 15 commits intomainfrom
codex/cherry-pick-hooks-perf-fix
Open

Optimize hooks-mode cherry-pick finalization performance#578
svarlamov wants to merge 15 commits intomainfrom
codex/cherry-pick-hooks-perf-fix

Conversation

@svarlamov
Copy link
Member

@svarlamov svarlamov commented Feb 24, 2026

Summary

  • align cherry-pick hooks-mode rewrite finalization with rebase-style terminal-only finalization
  • tighten cherry-pick terminal-step detection so heavy rewrite runs once at sequence end
  • skip managed repo lookup for prepare-commit-msg to remove per-commit overhead in hooks mode
  • add regression assertion that multi-commit cherry-pick emits exactly one cherry_pick_complete event with full mappings

Validation

  • cargo fmt -- --check
  • cargo clippy
  • cargo test -- --test-threads=8
  • cargo test --test hook_modes -- --test-threads=8
  • python3 scripts/benchmarks/git/benchmark_modes_vs_main.py --iterations-basic 5 --iterations-complex 4 --margin-pct 25.0 --margin-baseline current_wrapper --enforce-margin --work-root /tmp/git-ai-bench-after-full-1771956806
  • python3 scripts/benchmarks/git/benchmark_nasty_modes_vs_main.py --work-root /tmp/git-ai-nasty-pr-smoke-1771960740 --feature-commits 24 --main-commits 10 --side-commits 8 --files 4 --lines-per-file 400 --burst-every 6 --repetitions 1 --margin-pct 25 --margin-baseline current_wrapper --enforce-margin

Open with Devin

@git-ai-cloud-dev
Copy link

git-ai-cloud-dev bot commented Feb 24, 2026

Stats powered by Git AI

🧠 you    ███████░░░░░░░░░░░░░  36%
🤖 ai     ░░░░░░░█████████████  64%
More stats
  • 0.0 lines generated for every 1 accepted
  • 0 seconds waiting for AI
  • Top model: codex::gpt-5.3-codex (493 accepted lines, 0 generated lines)

AI code tracked with git-ai

@git-ai-cloud
Copy link

git-ai-cloud bot commented Feb 24, 2026

Stats powered by Git AI

🧠 you    ███████░░░░░░░░░░░░░  36%
🤖 ai     ░░░░░░░█████████████  64%
More stats
  • 0.0 lines generated for every 1 accepted
  • 0 seconds waiting for AI
  • Top model: codex::gpt-5.3-codex (493 accepted lines, 0 generated lines)

AI code tracked with git-ai

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant