@bussolabs/closeyourit-cli 0.18.0 → 0.20.0

package/dist/commands/seo/pages.js

@@ -0,0 +1,44 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_1 = require("../../base");
+const output_1 = require("../../lib/output");
+class SeoPages extends base_1.BaseCommand {
+    static description = 'List the pages the SEO check has visited, with their state';
+    static examples = [
+        '<%= config.bin %> seo pages',
+        '<%= config.bin %> seo pages --project acme-site',
+        '<%= config.bin %> seo pages --url /prezzi --json',
+    ];
+    static flags = {
+        project: core_1.Flags.string({ char: 'p', description: 'Filter by project id (UUID) or key' }),
+        url: core_1.Flags.string({ description: 'Filter by URL (partial match)' }),
+        ...base_1.pageFlag,
+    };
+    async run() {
+        const { flags } = await this.parse(SeoPages);
+        const query = new URLSearchParams({ page: String(flags.page) });
+        if (flags.project)
+            query.set('project_id', await this.resolveProjectId(flags.project));
+        if (flags.url)
+            query.set('url', flags.url);
+        const res = await this.api.get(`/cli/v1/seo/pages?${query.toString()}`);
+        const pages = res.data ?? [];
+        if (!this.jsonEnabled()) {
+            this.log((0, output_1.renderTable)(['PROJECT', 'PATH', 'STATUS', 'TITLE', 'H1', 'WORDS'], pages.map((page) => {
+                const headings = Array.isArray(page.h1s) ? page.h1s : [];
+                return [
+                    String(page.project?.key ?? ''),
+                    String(page.path ?? page.url ?? ''),
+                    (0, output_1.statusCell)(page.status_code),
+                    String(page.title ?? '—'),
+                    // No h1 is the finding a green score never reports: it belongs in the column, not hidden.
+                    headings.length === 0 ? 'missing' : String(headings.length),
+                    String(page.word_count ?? 0),
+                ];
+            })));
+        }
+        return res;
+    }
+}
+exports.default = SeoPages;

package/dist/commands/seo/promote.d.ts

@@ -0,0 +1,9 @@
+import { BaseCommand } from '../../base';
+export default class SeoPromote extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static args: {
+        id: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    run(): Promise<unknown>;
+}

package/dist/commands/seo/promote.js

@@ -0,0 +1,22 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_1 = require("../../base");
+const output_1 = require("../../lib/output");
+class SeoPromote extends base_1.BaseCommand {
+    static description = 'Open a ticket from an SEO finding, evidence included';
+    static examples = ['<%= config.bin %> seo promote <issue-id>'];
+    static args = {
+        id: core_1.Args.string({ description: 'SEO finding id', required: true }),
+    };
+    async run() {
+        const { args } = await this.parse(SeoPromote);
+        // Idempotent server-side: promoting twice returns R422-SEO-001, never a second ticket.
+        const res = await this.api.put(`/cli/v1/seo/${args.id}/promotion`);
+        if (!this.jsonEnabled()) {
+            this.log((0, output_1.renderRecord)(res.data ?? {}));
+        }
+        return res;
+    }
+}
+exports.default = SeoPromote;

package/dist/commands/seo/reopen.d.ts

@@ -0,0 +1,9 @@
+import { BaseCommand } from '../../base';
+export default class SeoReopen extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static args: {
+        id: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    run(): Promise<unknown>;
+}

package/dist/commands/seo/reopen.js

@@ -0,0 +1,21 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_1 = require("../../base");
+class SeoReopen extends base_1.BaseCommand {
+    static description = 'Reopen an ignored SEO finding: you changed your mind';
+    static examples = ['<%= config.bin %> seo reopen <issue-id>'];
+    static args = {
+        id: core_1.Args.string({ description: 'SEO finding id', required: true }),
+    };
+    async run() {
+        const { args } = await this.parse(SeoReopen);
+        // Reopening is deleting the ignore, not a verb of its own — hence DELETE on the same resource.
+        const res = await this.api.delete(`/cli/v1/seo/${args.id}/ignore`);
+        if (!this.jsonEnabled()) {
+            this.log(`SEO finding ${args.id} reopened.`);
+        }
+        return res;
+    }
+}
+exports.default = SeoReopen;

package/dist/commands/seo/rescan.d.ts

