fix: use console.warn for usage warnings in CLI layer

- Change "No scenario files found" etc. from logger/console.error to console.warn
- Change "Unknown command" from logger.error to console.warn
- Remove redundant logger.error before throw in resolveReporter

These are normal operational warnings, not errors.

Author: lambdalisue

src/commands/run.ts

@@ -141,7 +141,7 @@ export async function runCommand(
     );
 
     if (scenarioFiles.length === 0) {
-      logger.error("No scenario files found", { paths, includes, excludes });
+      console.warn("No scenario files found");
       return EXIT_CODE.NOT_FOUND;
     }
 
@@ -161,7 +161,7 @@ export async function runCommand(
     logger.debug("Scenarios loaded", { scenarioCount: scenarios.length });
 
     if (scenarios.length === 0) {
-      logger.error("No scenarios found in files");
+      console.warn("No scenarios found in files");
       return EXIT_CODE.NOT_FOUND;
     }
 
@@ -173,7 +173,7 @@ export async function runCommand(
     }
 
     if (scenarios.length === 0) {
-      logger.error("No scenarios matched the filter", { selectors });
+      console.warn("No scenarios matched the filter");
       return EXIT_CODE.NOT_FOUND;
     }
 

src/main.ts

@@ -68,8 +68,8 @@ export async function main(args: string[]): Promise<number> {
       return await listCommand(commandArgs, cwd);
 
     default:
-      logger.error("Unknown command", { command });
-      logger.error("Run 'probitas --help' for usage information");
+      console.warn(`Unknown command: ${command}`);
+      console.warn("Run 'probitas --help' for usage information");
       return EXIT_CODE.USAGE_ERROR;
   }
 }

src/utils.ts

@@ -50,10 +50,6 @@ export function resolveReporter(
 
   const fn = reporterMap[reporter];
   if (!fn) {
-    logger.error("Unknown reporter", {
-      reporter,
-      availableReporters: Object.keys(reporterMap),
-    });
     throw new Error(`Unknown reporter: ${reporter}`);
   }