Add server cross-compile and macOS desktop build CI jobs#369
Merged
Conversation
Restore build capabilities deleted in PR #360: server cross-compilation for x86_64/aarch64 musl targets and macOS desktop Tauri build. Also adds desktop-release-build justfile target for local unsigned builds. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add save-if to rust-cache in both new jobs to prevent cache poisoning from fork PRs (only save cache on main branch) - Update taiki-e/install-action SHA to match current v2 tag Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ings zizmor flags Swatinem/rust-cache as a cache poisoning vector regardless of save-if. These are validation-only jobs where caching is nice-to-have, not critical. Can re-add with actions/cache if build times are painful. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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
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
x86_64-unknown-linux-muslandaarch64-unknown-linux-musltargets usingcross@0.2.5— catches musl linking issues that native Ubuntu builds misscargo checkapproachdesktop-release-buildjustfile target for local unsigned builds with configurable target triplestagingjustfile target to prevent Tauri compile-time validation failuresThese restore build capabilities accidentally deleted in PR #360.
What's NOT in this PR
Test plan
just desktop-release-buildworks locally on macOSjust stagingno longer fails on sidecar validation🤖 Generated with Claude Code