Skip to content

Hold the add-project intent until an environment exists - #124

Merged
badcuban merged 1 commit into
mainfrom
fix/add-project-intent-race
Aug 7, 2026
Merged

Hold the add-project intent until an environment exists#124
badcuban merged 1 commit into
mainfrom
fix/add-project-intent-race

Conversation

@badcuban

@badcuban badcuban commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Root-causes the escalating palette suite failures (issue #117's second layer, after the ghost-click fix in #121): the add-project open intent was consumed unconditionally on palette mount, branching on whatever environment data existed at that instant. Clicking during startup hydration dead-ended the flow permanently. The intent now waits for environment options (effect re-runs on arrival; palette close cancels; hosted-static proceeds immediately to pairing guidance).

Local validation: suite pass rate went from 1/5 to 5/6; the residual create-folder failure now self-diagnoses via lazy timeout messages that report the palette's actual rendered state.

Real-user impact beyond tests: clicking Add a project during app startup no longer produces an unrecoverable dead palette.

Opening add-project is a one-shot intent, and it was consumed the
moment the palette mounted, against whatever environment data had
hydrated by then. A click during startup landed on an empty list: the
flow dead-ended with guidance meant for genuinely environment-less
sessions, and the arriving environments a moment later could not revive
it. This is also why the palette browser suites failed most local runs.
The intent now waits for the options to exist (the effect re-runs when
they arrive; closing the palette still cancels), while hosted-static
tabs keep their immediate pairing guidance.

Also teaches the suite's waitFor helpers to report what the palette
actually rendered at timeout, so the next residual failure diagnoses
itself.
@vercel

vercel Bot commented Aug 7, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
threadlines Skipped Skipped Aug 7, 2026 4:46pm

Request Review

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S labels Aug 7, 2026
@badcuban
badcuban merged commit e5e2dd6 into main Aug 7, 2026
15 checks passed
@badcuban
badcuban deleted the fix/add-project-intent-race branch August 7, 2026 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant