@automattic/i18n-check-webpack-plugin 1.0.1 → 1.0.5

package/.eslintignore

@@ -0,0 +1,3 @@
+# Not loaded by default eslint, but we use tools/js-tools/load-eslint-ignore.js in .eslintrc to munge it in.
+
+tests/fixtures/

package/CHANGELOG.md

@@ -5,6 +5,22 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.0.5] - 2022-02-16
+### Added
+- Add timing info to debug output.
+
+## [1.0.4] - 2022-01-27
+### Changed
+- Updated package dependencies.
+
+## [1.0.3] - 2022-01-25
+### Changed
+- Updated package dependencies.
+
+## [1.0.2] - 2022-01-18
+### Changed
+- General: update required node version to v16.13.2
+
 ## [1.0.1] - 2022-01-04
 ### Changed
 - Updated package dependencies
@@ -22,5 +38,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Added
 - Initial release.
 
+[1.0.5]: https://github.com/Automattic/i18n-check-webpack-plugin/compare/v1.0.4...v1.0.5
+[1.0.4]: https://github.com/Automattic/i18n-check-webpack-plugin/compare/v1.0.3...v1.0.4
+[1.0.3]: https://github.com/Automattic/i18n-check-webpack-plugin/compare/v1.0.2...v1.0.3
+[1.0.2]: https://github.com/Automattic/i18n-check-webpack-plugin/compare/v1.0.1...v1.0.2
 [1.0.1]: https://github.com/Automattic/i18n-check-webpack-plugin/compare/v1.0.0...v1.0.1
 [1.0.0]: https://github.com/Automattic/i18n-check-webpack-plugin/compare/v0.1.0...v1.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@automattic/i18n-check-webpack-plugin",
-	"version": "1.0.1",
+	"version": "1.0.5",
 	"description": "A Webpack plugin to check that WordPress i18n hasn't been mangled by Webpack optimizations.",
 	"homepage": "https://jetpack.com",
 	"bugs": {

package/src/I18nCheckPlugin.js

@@ -7,6 +7,7 @@ const GettextEntries = require( './GettextEntries.js' );
 const GettextExtractor = require( './GettextExtractor.js' );
 
 const debug = require( 'debug' )( PLUGIN_NAME + ':plugin' );
+const debugtiming = require( 'debug' )( PLUGIN_NAME + ':timing' );
 
 const schema = {
 	title: `${ PLUGIN_NAME } plugin options`,
@@ -193,11 +194,14 @@ class I18nCheckPlugin {
 					additionalAssets: true,
 				},
 				assets => {
+					const t0 = Date.now();
 					const promises = [];
 					for ( const filename of Object.keys( assets ) ) {
 						promises.push( this.#processAsset( compilation, filename, moduleCache ) );
 					}
-					return Promise.all( promises );
+					return Promise.all( promises ).finally( () => {
+						debugtiming( `Processed all assets in ${ Date.now() - t0 }ms` );
+					} );
 				}
 			);
 		} );
@@ -211,6 +215,7 @@ class I18nCheckPlugin {
 	 * @param {Map} moduleCache - Cache for processed modules.
 	 */
 	async #processAsset( compilation, filename, moduleCache ) {
+		const t0 = Date.now();
 		const asset = compilation.getAsset( filename );
 
 		// Detemine if we even need to process this asset. JavaScript assets seem to always
@@ -226,28 +231,36 @@ class I18nCheckPlugin {
 
 		// Extract strings from the source resources.
 		const sourceEntries = new Set();
+		const promises = [];
 		for ( const resource of asset.info.resources ) {
 			if ( ! moduleCache.has( resource ) ) {
-				const promise = new Promise( resolve => {
-					this.#extractor
-						.extractFromFile( resource, {
-							filename: npath.relative( compilation.compiler.context, resource ),
-						} )
-						.then( resolve );
+				const promise = this.#extractor.extractFromFile( resource, {
+					filename: npath.relative( compilation.compiler.context, resource ),
 				} );
 				moduleCache.set( resource, promise );
 			}
-			const resourceEntries = await moduleCache.get( resource );
-			resourceEntries.forEach( e => sourceEntries.add( e ) );
+			promises.push(
+				moduleCache.get( resource ).then( resourceEntries => {
+					resourceEntries.forEach( e => sourceEntries.add( e ) );
+				} )
+			);
 		}
 
 		// Extract strings from the asset.
 		const lintLogger = s => {
 			compilation[ this.#reportkey ].push( new Error( s ) );
 		};
-		const source = asset.source.source();
-		const babelFile = await this.#extractor.parse( source, { filename, lintLogger } );
-		const assetEntries = this.#extractor.extractFromAst( babelFile, { filename, lintLogger } );
+		let babelFile, assetEntries;
+		promises.push(
+			( async () => {
+				const source = asset.source.source();
+				babelFile = await this.#extractor.parse( source, { filename, lintLogger } );
+				assetEntries = this.#extractor.extractFromAst( babelFile, { filename, lintLogger } );
+			} )()
+		);
+
+		// Wait for all the extractions to complete.
+		await Promise.all( promises );
 
 		// Analyze. First, collect the missing entries and report any entries with lost translator comments.
 		const missingEntries = new GettextEntries();
@@ -348,6 +361,7 @@ class I18nCheckPlugin {
 				)
 			);
 		}
+		debugtiming( `Processed asset ${ filename } in ${ Date.now() - t0 }ms` );
 	}
 }