feat(review): filter unreviewable content from diff context - #1008

Open

guyoron1 wants to merge 21 commits into

fullsend-ai:mainfrom

guyoron1:feat/review-diff-filtering

Commits on Sep 10, 2026

feat(review): filter unreviewable content from diff context
guyoron1
committed
9a7d116
fix(review): review round 1 — header-path fallback, bounded buffering, threshold-exempt disclosure
guyoron1
committed
537480c
fix(review): review round 2 — header split, quoted renames, one-line summary, post-script exemption
guyoron1
committed
cdf3674
fix(review): review round 3 — src-not-bundle, marker gating, GitLab shape, tab paths
guyoron1
committed
9dc7de8
fix(review): review round 4 — info-only disclosure exemption, paired GitLab boundaries
guyoron1
committed
2775b14
docs(review): excluded-content disclosure names all exclusion categories
guyoron1
committed
951b7a3
fix(review): settle generated markers from the file head, not the hunk
guyoron1
committed
e388990
test(review): cover mid-file regen, forged +++ boundary, header cap
guyoron1
committed
a05fce5
docs(review): one disclosure finding per excluded file
guyoron1
committed
6a92945
docs(review): make the step 2c filter command runnable as written
guyoron1
committed
8f796ab
chore(lint): raise pr-review context budget to 16815
guyoron1
committed
9da767e
fix(review): classify the forge skills' per-file diff shape; drop their coarse jq pre-filter
guyoron1
committed
7a08cd6
docs(review): step 2c never clobbers the diff, and an all-excluded PR is not a failure
guyoron1
committed
a30cb3b
fix(review): dequote quoted ---/+++ paths so GitLab-shaped sections classify
guyoron1
committed
47cb000
fix(review): minified and sourcemap rules match case-insensitively like the lockfile rule
guyoron1
committed
0e54e8e
docs(review): provenance-warning uses the <pr> file sentinel the agent definition names
guyoron1
committed
c13e3b4
chore(lint): raise pr-review context budget to 17579
guyoron1
committed
81ecb38
docs(review): header comment discloses the header-phase buffer cap
guyoron1
committed
dd67633

Navigation Menu

Uh oh!

feat(review): filter unreviewable content from diff context - #1008

feat(review): filter unreviewable content from diff context#1008
guyoron1 wants to merge 21 commits into
fullsend-ai:mainfrom
guyoron1:feat/review-diff-filtering

Commits on Sep 10, 2026

feat(review): filter unreviewable content from diff context

fix(review): review round 1 — header-path fallback, bounded buffering, threshold-exempt disclosure

fix(review): review round 2 — header split, quoted renames, one-line summary, post-script exemption

fix(review): review round 3 — src-not-bundle, marker gating, GitLab shape, tab paths

fix(review): review round 4 — info-only disclosure exemption, paired GitLab boundaries

docs(review): excluded-content disclosure names all exclusion categories

fix(review): settle generated markers from the file head, not the hunk

test(review): cover mid-file regen, forged +++ boundary, header cap

docs(review): one disclosure finding per excluded file

docs(review): make the step 2c filter command runnable as written

chore(lint): raise pr-review context budget to 16815

fix(review): classify the forge skills' per-file diff shape; drop their coarse jq pre-filter

docs(review): step 2c never clobbers the diff, and an all-excluded PR is not a failure

fix(review): dequote quoted ---/+++ paths so GitLab-shaped sections classify

fix(review): minified and sourcemap rules match case-insensitively like the lockfile rule

docs(review): provenance-warning uses the <pr> file sentinel the agent definition names

chore(lint): raise pr-review context budget to 17579

docs(review): header comment discloses the header-phase buffer cap

Commits on Sep 11, 2026

fix(review): don't approve a disclosure-only PR to ready-for-merge

fix(review): keep GitLab header-only section's lines through the filter

fix(review): runtime-neutral filter path; shrink skill under budget