Skip to content

[SYMPP-779B] Gate the production dashboard build - #586

Open
Pimpmuckl wants to merge 1 commit into
mainfrom
2kftdpbmfedr2ihh
Open

[SYMPP-779B] Gate the production dashboard build#586
Pimpmuckl wants to merge 1 commit into
mainfrom
2kftdpbmfedr2ihh

Conversation

@Pimpmuckl

@Pimpmuckl Pimpmuckl commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Context

The full delivery gate did not prove that Vite could produce the production dashboard, and Elixir onboarding named an outdated toolchain.

TL;DR

Build production dashboard assets once in full local and PR gates; keep the fast Elixir loop unchanged.

Summary

  • Make root make all install locked frontend packages and run the production build once.
  • Build production assets once in the PR frontend job after its existing test and quality checks.
  • Document the root full gate and correct Elixir onboarding to pinned 1.20.1 / OTP 29.

Alternatives

  • Kept make -C elixir all unchanged instead of slowing the backend-only development loop.
  • Reused the existing npm scripts and lockfile instead of adding a helper target or dependency.

Test Plan

  • make -C elixir all - 1658/1662 passed; four existing Windows artifact tests report database_unsupported.
  • make -C elixir ci-full - not required for this quick-fix package.
  • npm ci && npm run build && npm test && npm run quality - all frontend gates passed in delivery order.
  • Dry-run gate counts - root build: 1; Elixir-only build: 0.
  • Review Suite fast rvw_06a253d1 - clean after verified findings were fixed.

Summary:
- install locked frontend dependencies and build dashboard assets once
- add the production build to the PR gate and correct onboarding

Rationale:
- fail delivery when Vite cannot produce the production dashboard
- keep fast backend iteration free of frontend build work

Tests:
- npm ci && npm run build && npm test && npm run quality
- make all (1658/1662 tests passed; 4 existing Windows artifact
  launcher failures report database_unsupported)
- dry-run counts: root build 1, Elixir-only build 0

Co-authored-by: Codex <codex@openai.com>
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