Skip to content

Improve commit-pathspec benchmark gating and fast-path coverage#580

Open
svarlamov wants to merge 3 commits intomainfrom
codex/pathspec-benchmark-fastpath-fixes
Open

Improve commit-pathspec benchmark gating and fast-path coverage#580
svarlamov wants to merge 3 commits intomainfrom
codex/pathspec-benchmark-fastpath-fixes

Conversation

@svarlamov
Copy link
Member

@svarlamov svarlamov commented Feb 24, 2026

Summary

  • add AI pathspec hotspot benchmark gate to perf workflows (smoke + nightly)
  • benchmark plain git vs git-ai using ratio threshold (<=2x), with multi-run median stability
  • harden benchmark plain-git baseline resolution to avoid wrapper contamination
  • optimize post-commit pathspec handling for attribution-empty AI entries and skip unnecessary stats work
  • add regression tests for empty-entry filtering, untracked pathspec handling, and post-commit fast-path prompt preservation
  • normalize snapshot tests across git_ai_version variants

Validation

  • cargo fmt -- --check
  • cargo clippy
  • cargo test test_post_commit_fast_path_preserves_prompt_metrics_for_empty_ai_entries -- --nocapture
  • cargo test --test commit_post_stats_benchmark benchmark_commit_post_command_ai_pathspec_hotspot -- --ignored --nocapture

Open with Devin

@git-ai-cloud
Copy link

git-ai-cloud bot commented Feb 24, 2026

Stats powered by Git AI

🧠 you    ████████████████████  100%
🤖 ai     ░░░░░░░░░░░░░░░░░░░░  0%
More stats
  • 0.0 lines generated for every 1 accepted
  • 0 seconds waiting for AI

AI code tracked with git-ai

@git-ai-cloud-dev
Copy link

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

Stats powered by Git AI

🧠 you    ████████████████████  100%
🤖 ai     ░░░░░░░░░░░░░░░░░░░░  0%
More stats
  • 0.0 lines generated for every 1 accepted
  • 0 seconds waiting for AI

AI code tracked with git-ai

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 7 additional findings.

Open in Devin Review

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