fix(desktop): make v0.6 Leptos dashboard viewable#56
Merged
Conversation
…ry point
Three fixes surfaced by local dogfood 2026-05-26, all "manifests lag the
Leptos migration":
- tauri.conf.json: correct before{Dev,Build}Command paths to `cd src-leptos`
(was `../src-leptos`). npm runs the script from desktop/, so the relative
path resolved to a nonexistent loci/src-leptos and the documented
`npm run tauri:dev` never booted.
- dashboard.rs: add #[serde(rename_all = "camelCase")] to PathArg +
HandoversArg so read_cron_states / read_handovers / start_state_watcher
receive `palacePath` (Tauri v2 wire convention expects camelCase).
Without it the reads failed and were swallowed by unwrap_or_default,
rendering an empty dashboard with no error.
- tauri.conf.json: window now resizable (1200x860, min 720x520) so the full
dashboard fits on one page instead of a fixed 680x520.
Co-Authored-By: Claude Opus 4.7 <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
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.
Summary
Three fixes surfaced during a local dogfood of the v0.6
loci wizarddashboard (2026-05-26). All three are cases of manifests lagging the Leptos migration — the documented entry point (npm run tauri:dev) never actually worked, and when forced through, the dashboard rendered empty with no error.before{Dev,Build}Commandusedcd ../src-leptos, butnpm run tauri:devruns fromdesktop/, so the path resolved to a nonexistentloci/src-leptos. Corrected tocd src-leptos. The documented dev invocation now boots.read_cron_states/read_handovers/start_state_watcherexpectpalacePath(Tauri v2 wire convention), but the frontend arg structs serialized snake_casepalace_path. The watcher surfaced the error; the two reads swallowed it viaunwrap_or_default, rendering an empty dashboard. Added#[serde(rename_all = "camelCase")].Scope
This is the "able to view" slice only. Known follow-ups deliberately left for a Phase-3.5 hardening pass (tracked separately):
start_state_watcherwatches<root>/croninstead of<root>/_palace/cron(live-refresh banner still shows; static read works)read_manifestexists + tested, frontend never calls it)Test plan
npm run tauri:devboots fromdesktop/(was broken)/Users/eris/Dev(40 cron jobs, handovers, CIQ 89.5)/opt/anaconda3and/usr/bin/python3paths (n/a — Rust/trunk only)🤖 Generated with Claude Code