td-ai-tools 1.0.6 → 1.0.8

package/README.md

@@ -33,6 +33,12 @@ npx td-ai-tools install
 npx td-ai-tools install --all
 npx td-ai-tools install pr-solver
 npx td-ai-tools install pr-solver horizon-component-library
+npx td-ai-tools update
+npx td-ai-tools update --all
+npx td-ai-tools update pr-solver
+npx td-ai-tools delete
+npx td-ai-tools delete --all
+npx td-ai-tools delete pr-solver
 ```
 
 The installer copies requested items into both agent layouts:
@@ -41,6 +47,10 @@ The installer copies requested items into both agent layouts:
 
 This keeps the installed assets available to both Claude-style and `.agents`-style project conventions.
 
+`install` now errors when the target item already exists. Use `update` to replace an existing installed skill or agent pack.
+
+`delete` removes installed items from both `.claude/` and `.agents/` target directories, and works on any installed skill or agent pack regardless of whether it is in the catalogue.
+
 ## Local Verification
 Run the local smoke test to package the repo and verify installation into a throwaway project:
 

package/bin/cli.js

@@ -44,7 +44,7 @@ function readFrontmatterField(mdPath, field) {
   return match ? match[1].trim() : '';
 }
 
-function installSkill(name) {
+function installSkill(name, { replaceExisting = false } = {}) {
   const src = path.join(SKILLS_DIR, name);
   if (!fs.existsSync(src)) {
     console.error(`  [error] Skill "${name}" not found.`);
@@ -52,13 +52,38 @@ function installSkill(name) {
   }
   for (const target of INSTALL_TARGETS) {
     const dest = path.join(TARGET_ROOT, target.root, 'skills', name);
+    if (fs.existsSync(dest) && !replaceExisting) {
+      console.error(`  [error] Skill "${name}" already exists at ${target.root}/skills/${name}/ (use "update" to replace).`);
+      return false;
+    }
+    if (fs.existsSync(dest) && replaceExisting) {
+      fs.rmSync(dest, { recursive: true, force: true });
+    }
     copyDir(src, dest);
-    console.log(`  [skill] ${name} -> ${target.root}/skills/${name}/`);
+    const action = replaceExisting ? 'updated' : 'installed';
+    console.log(`  [skill] ${name} ${action} -> ${target.root}/skills/${name}/`);
+  }
+  return true;
+}
+
+function deleteSkill(name) {
+  let deletedAny = false;
+  for (const target of INSTALL_TARGETS) {
+    const dest = path.join(TARGET_ROOT, target.root, 'skills', name);
+    if (fs.existsSync(dest)) {
+      fs.rmSync(dest, { recursive: true, force: true });
+      console.log(`  [skill] ${name} deleted from ${target.root}/skills/${name}/`);
+      deletedAny = true;
+    }
+  }
+  if (!deletedAny) {
+    console.error(`  [error] Skill "${name}" is not installed.`);
+    return false;
   }
   return true;
 }
 
-function installAgent(name) {
+function installAgent(name, { replaceExisting = false } = {}) {
   const src = path.join(AGENTS_DIR, name);
   if (!fs.existsSync(src)) {
     console.error(`  [error] Agent pack "${name}" not found.`);
@@ -66,12 +91,52 @@ function installAgent(name) {
   }
   for (const target of INSTALL_TARGETS) {
     const dest = path.join(TARGET_ROOT, target.root, 'agents', name);
+    if (fs.existsSync(dest) && !replaceExisting) {
+      console.error(`  [error] Agent pack "${name}" already exists at ${target.root}/agents/${name}/ (use "update" to replace).`);
+      return false;
+    }
+    if (fs.existsSync(dest) && replaceExisting) {
+      fs.rmSync(dest, { recursive: true, force: true });
+    }
     copyDir(src, dest);
-    console.log(`  [agent] ${name} -> ${target.root}/agents/${name}/`);
+    const action = replaceExisting ? 'updated' : 'installed';
+    console.log(`  [agent] ${name} ${action} -> ${target.root}/agents/${name}/`);
   }
   return true;
 }
 
+function deleteAgent(name) {
+  let deletedAny = false;
+  for (const target of INSTALL_TARGETS) {
+    const dest = path.join(TARGET_ROOT, target.root, 'agents', name);
+    if (fs.existsSync(dest)) {
+      fs.rmSync(dest, { recursive: true, force: true });
+      console.log(`  [agent] ${name} deleted from ${target.root}/agents/${name}/`);
+      deletedAny = true;
+    }
+  }
+  if (!deletedAny) {
+    console.error(`  [error] Agent pack "${name}" is not installed.`);
+    return false;
+  }
+  return true;
+}
+
+function getInstalled(type) {
+  const installed = new Set();
+  for (const target of INSTALL_TARGETS) {
+    const dir = path.join(TARGET_ROOT, target.root, type);
+    if (fs.existsSync(dir)) {
+      for (const entry of fs.readdirSync(dir)) {
+        if (fs.statSync(path.join(dir, entry)).isDirectory()) {
+          installed.add(entry);
+        }
+      }
+    }
+  }
+  return [...installed].sort();
+}
+
 function printList() {
   const skills = getAvailable(SKILLS_DIR);
   const agents = getAvailable(AGENTS_DIR);
@@ -97,14 +162,27 @@ function buildMenu() {
   return [...skills, ...agents];
 }
 
-function installItems(items) {
+function buildDeleteMenu() {
+  const skills = getInstalled('skills').map(name => ({ type: 'skill', name }));
+  const agents = getInstalled('agents').map(name => ({ type: 'agent', name }));
+  return [...skills, ...agents];
+}
+
+function installItems(items, options = {}) {
+  for (const item of items) {
+    if (item.type === 'skill') installSkill(item.name, options);
+    else installAgent(item.name, options);
+  }
+}
+
+function deleteItems(items) {
   for (const item of items) {
-    if (item.type === 'skill') installSkill(item.name);
-    else installAgent(item.name);
+    if (item.type === 'skill') deleteSkill(item.name);
+    else deleteAgent(item.name);
   }
 }
 
-async function interactiveInstall() {
+async function interactiveInstall(mode = 'install') {
   const menu = buildMenu();
   if (menu.length === 0) {
     console.log('No skills or agent packs available.');
@@ -120,7 +198,40 @@ async function interactiveInstall() {
 
   const rl = readline.createInterface({ input: process.stdin, output: process.stdout });
   return new Promise(resolve => {
-    rl.question('Select items to install: ', answer => {
+    rl.question(`Select items to ${mode}: `, answer => {
+      rl.close();
+      const trimmed = answer.trim().toLowerCase();
+      let selected;
+      if (trimmed === 'all') {
+        selected = menu;
+      } else {
+        const indices = trimmed.split(/\s+/).map(n => parseInt(n, 10) - 1);
+        selected = indices.filter(i => i >= 0 && i < menu.length).map(i => menu[i]);
+      }
+      console.log('');
+      installItems(selected, { replaceExisting: mode === 'update' });
+      resolve();
+    });
+  });
+}
+
+async function interactiveDelete() {
+  const menu = buildDeleteMenu();
+  if (menu.length === 0) {
+    console.log('No installed skills or agent packs to delete.');
+    return;
+  }
+
+  console.log('\nInstalled (enter numbers separated by spaces, or "all"):\n');
+  menu.forEach((item, i) => {
+    const label = item.type === 'skill' ? 'skill' : 'agent';
+    console.log(`  [${i + 1}] ${label}: ${item.name}`);
+  });
+  console.log('');
+
+  const rl = readline.createInterface({ input: process.stdin, output: process.stdout });
+  return new Promise(resolve => {
+    rl.question('Select items to delete: ', answer => {
       rl.close();
       const trimmed = answer.trim().toLowerCase();
       let selected;
@@ -131,7 +242,7 @@ async function interactiveInstall() {
         selected = indices.filter(i => i >= 0 && i < menu.length).map(i => menu[i]);
       }
       console.log('');
-      installItems(selected);
+      deleteItems(selected);
       resolve();
     });
   });
@@ -153,6 +264,22 @@ function resolveNames(names) {
   return items;
 }
 
+function resolveDeleteNames(names) {
+  const installedSkills = getInstalled('skills');
+  const installedAgents = getInstalled('agents');
+  const items = [];
+  for (const name of names) {
+    if (installedSkills.includes(name)) {
+      items.push({ type: 'skill', name });
+    } else if (installedAgents.includes(name)) {
+      items.push({ type: 'agent', name });
+    } else {
+      console.error(`  [error] "${name}" is not installed.`);
+    }
+  }
+  return items;
+}
+
 async function main() {
   const args = process.argv.slice(2);
 
@@ -164,6 +291,12 @@ Usage:
   npx td-ai-tools install             Interactive install
   npx td-ai-tools install --all       Install everything
   npx td-ai-tools install <name...>   Install specific skills or agent packs
+  npx td-ai-tools update              Interactive update (replaces existing items)
+  npx td-ai-tools update --all        Update everything
+  npx td-ai-tools update <name...>    Update specific skills or agent packs
+  npx td-ai-tools delete              Interactive delete
+  npx td-ai-tools delete --all        Delete everything
+  npx td-ai-tools delete <name...>    Delete specific skills or agent packs
 
 Skills are installed to:   .claude/skills/<name>/
                          .agents/skills/<name>/
@@ -178,7 +311,7 @@ Agent packs installed to:  .claude/agents/<name>/
   console.log('------------');
 
   if (!cmd || cmd === 'install' && args.length === 1) {
-    await interactiveInstall();
+    await interactiveInstall('install');
     return;
   }
 
@@ -199,6 +332,38 @@ Agent packs installed to:  .claude/agents/<name>/
     return;
   }
 
+  if (cmd === 'update') {
+    const rest = args.slice(1);
+    if (rest.length === 0) {
+      await interactiveInstall('update');
+      return;
+    }
+    if (rest[0] === '--all') {
+      console.log('');
+      installItems(buildMenu(), { replaceExisting: true });
+    } else {
+      console.log('');
+      installItems(resolveNames(rest), { replaceExisting: true });
+    }
+    return;
+  }
+
+  if (cmd === 'delete') {
+    const rest = args.slice(1);
+    if (rest.length === 0) {
+      await interactiveDelete();
+      return;
+    }
+    if (rest[0] === '--all') {
+      console.log('');
+      deleteItems(buildDeleteMenu());
+    } else {
+      console.log('');
+      deleteItems(resolveDeleteNames(rest));
+    }
+    return;
+  }
+
   console.error(`Unknown command: "${cmd}". Run with --help for usage.`);
   process.exit(1);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "td-ai-tools",
-  "version": "1.0.6",
+  "version": "1.0.8",
   "description": "Install agent skills and packs into your project",
   "scripts": {
     "smoke:install": "./scripts/smoke-install.sh"

package/skills/playwright-cli/CODEX-CONFIG.md

@@ -8,7 +8,7 @@ machine or one project clone location.
 
 - `config.toml.template` — the source of truth. Uses two placeholders:
   - `__PROJECT_ROOT__` — replaced with the absolute path to the repo root
-  - `__CHROME_PATH__` — replaced with the absolute path to the Chrome/Chromium binary
+  - `__BROWSER_PATH__` — replaced with the absolute path to the Chromium/Chrome binary
 - `setup-codex.sh` — reads the template, substitutes the placeholders, writes
   `<repo>/.codex/config.toml`.
 
@@ -27,7 +27,7 @@ The script prints the values it resolved, e.g.:
 ```
 wrote /home/you/code/lio/.codex/config.toml
   PROJECT_ROOT=/home/you/code/lio
