@ghl-ai/aw 0.1.69 → 0.1.70-beta.1

package/ecc.mjs

@@ -3,7 +3,7 @@ import { promisify } from "node:util";
 const execAsync = promisify(execCb);
 import {
   existsSync, readFileSync, readdirSync,
-  mkdirSync, rmSync, writeFileSync, renameSync,
+  mkdirSync, rmSync, writeFileSync,
 } from "node:fs";
 import { dirname, join } from "node:path";
 import { homedir } from "node:os";
@@ -16,7 +16,6 @@ export const AW_ECC_TAG = "v1.4.66";
 const REQUIRED_ECC_FILES = [
   "package.json",
   "scripts/install-apply.js",
-  "scripts/sync-ecc-to-codex.sh",
 ];
 
 const MARKETPLACE_NAME = "aw-marketplace";
@@ -24,19 +23,31 @@ const PLUGIN_KEY = `aw@${MARKETPLACE_NAME}`;
 
 function eccDir() { return join(homedir(), ".aw-ecc"); }
 
-// "claude" uses a no-commands module set so hooks, rules, shared references,
-// and install-state land in ~/.claude/ — while slash commands stay owned by
-// the marketplace plugin under /aw:*.
-// Using `--without baseline:commands` with broader profiles is unsafe because
-// some optional modules depend on commands-core and cause install-apply to fail.
+// File-copy targets use explicit non-skill, non-command module sets so hooks,
+// rules, shared references, and install-state can remain available while AW
+// skills come from platform-docs/.aw_registry/aw. Slash access is backed by
+// those skills directly, so aw-ecc command artifacts are intentionally skipped.
 const FILE_COPY_TARGETS = ["claude", "cursor", "codex"];
