okstra 0.73.0 → 0.75.0

package/runtime/validators/validate_improvement_report.py

@@ -23,6 +23,7 @@ from okstra_ctl.improvement_lenses import (
     ABSOLUTE_CANDIDATE_CAP,
     SOURCE_WORKERS,
 )
+from okstra_ctl.md_table import split_pipe_row
 
 
 _VERDICT_TOKENS = ("candidates-ready", "no-candidates", "blocked")
@@ -57,7 +58,7 @@ def _read_section_table(body: str, heading: str) -> list[list[str]]:
         s = line.strip()
         if not s.startswith("|") or not s.endswith("|"):
             continue
-        cells = [_CELL_ANCHOR_RE.sub("", c).strip() for c in s.strip("|").split("|")]
+        cells = [_CELL_ANCHOR_RE.sub("", c).strip() for c in split_pipe_row(s)]
         if all(set(c) <= set("-: ") for c in cells):
             continue
         rows.append(cells)

package/src/install.mjs

@@ -167,6 +167,34 @@ async function copyTreeIfChanged(srcRoot, dstRoot, opts) {
   return { copied, skipped, missingSource };
 }
 
+// Single-file variant of copyTreeIfChanged, used for the lead `okstra` skill
+// whose source is one file (agents/SKILL.md) rather than a skill directory.
+async function copyFileIfChanged(src, dst, opts) {
+  const { refresh = false, dryRun = false, mode } = opts ?? {};
+  let srcBuf;
+  try {
+    srcBuf = await fs.readFile(src);
+  } catch {
+    return { copied: 0, skipped: 0, missingSource: true };
+  }
+  let needsCopy = refresh;
+  if (!needsCopy) {
+    try {
+      const [srcHash, dstHash] = await Promise.all([hashFile(src), hashFile(dst)]);
+      needsCopy = srcHash !== dstHash;
+    } catch {
+      needsCopy = true;
+    }
+  }
+  if (!needsCopy) return { copied: 0, skipped: 1, missingSource: false };
+  if (dryRun) {
+    process.stdout.write(`[dry-run] copy ${src} -> ${dst}\n`);
+    return { copied: 1, skipped: 0, missingSource: false };
+  }
+  await writeFileAtomic(dst, srcBuf, mode);
+  return { copied: 1, skipped: 0, missingSource: false };
+}
+
 async function ensureSymlink(target, linkPath, opts) {
   const { dryRun = false } = opts ?? {};
   try {
@@ -244,7 +272,8 @@ async function installLinkMode(repoPath, paths, opts) {
   }
 
   const skillResult = await installSkillsLink(repoAbs, { dryRun, quiet });
-  await writeSkillsManifest(paths.home, skillResult.installed, { dryRun });
+  const leadSkillResult = await installLeadSkillLink(repoAbs, { dryRun, quiet });
+  await writeSkillsManifest(paths.home, [...skillResult.installed, ...leadSkillResult], { dryRun });
 
   const agentResult = await installAgentsLink(repoAbs, { dryRun, quiet });
   await writeAgentsManifest(paths.home, agentResult.installed, { dryRun });
@@ -487,6 +516,43 @@ async function installSkillsLink(repoAbs, opts) {
   return { installed: names };
 }
 
+// The lead `okstra` skill source is agents/SKILL.md (frontmatter `name: okstra`);
+// it doubles as the lead agent contract, so it lives under agents/ rather than
+// skills/. Claude Code only discovers skills at ~/.claude/skills/<name>/SKILL.md,
+// so it must be installed there as its own skill dir — otherwise install ships
+// every okstra-* helper skill but not the lead skill, and edits to agents/SKILL.md
+// never reach the running lead.
+const LEAD_SKILL_NAME = "okstra";
+
+async function installLeadSkillCopy(runtimeRoot, opts) {
+  const { refresh, dryRun, quiet } = opts;
+  const src = join(runtimeRoot, "agents", "SKILL.md");
+  const dst = join(CLAUDE_SKILLS_DIR, LEAD_SKILL_NAME, "SKILL.md");
+  const r = await copyFileIfChanged(src, dst, { refresh, dryRun, mode: 0o644 });
+  if (r.missingSource) {
+    if (!quiet) process.stdout.write("  skills/okstra: runtime/agents/SKILL.md missing — skipped\n");
+    return [];
+  }
+  if (!quiet) {
+    process.stdout.write(`  skills/okstra: ${r.copied ? "copied" : "unchanged"} -> ${dst}\n`);
+  }
+  return [LEAD_SKILL_NAME];
+}
+
+async function installLeadSkillLink(repoAbs, opts) {
+  const { dryRun, quiet } = opts;
+  const src = join(repoAbs, "agents", "SKILL.md");
+  if (!(await fileExists(src))) {
+    if (!quiet) process.stdout.write("  skills/okstra: <repo>/agents/SKILL.md missing — skipped\n");
+    return [];
+  }
+  const dst = join(CLAUDE_SKILLS_DIR, LEAD_SKILL_NAME, "SKILL.md");
+  if (!dryRun) await fs.mkdir(join(CLAUDE_SKILLS_DIR, LEAD_SKILL_NAME), { recursive: true });
+  const action = await ensureSymlink(src, dst, { dryRun });
+  if (!quiet) process.stdout.write(`  skills/okstra/SKILL.md: ${action}\n`);
+  return [LEAD_SKILL_NAME];
+}
+
 function parseInstallArgs(args) {
   const result = {
     dryRun: false,
@@ -623,7 +689,12 @@ export async function runInstall(args) {
   }
 
   const skillResult = await installSkillsCopy(runtimeRoot, opts);
-  await writeSkillsManifest(paths.home, skillResult.installed, { dryRun: opts.dryRun });
+  const leadSkillResult = await installLeadSkillCopy(runtimeRoot, opts);
+  await writeSkillsManifest(
+    paths.home,
+    [...skillResult.installed, ...leadSkillResult],
+    { dryRun: opts.dryRun },
+  );
 
   const agentResult = await installAgentsCopy(runtimeRoot, opts);
   await writeAgentsManifest(paths.home, agentResult.installed, { dryRun: opts.dryRun });

package/src/uninstall.mjs

@@ -18,6 +18,7 @@ const BIN_ENTRYPOINTS = [
 ];
 
 const FALLBACK_SKILL_NAMES = [
+  "okstra",
   "okstra-setup",
   "okstra-brief",
   "okstra-run",