-  CHROME_PATH=/usr/bin/google-chrome
+  BROWSER_PATH=/usr/bin/chromium
 ```
 
 Re-run it any time the template changes or you move the repo.
@@ -41,17 +41,19 @@ Re-run it any time the template changes or you move the repo.
 2. Falls back to walking three directories up from the script
    (`.agents/skills/playwright-cli` → repo root) if the repo isn't a git checkout
 
-**`CHROME_PATH`** — derived in this order:
+**`BROWSER_PATH`** — derived in this order:
 
-1. The `CHROME_PATH` environment variable, if set
-2. First match from `PATH` of: `google-chrome`, `google-chrome-stable`, `chromium`,
-   `chromium-browser`
-3. Falls back to `/usr/bin/google-chrome` with a warning if nothing is found
+1. The `BROWSER_PATH` environment variable, if set
+2. The legacy `CHROME_PATH` environment variable, if set (kept for backwards compat)
+3. First match from `PATH` of: `chromium`, `chromium-browser`, `google-chrome`,
+   `google-chrome-stable` — Chromium is preferred because Chrome is often missing
+   on WSL shells
+4. Falls back to `/usr/bin/chromium` with a warning if nothing is found
 
-Override the Chrome binary explicitly when needed:
+Override the browser binary explicitly when needed:
 
 ```bash
