okstra 0.101.1 → 0.102.1

package/runtime/validators/validate-run.py

@@ -1140,6 +1140,8 @@ PLANNING_REQUIRED_SECTIONS = (
     "Rollback",
     "Requirement Coverage",
     "Plan Body Verification",
+    "Cross-Project Dependencies",
+    "Decision Drafts",
 )
 
 # §5.7 implementation deliverables — substring scan against report body.
@@ -1152,6 +1154,7 @@ IMPLEMENTATION_REQUIRED_SECTIONS = (
     "Validation Evidence",
     "Verifier Results",
     "Rollback Verification",
+    "Manual User Test (Draft)",
     "Routing Recommendation",
 )
 
@@ -1163,6 +1166,7 @@ FINAL_VERIFICATION_REQUIRED_SECTIONS = (
     "Validation Evidence",
     "Read-only Command Log",
     "Conditional Acceptance Conditions",
+    "Manual User Test Results",
     "Routing Recommendation",
 )
 
@@ -1483,8 +1487,12 @@ def validate_final_report_data(report_path: Path, failures: list[str]) -> None:
 
     _validate_rationale_evidence(data, failures)
 
-    if (data.get("header") or {}).get("taskType") == "final-verification":
+    task_type = (data.get("header") or {}).get("taskType")
+    if task_type == "final-verification":
         _validate_final_verification_consistency(data, failures)
+    elif task_type == "implementation-planning":
+        _validate_implementation_planning_cross_project(data, failures)
+        _validate_implementation_planning_decision_drafts(data, failures)
 
 
 # path:line (foo.service.ts:268), report ID (C-001 / F-013 / G-crit-002 / RC-1),
@@ -1525,6 +1533,67 @@ def _validate_rationale_evidence(data: dict, failures: list[str]) -> None:
         )
 
 
