Skip to content

fix(onboarding): confirm app connection after successful init - #433

Closed
jomifepe wants to merge 1 commit into
mainfrom
codex/onboarding-init-completion
Closed

jomifepe wants to merge 1 commit into
mainfrom
codex/onboarding-init-completion

Conversation

@jomifepe

@jomifepe jomifepe commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Successful init now completes connectPrismic, allowing browser onboarding to confirm the local connection when the user returns. Completion uses the existing idempotent endpoint rather than read-then-toggle, so repeated or concurrent commands cannot undo progress. Reporting failure remains nonfatal, and Instant Start handoff reporting is preserved.

Validation

Passed formatting, lint, TypeScript, and two focused tests covering atomic concurrent/repeated reporting and failed requests. Added integration coverage for repeated successful init and the existing starter handoff.

The local staging integration suite is blocked before tests execute: its legacy repository-creation endpoint returns HTTP 404. The initial draft passed CI. This PR is now closed; follow-up commit 667f257 has been pushed to its branch, and the closed PR has been left closed. The new commit therefore has no PR validation run yet.

Release before enabling the editor onboarding experiment.

@jomifepe jomifepe closed this Oct 21, 2026
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