Skip to content

Canonicalize reviewer evidence before sanitization#1796

Merged
chubes4 merged 1 commit into
mainfrom
fix/canonical-reviewer-evidence-ref
Jul 14, 2026
Merged

Canonicalize reviewer evidence before sanitization#1796
chubes4 merged 1 commit into
mainfrom
fix/canonical-reviewer-evidence-ref

Conversation

@chubes4

@chubes4 chubes4 commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Canonicalize native transcript references before runtime-result sanitization, using realpath containment, no symlink traversal, byte digests, and a single relative reviewer-evidence descriptor.
  • Make upload staging consume and revalidate only the dedicated descriptor.
  • Cover hosted duplicate refs, distinct transcripts, stale digests, and reviewer-safe transcript projection.

Fixes #1795

Regression evidence: https://github.com/Automattic/build-with-wordpress/actions/runs/29350690551

Testing

  1. npm run build
  2. npm run test:agent-task-contracts
  3. npm run test:runtime-sources-materialization
  4. npm run test:native-agent-task-lifecycle
  5. npm run test:native-agent-task-interruption
  6. npm run test:native-agent-task-playground-e2e
  7. npm run test:redaction
  8. npm run test:production-boundary-enforcement
  9. actionlint .github/workflows/run-agent-task.yml .github/workflows/agent-task-contracts.yml

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode with openai/gpt-5.6-terra
  • Used for: Implemented the canonical reviewer-evidence boundary, added regression coverage, and ran verification with Chris Huber.

@chubes4 chubes4 merged commit 1abe135 into main Jul 14, 2026
4 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.

Canonicalize reviewer transcript refs before result sanitization

1 participant