@lamemind/loom-deck 0.73.0 → 0.73.1

package/dist/task-edit.js

@@ -3,7 +3,7 @@
 // isolata in `writeTaskEdit`, così la grammatica delle sostituzioni è testabile
 // senza toccare il filesystem.
 import { readFileSync, writeFileSync } from 'node:fs';
-import { TASK_ID_RE, findTaskFile, headerColumns } from './tasks.js';
+import { TASK_ID_RE, findTaskFile, overviewRows } from './tasks.js';
 // Il task file scrive la priorità per NOME (`- **Priority**: Med`), tasks.md la
 // scrive come GLIFO (colonna Pri). Due rappresentazioni dello stesso valore →
 // due mappe, entrambe keyed sul nome canonico di view.ts (unica fonte del rango).
@@ -83,10 +83,13 @@ export function initialDetail(current, prog, date = today()) {
  * toccate restano i token grezzi originali: nessun re-flow della tabella, il
  * diff resta di una riga. `ok:false` = id assente → il chiamante non scrive nulla.
  *
- * Le posizioni vengono dall'HEADER (`headerColumns`, stessa fonte di
- * `parseTasks`), mai da un indice cablato: qui si SCRIVE su disco, quindi una
- * colonna in più o in meno non produrrebbe una lettura sbagliata ma un titolo
- * sovrascritto col glifo. Header assente → `ok:false`, non si tira a indovinare.
+ * Le righe candidate vengono da `overviewRows`, lo stesso confine di
+ * `parseTasks`: solo dentro la Tasks Overview, con le colonne del suo HEADER e
+ * mai un indice cablato. Qui si SCRIVE su disco, quindi senza confine una riga
+ * cross-deps con lo stesso id (`| T07 | — | ✔️T08 |`) sopra la tabella verrebbe
+ * riscritta al posto della riga task, con Pri/Prog/titolo sopra le celle Parent e
+ * Cross Deps; e una colonna in più o in meno darebbe un titolo sovrascritto col
+ * glifo. Header assente → nessuna riga → `ok:false`, non si tira a indovinare.
  *
  * `desc` undefined ≠ `desc` vuota: la prima non tocca la cella (chiamata
  * legacy, solo pri/prog), la seconda la svuota davvero.
