Conversation
- Require `claude auth login` for Claude setup - Remove helper-command settings, contracts, and picker logic - Update related error and setup copy
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
github-actions
bot
added
size:L
vouch:trusted
- Replace layered atmospheric glows with cleaner grid and edge treatment - Remove unused section glow and reflection accents from hero/get-started
- Remove helper-command auth token resolution - Update Claude health checks and settings copy - Strip Anthropic credential env vars before launch
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes Claude auth token-helper support across contracts and the web app UI, and updates user-facing guidance to require authenticating Claude Code via the CLI (claude auth login).
Changes:
- Removes Claude token-helper command fields from shared provider start-option contracts and web app settings.
- Simplifies provider readiness/selection logic so Claude is only selectable when the server reports it ready/authenticated.
- Updates settings/setup UI copy and server doctor output to align with the
claude auth loginflow.
Reviewed changes
Copilot reviewed 30 out of 30 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/contracts/src/provider.test.ts | Removes token-helper expectations from provider session start parsing tests. |
| packages/contracts/src/orchestration.ts | Removes authTokenHelperCommand from ClaudeProviderStartOptions contract schema. |
| apps/web/src/routes/_chat.settings.tsx | Removes token-helper UI and updates Claude setup/auth guidance copy. |
| apps/web/src/lib/providerAvailability.ts | Removes token-helper override path from provider readiness logic. |
| apps/web/src/lib/providerAvailability.test.ts | Updates readiness tests to reflect Claude no longer selectable via token helper. |
| apps/web/src/components/chat/threadError.ts | Updates Claude login guidance text to claude auth login. |
| apps/web/src/components/chat/threadError.test.ts | Updates tests for revised Claude auth error diagnostics copy. |
| apps/web/src/components/chat/ProviderSetupCard.tsx | Updates Claude provider setup instructions and note. |
| apps/web/src/components/ChatView.tsx | Removes token-helper setting from selectable-provider computation dependencies. |
| apps/web/src/appSettings.ts | Removes token-helper setting and stops emitting it in provider start options. |
| apps/web/src/appSettings.test.ts | Updates app settings defaults and provider start options tests accordingly. |
| apps/server/src/doctor.ts | Updates doctor output to require claude auth login (no env/token-helper mention). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
claudeAuthTokenHelperCommandfrom shared contracts, app settings, provider selection, and settings UI.claude auth logininstead of API-key or token-helper based auth.Testing
bun fmtbun lintbun typecheck