Fix/workflow test isolation#191
Closed
Jack-261108 wants to merge 4 commits intoclaude-code-best:mainfrom
Closed
Conversation
added 4 commits
April 8, 2026 09:32
Improve multiline tool rendering and agent token accounting while removing obsolete WebFetch preflight settings.
Allow voice mode to work with supported API key STT providers in addition to Claude.ai OAuth, and align terminal truncation rendering with actual display behavior.
Add GitHub release and Homebrew tap automation, stabilize the packaged CLI entrypoint, and clean up duplicated build hooks in the publish flow.
Resolve shellcheck issues in the Homebrew sync workflow and isolate provider-related tests from global mocks and locale-dependent state.
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThis PR implements end-to-end release automation and distribution infrastructure via GitHub Actions workflows, introduces OpenAI-based STT support for voice mode alongside Anthropic OAuth, refactors tool result validation and terminal output handling, and updates related UI components and agent token tracking. Changes
Sequence DiagramssequenceDiagram
actor User
participant GitHub as GitHub Actions
participant Build as Build Process
participant Release as Release Asset
participant Homebrew as Homebrew Tap
User->>GitHub: Push tag v*
GitHub->>Build: Trigger release workflow
Build->>Build: Run tests
Build->>Build: Build artifacts (arm64, x86_64)
Build->>Build: Generate manifest.json + SHA256SUMS
Build->>Build: Create Homebrew formula
Build->>Release: Upload tarballs & metadata
Note over GitHub,Release: Release published
GitHub->>Homebrew: Trigger sync workflow
Homebrew->>Homebrew: Fetch manifest.json
Homebrew->>Homebrew: Validate checksums
Homebrew->>Homebrew: Create/update PR with formula
Homebrew->>Homebrew: Enable auto-merge
Homebrew->>GitHub: Complete (summary to GITHUB_STEP_SUMMARY)
sequenceDiagram
actor User
participant CLI as Voice Command
participant Provider as Provider Selector
participant STT as STT Service
participant OpenAI as OpenAI API
participant Anthropic as Anthropic OAuth
User->>CLI: Enable voice mode
CLI->>Provider: Check available providers
Provider->>Anthropic: Has OAuth token?
Provider->>OpenAI: Has API key?
alt Provider Available
Provider->>STT: Select provider
alt Provider is OpenAI
STT->>STT: Buffer PCM audio
STT->>STT: Convert to WAV
STT->>OpenAI: Upload & transcribe
OpenAI-->>STT: Transcript
STT-->>User: Final transcript
else Provider is Anthropic OAuth
STT->>Anthropic: Connect voice_stream
Anthropic-->>STT: Real-time transcription
STT-->>User: Final transcript
end
else No Provider
Provider-->>CLI: unavailable
CLI-->>User: Configure OAuth or API key
end
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
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.
Summary by CodeRabbit
Release Notes
New Features
brew install claude-code-best)Improvements
Documentation