Repository navigation
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
|
Thanks for the pull request, @Xuoon. It was closed automatically because it does not meet the requirements below yet. This is an automated check, not a review of the change. Contributor approval. bb reviews pull requests only from approved contributors, and you are not on the list yet. Join the #contributors channel in Discord (https://discord.gg/kvBU6tJhcJ), post a link to your pull request, and explain what it changes and why. A maintainer then adds you to the list and reopens the pull request. See CONTRIBUTING.md for details. |
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.
Human comments
What was wrong
buildSessionEnvsetsCLAUDE_CODE_ENTRYPOINT=cli(#84), which Claude Code sends ascc_entrypoint=sdk-cliin SDK mode. Anthropic meterssdk-*entrypoints at roughly 1.6-2.9x the 5-hour usage of the interactive CLI for the same tokens (anthropics/claude-code#97074), so subscription users hit the 5-hour limit sooner in bb. T3 Code fixed the same problem by sending its own name (pingdotgg/t3code#28410). Theo says he got "semi-solid permission" from Anthropic to merge that change (video, 14:37).What changed
bridge.ts: Claude Code sessions default toCLAUDE_CODE_ENTRYPOINT=bb. An inherited value is still replaced. A value in the thread'senvVarsnow wins, so a user can opt out.initialization-probe.ts: the model-list and slash-command probes sendbbtoo. Until now they passed through whatever the bridge inherited.CLAUDE_AGENT_SDK_CLIENT_APPis still removed, as before.claude-code-providerskill anddocs/configuration.md.CLAUDE_AGENT_SDK_CLIENT_APP, which sendssdk-ts. T3 Code measuredsdk-tsat the higher rate (Claude 5h limit drains about 1.7x faster in T3 Code than in the Claude Code CLI (Anthropic Agent SDK metering) pingdotgg/t3code#24649).HOST_DAEMON_PROTOCOL_VERSIONbump.How you verified
claude -p --input-format stream-json --output-format stream-json) against a local server:cli:cc_entrypoint=sdk-cli,User-Agent: claude-cli/2.1.317 (external, sdk-cli)bb:cc_entrypoint=bb,User-Agent: claude-cli/2.1.317 (external, bb)agent-sdk/<version>.cc_entrypointis the same.it.eachtest inbridge.test.ts: an inheritedinheritedbecomesbb, and a configuredcustomstayscustom. Thecustomcase fails when the default is spread afterenvOverrides.thread/startenv test expectsbb.bb.main(Expected: "bb",Received: "cli").pnpm exec turbo run typecheck test lint --filter=bb-plugin-provider-claude-code: 431 tests pass, lint is clean, andoxfmt --checkpasses on the changed files.bbvssdk-clion a real account. That evidence is T3 Code's measurement fort3-code.Fixes #6951