feat(validate): add explicit Tier 2 and Tier 3 gating controls - #23
Conversation
Signed-off-by: Narendran Raghavan <nraghavan@nvidia.com>
…e-gating Signed-off-by: Narendran Raghavan <nraghavan@nvidia.com>
rng1995
left a comment
There was a problem hiding this comment.
Reviewed the explicit Tier 2/Tier 3 gate selection, source preflight, policy digest, and JSON/HTML/Markdown reporting consistency. I found no blocking code issues. The focused command, source, reporter, policy, and evaluation tests passed locally (86 tests), and the GitHub check matrix is green.
rng1995
left a comment
There was a problem hiding this comment.
Review found one blocking reporter/exit consistency issue; details inline. The stacked branch is otherwise current with its declared base and has no existing unresolved threads.
Signed-off-by: Narendran Raghavan <nraghavan@nvidia.com>
rng1995
left a comment
There was a problem hiding this comment.
Ready for an independent maintainer approval. All actionable review feedback is resolved. Final head 4f67972 is conflict-free against the exact declared base naren/tier3-provider-preflight at d407bd2: GitHub reports MERGEABLE/CLEAN, the branch is 0 behind / 3 ahead, and local git merge-tree produced 1589d694503e9e78384b38389c8394b61203f74f without conflicts. Every final-SHA CI/security/package check is green. Local verification: 3,339 passed, 8 skipped, 3 deselected with 79% coverage; focused gating/source/reporting matrix 87 passed; ruff, OSS boundary, diff check, and wheel/sdist build passed. GitHub rejected my APPROVE submission because this authenticated account is the PR author and cannot approve its own pull request.
Landing status
This PR merged into the staging branch
naren/tier3-provider-preflight, not intomain. Its base PR #19 is closed and was intentionally not merged, so the flags and reporting contract below are not available from the current GitHub release line.Follow-up #49 replays the reviewed gating commits onto current
main, adds the current Fern and repository documentation, and supersedes this PR for the release line. Review and merge #49 rather than attempting to retarget this already-merged stacked PR.Summary
Adds explicit Tier 2 and Tier 3 gating controls and makes CLI, JSON, Markdown, and HTML consume the same finalized gating policy.
Source and dependency
d2ae260from the0.8.3line (ed7ae8e8396ec0788135741192e6d75e4ba23724).Public-compatible behavior
--no-block-on-dedupmakes it advisory.--block-on-agent-evalmakes it blocking.Adaptations and exclusions
Validation
git diff --check, packaging, and OSS-boundary checks: passed.Risk controls