feat: add \sql_execute_failure\ telemetry for SQL execution errors

\`core_failure\` is for internal tool failures. SQL execution via the
dispatcher is a separate concern — soft errors are returned as results
(not thrown), so \`core_failure\` never fires for them.

New \`sql_execute_failure\` event captures: warehouse type, query type,
error message (truncated to 500 chars), and PII-masked SQL. Fires from
the \`sql.execute\` handler catch path.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: suryaiyer95

docs/docs/configure/telemetry.md

@@ -34,6 +34,7 @@ We collect the following categories of events:
 | `mcp_server_census` | MCP server capabilities after connect (tool and resource counts — no tool names) |
 | `context_overflow_recovered` | Context overflow is handled (strategy) |
 | `skill_used` | A skill is loaded (skill name and source — `builtin`, `global`, or `project` — no skill content) |
+| `sql_execute_failure` | A SQL execution fails (warehouse type, query type, error message, PII-masked SQL — no raw values) |
 | `core_failure` | A tool failure occurs — error category, error message (truncated to 500 chars), and PII-masked arguments (string literals in SQL replaced with `?`, sensitive keys like `password`/`token`/`secret` fully redacted) |
 
 Each event includes a timestamp, anonymous session ID, and the CLI version.

packages/opencode/src/altimate/native/connections/register.ts

@@ -228,6 +228,7 @@ register("sql.execute", async (params: SqlExecuteParams): Promise<SqlExecuteResu
     } catch {}
     return result
   } catch (e) {
+    const errorMsg = String(e)
     try {
       Telemetry.track({
         type: "warehouse_query",
@@ -239,11 +240,21 @@ register("sql.execute", async (params: SqlExecuteParams): Promise<SqlExecuteResu
         duration_ms: Date.now() - startTime,
         row_count: 0,
         truncated: false,
-        error: String(e).slice(0, 500),
+        error: errorMsg.slice(0, 500),
         error_category: categorizeQueryError(e),
       })
+      Telemetry.track({
+        type: "sql_execute_failure",
+        timestamp: Date.now(),
+        session_id: Telemetry.getContext().sessionId,
+        warehouse_type: warehouseType,
+        query_type: detectQueryType(params.sql),
+        error_message: errorMsg.slice(0, 500),
+        masked_sql: Telemetry.maskArgs({ sql: params.sql }),
+        duration_ms: Date.now() - startTime,
+      })
     } catch {}
-    return { columns: [], rows: [], row_count: 0, truncated: false, error: String(e) } as SqlExecuteResult & { error: string }
+    return { columns: [], rows: [], row_count: 0, truncated: false, error: errorMsg } as SqlExecuteResult & { error: string }
   }
 })
 

packages/opencode/src/altimate/telemetry/index.ts

@@ -341,6 +341,16 @@ export namespace Telemetry {
         skill_source: "builtin" | "global" | "project"
         duration_ms: number
       }
+    | {
+        type: "sql_execute_failure"
+        timestamp: number
+        session_id: string
+        warehouse_type: string
+        query_type: string
+        error_message: string
+        masked_sql: string
+        duration_ms: number
+      }
     | {
         type: "core_failure"
         timestamp: number