fix(cli): set exitCode=1 when schema check fails with no permission issues

Previously, when handleSchemaIssues threw and no permission issues were
detected, totalFound was 0 and the command reported 'No issues found'
despite the schema check failure. Now anyFailed is checked alongside
totalFound to ensure failures are always surfaced with a non-zero exit.

Author: BYK

src/commands/cli/fix.ts

@@ -342,20 +342,26 @@ export const fixCommand = buildCommand({
     }
 
     const totalFound = perm.found + schema.found;
+    const anyFailed = perm.repairFailed || schema.repairFailed;
 
-    if (totalFound === 0) {
+    if (totalFound === 0 && !anyFailed) {
       stdout.write(
         "No issues found. Database schema and permissions are correct.\n"
       );
       return;
     }
 
     if (dryRun) {
-      stdout.write("Run 'sentry cli fix' to apply fixes.\n");
+      if (totalFound > 0) {
+        stdout.write("Run 'sentry cli fix' to apply fixes.\n");
+      }
+      if (anyFailed) {
+        proc.exitCode = 1;
+      }
       return;
     }
 
-    if (perm.repairFailed || schema.repairFailed) {
+    if (anyFailed) {
       proc.exitCode = 1;
     } else {
       stdout.write("All issues repaired successfully.\n");