td-ai-tools 1.0.6 → 1.0.7

package/README.md

@@ -33,6 +33,9 @@ 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
 ```
 
 The installer copies requested items into both agent layouts:
@@ -41,6 +44,8 @@ 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.
+
 ## 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,21 @@ 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 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,8 +74,16 @@ 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;
 }
@@ -97,14 +113,14 @@ function buildMenu() {
   return [...skills, ...agents];
 }
 
-function installItems(items) {
+function installItems(items, options = {}) {
   for (const item of items) {
-    if (item.type === 'skill') installSkill(item.name);
-    else installAgent(item.name);
+    if (item.type === 'skill') installSkill(item.name, options);
+    else installAgent(item.name, options);
   }
 }
 
-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 +136,7 @@ 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;
@@ -131,7 +147,7 @@ async function interactiveInstall() {
         selected = indices.filter(i => i >= 0 && i < menu.length).map(i => menu[i]);
       }
       console.log('');
-      installItems(selected);
+      installItems(selected, { replaceExisting: mode === 'update' });
       resolve();
     });
   });
@@ -164,6 +180,9 @@ 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
 
 Skills are installed to:   .claude/skills/<name>/
                          .agents/skills/<name>/
@@ -178,7 +197,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 +218,22 @@ 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;
+  }
+
   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.7",
   "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"