refactor: refactor release

Author: zdm

lib/commands/package.js

@@ -8,9 +8,14 @@ export default class extends Command {
             "commands": {
                 "release": {
                     "short": "R",
-                    "title": "Release and publish package",
+                    "title": "Release package",
                     "module": () => new URL( "package/release.js", import.meta.url ),
                 },
+                "publish": {
+                    "short": "P",
+                    "title": "Publish package",
+                    "module": () => new URL( "package/publish.js", import.meta.url ),
+                },
                 "update-dependencies": {
                     "short": "u",
                     "title": "Update package dependencies",

lib/commands/package/publish.js

@@ -0,0 +1,60 @@
+import Command from "#lib/command";
+
+export default class extends Command {
+
+    // static
+    static cli () {
+        return {
+            "options": {
+                "access": {
+                    "short": "a",
+                    "description": "Set package access level.",
+                    "schema": { "enum": [ "public", "private" ] },
+                },
+                "yes": {
+                    "short": "y",
+                    "description": `answer "YES" on all questions`,
+                    "default": false,
+                    "schema": { "type": "boolean" },
+                },
+            },
+            "arguments": {
+                "commit-reference": {
+                    "description": "Git commit reference to publish.",
+                    "default": "HEAD",
+                    "schema": { "type": "string" },
+                },
+            },
+        };
+    }
+
+    // public
+    async run () {
+        const pkg = this._findGitPackage();
+        if ( !pkg ) return result( [ 500, "Unable to find root package" ] );
+
+        var res;
+
+        res = await pkg.npm.publish( {
+            "commitRef": process.cli.arguments[ "commit-reference" ],
+            "repeatOnError": !process.cli.options.yes,
+            "accessStatus": process.cli.options.access,
+        } );
+        if ( !res.ok ) return res;
+
+        // get sub-packages
+        const subPackages = pkg.subPackages;
+
+        // publish sub-packages
+        for ( const pkg of subPackages ) {
+            res = await pkg.npm.publish( {
+                "commitRef": process.cli.arguments[ "commit-reference" ],
+                "repeatOnError": !process.cli.options.yes,
+                "accessStatus": process.cli.options.access,
+            } );
+            if ( !res.ok ) return res;
+        }
+
+        return result( 200 );
+    }
+}

lib/commands/package/release.js

@@ -12,6 +12,12 @@ export default class extends Command {
                     "default": false,
                     "schema": { "type": "boolean" },
                 },
+                "publish": {
+                    "short": "p",
+                    "description": "publish package",
+                    "default": false,
+                    "schema": { "type": "boolean" },
+                },
             },
             "arguments": {
                 "pre-release-tag": {
@@ -34,6 +40,7 @@ export default class extends Command {
 
         const release = await pkg.release( {
             "preReleaseTag": process.cli.arguments[ "pre-release-tag" ],
+            "publish": process.cli.options.publish,
             "yes": process.cli.options.yes,
         } );
 

lib/npm.js

@@ -70,16 +70,12 @@ export default class Npm {
                 proc.once( "close", code => {
                     var res;
 
-                    if ( code ) {
-                        let data = Buffer.concat( stdout );
-                        data = JSON.parse( data );
+                    const data = JSON.parse( Buffer.concat( stdout ) );
 
+                    if ( code ) {
                         res = result( [ 500, data.error.summary ], data );
                     }
                     else {
-                        let data = Buffer.concat( stdout );
-                        data = JSON.parse( data );
-
                         res = result( 200, data );
                     }
 
@@ -145,10 +141,8 @@ export default class Npm {
         return res;
     }
 
-    async setPackageAccessStatus ( packageName, privateAccess, { cwd, registry } = {} ) {
-        const args = [ "access", "set", "status=" + ( privateAccess
-            ? "private"
-            : "public" ), packageName ];
+    async setPackageAccessStatus ( packageName, accessStatus, { cwd, registry } = {} ) {
+        const args = [ "access", "set", "status=" + accessStatus, packageName ];
 
         const res = await this.exec( args, {
             cwd,
@@ -208,7 +202,7 @@ export default class Npm {
         } );
     }
 
-    async publish ( { executablesPatterns, packPath, "private": privateAccess, tag, cwd, registry } = {} ) {
+    async publish ( { executablesPatterns, packPath, accessStatus, tag, cwd, registry } = {} ) {
         if ( !packPath ) {
             const res = await this.pack( {
                 executablesPatterns,
@@ -223,8 +217,8 @@ export default class Npm {
 
         const args = [ "publish" ];
 
-        if ( privateAccess != null ) {
-            args.push( "--access", privateAccess
+        if ( accessStatus ) {
+            args.push( "--access", accessStatus === "private"
                 ? "restricted"
                 : "public" );
         }