@@ -104,23 +107,14 @@ export function updateTasksMdRow(content, id, priGlyph, progGlyph, desc) {
     if (!TASK_ID_RE.test(id))
         return { content, ok: false };
     const lines = content.split('\n');
-    const cols = lines.reduce((found, l) => found ?? headerColumns(l), null);
-    if (!cols)
-        return { content, ok: false };
-    let ok = false;
-    const out = lines.map((line) => {
-        if (ok)
-            return line;
-        if (!line.trim().startsWith('|'))
-            return line;
+    for (const { index, line, id: rowId, cols } of overviewRows(lines)) {
+        if (rowId !== id)
+            continue;
         const cells = line.split('|');
-        if ((cells[1] ?? '').trim() !== id)
-            return line;
         // Serve la cella desc PIÙ la coda dopo il `|` finale: senza la coda lo
         // splice inserirebbe invece di sostituire.
         if (cells.length < cols.desc + 2)
-            return line; // riga malformata → non toccarla
-        ok = true;
+            continue; // riga malformata → non toccarla
         cells[cols.pri] = ` ${priGlyph} `;
         cells[cols.prog] = ` ${progGlyph} `;
         if (desc !== undefined) {
@@ -128,9 +122,10 @@ export function updateTasksMdRow(content, id, priGlyph, progGlyph, desc) {
             // ben formata) e va conservata, o la riga perderebbe il bordo destro.
             cells.splice(cols.desc, cells.length - cols.desc - 1, ` ${sanitizeCell(desc)} `);
         }
-        return cells.join('|');
-    });
-    return { content: ok ? out.join('\n') : content, ok };
+        lines[index] = cells.join('|');
+        return { content: lines.join('\n'), ok: true };
+    }
+    return { content, ok: false };
 }
 /**
  * Rende un testo scrivibile dentro una CELLA di tabella markdown. Due caratteri

package/dist/tasks.js

@@ -59,31 +59,60 @@ export function headerColumns(line) {
         return null;
     return { pri, prog, desc: prog + 1 };
 }
-// Estrae le righe `| Tnn | Pri | Prog | Task |` della Tasks Overview. Righe
-// header/separatore non matchano `^T\d+$` → scartate.
-//
-// Le righe si parsano solo DOPO aver incontrato l'header: senza indici derivati
-// non si tira a indovinare, la lista resta vuota. Un fallimento visibile invece
-// di una cella letta al posto sbagliato.
-export function parseTasks(content) {
-    const tasks = [];
+/**
+ * Le righe task della Tasks Overview: FONTE UNICA del confine della tabella,
+ * condivisa da `parseTasks` (lettura) e `updateTasksMdRow` (scrittura su disco),
+ * come `TASK_ID_RE` e `headerColumns` — due confini scritti in due posti
+ * divergerebbero, e quello dello scrittore è quello che fa danno.
+ *
+ * La tabella apre all'header (`headerColumns`) e chiude alla PRIMA riga che non
+ * inizia con `|`; un secondo header ne riapre una, con le proprie colonne. Il
+ * confine serve perché la prima cella `^T\d+$` non basta a riconoscere una riga
+ * task: le righe della tabella cross-deps dell'Execution Plan
+ * (`| T07 | — | ✔️T08 |`) hanno la stessa forma appena perdono il 🔒 davanti
+ * all'id, e senza confine diventavano task doppie con Pri = cella Parent e
+ * Prog = cella Cross Deps.
+ *
+ * Il titolo `## Tasks Overview` non entra nel criterio: il gate è l'header, così
+ * il deck non dipende dal nome della sezione. Il prezzo: una riga vuota o un
+ * commento DENTRO la tabella la chiude, e le righe sotto spariscono dal deck
+ * senza errore.
+ */
+export function overviewRows(lines) {
+    const rows = [];
     let cols = null;
-    for (const line of content.split('\n')) {
+    lines.forEach((line, index) => {
         const header = headerColumns(line);
         if (header) {
             cols = header;
-            continue;
+            return;
         }
         if (!cols)
-            continue;
+            return;
         const t = line.trim();
-        if (!t.startsWith('|'))
-            continue;
-        const cells = t.split('|').map((c) => c.trim());
+        if (!t.startsWith('|')) {
+            cols = null;
+            return;
+        }
+        // cells[0] = '' (prima del primo |): l'id è la seconda cella. Header e
+        // separatore non matchano `^T\d+$` → scartati.
+        const id = (t.split('|')[1] ?? '').trim();
+        if (TASK_ID_RE.test(id))
+            rows.push({ index, line, id, cols });
+    });
+    return rows;
+}
+// Estrae le righe `| Tnn | Pri | Prog | Task |` della Tasks Overview — il
+// confine della tabella sta in `overviewRows`.
+//
+// Le righe si parsano solo DOPO aver incontrato l'header: senza indici derivati
+// non si tira a indovinare, la lista resta vuota. Un fallimento visibile invece
+// di una cella letta al posto sbagliato.
+export function parseTasks(content) {
+    const tasks = [];
+    for (const { line, id, cols } of overviewRows(content.split('\n'))) {
         // cells[0] = '' (prima del primo |), cells[last] = '' (dopo l'ultimo |)
-        const id = cells[1];
-        if (!TASK_ID_RE.test(id))
-            continue;
+        const cells = line.trim().split('|').map((c) => c.trim());
         const pri = cells[cols.pri] ?? '';
         const prog = cells[cols.prog] ?? '';
         // desc = colonna finale; join per resistere a eventuali `|` nella descrizione.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lamemind/loom-deck",
-  "version": "0.73.0",
+  "version": "0.73.1",
   "description": "Deck TUI Ink per-progetto della famiglia loom: legge tasks.md e spawna sessioni Claude Code bound via LOOM_TASK",
   "type": "module",
   "bin": {