+def _validate_implementation_planning_cross_project(data: dict, failures: list[str]) -> None:
+    """타 프로젝트 의존을 DM 행(`kind == 'cross-project'`)으로 선언했다면
+    `crossProjectDependencies` 에 `direction == 'upstream-precondition'` 행이
+    반드시 있어야 한다 — 실제 선행 필수 의존이 soft `recommendedNextSteps`
+    추천으로 새어나가지 못하게 강제한다. 각 XP 행의 필드 비-빈은 스키마가
+    보장하고, 이 검사는 cross-project DM 신호가 있을 때 XP 행이 *존재*하는지를 본다.
+    """
+    ip = data.get("implementationPlanning")
+    if not isinstance(ip, dict):
+        return
+    dm_rows = ip.get("dependencyMigrationRisk") or []
+    has_cross_project_dm = any(
+        isinstance(r, dict) and r.get("kind") == "cross-project" for r in dm_rows
+    )
+    if not has_cross_project_dm:
+        return
+    xp_rows = ip.get("crossProjectDependencies") or []
+    has_upstream = any(
+        isinstance(r, dict) and r.get("direction") == "upstream-precondition"
+        for r in xp_rows
+    )
+    if not has_upstream:
+        failures.append(
+            "final-report data.json: a dependencyMigrationRisk row has "
+            "kind='cross-project' but crossProjectDependencies carries no "
+            "direction='upstream-precondition' entry. Record the cross-project "
+            "dependency as a mandatory precondition (concrete requiredWork / "
+            "verificationSignal / howToStart) — not a soft Recommended Next Step."
+        )
+
+
+def _validate_implementation_planning_decision_drafts(data: dict, failures: list[str]) -> None:
+    """`decisionDrafts` 가 비어있지 않으면 어느 stage 의 stepwiseExecution step 이
+    `.okstra/decisions/` 파일을 생성하는 materialization step 을 포함해야 한다.
+    프로파일 Decision-record evaluation 의 "approved plan stepwise MUST include
+    Create .okstra/decisions/<NNNN>-<slug>.md" 를 실제 강제한다. draft 존재 자체가
+    trigger 이므로 self-contained 하다.
+    """
+    ip = data.get("implementationPlanning")
+    if not isinstance(ip, dict):
+        return
+    if not (ip.get("decisionDrafts") or []):
+        return
+    has_materialization = any(
+        ".okstra/decisions/" in (step.get("files") or "")
+        or ".okstra/decisions/" in (step.get("action") or "")
+        for stage in (ip.get("stages") or [])
+        if isinstance(stage, dict)
+        for step in (stage.get("stepwiseExecution") or [])
+        if isinstance(step, dict)
+    )
+    if not has_materialization:
+        failures.append(
+            "final-report data.json: implementationPlanning.decisionDrafts is "
+            "non-empty but no stage's stepwiseExecution creates a "
+            "`.okstra/decisions/<NNNN>-<slug>.md` file. The approved plan must "
+            "materialize each decision draft via a stepwise step (profile "
+            "Decision-record evaluation)."
+        )
+
+
 def _validate_final_verification_consistency(data: dict, failures: list[str]) -> None:
     """Enforce verdict ↔ blocker/condition/routing consistency on the
     final-verification data.json (SSOT). The schema guarantees field SHAPE;

package/src/commands/lifecycle/install.mjs

@@ -7,16 +7,16 @@ import { resolvePaths } from "../../lib/paths.mjs";
 import { fileExists } from "../../lib/proc.mjs";
 import { buildRuntimeManifest, RUNTIMES_MANIFEST_REL } from "../../lib/runtime-manifest.mjs";
 import { normalizeRuntimeRequest, resolveRuntime } from "../../lib/runtime-resolver.mjs";
-import { OBSOLETE_INTERNAL_SKILL_NAMES, USER_SKILL_NAMES } from "../../lib/skill-catalog.mjs";
+import { OBSOLETE_SKILL_NAMES, USER_SKILL_NAMES } from "../../lib/skill-catalog.mjs";
 
 const SKILLS_MANIFEST_REL = "installed-skills.json";
 const AGENTS_MANIFEST_REL = "installed-agents.json";
 const USER_HOME = homedir();
 const CLAUDE_HOME = join(USER_HOME, ".claude");
-const CODEX_HOME = join(USER_HOME, ".codex");
+const AGENT_HOME = join(USER_HOME, ".agent");
 const OMP_AGENT_HOME = join(USER_HOME, ".omp", "agent");
 const CLAUDE_SKILLS_DIR = join(CLAUDE_HOME, "skills");
-const CODEX_SKILLS_DIR = join(CODEX_HOME, "skills");
+const AGENT_SKILLS_DIR = join(AGENT_HOME, "skills");
 const OMP_SKILLS_DIR = join(OMP_AGENT_HOME, "skills");
 const CLAUDE_AGENTS_DIR = join(CLAUDE_HOME, "agents");
 
@@ -82,8 +82,8 @@ Effect (copy mode):
   ${"$HOME"}/.okstra/prompts       <- runtime/prompts  (lead contracts + coding-preflight pack)
   ${"$HOME"}/.okstra/templates/settings.local.json <- runtime/templates/reports/settings.template.json
   ${"$HOME"}/.claude/skills/<name> <- skills when ${"$HOME"}/.claude exists
-  ${"$HOME"}/.codex/skills/<name>  <- skills when ${"$HOME"}/.codex exists
-  ${"$HOME"}/.omp/agent/skills/<name> <- skills fallback when Claude/Codex homes are absent
+  ${"$HOME"}/.agent/skills/<name>  <- skills when ${"$HOME"}/.agent exists
+  ${"$HOME"}/.omp/agent/skills/<name> <- skills fallback when Claude/Agent homes are absent
   ${"$HOME"}/.claude/agents/<worker>.md <- worker agents when ${"$HOME"}/.claude exists
   ${"$HOME"}/.okstra/installed-runtimes.json <- install request/resolution manifest
   ${"$HOME"}/.okstra/installed-skills.json   <- target-aware manifest of installed skills
@@ -96,7 +96,7 @@ Effect (link mode):
   ${"$HOME"}/.okstra/prompts          -> <repo>/prompts
   ${"$HOME"}/.okstra/templates/settings.local.json -> <repo>/templates/reports/settings.template.json
   ${"$HOME"}/.claude/skills/<name>    -> <repo>/skills/<name> when ${"$HOME"}/.claude exists
-  ${"$HOME"}/.codex/skills/<name>     -> <repo>/skills/<name> when ${"$HOME"}/.codex exists
+  ${"$HOME"}/.agent/skills/<name>     -> <repo>/skills/<name> when ${"$HOME"}/.agent exists
   ${"$HOME"}/.omp/agent/skills/<name> -> <repo>/skills/<name> fallback
   ${"$HOME"}/.claude/agents/<worker>.md -> <repo>/agents/workers/<worker>.md when ${"$HOME"}/.claude exists
   ${"$HOME"}/.okstra/dev-link         <- <repo> path stamp
@@ -401,12 +401,12 @@ async function installLinkMode(repoPath, paths, opts) {
 
 async function detectSkillTargets() {
   const targets = [];
-  const [hasClaude, hasCodex] = await Promise.all([dirExists(CLAUDE_HOME), dirExists(CODEX_HOME)]);
+  const [hasClaude, hasAgent] = await Promise.all([dirExists(CLAUDE_HOME), dirExists(AGENT_HOME)]);
   if (hasClaude) {
     targets.push({ provider: "claude", home: CLAUDE_HOME, skillsDir: CLAUDE_SKILLS_DIR });
   }
-  if (hasCodex) {
-    targets.push({ provider: "codex", home: CODEX_HOME, skillsDir: CODEX_SKILLS_DIR });
+  if (hasAgent) {
+    targets.push({ provider: "agent", home: AGENT_HOME, skillsDir: AGENT_SKILLS_DIR });
   }
   if (targets.length === 0 && (await dirExists(OMP_AGENT_HOME))) {
     targets.push({ provider: "omp", home: OMP_AGENT_HOME, skillsDir: OMP_SKILLS_DIR });
@@ -637,7 +637,7 @@ async function installSkillsCopy(runtimeRoot, target, opts) {
     copied += r.copied;
     skipped += r.skipped;
   }
-  const pruned = await pruneObsoleteInternalSkills(target, opts);
+  const pruned = await pruneObsoleteSkills(target, opts);
   if (!quiet) {
     process.stdout.write(
       `  ${target.provider} skills: copied=${copied} skipped=${skipped} pruned=${pruned} -> ${target.skillsDir}/ (${USER_SKILL_NAMES.length} skills)\n`,
@@ -656,19 +656,18 @@ async function installSkillsLink(repoAbs, target, opts) {
     const action = await ensureSymlink(src, dst, { dryRun });
     if (!quiet) process.stdout.write(`  ${target.provider} skills/${name}: ${action}\n`);
   }
-  const pruned = await pruneObsoleteInternalSkills(target, opts);
+  const pruned = await pruneObsoleteSkills(target, opts);
   if (!quiet && pruned > 0) {
     process.stdout.write(`  ${target.provider} obsolete internal skills: pruned=${pruned}\n`);
   }
   return { ...target, installed: [...USER_SKILL_NAMES] };
 }
 
-// Remove former internal skill directories left by older installs. Exact-name
-// only (OBSOLETE_INTERNAL_SKILL_NAMES) — those contracts now ship as runtime
-// resources under ~/.okstra/prompts/, not as agent skills.
-async function pruneObsoleteInternalSkills(target, opts) {
+// Remove former skill directories left by older installs. Exact-name only
+// (OBSOLETE_SKILL_NAMES) — never wildcard-delete okstra-*.
+async function pruneObsoleteSkills(target, opts) {
   let pruned = 0;
-  for (const name of OBSOLETE_INTERNAL_SKILL_NAMES) {
+  for (const name of OBSOLETE_SKILL_NAMES) {
     const path = join(target.skillsDir, name);
     if (!(await pathEntryExists(path))) continue;
     pruned += 1;
@@ -683,7 +682,7 @@ async function pruneObsoleteInternalSkills(target, opts) {
 
 async function installSkillTargetsCopy(runtimeRoot, targets, opts) {
   if (targets.length === 0) {
-    if (!opts.quiet) process.stdout.write("  skills: no Claude, Codex, or OMP agent home found — skipped\n");
+    if (!opts.quiet) process.stdout.write("  skills: no Claude, Agent, or OMP agent home found — skipped\n");
     return [];
   }
   return await Promise.all(targets.map((target) => installSkillsCopy(runtimeRoot, target, opts)));
@@ -691,7 +690,7 @@ async function installSkillTargetsCopy(runtimeRoot, targets, opts) {
 
 async function installSkillTargetsLink(repoAbs, targets, opts) {
   if (targets.length === 0) {
-    if (!opts.quiet) process.stdout.write("  skills: no Claude, Codex, or OMP agent home found — skipped\n");
+    if (!opts.quiet) process.stdout.write("  skills: no Claude, Agent, or OMP agent home found — skipped\n");
     return [];
   }
   return await Promise.all(targets.map((target) => installSkillsLink(repoAbs, target, opts)));

package/src/commands/lifecycle/uninstall.mjs

@@ -15,6 +15,8 @@ const FALLBACK_SKILL_NAMES = [
   "okstra-memory",
   "okstra-inspect",
   "okstra-schedule",
+  "okstra-container-build",
+  "okstra-container",
   "okstra-context-loader",
   "okstra-team-contract",
   "okstra-convergence",
@@ -38,15 +40,15 @@ const FALLBACK_AGENT_NAMES = [
 
 const USER_HOME = homedir();
 const CLAUDE_HOME = join(USER_HOME, ".claude");
-const CODEX_HOME = join(USER_HOME, ".codex");
+const AGENT_HOME = join(USER_HOME, ".agent");
 const OMP_AGENT_HOME = join(USER_HOME, ".omp", "agent");
 const CLAUDE_SKILLS_DIR = join(CLAUDE_HOME, "skills");
-const CODEX_SKILLS_DIR = join(CODEX_HOME, "skills");
+const AGENT_SKILLS_DIR = join(AGENT_HOME, "skills");
 const OMP_SKILLS_DIR = join(OMP_AGENT_HOME, "skills");
 const CLAUDE_AGENTS_DIR = join(CLAUDE_HOME, "agents");
 const FALLBACK_SKILL_ROOTS = [
   CLAUDE_SKILLS_DIR,
-  CODEX_SKILLS_DIR,
+  AGENT_SKILLS_DIR,
   OMP_SKILLS_DIR,
 ];
 const SKILLS_MANIFEST_REL = "installed-skills.json";
@@ -61,7 +63,7 @@ Usage:
                                 installed-skills.json, installed-agents.json} AND
                                 every skill target recorded in
                                 installed-skills.json
-                                (~/.claude/skills, ~/.codex/skills, or
+                                (~/.claude/skills, ~/.agent/skills, or
                                 ~/.omp/agent/skills). Claude worker agents are
                                 removed from ~/.claude/agents when installed.
                                 Missing manifests fall back to hard-coded
@@ -73,7 +75,7 @@ Usage:
                                 (<PROJECT>/.claude/settings.local.json symlink)
                                 are NOT touched — remove them manually if needed.
   okstra uninstall --purge      Remove the entire ~/.okstra directory AND
-                                okstra skills in Claude/Codex/OMP homes AND
+                                okstra skills in Claude/Agent/OMP homes AND
                                 ~/.claude/agents/<*-worker.md> (DESTRUCTIVE).
                                 Requires -y or an interactive confirmation
   okstra uninstall --dry-run    Print the plan without touching disk
@@ -137,7 +139,7 @@ export async function runUninstall(args) {
   if (opts.purge) {
     if (!opts.yes && !opts.dryRun) {
       const ok = await promptConfirm(
-        `purge entire ${paths.home} AND okstra skills in Claude/Codex/OMP homes? user data will be lost.`,
+        `purge entire ${paths.home} AND okstra skills in Claude/Agent/OMP homes? user data will be lost.`,
       );
       if (!ok) {
         process.stdout.write("aborted.\n");

package/src/lib/skill-catalog.mjs

@@ -1,5 +1,5 @@
 // Single source of truth for which okstra skills are user-facing and which
-// former internal skill directories must be pruned from agent skill homes on
+// former or renamed skill directories must be pruned from agent skill homes on
 // (re)install. The Claude plugin manifest (.claude-plugin/plugin.json) and the
 // installer both derive their skill list from here, so the public surface
 // cannot drift between discovery and install.
@@ -11,26 +11,27 @@ export const USER_SKILL_NAMES = Object.freeze([
   "okstra-memory",
   "okstra-inspect",
   "okstra-schedule",
-  "okstra-container",
+  "okstra-container-build",
 ]);
 
-// Names okstra used to install as skills before lead/internal contracts moved
-// into the runtime resource tree (~/.okstra/prompts/). Install prunes these
-// exact directories from each target skill home so upgrades do not leave the
-// old internal skills behind. Exact-name only — never wildcard.
-export const OBSOLETE_INTERNAL_SKILL_NAMES = Object.freeze([
+// Names okstra used to install as skills before they were renamed or moved
+// into the runtime resource tree. Install prunes these exact directories from
+// each target skill home so upgrades do not leave stale slash commands behind.
+// Exact-name only — never wildcard.
+export const OBSOLETE_SKILL_NAMES = Object.freeze([
   "okstra",
   "okstra-context-loader",
   "okstra-team-contract",
   "okstra-convergence",
   "okstra-report-writer",
   "okstra-coding-preflight",
+  "okstra-container",
 ]);
 
 export function userSkillNames() {
   return [...USER_SKILL_NAMES];
 }
 
-export function obsoleteInternalSkillNames() {
-  return [...OBSOLETE_INTERNAL_SKILL_NAMES];
+export function obsoleteSkillNames() {
+  return [...OBSOLETE_SKILL_NAMES];
 }