quackage 1.0.61 → 1.0.63

package/gulp/Quackage-Gulpfile.js

@@ -70,7 +70,13 @@ console.log(`   > Building to [${_CONFIG.LibraryUniminifiedFileName}] and [${_CO
 console.log(`--> Gulp is taking over!`);
 
 const libBrowserify = require('browserify');
-const libGulp = require('gulp');
+
+// Resolve gulp from the consuming project's CWD so that the task
+// registry is shared with the gulp CLI (which also resolves from CWD).
+// Without this, `file:` dev-dependency references to quackage cause
+// two separate gulp instances — tasks registered here become invisible
+// to the CLI, resulting in "Task never defined: default".
+const libGulp = require(require.resolve('gulp', { paths: [process.cwd()] }));
 
 const libVinylSourceStream = require('vinyl-source-stream');
 const libVinylBuffer = require('vinyl-buffer');
@@ -78,28 +84,74 @@ const libVinylBuffer = require('vinyl-buffer');
 const libSourcemaps = require('gulp-sourcemaps');
 const libBabel = require('gulp-babel');
 const libTerser = require('gulp-terser');
+const { Transform } = require('stream');
+
+/**
+ * Create a configured browserify instance with common settings.
+ *
+ * Applies:
+ *   - BrowserifyIgnore entries from config
+ *   - Global transform to rewrite `require('node:xxx')` → `require('xxx')`
+ *     (needed for packages like find-my-way v9 that use Node.js prefixed builtins)
+ *
+ * @param {object} pConfig - The gulpfile config object
+ * @returns {object} Configured browserify instance
+ */
+function createBrowserifyInstance(pConfig)
+{
+	var tmpBrowserify = libBrowserify(
+		{
+			entries: pConfig.EntrypointInputSourceFile,
+			standalone: pConfig.LibraryObjectName,
+			debug: true
+		});
+
+	// Ignore modules that should not be bundled (e.g. WASM loaders loaded via <script> tag)
+	if (Array.isArray(pConfig.BrowserifyIgnore))
+	{
+		for (var i = 0; i < pConfig.BrowserifyIgnore.length; i++)
+		{
+			tmpBrowserify.ignore(pConfig.BrowserifyIgnore[i]);
+		}
+	}
+
+	// Global transform: rewrite require('node:xxx') → require('xxx')
+	// Some npm packages (e.g. find-my-way v9) use the `node:` prefix for
+	// built-in modules, which browserify cannot resolve. This transform
+	// strips the prefix so browserify resolves the standard module name.
+	tmpBrowserify.transform(
+		function(pFile)
+		{
+			var tmpChunks = [];
+			return new Transform(
+				{
+					transform: function(pChunk, pEncoding, fCallback)
+					{
+						tmpChunks.push(pChunk);
+						fCallback();
+					},
+					flush: function(fCallback)
+					{
+						var tmpSource = Buffer.concat(tmpChunks).toString('utf8');
+						tmpSource = tmpSource.replace(
+							/require\(\s*['"]node:([^'"]+)['"]\s*\)/g,
+							"require('$1')"
+						);
+						this.push(tmpSource);
+						fCallback();
+					}
+				});
+		},
+		{ global: true }
+	);
+
+	return tmpBrowserify;
+}
 
 // Build the module for the browser
 libGulp.task('minified',
 	() => {
-		// set up the custom browserify instance for this task
-		var tmpBrowserify = libBrowserify(
-			{
-				entries: _CONFIG.EntrypointInputSourceFile,
-				standalone: _CONFIG.LibraryObjectName,
-				debug: true
-			});
-
-		// Ignore modules that should not be bundled (e.g. WASM loaders loaded via <script> tag)
-		if (Array.isArray(_CONFIG.BrowserifyIgnore))
-		{
-			for (var i = 0; i < _CONFIG.BrowserifyIgnore.length; i++)
-			{
-				tmpBrowserify.ignore(_CONFIG.BrowserifyIgnore[i]);
-			}
-		}
-
-		return tmpBrowserify.bundle()
+		return createBrowserifyInstance(_CONFIG).bundle()
 			.pipe(libVinylSourceStream(_CONFIG.LibraryMinifiedFileName))
 			.pipe(libVinylBuffer())
 			.pipe(libSourcemaps.init({loadMaps: true}))
@@ -114,24 +166,7 @@ libGulp.task('minified',
 // Build the module for the browser
 libGulp.task('debug',
 	() => {
-		// set up the custom browserify instance for this task
-		var tmpBrowserify = libBrowserify(
-			{
-				entries: _CONFIG.EntrypointInputSourceFile,
-				standalone: _CONFIG.LibraryObjectName,
-				debug: true
-			});
-
-		// Ignore modules that should not be bundled (e.g. WASM loaders loaded via <script> tag)
-		if (Array.isArray(_CONFIG.BrowserifyIgnore))
-		{
-			for (var i = 0; i < _CONFIG.BrowserifyIgnore.length; i++)
-			{
-				tmpBrowserify.ignore(_CONFIG.BrowserifyIgnore[i]);
-			}
-		}
-
-		return tmpBrowserify.bundle()
+		return createBrowserifyInstance(_CONFIG).bundle()
 			.pipe(libVinylSourceStream(_CONFIG.LibraryUniminifiedFileName))
 			.pipe(libVinylBuffer())
 			.pipe(libSourcemaps.init({loadMaps: true}))

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "quackage",
-    "version": "1.0.61",
+    "version": "1.0.63",
     "description": "Building.  Testing.  Quacking.  Reloading.",
     "main": "source/Quackage-CLIProgram.js",
     "scripts": {

package/source/Quackage-CLIProgram.js

@@ -20,6 +20,7 @@ let _Pict = new libCLIProgram(
 		require('./commands/Quackage-Command-UpdatePackage.js'),
 		require('./commands/Quackage-Command-UpdatePackage-Luxury.js'),
 		require('./commands/Quackage-Command-Lint.js'),
+		require('./commands/Quackage-Command-CheckDependencies.js'),
 		require('./commands/Quackage-Command-UpdateNodeGitignore.js'),
 
 		// Mocha test execution

package/source/commands/Quackage-Command-Build.js

@@ -69,7 +69,15 @@ class QuackageCommandBuild extends libCommandLineCommand
 				}
 
 				// ## .gulpfile-quackage-config.json
-				libFS.writeFileSync(`${this.fable.AppData.CWD}/.gulpfile-quackage-config.json`, this.fable.parseTemplateByHash('Gulpfile-Configuration', pAction));
+				let tmpGulpfileConfigContent = this.fable.parseTemplateByHash('Gulpfile-Configuration', pAction);
+				// Allow per-action BrowserifyIgnore overrides from GulpExecutions entries
+				if (Array.isArray(pAction.BrowserifyIgnore) && pAction.BrowserifyIgnore.length > 0)
+				{
+					let tmpGulpfileConfig = JSON.parse(tmpGulpfileConfigContent);
+					tmpGulpfileConfig.BrowserifyIgnore = pAction.BrowserifyIgnore;
+					tmpGulpfileConfigContent = JSON.stringify(tmpGulpfileConfig, null, 4);
+				}
+				libFS.writeFileSync(`${this.fable.AppData.CWD}/.gulpfile-quackage-config.json`, tmpGulpfileConfigContent);
 				// ## .gulpfile-quackage.js
 				libFS.writeFileSync(`${this.fable.AppData.CWD}/.gulpfile-quackage.js`, this.fable.parseTemplateByHash('Gulpfile-QuackageBase', { AppData: this.fable.AppData, Record: pAction }));
 				// ## gulpfile.js

package/source/commands/Quackage-Command-CheckDependencies.js

@@ -0,0 +1,134 @@
+const libCommandLineCommand = require('pict-service-commandlineutility').ServiceCommandLineCommand;
+const libFS = require('fs');
+const libPath = require('path');
+
+class QuackageCommandCheckDependencies extends libCommandLineCommand
+{
+	constructor(pFable, pManifest, pServiceHash)
+	{
+		super(pFable, pManifest, pServiceHash);
+
+		this.options.CommandKeyword = 'check-dependencies';
+		this.options.Description = 'Check package.json for local file:// references and list all dependency versions';
+		this.options.Aliases.push('check-deps');
+		this.options.Aliases.push('checkdeps');
+
+		this.options.CommandOptions.push({ Name: '-a, --allow-file-package-references', Description: 'Allow file:// references without returning an error exit code', Default: false });
+
+		this.addCommand();
+	}
+
+	onRunAsync(fCallback)
+	{
+		let tmpPackage = this.fable.AppData.Package;
+		let tmpCWD = this.fable.AppData.CWD;
+		let tmpAllowFileReferences = this.CommandOptions.allowFilePackageReferences ? true : false;
+
+		let tmpFileReferenceCount = 0;
+		let tmpMismatchCount = 0;
+
+		this.log.info(`Checking dependencies for ${tmpPackage.name}@${tmpPackage.version} ...`);
+
+		let tmpDependencySections = ['dependencies', 'devDependencies', 'peerDependencies', 'optionalDependencies'];
+
+		for (let i = 0; i < tmpDependencySections.length; i++)
+		{
+			let tmpSectionName = tmpDependencySections[i];
+
+			if (!tmpPackage.hasOwnProperty(tmpSectionName))
+			{
+				continue;
+			}
+
+			let tmpDependencies = tmpPackage[tmpSectionName];
+			let tmpDependencyNames = Object.keys(tmpDependencies);
+
+			if (tmpDependencyNames.length < 1)
+			{
+				continue;
+			}
+
+			console.log('');
+			this.log.info(`--- ${tmpSectionName} ---`);
+
+			for (let j = 0; j < tmpDependencyNames.length; j++)
+			{
+				let tmpDepName = tmpDependencyNames[j];
+				let tmpDepVersion = tmpDependencies[tmpDepName];
+				let tmpIsFileReference = (typeof(tmpDepVersion) === 'string') && tmpDepVersion.startsWith('file://');
+
+				// Try to read the installed version from node_modules
+				let tmpInstalledVersion = '(not installed)';
+				try
+				{
+					let tmpInstalledPackagePath = libPath.join(tmpCWD, 'node_modules', tmpDepName, 'package.json');
+					if (libFS.existsSync(tmpInstalledPackagePath))
+					{
+						let tmpInstalledPackage = JSON.parse(libFS.readFileSync(tmpInstalledPackagePath, 'utf8'));
+						tmpInstalledVersion = tmpInstalledPackage.version || '(unknown)';
+					}
+				}
+				catch (pError)
+				{
+					tmpInstalledVersion = '(error reading)';
+				}
+
+				let tmpVersionMatch = true;
+				// For non-file references, check if the installed version satisfies the specified range
+				if (!tmpIsFileReference && tmpInstalledVersion !== '(not installed)' && tmpInstalledVersion !== '(error reading)' && tmpInstalledVersion !== '(unknown)')
+				{
+					// Simple display check -- show mismatch if the specified version doesn't contain the installed version
+					// This is intentionally simple; a full semver check would require a library
+					if (tmpDepVersion !== tmpInstalledVersion && !tmpDepVersion.includes(tmpInstalledVersion))
+					{
+						tmpVersionMatch = false;
+						tmpMismatchCount++;
+					}
+				}
+
+				if (tmpIsFileReference)
+				{
+					tmpFileReferenceCount++;
+					this.log.warn(`  [FILE REF] ${tmpDepName}: ${tmpDepVersion}  -->  installed: ${tmpInstalledVersion}`);
+				}
+				else if (!tmpVersionMatch)
+				{
+					this.log.info(`  [MISMATCH] ${tmpDepName}: ${tmpDepVersion}  -->  installed: ${tmpInstalledVersion}`);
+				}
+				else
+				{
+					this.log.info(`  [OK]       ${tmpDepName}: ${tmpDepVersion}  -->  installed: ${tmpInstalledVersion}`);
+				}
+			}
+		}
+
+		console.log('');
+
+		if (tmpMismatchCount > 0)
+		{
+			this.log.warn(`Found ${tmpMismatchCount} version mismatch(es) between package.json and node_modules.`);
+		}
+
+		if (tmpFileReferenceCount > 0)
+		{
+			this.log.warn(`Found ${tmpFileReferenceCount} file:// reference(s) in package.json!`);
+			if (!tmpAllowFileReferences)
+			{
+				this.log.error(`Refusing to publish with local file:// references. Use --allow-file-package-references to override.`);
+				this.fable.QuackageProcess.exitParentProcess(1);
+			}
+			else
+			{
+				this.log.info(`File references allowed by --allow-file-package-references flag.`);
+			}
+		}
+		else
+		{
+			this.log.info(`No file:// references found.  All clear for publishing!`);
+		}
+
+		return fCallback();
+	};
+}
+
+module.exports = QuackageCommandCheckDependencies;

