Skip to content

Fix Codex auth URL parsing#16

Open
guo-yu wants to merge 1 commit into
mainfrom
codex/issue-15-codex-auth-url
Open

Fix Codex auth URL parsing#16
guo-yu wants to merge 1 commit into
mainfrom
codex/issue-15-codex-auth-url

Conversation

@guo-yu

@guo-yu guo-yu commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Fixes #15.

Summary:

  • Strip terminal control sequences from CLI login output before extracting provider URLs and device codes.
  • Add a Codex regression test for colorized device-login output so the returned URL remains browser-safe.

Tests:

  • pnpm --filter @wanman/runtime test -- auth-manager.spawn.test.ts
  • pnpm --filter @wanman/runtime test -- auth.test.ts
  • pnpm --filter @wanman/runtime typecheck

@guo-yu guo-yu mentioned this pull request Jun 2, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Codex provider connection fails on wanman.ai

2 participants

@guo-yu @codex