prepare-package 1.2.2 → 1.2.4

package/dist/index.js

@@ -6,6 +6,40 @@ const logger = require('./logger');
 
 // const argv = require('yargs').argv;
 
+// Helper function to check for local dependencies
+function checkForLocalDependencies(packageJSON) {
+  const localDeps = [];
+  const depTypes = [
+    'dependencies',
+    'devDependencies',
+    'peerDependencies',
+    'optionalDependencies'
+  ];
+
+  depTypes.forEach(type => {
+    if (!packageJSON[type]) {
+      return;
+    }
+
+    Object.entries(packageJSON[type]).forEach(([name, version]) => {
+      // Check for file: protocol or relative/absolute paths
+      if (version.startsWith('file:')
+        || version.startsWith('../')
+        || version.startsWith('./')
+        || version.startsWith('/')
+        || version.startsWith('~')) {
+        localDeps.push({
+          type,
+          name,
+          value: version
+        });
+      }
+    });
+  });
+
+  return localDeps;
+}
+
 module.exports = async function (options) {
   // Set the options
   options = options || {};
@@ -14,6 +48,13 @@ module.exports = async function (options) {
   options.isPostInstall = typeof options.isPostInstall === 'undefined' ? false : options.isPostInstall;
   options.singleFile = options.singleFile || null; // For watch mode - single file to process
 
+  // Detect if running via npm publish
+  // When npm publish runs, it sets npm_command to 'publish' even though lifecycle_event is 'prepare'
+  const isPublishing = process.env.npm_command === 'publish'
+    || process.env.npm_lifecycle_event === 'prepublishOnly'
+    || process.env.npm_lifecycle_event === 'prepublish';
+  const isManualPrepare = process.env.npm_lifecycle_event === 'prepare' && !isPublishing;
+
   // Set the paths
   const theirPackageJSONPath = path.resolve(options.cwd, 'package.json');
   const theirPackageJSONExists = jetpack.exists(theirPackageJSONPath);
@@ -23,6 +64,20 @@ module.exports = async function (options) {
   const theirPackageJSON = theirPackageJSONExists ? require(theirPackageJSONPath) : {};
   const isLivePreparation = theirPackageJSON.name !== 'prepare-package';
 
+
+  // Check for local dependencies when publishing
+  if (isPublishing) {
+    const hasLocalDeps = checkForLocalDependencies(theirPackageJSON);
+    if (hasLocalDeps.length > 0) {
+      logger.error('Cannot publish with local dependencies!');
+      logger.error('The following dependencies use local file references:');
+      hasLocalDeps.forEach(dep => {
+        logger.error(`  - ${dep.type}: ${dep.name} -> ${dep.value}`);
+      });
+      throw new Error('Publishing blocked: Remove local file dependencies before publishing to npm');
+    }
+  }
+
   // const options = {
   //   purge: argv['--purge'] || argv['-p'],
   // };
@@ -49,12 +104,16 @@ module.exports = async function (options) {
   // console.log(chalk.blue(`[prepare-package]: output=${theirPackageJSON.preparePackage.output}`));
   // console.log(chalk.blue(`[prepare-package]: main=${theirPackageJSON.main}`));
   if (!options.singleFile) {
-    logger.log(`Starting...`);
+    logger.log(`Starting...${isPublishing ? ' (via npm publish)' : ''}`);
     logger.log({
       purge: options.purge,
       input: theirPackageJSON.preparePackage.input,
       output: theirPackageJSON.preparePackage.output,
       main: theirPackageJSON.main,
+
+      // lifecycle: process.env.npm_lifecycle_event,
+      // command: process.env.npm_command,
+      isPublishing: isPublishing,
     });
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prepare-package",
-  "version": "1.2.2",
+  "version": "1.2.4",
   "description": "Prepare a Node.js package before being published",
   "main": "dist/index.js",
   "exports": {
@@ -40,9 +40,9 @@
   },
   "dependencies": {
     "chalk": "^4.1.2",
-    "chokidar": "^3.5.3",
-    "fs-jetpack": "^4.3.1",
-    "wonderful-fetch": "^1.3.2"
+    "chokidar": "^4.0.3",
+    "fs-jetpack": "^5.1.0",
+    "wonderful-fetch": "^1.3.3"
   },
   "devDependencies": {
     "mocha": "^8.4.0"