Skip to content

fix: stabilize deploy validation - #10

Merged
kody-bot merged 11 commits into
mainfrom
cursor/ws-mcp-ui-3877
Jul 29, 2026
Merged

fix: stabilize deploy validation#10
kody-bot merged 11 commits into
mainfrom
cursor/ws-mcp-ui-3877

Conversation

@kentcdodds

@kentcdodds kentcdodds commented Jul 29, 2026

Copy link
Copy Markdown
Member

Summary

  • lint repository areas in isolated sequential processes so TypeScript state is released and memory remains bounded
  • eagerly prebundle dependencies for both solution servers and the shared renderer
  • mount renderer iframes only after the real postMessage listener is active, and bridge action responses deterministically
  • install Chromium on Node 22 and keep one CI-owned renderer alive across the complete solution suite
  • preserve every solution test and its behavioral assertions, with 10-second browser-event ceilings under CI load

Failure addressed

Deploy workflow run https://github.com/epicweb-dev/mcp-ui/actions/runs/30413812797 failed because monolithic ESLint accumulated beyond the Node heap on macOS and Windows, while Vite invalidated newly discovered prebundles before Vitest discovered or completed test files.

Validation

  • npm run lint
  • npm run typecheck
  • Chromium installs with Linux dependencies on the browser job’s Node 22 runtime
  • Setup passes on Node 26 across Linux, macOS, and Windows
  • shared renderer exits are propagated immediately
  • real iframe lifecycle and action messages remain the source of test assertions
  • current PR CI is the final source of truth
Open in Web Open in Cursor 

cursoragent and others added 3 commits July 29, 2026 01:39
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kody-bot
kody-bot marked this pull request as ready for review July 29, 2026 02:04
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Comment thread .github/workflows/validate.yml Outdated
cursoragent and others added 5 commits July 29, 2026 02:36
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
# Conflicts:
#	.github/workflows/validate.yml

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5570170. Configure here.

Comment thread .github/workflows/validate.yml
Comment thread epicshop/epic-me/app/routes/mcp-ui-renderer.tsx Outdated
cursoragent and others added 2 commits July 29, 2026 03:34
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kody-bot
kody-bot merged commit 526e58f into main Jul 29, 2026
6 checks passed
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.

3 participants