@worca/app 1.2.0 → 1.3.0

package/src/core/pipeline-delete.mjs

@@ -169,7 +169,7 @@ export async function archivePipeline({ projectDir = null, key = null, workspace
   try {
     const branch = state?.branch?.feature;
     if (branch && state?.projectDir && await hasGh()) {
-      const pr = await findPrForBranch({ projectDir: state.projectDir, head: branch });
+      const pr = await findPrForBranch({ projectDir: state.projectDir, head: branch, prUrl: row.pr_url || null });
       if (pr) persistPrState(row.id, pr);
     }
   } catch (e) {

package/src/core/plugin-store.mjs

@@ -28,6 +28,7 @@ import { loadAgentRegistry } from './agent-registry.mjs';
 import { pluginModelSecretStatus } from './plugin-models.mjs';
 import { referencedPluginModels } from './config.mjs';
 import { clearBindingsForPlugin } from './source-bindings.mjs';
+import { parseFrontmatter } from './frontmatter.mjs';
 
 const execFileP = promisify(execFile);
 const defaultExec = (cmd, args, opts = {}) =>
@@ -55,15 +56,6 @@ function insideDir(dir, rel) {
   return resolve(root, rel).startsWith(root + sep);
 }
 
-/** Private copy of workflows.mjs:66-77 parseFrontmatterTools (module-private there). */
-function frontmatterTools(text) {
-  const m = /^---\s*\n([\s\S]*?)\n---/.exec(text);
-  if (!m) return [];
-  const line = m[1].split(/\r?\n/).find((l) => /^tools\s*:/.test(l));
-  if (!line) return [];
-  return line.replace(/^tools\s*:/, '').split(',').map((s) => s.trim()).filter(Boolean);
-}
-
 /** The "Will install" consent inventory (spec §6.1, design §9.4): agents +
  *  their frontmatter tools, sources + their secret fields, models with their
  *  base-URL value VERBATIM (a model env can redirect all API traffic — the
@@ -89,7 +81,7 @@ export function buildInstallInventory(versionDir) {
         if (typeof af === 'string' && af.trim() && insideDir(aDir, af.trim())) mdFile = af.trim();
       } catch { /* unreadable sidecar: fall back to the sibling */ }
       let tools = [];
-      try { tools = frontmatterTools(readFileSync(join(aDir, mdFile), 'utf8')); } catch { /* md missing */ }
+      try { tools = parseFrontmatter(readFileSync(join(aDir, mdFile), 'utf8'))?.tools ?? []; } catch { /* md missing */ }
       agents.push({ key, tools });
     }
   }

package/src/core/preflight.mjs

@@ -322,9 +322,8 @@ export async function detectToolsPerProject(projectDirs) {
  * required version >= 2.1.220) rather than failing the run: R1(a)/(c) still hold via
  * the cwd and ancestor mechanisms, and R1(b) is reported as DEGRADED.
  *
- * Deliberately NO `addDir` field: `--add-dir` is not probed because no shipped
- * feature uses it (§5.3 — it needs `CLAUDE_CODE_ADDITIONAL_DIRECTORIES_CLAUDE_MD=1`
- * to carry memory at all, E2), and a probe field no consumer reads is dead weight.
+ * No `addDir` preflight probe: Ask Worca's memory mount passes `--add-dir`
+ * (claude-runner `addDirs`), verified by the 2026-09-13 rules probe, not by preflight.
  *
  * Never throws: a missing binary / hung process resolves to
  * `{ mcpConfig: false, version: null }`, which the caller treats the same as an old

package/src/core/projects.mjs

@@ -144,16 +144,31 @@ export async function addProject(input) {
 }
 
 /**
- * Remove a project by name (case-insensitive). Absent name is a no-op.
+ * Remove a project by name (case-insensitive). Absent name is a no-op. Also prunes any metrics
+ * worktree that belonged to this project's repository, waiting up to ~10s for it: the wait is
+ * best-effort — a worktree still held by a running flush is skipped and swept on the next start.
  * @param {string} name
  * @returns {Promise<Array<{key:string, name:string, path:string, exists:boolean}>>}
  */
 export async function removeProject(name) {
   const key = (typeof name === 'string' ? name : '').trim();
+  let removedPath = null;
   if (key) {
     tx(() => {
+      const row = prepare('SELECT path FROM projects WHERE name = ? COLLATE NOCASE').get(key);
+      removedPath = row ? row.path : null;
       prepare('DELETE FROM projects WHERE name = ? COLLATE NOCASE').run(key);
     });
   }
+  if (removedPath) {
+    // Dynamic import: metrics/sync.mjs imports this module (worcaHome, listProjects).
+    try {
+      const { pruneMetricsWorktreesFor } = await import('./metrics/sync.mjs');
+      // Bounded: each removal waits for the slug lock, and a running flush can hold it for minutes.
+      // The DELETE request must not hang on that; the prune finishes in the background.
+      const prune = pruneMetricsWorktreesFor(removedPath).catch(() => []);
+      await Promise.race([prune, new Promise((r) => setTimeout(r, 10_000).unref?.())]);
+    } catch { /* best-effort: a leftover worktree is harmless and re-pruned by git */ }
+  }
   return listProjects();
 }