@lifeaitools/rdc-skills 0.37.5 → 0.37.7

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "rdc",
-  "version": "0.37.5",
+  "version": "0.37.7",
   "description": "RDC typed-agent dispatch skill suite for Claude Code — plan, build, review, overnight unattended builds with work-item tracking and TDD enforcement.",
   "author": {
     "name": "LIFEAI",

package/clauth-plugin.json

@@ -1,7 +1,7 @@
 {
   "schema": "lifeai.plugin.v1",
   "id": "rdc-skills",
-  "version": "0.37.5",
+  "version": "0.37.7",
   "publisher": "LIFEAI",
   "core": false,
   "documentation": {

package/git-sha.json

@@ -1,3 +1,3 @@
 {
-  "sha": "a96832c1aa663240fecf042c6b41d7017c8bce47"
+  "sha": "9621389d4e4724ae260e0d4e91651c10b3ac0e2c"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lifeaitools/rdc-skills",
-  "version": "0.37.5",
+  "version": "0.37.7",
   "description": "RDC typed-agent dispatch skill suite for Claude Code - plan, build, review, overnight builds",
   "keywords": [
     "claude-code",
@@ -19,7 +19,7 @@
     "type": "plugin",
     "skills": "skills/",
     "guides": "guides/",
-    "version": "0.37.5",
+    "version": "0.37.7",
     "commands": "commands/"
   },
   "bin": {

package/scripts/install-rdc-skills.js

@@ -951,7 +951,7 @@ function findCodexTargets() {
   return targets;
 }
 
-function registerCodexTarget(targetDir) {
+function registerCodexTarget(targetDir, { install = false } = {}) {
   fs.mkdirSync(targetDir, { recursive: true });
 
   // Clean: remove dirs that are rdc skills — by prefix OR by frontmatter name
@@ -967,11 +967,21 @@ function registerCodexTarget(targetDir) {
     }
   }
 
-  // Do not reinstall file-based copies. Codex discovers the live catalog via
-  // [mcp_servers.rdc-skills]. Keeping the same skills in ~/.codex and ~/.agents
-  // registers every skill two or three times, exhausts Codex's skill metadata
-  // budget, and emits a red startup error on every session.
-  return { removed, copied: 0 };
+  if (!install) return { removed, copied: 0 };
+
+  // Install into exactly ONE Codex skill dir. Copies in both ~/.codex and
+  // ~/.agents registered every skill twice and exhausted Codex's skill metadata
+  // budget; bf88292 answered that by installing none, which left Codex with
+  // rdc: skills only while the remote MCP was reachable.
+  const skillsSrc = path.join(repoRoot, 'skills');
+  let copied = 0;
+  for (const entry of fs.readdirSync(skillsSrc, { withFileTypes: true })) {
+    if (!entry.isDirectory()) continue;
+    if (!fs.existsSync(path.join(skillsSrc, entry.name, 'SKILL.md'))) continue;
+    copyDirRecursive(path.join(skillsSrc, entry.name), path.join(targetDir, `rdc-${entry.name}`));
+    copied++;
+  }
+  return { removed, copied };
 }
 
 // ── Step 6: Zip for claude.ai / distribution ─────────────────────────────────
@@ -1372,21 +1382,24 @@ async function main() {
     info('[2.5] MCP        — rdc-skills endpoint verified (claude + codex)');
   }
 
-  // 2.6. Codex migration: MCP is authoritative; purge legacy file copies.
+  // 2.6. Codex: rdc skill files in ONE dir (the explicit one, else ~/.codex/skills);
+  // every other Codex skill dir is only cleaned, so nothing registers twice.
   const codexTargets = findCodexTargets();
-  if (codexTargets.length > 0) {
-    let copiedTotal = 0;
-    let removedTotal = 0;
-    for (const target of codexTargets) {
-      const { removed, copied } = registerCodexTarget(target.targetDir);
-      copiedTotal += copied;
-      removedTotal += removed;
-      info(`       ${target.label.padEnd(15)}: ${target.targetDir} (${removed} stale removed; MCP authoritative)`);
-    }
-    ok(`[2.6] Codex      — ${removedTotal} stale file-based skill(s) removed across ${codexTargets.length} target(s); MCP authoritative`);
-  } else {
-    info('[2.6] Codex      — skipped (no Codex skill dirs found; use --codex-root or --codex-skill-dir)');
-  }
+  const primaryCodexDir = explicitCodexSkillDir || path.join(os.homedir(), '.codex', 'skills');
+  if (!codexTargets.some((t) => t.targetDir === primaryCodexDir)) {
+    codexTargets.unshift({ label: 'global .codex', targetDir: primaryCodexDir });
+  }
+  let copiedTotal = 0;
+  let removedTotal = 0;
+  for (const target of codexTargets) {
+    const install = target.targetDir === primaryCodexDir;
+    const { removed, copied } = registerCodexTarget(target.targetDir, { install });
+    copiedTotal += copied;
+    removedTotal += removed;
+    info(`       ${target.label.padEnd(15)}: ${target.targetDir} (${install ? `${copied} installed` : 'cleaned only'}, ${removed} stale removed)`);
+  }
+  if (copiedTotal === 0) throw new Error(`Codex skill install wrote nothing to ${primaryCodexDir}`);
+  ok(`[2.6] Codex      — ${copiedTotal} skill(s) installed in ${primaryCodexDir}; ${removedTotal} stale copies removed across ${codexTargets.length} dir(s)`);
 
   // 2.7. Symlinks in regen-root/.claude/skills/ (FS MCP + claude.ai access)
   if (codexRoot) {

package/scripts/verify-live-install.mjs

@@ -178,10 +178,16 @@ function checkCodex(truth) {
     const rdcDirs = dirs.filter((e) => /^rdc-/.test(e.name) || readFrontmatterName(path.join(t.dir, e.name, 'SKILL.md'))?.startsWith('rdc:'));
     results.push({ label: t.label, dir: t.dir, count: rdcDirs.length });
   }
-  const pass = results.every((r) => r.count === 0);
-  const detail = results.map((r) =>
-    `${r.label}: ${r.count} legacy file-based rdc skill(s)${r.count === 0 ? '' : ' [MCP DUPLICATES PRESENT]'}`
-  ).join('; ');
+  // Codex keeps rdc skills as files in exactly one dir (~/.codex/skills), so it
+  // works without the network; any copy elsewhere registers the skill twice.
+  const skillsSrc = path.join(repoRoot, 'skills');
+  const expected = fs.readdirSync(skillsSrc, { withFileTypes: true })
+    .filter((e) => e.isDirectory() && fs.existsSync(path.join(skillsSrc, e.name, 'SKILL.md'))).length;
+  const isPrimary = (r) => r.label === 'global .codex';
+  const pass = results.some(isPrimary) && results.every((r) => (isPrimary(r) ? r.count === expected : r.count === 0));
+  const detail = results.map((r) => (isPrimary(r)
+    ? `${r.label}: ${r.count}/${expected} rdc skill(s) installed`
+    : `${r.label}: ${r.count} rdc skill(s)${r.count === 0 ? '' : ' [DUPLICATES — would register twice]'}`)).join('; ');
   return { surface: 'Codex', pass, detail };
 }
 

package/skills/deploy/SKILL.md

@@ -27,6 +27,20 @@ Run it from the regen-root checkout you are working in. That is the whole
 instruction. There is no checklist to follow, no mode to pick, and no decision
 for you to make on the success path.
 
+## Local build contract
+
+Standard dev deployment builds on the workstation first. `DEPLOY_MGR_URL`
+must name a loopback manager whose health response reports `host.role: local`.
+The program requests `builder: local` explicitly; Docker or build failure stops
+the deployment. It never falls back to a remote builder.
+
+The target must have a registered image route. Legacy remote build/reload routes
+are refused until a proper local artifact build-and-ship route is implemented.
+Do not enable an image route without its runtime and build configuration.
+
+The manager's explicitly requested remote maintenance operations remain separate
+from this standard command. `promote` remains build-free and approval-gated.
+
 ## Exit 0 — you are done
 
 Print the URL the program reports, and stop.
@@ -46,8 +60,8 @@ followed by the same record as one line of JSON.
 | exit | meaning | first move |
 |---|---|---|
 | 2 | usage — bad or missing arguments | read the usage line; this program takes a slug and optionally `promote` |
-| 3 | deploy manager unreachable | `pm2 restart regen-deploy-mgr` (it is a local PM2 process), then re-run |
-| 4 | slug not resolvable or not routable | the block names what is missing — a registry row, or an allowlist entry |
+| 3 | deploy manager unavailable or unsuitable for local building | read the block; restore the workstation manager and its local host identity, then re-run |
+| 4 | slug not resolvable or no local artifact route | the block names the missing registry or image route; repair that route before deployment |
 | 5 | a submitted job failed or was rejected | the block carries the manager's own step, command, exit code and stderr tail |
 | 6 | the job succeeded but the health probe did not | bytes shipped, app did not come up — read the job the block names |
 | 7 | polling exceeded the deadline | the job is NOT cancelled; the block gives the URL to read it |
@@ -110,9 +124,10 @@ manager, where it runs, rather than in a document that hopes to be read.
 
 A dev deploy needs no approval. `promote` puts bytes on production and needs
 explicit go-ahead for that call. The program itself refuses to promote anything
-that is not an image-model app, because promote retags the digest dev already
-runs — **a promote never builds**, and one that fell back to building would ship
-bytes production has never seen under a name claiming it had.
+that is not an image-model app. **A promote never builds**; it retags an existing
+image through the manager. The current manager resolves the configured dev tag
+(normally `develop`); a successful dev deploy alone does not prove that this
+mutable tag still identifies the running dev image when promotion begins.
 
 ## Verification