Repository navigation
Final review round: replay trail, trust wording, live-run numbers - #11
Merged
Merged
Conversation
- The replay's "What Base saw" shows each recorded step's time on Base, no block number of the replay's own chain, and says it was recorded from Base in the live run. - The replay's deposit deadline is the run's: the harvest takes the Zcash tip as the vault opened (recording.json: zcashTip only). - Trust, stated plainly in the README, SUBMISSION, JURY_FAQ, the docs page and the app: why there are signers, what they see, what three of them could do and what follows. - The live run's numbers in place of the forks': the loan's and the swap's timings, the leak scan (none of 181 identifiers), the cbZEC market as read on 27 October; "each on-chain step" where a step has no transaction. - Mocks: the market's liquidity and bots are listed; the onion service, never deployed, is no longer listed as real. - deploy-vercel.sh: a live build needs its backend's /health to answer; without a live backend, /live/ redirects to the replay at /app/. - Site: the hero comes in at once; on a phone the scene's two tags no longer land on the eyebrow; the video has a label and its captions are re-split. - .gitignore: data/ at any depth and the app's loan key backups; the watchdog example names no real address. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The last review round before the submission: no contract or guest changes.
App (the replay)
recording.jsonchanges inzcashTiponly.Docs and site
Tooling
scripts/deploy-vercel.sh: a live build needs its backend's/healthto answer; without a live backend,/live/redirects to the replay at/app/..gitignore:data/at any depth and the app's loan key backups; the watchdog example names no real address.Checked locally
vitest run464 passed, 1 skipped (the harvest itself);tsc --noEmitclean.bash scripts/deploy-vercel.sh --build-only: the replay export checks out (16 pages, 152 files),vercel.jsoncarries the/live/redirect.🤖 Generated with Claude Code