Skip to content

chore(warden): Enable incremental review - #8199

Merged
sentrivana merged 1 commit into
masterfrom
chore/warden-incremental-review
Nov 6, 2026
Merged

sentrivana merged 1 commit into
masterfrom
chore/warden-incremental-review

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Turns on Warden incremental review for find-bugs, code-review, skill-scanner. On each push, these skills now skip hunks that an earlier clean run on the same PR already reviewed, instead of re-reviewing the whole PR. Other skills in this repo keep full review.

Warden falls back to a full review when there is no usable baseline: first push, force push, config or skill change, or an earlier run that requested changes or had failures. This needs the org base config to set incrementalApp = "sentry-warden" (getsentry/.github#223); until that lands, Warden warns and keeps doing full reviews.

via Jeffrey Hung.

--

View Session [Sentry]

Skip hunks an earlier clean Warden run on the PR already reviewed for: find-bugs, code-review, skill-scanner.

Co-Authored-By: Jeffrey Hung <jeffrey.hung@sentry.io>
@sentry-junior
sentry-junior Bot requested a review from a team as a code owner November 5, 2026 22:08
@sentry-junior
sentry-junior Bot requested a review from Jeffreyhung November 5, 2026 22:10
@github-actions

github-actions Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Coverage Results 📊

✅ 137214 passed | ⏭️ 7456 skipped | Total: 144670 | Pass Rate: 94.85% | Execution Time: 446m 5s

📊 Comparison with Base Branch

Metric Change
Total Tests —
Passed Tests —
Failed Tests —
Skipped Tests —

✨ Test counts unchanged from base.

All tests are passing successfully.

✅ Patch coverage is 100.00% (no changed executable lines found; target 80%).
Project statement coverage is 90.30% (down 0.03 percentage points from base (1baa462) to head (7cd868f)).

Coverage diff
@@            Coverage Diff             @@
##        master     #8199       +/-##
==========================================
- Coverage    90.33%    90.30%    -0.03%
==========================================
  Files          203       203         —
  Tracked lines     26922     26922         —
  Branches     10056     10056         —
==========================================
- Hits         24319     24311        -8
- Misses        2603      2611        +8
- Partials      1516      1517        +1

Generated by Coverage Action

@sentrivana
sentrivana merged commit 5a099b8 into master Nov 6, 2026
172 checks passed
@sentrivana
sentrivana deleted the chore/warden-incremental-review branch November 6, 2026 08:25
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