fix(api): give the error-samples payload mock its cutAttributeBytes - #1171
Merged
Merged
Maple Review Bot / Maple / review
succeeded
Sep 30, 2026 in 34s
Confidence 5/5 · No issues found
🟢 Confidence 5/5 · safe to merge
Test-only fixture fix; the schema requirement and the fallback byte counts both check out against the assertions in the same test.
quality 100/100 · no findings · tests covered · risk low
Adds the missing cutAttributeBytes field to the error-samples payload mock in ai-sessions.http.test.ts, restoring the row decode that the new schema requires. One-line test-fixture fix, safe to merge.
ai-sessions.http.test.tspayload mock row now carriescutAttributeBytes: {}
What was checked
aiToolErrorPayloadsRowSchemarequirescutAttributeBytes(query-engine-integrations/src/ai/ai-tools.ts:964), so the mock previously failed decode- With
{},aiToolErrorPayloadfalls back toutf8.encode(text).length, which matches the assertedargumentsBytes: 2andresultBytesat test lines 1710-1712 - Other tests in the file decode no payload rows (only occurrences rows or
[]), so no second mock is missing the field
6eecaf0 · Updated on every push. Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.
Loading