Skip to content

Commit 41c218f

Browse files
harjothkharaclaude
andcommitted
fix(hermes): drop delta from reasoning extraction chain
Addresses Greptile review: `delta` is the field that carries streamed assistant text in `message.delta` events, so keeping it in the `reasoning.available` extraction chain risked silently treating a non-reasoning `delta` payload as reasoning text. The remaining six field names are semantically specific; ambiguous payloads now fall back to the "Hermes reasoning available" placeholder, matching the stated safe-fallback contract. Kept `.ts` and `.cjs` mirror in sync. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 81f94b7 commit 41c218f

2 files changed

Lines changed: 2 additions & 4 deletions

File tree

packages/adapters/hermes/gateway-ui-parser.cjs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,7 @@ function extractDirectReasoningText(data) {
3535
asString(data.thinking).trim() ||
3636
asString(data.text).trim() ||
3737
asString(data.summary).trim() ||
38-
asString(data.content).trim() ||
39-
asString(data.delta).trim()
38+
asString(data.content).trim()
4039
);
4140
}
4241

packages/adapters/hermes/src/gateway/ui/parse-stdout.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,7 @@ function extractDirectReasoningText(data: Record<string, unknown>): string {
4141
asString(data.thinking).trim() ||
4242
asString(data.text).trim() ||
4343
asString(data.summary).trim() ||
44-
asString(data.content).trim() ||
45-
asString(data.delta).trim()
44+
asString(data.content).trim()
4645
);
4746
}
4847

0 commit comments

Comments
 (0)