Skip to content

feat(auth): bind loopback callback lifetime - #90

Open
Windsor Nguyễn (windsornguyen) wants to merge 1 commit into
feat/cli-oauth-http-boundaryfrom
feat/cli-oauth-loopback-listener
Open

feat(auth): bind loopback callback lifetime#90
Windsor Nguyễn (windsornguyen) wants to merge 1 commit into
feat/cli-oauth-http-boundaryfrom
feat/cli-oauth-loopback-listener

Conversation

@windsornguyen

Copy link
Copy Markdown
Member

Note

Authorship: AI-assisted.

Summary

Bind browser authorization to one loopback listener with verified state and issuer.

Public CLI activation follows in a later layer. Hosted authentication requires a compatible server deployment.

Changes

  • src/auth/README.md
  • src/auth/oauth/callback.ts
  • src/auth/oauth/loopback.ts
  • src/auth/oauth/page.ts
  • tests/auth/callback-listener.test.ts

Verification

Verified at c4c362787e773ce61f7106a643c8f14601895464 against its immediate parent.

  • All four type-check scopes and the source build passed.
  • All 70 discovered TypeScript and native-JavaScript tests passed.
  • The macOS native keyring test wrote, read, and deleted a disposable entry.
  • Source/build public-content scans and the actual package archive check passed.
  • The installed preview passed all 25 feedback tests.
  • Frozen pnpm 10.34.5 installation passed.

Scope

Human-written additions + deletions: 378.

Depends on #89. Retains #60: valid automatic retry keys.

The layer keeps its boundary implementation and direct contract tests together. Command activation and unrelated integrations remain separate.

Review focus

Credential rejection must stop its selected path. Keep token persistence, revocation, safe output, and caller identity consistent with the tested contract.

Documentation

Authentication contract at this head.

Reviewers

Signed-off-by: Windsor Nguyen <93564933+windsornguyen@users.noreply.github.com>
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