Repository navigation
Roster on the site: plain Memory64 message, exact Examples blurb - #7
Merged
Merged
Conversation
The site links the Roster app, which needs Memory64: a browser without it (Safari) now gets the editor's plain message instead of the raw start error. The Examples blurb says what the site has: no notary, one roster notarized when the site was built, which the verify page checks again. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016R1ctPHanJvc74mwoJrYgd
Owner
Author
Final summary (stop time, 2026-10-22 18:00 UTC)This PR has two small fixes to the Roster demo on the site:
A review of the site version, run while #6 was merging, found both. Checked locally:
It is a draft, and CI run 39826434271 had not finished at stop time. What this session did: #6, the Roster demo, merged at 17:38 UTC.
CI
Numbers (CI machines, to be re-measured on the reference Mac)
Open problems
Next steps for you
Generated by Claude Code |
FawadHa1der
marked this pull request as ready for review
October 26, 2026 16:01
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.
A follow-up to #6 with two small fixes. A review of the site version, run while #6 was merging, found both.
examples/roster.html: the site now links the Roster app, which needs WebAssembly Memory64. In a browser without it (Safari), the page used to show the raw "LeanUI failed to start" error. It now shows the same plain message as the photo editor (examples/editor.html). The verify page is wasm32 and works in Safari regardless.examples/Site.lean: the Examples blurb said "a roster comes with a proof", but the site has no notary. It now says what the site has: one roster notarized when the site was built, which an independent kernel checks again in your browser.Checks
lake build leanui-sitepasses.tests/browser/roster.mjspasses in Chromium with the new page (84 checks).Generated by Claude Code