Skip to content

fix: require complete source verification - #16

Merged
royalpinto007 merged 2 commits into
AgentPostmortem:mainfrom
be-student:codex/10-complete-source-verification
Sep 6, 2026
Merged

fix: require complete source verification#16
royalpinto007 merged 2 commits into
AgentPostmortem:mainfrom
be-student:codex/10-complete-source-verification

Conversation

@be-student

Copy link
Copy Markdown
Contributor

What & why

Supplying an empty or partial source_contents mapping could still produce a valid receipt verdict. Explicit source verification now requires coverage of every source recorded in the receipt.

Changes

  • distinguish an omitted source mapping from an explicitly empty mapping
  • report all missing source IDs alongside any content mismatches
  • document the complete-mapping contract and add empty, partial, and combined-failure tests

Checklist

  • Focused change with a clear description
  • Ran format / lint / tests (see README)
  • Updated docs or CHANGELOG if relevant
  • Linked any related issue

Closes #10

Validation: pytest -q; ruff check .; python examples/demo_rag.py; git diff --check.

@royalpinto007
royalpinto007 force-pushed the codex/10-complete-source-verification branch from 09747a9 to e795217 Compare September 6, 2026 19:51
@royalpinto007
royalpinto007 merged commit 6afb5cb into AgentPostmortem:main Sep 6, 2026
2 checks passed
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.

Empty or partial source_contents passes as verified

2 participants