Skip to content

feat: configurable OAuth callback port & remove client flags#71

Merged
lollipop-onl merged 10 commits into
mainfrom
feat/oauth-port-and-flag-cleanup
Mar 10, 2026
Merged

feat: configurable OAuth callback port & remove client flags#71
lollipop-onl merged 10 commits into
mainfrom
feat/oauth-port-and-flag-cleanup

Conversation

@lollipop-onl
Copy link
Copy Markdown
Collaborator

@lollipop-onl lollipop-onl commented Mar 10, 2026

Summary

  • startCallbackServer にポート引数を追加(デフォルト 5033、0 で OS ランダム割り当て)
  • --client-id / --client-secret CLI フラグを削除し、環境変数 (BACKLOG_OAUTH_CLIENT_ID, BACKLOG_OAUTH_CLIENT_SECRET) のみに統一
  • 環境変数 BACKLOG_OAUTH_PORT を追加(OAuth コールバックポートの変更が可能に)
  • oauth-callback テストをランダムポートに移行し、ポート競合による不安定さを解消

Test plan

  • backlog-utils テスト全パス(ランダムポート化済み)
  • cli テスト全パス(フラグ削除反映済み)
  • 全453テスト、95ファイルパス
  • BACKLOG_OAUTH_PORT=8080 bee auth login -m oauth で任意ポートでの OAuth フローが動作すること

🤖 Generated with Claude Code

lollipop-onl and others added 8 commits March 10, 2026 12:43
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…H_PORT support

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ive prompt

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@lollipop-onl lollipop-onl force-pushed the feat/oauth-port-and-flag-cleanup branch from 049cad6 to 8145ee1 Compare March 10, 2026 03:45
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 10, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 92.27% 1779 / 1928
🔵 Statements 92.35% 1849 / 2002
🔵 Functions 90.78% 384 / 423
🔵 Branches 77.86% 837 / 1075
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
apps/cli/src/commands/auth/login.ts 98.18% 71.42% 100% 98.14% 79
packages/backlog-utils/src/oauth-callback.ts 95.55% 92.85% 88.88% 95.55% 85-86
Generated in workflow #358 for commit 0ed7077 by the Vitest Coverage Report Action

lollipop-onl and others added 2 commits March 10, 2026 12:49
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@lollipop-onl lollipop-onl merged commit 40e954a into main Mar 10, 2026
7 checks passed
@lollipop-onl lollipop-onl deleted the feat/oauth-port-and-flag-cleanup branch March 10, 2026 03:59
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