Skip to content

Simplify CLI guest login upgrade#309

Merged
leggetter merged 3 commits into
feat/cli-guest-tracking-fixesfrom
codex/simplify-cli-guest-flow-cli
Jul 7, 2026
Merged

Simplify CLI guest login upgrade#309
leggetter merged 3 commits into
feat/cli-guest-tracking-fixesfrom
codex/simplify-cli-guest-flow-cli

Conversation

@alexbouchardd

Copy link
Copy Markdown
Contributor

Summary

  • remove guest_user_id from CLI profile/config handling
  • refresh guest sign-in links through POST /cli/guest
  • make hookdeck login reuse the existing guest key and wait for it to validate as non-guest

Tests

  • go test ./pkg/config ./pkg/hookdeck ./pkg/login ./pkg/gateway/mcp
  • go test -tags guest ./test/acceptance
  • git diff --check

alexbouchardd and others added 3 commits July 3, 2026 14:31
Refresh calls POST /cli/guest with link_context only; empty device_name
was rejected by server Joi validation (422). Add omitempty and assert the
key is absent in unit and guest acceptance tests.

Co-Authored-By: Claude <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
…ixes

Bring #308 handling into simplify guest flow: CI keys fail fast on
project list/use, login rejects headless CI keys, keep waitForGuestUpgrade
for active guest profiles.

Co-Authored-By: Claude <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@leggetter leggetter merged commit f147d49 into feat/cli-guest-tracking-fixes Jul 7, 2026
1 check passed
@leggetter leggetter deleted the codex/simplify-cli-guest-flow-cli branch July 7, 2026 14:39
@leggetter

Copy link
Copy Markdown
Collaborator

Alex's simplify guest-login work now lives on feat/cli-guest-tracking-fixes (folded in via fast-forward).

Review here: #306#306

That PR carries the simplified login flow (waitForGuestUpgrade, guest link refresh without device_name, project-scoped credential handling from #308).

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