Skip to content

Commit ddb0807

Browse files
toothbrushclaude
andcommitted
logout: fix stale revokeCurrentFunc doc comment
The comment claimed the impl resolves its own data-API bearer; the caller now resolves the active context's core URL + token and binds them into the closure. Describe the actual contract. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Entire-Checkpoint: 2525dd1ceb9f
1 parent 9901fbf commit ddb0807

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

cmd/entire/cli/logout.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,10 @@ type tokenStore interface {
1919
DeleteToken(baseURL string) error
2020
}
2121

22-
// revokeCurrentFunc revokes the CLI's current token server-side. The
23-
// implementation resolves its own data-API bearer (same audience-
24-
// matching rule as sessionLister); callers don't pass the keyring
25-
// entry through.
22+
// revokeCurrentFunc revokes the CLI's current login session server-side.
23+
// The caller resolves the active context's core URL + bearer up-front and
24+
// binds them into the closure, so the revocation hits the same core that
25+
// `auth status` lists.
2626
type revokeCurrentFunc func(ctx context.Context) error
2727

2828
// clearContextFunc removes the active contexts.json context (and its

0 commit comments

Comments
 (0)