@lamemind/loom-deck 0.72.2 → 0.73.1

package/README.md

@@ -405,8 +405,8 @@ Apre una tab Ptyxis nella window attiva con `LOOM_TASK=T18 claude 'recap stato t
 
 I testi del catalogo stanno in `scripts/prompt-catalog` (formato `kind<TAB>template`, `{TASK}` interpolato), non dentro lo script: li legge anche il deck, per mostrare nel detail il prompt prima dello spawn. `--prompt` passa invece un testo **letterale** ed è esclusivo con `--prompt-kind` — sono due modi di dire la stessa cosa, e accettarli insieme costringerebbe a stabilire quale vince. Il testo viene quotato per la shell in-tab, apici singoli compresi: è l'unico ingresso di testo libero in una riga che una shell parsa.
 
-La tab porta anche `PTYXIS_PROFILE` forzata al profilo bindato al progetto nel registry (`bindings/claude`), letto da dconf: è la chiave con cui loom-compass associa a un progetto lo stato annunciato dagli hook (running/ask/done). Una tab `ptyxis --tab` nuda erediterebbe il profilo di default, e l'annuncio finirebbe keyed su un UUID che nessun progetto dichiara — pallino fermo su idle, senza alcun errore visibile. Override o disattivazione via `LOOM_DECK_STATE_PROFILE`
-(settata a vuoto → nessun annuncio); progetto non registrato → nessun prefisso.
+La tab porta anche `PTYXIS_PROFILE` forzata all'`id` del progetto, letto da `.claude/loom-works.json`: è la chiave con cui loom-compass associa a un progetto lo stato annunciato dagli hook (running/ask/done). Il nome della variabile è storico — non contiene l'UUID di un profilo Ptyxis. Una tab `ptyxis --tab` nuda porterebbe l'UUID del profilo di default, e l'annuncio finirebbe keyed su una chiave che nessun progetto dichiara — badge fermo, senza alcun errore visibile. Un `id` fuori da `^[A-Za-z0-9_-]+$` viene scartato con un avviso su stderr, perché il valore entra in una riga che una shell parsa. Override o disattivazione via `LOOM_DECK_STATE_PROFILE`
+(settata a vuoto → nessun annuncio); progetto senza file config → nessun prefisso.
 
 ## Sviluppo (TUI Ink)
 

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.72.2",
+  "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": {

package/scripts/deck-run

@@ -45,25 +45,19 @@
 # Serve a chi spawna deck-run e vuole mostrare l'invocazione vera senza
 # ricomporla per conto proprio; le diagnostiche restano tutte su stderr.
 #
-# Modalità spawn (LOOM_DECK_SPAWN_MODE):
-#   inline   → ptyxis --tab -- <cmd>            (default)  comando inline, LOOM_TASK diretta, zero dconf
-#   profile  → ptyxis --tab-with-profile=<UUID>            riusa il profilo, riscrive il custom-command via dconf
-#
-# La scelta fra le due è la Decisione aperta §9 della proposta: la modalità
-# "inline" è il default perché non muta stato dconf condiviso. Vedi la nota
-# di decisione nella task folder di T18.
+# Spawn: `ptyxis --tab -- bash -lc <cmd>`, comando inline sul profilo di default
+# di Ptyxis — LOOM_TASK e chiave di stato viaggiano dentro il comando, nessuna
+# scrittura dconf e nessun profilo dedicato.
 #
 # Env:
-#   LOOM_DECK_SPAWN_MODE   inline|profile          (default: inline)
-#   LOOM_DECK_PROFILE_UUID UUID profilo Ptyxis     (default: cc-host)
 #   LOOM_DECK_WORKDIR      dir di lavoro della tab (default: $PWD)
 #   LOOM_DECK_ENTER_PROMPT override del prompt iniziale, {TASK}=TaskID (vince sul kind, tranne `none`)
 #                          T161: inerte sugli spawn che arrivano dal deck TUI —
 #                          quelli passano SEMPRE `--prompt`, su cui non ha mai
 #                          vinto. Resta per le invocazioni a mano e per i test.
 #   LOOM_DECK_INTAB_CMD    override comando in-tab (default: la sessione CC; usato per i test)
-#   LOOM_DECK_STATE_PROFILE  PTYXIS_PROFILE annunciata a compass (default: bindings/claude
-#                            del progetto, letto da dconf; settata a vuoto = nessun annuncio)
+#   LOOM_DECK_STATE_PROFILE  PTYXIS_PROFILE annunciata a compass (default: `id` del file
+#                            config del progetto; settata a vuoto = nessun annuncio)
 #   LOOM_DECK_PERMISSION_MODE  override del permissionMode del file config (default: campo file, poi 'manual')
 #   LOOM_DECK_MODEL        modello quando --model non è passato (batte il catalogo, default: fable)
 #
@@ -89,7 +83,6 @@ FORK=0
 # la tab finisce nella finestra di un altro progetto, o in una a caso.
 #
 # Non è un quinto asse: non cambia cosa gira nella tab, solo dove la tab nasce.
-# Vale identico nei due rami di spawn (inline e profile).
 NEW_WINDOW=0
 TITLE_NOTE=""
 # Vuoto = flag non passato (≠ un default già risolto): serve a distinguere
@@ -272,8 +265,6 @@ if [[ $PROMPT_GIVEN -eq 1 && -n "$PROMPT_KIND" ]]; then
   exit 2
 fi
 
-MODE="${LOOM_DECK_SPAWN_MODE:-inline}"
-PROFILE_UUID="${LOOM_DECK_PROFILE_UUID:-5a36ae48df1c4d4882f43060e3e59656}"
 WORKDIR="${LOOM_DECK_WORKDIR:-$PWD}"
 
 # ── Titolo tab = label standard loom (matchabile da compass) ──────────────────
@@ -471,40 +462,34 @@ case "${MODEL:-}" in
 esac
 MODEL_FLAG="--model ${MODEL} "
 
-# ── Profilo di stato per compass ─────────────────────────────────────────────
+# ── Chiave di stato per compass ──────────────────────────────────────────────
 # Lo stato di una sessione (running/ask/done) viaggia verso compass via D-Bus
 # keyed su $PTYXIS_PROFILE (hook Claude → `compass <stato>`), e compass lo mappa
-# a un progetto SOLO se quell'UUID compare in `bindings/<surface>/profile` del
-# registry. Ma nel modo `inline` la tab nasce da `ptyxis --tab` nudo, che eredita
-# il profilo DEFAULT di Ptyxis: l'annuncio arriva keyed su un UUID che nessun
-# progetto dichiara → stato scartato, pallino fermo su idle per sempre. Il match
-# della FINESTRA non se ne accorge (passa dal titolo, non dal profilo), quindi il
-# progetto risulta presente ma senza stato — un guasto che non si manifesta come
-# errore, solo come pallino che non cambia mai.
-# Rimedio: forzare in-tab la PTYXIS_PROFILE del binding claude del progetto, così
-# l'annuncio è keyed sull'UUID che compass cerca davvero. Il profilo bindato NON
-# è usabile come profilo vero della tab (`--tab-with-profile`): ha
-# use-custom-command=true, che fa scartare il comando inline — vedi il ramo
-# `profile` sotto, che per questo deve riscrivere dconf.
+# a un progetto SOLO se quella chiave è l'`id` di un progetto del registry. La
+# tab nasce da `ptyxis --tab` nudo, e Ptyxis esporta da sé PTYXIS_PROFILE col
+# valore del profilo DEFAULT: senza un prefisso l'annuncio arriverebbe keyed su
+# un UUID che nessun progetto dichiara → stato scartato, badge fermo per sempre.
+# Il match della FINESTRA non se ne accorge (passa dal titolo, non dalla
+# chiave), quindi il progetto risulta presente ma senza stato — un guasto che
+# non si manifesta come errore, solo come badge che non cambia mai.
+# Rimedio: forzare in-tab PTYXIS_PROFILE=<id del progetto>. Il nome della
+# variabile è storico: non contiene l'UUID di un profilo Ptyxis, e nessuna
+# chiamata a Ptyxis la usa. È la stessa chiave che mette il bottone 🤖 di compass.
 #
-# `${VAR+set}`, non `${VAR:-}`: env settata anche a VUOTO = "salta il lookup"
-# (deterministico per i test, che altrimenti leggerebbero il dconf della macchina
-# su cui gira la suite). Non settata = risolvi dal registry.
-# Assente/non risolvibile → nessun prefisso: si degrada al comportamento di prima
-# (stato orfano) invece di rompere lo spawn.
+# `${VAR+set}`, non `${VAR:-}`: env settata anche a VUOTO = "nessun annuncio"
+# (deterministico per i test). Non settata = `id` del file config del progetto.
+# Assente/non risolvibile → nessun prefisso: si degrada a stato orfano invece di
+# rompere lo spawn.
 if [[ -n "${LOOM_DECK_STATE_PROFILE+set}" ]]; then
   STATE_PROFILE="${LOOM_DECK_STATE_PROFILE}"
 else
   STATE_PROFILE=""
-  if [[ -n "$_cfg" ]] && command -v dconf >/dev/null 2>&1; then
-    _pid="$(jq -r '.id // empty' "$_cfg" 2>/dev/null || true)"
-    if [[ -n "$_pid" ]]; then
-      STATE_PROFILE="$(dconf read "/org/lamemind/loom/projects/${_pid}/bindings/claude/profile" 2>/dev/null | tr -d "'" || true)"
-    fi
+  if [[ -n "$_cfg" ]]; then
+    STATE_PROFILE="$(jq -r '.id // empty' "$_cfg" 2>/dev/null || true)"
   fi
 fi
-# Whitelist come per la nota del titolo: il valore entra in `bash -lc "…"`, e un
-# registry è pur sempre un file editabile a mano. Un UUID Ptyxis è esadecimale;
+# Whitelist come per la nota del titolo: il valore entra in `bash -lc "…"`, e il
+# file config è editabile a mano. Lo schema non vincola la forma dell'`id`:
 # accetto l'alfabeto id-safe e scarto il resto invece di quotare.
 if [[ -n "$STATE_PROFILE" && ! "$STATE_PROFILE" =~ ^[A-Za-z0-9_-]+$ ]]; then
   echo "profilo di stato ignorato (caratteri non ammessi): '${STATE_PROFILE}'" >&2
@@ -688,33 +673,10 @@ IN_TAB_CMD="${LOOM_DECK_INTAB_CMD:-$_default_intab}"
 # vedono comunque quello che vedevano prima.
 printf 'LOOM_DECK_INTAB %s\n' "$IN_TAB_CMD"
 
-# Verbo di apertura (T162): tab nella finestra attiva, o finestra nuova. Una
-# variabile e non due rami duplicati per ogni modalità di spawn — `--new-window`
-# è l'unica differenza fra i due casi.
+# Verbo di apertura (T162): tab nella finestra attiva, o finestra nuova —
+# `--new-window` è l'unica differenza fra i due casi.
 if [[ $NEW_WINDOW -eq 1 ]]; then WIN_FLAG="--new-window"; else WIN_FLAG="--tab"; fi
 
-case "$MODE" in
-  inline)
-    # --tab: tab nella window attiva. -- <cmd>: comando inline nella tab.
-    # LOOM_TASK viaggia dentro il comando → nessuna scrittura dconf.
-    exec ptyxis "$WIN_FLAG" -d "$WORKDIR" -T "${TITLE}" -- bash -lc "$IN_TAB_CMD"
-    ;;
-  profile)
-    # --tab-with-profile: riusa il profilo (custom-command FISSO) → per iniettare
-    # LOOM_TASK+task va riscritto il custom-command al volo via dconf (stato condiviso).
-    # Con --new-window i due flag convivono: la finestra nasce e il profilo
-    # decide cosa gira nella sua prima tab.
-    dpath="/org/gnome/Ptyxis/Profiles/${PROFILE_UUID}/"
-    dconf write "${dpath}use-custom-command" true
-    dconf write "${dpath}custom-command" "'bash -lc \"${IN_TAB_CMD//\"/\\\"}\"'"
-    if [[ $NEW_WINDOW -eq 1 ]]; then
-      exec ptyxis --new-window --tab-with-profile="$PROFILE_UUID" -d "$WORKDIR" -T "${TITLE}"
-    else
-      exec ptyxis --tab-with-profile="$PROFILE_UUID" -d "$WORKDIR" -T "${TITLE}"
-    fi
-    ;;
-  *)
-    echo "LOOM_DECK_SPAWN_MODE ignoto: '$MODE' (usa 'inline' o 'profile')" >&2
-    exit 2
-    ;;
-esac
+# -- <cmd>: comando inline nella tab. LOOM_TASK e chiave di stato viaggiano
+# dentro il comando → nessuna scrittura dconf, nessun profilo dedicato.
+exec ptyxis "$WIN_FLAG" -d "$WORKDIR" -T "${TITLE}" -- bash -lc "$IN_TAB_CMD"