fix: [AI-5975] decouple metadata.success from domain outcomes in finding tools

Five tools were suppressing `sql_quality` telemetry because their
`metadata.success` tracked domain outcomes (SQL invalid, policy violated,
queries not equivalent) rather than engine execution success.

`tool.ts` gate: `!isSoftFailure && findings.length > 0`
- `isSoftFailure = metadata.success === false`
- Tools that found issues had `success: false` → findings suppressed

Fix: set `success: true` when the engine ran (even if it found problems).
Domain outcomes remain in dedicated fields (`valid`, `pass`, `equivalent`,
`fixed`). Only catch blocks set `success: false` (real engine crashes).

Affected tools:
- `altimate_core_validate` — validation errors now emit `sql_quality`
- `altimate_core_semantics` — semantic issues now emit `sql_quality`
- `altimate_core_policy` — policy violations now emit `sql_quality`
- `altimate_core_equivalence` — differences now emit `sql_quality`
- `altimate_core_fix` — unfixable errors now emit `sql_quality`

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

Author: suryaiyer95

packages/opencode/src/altimate/tools/altimate-core-policy.ts

@@ -31,7 +31,7 @@ export const AltimateCorePolicyTool = Tool.define("altimate_core_policy", {
       return {
         title: `Policy: ${data.pass ? "PASS" : "VIOLATIONS FOUND"}`,
         metadata: {
-          success: result.success,
+          success: true, // engine ran — violations are findings, not failures
           pass: data.pass,
           has_schema: hasSchema,
           dialect: "snowflake",

packages/opencode/src/altimate/tools/altimate-core-semantics.ts

@@ -35,7 +35,7 @@ export const AltimateCoreSemanticsTool = Tool.define("altimate_core_semantics",
       return {
         title: hasError ? "Semantics: ERROR" : `Semantics: ${data.valid ? "VALID" : `${issueCount} issues`}`,
         metadata: {
-          success: result.success,
+          success: true, // engine ran — semantic issues are findings, not failures
           valid: data.valid,
           issue_count: issueCount,
           has_schema: hasSchema,

packages/opencode/src/altimate/tools/altimate-core-validate.ts

@@ -34,7 +34,7 @@ export const AltimateCoreValidateTool = Tool.define("altimate_core_validate", {
       return {
         title: `Validate: ${data.valid ? "VALID" : "INVALID"}`,
         metadata: {
-          success: result.success,
+          success: true, // engine ran — validation errors are findings, not failures
           valid: data.valid,
           has_schema: hasSchema,
           dialect: "snowflake",