Prefer Claude user ID for telemetry identity#1249
Merged
juliusmarminge merged 1 commit intomainfrom Mar 20, 2026
Merged
Conversation
- Read `~/.claude.json` when Codex account ID is unavailable - Fall back to the existing anonymous telemetry ID
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
github-actions
bot
added
size:S
aaditagrawal
pushed a commit
to aaditagrawal/t3code
that referenced
this pull request
Mar 21, 2026
7 tasks
gabrielMalonso
pushed a commit
to gabrielMalonso/t3code
that referenced
this pull request
Mar 21, 2026
bulbulogludemir
added a commit
to bulbulogludemir/krabbycode
that referenced
this pull request
Mar 21, 2026
Ported from pingdotgg/t3code (bc12421..9e29c9d): - feat: persist modelOptions, refactor provider-specific logic (pingdotgg#1121) Sticky model/options in composer, composerProviderRegistry, shared resolveSelectableModel utility - fix(ChatView): update scroll button styles (pingdotgg#1246) - fix(web): show health banner for selected provider (pingdotgg#1223) - fix(web): update add-project toggle label in cancel state (pingdotgg#1248) - Prefer Claude user ID for telemetry identity (pingdotgg#1249) - feat: configurable base directory (pingdotgg#826) Renamed stateDir → baseDir, T3CODE_STATE_DIR → T3CODE_HOME, deriveServerPaths for attachmentsDir/logsDir/dbPath/anonymousIdPath All changes adapted for Krabby branding and merged with existing features. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.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
1 participant
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.
~/.claude.jsonwhen Codex account ID is unavailableWhat Changed
Why
UI Changes
Checklist
Note
Prefer Claude user ID from
~/.claude.jsonfor telemetry identityUpdates
getTelemetryIdentifierin Identify.ts to check~/.claude.jsonfor auserIDfield after the Codex account ID but before falling back to the anonymous ID. A newgetClaudeUserIdeffect reads and decodes the file usingClaudeJsonSchema, and the resulting ID is hashed like other identifiers. Risk: adds a file read and JSON decode on every telemetry identifier resolution.Macroscope summarized 98c8c03.