Skip to content

Add Claude Code provider support across server, contracts, and web#530

Closed
projectashik wants to merge 4 commits intopingdotgg:mainfrom
projectashik:feat/claude
Closed

Add Claude Code provider support across server, contracts, and web#530
projectashik wants to merge 4 commits intopingdotgg:mainfrom
projectashik:feat/claude

Conversation

@projectashik
Copy link

@projectashik projectashik commented Mar 8, 2026

  • add Claude adapter/service and register it alongside Codex
  • extend provider/runtime schemas, health checks, and orchestration flow for Claude
  • update web settings/session logic to handle Claude as a selectable provider
  • expand tests to cover Claude provider registry and runtime ingestion

Note

Add Claude Code provider support by implementing ClaudeAdapter.makeClaudeAdapterLive and wiring provider selection, health checks, models, and UI across server, contracts, and web

Introduce a full Claude CLI adapter with session and turn streaming, register claude in the provider registry and orchestration types, add Claude health checks and status parsing, define Claude models and defaults in contracts, and expose Claude selection and custom models in the web UI. Key entry points: ClaudeAdapter.ts, ProviderAdapterRegistry.ts, ProviderHealth.ts, and model.ts.

📍Where to Start

Start with the Claude provider implementation in makeClaudeAdapterLive in ClaudeAdapter.ts, then follow registration in ProviderAdapterRegistry.ts and health checks in ProviderHealth.ts.

Macroscope summarized 9bea35f.

- add Claude adapter/service and register it alongside Codex
- extend provider/runtime schemas, health checks, and orchestration flow for Claude
- update web settings/session logic to handle Claude as a selectable provider
- expand tests to cover Claude provider registry and runtime ingestion
@coderabbitai
Copy link

coderabbitai bot commented Mar 8, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 8997a315-d685-4cca-902d-2e82363e80d2

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

- Default legacy settings snapshots to empty custom model arrays
- Guard chat settings accessors against undefined custom model lists
- Add regression test for loading persisted settings without Claude models
- Ignore stale turn-complete events unless they match the current turn
- Emit `session.state.changed` only for the active turn to prevent false transitions
- Mark `hasConversation` on process exit only when resume cursor matches session
- add ClaudeAdapter lifecycle tests for stale exit and resumability behavior
- parameterize auth status parsing messages by provider context
- verify Claude health checks report Claude-specific auth guidance
@brrock
Copy link

brrock commented Mar 8, 2026

they have their own pr and are awating antrhopic's permission to merge #179

@maria-rcks
Copy link
Collaborator

closing this for now because it does not follow our contribution guidelines:
https://github.com/pingdotgg/t3code/blob/main/CONTRIBUTING.md

for ui changes, please include clear before/after screenshots, and a short video if the change depends on motion or interaction details.

feel free to update the pr with that context and reopen it.

@maria-rcks maria-rcks closed this Mar 9, 2026
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.

3 participants