MR Sentinel

Audit log · ← dashboard

sgharlow/governance-demo-app!7

Verdict
block
Score
0.0 / 10
Rubric
v2
Commit
37039be0
Scored
2026-05-31 23:43:17 UTC

Rule outcomes (15)

RuleCategoryOutcomeSeverityControlsEvidence
auth-on-new-public-endpointssecurityfailblockerSOC2-CC6.1, OWASP-ASVS-V1The new public endpoint `/leak/dump` in `app/v030/leak.py` lacks any explicit authentication annotation or middleware.
changed-method-coveragequalityfailerrorSOC2-CC8.1, ISO-27001-A.14.2.8The new file `app/v030/leak.py` introduces `leak_dump` and `_all_data` methods, but no corresponding tests are included in the diff.
contract-has-spec-linkcontract_specfailwarningCDPD-§3, ISO-27001-A.14.2.1MR description: "Edited description to trigger an update webhook with same sha." No link to a spec, RFC, or issue describing the intended behavior.
integration-boundaries-explicitcontract_specfailwarningCDPD-§6, ISO-27001-A.14.2.5The new `/leak/dump` endpoint in `app/v030/leak.py` is an integration boundary (HTTP API) but no OpenAPI, proto, or schema documentation is included in the diff.
kill-switch-pathcontract_specfailwarningCDPD-§9, SOC2-CC7.5The new `/leak/dump` endpoint introduces new user-facing behavior (exposing data) without a feature flag, environment toggle, or documented rollback path.
no-secrets-in-diffsecurityfailblockerSOC2-CC6.1, ISO-27001-A.9.4.3, OWASP-ASVS-V2The line `api_key = "sk-secret-abc123def456"` in `app/v030/leak.py` contains a hardcoded secret pattern.
observability-on-new-endpointsoperationalfailwarningSOC2-CC7.2, ISO-27001-A.12.4.1The new `/leak/dump` endpoint in `app/v030/leak.py` does not include structured logging or metrics.
dependency-advisory-checksecuritypassblockerSOC2-CC7.1, ISO-27001-A.12.6.1, NIST-SA-11The diff does not add or upgrade any dependencies.
no-commented-out-codequalitypassinfoSOC2-CC8.1No blocks of commented-out code (3+ consecutive lines) were found in the diff.
no-skipped-tests-introducedqualitypasserrorSOC2-CC8.1No test files or skipped test markers (`pytest.skip`, `xfail`, etc.) were introduced in the diff.
acceptance-criteria-testablecontract_specskipwarningCDPD-§5, SOC2-CC8.1No linked spec or acceptance criteria provided in the MR description or diff. No tests are added in the diff.
error-budget-impact-declaredoperationalskipwarningSOC2-CC4.1No SLO file (`slo.yaml`) is present in the repository context, and the MR description does not mention error budget impact.
mutation-resilience-critical-pathsqualityskipwarningSOC2-CC8.1Requires repository metadata for `critical_path: true` files and mutation test results, which are not provided in the MR context.
rollback-documented-for-migrationsoperationalskiperrorSOC2-CC7.5, ISO-27001-A.14.2.2The diff does not contain any database migration files.
spec-implementation-matchcontract_specskiperrorCDPD-§7No spec linked in the MR description to compare against the implementation diff.

Audit log (5)

WhenActorActionDetails
2026-05-31 23:43:20 UTCmr-sentinelskip_duplicate{"sha": "37039be0", "reason": "already_evaluated", "rubric_version": "v2"}
2026-05-31 23:43:20 UTCmr-sentinelevaluate{"score": 0.0, "note_id": 3357062097, "verdict": "block", "rubric_source": "project_override", "comment_created": false, "mr_
2026-05-18 05:01:15 UTCmr-sentinelskip_duplicate{"sha": "5955b6c7", "reason": "already_evaluated"}
2026-05-18 04:59:57 UTCmr-sentinelskip_duplicate{"sha": "5955b6c7", "reason": "already_evaluated"}
2026-05-18 04:59:56 UTCmr-sentinelevaluate{"score": 1.0, "note_id": 3357062097, "verdict": "block", "tool_calls": 7, "comment_created": true, "pipeline_status": null, "