Skip to content

fix: harden review receipt evidence#56

Merged
MageByte-Zero merged 7 commits into
mainfrom
codex/receipt-integrity
Jul 13, 2026
Merged

fix: harden review receipt evidence#56
MageByte-Zero merged 7 commits into
mainfrom
codex/receipt-integrity

Conversation

@MageByte-Zero

Copy link
Copy Markdown
Owner

Fixes #45.

What changed

  • Anchors review evidence paths to the change directory and requires a physical, non-symlink review overlay.
  • Verifies review ranges are real Git commits with an ancestor relationship, then stores their canonical commit IDs.
  • Revalidates persisted review receipts before dependent waves or closing can rely on them.
  • Migrates fixtures to real Git evidence and documents the stricter receipt contract.

Root cause

Receipt metadata accepted caller-controlled paths and SHA-like strings without proving that the evidence or commit range was trustworthy.

Validation

  • npm run build
  • npm test
  • npm run validate -- docs/examples/add-dark-mode
  • npm run validate -- docs/examples/refactor-auth-boundary
  • node scripts/lint/lint-skills.mjs --include token
  • node scripts/spec-superflow.mjs doctor

No release, tag, or npm publish is included.

@MageByte-Zero
MageByte-Zero marked this pull request as ready for review July 13, 2026 05:56
@MageByte-Zero
MageByte-Zero merged commit 1ed89cb into main Jul 13, 2026
6 checks passed
@MageByte-Zero
MageByte-Zero deleted the codex/receipt-integrity branch July 13, 2026 05:56
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.

不并行和跳步BUG

1 participant