Skip to content

fix: avoid wrapping OAuth URLs in setup UI#308

Open
HwangJohn wants to merge 2 commits into
langchain-ai:mainfrom
HwangJohn:fix/oauth-url-copy-safe
Open

fix: avoid wrapping OAuth URLs in setup UI#308
HwangJohn wants to merge 2 commits into
langchain-ai:mainfrom
HwangJohn:fix/oauth-url-copy-safe

Conversation

@HwangJohn

@HwangJohn HwangJohn commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove the raw wrapped OAuth URL from the bordered setup UI
  • keep the terminal hyperlink and clipboard status so copied URLs do not include Ink border characters
  • show a provider-specific fallback command such as openwiki auth gmail when clipboard support is unavailable

Fixes #296.

Testing

  • corepack pnpm exec prettier --check src/credentials.tsx test/credentials.test.ts
  • corepack pnpm exec vitest run test/credentials.test.ts test/openai-chatgpt-credentials.test.ts
  • corepack pnpm run typecheck
  • corepack pnpm run lint:check
    Note: corepack pnpm test currently fails locally on Windows in test/env-behavior.test.ts > writes the env file with 0600 permissions (mode & 0o077 receives 54). Targeted tests above pass.

@HwangJohn
HwangJohn marked this pull request as ready for review July 13, 2026 11:24
@HwangJohn
HwangJohn force-pushed the fix/oauth-url-copy-safe branch from 08d37d5 to 040517f Compare July 13, 2026 23:46
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.

[Bug]: Auth URL box-drawing characters get copied into the URL

1 participant