Rust Rewrite with Deck Federation and Yaml Deck Source#60
Open
jeprecated wants to merge 142 commits into
Open
Conversation
This was referenced May 25, 2026
…mplete except deferred 0130/0150
De-risk runtime swap, not a rewrite. Removes the entire iced/wgpu/tiny-skia tree and replaces it with a minimal Leptos 0.8 CSR bootstrap, keeping the imperative DOM layer (publish_*, event Closures, generation counters, localStorage helpers) byte-for-byte functional. - Cargo.toml: drop both iced deps; add leptos 0.8.20 (csr) + console_error_panic_hook under the wasm32 target; description now says Leptos. - main.rs: wasm32 entry mounts leptos::mount::mount_to_body(App) with panic hook; native main() is empty so --all-targets builds stay green. - lib.rs: delete run()/theme()/WorkbenchApp/Message/the Iced view()/panel() and the now-orphaned native no-op stubs. Add a wasm32-gated Leptos App component that, on mount, performs exactly the former bootstrap: probe -> loading, then fetch /api/workspace -> publish_workspace_probe(loaded/error) and fetch note pivot -> publish_note_pivot_panel / swallow stale / publish_note_pivot_error. Exact status strings preserved (asserted by E2E). WorkspaceSummary and from_workspace_json unchanged (tests/workspace_summary.rs untouched, green). - crates/brain-brew-cli/assets/workbench: regenerated via workbench-ui-embed; shipped wasm shrinks 3.69MB -> 386KB with zero iced/wgpu strings. Cargo.lock: wasm-bindgen held at 0.2.125 (no skew vs nix cli 0.2.121); iced/ wgpu/tiny-skia removed entirely. Validation: fmt:check clean; workspace tests (excl e2e) green; clippy -D warnings clean (native + wasm32); trunk WASM build succeeds; 13/13 E2E green. Zero edits to the e2e oracle, the server, index.html, or tests/workspace_summary.rs.
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
1 participant
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.
Issue/Discussion #4 is now achieved with this full rewrite. Only 6 years later! 😁
Ultimate Geography (in its entirety) is now a test fixture. The CrowdAnki output from before the rewrite and now is identical and fully tested for all regressions.
If its the same then why bother? This rewrite adds the following cababilities:
I took a lot of inspiration from declarative systems like NixOS while doing this.
This is a work in progress, and thus marked as an alpha for now. Though parity in the output is achieved, which is the main concern. More to come.