@@ -0,0 +1,9 @@
+import { BaseCommand } from '../../base';
+export default class SeoRescan extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static flags: {
+        site: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+    };
+    run(): Promise<unknown>;
+}

package/dist/commands/seo/rescan.js

@@ -0,0 +1,21 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_1 = require("../../base");
+class SeoRescan extends base_1.BaseCommand {
+    static description = 'Check a site now, without waiting for the next scheduled run';
+    static examples = ['<%= config.bin %> seo rescan --site <site-id>'];
+    // Required: a check always targets one site (a project can declare one per environment).
+    static flags = {
+        site: core_1.Flags.string({ description: 'Site id (from `seo-sites list`)', required: true }),
+    };
+    async run() {
+        const { flags } = await this.parse(SeoRescan);
+        const res = await this.api.post('/cli/v1/seo/rescan', { site_id: flags.site });
+        if (!this.jsonEnabled()) {
+            this.log(`Check queued for site ${flags.site}.`);
+        }
+        return res;
+    }
+}
+exports.default = SeoRescan;

package/dist/commands/seo/show.d.ts

@@ -0,0 +1,9 @@
+import { BaseCommand } from '../../base';
+export default class SeoShow extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static args: {
+        id: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    run(): Promise<unknown>;
+}

package/dist/commands/seo/show.js

@@ -0,0 +1,35 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_1 = require("../../base");
+const output_1 = require("../../lib/output");
+/** A nested object of the payload, or `{}` when the field is absent. */
+function nested(data, key) {
+    const value = data[key];
+    return value && typeof value === 'object' ? value : {};
+}
+class SeoShow extends base_1.BaseCommand {
+    static description = 'Show a single SEO finding, with the evidence behind it';
+    static examples = ['<%= config.bin %> seo show <issue-id>'];
+    static args = {
+        id: core_1.Args.string({ description: 'SEO finding id', required: true }),
+    };
+    async run() {
+        const { args } = await this.parse(SeoShow);
+        const res = await this.api.get(`/cli/v1/seo/${args.id}`);
+        if (!this.jsonEnabled()) {
+            const data = res.data ?? {};
+            // Evidence in a block of its own: it is the reason the finding exists, and burying it among
+            // the metadata means reopening the browser to see what was actually found.
+            this.log((0, output_1.renderRecord)({ ...data, evidence: undefined }));
+            this.log((0, output_1.section)('Evidence', '🔎'));
+            this.log((0, output_1.renderRecord)(nested(data, 'evidence')));
+            this.log((0, output_1.section)('Site', '🌐'));
+            this.log((0, output_1.renderRecord)(nested(data, 'site')));
+            this.log((0, output_1.section)('Project', '📦'));
+            this.log((0, output_1.renderRecord)(nested(data, 'project')));
+        }
+        return res;
+    }
+}
+exports.default = SeoShow;

package/dist/commands/seo-sites/create.d.ts

@@ -0,0 +1,14 @@
+import { BaseCommand } from '../../base';
+export default class SeoSitesCreate extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static flags: {
+        url: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+        environment: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+        frequency: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'max-pages': import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'no-sitemap': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        project: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+    };
+    run(): Promise<unknown>;
+}

package/dist/commands/seo-sites/create.js

@@ -0,0 +1,43 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_1 = require("../../base");
+const output_1 = require("../../lib/output");
+class SeoSitesCreate extends base_1.BaseCommand {
+    static description = 'Declare a site to keep an eye on (one per project and environment)';
+    static examples = [
+        '<%= config.bin %> seo-sites create --project acme-site --url https://www.acme.example --environment <env-id>',
+        '<%= config.bin %> seo-sites create -p acme-site --url https://www.acme.example --environment <env-id> --frequency weekly --max-pages 50',
+    ];
+    static flags = {
+        ...base_1.projectFlag,
+        url: core_1.Flags.string({ description: 'Public address the visit starts from', required: true }),
+        environment: core_1.Flags.string({ description: 'Environment id (must be declared by the project)', required: true }),
+        frequency: core_1.Flags.string({ description: 'How often to check it: daily (default) or weekly', options: ['daily', 'weekly'] }),
+        'max-pages': core_1.Flags.integer({ description: 'How many pages to visit at most each run' }),
+        'no-sitemap': core_1.Flags.boolean({ description: "Don't read the sitemap: start from the home page and follow links only" }),
+    };
+    async run() {
+        const { flags } = await this.parse(SeoSitesCreate);
+        const projectId = await this.resolveProjectId(flags.project);
+        const body = {
+            project_id: projectId,
+            environment_id: flags.environment,
+            base_url: flags.url,
+        };
+        if (flags.frequency !== undefined)
+            body.frequency = flags.frequency;
+        if (flags['max-pages'] !== undefined)
+            body.max_pages = flags['max-pages'];
+        if (flags['no-sitemap'])
+            body.follow_sitemap = false;
+        const res = await this.api.post('/cli/v1/seo_sites', body);
+        if (!this.jsonEnabled()) {
+            this.log((0, output_1.renderRecord)(res.data ?? {}));
+            // The first run starts on creation: without saying so, an empty list looks like a failure.
+            this.log('First check queued.');
+        }
+        return res;
+    }
+}
+exports.default = SeoSitesCreate;

