Automated Testing: Allow console logging in all bin, scripts, tools files

packages/babel-preset-default/bin/index.js

@@ -29,7 +29,6 @@ builder( {
 	)
 	.then( ( output ) => writeFile( './build/polyfill.min.js', output.code ) )
 	.catch( ( error ) => {
-		// eslint-disable-next-line no-console
 		console.log( error );
 		process.exit( 1 );
 	} );

packages/react-native-editor/bin/extract-files-from-sourcemap.js

@@ -29,26 +29,22 @@ if ( require.main === module ) {
 
 	// Validate arguments
 	if ( ! mapFile ) {
-		// eslint-disable-next-line no-console
 		console.error( `Map file argument is required.` );
 		process.exit( 1 );
 	}
 	if ( ! targetDir ) {
-		// eslint-disable-next-line no-console
 		console.error( `Target directory argument is required.` );
 		process.exit( 1 );
 	}
 	try {
 		fs.accessSync( mapFile );
 	} catch {
-		// eslint-disable-next-line no-console
 		console.error( `Map file "${ mapFile } doesn't exist.` );
 		process.exit( 1 );
 	}
 	try {
 		fs.accessSync( targetDir );
 	} catch {
-		// eslint-disable-next-line no-console
 		console.error( `Target directory "${ targetDir }"" doesn't exist.` );
 		process.exit( 1 );
 	}
@@ -88,14 +84,12 @@ if ( require.main === module ) {
 			try {
 				fs.accessSync( compiledFile );
 			} catch {
-				// eslint-disable-next-line no-console
 				console.warn(
 					`Couldn't find matching build file for Typescript file "${ compiledFile }".`
 				);
 				return file;
 			}
 
-			// eslint-disable-next-line no-console
 			console.log(
 				`Using compiled file "${ compiledFile }" for Typescript file "${ file }"`
 			);
@@ -110,7 +104,6 @@ if ( require.main === module ) {
 			copyFileToDir( sourceFile, targetDir )
 		);
 	} catch ( error ) {
-		// eslint-disable-next-line no-console
 		console.error(
 			`Something went wrong when copying source files to "${ targetDir }":`,
 			error

packages/react-native-editor/bin/extract-used-strings.js

@@ -143,7 +143,6 @@ const generatePotFiles = ( plugins, potFilesDir ) => {
 			}
 		);
 	} catch ( error ) {
-		// eslint-disable-next-line no-console
 		console.error( error );
 		process.exit( 1 );
 	}

packages/react-native-editor/bin/i18n-translations-download.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-/* eslint-disable no-console */
 
 /**
  * External dependencies
@@ -243,7 +242,7 @@ const getUsedStrings = ( usedStringsFile, domain ) => {
 const generateIndexFile = ( translations, pluginDir ) => {
 	const indexNative = `/* THIS IS A GENERATED FILE. DO NOT EDIT DIRECTLY. */
 /* eslint-disable prettier/prettier */
-	
+
 	const translations = {
 	${ translations
 		.filter( Boolean )
@@ -253,9 +252,9 @@ const generateIndexFile = ( translations, pluginDir ) => {
 		)
 		.join( '\n' ) }
 	};
-	
+
 	export const getTranslation = ( locale ) => translations[ locale ];
-	
+
 /* eslint-enable prettier/prettier */
 `;
 
@@ -312,5 +311,3 @@ if ( require.main === module ) {
 		generateIndexFile( translations, pluginDir );
 	} );
 }
-
-/* eslint-enable no-console */

packages/scripts/config/jest-environment-puppeteer/index.js

@@ -32,8 +32,7 @@
 	// https://github.com/facebook/jest/blob/1be8d737abd0e2f30e3314184a0efc372ad6d88f/packages/jest-jasmine2/src/jasmine/Env.ts#L250-L251
 	// Emitting an uncaughtException event to the process will throw an
 	// empty error which is very hard to debug in puppeteer context.
-	// eslint-disable-next-line no-console
 	console.error( error );
 };
 
 const KEYS = {
@@ -70,8 +69,7 @@
 					// eslint-disable-next-line no-debugger
 					debugger;
 				} );
-				// eslint-disable-next-line no-console
 				console.log(
 					chalk.blue(
 						'\n\n🕵️‍  Code is paused, press enter to resume'
 					)

packages/scripts/config/jest-github-actions-reporter/index.js

@@ -28,8 +28,7 @@
 		const messages = getMessages( _aggregatedResults.testResults );
 
 		for ( const message of messages ) {
-			// eslint-disable-next-line no-console
 			console.log( message );
 		}
 	}
 }

packages/scripts/scripts/lint-js.js

@@ -43,7 +43,6 @@ if ( ! hasLintConfig ) {
 	];
 	const hasLegacyConfig = legacyConfigFiles.some( hasProjectFile );
 	if ( hasLegacyConfig ) {
-		// eslint-disable-next-line no-console
 		console.warn(
 			'Warning: Legacy eslintrc configuration detected. ' +
 				'ESLint v10 no longer supports eslintrc files. ' +

packages/scripts/scripts/packages-update.js

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 /**
  * External dependencies
  */
@@ -82,4 +81,3 @@ function updatePackageJSON() {
 }
 
 updatePackageJSON();
-/* eslint-enable no-console */

packages/scripts/scripts/test-playwright.js

@@ -41,7 +41,6 @@ try {
 	// First, try to load the package installed from among the optional peerDependencies.
 	loadConfig = require( '@wordpress/env/lib/config' ).loadConfig;
 } catch {
-	// eslint-disable-next-line no-console
 	console.log(
 		'Notice: Could not find @wordpress/env package. Using WP_BASE_URL environment variable or else the default http://localhost:8889 URL for tests.'
 	);

packages/scripts/utils/cli.js

@@ -38,14 +38,12 @@ const hasFileArgInCLI = () => getFileArgsFromCLI().length > 0;
 
 const handleSignal = ( signal ) => {
 	if ( signal === 'SIGKILL' ) {
-		// eslint-disable-next-line no-console
 		console.log(
 			'The script failed because the process exited too early. ' +
 				'This probably means the system ran out of memory or someone called ' +
 				'`kill -9` on the process.'
 		);
 	} else if ( signal === 'SIGTERM' ) {
-		// eslint-disable-next-line no-console
 		console.log(
 			'The script failed because the process exited too early. ' +
 				'Someone might have called `kill` or `killall`, or the system could ' +
@@ -57,13 +55,11 @@ const handleSignal = ( signal ) => {
 
 const spawnScript = ( scriptName, args = [], nodeArgs = [] ) => {
 	if ( ! scriptName ) {
-		// eslint-disable-next-line no-console
 		console.log( 'Script name is missing.' );
 		exit( 1 );
 	}
 
 	if ( ! hasScriptFile( scriptName ) ) {
-		// eslint-disable-next-line no-console
 		console.log(
 			'Unknown script "' +
 				scriptName +

tools/api-docs/gen-components-docs/index.mjs

@@ -142,7 +142,6 @@ await Promise.all(
 		);
 
 		try {
-			// eslint-disable-next-line no-console
 			console.log( `Writing docs to ${ outputFile }` );
 			return fs.writeFile( outputFile, docs );
 		} catch ( e ) {

tools/api-docs/gen-theme-reference.mjs

@@ -290,7 +290,6 @@ async function main() {
 }
 
 main().catch( ( error ) => {
-	// eslint-disable-next-line no-console
 	console.error( error );
 	process.exit( 1 );
 } );

tools/api-docs/update-api-docs.js

@@ -246,7 +246,6 @@ glob.stream( [
 				cwd: ROOT_DIR,
 			} );
 		} catch ( error ) {
-			// eslint-disable-next-line no-console
 			console.error( error );
 			process.exit( 1 );
 		}

tools/eslint/config.mjs

@@ -564,15 +564,7 @@ export default dedupePlugins( [
 
 	// Override: CLI/bin/env files — allow console.
 	{
-		files: [
-			'bin/**/*.js',
-			'bin/**/*.mjs',
-			'tools/**/*.js',
-			'tools/**/*.mjs',
-			'tools/**/*.cjs',
-			'packages/env/**',
-			'packages/theme/bin/**/*.[tj]s?(x)',
-		],
+		files: [ '**/{bin,scripts,tools}/**', 'packages/env/**' ],
 		rules: {
 			'no-console': 'off',
 		},