Skip to content

fix(setup): report account conflicts clearly and prepare 1.0.6 - #98

Merged
escott- merged 3 commits into
mainfrom
fix/signed-setup-account-conflict
Sep 12, 2026
Merged

escott- merged 3 commits into
mainfrom
fix/signed-setup-account-conflict

Conversation

@escott-

@escott- escott- commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

Signed-in setup commands printed “Signed in as” before verifying an existing machine account. A conflicting account or verification failure could then stop setup after that misleading success message.

Report sign-in success only after credential resolution, preserve existing credentials on non-interactive conflicts, and explain the fresh-command, interactive-confirmation and CI/environment recovery paths. This completes the installer portion of contextstream/contextstream#591; the matching documentation is already live. Prepare version 1.0.6 across Rust, npm and MCP Registry metadata.

Test plan

  • Real CLI tests use an isolated temporary home and local mock API: conflicting accounts and API verification errors fail without sign-in success or credential changes.
  • Both regression tests fail on unchanged main and pass with this fix.
  • Release-contract tests: 12 passed.
  • Formatting check passed; refreshed full CI must pass before merge.
  • No real account credentials were used or changed. Publishing follows the protected release workflow.

Signed-off-by: escott- <escott05@gmail.com>
@escott-
escott- force-pushed the fix/signed-setup-account-conflict branch from c173839 to 722cb62 Compare September 12, 2026 16:25
@escott- escott- closed this Sep 12, 2026
@escott- escott- reopened this Sep 12, 2026
Signed-off-by: escott- <escott05@gmail.com>
@escott- escott- changed the title fix(setup): explain account conflicts before reporting sign-in success fix(setup): report account conflicts clearly and prepare 1.0.6 Sep 12, 2026
@escott-
escott- marked this pull request as ready for review September 12, 2026 19:27
Signed-off-by: escott- <escott05@gmail.com>
@escott-
escott- merged commit df13d28 into main Sep 12, 2026
10 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.

2 participants