package/dist/commands/seo-sites/delete.d.ts

@@ -0,0 +1,9 @@
+import { BaseCommand } from '../../base';
+export default class SeoSitesDelete extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static args: {
+        id: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    run(): Promise<unknown>;
+}

package/dist/commands/seo-sites/delete.js

@@ -0,0 +1,21 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_1 = require("../../base");
+class SeoSitesDelete extends base_1.BaseCommand {
+    static description = 'Stop watching a site and remove its pages and findings';
+    static examples = ['<%= config.bin %> seo-sites delete <site-id>'];
+    static args = {
+        id: core_1.Args.string({ description: 'Site id', required: true }),
+    };
+    async run() {
+        const { args } = await this.parse(SeoSitesDelete);
+        const res = await this.api.delete(`/cli/v1/seo_sites/${args.id}`);
+        if (!this.jsonEnabled()) {
+            // Pages and findings go with it: to keep the history, pause the site instead (`update --disable`).
+            this.log(`Site ${args.id} removed, with its pages and findings.`);
+        }
+        return res;
+    }
+}
+exports.default = SeoSitesDelete;

package/dist/commands/seo-sites/list.d.ts

@@ -0,0 +1,10 @@
+import { BaseCommand } from '../../base';
+export default class SeoSitesList extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static flags: {
+        page: import("@oclif/core/lib/interfaces").OptionFlag<number, import("@oclif/core/lib/interfaces").CustomOptions>;
+        project: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+    };
+    run(): Promise<unknown>;
+}

package/dist/commands/seo-sites/list.js

@@ -0,0 +1,38 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_1 = require("../../base");
+const output_1 = require("../../lib/output");
+class SeoSitesList extends base_1.BaseCommand {
+    static description = 'List the sites under SEO check';
+    static examples = [
+        '<%= config.bin %> seo-sites list',
+        '<%= config.bin %> seo-sites list --project acme-site --json',
+    ];
+    static flags = {
+        project: core_1.Flags.string({ char: 'p', description: 'Filter by project id (UUID) or key' }),
+        ...base_1.pageFlag,
+    };
+    async run() {
+        const { flags } = await this.parse(SeoSitesList);
+        const query = new URLSearchParams({ page: String(flags.page) });
+        if (flags.project)
+            query.set('project_id', await this.resolveProjectId(flags.project));
+        const res = await this.api.get(`/cli/v1/seo_sites?${query.toString()}`);
+        const sites = res.data ?? [];
+        if (!this.jsonEnabled()) {
+            this.log((0, output_1.renderTable)(['ID', 'PROJECT', 'ENV', 'SITE', 'EVERY', 'OPEN', 'LAST CHECK'], sites.map((site) => [
+                String(site.id ?? ''),
+                String(site.project?.key ?? ''),
+                String(site.environment?.code ?? ''),
+                String(site.base_url ?? ''),
+                String(site.frequency ?? ''),
+                String(site.open_issues_count ?? 0),
+                // A failed run must never look like a successful one: the reason wins over the date.
+                site.last_error ? (0, output_1.statusCell)(`failed: ${site.last_error}`) : String(site.last_audited_at ?? 'never'),
+            ])));
+        }
+        return res;
+    }
+}
+exports.default = SeoSitesList;

package/dist/commands/seo-sites/update.d.ts

