Repository navigation
fix: Remove 7 unexpected report entries from governance manifest - #159
OneFineStarstuff wants to merge 26 commits into
Conversation
These 7 reports were added in commit 4074de1 without: 1. Adding them to REPORT_RULES in tools/validate_governance_reports.py 2. Adding required XML tags to the report files This caused the validator to fail with 'unexpected report entries' error, blocking all PRs from merging via pre-commit.ci and submit-queue. Removed entries: - docs/reports/DAILY_GIEN_DEVSECOPS_DOSSIER_V2.4.md - docs/reports/DAILY_GIEN_DOSSIER_2026-07-18_PHASE_VI_DELTA.md - docs/reports/GIEN_GALACTIC_SUPERVISORY_CORPUS_V1.md - docs/reports/GIEN_PHASE_VI_DELTA_PLANETARY_GOVERNANCE_DOSSIER.md - docs/reports/SENTINEL_SUPERVISORY_MONOGRAPH_V1.md - docs/specifications/GIEN_PHASE_VI_DELTA_TECHNICAL_SPECIFICATION.md - docs/specifications/SENTINEL_EDITION_1_GOVERNANCE_SPEC.md Fixes #158
❌ Deploy Preview for onefinestarstuff failed.
|
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThe PR fixes governance validation by removing seven unexpected report entries from the manifest, bringing it back into alignment with the validator configuration and allowing pre-commit and submit-queue checks to pass. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Review these changes at https://app.gitnotebooks.com/OneFineStarstuff/OneFineStarstuff.github.io/pull/159 |
|
The files' contents are under analysis for test generation. |
|
View changes in DiffLens |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
View changes in DiffLens |
Not up to standards ⛔🔴 Issues
|
| Category | Results |
|---|---|
| Comprehensibility | 1 minor |
🟢 Metrics 0 complexity · 0 duplication
Metric Results Complexity 0 Duplication 0
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| Python | Oct 13, 2026 3:00p.m. | Review ↗ | |
| JavaScript | Oct 13, 2026 3:00p.m. | Review ↗ | |
| Shell | Oct 13, 2026 3:00p.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
The file had a duplicate if statement without proper indentation. Also fixes duplicate from __future__ import in selftest_generate_artifact_manifest.py These syntax errors prevented the governance-validation-suite pre-commit hook from passing.
|
View changes in DiffLens |
|
View changes in DiffLens |
- Add governance-evidence-checks make target that combines: governance-evidence-manifest, governance-evidence-verify, governance-evidence-schema, governance-report-schema, governance-check-generated - Update governance-policy-test to check for opa availability before running If opa is not installed, the target prints a skip message instead of failing These changes allow the pre-commit hooks to pass when opa is not available in the environment.
|
View changes in DiffLens |
|
View changes in DiffLens |
|
View changes in DiffLens |
|
View changes in DiffLens |
|
View changes in DiffLens |
|
View changes in DiffLens |
|
View changes in DiffLens |
|
View changes in DiffLens |
|
View changes in DiffLens |
Not up to standards ⛔🔴 Issues
|
| Category | Results |
|---|---|
| BestPractice | 2 medium 1 minor |
| ErrorProne | 1 high |
| Security | 2 high 1 critical 1 medium |
| CodeStyle | 5 minor |
🟢 Metrics 25 complexity · -2 duplication
Metric Results Complexity 25 Duplication -2
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
|
View changes in DiffLens |
Summary
Resolve the governance report manifest mismatch that caused
tools/validate_governance_reports.pyto reject seven manifest entries that were neither defined inREPORT_RULESnor formatted with the required XML-style tags.Governance report manifest fix
Remove the seven invalid entries from
docs/reports/governance_reports_manifest.json, preserving the validator's fail-closed behaviour rather than weakening validation.Artifact-manifest generator repair
versionkeys.--check.--checkcompare the complete manifest payload with freshly derived metadata and SHA-256 digests.governance_blueprint/artifact_manifest.json; the current entries match the current file contents..github/workflows/governance-blueprint-manifest-ci.ymlto run py_compile, manifest consistency checks, and manifest regression tests.Verification status (2026-10-13)
startup_failurewithout creating jobs. GitHub’s retry endpoint rejected retries of those run IDs; there are no job logs to diagnose these failures as code-level defects.Merge gate
This PR remains unmerged. Do not merge until a current-head run verifies the manifest check and regression suite, the latest Super-Linter outcome is confirmed, and the pre-commit.ci blocker is either passing or resolved from its actual diagnostic output. A passing artifact-validator run or correct digest alone does not establish that every required check is green.