fix(scan github): don't print success counts when returning blocking error

Before: on rate-limit / auth abort, the function emitted
"N GitHub repos processed / 0 with supported Manifest files" followed
by logger.fail(...). The success lines implied partial progress even
though we're about to return ok:false, which is confusing output.
Move the success counters past the blocking-error return so they only
print on actual completion.

Author: jdalton

packages/cli/src/commands/scan/create-scan-from-github.mts

@@ -148,14 +148,14 @@ export async function createScanFromGithub({
     }
   }
 
-  logger.success(reposScanned, 'GitHub repos processed')
-  logger.success(scansCreated, 'with supported Manifest files')
-
   if (blockingError) {
     logger.fail(blockingError.message)
     return blockingError
   }
 
+  logger.success(reposScanned, 'GitHub repos processed')
+  logger.success(scansCreated, 'with supported Manifest files')
+
   // If every repo failed but not for a known-blocking reason, treat
   // the run as an error so scripts know something went wrong instead
   // of inferring success from an ok: true with 0 scans.