chore: minor fix

Author: Chrilleweb

src/commands/scanUsage.ts

@@ -43,7 +43,9 @@ export async function scanUsage(opts: ScanUsageOptions): Promise<ExitResult> {
   let scanResult = await scanCodebase(opts);
 
   // Filter out commented usages
-  scanResult.used = skipCommentedUsages(scanResult.used);
+  const filteredUsages = skipCommentedUsages(scanResult.used);
+  scanResult.used = filteredUsages;
+  scanResult.logged = filteredUsages.filter((usage) => usage.isLogged);
 
   // Measure duration
   const endTime = performance.now();

test/unit/commands/scanUsage.test.ts

@@ -389,6 +389,47 @@ it('skips prompt when type is none and json is true', async () => {
     );
   });
 
+  it('removes logged warning for block-commented console env usage', async () => {
+    vi.mocked(scanCodebase).mockResolvedValue({
+      ...baseScanResult,
+      used: [
+        {
+          variable: 'TEST_KEY',
+          file: 'src/index.ts',
+          line: 4,
+          column: 1,
+          pattern: 'process.env',
+          context: '/* console.log(process.env.TEST_KEY); */',
+          isLogged: true,
+        },
+      ],
+      logged: [
+        {
+          variable: 'TEST_KEY',
+          file: 'src/index.ts',
+          line: 4,
+          column: 1,
+          pattern: 'process.env',
+          context: '/* console.log(process.env.TEST_KEY); */',
+          isLogged: true,
+        },
+      ],
+    });
+    vi.mocked(determineComparisonFile).mockResolvedValue({ type: 'none' });
+
+    await scanUsage({ ...baseOpts, isCiMode: true });
+
+    expect(printScanResult).toHaveBeenCalledWith(
+      expect.objectContaining({
+        logged: [],
+        used: [],
+      }),
+      expect.anything(),
+      expect.anything(),
+      expect.anything(),
+    );
+  });
+
   it('returns exitWithError true in JSON strict mode for each warning type', async () => {
     const cases: Partial<ScanResult>[] = [
       { duplicates: { env: [{ key: 'A', count: 2 }] } },