@@ -0,0 +1,17 @@
+import { BaseCommand } from '../../base';
+export default class SeoSitesUpdate extends BaseCommand {
+    static description: string;
+    static examples: string[];
+    static args: {
+        id: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    static flags: {
+        url: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        frequency: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'max-pages': import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        sitemap: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        enable: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        disable: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+    };
+    run(): Promise<unknown>;
+}

package/dist/commands/seo-sites/update.js

@@ -0,0 +1,45 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_1 = require("../../base");
+const output_1 = require("../../lib/output");
+class SeoSitesUpdate extends base_1.BaseCommand {
+    static description = 'Change how a site is checked (project and environment are its identity: they never change)';
+    static examples = [
+        '<%= config.bin %> seo-sites update <site-id> --frequency weekly',
+        '<%= config.bin %> seo-sites update <site-id> --max-pages 200 --disable',
+    ];
+    static args = {
+        id: core_1.Args.string({ description: 'Site id', required: true }),
+    };
+    static flags = {
+        url: core_1.Flags.string({ description: 'Public address the visit starts from' }),
+        frequency: core_1.Flags.string({ description: 'How often to check it: daily or weekly', options: ['daily', 'weekly'] }),
+        'max-pages': core_1.Flags.integer({ description: 'How many pages to visit at most each run' }),
+        sitemap: core_1.Flags.boolean({ description: 'Read the sitemap', allowNo: true }),
+        enable: core_1.Flags.boolean({ description: 'Resume the checks', exclusive: ['disable'] }),
+        disable: core_1.Flags.boolean({ description: 'Pause the checks (the site and its findings stay)', exclusive: ['enable'] }),
+    };
+    async run() {
+        const { args, flags } = await this.parse(SeoSitesUpdate);
+        const body = {};
+        if (flags.url !== undefined)
+            body.base_url = flags.url;
+        if (flags.frequency !== undefined)
+            body.frequency = flags.frequency;
+        if (flags['max-pages'] !== undefined)
+            body.max_pages = flags['max-pages'];
+        if (flags.sitemap !== undefined)
+            body.follow_sitemap = flags.sitemap;
+        if (flags.enable)
+            body.enabled = true;
+        if (flags.disable)
+            body.enabled = false;
+        const res = await this.api.put(`/cli/v1/seo_sites/${args.id}`, body);
+        if (!this.jsonEnabled()) {
+            this.log((0, output_1.renderRecord)(res.data ?? {}));
+        }
+        return res;
+    }
+}
+exports.default = SeoSitesUpdate;

package/dist/commands/servers/ignored-containers.d.ts

@@ -0,0 +1,15 @@
+import { BaseCommand } from '../../base';
+export default class ServersIgnoredContainers extends BaseCommand {
+    static args: {
+        id: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    static description: string;
+    static examples: string[];
+    static flags: {
+        clear: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        set: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+    };
+    run(): Promise<unknown>;
+    private render;
+    private split;
+}

package/dist/commands/servers/ignored-containers.js

@@ -0,0 +1,51 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_1 = require("../../base");
+// CYRA-519 — i nomi che su una macchina NON sono servizi (container di compilazione e test, che
+// nascono e muoiono a ogni lavorazione) si scrivevano solo dalla pagina web: chi lavora da terminale
+// doveva aprire il browser per una lista di due righe.
+//
+// Senza flag il comando legge; --set sostituisce l'elenco, --clear lo svuota. Sostituisce e non
+// accoda: un elenco di due nomi si riscrive per intero, e "aggiungi" andrebbe letto prima.
+class ServersIgnoredContainers extends base_1.BaseCommand {
+    static args = {
+        id: core_1.Args.string({ description: 'Server id', required: true }),
+    };
+    static description = 'Read or set the container names this server does not treat as services';
+    static examples = [
+        '<%= config.bin %> servers ignored-containers <server-id>',
+        '<%= config.bin %> servers ignored-containers <server-id> --set buildkit,runner',
+        '<%= config.bin %> servers ignored-containers <server-id> --clear',
+    ];
+    static flags = {
+        clear: core_1.Flags.boolean({ description: 'Remove every ignored container name', exclusive: ['set'] }),
+        set: core_1.Flags.string({ description: 'Comma-separated names to ignore (replaces the list)' }),
+    };
+    async run() {
+        const { args, flags } = await this.parse(ServersIgnoredContainers);
+        const path = `/cli/v1/servers/${encodeURIComponent(args.id)}`;
+        if (flags.set === undefined && !flags.clear) {
+            const res = await this.api.get(path);
+            if (!this.jsonEnabled())
+                this.log(this.render(res.data));
+            return res;
+        }
+        const patterns = flags.clear ? [] : this.split(flags.set ?? '');
+        const res = await this.api.put(path, { ignored_container_patterns: patterns });
+        if (!this.jsonEnabled())
+            this.log(this.render(res.data));
+        return res;
+    }
+    render(data) {
+        const patterns = data?.ignored_container_patterns ?? [];
+        return patterns.length === 0 ? 'No ignored container names' : patterns.join('\n');
+    }
+    split(value) {
+        return value
+            .split(',')
+            .map((name) => name.trim())
+            .filter(Boolean);
+    }
+}
+exports.default = ServersIgnoredContainers;

package/dist/commands/tickets/comment.d.ts

@@ -1,13 +1,24 @@
+import { type Interfaces } from '@oclif/core';
 import { BaseCommand } from '../../base';
 export default class TicketsComment extends BaseCommand {
     static args: {
-        id: import("@oclif/core/lib/interfaces").Arg<string, Record<string, unknown>>;
+        id: Interfaces.Arg<string, Record<string, unknown>>;
     };
     static description: string;
     static examples: string[];
     static flags: {
-        body: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
-        project: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+        body: Interfaces.OptionFlag<string, Interfaces.CustomOptions>;
+        project: Interfaces.OptionFlag<string, Interfaces.CustomOptions>;
     };
+    /** Ticket e progetto come li ha scritti l'utente: il suggerimento su R422 li ripete tali e quali. */
+    private target?;
     run(): Promise<unknown>;
+    /**
+     * Il commento troppo lungo va scritto altrove, e il messaggio del server dice solo «use the work
+     * report»: prima di CYCL-31 quel campo dal terminale non esisteva e l'unica uscita era aprire il
+     * sito. Ora esiste, quindi al messaggio si accoda il comando pronto da eseguire, con il ticket e
+     * il progetto già dentro. Codice e testo del server restano intatti — anche in `--json`, dove il
+     * suggerimento viaggia in coda al `message` e il `code` resta `R422-COMMENT-002`.
+     */
+    protected catch(error: Interfaces.CommandError): Promise<unknown>;
 }

package/dist/commands/tickets/comment.js

@@ -2,7 +2,13 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const core_1 = require("@oclif/core");
 const base_1 = require("../../base");
+const api_1 = require("../../lib/api");
+const limits_1 = require("../../lib/limits");
 const output_1 = require("../../lib/output");
+// Rifiuto del server per un commento oltre il limite: il messaggio rimanda «al work report». Il
+// codice resta quello del backend, propagato tale e quale (vedi errors/error-codes.ts) — quello che
+// si aggiunge è il comando che quel campo lo scrive davvero (CYCL-31).
+const COMMENT_TOO_LONG = 'R422-COMMENT-002';
 class TicketsComment extends base_1.BaseCommand {
     static args = {
         id: core_1.Args.string({ description: 'Ticket id', required: true }),
@@ -13,10 +19,18 @@ class TicketsComment extends base_1.BaseCommand {
     ];
     static flags = {
         ...base_1.projectFlag,
-        body: core_1.Flags.string({ description: 'Comment body', required: true }),
+        // Il tetto è dichiarato qui e la misura resta al server: il suo rifiuto porta il comando pronto
+        // per scrivere altrove il testo che non ci sta (vedi `catch`), che un controllo locale toglierebbe.
+        body: core_1.Flags.string({
+            description: `Comment body (max ${limits_1.LENGTH_LIMITS.comment} characters — longer text goes in the work report)`,
+            required: true,
+        }),
     };
+    /** Ticket e progetto come li ha scritti l'utente: il suggerimento su R422 li ripete tali e quali. */
+    target;
     async run() {
         const { args, flags } = await this.parse(TicketsComment);
+        this.target = { id: args.id, project: flags.project };
         const projectId = await this.resolveProjectId(flags.project);
         const res = await this.api.post(`/cli/v1/projects/${projectId}/tickets/${args.id}/comments`, { body: flags.body });
         if (!this.jsonEnabled()) {
@@ -25,5 +39,19 @@ class TicketsComment extends base_1.BaseCommand {
         }
         return res;
     }
+    /**
+     * Il commento troppo lungo va scritto altrove, e il messaggio del server dice solo «use the work
+     * report»: prima di CYCL-31 quel campo dal terminale non esisteva e l'unica uscita era aprire il
+     * sito. Ora esiste, quindi al messaggio si accoda il comando pronto da eseguire, con il ticket e
+     * il progetto già dentro. Codice e testo del server restano intatti — anche in `--json`, dove il
+     * suggerimento viaggia in coda al `message` e il `code` resta `R422-COMMENT-002`.
+     */
+    async catch(error) {
+        if (error instanceof api_1.ApiRequestError && error.code === COMMENT_TOO_LONG && this.target) {
+            const hint = `Write it with: ${this.config.bin} tickets report ${this.target.id} --project ${this.target.project} --body-file <file>`;
+            return super.catch(new api_1.ApiRequestError(error.status, error.code, `${error.message} ${hint}`, error.details));
+        }
+        return super.catch(error);
+    }
 }
 exports.default = TicketsComment;

package/dist/commands/tickets/create.d.ts

@@ -3,6 +3,12 @@ export default class TicketsCreate extends BaseCommand {
     static description: string;
     static examples: string[];
     static flags: {
+        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>;
+        '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>;
         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>;
@@ -13,12 +19,6 @@ export default class TicketsCreate extends BaseCommand {
         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>;
         title: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
         kind: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
         project: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;

package/dist/commands/tickets/create.js

@@ -2,26 +2,45 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const core_1 = require("@oclif/core");
 const base_1 = require("../../base");
+const limits_1 = require("../../lib/limits");
 const output_1 = require("../../lib/output");
 const ticket_body_1 = require("../../lib/ticket-body");
+const ticket_lookup_1 = require("../../lib/ticket-lookup");
 class TicketsCreate extends base_1.BaseCommand {
-    static description = 'Create a ticket';
+    static description = 'Create a ticket (status defaults to open, priority to medium; both take a code or a label)';
     static examples = [
         '<%= config.bin %> tickets create --project acme-api --title "Checkout fails"',
         '<%= config.bin %> tickets create -p acme-api --title "Login bug" --kind bug --step-given "logged out" --step-when "I submit" --step-then "it 500s"',
         '<%= config.bin %> tickets create -p acme-api --title "Checkout" --scenarios \'[{"title":"Happy","step_given":"logged in","step_when":"I pay","step_then":"order created"}]\' --condition "confirmation email is sent"',
+        '<%= config.bin %> tickets create -p acme-api --title "Payment retry" --status in_progress --priority High',
     ];
     static flags = {
         ...base_1.projectFlag,
-        title: core_1.Flags.string({ description: 'Ticket title', required: true }),
+        title: core_1.Flags.string({ description: `Ticket title (max ${limits_1.LENGTH_LIMITS.title} characters)`, required: true }),
         kind: core_1.Flags.string({ description: 'Ticket kind (bug, story, task, epic)', default: 'bug', options: ['bug', 'story', 'task', 'epic'] }),
         ...ticket_body_1.ticketBodyFlags,
+        // Su un ticket che nasce, omettere stato e priorità non li lascia com'erano: applica i predefiniti.
+        ...ticket_lookup_1.newTicketLookupFlags,
     };
     async run() {
         const { flags } = await this.parse(TicketsCreate);
+        // Un ticket nuovo non ha niente da salvaguardare: il tetto vale pieno, e i campi troppo lunghi
+        // si leggono tutti in questo unico rifiuto invece che uno per tentativo (CYCL-32).
+        const tooLong = (0, limits_1.tooLongMessage)((0, ticket_body_1.ticketLengthChecks)(flags));
+        if (tooLong)
+            this.error(tooLong, { exit: 2 });
         const projectId = await this.resolveProjectId(flags.project);
         const body = { title: flags.title, kind: flags.kind };
         (0, ticket_body_1.applyTicketBody)(body, flags);
+        // Stato e priorità il server li pretende ("Status must exist and Priority must exist") mentre
+        // l'aiuto li dava per opzionali: chi non li indica prende i predefiniti, chi li indica li scrive
+        // come li legge (`open`, `In Progress`) invece di andare a cercarne l'UUID (CYCL-34).
+        const [statusId, priorityId] = await Promise.all([
+            this.resolveTicketLookupId(ticket_lookup_1.TICKET_STATUS, flags.status),
+            this.resolveTicketLookupId(ticket_lookup_1.TICKET_PRIORITY, flags.priority),
+        ]);
+        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:');