Repository navigation
docs(readme): remove synthetic proof and recall-drill download links - #3498
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Signed-off-by: Christopher <131207452+PetrefiedThunder@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: d314a2a6-2061-4c81-8530-f680f6031bd9) |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes. ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
PetrefiedThunder
left a comment
There was a problem hiding this comment.
VS Code Muse review at 086bc73: FAIL
Both removals are confirmed: the synthetic /proof download line and the /proof/recall-drill line are gone.
Blocking: README.md:17 (the old line 18) still says "drop the public audit bundle into the verifier". That public bundle is synthetic:
- frontend/public/proof/audit-bundle.zip (and audit-bundle-tampered.zip) are built by scripts/build_audit_bundle.sh from fsma-204-sample.csv.
- The manifest the script writes says
"_comment": "SYNTHETIC — RegEngine /proof audit bundle. Hashes are real; data is not."(build_audit_bundle.sh:76).
Under the real-data rule, the README can't point users at it. app/verify/page.tsx:66-68 already says "Export your own bundle... the audit bundle ZIP you exported from your RegEngine workspace". Please reword the line to match, for example: "regengine.co/verify: drop an audit bundle exported from your own workspace into the verifier and confirm file-hash and Merkle-root integrity."
Follow-up (separate PR): remove the synthetic files in frontend/public/proof/ (audit-bundle.zip, audit-bundle-tampered.zip, epcis-sample.jsonld, fsma-204-sample.csv), scripts/build_audit_bundle.sh, and demo_exports/sample_audit_bundle.json.
…own workspace Signed-off-by: Christopher <131207452+PetrefiedThunder@users.noreply.github.com>
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: 684958b7-a99f-4951-94ab-2411e711b26f) |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: b5f34cca-f618-44c6-98ac-385ecfcc7bbe) |
PR Blocker ChecklistHead SHA: Status: Ready
|
PetrefiedThunder
left a comment
There was a problem hiding this comment.
REVIEW: PASS 00dd6ab (content). Authored delta since 086bc73 is c9a8e32: README.md:17 now reads "drop an audit bundle exported from your own workspace into the verifier…"; no synthetic/public sample bundle presented as real, no made-up data added. 00dd6ab is a merge of main; its other changes are main's, not this PR's. Note: this PR was already merged at 3:20 AM PT (during the freeze window) with CI failing: PR Gate: Secrets and CI Status Gate fail (21 pass, 3 skipped). This review is after the fact; it is not an approval of that merge.
Removes README lines 17 and 19, which advertised the synthetic /proof downloads and the frozen-synthetic recall drill. README only. proof_recall_router.py, the samples and the recall-drill page are untouched; they wait for #3471.
Note: the /verify bullet (line 18) still says 'the public audit bundle'; left as is for review.
Note
Low Risk
Documentation-only README edits with no application, API, or deployment behavior changes.
Overview
README “Public proof surface” no longer lists synthetic try-it-yourself links for
/proofdownloads (FSMA CSV, EPCIS bundle, audit ZIP) or the frozen-dataset/proof/recall-drilldemo.The section now points readers only at
regengine.co/verify, with copy updated so they verify audit bundles exported from their own workspace instead of a shared public bundle. Header links (e.g. “Public proof”) and runtime proof routes are unchanged in this PR.Reviewed by Cursor Bugbot for commit 00dd6ab. Bugbot is set up for automated code reviews on this repo. Configure here.