style: refactor code

Author: zdm

lib/git.js

@@ -112,7 +112,7 @@ export default class Git extends CoreGit {
         return res;
     }
 
-    async getReleaseTags ( { allowMajorTag } = {} ) {
+    async getReleaseTags ( { majorTagEnabled } = {} ) {
         const res = await this.getTags();
         if ( !res.ok ) return res;
 
@@ -194,7 +194,7 @@ export default class Git extends CoreGit {
 
             // tag is not set
             if ( tags[ tag ].current == null ) {
-                if ( tags[ tag ].isMajorTag && !allowMajorTag ) {
+                if ( tags[ tag ].isMajorTag && !majorTagEnabled ) {
                     delete releaseTags[ tag ];
                 }
                 else if ( tags[ tag ].version ) {
@@ -203,7 +203,7 @@ export default class Git extends CoreGit {
             }
 
             // major tag is set, but not allowed
-            else if ( tags[ tag ].isMajorTag && !allowMajorTag ) {
+            else if ( tags[ tag ].isMajorTag && !majorTagEnabled ) {
                 releaseTags[ tag ].action = "delete";
             }
 

lib/package.js

@@ -478,7 +478,7 @@ export default class Package {
             if ( !res.ok ) return res;
         }
 
-        res = await this.git.getReleaseTags( { "allowMajorTag": this.cliConfig?.release.allowMajorTag } );
+        res = await this.git.getReleaseTags( { "majorTagEnabled": this.cliConfig?.release.majorTagEnabled } );
         if ( !res.ok ) return res;
         const tags = res.data;
 

lib/package/release.js

@@ -88,7 +88,7 @@ export default class Publish {
             this.#originalBranch = status.head.branch;
 
             // check allowed branch
-            if ( this.#pkg.cliConfig?.release.allowedBranches && !this.#pkg.cliConfig.release.allowedBranches.includes( this.#originalBranch ) ) {
+            if ( this.#pkg.cliConfig?.release.releaseBranches && !this.#pkg.cliConfig.release.releaseBranches.includes( this.#originalBranch ) ) {
                 return result( [ 500, "Unable to create release from the current branch" ] );
             }
 
@@ -154,7 +154,7 @@ export default class Publish {
 
             this.#majorLatestTag = `v${ this.#currentRelease.majorNumber }.latest`;
 
-            if ( this.#pkg.cliConfig?.release.allowMajorTag ) {
+            if ( this.#pkg.cliConfig?.release.majorTagEnabled ) {
                 this.#majorTag = `v${ this.#currentRelease.majorNumber }`;
             }
         }

resources/cli.config.yaml

@@ -52,8 +52,8 @@ meta:
 
 release:
   enabled: false
-  allowMajorTag: false
-  allowedBranches: [main]
+  releaseBranches: [main]
+  majorTagEnabled: false
 
 docker:
   composeFile: compose.yaml

resources/schemas/cli.config.schema.yaml

@@ -109,13 +109,13 @@ properties:
     type: object
     properties:
       enabled: { type: boolean }
-      allowMajorTag: { type: boolean }
-      allowedBranches:
+      releaseBranches:
         anyOf:
           - type: "null"
           - { type: array, items: { type: string }, minItems: 1, uniqueItems: true }
+      majorTagEnabled: { type: boolean }
     additionalProperties: false
-    required: [enabled, allowMajorTag, allowedBranches]
+    required: [enabled, releaseBranches, majorTagEnabled]
 
   docker:
     type: object