Skip to content

Commit 296c165

Browse files
Qardclaude
andcommitted
fix(anthropic): enable messages.stream() in auto-instrumentation e2e tests (#1592)
The root cause of #1592 was fixed by PR #1617: `patch-tracing-channel.ts` now preserves the original `APIPromise` object (with `.withResponse()`) instead of wrapping it in a plain `.then()` chain when the result is a Promise subclass. Remove the `useMessagesStreamHelper: false` workaround that was causing the auto-hook instrumentation path to avoid `client.messages.stream()`. The 140 anthropic e2e tests all pass with `messages.stream()` enabled in both the wrapper and auto-hook paths. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent c2d24c1 commit 296c165

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

e2e/scenarios/anthropic-instrumentation/scenario.impl.mjs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,6 @@ export async function runWrappedAnthropicInstrumentation(Anthropic, options) {
251251
export async function runAutoAnthropicInstrumentation(Anthropic, options) {
252252
await runAnthropicInstrumentationScenario(Anthropic, {
253253
...options,
254-
useMessagesStreamHelper: false,
255254
});
256255
}
257256

0 commit comments

Comments
 (0)