@bussolabs/closeyourit-cli 0.25.0 → 0.25.1

package/dist/base.js

@@ -36,6 +36,13 @@ function paginationError(opts) {
         return `Invalid --per ${opts.per}: ask for at least 1 row.`;
     return undefined;
 }
+/**
+ * `this.exit(n)` di oclif lancia un ExitError: si riconosce dal codice `EEXIT`, non dalla classe —
+ * importarla legherebbe la base a un percorso interno di oclif che cambia fra le versioni.
+ */
+function isExitError(error) {
+    return typeof error === 'object' && error !== null && error.code === 'EEXIT';
+}
 class BaseCommand extends core_1.Command {
     static enableJsonFlag = true;
     /**
@@ -205,6 +212,12 @@ class BaseCommand extends core_1.Command {
      * swallowed (empty stdout, exit looks like a silent success) for machine consumers (CYCL-1).
      */
     async catch(error) {
+        // CYCL-52 — `this.exit(n)` è una decisione presa dal comando, non un guasto da raccontare: chi
+        // l'ha chiamata ha già detto la sua sull'uscita. Senza questa riga l'ExitError arriva quaggiù e
+        // il ramo generico emette un SECONDO envelope, così chi legge in modalità dati si trova due
+        // oggetti JSON sulla stessa uscita e non sa quale dei due sia la risposta.
+        if (isExitError(error))
+            throw error;
         if (error instanceof api_1.ApiRequestError) {
             if (this.jsonEnabled()) {
                 this.logJson({ error: { code: error.code, message: error.message, ...(error.details === undefined ? {} : { details: error.details }) } });

package/dist/commands/tickets/create.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const core_1 = require("@oclif/core");
 const base_1 = require("../../base");
+const error_codes_1 = require("../../errors/error-codes");
 const limits_1 = require("../../lib/limits");
 const output_1 = require("../../lib/output");
 const ticket_body_1 = require("../../lib/ticket-body");
@@ -42,9 +43,29 @@ class TicketsCreate extends base_1.BaseCommand {
         body.status_id = statusId;
         body.priority_id = priorityId;
         const res = await this.api.post(`/cli/v1/projects/${projectId}/tickets`, body);
-        if (!this.jsonEnabled()) {
-            this.log('Ticket created:');
-            this.log((0, output_1.renderRecord)(res.data ?? {}));
+        // CYCL-52 — da qui in poi il ticket ESISTE sul server, e qualunque cosa vada storta mentre lo si
+        // racconta non lo cancella. Se il comando esce non-zero senza dire niente, chi legge conclude che
+        // non è stato creato e lo ricrea: è così che sono nati CYRA-755/756, due volte lo stesso ticket.
+        // Perciò il codice e l'id vengono emessi comunque, e solo dopo l'errore vero risale.
+        try {
+            if (!this.jsonEnabled()) {
+                this.log('Ticket created:');
+                this.log((0, output_1.renderRecord)(res.data ?? {}));
+            }
+        }
+        catch (error) {
+            const creato = res.data ?? {};
+            const identita = { code: creato.code, id: creato.id };
+            const motivo = error instanceof Error ? error.message : String(error);
+            // In modalità dati l'envelope è UNO: se si lasciasse risalire l'errore, la base ne emetterebbe
+            // un secondo e chi legge si troverebbe due oggetti sulla stessa uscita. Qui data ed error
+            // stanno insieme — il ticket c'è, e c'è anche cosa è andato storto — e si esce non-zero.
+            if (this.jsonEnabled()) {
+                this.logJson({ data: identita, error: { code: error_codes_1.ErrorCodes.System.unexpected, message: motivo } });
+                return this.exit(1);
+            }
+            this.logToStderr(`Ticket creato (${String(identita.code ?? identita.id ?? 'senza codice')}), ma la resa è fallita: ${motivo}`);
+            throw error;
         }
         return res;
     }