Commit 780b5a4
fix(runtime): bump Claude Agent SDK to 0.3.220, guard the launched binary
The version guard validated the global `claude` on PATH while the Agent SDK
launched its own bundled (2.1.90) binary — a false-positive compatibility
check (PR #162 review, P1): the guard passed on 2.1.220 while query() ran
2.1.90 and still crashed on the empty attribution strings.
Resolve via the reviewer's option 1: bump @anthropic-ai/claude-agent-sdk
0.2.90 -> 0.3.220, which bundles Claude Code 2.1.220 (>= 2.1.191) through
per-platform native binaries, and rewrite the guard so it inspects the exact
binary query() will launch:
- explicit pathToClaudeCodeExecutable configured -> spawn `<path> --version`
(buildClaudeQueryOptions forwards the same path to query());
- otherwise -> read the version of the SDK's bundled binary from its
manifest.json (readBundledClaudeVersion), resolved via the SDK main entry
since the package exports restrict the manifest subpath. No spawn, no PATH
lookup, no platform/musl ambiguity.
Removes the misleading discoveredExecutablePath (PATH-CLI) fallback that
produced the false positive. Also routes model discovery through the same
guard (previously bypassed). The empty-attribution suppression contract and
unrelated-settings pass-through from prior iterations are unchanged.
Verification: runtime 899 passed (1 integration skipped), agent 376, shared
206; typecheck/format/lint/build green; runtime coverage 85.69%
(version.ts 88.88%). Bundled CC confirmed at 2.1.220 via the manifest.
Co-Authored-By: Claude <noreply@anthropic.com>1 parent e4c61eb commit 780b5a4
14 files changed
Lines changed: 537 additions & 420 deletions
File tree
- .docker
- docs
- packages
- api
- runtime
- src
- __tests__
- adapters/claude
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
64 | 64 | | |
65 | 65 | | |
66 | 66 | | |
67 | | - | |
68 | | - | |
69 | | - | |
70 | | - | |
71 | | - | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
72 | 76 | | |
73 | 77 | | |
74 | 78 | | |
| |||
117 | 121 | | |
118 | 122 | | |
119 | 123 | | |
| 124 | + | |
| 125 | + | |
120 | 126 | | |
121 | 127 | | |
122 | 128 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
| 9 | + | |
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| |||
130 | 130 | | |
131 | 131 | | |
132 | 132 | | |
133 | | - | |
| 133 | + | |
134 | 134 | | |
135 | 135 | | |
136 | 136 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
197 | 197 | | |
198 | 198 | | |
199 | 199 | | |
200 | | - | |
| 200 | + | |
201 | 201 | | |
202 | | - | |
| 202 | + | |
| 203 | + | |
| 204 | + | |
203 | 205 | | |
204 | 206 | | |
205 | 207 | | |
| |||
0 commit comments