package/source/commands/Quackage-Command-CopyFilesFromTo.js

@@ -43,7 +43,7 @@ class QuackageCommandCopyFilesFromTo extends libCommandLineCommand
 		{
 			let tmpErrorMessage = `Not even the git checkout location has an installation of copy-files-from-to at [${tmpCFFTLocation}]... building cannot commence.  We also tried CWD [${tmpCWDCFFTLocation}] and relative node_modules [${tmpRelativePackageCFFTLocation}].  Sorry!  Maybe you need to run "npm install" somewhere??`;
 			this.log.info(tmpErrorMessage)
-			return fActionCallback(new Error(tmpErrorMessage));
+			return fCallback(new Error(tmpErrorMessage));
 		}
 		this.log.info(`Quackage found copy-files-from-to at [${tmpCFFTLocation}] ... executing build from there.`);
 

package/source/commands/Quackage-Command-RunMochaTests.js

@@ -44,7 +44,7 @@ class QuackageCommandBuild extends libCommandLineCommand
 		{
 			let tmpErrorMessage = `Not even the git checkout location has an installation of mocha at [${tmpMochaLocation}]... building cannot commence.  We also tried CWD [${tmpCWDMochaLocation}] and relative node_modules [${tmpRelativePackageMochaLocation}].  Sorry!  Maybe you need to run "npm install" somewhere??`;
 			this.log.info(tmpErrorMessage)
-			return fActionCallback(new Error(tmpErrorMessage));
+			return fCallback(new Error(tmpErrorMessage));
 		}
 		this.log.info(`Quackage found mocha at [${tmpMochaLocation}] ... executing build from there.`);