Fix CI still running full scan

Author: Alexis Grenier

packages/react-doctor/src/cli.ts

@@ -156,8 +156,8 @@ const program = new Command()
       const resolvedDirectory = path.resolve(directory);
       const userConfig = loadConfig(resolvedDirectory);
 
-      if (!isScoreOnly) {
-        logger.log(`react-doctor v${VERSION}`);
+      if (!isScoreOnly && !flags.hideBranding) {
+        logger.log(\`react-doctor v\${VERSION}\`);
         logger.break();
       }
 
@@ -181,7 +181,7 @@ const program = new Command()
         isScoreOnly,
       );
 
-      if (isDiffMode && diffInfo && !isScoreOnly) {
+      if (isDiffMode && diffInfo && !isScoreOnly && !flags.hideBranding) {
         if (diffInfo.isCurrentChanges) {
           logger.log("Scanning uncommitted changes");
         } else {
@@ -201,8 +201,8 @@ const program = new Command()
           if (projectDiffInfo) {
             const changedSourceFiles = filterSourceFiles(projectDiffInfo.changedFiles);
             if (changedSourceFiles.length === 0) {
-              if (!isScoreOnly) {
-                logger.dim(`No changed source files in ${projectDirectory}, skipping.`);
+              if (!isScoreOnly && !flags.hideBranding) {
+                logger.dim(\`No changed source files in \${projectDirectory}, skipping.\`);
                 logger.break();
               }
               continue;
@@ -211,13 +211,13 @@ const program = new Command()
           }
         }
 
-        if (!isScoreOnly) {
-          logger.dim(`Scanning ${projectDirectory}...`);
+        if (!isScoreOnly && !flags.hideBranding) {
+          logger.dim(\`Scanning \${projectDirectory}...\`);
           logger.break();
         }
         const scanResult = await scan(projectDirectory, { ...scanOptions, includePaths });
         allDiagnostics.push(...scanResult.diagnostics);
-        if (!isScoreOnly) {
+        if (!isScoreOnly && !flags.hideBranding) {
           logger.break();
         }
       }

packages/react-doctor/src/scan.ts

@@ -587,6 +587,10 @@ export const scan = async (
   }
 
   if (diagnostics.length === 0) {
+    if (options.noBranding) {
+      logger.log(buildNoBrandingReport(diagnostics, scoreResult));
+      return { diagnostics, scoreResult, skippedChecks };
+    }
     if (hasSkippedChecks) {
       const skippedLabel = skippedChecks.join(" and ");
       logger.warn(
@@ -610,7 +614,7 @@ export const scan = async (
     return { diagnostics, scoreResult, skippedChecks };
   }
 
-  printDiagnostics(diagnostics, options.verbose);
+  if (!options.noBranding) printDiagnostics(diagnostics, options.verbose);
 
   const displayedSourceFileCount = isDiffMode ? includePaths.length : projectInfo.sourceFileCount;
 
@@ -625,7 +629,7 @@ export const scan = async (
     options.noBranding,
   );
 
-  if (hasSkippedChecks) {
+  if (hasSkippedChecks && !options.noBranding) {
     const skippedLabel = skippedChecks.join(" and ");
     logger.break();
     logger.warn(`  Note: ${skippedLabel} checks failed — score may be incomplete.`);