ci(lint): enable no-console rule (#16053)

Enable `no-console` rule in our linting setup. Add `oxlint-disable` comments where we do want to log (in scripts).

Author: overlookmotel

.github/scripts/check-conformance-changes.js

@@ -1,5 +1,7 @@
 #!/usr/bin/env node
 
+// oxlint-disable no-console
+
 /**
  * Check if conformance tests should run based on changed files.
  * Uses cargo tree to determine dependencies of oxc_coverage crate.

.github/scripts/generate-benchmark-matrix.js

@@ -1,5 +1,7 @@
 #!/usr/bin/env node
 
+// oxlint-disable no-console
+
 /**
  * Generate a dynamic matrix for benchmark jobs based on affected components.
  * This script determines which benchmark components need to run based on changed files.

.github/scripts/get-changed-files.js

@@ -1,5 +1,7 @@
 #!/usr/bin/env node
 
+// oxlint-disable no-console
+
 /**
  * Get changed files from GitHub events (pull request or push).
  * This module provides a reusable function for detecting changed files.

.github/scripts/utils.js

@@ -2,6 +2,8 @@
  * Common utilities for GitHub Actions scripts
  */
 
+// oxlint-disable no-console
+
 const { execSync } = require('child_process');
 
 /**

apps/oxfmt/scripts/build.js

@@ -1,3 +1,5 @@
+// oxlint-disable no-console
+
 import { execSync } from 'node:child_process';
 import { copyFileSync, mkdirSync, readdirSync, readFileSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';

apps/oxlint/scripts/build.ts

@@ -1,3 +1,5 @@
+// oxlint-disable no-console
+
 import { execSync } from 'node:child_process';
 import { copyFileSync, readdirSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';

crates/oxc_traverse/scripts/build.mjs

@@ -44,6 +44,7 @@ await Promise.all([
 async function writeToFile(filename, code) {
   code = `${PREAMBLE}${code}`;
   const path = pathJoin(outputDirPath, filename);
+  // oxlint-disable-next-line no-console
   console.log('Writing:', path);
   await writeFile(path, code);
   await execAsync(`rustfmt ${JSON.stringify(path)}`);

napi/parser/example.js

@@ -25,5 +25,5 @@ const file = args.positionals[0] ?? 'test.js';
 
 const code = fs.readFileSync(file, 'utf-8');
 const result = parseSync(file, code, args.values);
-// oxlint-disable-next-line typescript-eslint/no-misused-spread
+// oxlint-disable-next-line no-console, typescript-eslint/no-misused-spread
 console.dir({ ...result }, { depth: Infinity });

napi/parser/scripts/visitor-keys.js

@@ -1,4 +1,5 @@
 import { visitorKeys } from '@typescript-eslint/visitor-keys';
 
 const keys = Object.entries(visitorKeys).map(([name, keys]) => ({ name, keys }));
+// oxlint-disable-next-line no-console
 console.log(JSON.stringify(keys));

npm/oxfmt/scripts/generate-packages.js

@@ -1,5 +1,7 @@
 // Code copied from [Rome](https://github.com/rome/tools/blob/lsp/v0.28.0/npm/rome/scripts/generate-packages.mjs)
 
+// oxlint-disable no-console
+
 import * as fs from 'node:fs';
 import { resolve } from 'node:path';
 import { fileURLToPath } from 'node:url';