MR Sentinel

Audit log · ← dashboard

sgharlow/governance-demo-app!9

Verdict
block
Score
1.0 / 10
Rubric
v2
Commit
ab0191ce
Scored
2026-06-01 00:43:51 UTC

Rule outcomes (15)

RuleCategoryOutcomeSeverityControlsEvidence
auth-on-new-public-endpointssecurityfailblockerSOC2-CC6.1, OWASP-ASVS-V1The new `/admin/dump-patients` endpoint lacks an explicit authentication or authorization decorator, making it publicly accessible.
changed-method-coveragequalityfailerrorSOC2-CC8.1, ISO-27001-A.14.2.8The new `dump_all_patients` method in `app/routes/admin.py` has no corresponding tests in the diff.
contract-has-spec-linkcontract_specfailwarningCDPD-§3, ISO-27001-A.14.2.1MR description states "No spec issue — small addition."
error-budget-impact-declaredoperationalfailwarningSOC2-CC4.1The MR description does not declare the expected error-budget impact for the new `/admin/dump-patients` endpoint.
integration-boundaries-explicitcontract_specfailwarningCDPD-§6, ISO-27001-A.14.2.5The new `/admin/dump-patients` REST endpoint does not include an OpenAPI fragment or schema definition in the diff.
kill-switch-pathcontract_specfailwarningCDPD-§9, SOC2-CC7.5The new `/admin/dump-patients` endpoint is enabled unconditionally without a feature flag or documented rollback procedure.
observability-on-new-endpointsoperationalfailwarningSOC2-CC7.2, ISO-27001-A.12.4.1The new `/admin/dump-patients` endpoint does not include structured logging for request details (ID, latency, status) or metrics (counter/histogram).
dependency-advisory-checksecuritypassblockerSOC2-CC7.1, ISO-27001-A.12.6.1, NIST-SA-11No dependency changes (additions or upgrades) detected in the diff.
no-commented-out-codequalitypassinfoSOC2-CC8.1No blocks of commented-out code (3+ consecutive lines) detected in the diff.
no-secrets-in-diffsecuritypassblockerSOC2-CC6.1, ISO-27001-A.9.4.3, OWASP-ASVS-V2No secret patterns (API keys, tokens, private keys, connection strings with passwords) detected in the diff.
no-skipped-tests-introducedqualitypasserrorSOC2-CC8.1No new test skips (e.g., `pytest.skip`, `xfail`, `@Ignore`) detected in the diff.
rollback-documented-for-migrationsoperationalpasserrorSOC2-CC7.5, ISO-27001-A.14.2.2No database migration files detected in the diff.
acceptance-criteria-testablecontract_specskipwarningCDPD-§5, SOC2-CC8.1No spec linked in MR description, so no acceptance criteria to evaluate against tests.
mutation-resilience-critical-pathsqualityskipwarningSOC2-CC8.1No information provided regarding `critical_path` labels or mutation test results for the affected files.
spec-implementation-matchcontract_specskiperrorCDPD-§7No spec linked in MR description, so cannot compare implementation against a spec.

Audit log (4)

WhenActorActionDetails
2026-06-01 00:43:54 UTCmr-sentinelskip_duplicate{"sha": "ab0191ce", "reason": "already_evaluated", "rubric_version": "v2"}
2026-06-01 00:43:53 UTCmr-sentinelevaluate{"score": 1.0, "note_id": 3361770499, "verdict": "block", "rubric_source": "project_override", "comment_created": false, "mr_
2026-05-19 05:11:43 UTCmr-sentinelskip_duplicate{"sha": "aac1c48a", "reason": "already_evaluated", "rubric_version": "v2"}
2026-05-19 05:11:42 UTCmr-sentinelevaluate{"score": 2.0, "note_id": 3361770499, "verdict": "block", "tool_calls": 8, "rubric_source": "project_override", "comment_crea