@adonisjs/assembler 6.1.3-21 → 6.1.3-23

package/build/src/code_transformer/main.js

@@ -63,28 +63,28 @@ export class CodeTransformer {
         /**
          * Delete the existing middleware if it exists
          */
-        const existingMiddleware = arrayLiteralExpression
+        const existingMiddlewareIndex = arrayLiteralExpression
             .getElements()
             .findIndex((element) => element.getText() === middleware);
-        if (existingMiddleware !== -1) {
-            arrayLiteralExpression.removeElement(existingMiddleware);
-        }
-        /**
-         * Add the middleware to the top or bottom of the array
-         */
-        if (middlewareEntry.position === 'before') {
-            arrayLiteralExpression.insertElement(0, middleware);
-        }
-        else {
-            arrayLiteralExpression.addElement(middleware);
+        if (existingMiddlewareIndex === -1) {
+            /**
+             * Add the middleware to the top or bottom of the array
+             */
+            if (middlewareEntry.position === 'before') {
+                arrayLiteralExpression.insertElement(0, middleware);
+            }
+            else {
+                arrayLiteralExpression.addElement(middleware);
+            }
         }
     }
     /**
      * Add a new middleware to the named middleware of the given file
      */
     #addToNamedMiddleware(file, middlewareEntry) {
-        if (!middlewareEntry.name)
+        if (!middlewareEntry.name) {
             throw new Error('Named middleware requires a name.');
+        }
         const callArguments = file
             .getVariableDeclarationOrThrow('middleware')
             .getInitializerIfKindOrThrow(SyntaxKind.CallExpression)
@@ -100,20 +100,21 @@ export class CodeTransformer {
          * Check if property is already defined. If so, remove it
          */
         const existingProperty = namedMiddlewareObject.getProperty(middlewareEntry.name);
-        if (existingProperty)
-            existingProperty.remove();
-        /**
-         * Add the named middleware
-         */
-        const middleware = `${middlewareEntry.name}: () => import('${middlewareEntry.path}')`;
-        namedMiddlewareObject.insertProperty(0, middleware);
+        if (!existingProperty) {
+            /**
+             * Add the named middleware
+             */
+            const middleware = `${middlewareEntry.name}: () => import('${middlewareEntry.path}')`;
+            namedMiddlewareObject.insertProperty(0, middleware);
+        }
     }
     /**
      * Write a leading comment
      */
     #addLeadingComment(writer, comment) {
-        if (!comment)
+        if (!comment) {
             return writer.blankLine();
+        }
         return writer
             .blankLine()
             .writeLine('/*')
@@ -173,7 +174,7 @@ export class CodeTransformer {
         if (!Node.isObjectLiteralExpression(objectLiteralExpression)) {
             throw new Error(`The second argument of Env.create is not an object literal.`);
         }
-        let firstAdded = false;
+        let shouldAddComment = true;
         /**
          * Add each variable validation
          */
@@ -182,18 +183,29 @@ export class CodeTransformer {
              * Check if the variable is already defined. If so, remove it
              */
             const existingProperty = objectLiteralExpression.getProperty(variable);
-            if (existingProperty)
-                existingProperty.remove();
-            objectLiteralExpression.addPropertyAssignment({
-                name: variable,
-                initializer: validation,
-                leadingTrivia: (writer) => {
-                    if (firstAdded)
-                        return;
-                    firstAdded = true;
-                    return this.#addLeadingComment(writer, definition.leadingComment);
-                },
-            });
+            /**
+             * Do not add leading comment if one or more properties
+             * already exists
+             */
+            if (existingProperty) {
+                shouldAddComment = false;
+            }
+            /**
+             * Add property only when the property does not exist
+             */
+            if (!existingProperty) {
+                objectLiteralExpression.addPropertyAssignment({
+                    name: variable,
+                    initializer: validation,
+                    leadingTrivia: (writer) => {
+                        if (!shouldAddComment) {
+                            return;
+                        }
+                        shouldAddComment = false;
+                        return this.#addLeadingComment(writer, definition.leadingComment);
+                    },
+                });
+            }
         }
         file.formatText(this.#editorSettings);
         await file.save();

package/build/src/code_transformer/rc_file_transformer.js

@@ -139,7 +139,7 @@ export class RcFileTransformer {
      * Add a new command to the rcFile
      */
     addCommand(commandPath) {
-        const commandsProperty = this.#getPropertyAssignmentInDefineConfigCall('providers', '[]');
+        const commandsProperty = this.#getPropertyAssignmentInDefineConfigCall('commands', '[]');
         const commandsArray = commandsProperty.getInitializerIfKindOrThrow(SyntaxKind.ArrayLiteralExpression);
         const commandString = `() => import('${commandPath}')`;
         /**

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@adonisjs/assembler",
   "description": "Provides utilities to run AdonisJS development server and build project for production",
-  "version": "6.1.3-21",
+  "version": "6.1.3-23",
   "engines": {
     "node": ">=18.16.0"
   },