@timekast/factory 2.1.0 → 2.2.0

package/dist/commands/ticket.js

@@ -4,8 +4,10 @@
  * A derivative documents a kit defect as a markdown file under `project/factory/`
  * (the `fx-factory-tickets` convention). Until now that file was a dead end: it sat
  * in the repo until somebody opened the Factory by hand. This command delivers it as
- * a GitHub issue in the Factory repo and stamps the resulting URL back into the
- * ticket's canonical `**GitHub issue:**` slot.
+ * a GitHub issue in the Factory repo and then DELETES the local file: it was only a
+ * draft, and from here on the issue is the one source of state (open / closed,
+ * labels). A local copy with its own status only drifted from GitHub, and a second
+ * `push` of it opened a duplicate issue (`fx-factory-tickets §6`).
  *
  * THREE THINGS THIS COMMAND DELIBERATELY DOES NOT DO:
  *
@@ -34,7 +36,7 @@
  * preflight degrades to a note + `exit 0` — the same `not-configured` contract
  * `factory backlog push` has without `BACKLOG_API_KEY`, never a stack trace.
  */
-import { existsSync, readFileSync, realpathSync, statSync, writeFileSync } from 'node:fs';
+import { existsSync, readFileSync, realpathSync, statSync, unlinkSync } from 'node:fs';
 import path from 'node:path';
 import { execa } from 'execa';
 import prompts from 'prompts';
@@ -163,36 +165,6 @@ export function resolveTicketPath(rootDir, filePath) {
     return realTarget;
 }
 // ─────────────────────────────────────────────────────────────────────────────
