Skip to content

implemented the fix - #493

Merged
Ayinkx merged 5 commits into
AyinkxLab:mainfrom
theFirstCodeManiac:feat/guard-review-output-validation
Oct 15, 2026
Merged

Ayinkx merged 5 commits into
AyinkxLab:mainfrom
theFirstCodeManiac:feat/guard-review-output-validation

Conversation

@theFirstCodeManiac

Copy link
Copy Markdown
Contributor

Closes #129

Guard Review Output Validation

Summary

Added validation guardrails to prevent the review service from accepting or generating unverified files, findings, coverage values, or metrics.

Changes

  • Reject parsed model output when it cannot be validated against the available review data.
  • Prevent unsupported files and findings from being surfaced as valid results.
  • Prevent fabricated or unverified coverage numbers and metrics from being accepted.
  • Removed coercion of invalid model output into seemingly valid review data.

Result

The review service now fails safely when model output cannot be validated, ensuring review results are based only on verifiable data.

theFirstCodeManiac and others added 5 commits October 7, 2026 08:54
…p link

The README hero rewrite (c131892) dropped the docs/soroban-workflow.md entry and pointed the Roadmap bullet at docs/roadmap.md, which has never existed in this repository. That left main red: tests/test_docs_links.py::test_relative_doc_links_resolve[README.md] and ::test_soroban_workflow_guide_is_linked_from_readme both fail, and because pull_request workflows test the merge commit, every open PR inherits the same two failures.
@Ayinkx
Ayinkx merged commit 9de76ae into AyinkxLab:main Oct 15, 2026
5 checks passed
@Ayinkx
Ayinkx self-requested a review October 15, 2026 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Prevent fabricated findings, coverage, or metrics

2 participants