-CHROME_PATH=/opt/google/chrome/chrome .agents/skills/playwright-cli/setup-codex.sh
+BROWSER_PATH=/usr/bin/chromium .agents/skills/playwright-cli/setup-codex.sh
 ```
 
 ## Why a generator instead of variable expansion

package/skills/playwright-cli/SKILL.md

@@ -158,6 +158,7 @@ playwright-cli video-stop video.webm
 ## Open parameters
 ```bash
 # Use specific browser when creating session
+playwright-cli open --browser=chromium
 playwright-cli open --browser=chrome
 playwright-cli open --browser=firefox
 playwright-cli open --browser=webkit

package/skills/playwright-cli/config.toml.template

@@ -19,10 +19,10 @@ inherit = "all"
 
 [shell_environment_policy.set]
 PLAYWRIGHT_BROWSERS_PATH = "__PROJECT_ROOT__/.artifacts/playwright/ms-playwright"
-PLAYWRIGHT_MCP_BROWSER = "chrome"
-PLAYWRIGHT_MCP_EXECUTABLE_PATH = "__CHROME_PATH__"
-PLAYWRIGHT_DAEMON_SOCKETS_DIR = "__PROJECT_ROOT__/.pw"
-BROWSER = "google-chrome"
+PLAYWRIGHT_MCP_BROWSER = "chromium"
+PLAYWRIGHT_MCP_EXECUTABLE_PATH = "__BROWSER_PATH__"
+PLAYWRIGHT_DAEMON_SOCKETS_DIR = "__PROJECT_ROOT__/.tmp/codex-playwright/sockets"
+BROWSER = "chromium"
 HOME = "__PROJECT_ROOT__/.tmp/codex-playwright/home"
 XDG_CONFIG_HOME = "__PROJECT_ROOT__/.tmp/codex-playwright/config"
 XDG_STATE_HOME = "__PROJECT_ROOT__/.tmp/codex-playwright/state"

