fix: [AI-5975] propagate tool error messages to telemetry#429

Open

suryaiyer95 wants to merge 12 commits intomainfrom

fix/ai-5975-tool-error-propagation

Commits on Mar 24, 2026

fix: [AI-5975] propagate actual error messages to telemetry instead of "unknown error"

suryaiyer95
and
claude
committed
fix: [AI-5975] early-return "No schema provided" error for `validate`, `semantics`, `equivalence`

suryaiyer95
and
claude
committed
fix: [AI-5975] add `.filter(Boolean)` to all error extractors to prevent empty string errors

suryaiyer95
and
claude
committed
merge: resolve conflict in `sql-analyze.ts` — take main's spread pattern for error

suryaiyer95
and
claude
committed
fix: [AI-5975] treat "not equivalent" as valid result in `altimate_core_equivalence`

suryaiyer95
and
claude
committed
fix: [AI-5975] address PR review feedback

suryaiyer95
and
claude
committed
fix: [AI-5975] use error-driven title in semantics, generic error label in analyze

suryaiyer95
and
claude
committed
fix: [AI-5975] ensure `metadata.error` is never `undefined` on success paths

suryaiyer95
and
claude
committed
fix: [AI-5975] dispatcher wrappers: `success` means "handler completed", not "result was positive"

suryaiyer95
and
claude
committed
fix: [AI-5975] address code review findings for tool wrappers

suryaiyer95
and
claude
committed
fix: [AI-5975] revert validate tool to use dispatcher's `result.success` flag

suryaiyer95
and
claude
committed
fix: [AI-5975] make error propagation tests self-contained with mocks

suryaiyer95
and
claude
committed