Skip to content

(MOT-4184) fix(ci): package all console frontend bundles - #601

Merged
ytallo merged 1 commit into
mainfrom
fix/console-release-ui
Jul 24, 2026
Merged

(MOT-4184) fix(ci): package all console frontend bundles#601
ytallo merged 1 commit into
mainfrom
fix/console-release-ui

Conversation

@ytallo

@ytallo ytallo commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Refs MOT-4184

Console releases now package both the embedded SPA and the injectable UI assets, so the published console can host worker-provided pages such as eval.

Technical details

The shared Rust release workflow detects every frontend directory available on a worker, builds each once, and restores their original paths in every platform build.

Validation

  • pnpm install --frozen-lockfile && pnpm build in console/ui
  • pnpm install --frozen-lockfile && pnpm build in console/web
  • YAML parse and whitespace validation for the release workflow

Summary by CodeRabbit

  • Bug Fixes
    • Improved release builds for components that include multiple frontend bundles.
    • Ensured all detected frontend assets are built, packaged, and restored correctly during Linux builds.
    • Prevented unnecessary frontend rebuilds when compiled assets are already available.

@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
workers Ready Ready Preview, Comment Jul 24, 2026 9:43pm
workers-tech-spec Ready Ready Preview, Comment Jul 24, 2026 9:43pm

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

skill-check — worker

0 verified, 49 skipped (no docs/).

Layer Result
structure
vale
ai
render

Four for four. Nicely done.

@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e8d2ea76-caee-4d83-8c61-58fc44398131

📥 Commits

Reviewing files that changed from the base of the PR and between 7bfc1fe and d034485.

📒 Files selected for processing (1)
  • .github/workflows/_rust-binary.yml

📝 Walkthrough

Walkthrough

The Rust binary workflow now detects and builds all available frontend bundles, stages them in a unified artifact, and restores the matching web/dist and ui/dist paths across matrix build shards.

Changes

Frontend bundle workflow

Layer / File(s) Summary
Detect and stage frontend bundles
.github/workflows/_rust-binary.yml
The Linux web-build job detects available web and ui frontends, builds each one, stages their dist directories, and uploads a unified artifact.
Restore bundles for matrix builds
.github/workflows/_rust-binary.yml
Matrix shards download the unified artifact into the worker root so frontend outputs are restored before cargo build.
Estimated code review effort: 3 (Moderate) ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant web-build
  participant pnpm
  participant frontend-bundles
  participant matrix-shard
  participant cargo-build
  web-build->>web-build: Detect web and ui frontends
  web-build->>pnpm: Build each detected frontend
  pnpm-->>web-build: Return dist outputs
  web-build->>frontend-bundles: Upload staged bundles
  matrix-shard->>frontend-bundles: Download bundles into worker root
  frontend-bundles-->>matrix-shard: Restore web/dist and ui/dist
  matrix-shard->>cargo-build: Build with restored frontend content
Loading

Possibly related PRs

  • iii-hq/workers#591: Updates the same workflow’s frontend bundle selection and restoration behavior.

Suggested labels: no-ticket

Suggested reviewers: sergiofilhowz

Poem

I’m a rabbit with bundles tucked neat,
web and ui now hop in one fleet.
Built once in the burrow,
Restored without sorrow,
Then Rust gets its frontend treat! 🐇

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: packaging all console frontend bundles in CI.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/console-release-ui

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ytallo
ytallo merged commit 83bc78b into main Jul 24, 2026
16 of 17 checks passed
@ytallo
ytallo deleted the fix/console-release-ui branch July 24, 2026 21:44
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