feat(polymarket): return published evidence digests - #110
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthrough
ChangesPublished evidence digest reporting
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
Change contract: Return the exact published NDJSON and manifest SHA-256 digests as a publication-only carrier after the immutable evidence triplet is installed.
Out of scope: Constructing the verifier-side
PolymarketEvidenceTrustAnchor; authenticating an OSS object or catalog selection; changing triplet bytes, publication ordering, or no-clobber behavior; snapshot compilation or research consumption.Dependency or merge order: None. The carrier is deliberately named
PublishedPolymarketEvidenceDigestsand requires an explicit downstream conversion into the separately typed verifier trust anchor.Focused validation:
cargo test --locked -p hft-collector polymarket_evidence_artifactcargo clippy --locked -p hft-collector --all-targets --no-deps -- -D warningsrustfmt --edition 2021 --check rust_hft/tools/collector/src/polymarket_evidence_artifact.rsgit diff --checkpublished_result_returns_digests_for_the_exact_tripletrecomputes the manifest digest from the exact installed bytes.Rollout/rollback impact: Additive publisher JSON output only; artifact data, manifest,
_SUCCESS, and publication safety behavior are unchanged. Rollback removes the returned digest carrier without invalidating already-published triplets.Scope exception: None (1 file, 82 added lines).
Summary by CodeRabbit