feinai 0.7.1 → 0.8.0

package/CHANGELOG.md

@@ -3,6 +3,31 @@
 All notable changes to this project will be documented in this file.
 Format follows [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
+## [0.8.0] - 2026-09-02
+
+### Changed
+
+- **Dashboard rediseñado: el detalle de spec/task pasa de modal overlay a páginas navegables.** Rutas por hash, linkeables y compartibles: `#/`, `#/specs`, `#/tasks`, `#/spec/:id`, `#/task/:id`, `#/new/spec`, `#/new/task`. El botón atrás del navegador funciona.
+- **Eliminados los scrolls anidados.** El documento es el único contenedor con scroll: se quitaron los `max-height` de `.list` (600px), `.markdown` (400px), `.modal` (90vh), la lista de archivos del worktree (120px) y los payloads de eventos (120px). Sobrevive sólo el dropdown de búsqueda (es un menú) y el scroll horizontal de bloques de código y tablas.
+- Breadcrumbs jerárquicos (`feinai / Specs / SPEC-012 / TASK-012-B`) con el crumb actual teñido por el estado real de la entidad.
+- Los filtros viven en la URL (`#/tasks?status=pending`); los stats del header linkean a su lista filtrada.
+- `prompt()` / `confirm()` nativos reemplazados por un diálogo dentro de la página.
+
+### Added
+
+- Soporte de tablas markdown en el renderer del dashboard (antes se veían como texto crudo con pipes).
+- Los IDs en `blocked_by` son links a la task correspondiente.
+
+### Fixed
+
+- `feinai server --down` sólo detiene el servidor registrado de este proyecto, no cualquier proceso en el puerto.
+- Lifecycle de specs huérfanas (SPEC-124) y normalización de estados a inglés.
+
+## [0.7.2] - 2026-06-22
+
+### Fixed
+- `isPortInUse` / `pidsOnPort` / `portOfPid` usaban `result.exitCode` en vez de `result.status` (API de `child_process.spawnSync`), causando que `findFreePort` nunca detectara puertos ocupados
+
 ## [0.7.1] - 2026-06-22
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "feinai",
-  "version": "0.7.1",
+  "version": "0.8.0",
   "description": "Task & spec manager for AI agents — parallel worktrees, live dashboard, SDD skills",
   "type": "module",
   "bin": {

package/skills/feinai-implement/SKILL.md

@@ -64,14 +64,30 @@ Run the gates defined in the task (`quality_gates`). If the task does not specif
 
 Gates pass:
 ```bash
-# From the worktree:
+# From the worktree — include feinai.db in the task commit:
+feinai done <TASK-ID> --result "gates ✓"
+feinai git add .feinai/feinai.db
+feinai git commit --amend --no-edit
+
 feinai git push origin HEAD:main
+```
 
+If push is rejected (another agent pushed first — conflict on `.feinai/feinai.db`):
+```bash
+# Accept origin's db (it's the authoritative state), re-apply done, re-push:
+feinai git fetch origin main
+feinai git checkout origin/main -- .feinai/feinai.db
+feinai done <TASK-ID> --result "gates ✓"
+feinai git add .feinai/feinai.db
+feinai git commit --amend --no-edit
+feinai git push origin HEAD:main
+```
+
+Push succeeds:
+```bash
 # From the repo root:
 feinai git worktree remove .worktrees/<TASK-ID>
 feinai git complete
-
-feinai done <TASK-ID> --result "gates ✓"
 ```
 
 Gates fail → follow "If something fails".

package/src/cli.ts

@@ -53,9 +53,11 @@ import {
   clearServerState,
   findFreePort,
   repairServerState,
+  killOrphansByProjectDir,
+  type OrphanKillResult,
 } from "./server-state";
 
-const VERSION = "0.7.1";
+const VERSION = "0.8.0";
 
 interface ParsedArgs {
   positional: string[];
@@ -77,6 +79,7 @@ const FLAG_NAMES = new Set([
   "daemon",
   "yes",
   "clear-blocked-by",
+  "no-preflight",
 ]);
 const MULTI_OPTIONS = new Set(["package", "gate", "blocked-by"]);
 
@@ -257,7 +260,7 @@ SERVER:
     --port <N>                      Port (default: 8272 — TASC on phone keypad)
     --host <addr>                   Bind host (default: 127.0.0.1)
   server --daemon / -d              Start detached (no job control noise)
-  server --down                     Stop the running feinai server (by port)
+  server --down                     Stop this project's recorded feinai server
 
 GLOBAL FLAGS:
   --json                            Output as JSON
@@ -777,61 +780,48 @@ function cmdSpec(rest: string[], args: ParsedArgs, format: OutputFormat): void {
 }
 
 async function cmdServer(args: ParsedArgs): Promise<void> {
-  const port = Number(args.options.port ?? "8272");
-
-  // --down: stop the project's recorded server, fallback to default port
+  // --down: stop this project's recorded server AND any orphan servers that
+  // share the project directory, so stale/untracked processes are cleaned up.
   if (args.flags["down"]) {
-    const db = findDbPath() ? openDb() : null;
-    let targetPort = port;
-    let targetPids: number[] = [];
-
-    if (db) {
-      // Try recorded server first
-      repairServerState(db);
-      const record = readServerState(db);
-      if (record) {
-        targetPort = record.port;
-        targetPids = [record.pid];
-      }
-    }
-
-    if (Number.isNaN(targetPort) || targetPort < 1 || targetPort > 65535) {
-      console.error("Error: --port must be a valid port number");
-      if (db) db.close();
-      process.exit(1);
-    }
-
-    // If no recorded pids, fall back to lsof probe on target port
-    if (targetPids.length === 0) {
-      const result = Bun.spawnSync(["lsof", "-ti", `tcp:${targetPort}`]);
-      targetPids = new TextDecoder()
-        .decode(result.stdout)
-        .trim()
-        .split("\n")
-        .filter(Boolean)
-        .map((s) => Number(s))
-        .filter((n) => !isNaN(n));
+    if (!findDbPath()) {
+      console.error(
+        "Error: no .feinai/feinai.db found. Cannot stop server without project context.",
+      );
+      process.exit(2);
     }
 
-    if (targetPids.length === 0) {
-      console.log(`No process found listening on port ${targetPort}.`);
-      if (db) db.close();
-      return;
-    }
+    const dbPath = findDbPath()!;
+    const db = openDb();
+    repairServerState(db);
+    const record = readServerState(db);
+    const killed: OrphanKillResult[] = [];
 
-    for (const pid of targetPids) {
+    if (record) {
       try {
-        process.kill(Number(pid), "SIGTERM");
-        console.log(`Stopped feinai server (PID ${pid}).`);
+        process.kill(record.pid, "SIGTERM");
+        killed.push({ pid: record.pid, port: record.port });
       } catch {
-        console.error(`Failed to kill PID ${pid}.`);
+        console.error(
+          `Failed to stop feinai server (PID ${record.pid}, port ${record.port}).`,
+        );
       }
+      clearServerState(db);
     }
+    db.close();
 
-    // Clear the row after stopping
-    if (db) {
-      clearServerState(db);
-      db.close();
+    // Also kill any unregistered servers whose CWD matches this project.
+    // The DB lives at <projectDir>/.feinai/feinai.db, so go up two levels.
+    const projectDir = dirname(dirname(dbPath));
+    const orphans = killOrphansByProjectDir(projectDir);
+    killed.push(...orphans);
+
+    if (killed.length === 0) {
+      console.log("No running feinai server found for this project.");
+    } else {
+      const summary = killed
+        .map((k) => `PID ${k.pid}${k.port ? ` (port ${k.port})` : ""}`)
+        .join(", ");
+      console.log(`Stopped feinai server(s): ${summary}`);
     }
     return;
   }
@@ -844,17 +834,21 @@ async function cmdServer(args: ParsedArgs): Promise<void> {
 
   const db = ensureDb();
   const host = args.options.host ?? "127.0.0.1";
-
-  // Pre-flight consistency check: repair stale records, reject if valid server exists
-  repairServerState(db);
-  const existing = readServerState(db);
-  if (existing) {
-    console.error(
-      `feinai server is already running for this project at http://127.0.0.1:${existing.port}`,
-    );
-    console.error(`Stop it first with: feinai server --down --port ${existing.port}`);
-    db.close();
-    process.exit(1);
+  const noPreflight = args.flags["no-preflight"];
+
+  // Pre-flight consistency check: repair stale records, reject if valid server exists.
+  // Skipped when the child is launched by the daemon parent (it already validated).
+  if (!noPreflight) {
+    repairServerState(db);
+    const existing = readServerState(db);
+    if (existing) {
+      console.error(
+        `feinai server is already running for this project at http://127.0.0.1:${existing.port}`,
+      );
+      console.error(`Stop it first with: feinai server --down`);
+      db.close();
+      process.exit(1);
+    }
   }
 
   // Determine port: if --port is given, use it; otherwise auto-increment from 8272
@@ -889,14 +883,25 @@ async function cmdServer(args: ParsedArgs): Promise<void> {
     } else {
       childArgs = [process.execPath, ...process.argv.slice(1).filter(noDaemon)];
     }
-    // Pass the resolved port so the child doesn't re-resolve (which could pick a different port)
-    if (!args.options.port) {
-      childArgs.push("--port", String(resolvedPort));
+    // Force the resolved port and the internal no-preflight flag onto the child.
+    // Strip any existing --port so the child always uses the port we resolved here.
+    for (let i = 0; i < childArgs.length; ) {
+      if (childArgs[i] === "--port") {
+        childArgs.splice(i, 2);
+        continue;
+      }
+      i++;
     }
+    childArgs.push("--port", String(resolvedPort), "--no-preflight");
+
     const child = Bun.spawn(childArgs, { detached: true, stdio: ["ignore", "ignore", "ignore"] });
+
+    // The parent owns the DB record: write it immediately with the child's pid
+    // so the server is tracked before the parent exits.
+    writeServerState(db, resolvedPort, child.pid);
     child.unref();
     console.log(`feinai dashboard → http://${host}:${resolvedPort}`);
-    console.log(`Stop with: feinai server --down --port ${resolvedPort}`);
+    console.log(`Stop with: feinai server --down`);
     db.close();
     return;
   }
@@ -905,29 +910,27 @@ async function cmdServer(args: ParsedArgs): Promise<void> {
   const { startServer } = await import("./server");
   const server = startServer({ port: resolvedPort, host });
 
-  // Write the server state row after successful bind
-  writeServerState(db, resolvedPort, process.pid);
+  // Write the server state row after successful bind (only when the CLI is the
+  // actual server process; daemon children skip this because the parent wrote it).
+  if (!noPreflight) {
+    writeServerState(db, resolvedPort, process.pid);
+  }
   db.close();
 
   console.log(`feinai dashboard listening at ${server.url}`);
-  console.log(`Stop with: feinai server --down --port ${resolvedPort}`);
+  console.log(`Stop with: feinai server --down`);
 
-  // Keep process alive until SIGINT
-  process.on("SIGINT", () => {
+  const shutdown = () => {
     console.log("\nStopping server...");
     const cleanupDb = ensureDb();
     clearServerState(cleanupDb);
     cleanupDb.close();
     server.stop();
     process.exit(0);
-  });
+  };
 
-  // Keep process alive until SIGINT
-  process.on("SIGINT", () => {
-    console.log("\nStopping server...");
-    server.stop();
-    process.exit(0);
-  });
+  process.on("SIGINT", shutdown);
+  process.on("SIGTERM", shutdown);
 }
 
 function cmdPlan(rest: string[], args: ParsedArgs, format: OutputFormat): void {