-const CLAUDE_FILE_COPY_MODULES = [
-  "rules-core",
-  "agents-core",
-  "hooks-runtime",
-  "platform-configs",
-  "workflow-quality",
-];
+const FILE_COPY_MODULES_BY_TARGET = {
+  claude: [
+    "rules-core",
+    "agents-core",
+    "hooks-runtime",
+    "platform-configs",
+  ],
+  cursor: [
+    "rules-core",
+    "agents-core",
+    "hooks-runtime",
+    "platform-configs",
+  ],
+  codex: [
+    "rules-core",
+    "agents-core",
+    "hooks-runtime",
+    "platform-configs",
+  ],
+};
 
 const TARGET_STATE = {
   claude: { state: ".claude/ecc/install-state.json" },
@@ -241,12 +252,11 @@ function cloneOrUpdate(tag, dest) {
 
 /**
  * Transform canonical /aw: references to Cursor-compatible /aw- in installed
- * skill and rule files. Cursor namespaces commands via directory structure
- * (commands/aw/plan.md → /aw-plan) rather than colons.
+ * rule files. Cursor's model-side rule still sees hyphenated route mentions in
+ * some generated home instructions.
  */
 function transformCursorAwRefs(home) {
   const dirs = [
-    join(home, ".cursor", "skills"),
     join(home, ".cursor", "rules"),
   ];
   for (const dir of dirs) {
@@ -280,57 +290,16 @@ function uninstallClaudePlugin() {
   try { run(`claude plugin marketplace remove ${MARKETPLACE_NAME}`); } catch { /* not registered */ }
 }
 
-/**
- * Move ecc command files from ~/.cursor/commands/*.md → ~/.cursor/commands/aw/*.md
- * so Cursor exposes them as /aw:tdd, /aw:plan — consistent with Claude Code's plugin namespace.
- * Also updates the ecc-install-state.json paths so nuke can clean them up correctly.
- */
-function namespaceCursorCommands(home) {
-  const commandsDir = join(home, ".cursor", "commands");
-  const awDir = join(commandsDir, "aw");
-  if (!existsSync(commandsDir)) return;
-
-  // Move any flat .md files (not already in a subdirectory) into aw/
-  const moved = [];
-  for (const file of readdirSync(commandsDir)) {
-    if (!file.endsWith(".md") || file.startsWith(".")) continue;
-    const src = join(commandsDir, file);
-    mkdirSync(awDir, { recursive: true });
-    const dest = join(awDir, file);
+function pruneEccAuthoredSkillsAndCommands(repoDir) {
+  // The local aw-ecc clone is now only a compatibility runtime for hooks,
+  // rules, agents, and configs. Active AW skills live in platform-docs under
+  // ~/.aw/.aw_registry/aw/skills, and slash adapters point at those SKILL.md
+  // files directly.
+  for (const relPath of ["commands", "skills"]) {
     try {
-      renameSync(src, dest);
-      moved.push({ from: src, to: dest });
+      rmSync(join(repoDir, relPath), { recursive: true, force: true });
     } catch { /* best effort */ }
   }
-
-  if (moved.length === 0) return;
-
-  // Update install-state so nuke removes files from the new aw/ location
-  const statePath = join(home, ".cursor", "ecc-install-state.json");
-  if (!existsSync(statePath)) return;
-  try {
-    const state = JSON.parse(readFileSync(statePath, "utf8"));
-    const pathMap = new Map(moved.map(({ from, to }) => [from, to]));
-    state.operations = (state.operations || []).map((op) => {
-      const newPath = op.destinationPath && pathMap.get(op.destinationPath);
-      return newPath ? { ...op, destinationPath: newPath } : op;
-    });
-    writeFileSync(statePath, JSON.stringify(state, null, 2));
-  } catch { /* best effort */ }
-}
-
-/**
- * Run scripts/sync-ecc-to-codex.sh from the cloned ecc repo.
- * Generates ~/.codex/prompts/*.md (Codex equivalent of slash commands),
- * using aw-*.md for AW-namespaced commands and ecc-*.md for the rest,
- * and merges ~/.codex/AGENTS.md. Best-effort — failure doesn't block init.
- */
-function syncEccToCodex(repoDir) {
-  const syncScript = join(repoDir, "scripts", "sync-ecc-to-codex.sh");
-  if (!existsSync(syncScript)) return;
-  try {
-    run(`bash "${syncScript}"`, { cwd: homedir() });
-  } catch { /* best effort — codex sync failure is non-blocking */ }
 }
 
 export async function installAwEcc(
@@ -347,6 +316,7 @@ export async function installAwEcc(
   try {
     if (eccSpinner) eccSpinner.start('Cloning aw-ecc engine...');
     await cloneOrUpdateAsync(AW_ECC_TAG, repoDir);
+    pruneEccAuthoredSkillsAndCommands(repoDir);
     if (eccSpinner) eccSpinner.message('Installing aw-ecc dependencies...');
 
     // Claude Code: plugin install via marketplace CLI (proper agent dispatch)
@@ -381,20 +351,17 @@ export async function installAwEcc(
 
           // Always use HOME as cwd so files land in ~/.<target>/ globally.
           const runCwd = homedir();
-          const installArgs = target === "claude"
-            ? `--target ${target} --modules ${CLAUDE_FILE_COPY_MODULES.join(",")}`
-            : `--target ${target} --profile full`;
+          const installModules = FILE_COPY_MODULES_BY_TARGET[target] || [];
+          const installArgs = installModules.length > 0
+            ? `--target ${target} --modules ${installModules.join(",")}`
+            : `--target ${target}`;
           run(
             `node ${join(repoDir, "scripts/install-apply.js")} ${installArgs}`,
             { cwd: runCwd },
           );
           if (target === "cursor") {
-            namespaceCursorCommands(runCwd);
             transformCursorAwRefs(home);
           }
-          if (target === "codex") {
-            syncEccToCodex(repoDir);
-          }
           restoreProtectedConfigs(snapshot);
         } catch { /* target not supported — skip */ }
       }));
@@ -446,8 +413,8 @@ export function uninstallAwEcc({ silent = false } = {}) {
     } catch { /* corrupted state — skip */ }
   }
 
-  // Codex: remove generated prompt files from sync-ecc-to-codex.sh
-  // (not tracked in install-state — cleaned via manifests, with prefix fallback)
+  // Codex: remove legacy generated prompt files from older ECC sync flows
+  // (not tracked in install-state — cleaned via manifests, with prefix fallback).
   const codexPromptsDir = join(HOME, ".codex", "prompts");
   if (existsSync(codexPromptsDir)) {
     try {

package/git.mjs

@@ -5,7 +5,14 @@ import { mkdtempSync, existsSync, lstatSync, rmSync, readFileSync, symlinkSync,
 import { join, basename, dirname } from 'node:path';
 import { homedir, tmpdir } from 'node:os';
 import { promisify } from 'node:util';
-import { REGISTRY_BASE_BRANCH, REGISTRY_DIR, DOCS_SOURCE_DIR, AW_DOCS_DIR, RULES_SOURCE_DIR } from './constants.mjs';
+import {
+  REGISTRY_BASE_BRANCH,
+  REGISTRY_DIR,
+  AW_REGISTRY_NAMESPACE_DIR,
+  DOCS_SOURCE_DIR,
+  AW_DOCS_DIR,
+  RULES_SOURCE_DIR,
+} from './constants.mjs';
 
 const exec = promisify(execCb);
 
@@ -155,6 +162,7 @@ export function includeToSparsePaths(paths) {
   }
   result.add(`${REGISTRY_DIR}/AW-PROTOCOL.md`);
   if (paths.includes('platform')) {
+    result.add(AW_REGISTRY_NAMESPACE_DIR);
     result.add(DOCS_SOURCE_DIR);
     result.add(AW_DOCS_DIR);
     result.add(RULES_SOURCE_DIR);
@@ -389,14 +397,9 @@ export async function fetchAndMerge(awHome, { silent = true } = {}) {
   // git 2.46+ bug on blob:none + no-cone sparse-checkout repos that silently
   // drops bare-name patterns (e.g. "content", "CODEOWNERS") when HEAD advances.
   //
-  // --autostash: AW writes into the registry working tree from external
-  // sources (ensureAwRuntimeHook copies ~/.aw-ecc/.../session-start.sh into a
-  // tracked path; transformCursorAwRefs rewrites /aw: → /aw- through Cursor
-  // skill directory symlinks that resolve into .aw_registry/). When those
-  // versions drift, the working tree is dirty at rebase time and rebase
-  // refuses to run, silently aborting the entire pull. Autostash stashes the
-  // dirty bytes, rebases cleanly, and reapplies them — so the registry stays
-  // in sync even when external sources have leaked into the tracked tree.
+  // --autostash: AW can have local registry edits pending when sync runs.
+  // Without autostash, git refuses to rebase and the pull silently aborts.
+  // Autostash stashes dirty bytes, rebases cleanly, and reapplies them.
   try {
     await exec(`git -C "${awHome}" rebase --autostash origin/${REGISTRY_BASE_BRANCH}`);
     updated = true;

package/hooks/codex-home.mjs

@@ -26,10 +26,10 @@ const CODEX_HOME_PHASE_BLUEPRINTS = {
         marker: this.scriptMarker,
         phase: 'SessionStart',
         targetCandidates: [
-          '$HOME/.aw_registry/platform/core/skills/using-aw-skills/hooks/session-start.sh',
-          '$HOME/.aw/.aw_registry/platform/core/skills/using-aw-skills/hooks/session-start.sh',
+          '$HOME/.aw_registry/aw/skills/using-aw-skills/hooks/session-start.sh',
+          '$HOME/.aw/.aw_registry/aw/skills/using-aw-skills/hooks/session-start.sh',
         ],
-        warningMessage: 'WARNING: AW using-aw-skills hook not found in ~/.aw_registry. Run aw init or aw pull platform.',
+        warningMessage: 'WARNING: AW using-aw-skills hook not found in ~/.aw_registry/aw. Run aw init or aw pull platform.',
         telemetryHookPath: '$HOME/.aw-ecc/scripts/hooks/aw-usage-session-start.js',
         harnessEnv: 'codex',
       });

package/link.mjs

@@ -158,10 +158,12 @@ function cleanSymlinksRecursive(dir) {
 }
 
 /**
- * Compute flat IDE name: namespace-slug (always includes namespace).
+ * Compute flat IDE names. The top-level `aw` namespace is already encoded in
+ * its skill and command names, so keep those names stable instead of producing
+ * aw-aw-plan or aw-plan.md variants.
  */
-function flatName(ns, name) {
-  return `${ns}-${name}`;
+function flatRegistryName(ns, ...parts) {
+  return (ns === 'aw' ? parts : [ns, ...parts]).join('-');
 }
 
 /**
@@ -202,7 +204,7 @@ export function linkWorkspace(cwd, awDirOverride = null, { silent = false } = {}
     for (const type of FILE_TYPES) {
       for (const { typeDirPath, segments } of findNestedTypeDirs(join(awDir, ns), type)) {
         for (const file of readdirSync(typeDirPath).filter(f => f.endsWith('.md') && !f.startsWith('.'))) {
-          const flat = [ns, ...segments, file].join('-');
+          const flat = flatRegistryName(ns, ...segments, file);
 
           for (const ide of IDE_DIRS) {
             const linkDir = join(cwd, ide, type);
@@ -219,7 +221,7 @@ export function linkWorkspace(cwd, awDirOverride = null, { silent = false } = {}
     // Skills: per-skill directory symlinks (recursive for nested domain dirs)
     for (const { typeDirPath: skillsDir, segments } of findNestedTypeDirs(join(awDir, ns), 'skills')) {
       for (const skill of listDirs(skillsDir)) {
-        const flat = [ns, ...segments, skill].join('-');
+        const flat = flatRegistryName(ns, ...segments, skill);
 
         for (const ide of IDE_DIRS) {
           const linkDir = join(cwd, ide, 'skills');
@@ -233,7 +235,7 @@ export function linkWorkspace(cwd, awDirOverride = null, { silent = false } = {}
 
       for (const { skillDirPath, segments: skillSegments } of findSkillRootDirs(skillsDir)) {
         if (skillSegments.length <= 1) continue;
-        const flat = [ns, ...segments, ...skillSegments].join('-');
+        const flat = flatRegistryName(ns, ...segments, ...skillSegments);
 
         for (const ide of IDE_DIRS) {
           const linkDir = join(cwd, ide, 'skills');
@@ -254,7 +256,7 @@ export function linkWorkspace(cwd, awDirOverride = null, { silent = false } = {}
       if (isColocated) {
         // Colocated evals: agents/evals/<slug>/ — one level of slug dirs
         for (const evalSlug of listDirs(evalsDir)) {
-          const flat = [ns, ...segments, evalSlug].join('-');
+          const flat = flatRegistryName(ns, ...segments, evalSlug);
           for (const ide of IDE_DIRS) {
             const linkDir = join(cwd, ide, 'evals');
             mkdirSync(linkDir, { recursive: true });
@@ -269,7 +271,7 @@ export function linkWorkspace(cwd, awDirOverride = null, { silent = false } = {}
         for (const subType of listDirs(evalsDir)) {
           const subDir = join(evalsDir, subType);
           for (const evalName of listDirs(subDir)) {
-            const flat = [ns, ...segments, subType, evalName].join('-');
+            const flat = flatRegistryName(ns, ...segments, subType, evalName);
             for (const ide of IDE_DIRS) {
               const linkDir = join(cwd, ide, 'evals');
               mkdirSync(linkDir, { recursive: true });
@@ -310,7 +312,7 @@ export function linkWorkspace(cwd, awDirOverride = null, { silent = false } = {}
       for (const { typeDirPath: skillsDir, segments } of findNestedTypeDirs(join(awDir, ns), 'skills')) {
         mkdirSync(agentsSkillsDir, { recursive: true });
         for (const skill of listDirs(skillsDir)) {
-          const flat = [ns, ...segments, skill].join('-');
+          const flat = flatRegistryName(ns, ...segments, skill);
           const linkPath = join(agentsSkillsDir, flat);
           const targetPath = join(skillsDir, skill);
           const relTarget = relative(agentsSkillsDir, targetPath);
@@ -319,7 +321,7 @@ export function linkWorkspace(cwd, awDirOverride = null, { silent = false } = {}
 
         for (const { skillDirPath, segments: skillSegments } of findSkillRootDirs(skillsDir)) {
           if (skillSegments.length <= 1) continue;
-          const flat = [ns, ...segments, ...skillSegments].join('-');
+          const flat = flatRegistryName(ns, ...segments, ...skillSegments);
           const linkPath = join(agentsSkillsDir, flat);
           const relTarget = relative(agentsSkillsDir, skillDirPath);
           if (linkNestedSkillRoot(relTarget, linkPath, { silent })) created++;
@@ -346,7 +348,7 @@ export function linkWorkspace(cwd, awDirOverride = null, { silent = false } = {}
   for (const ns of namespaces) {
     for (const { typeDirPath: commandsDir, segments } of findNestedTypeDirs(join(awDir, ns), 'commands')) {
       for (const file of readdirSync(commandsDir).filter(f => f.endsWith('.md') && !f.startsWith('.'))) {
-        const cmdFileName = [ns, ...segments, file].join('-');
+        const cmdFileName = flatRegistryName(ns, ...segments, file);
 
         for (const ide of IDE_DIRS) {
           const linkDir = join(cwd, ide, 'commands', 'aw');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ghl-ai/aw",
-  "version": "0.1.69",
+  "version": "0.1.70-beta.1",
   "description": "Agentic Workspace CLI — pull, push & manage agents, skills and commands from the registry",
   "type": "module",
   "bin": {

package/startup.mjs

@@ -83,8 +83,10 @@ function resolveRegistryRoot(homeDir = homedir()) {
   ].find(existsSync) || null;
 }
 
-function awRuntimeHookSourcePath(homeDir = homedir()) {
-  return join(homeDir, '.aw-ecc', 'skills', 'using-aw-skills', 'hooks', 'session-start.sh');
+function awRuntimeHookDestinationPath(homeDir = homedir()) {
+  const registryRoot = resolveRegistryRoot(homeDir);
+  if (!registryRoot) return null;
+  return join(registryRoot, 'aw', 'skills', 'using-aw-skills', 'hooks', 'session-start.sh');
 }
 
 function readJson(filePath, fallback = {}) {
@@ -393,27 +395,10 @@ function hasCodexSessionStartScript(homeDir = homedir()) {
 }
 
 export function ensureAwRuntimeHook(homeDir = homedir()) {
-  const sourcePath = awRuntimeHookSourcePath(homeDir);
-  const registryRoot = resolveRegistryRoot(homeDir);
-  if (!existsSync(sourcePath) || !registryRoot) return [];
-
-  const destinationPath = join(
-    registryRoot,
-    'platform',
-    'core',
-    'skills',
-    'using-aw-skills',
-    'hooks',
-    'session-start.sh',
-  );
-  const sourceContent = readFileSync(sourcePath, 'utf8');
-  const existingContent = existsSync(destinationPath) ? readFileSync(destinationPath, 'utf8') : null;
-  if (existingContent === sourceContent) return [];
-
-  mkdirSync(dirname(destinationPath), { recursive: true });
-  writeFileSync(destinationPath, sourceContent);
+  const destinationPath = awRuntimeHookDestinationPath(homeDir);
+  if (!destinationPath || !existsSync(destinationPath)) return [];
   try { chmodSync(destinationPath, 0o755); } catch { /* best effort */ }
-  return [destinationPath];
+  return [];
 }
 
 function hasCodexHooksEnabled(homeDir = homedir()) {

package/c4/eccRegistryBridge.mjs

@@ -1,236 +0,0 @@
-/**
- * c4/eccRegistryBridge.mjs — Codex-only Bug A workaround.
- *
- * Codex Web's upstream `aw-session-start.sh` enumerator scans a fixed
- * registry path:
- *
- *   ~/.aw/.aw_registry/platform/core/skills/<name>/SKILL.md
- *
- * ECC ships skills under a different path:
- *
- *   ~/.aw-ecc/skills/<name>/SKILL.md
- *
- * On Codex Web, neither Claude's plugin marketplace nor Cursor's direct
- * `~/.cursor/skills/...` resolution applies. The only way Codex can see
- * ECC skills is if the registry path is populated. We bridge by symlinking
- * each ECC skill's SKILL.md into the registry layout. The default mode keeps
- * the original non-clobbering behavior. `preferEccSkills` only retargets
- * existing symlinks; regular files are treated as user-owned and preserved.
- *
- * Also installs a fallback symlink at `<homeOf(awHome)>/.aw_registry` →
- * `<awHome>/.aw_registry` because some legacy hooks read from that path.
- *
- * **Scope**: Codex Web only. The orchestrator is responsible for *not*
- * invoking this on claude-web or cursor-cloud; the module itself is
- * harness-agnostic and just performs the bridge.
- */
-
-import {
-  existsSync,
-  lstatSync,
-  readlinkSync,
-  readdirSync,
-  mkdirSync,
-  symlinkSync,
-  unlinkSync,
-} from 'node:fs';
-import { dirname, join } from 'node:path';
-
-/** True iff `path` is a symlink that points to a nonexistent target. */
-function isBrokenSymlink(path) {
-  try {
-    const st = lstatSync(path);
-    if (!st.isSymbolicLink()) return false;
-    return !existsSync(path);
-  } catch {
-    return false;
-  }
-}
-
-/** True iff `path` is a symlink that already points at `target`. */
-function isSymlinkTo(path, target) {
-  try {
-    const st = lstatSync(path);
-    if (!st.isSymbolicLink()) return false;
-    return readlinkSync(path) === target;
-  } catch {
-    return false;
-  }
-}
-
-/** True iff `path` is any symlink, regardless of where it points. */
-function isSymlink(path) {
-  try {
-    return lstatSync(path).isSymbolicLink();
-  } catch {
-    return false;
-  }
-}
-
-/**
- * Install the fallback symlink `~/.aw_registry → ~/.aw/.aw_registry` if not
- * present and not blocked by an existing real directory. Best-effort.
- */
-function installRegistryFallbackLink(awHome) {
-  const home = dirname(awHome);
-  const fallback = join(home, '.aw_registry');
-  const target = join(awHome, '.aw_registry');
-
-  if (!existsSync(target)) {
-    // Nothing to point at yet — skip silently.
-    return;
-  }
-
-  let existsAtFallback = false;
-  try {
-    lstatSync(fallback);
-    existsAtFallback = true;
-  } catch {
-    existsAtFallback = false;
-  }
-
-  if (existsAtFallback) return; // do not clobber existing dir/symlink
-
-  try {
-    symlinkSync(target, fallback);
-  } catch {
-    // best-effort
-  }
-}
-
-/**
- * Symlink every ECC skill into the registry layout. Idempotent.
- *
- * Result shape:
- *   - linked   — skill names freshly bridged this call
- *   - skipped  — skill names whose registry target already existed and was preserved
- *   - replaced — skill names whose registry target was replaced with the ECC symlink
- *   - broken   — skill directories that exist in eccHome/skills/ but have no SKILL.md
- *                (broken ECC layout — distinct from already-bridged so the orchestrator
- *                can surface a real diagnostic rather than treating it as success)
- *   - reason   — only set when the result is fully diagnostic:
- *                  'ecc-missing'    — eccHome/skills/ does not exist
- *                  'already-bridged' — every well-formed skill was already linked
- *                                     and there were no broken skills
- *
- * @param {object} opts
- * @param {string} opts.awHome    Path of the AW home (e.g. ~/.aw).
- * @param {string} opts.eccHome   Path of the ECC home (e.g. ~/.aw-ecc).
- * @param {boolean} [opts.preferEccSkills=false]
- *   Retarget existing registry SKILL.md symlinks to ECC. Regular files are
- *   never replaced because they may be user-owned overrides.
- * @returns {{
- *   linked: string[],
- *   skipped: string[],
- *   replaced: string[],
- *   broken: string[],
- *   reason?: 'already-bridged' | 'ecc-missing'
- * }}
- */
-export function applyEccRegistryBridge({ awHome, eccHome, preferEccSkills = false } = {}) {
-  if (!awHome || !eccHome) {
-    throw new Error('applyEccRegistryBridge: awHome and eccHome are required');
-  }
-
-  const eccSkillsDir = join(eccHome, 'skills');
-  if (!existsSync(eccSkillsDir)) {
-    return { linked: [], skipped: [], broken: [], reason: 'ecc-missing' };
-  }
-
-  const linked = [];
-  const skipped = [];
-  const replaced = [];
-  const broken = [];
-
-  let entries;
-  try {
-    entries = readdirSync(eccSkillsDir, { withFileTypes: true });
-  } catch {
-    return { linked: [], skipped: [], broken: [], reason: 'ecc-missing' };
-  }
-
-  let skillDirCount = 0;
-  for (const entry of entries) {
-    if (!entry.isDirectory()) continue;
-    skillDirCount += 1;
-    const name = entry.name;
-    const source = join(eccSkillsDir, name, 'SKILL.md');
-
-    // ECC skill directory exists but lacks SKILL.md → broken layout.
-    if (!existsSync(source)) {
-      broken.push(name);
-      continue;
-    }
-
-    const targetDir = join(awHome, '.aw_registry/platform/core/skills', name);
-    const target = join(targetDir, 'SKILL.md');
-
-    // If target already exists and is healthy, preserve regular files. The
-    // optional preference mode may only retarget generated symlinks; it must
-    // not delete user-owned registry files.
-    let targetExists = false;
-    try {
-      lstatSync(target);
-      targetExists = true;
-    } catch {
-      targetExists = false;
-    }
-
-    if (targetExists && !isBrokenSymlink(target)) {
-      if (isSymlinkTo(target, source)) {
-        skipped.push(name);
-        continue;
-      }
-
-      if (!preferEccSkills || !isSymlink(target)) {
-        skipped.push(name);
-        continue;
-      }
-
-      try {
-        unlinkSync(target);
-      } catch {
-        broken.push(name);
-        continue;
-      }
-
-      try {
-        symlinkSync(source, target);
-        replaced.push(name);
-      } catch {
-        broken.push(name);
-      }
-      continue;
-    }
-
-    // Either absent or a broken symlink — replace.
-    mkdirSync(targetDir, { recursive: true });
-    if (targetExists) {
-      try { unlinkSync(target); } catch { /* best-effort */ }
-    }
-    try {
-      symlinkSync(source, target);
-      linked.push(name);
-    } catch {
-      // Filesystem may forbid symlinks (rare on Codex Web). Treat as broken
-      // so the orchestrator can surface the real diagnostic.
-      broken.push(name);
-    }
-  }
-
-  installRegistryFallbackLink(awHome);
-
-  const result = { linked, skipped, replaced, broken };
-  // Only flag 'already-bridged' when every well-formed skill was already linked
-  // AND no broken skills exist — i.e. this run was a true no-op.
-  if (
-    skillDirCount > 0 &&
-    linked.length === 0 &&
-    replaced.length === 0 &&
-    broken.length === 0 &&
-    skipped.length === skillDirCount
-  ) {
-    result.reason = 'already-bridged';
-  }
-  return result;
-}