rowboat/apps/x/packages
Ramnique Singh 054f807925 fix(x): record builtin error envelopes as isError in durable results
Builtins signal failure by returning an envelope ({error}, {success:
false, error|message}, composio's {successful: false}) instead of
throwing, and the tool-registry bridge marked every non-throwing result
isError: false — so the durable log recorded most builtin failures as
successes, misleading the reducer's record, the renderer's error state,
telemetry, and the tools_extended gate. The spec's ToolResultData.isError
channel was effectively populated only by throws.

The bridge now detects the envelope at its single chokepoint and sets
isError accordingly. Model-facing bytes are unchanged (encoding ignores
isError). Deliberately not an error: executeCommand's success:false for
a plain non-zero exit with no error/message — a failing grep is a
result, not a tool failure. Composio success passthrough (error: null)
stays non-error via the truthy-string check. The authoring convention is
documented on the catalog schema.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-07-11 12:31:47 +05:30
..
core fix(x): record builtin error envelopes as isError in durable results 2026-07-11 12:31:47 +05:30
shared fix(x): fail-closed tool permissions with per-tool catalog declarations 2026-07-10 23:07:11 +05:30