Add a complementary soft layer to the hard gates: advisory_flags surface
governance hints (e.g. oversized-but-not-blocked PRs -> suggest splitting) in
the scorecard WITHOUT affecting score or verdict. Hard gates stay the real
teeth; advisory leaves the call to humans.
- additive & backward-compatible: default off; policies lacking the section
behave exactly as before (verified: default policy on real PR #275 yields
zero advisory section, PASS 87/100 unchanged)
- deterministic: consumes only already-collected PR data (no wall-clock input),
preserving same-input -> same-score -> same-verdict
- first flag large_pr_files (changed-files >= threshold) answers the
166-open-PR backlog reality where big PRs should be flagged, not blocked
- evaluate_advisory_flags() is separate from score_dimensions/decide_verdict;
the 5-dim scoring and verdict logic are untouched
- 7 new unit tests (15 total) incl. an invariant that enabling advisory does
NOT change total/verdict; verified live on real open PR #275
- REFERENCE.md 1.9 + gatekeeper.yaml + verification.md E + demo scorecard
Extends the merged gatekeeper line (#90 skill -> #219 workflow -> #220 issueops).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
wbtiger
NeeNe