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

Merged

anandgupta42 merged 29 commits intomainfrom

fix/ai-5975-tool-error-propagation

Mar 25, 2026

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
fix: [AI-5975] propagate tool error messages to telemetry across all 25 tools

suryaiyer95
and
claude
committed
fix: [AI-5975] add missing `dialect` fields to test args for type-check

suryaiyer95
and
claude
committed
fix: [AI-5975] remove spurious `dialect` from grade test (no such param)

suryaiyer95
and
claude
committed
Merge remote-tracking branch 'origin/main' into fix/ai-5975-tool-error-propagation
suryaiyer95
committed
fix: [AI-5975] update test expectation to match error title format after merge

suryaiyer95
and
claude
committed
feat: [AI-5975] add sql_quality telemetry for issue prevention metrics (#446)

suryaiyer95
and
claude
authored
Merge branch 'main' into fix/ai-5975-tool-error-propagation
anandgupta42
authored

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

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

fix: [AI-5975] propagate tool error messages to telemetry#429
anandgupta42 merged 29 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"

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

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

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

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

fix: [AI-5975] address PR review feedback

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

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

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

fix: [AI-5975] address code review findings for tool wrappers

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

fix: [AI-5975] make error propagation tests self-contained with mocks

fix: [AI-5975] propagate tool error messages to telemetry across all 25 tools

fix: [AI-5975] add missing `dialect` fields to test args for type-check

fix: [AI-5975] remove spurious `dialect` from grade test (no such param)

Merge remote-tracking branch 'origin/main' into fix/ai-5975-tool-error-propagation

fix: [AI-5975] update test expectation to match error title format after merge

feat: [AI-5975] add sql_quality telemetry for issue prevention metrics (#446)

Merge branch 'main' into fix/ai-5975-tool-error-propagation

Commits on Mar 25, 2026

Merge branch 'main' into fix/ai-5975-tool-error-propagation

Merge branch 'main' into fix/ai-5975-tool-error-propagation

fix: add missing `altimate_change` markers and prettier formatting in `tool.ts`

style: prettier formatting on all changed tool files

fix: preserve original formatting of `errorMsg` ternary to avoid marker guard false positive

fix: restore original `errorMsg` indentation to match `main`

fix: correct indentation of `altimate_change end` marker in `tool.ts`

fix: propagate `metadata.error` in `sql-optimize` and `sql-translate`, compute `has_schema` in `sql-analyze`

fix: remove unused `@ts-expect-error` directives now that `define()` generic is simplified

Merge remote-tracking branch 'origin/main' into fix/ai-5975-tool-error-propagation