ci: Add typecheck and build workflow for consumer UI - #5
Merged
Conversation
The consumer Portal Plugin has no automated checks, so a dependency upgrade lands with no signal on whether it broke the app. Add a workflow that installs, type-checks and builds it on every push and pull request against main, so a change like the pending react-router security bump can prove itself before merge. Key changes: - Trigger on push and on pull_request targeting main - Set up Node from ui/consumer/.nvmrc and Bun via oven-sh/setup-bun - Install with bun install --frozen-lockfile, then run typecheck and build - Commit ui/consumer/bun.lock, which the repo builds against (Dockerfile COPYs it) but never had checked in, so a frozen install had nothing to freeze against Related to #4 Claude-Session: https://claude.ai/code/session_01T2Bmcs7qWPaw1R6HnmrmBq
kevwilliams
approved these changes
Sep 11, 2026
kevwilliams
left a comment
Contributor
There was a problem hiding this comment.
Adds a straightforward CI workflow (checkout, setup-node from .nvmrc, setup-bun, frozen-lockfile install, typecheck, build) scoped to ui/consumer, plus the missing bun.lock it depends on. Triggers and working directory are correct, scripts match package.json, no secrets or elevated permissions are involved, and there's no pull_request_target or injected untrusted input. Low risk, closes a real CI gap.
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
A dependency upgrade to the consumer UI plugin lands today with no signal on whether it broke the app, since nothing runs on push or on a pull request.
That already blocks a pending router security fix, because only a build reveals whether the new version breaks routing.
This adds a workflow that installs the app's dependencies, type-checks it, and builds it on every push and pull request, so an upgrade proves itself before merge.
A frozen install needs a lockfile to freeze against, and the app never had one committed, so this change adds that too.
Details
Test plan
Related to #4
https://claude.ai/code/session_01T2Bmcs7qWPaw1R6HnmrmBq