Skip to content

ci(verify): stage CLI bundle resources before cargo steps (issue 17) - #40

Merged
Xxx91n merged 2 commits into
mainfrom
arch/17-verify-cli-staging
Sep 5, 2026
Merged

ci(verify): stage CLI bundle resources before cargo steps (issue 17)#40
Xxx91n merged 2 commits into
mainfrom
arch/17-verify-cli-staging

Conversation

@Xxx91n

@Xxx91n Xxx91n commented Sep 4, 2026

Copy link
Copy Markdown
Owner

Root cause

verify job's cargo test/cargo check on frontend/src-tauri triggers tauri_build::build() (build.rs), which validates the five files declared in tauri.conf.json → bundle.resources under frontend/src-tauri/bin/. Locally frontend/scripts/prebuild.mjs stages them during npm run build/tauri build, but the verify job runs cargo directly — every main push failed at "Run Tauri crate tests" with resource path bin/env-manager-cli.exe doesn't exist (run 33880367303).

Fix

  • Insert a fail-closed staging step ("Stage CLI artifacts into Tauri resource dir (issue 17)") after "Build CLI" and before any cargo compile: copies env-manager-cli.exe/.dll/.runtimeconfig.json/.deps.json from bin/Release/net10.0-windows/ and AGENTS.cli.md from the repo root into frontend/src-tauri/bin/; any missing file exits 1 (5/5 check).
  • Local responsibilities unchanged (build.mjs/prebuild.mjs untouched); no changes to cargo test/check assertions or tauri.conf.json.
  • Docs synced in the same commit: docs/build-and-release.md CI section + AGENTS.md Tauri-shell bullet.

Part of architecture-recovery ticket 17 (main push CI always green).

Env Manager Bot added 2 commits September 5, 2026 02:07
The verify job cargo compiles of the Tauri crate validate tauri.conf.json
bundle.resources, but the five declared files were never staged in CI
(locally frontend/scripts/prebuild.mjs does it during npm run build),
so every main push failed at "Run Tauri crate tests". Insert a fail-closed
staging step after "Build CLI" copying the five resources into
frontend/src-tauri/bin/. Docs synced: build-and-release.md CI section +
AGENTS.md Tauri-shell bullet.
…y (issue 17)

Run 33904993947 proved the five CLI resources stage correctly (5/5 logged)
and the resource gate passed, but compile then hit a second gate:
tauri::generate_context! (main.rs:1409) panics when frontendDist
("../../dist", repo-root dist/) does not exist - the real dist/ is only
produced by "Build frontend", which runs after the cargo steps. Seed a
dist/index.html placeholder in the same staging step; vite build
overwrites it with real output later.
@Xxx91n
Xxx91n merged commit 96abc6b into main Sep 5, 2026
9 checks passed
@Xxx91n
Xxx91n deleted the arch/17-verify-cli-staging branch September 5, 2026 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant