feat(claude-trace): Add support for native bun binary#56
Open
Richard-Weiss wants to merge 2 commits intobadlogic:mainfrom
Open
feat(claude-trace): Add support for native bun binary#56Richard-Weiss wants to merge 2 commits intobadlogic:mainfrom
Richard-Weiss wants to merge 2 commits intobadlogic:mainfrom
Conversation
- Detect native binaries (ELF/Mach-O/PE) via magic bytes - Use HTTP reverse proxy with ANTHROPIC_BASE_URL for compiled Claude - Add optional --include-sensitive-headers flag to disable header redaction Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Use 'where' instead of 'which' on Windows - Handle 'where' returning multiple lines - Skip bash wrapper detection on Windows - Add Windows paths (.local/bin/claude.exe, %APPDATA%/npm/claude.cmd) - Add missing paths (~/.claude/bin/claude, /opt/homebrew/bin/claude) Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Author
|
I tested it on Win 10. Got some issues because of the Claude Code NPM dependency, so kicked that out. After that having to run build in Git Bash because of cp commands in the build script. But, the reverse proxy approach worked there too, so seems "good enough" to me for the scope of this PR. |
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
2 participants
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.
Adds support for tracing Claude when installed as a native binary (Bun-compiled) instead of Node.js. Uses a reverse proxy approach since the
--requireinterceptor only works with Node.--include-sensitive-headersthat logs bearer token and other credentialsI only tested it on Ubuntu Server so would be nice if someone on Mac and Windows were to test it before any possible merge.
Should resolve #37 and #48.