-// Stamping (`fx-factory-tickets §6.2`)
-// ─────────────────────────────────────────────────────────────────────────────
-/** Canonical header field the delivered URL is stamped into. It is the ONLY field a tool writes. */
-const GITHUB_ISSUE_FIELD = '**GitHub issue:**';
-const ESTADO_FIELD = '**Estado:**';
-/**
- * Write `url` into the ticket's `**GitHub issue:**` slot, which is born holding an
- * em-dash and sits immediately after `**Estado:**`. Stamping the URL does NOT touch
- * `Estado` — delivering is not triaging (`fx-factory-tickets §6.2`).
- *
- * A ticket written before the convention has no such field; it is inserted right
- * after `Estado` rather than dropped, so the ticket stops being a dead end either
- * way. With no header block at all, nothing is changed and the caller warns.
- */
-export function stampGitHubIssueUrl(content, url) {
-    const stamped = `${GITHUB_ISSUE_FIELD} ${url}`;
-    const lines = content.split('\n');
-    const fieldIdx = lines.findIndex((line) => line.startsWith(GITHUB_ISSUE_FIELD));
-    if (fieldIdx >= 0) {
-        lines[fieldIdx] = stamped;
-        return { content: lines.join('\n'), changed: true };
-    }
-    const estadoIdx = lines.findIndex((line) => line.startsWith(ESTADO_FIELD));
-    if (estadoIdx >= 0) {
-        lines.splice(estadoIdx + 1, 0, stamped);
-        return { content: lines.join('\n'), changed: true };
-    }
-    return { content, changed: false };
-}
-// ─────────────────────────────────────────────────────────────────────────────
 // Terminal safety
 // ─────────────────────────────────────────────────────────────────────────────
 /**
@@ -271,7 +243,7 @@ async function defaultConfirm(message) {
  *
  * Order matters and is asserted by the unit test: containment → read → size →
  * (`--dry-run` stops here) → gh session → publish gate → create → verify
- * assignee/label → stamp. Every guard that can reject runs BEFORE `gh` is ever
+ * assignee/label → delete the draft. Every guard that can reject runs BEFORE `gh` is ever
  * spawned.
  */
 export async function runTicketPush(filePath, flags = parseTicketPushFlags([]), rootDir = detectRepo(process.cwd()).repoRoot ?? process.cwd(), deps = {}) {
@@ -387,8 +359,8 @@ export async function runTicketPush(filePath, flags = parseTicketPushFlags([]),
     if (!url) {
         console.log(`✅ Issue creado en ${FACTORY_REPO}.`);
         console.log(stdout.trim());
-        console.warn('⚠️  No se pudo leer la URL de la salida de `gh`, así que el ticket local no se estampó. ' +
-            'Copia la URL de arriba al campo `**GitHub issue:**`.');
+        console.warn('⚠️  No se pudo leer la URL de la salida de `gh`, así que el borrador local NO se borró. ' +
+            'Revisa el issue de arriba y borra el archivo a mano (no lo vuelvas a entregar: abriría otro issue).');
         return;
     }
     console.log(`✅ Issue creado: ${url}`);
@@ -407,22 +379,19 @@ export async function runTicketPush(filePath, flags = parseTicketPushFlags([]),
     //    permission is `read` — so the two notification mechanisms of this channel can
     //    fail at once, without a word, unless we look.
     await verifyIssueFields(url, flags, labelDropped, assigneeDropped);
-    // 6. Stamp the URL back. If this fails, the URL is already on screen (step above):
-    //    never leave the dev without the URL of an issue that already exists.
-    const stamp = stampGitHubIssueUrl(body, url);
-    if (!stamp.changed) {
-        console.warn(`⚠️  \`${shown}\` no tiene el campo \`${GITHUB_ISSUE_FIELD}\` ni \`${ESTADO_FIELD}\`, así que no se estampó. ` +
-            'Agrega la URL a mano (`fx-factory-tickets §6.2`).');
-        return;
-    }
+    // 6. The draft is delivered: delete it. From here the issue is the only source of
+    //    state. If this fails, the URL is already on screen (step above) — never leave
+    //    the dev without the URL of an issue that already exists, and never fail a
+    //    delivery that succeeded.
     try {
-        (deps.writeFile ?? ((p, c) => writeFileSync(p, c, 'utf8')))(target, stamp.content);
-        console.log(`   URL estampada en \`${shown}\` (el \`Estado\` no cambia: entregar no es triar).`);
+        (deps.removeFile ?? unlinkSync)(target);
+        console.log(`   Borré el borrador \`${shown}\`: el estado vive en el issue. ` +
+            'Commitea el borrado (`docs(factory): …`).');
     }
     catch (error) {
-        console.warn(`⚠️  El issue existe (${url}) pero \`${shown}\` no se pudo actualizar: ` +
+        console.warn(`⚠️  El issue existe (${url}) pero \`${shown}\` no se pudo borrar: ` +
             `${error instanceof Error ? error.message : String(error)}\n` +
-            `   Copia la URL al campo \`${GITHUB_ISSUE_FIELD}\` a mano.`);
+            '   Bórralo a mano; entregarlo otra vez abriría un issue duplicado.');
     }
 }
 /**

package/dist/index.js

@@ -191,8 +191,8 @@ Sobre \`backlog\`:
 
 Sobre \`ticket\`:
   \`ticket push <archivo>\` entrega un factory-ticket de \`project/factory/\` al repo
-  del Factory como un issue de GitHub, y estampa la URL de vuelta en el campo
-  \`**GitHub issue:**\` del ticket (el \`Estado\` no cambia: entregar no es triar).
+  del Factory como un issue de GitHub y, entregado, BORRA el archivo local: era un
+  borrador, y el estado vive en el issue (commitea el borrado con \`docs(factory): …\`).
   Cero credenciales nuevas: usa tu propia sesión de \`gh\`; sin sesión avisa y sale 0.
   El cuerpo se publica ÍNTEGRO en un repo compartido, así que hay compuerta:
     --dry-run        Imprime destino, asignado, etiqueta y título; no crea nada

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@timekast/factory",
-  "version": "2.1.0",
+  "version": "2.2.0",
   "description": "Public, thin CLI to bootstrap and maintain TimeKast Factory derived projects.",
   "type": "module",
   "license": "UNLICENSED",