fix: allowlist CLI commands in no-console-log test

telemetry.ts is a CLI command that correctly uses console.log for
user-facing output, not MCP stdio. Added allowlist for commands/ files.

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

Author: 2 people

tests/unit/no-console-log.test.ts

@@ -28,6 +28,11 @@ function collectTsFiles(dir: string): string[] {
   return results;
 }
 
+// CLI commands run outside MCP stdio transport — console.log is correct there
+const CLI_COMMAND_ALLOWLIST = new Set([
+  "src/commands/telemetry.ts",
+]);
+
 describe("no console.log in src/", () => {
   const srcDir = join(__dirname, "../../src");
   const files = collectTsFiles(srcDir);
@@ -36,18 +41,21 @@ describe("no console.log in src/", () => {
     expect(files.length).toBeGreaterThan(0);
   });
 
-  it("should not contain console.log in any src/ file", () => {
+  it("should not contain console.log in any src/ file (except CLI commands)", () => {
     const violations: string[] = [];
 
     for (const file of files) {
+      const relative = file.replace(srcDir, "src");
+      // CLI commands use stdout directly — not MCP stdio
+      if (CLI_COMMAND_ALLOWLIST.has(relative)) continue;
+
       const content = readFileSync(file, "utf-8");
       const lines = content.split("\n");
       for (let i = 0; i < lines.length; i++) {
         const line = lines[i];
         // Skip comments
         if (line.trimStart().startsWith("//") || line.trimStart().startsWith("*")) continue;
         if (line.includes("console.log")) {
-          const relative = file.replace(srcDir, "src");
           violations.push(`${relative}:${i + 1}: ${line.trim()}`);
         }
       }