ci: improve publish command (#786)

Author: BioPhoton

packages/cli/project.json

@@ -16,7 +16,7 @@
       }
     },
     "publish": {
-      "command": "node tools/scripts/publish.mjs cli {args.ver} {args.tag}",
+      "command": "node tools/scripts/publish.mjs --name=cli --ver={args.ver} --tag={args.tag}",
       "dependsOn": ["build"]
     },
     "lint": {

packages/core/project.json

@@ -16,7 +16,7 @@
       }
     },
     "publish": {
-      "command": "node tools/scripts/publish.mjs core {args.ver} {args.tag}",
+      "command": "node tools/scripts/publish.mjs --name=core --ver={args.ver} --tag={args.tag}",
       "dependsOn": ["build"]
     },
     "lint": {

packages/models/project.json

@@ -16,7 +16,7 @@
       }
     },
     "publish": {
-      "command": "node tools/scripts/publish.mjs models {args.ver} {args.tag}",
+      "command": "node tools/scripts/publish.mjs --name=models --ver={args.ver} --tag={args.tag}",
       "dependsOn": ["build"]
     },
     "lint": {

packages/nx-plugin/project.json

@@ -38,7 +38,7 @@
       }
     },
     "publish": {
-      "command": "node tools/scripts/publish.mjs nx-plugin {args.ver} {args.tag}",
+      "command": "node tools/scripts/publish.mjs --name=nx-plugin --ver={args.ver} --tag={args.tag}",
       "dependsOn": ["build"]
     },
     "lint": {

packages/plugin-coverage/project.json

@@ -39,7 +39,7 @@
       }
     },
     "publish": {
-      "command": "node tools/scripts/publish.mjs plugin-coverage {args.ver} {args.tag}",
+      "command": "node tools/scripts/publish.mjs --name=plugin-coverage --ver={args.ver} --tag={args.tag}",
       "dependsOn": ["build"]
     }
   },

packages/plugin-eslint/project.json

@@ -17,7 +17,7 @@
       }
     },
     "publish": {
-      "command": "node tools/scripts/publish.mjs plugin-eslint {args.ver} {args.tag}",
+      "command": "node tools/scripts/publish.mjs --name=plugin-eslint --ver={args.ver} --tag={args.tag}",
       "dependsOn": ["build"]
     },
     "lint": {

packages/plugin-js-packages/project.json

@@ -39,7 +39,7 @@
       }
     },
     "publish": {
-      "command": "node tools/scripts/publish.mjs plugin-js-packages {args.ver} {args.tag}",
+      "command": "node tools/scripts/publish.mjs --name=plugin-js-packages --ver={args.ver} --tag={args.tag}",
       "dependsOn": ["build"]
     }
   },

packages/utils/project.json

@@ -16,7 +16,7 @@
       }
     },
     "publish": {
-      "command": "node tools/scripts/publish.mjs utils {args.ver} {args.tag}",
+      "command": "node tools/scripts/publish.mjs --name=utils --ver={args.ver} --tag={args.tag}",
       "dependsOn": ["build"]
     },
     "lint": {

tools/scripts/publish.mjs

@@ -7,8 +7,10 @@
  * You might need to authenticate with NPM before running this script.
  */
 import devkit from '@nx/devkit';
-import { execSync } from 'child_process';
-import { readFileSync, writeFileSync } from 'fs';
+import { execSync } from 'node:child_process';
+import { readFileSync, writeFileSync } from 'node:fs';
+import yargs from 'yargs';
+import { hideBin } from 'yargs/helpers';
 
 const { readCachedProjectGraph } = devkit;
 
@@ -19,16 +21,28 @@ function invariant(condition, message) {
   }
 }
 
-// Executing publish script: node path/to/publish.mjs {name} --version {version} --tag {tag}
-// Default "tag" to "next" so we won't publish the "latest" tag by accident.
-const [, , name, version, tag = 'next'] = process.argv;
-
 // A simple SemVer validation to validate the version
 const validVersion = /^\d+\.\d+\.\d+(-\w+\.\d+)?/;
-invariant(
-  version && validVersion.test(version),
-  `No version provided or version did not match Semantic Versioning, expected: #.#.#-tag.# or #.#.#, got ${version}.`,
-);
+
+// Executing publish script: node path/to/publish.mjs {name} --version {version} --tag {tag}
+// Default "tag" to "next" so we won't publish the "latest" tag by accident.
+const {
+  name,
+  ver: version,
+  tag,
+} = yargs(hideBin(process.argv))
+  .options({
+    name: { type: 'string', demandOption: true },
+    ver: { type: 'string', demandOption: true },
+    tag: { type: 'string', default: 'next' },
+  })
+  .coerce('ver', ver => {
+    invariant(
+      ver && validVersion.test(ver),
+      `No version provided or version did not match Semantic Versioning, expected: #.#.#-tag.# or #.#.#, got ${ver}.`,
+    );
+    return ver;
+  }).argv;
 
 const graph = readCachedProjectGraph();
 const project = graph.nodes[name];

tools/scripts/start-local-registry.ts

@@ -25,16 +25,7 @@ export default async () => {
   // is is also possible to use nx release to publish the packages to the local registry
   execFileSync(
     'npx',
-    [
-      'nx',
-      'run-many',
-      '--targets',
-      'publish',
-      '--ver',
-      version,
-      '--tag',
-      'e2e',
-    ],
+    ['nx', 'run-many', '--targets=publish', `--ver=${version}`, '--tag=e2e'],
     { env: process.env, stdio: 'inherit', shell: true },
   );
 };