fix(desktop): remove stale sprout-admin prereq, add sidecar tooling#371
Merged
Conversation
Token minting switched to mint_token_via_api() (NIP-98 HTTP) but the UI still gated the mint toggle on sprout-admin binary availability. Remove the dead prereq check from Rust types, TS types, Doctor panel, and create-agent dialog. Minting now works in release DMG builds without shipping an unused operator binary. Also: DRY up justfile sidecar stubs into _ensure-sidecar-stubs recipe, add scripts/bundle-sidecars.sh for release builds.
fsola-sq
added a commit
that referenced
this pull request
Apr 20, 2026
…-binding * origin/main: fix(desktop): eliminate agent startup beachball (#374) fix(desktop): resolve agent command path for DMG builds (#372) fix(desktop): remove stale sprout-admin prereq, add sidecar tooling (#371) Add server cross-compile and macOS desktop build CI jobs (#369) Fix forum post card bugs on desktop and mobile (#370) fix(desktop): kill WebSocket flood and fix Markdown <p><div> nesting (#368) perf: caching, batched DM resolution, bounded audit, global kind index (#367) fix: staging to generate stubs as needed (#366) chore(deps): update rust crate axum to v0.8.9 (#365) chore(deps): update dependency @tanstack/react-router to v1.168.22 (#364) feat(desktop): autoscroll thread sidebar for new replies (#363) fix(desktop): eliminate 10+ second UI freeze on startup (#361) feat(desktop): bundle sprout-acp and sprout-mcp-server as Tauri sidecars (#362) Remove release pipeline from public repo (#360) Amp-Thread-ID: https://ampcode.com/threads/T-019dab7a-5979-7401-83a1-509b9adfe4a0 Co-authored-by: Amp <amp@ampcode.com> # Conflicts: # crates/sprout-relay/src/state.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.
Summary
sprout-adminprerequisite check that gated the mint toggle in the Create Agent dialog. Token minting usesmint_token_via_api()(NIP-98 HTTP) — the binary was never invoked, but the UI check prevented minting in release DMG builds wheresprout-adminwasn't on PATH._ensure-sidecar-stubsrecipe.scripts/bundle-sidecars.shfor copying release binaries into the Tauri sidecar directory.Changes
Rust: Removed
DEFAULT_ADMIN_COMMANDconst andadminfield fromManagedAgentPrereqsInfoTypeScript: Removed
adminfrom all type definitions, raw Tauri bridge mappings, and e2e mock dataUI: Removed "Token minting" from Doctor panel, removed
isMintSupportedgating from Create Agent dialogBuild: DRYed justfile sidecar stubs, added
bundle-sidecars.shTest plan
cargo checkpassestsc --noEmitpassesbiome checkpassesjust desktop-tauri-checkcompiles clean🤖 Generated with Claude Code