Skip to content

feat(e2e): diff-aware AI review with e2e-last-green tracking#2893

Merged
louisgv merged 1 commit into
OpenRouterLabs:mainfrom
AhmedTMM:feat/diff-aware-review
Mar 23, 2026
Merged

feat(e2e): diff-aware AI review with e2e-last-green tracking#2893
louisgv merged 1 commit into
OpenRouterLabs:mainfrom
AhmedTMM:feat/diff-aware-review

Conversation

@AhmedTMM
Copy link
Copy Markdown
Collaborator

Summary

  • AI log review now includes git diff since last green E2E run for causal analysis
  • New e2e-last-green tag tracks the last fully passing run as a baseline
  • AI can now correlate errors with specific commits (e.g. "this 404 likely caused by abc123 which deleted file Y")
  • Extracted from feat(e2e): diff-aware AI review + predictive reference checking #2876 without the broken reference checker

Changes

  • sh/e2e/lib/ai-review.sh — added _get_diff_since_last_green, mark_e2e_green, diff context in prompt
  • sh/e2e/e2e.sh — calls mark_e2e_green after all tests pass

Test plan

  • Run E2E, verify AI review includes diff context when e2e-last-green tag exists
  • Verify tag advances after a fully passing run
  • Verify graceful skip when tag doesn't exist yet (first run)

🤖 Generated with Claude Code

AI log review now includes the git diff since the last fully passing
E2E run, enabling causal analysis like "this 404 likely caused by
commit abc123 which deleted file Y". After a fully green run, the
e2e-last-green tag advances to HEAD as the new baseline.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@louisgv louisgv left a comment

Choose a reason for hiding this comment

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

Security review passed (-- security/pr-reviewer)

Code quality: Clean, well-documented
Command injection: None detected
Shell safety: macOS bash 3.x compatible
Variable handling: Proper escaping via env vars + TypeScript
Credential exposure: None
Force operations: Acceptable (tag advancement only)

All security checks passed. Merging.

@louisgv louisgv merged commit 6aeb9ba into OpenRouterLabs:main Mar 23, 2026
6 checks passed
@AhmedTMM AhmedTMM deleted the feat/diff-aware-review branch April 7, 2026 00:40
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.

2 participants