@bussolabs/closeyourit-cli 0.19.0 → 0.21.0

package/dist/lib/ticket-body.d.ts

@@ -1,4 +1,11 @@
+import { type LengthCheck } from './limits';
 export declare const ticketBodyFlags: {
+    'assignee-id': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+    'milestone-id': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+    'parent-id': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+    'platform-id': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+    status: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+    priority: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     description: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     'technical-analysis': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     scenarios: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
@@ -9,12 +16,10 @@ export declare const ticketBodyFlags: {
     condition: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     weight: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
     'due-at': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-    'status-id': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-    'priority-id': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-    'assignee-id': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-    'milestone-id': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-    'parent-id': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-    'platform-id': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+};
+export declare const CLEARABLE_FIELDS: Record<string, string>;
+export declare const clearFlag: {
+    clear: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
 };
 type BodyFlags = Record<string, unknown>;
 export declare function scenariosFromFlags(flags: BodyFlags): Array<Record<string, unknown>> | undefined;
@@ -22,4 +27,15 @@ export declare function conditionsFromFlags(flags: BodyFlags): Array<{
     text: string;
 }> | undefined;
 export declare function applyTicketBody(body: BodyFlags, flags: BodyFlags): void;
+/**
+ * I campi del corpo che il server misura, pronti per `tooLongMessage` (CYCL-32): titolo, descrizione
+ * e analisi tecnica. Scenari e condizioni non ci sono — non hanno tetto.
+ *
+ * `current` è il ticket com'è adesso, per i comandi che lo leggono prima di scrivere: fa da
+ * salvaguardia, così un testo scritto prima che il tetto esistesse resta correggibile finché non si
+ * allunga. Chi non lo conosce lo omette e il tetto vale pieno.
+ */
+export declare function ticketLengthChecks(flags: BodyFlags, current?: BodyFlags): LengthCheck[];
+/** Marca come vuoti i campi elencati da `--clear`: null, o lista vuota per le piattaforme. */
+export declare function applyTicketClears(body: BodyFlags, fields: string[] | undefined): void;
 export {};

package/dist/lib/ticket-body.js

@@ -1,17 +1,25 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.ticketBodyFlags = void 0;
+exports.clearFlag = exports.CLEARABLE_FIELDS = exports.ticketBodyFlags = void 0;
 exports.scenariosFromFlags = scenariosFromFlags;
 exports.conditionsFromFlags = conditionsFromFlags;
 exports.applyTicketBody = applyTicketBody;
+exports.ticketLengthChecks = ticketLengthChecks;
+exports.applyTicketClears = applyTicketClears;
 const core_1 = require("@oclif/core");
+const limits_1 = require("./limits");
+const ticket_lookup_1 = require("./ticket-lookup");
 // Flag del corpo ticket condivisi da `tickets create` e `tickets update`: descrizione (sommario
 // human-simple) + N scenari BDD + N condizioni DoD + analisi tecnica (registro tecnico separato) +
 // metadati. Gli scenari si passano come JSON (--scenarios) oppure, per un SINGOLO scenario, con le
 // scorciatoie --step-*. Le condizioni DoD sono --condition (ripetibile). Parità piena col form web.
 exports.ticketBodyFlags = {
-    description: core_1.Flags.string({ description: 'Free-form description (human-simple summary)' }),
-    'technical-analysis': core_1.Flags.string({ description: 'Technical analysis (stack, cause) — kept out of the human-simple body' }),
+    description: core_1.Flags.string({
+        description: `Free-form description (human-simple summary, max ${limits_1.LENGTH_LIMITS.description} characters)`,
+    }),
+    'technical-analysis': core_1.Flags.string({
+        description: `Technical analysis (stack, cause) — kept out of the human-simple body (max ${limits_1.LENGTH_LIMITS.technicalAnalysis} characters)`,
+    }),
     scenarios: core_1.Flags.string({
         description: 'BDD scenarios as a JSON array: [{"title","step_given","step_when","step_then","step_expected"}]',
     }),
@@ -22,13 +30,34 @@ exports.ticketBodyFlags = {
     condition: core_1.Flags.string({ description: 'Definition-of-Done line (repeatable)', multiple: true }),
     weight: core_1.Flags.string({ description: 'Story points (positive integer)' }),
     'due-at': core_1.Flags.string({ description: 'Due date (ISO 8601)' }),
-    'status-id': core_1.Flags.string({ description: 'Status lookup id' }),
-    'priority-id': core_1.Flags.string({ description: 'Priority lookup id' }),
+    // Stato e priorità si indicano per code, etichetta o id, e il comando li risolve (CYCL-34).
+    ...ticket_lookup_1.ticketLookupFlags,
     'assignee-id': core_1.Flags.string({ description: 'Assignee account id' }),
     'milestone-id': core_1.Flags.string({ description: 'Milestone id (of this project)' }),
     'parent-id': core_1.Flags.string({ description: 'Parent epic id (of this project) — pass an empty string to detach' }),
     'platform-id': core_1.Flags.string({ description: 'Affected platform id (repeatable)', multiple: true }),
 };
+// Campi che `tickets update` svuota solo se glielo si chiede (`--clear <campo>`): nome del flag che
+// li imposta → chiave del corpo. Svuotarli è una richiesta a sé, mai l'effetto collaterale di un
+// campo omesso (CYCL-30). Scenari e condizioni non ci sono: il backend li sostituisce quando il
+// corpo ne porta di nuovi, li lascia intatti quando li omette, e un array vuoto non li cancella.
+exports.CLEARABLE_FIELDS = {
+    description: 'description',
+    'technical-analysis': 'technical_analysis',
+    weight: 'weight',
+    'due-at': 'due_at',
+    assignee: 'assignee_id',
+    milestone: 'milestone_id',
+    parent: 'parent_id',
+    platforms: 'platform_ids',
+};
+exports.clearFlag = {
+    clear: core_1.Flags.string({
+        description: `Empty a field on purpose (repeatable): ${Object.keys(exports.CLEARABLE_FIELDS).join(', ')}`,
+        multiple: true,
+        options: Object.keys(exports.CLEARABLE_FIELDS),
+    }),
+};
 const STEP_KEYS = [
     ['step-given', 'step_given'],
     ['step-when', 'step_when'],
@@ -64,6 +93,8 @@ function conditionsFromFlags(flags) {
     return conditions.map((text) => ({ text }));
 }
 // Popola `body` coi campi del corpo presenti nei flag (chiavi snake_case attese dall'API CLI).
+// Stato e priorità restano fuori: vanno risolti in id contro le lookup dell'organizzazione, cioè
+// con una chiamata, e ogni comando li aggiunge dopo — con o senza predefinito (CYCL-34).
 function applyTicketBody(body, flags) {
     if (flags.description !== undefined)
         body.description = flags.description;
@@ -73,10 +104,6 @@ function applyTicketBody(body, flags) {
         body.weight = flags.weight;
     if (flags['due-at'] !== undefined)
         body.due_at = flags['due-at'];
-    if (flags['status-id'] !== undefined)
-        body.status_id = flags['status-id'];
-    if (flags['priority-id'] !== undefined)
-        body.priority_id = flags['priority-id'];
     if (flags['assignee-id'] !== undefined)
         body.assignee_id = flags['assignee-id'];
     if (flags['milestone-id'] !== undefined)
@@ -92,3 +119,38 @@ function applyTicketBody(body, flags) {
     if (conditions !== undefined)
         body.conditions_attributes = conditions;
 }
+/**
+ * I campi del corpo che il server misura, pronti per `tooLongMessage` (CYCL-32): titolo, descrizione
+ * e analisi tecnica. Scenari e condizioni non ci sono — non hanno tetto.
+ *
+ * `current` è il ticket com'è adesso, per i comandi che lo leggono prima di scrivere: fa da
+ * salvaguardia, così un testo scritto prima che il tetto esistesse resta correggibile finché non si
+ * allunga. Chi non lo conosce lo omette e il tetto vale pieno.
+ */
+function ticketLengthChecks(flags, current = {}) {
+    // `orthography`: il server corregge l'ortografia italiana di questi tre campi prima di misurarli.
+    return [
+        { flag: '--title', value: flags.title, max: limits_1.LENGTH_LIMITS.title, previous: current.title, orthography: true },
+        {
+            flag: '--description',
+            value: flags.description,
+            max: limits_1.LENGTH_LIMITS.description,
+            previous: current.description,
+            orthography: true,
+        },
+        {
+            flag: '--technical-analysis',
+            value: flags['technical-analysis'],
+            max: limits_1.LENGTH_LIMITS.technicalAnalysis,
+            previous: current.technical_analysis,
+            orthography: true,
+        },
+    ];
+}
+/** Marca come vuoti i campi elencati da `--clear`: null, o lista vuota per le piattaforme. */
+function applyTicketClears(body, fields) {
+    for (const field of fields ?? []) {
+        const key = exports.CLEARABLE_FIELDS[field];
+        body[key] = key === 'platform_ids' ? [] : null;
+    }
+}

package/dist/lib/ticket-lookup.d.ts

@@ -0,0 +1,53 @@
+/**
+ * Le due lookup d'organizzazione che ogni ticket si porta addosso: stato e priorità (CYCL-34).
+ *
+ * Il server le pretende entrambe alla creazione (R422-TICKET-001 "Status must exist and Priority
+ * must exist") mentre l'aiuto della CLI le dava per opzionali, e i loro id andavano cercati prima
+ * con `tickets statuses`/`tickets priorities` e incollati come UUID. Qui vivono l'endpoint da
+ * leggere, il code che vale quando il ticket nasce senza che li si sia indicati, e come si chiama
+ * il campo nei messaggi.
+ */
+export interface TicketLookup {
+    /** Lookup d'organizzazione (non di progetto): gli id valgono per un ticket di qualunque progetto. */
+    path: string;
+    /** Nome del campo nei messaggi d'errore. */
+    name: string;
+    /** Il flag che lo imposta. */
+    flag: string;
+    /** Il code che vale quando il ticket nasce e non lo si è indicato. */
+    fallback: string;
+    errorCode: string;
+}
+export declare const TICKET_STATUS: TicketLookup;
+export declare const TICKET_PRIORITY: TicketLookup;
+/**
+ * `--status`/`--priority` di un comando che modifica un ticket già esistente. `--status-id` e
+ * `--priority-id` restano come alias: erano l'unico modo di indicarli, e gli script che li usano
+ * continuano a funzionare senza riscritture.
+ */
+export declare const ticketLookupFlags: {
+    status: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+    priority: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+};
+/** Gli stessi due flag su un comando che crea il ticket, dove ometterli ha un valore predefinito. */
+export declare const newTicketLookupFlags: {
+    status: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+    priority: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+};
+/**
+ * Le voci che corrispondono al riferimento scritto a mano: prima il code, poi l'etichetta, senza
+ * badare alle maiuscole. Il code ha la precedenza perché è il nome esatto della voce: senza,
+ * l'etichetta di un'altra ("Urgent" su `high`) farebbe concorrenza al code di quella cercata
+ * (`urgent`) e una scelta legittima diventerebbe un'ambiguità da rifiutare.
+ */
+export declare function matchTicketLookup(rows: unknown, value: string): Array<{
+    id: string;
+}>;
+/**
+ * Il rifiuto di un riferimento che non si risolve: dice cos'è stato cercato, come si scrive il flag
+ * e quali voci esistono davvero — così scoprirle non costa un secondo comando.
+ */
+export declare function ticketLookupErrorMessage(lookup: TicketLookup, wanted: string, rows: unknown, { matched, isDefault }: {
+    matched: number;
+    isDefault: boolean;
+}): string;

package/dist/lib/ticket-lookup.js

@@ -0,0 +1,78 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.newTicketLookupFlags = exports.ticketLookupFlags = exports.TICKET_PRIORITY = exports.TICKET_STATUS = void 0;
+exports.matchTicketLookup = matchTicketLookup;
+exports.ticketLookupErrorMessage = ticketLookupErrorMessage;
+const core_1 = require("@oclif/core");
+const error_codes_1 = require("../errors/error-codes");
+exports.TICKET_STATUS = {
+    path: '/cli/v1/types/ticket_statuses',
+    name: 'Status',
+    flag: '--status',
+    fallback: 'open',
+    errorCode: error_codes_1.ErrorCodes.TicketStatus.notFound,
+};
+exports.TICKET_PRIORITY = {
+    path: '/cli/v1/types/ticket_priorities',
+    name: 'Priority',
+    flag: '--priority',
+    fallback: 'medium',
+    errorCode: error_codes_1.ErrorCodes.TicketPriority.notFound,
+};
+/**
+ * `--status`/`--priority` di un comando che modifica un ticket già esistente. `--status-id` e
+ * `--priority-id` restano come alias: erano l'unico modo di indicarli, e gli script che li usano
+ * continuano a funzionare senza riscritture.
+ */
+exports.ticketLookupFlags = {
+    status: core_1.Flags.string({
+        aliases: ['status-id'],
+        description: 'Status code, label or id (e.g. open, "In Progress") — unchanged when omitted',
+    }),
+    priority: core_1.Flags.string({
+        aliases: ['priority-id'],
+        description: 'Priority code, label or id (e.g. medium, High) — unchanged when omitted',
+    }),
+};
+/** Gli stessi due flag su un comando che crea il ticket, dove ometterli ha un valore predefinito. */
+exports.newTicketLookupFlags = {
+    status: core_1.Flags.string({
+        aliases: ['status-id'],
+        description: `Status code, label or id (e.g. open, "In Progress") — default: ${exports.TICKET_STATUS.fallback}`,
+    }),
+    priority: core_1.Flags.string({
+        aliases: ['priority-id'],
+        description: `Priority code, label or id (e.g. medium, High) — default: ${exports.TICKET_PRIORITY.fallback}`,
+    }),
+};
+/**
+ * Le voci che corrispondono al riferimento scritto a mano: prima il code, poi l'etichetta, senza
+ * badare alle maiuscole. Il code ha la precedenza perché è il nome esatto della voce: senza,
+ * l'etichetta di un'altra ("Urgent" su `high`) farebbe concorrenza al code di quella cercata
+ * (`urgent`) e una scelta legittima diventerebbe un'ambiguità da rifiutare.
+ */
+function matchTicketLookup(rows, value) {
+    const wanted = value.trim().toLowerCase();
+    const list = (Array.isArray(rows) ? rows : []);
+    const identified = list.filter((row) => typeof row.id === 'string' && row.id !== '');
+    const sameAs = (field) => typeof field === 'string' && field.trim().toLowerCase() === wanted;
+    const byCode = identified.filter((row) => sameAs(row.code));
+    return byCode.length > 0 ? byCode : identified.filter((row) => sameAs(row.label));
+}
+/**
+ * Il rifiuto di un riferimento che non si risolve: dice cos'è stato cercato, come si scrive il flag
+ * e quali voci esistono davvero — così scoprirle non costa un secondo comando.
+ */
+function ticketLookupErrorMessage(lookup, wanted, rows, { matched, isDefault }) {
+    const known = (Array.isArray(rows) ? rows : [])
+        .map((row) => (typeof row.code === 'string' && row.code !== '' ? row.code : String(row.label ?? '')))
+        .filter((name) => name !== '')
+        .join(', ');
+    const hint = `Pass ${lookup.flag} with a code, a label or an id${known === '' ? '' : ` — known: ${known}`}.`;
+    if (matched > 1)
+        return `${lookup.name} "${wanted}" matches ${matched} entries. ${hint}`;
+    if (isDefault) {
+        return `This organization has no ${lookup.name.toLowerCase()} "${wanted}", the default for a new ticket. ${hint}`;
+    }
+    return `${lookup.name} not found: "${wanted}". ${hint}`;
+}

package/dist/lib/ticket-report.d.ts

@@ -0,0 +1,7 @@
+/**
+ * Una versione del resoconto di lavorazione: intestazione coi metadati + il testo per intero
+ * (CYCL-31). Il corpo passa da `sanitizeMultiline` e non da `renderRecord`: quest'ultimo appiattisce
+ * ogni valore con `sanitize`, che trasforma gli a capo in spazi — su un resoconto di migliaia di
+ * caratteri significherebbe stamparlo come un muro di testo su una riga sola.
+ */
+export declare function renderReport(report: Record<string, unknown>): string;

package/dist/lib/ticket-report.js

@@ -0,0 +1,14 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.renderReport = renderReport;
+const output_1 = require("./output");
+/**
+ * Una versione del resoconto di lavorazione: intestazione coi metadati + il testo per intero
+ * (CYCL-31). Il corpo passa da `sanitizeMultiline` e non da `renderRecord`: quest'ultimo appiattisce
+ * ogni valore con `sanitize`, che trasforma gli a capo in spazi — su un resoconto di migliaia di
+ * caratteri significherebbe stamparlo come un muro di testo su una riga sola.
+ */
+function renderReport(report) {
+    const header = `── version ${(0, output_1.sanitize)(report.version ?? '?')} · ${(0, output_1.sanitize)(report.author ?? '?')} · ${(0, output_1.sanitize)(report.source ?? '-')} · ${(0, output_1.sanitize)(report.created_at ?? '-')}`;
+    return `${header}\n${(0, output_1.sanitizeMultiline)(report.body ?? '')}`;
+}