Skip to content

fix(ai): give the get_agent_tool_error payload fixture its cutAttributeBytes - #1172

Merged
JeremyFunk merged 1 commit into
mainfrom
fix/agent-tools-payload-mock
Sep 30, 2026
Merged

fix(ai): give the get_agent_tool_error payload fixture its cutAttributeBytes#1172
JeremyFunk merged 1 commit into
mainfrom
fix/agent-tools-payload-mock

fix(ai): give the get_agent_tool_error payload fixture its cutAttribu…

cbfad3e
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review succeeded Sep 30, 2026 in 18s

Confidence 5/5 · No issues found

🟢 Confidence 5/5 · safe to merge
quality 100/100 · no findings · tests not needed · risk low

Adds the now-required cutAttributeBytes: {} to the payload() fixture in agent-tools.test.ts, restoring the @maple/ai suite after aiToolErrorPayloadsRowSchema gained that column. Test-only, safe to merge.

  • payload() fixture in agent-tools.test.ts now carries cutAttributeBytes: {}
What was checked
  • aiToolErrorPayloadsRowSchema requires cutAttributeBytes: Schema.Record(String, CHNumber) (packages/query-engine-integrations/src/ai/ai-tools.ts:964), so {} decodes
  • aiToolErrorPayload falls back to utf8 length when no key matches (ai-tools.ts:1064), so the 12,000 bytes total and argumentsBytes assertions still hold
  • Fixture rows feed the fake warehouse only; no production code path is touched

cbfad3e · Updated on every push. Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.

View more details on Maple Review Bot