add skip npm option

fixtures/pnpm/star-package/do-not-publish/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "do-not-publish",
+  "version": "1.0.0",
+  "description": "",
+  "main": "index.js",
+  "scripts": {
+    "test": "echo \"Error: no test specified\" && exit 1"
+  },
+  "keywords": [],
+  "author": "",
+  "license": "ISC",
+  "release-plan": {
+    "skipNpmPublish": true
+  }
+}

package.json

@@ -20,7 +20,7 @@
     "lint:js": "eslint . --cache",
     "lint:js:fix": "eslint . --fix",
     "prepare": "pnpm run build",
-    "test": "vitest",
+    "test": "vitest run",
     "coverage": "vitest run --coverage"
   },
   "dependencies": {

src/interdep.test.ts

@@ -34,6 +34,7 @@ describe('interdep', function () {
 
       expect(Array(...answer.keys())).toMatchInlineSnapshot(`
         [
+          "do-not-publish",
           "star-package",
         ]
       `);

src/publish.test.ts

@@ -6,6 +6,7 @@ import {
   IssueReporter,
 } from './publish.js';
 import { Solution } from './plan.js';
+import { getPackages } from './interdep.js';
 
 // we aren't currently using this so we can just ignore for now
 const reporter = new IssueReporter();
@@ -217,7 +218,12 @@ describe('publish', function () {
         new Map([
           [
             'release-plan',
-            { oldVersion: '0.8.1', newVersion: '0.9.0', impact: 'minor' },
+            {
+              oldVersion: '0.8.1',
+              newVersion: '0.9.0',
+              impact: 'minor',
+              pkgJSONPath: './package.json',
+            },
           ],
         ]) as Solution,
         reporter,
@@ -241,6 +247,58 @@ describe('publish', function () {
         }
       `);
     });
+
+    it('skips publishing if npmSkipPublish is specified in package.json', async function () {
+      const consoleSpy = vi.spyOn(process.stdout, 'write');
+      const packages = getPackages('./fixtures/pnpm/star-package');
+      const publishResult = await npmPublish(
+        new Map([
+          [
+            'do-not-publish',
+            {
+              oldVersion: '0.8.1',
+              newVersion: '0.9.0',
+              impact: 'minor',
+              pkgJSONPath: packages.get('do-not-publish')?.pkgJSONPath,
+            },
+          ],
+          [
+            'star-package',
+            {
+              oldVersion: '0.8.1',
+              newVersion: '0.9.0',
+              impact: 'minor',
+              pkgJSONPath: packages.get('star-package')?.pkgJSONPath,
+            },
+          ],
+        ]) as Solution,
+        reporter,
+        {
+          tag: 'best-tag',
+          dryRun: true,
+        },
+        'face',
+      );
+
+      expect(consoleSpy.mock.lastCall?.[0]).toMatchInlineSnapshot(`
+        "
+         ℹ️ --dryRun active. Adding \`--dry-run\` flag to \`face publish\` for star-package, which would publish version 0.9.0
+        "
+      `);
+
+      expect(publishResult).toMatchInlineSnapshot(`
+        {
+          "args": [
+            "publish",
+            "--tag=best-tag",
+            "--dry-run",
+          ],
+          "released": Map {
+            "star-package" => "0.9.0",
+          },
+        }
+      `);
+    });
   });
 
   describe('createGithubRelease', function () {

src/publish.ts

@@ -6,7 +6,7 @@ import latestVersion from 'latest-version';
 import { dirname } from 'path';
 import PackageJson from '@npmcli/package-json';
 import parseGithubUrl from 'parse-github-repo-url';
-import fsExtra from 'fs-extra';
+import fsExtra, { readJSONSync } from 'fs-extra';
 
 const { existsSync } = fsExtra;
 
@@ -266,6 +266,14 @@ export async function npmPublish(
       continue;
     }
 
+    const pkg = readJSONSync(entry.pkgJSONPath);
+    if (pkg['release-plan']?.skipNpmPublish) {
+      info(
+        `skipping publish for ${pkgName}, as config option skipNpmPublish is set in its package.json`,
+      );
+      continue;
+    }
+
     const preExisting = await doesVersionExist(
       pkgName,
       entry.newVersion,
@@ -294,6 +302,7 @@ export async function npmPublish(
         stderr: 'inherit',
         stdout: 'inherit',
       });
+      released.set(pkgName, entry.newVersion);
     } catch (err) {
       reporter.reportFailure(
         `Failed to ${packageManager} publish ${pkgName} - Error: ${err.message}`,