fix(start:headless): swap removed start:mock-idp for start:dev-idp#2917
fix(start:headless): swap removed start:mock-idp for start:dev-idp#2917mfbx9da4 wants to merge 1 commit into
Conversation
start:mock-idp referenced ./mock-speakeasy-idp/main, which no longer exists. mprocs.yaml already uses start:dev-idp; align the headless script with it.
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Summary
.mise-tasks/start/headless.shinvokedmise run start:mock-idp, which still points at./mock-speakeasy-idp/main— a directory that was deleted in PR #2825 (fix(auth): admin override, slug dedup, and re-apply Speakeasy IDP removal). Runningmise run start:headlesstherefore fails immediately withstat .../mock-speakeasy-idp/main: directory not foundand never starts the IDP, which in turn blocks login from a headless dev environment.mprocs.yamlwas already updated to usestart:dev-idpat the time of the removal, so this is an orphaned reference in the headless flow. This patch:start:mock-idp→start:dev-idp, andmock-idp.log→dev-idp.logso the new task's logs land in the path the user is told about.Why this matters
mise run start:headlessis the documented headless equivalent ofmadprocs. Without this fix, anyone bringing up Gram outside the TUI hits a stale-task failure during onboarding and has to bypass the script. Reported during a fresh-checkout login walkthrough where everything else needed to be hand-wired before the dashboard could authenticate.Related stale references (not in this PR)
.mise-tasks/seed.mtscarries the same bug: it still tries to authenticate via${SPEAKEASY_SERVER_ADDRESS}/v1/speakeasy_provider/login, which was themock-speakeasy-idpAPI. NeitherSPEAKEASY_SERVER_ADDRESSnorSPEAKEASY_SECRET_KEYis defined anywhere inmise.tomlafter #2825.mise run seedcurrently fails withSPEAKEASY_SERVER_ADDRESS is not set. Fixing it requires re-implementing the auth bootstrap againstdev-idp's mock-workos / OAuth2 flow (cookie-jar +/rpc/auth.login→/rpc/auth.callback), so it deserves its own PR.Test plan
mise run start:headlessreachesAll processes started.without the IDP task crashing.tail /tmp/gram-dev/dev-idp.logshowsdev-idp listeningon:35291.https://localhost:5173completes end-to-end against the mock-workos IDP.