package/skills/playwright-cli/setup-codex.sh

@@ -11,19 +11,19 @@ else
 fi
 OUTPUT="$PROJECT_ROOT/.codex/config.toml"
 
-CHROME_PATH="${CHROME_PATH:-$(command -v google-chrome || command -v google-chrome-stable || command -v chromium || command -v chromium-browser || true)}"
-if [ -z "$CHROME_PATH" ]; then
-  echo "warning: no Chrome/Chromium binary found on PATH; set CHROME_PATH=/path/to/chrome and rerun" >&2
-  CHROME_PATH="/usr/bin/google-chrome"
+BROWSER_PATH="${BROWSER_PATH:-${CHROME_PATH:-$(command -v chromium || command -v chromium-browser || command -v google-chrome || command -v google-chrome-stable || true)}}"
+if [ -z "$BROWSER_PATH" ]; then
+  echo "warning: no Chromium/Chrome binary found on PATH; set BROWSER_PATH=/path/to/browser and rerun" >&2
+  BROWSER_PATH="/usr/bin/chromium"
 fi
 
 mkdir -p "$(dirname "$OUTPUT")"
 
 sed \
   -e "s|__PROJECT_ROOT__|$PROJECT_ROOT|g" \
-  -e "s|__CHROME_PATH__|$CHROME_PATH|g" \
+  -e "s|__BROWSER_PATH__|$BROWSER_PATH|g" \
   "$TEMPLATE" > "$OUTPUT"
 
 echo "wrote $OUTPUT"
 echo "  PROJECT_ROOT=$PROJECT_ROOT"
-echo "  CHROME_PATH=$CHROME_PATH"
+echo "  BROWSER_PATH=$BROWSER_PATH"