td-ai-tools 1.1.10 → 1.1.12

package/README.md

@@ -36,9 +36,11 @@ npx td-ai-tools list
 npx td-ai-tools install
 npx td-ai-tools install --all
 npx td-ai-tools install pr-solver
+npx td-ai-tools install --setup playwright-cli
 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 --setup playwright-cli
 npx td-ai-tools update pr-solver
 npx td-ai-tools delete
 npx td-ai-tools delete --all
@@ -53,6 +55,8 @@ If a skill bundles a sub-agent prompt (any `.md` with a `name:` field in `skills
 
 This keeps the installed assets available to both Claude-style and `.agents`-style project conventions.
 
+Some skills provide a recognized setup command (`setup.sh`, `scripts/setup.sh`, or `package.json` with `scripts.setup`). Non-interactive installs and updates run setup only when you pass `--setup`; interactive installs/updates ask for confirmation only when the selected skills include a recognized setup command. Accepted setup runs once inside each installed copy: `.claude/skills/<name>/` and `.agents/skills/<name>/`.
+
 `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.

package/bin/cli.js

@@ -12,7 +12,7 @@ import {
 } from '../lib/catalog.js';
 import { readFrontmatterField } from '../lib/frontmatter.js';
 import {
-  installSkill, installAgent, deleteSkill, deleteAgent,
+  installSkill, installAgent, deleteSkill, deleteAgent, skillHasSetup,
 } from '../lib/installer.js';
 
 const __filename = fileURLToPath(import.meta.url);
@@ -139,13 +139,18 @@ function hintFor(type, name) {
  *
  * @param {MenuItem[]} items - Items to install.
  * @param {{replaceExisting?: boolean}} [options] - Forwarded to the installer.
- * @returns {void}
+ * @returns {boolean} Whether every installer reported success.
  */
 function installItems(items, options = {}) {
+  const { runSetup = false, ...installOptions } = options;
+  let ok = true;
   for (const item of items) {
-    if (item.type === 'skill') installSkill(ctx, item.name, { ...options, report: status });
-    else installAgent(ctx, item.name, { ...options, report: status });
+    const installed = item.type === 'skill'
+      ? installSkill(ctx, item.name, { ...installOptions, runSetup, report: status })
+      : installAgent(ctx, item.name, { ...installOptions, report: status });
+    if (!installed) ok = false;
   }
+  return ok;
 }
 
 /**
@@ -354,6 +359,52 @@ function fromGroupValues(values) {
   });
 }
 
+
+/**
+ * Whether any selected skill has a recognized setup script.
+ *
+ * @param {MenuItem[]} items - Items selected for install/update.
+ * @returns {boolean}
+ */
+function hasSetupSkills(items) {
+  return items.some(item => item.type === 'skill' && skillHasSetup(ctx, item.name));
+}
+
+/**
+ * Prompts before running setup for selected skills that provide a recognized setup script.
+ *
+ * @param {MenuItem[]} items - Items selected for install/update.
+ * @returns {Promise<boolean>} Whether setup should run.
+ */
+async function confirmSetup(items) {
+  if (!hasSetupSkills(items)) return false;
+  const answer = await p.confirm({
+    message: 'Run setup scripts for selected skills that provide one? Setup runs in both .claude and .agents copies.',
+    initialValue: false,
+  });
+  if (p.isCancel(answer)) {
+    p.cancel('Cancelled.');
+    process.exit(0);
+  }
+  return Boolean(answer);
+}
+
+/**
+ * Removes recognized global install flags from positional arguments.
+ *
+ * @param {string[]} args - Raw command arguments after the command name.
+ * @returns {{rest: string[], runSetup: boolean}}
+ */
+function parseInstallFlags(args) {
+  let runSetup = false;
+  const rest = [];
+  for (const arg of args) {
+    if (arg === '--setup') runSetup = true;
+    else rest.push(arg);
+  }
+  return { rest, runSetup };
+}
+
 /**
  * Runs the interactive install/update flow (multiselect → apply). Exits the
  * process on a non-TTY stdin or user cancellation.
@@ -395,7 +446,13 @@ async function interactiveInstall(mode = 'install') {
     return;
   }
 
-  installItems(items, { replaceExisting: mode === 'update' });
+  const runSetup = await confirmSetup(items);
+  const ok = installItems(items, { replaceExisting: mode === 'update', runSetup });
+  if (!ok) {
+    p.outro(pc.red(`${mode === 'update' ? 'Update' : 'Install'} completed with errors.`));
+    process.exitCode = 1;
+    return;
+  }
   p.outro(mode === 'update' ? pc.green('Update complete.') : pc.green('Install complete.'));
 }
 
@@ -527,9 +584,13 @@ const HELP_TEXT = `Usage:
   npx td-ai-tools list                List available skills and agent packs
   npx td-ai-tools install             Interactive install
   npx td-ai-tools install --all       Install everything
+  npx td-ai-tools install --setup <name...>
+                                      Install and run recognized skill setup scripts
   npx td-ai-tools install <name...>   Install specific skills or agent packs
   npx td-ai-tools update              Interactive update (installed items in the catalog)
   npx td-ai-tools update --all        Update all installed items found in the catalog
+  npx td-ai-tools update --setup <name...>
+                                      Update and run recognized skill setup scripts
   npx td-ai-tools update <name...>    Update specific installed skills or agent packs
   npx td-ai-tools delete              Interactive delete
   npx td-ai-tools delete --all        Delete everything
@@ -581,17 +642,17 @@ async function main() {
   }
 
   if (cmd === 'install') {
-    const rest = args.slice(1);
+    const { rest, runSetup } = parseInstallFlags(args.slice(1));
     if (rest[0] === '--all') {
-      installItems(buildMenu());
+      if (!installItems(buildMenu(), { runSetup })) process.exitCode = 1;
     } else {
-      installItems(resolveNames(rest));
+      if (!installItems(resolveNames(rest), { runSetup })) process.exitCode = 1;
     }
     return;
   }
 
   if (cmd === 'update') {
-    const rest = args.slice(1);
+    const { rest, runSetup } = parseInstallFlags(args.slice(1));
     if (rest.length === 0) {
       await interactiveInstall('update');
       return;
@@ -602,9 +663,9 @@ async function main() {
         status('info', 'No installed skills or agent packs match the catalog.');
         return;
       }
-      installItems(menu, { replaceExisting: true });
+      if (!installItems(menu, { replaceExisting: true, runSetup })) process.exitCode = 1;
     } else {
-      installItems(resolveUpdateNames(rest), { replaceExisting: true });
+      if (!installItems(resolveUpdateNames(rest), { replaceExisting: true, runSetup })) process.exitCode = 1;
     }
     return;
   }

package/lib/installer.js

@@ -6,6 +6,7 @@
  */
 import fs from 'node:fs';
 import path from 'node:path';
+import { spawnSync } from 'node:child_process';
 import { copyDir } from './fs-utils.js';
 import { bundledAgentsIn } from './catalog.js';
 
@@ -24,6 +25,82 @@ import { bundledAgentsIn } from './catalog.js';
 /** @type {ReportFn} */
 const noop = () => {};
 
+/**
+ * Recognized setup commands, in priority order, relative to an installed skill.
+ * @type {{file: string, command: string, args: string[]}[]}
+ */
+const SKILL_SETUP_CANDIDATES = [
+  { file: 'setup.sh', command: 'bash', args: ['setup.sh'] },
+  { file: path.join('scripts', 'setup.sh'), command: 'bash', args: [path.join('scripts', 'setup.sh')] },
+  { file: 'package.json', command: 'npm', args: ['run', 'setup'] },
+];
+
+/**
+ * Finds the setup command for a skill directory, if one uses a recognized convention.
+ *
+ * @param {string} skillDir - Source or installed skill directory.
+ * @returns {({file: string, command: string, args: string[]}|null)} The setup command, or `null`.
+ */
+export function findSkillSetup(skillDir) {
+  for (const candidate of SKILL_SETUP_CANDIDATES) {
+    const filePath = path.join(skillDir, candidate.file);
+    if (!fs.existsSync(filePath)) continue;
+    if (candidate.file === 'package.json') {
+      try {
+        const pkg = JSON.parse(fs.readFileSync(filePath, 'utf8'));
+        if (!pkg?.scripts?.setup) continue;
+      } catch {
+        continue;
+      }
+    }
+    return candidate;
+  }
+  return null;
+}
+
+/**
+ * Whether a catalog skill provides a setup command using a recognized convention.
+ *
+ * @param {Ctx} ctx
+ * @param {string} name - Skill name.
+ * @returns {boolean}
+ */
+export function skillHasSetup(ctx, name) {
+  return Boolean(findSkillSetup(path.join(ctx.skillsDir, name)));
+}
+
+/**
+ * Runs a skill setup command from the installed skill directory.
+ *
+ * @param {string} name - Skill name.
+ * @param {Target} target - The destination layout.
+ * @param {string} dest - Installed skill directory.
+ * @param {ReportFn} report
+ * @returns {boolean} Whether setup succeeded or no setup existed.
+ */
+function runSkillSetup(name, target, dest, report) {
+  const setup = findSkillSetup(dest);
+  if (!setup) return true;
+
+  report('step', `setup: ${name} running ${setup.command} ${setup.args.join(' ')} in ${target.root}/skills/${name}/`);
+  const result = spawnSync(setup.command, setup.args, {
+    cwd: dest,
+    stdio: 'inherit',
+    shell: false,
+  });
+  if (result.error) {
+    report('error', `setup: ${name} failed in ${target.root}/skills/${name}/: ${result.error.message}`);
+    return false;
+  }
+  if (result.status !== 0) {
+    report('error', `setup: ${name} failed in ${target.root}/skills/${name}/ with exit code ${result.status}`);
+    return false;
+  }
+  report('success', `setup: ${name} completed in ${target.root}/skills/${name}/`);
+  return true;
+}
+
+
 /**
  * Copies an item's bundled sub-agents into a target's `agents/` directory.
  *
@@ -52,10 +129,12 @@ function registerBundledAgents(ctx, target, srcDir, report) {
  * @param {string} name - Skill name.
  * @param {object} [options]
  * @param {boolean} [options.replaceExisting=false] - Overwrite an existing install (used by `update`).
+ * @param {boolean} [options.runSetup=false] - Run recognized setup scripts after each target copy.
  * @param {ReportFn} [options.report] - Progress sink.
- * @returns {boolean} `false` if the skill is not in the catalog, otherwise `true`.
+ * @returns {boolean} `false` if the skill is not in the catalog or setup fails, otherwise `true`.
  */
-export function installSkill(ctx, name, { replaceExisting = false, report = noop } = {}) {
+export function installSkill(ctx, name, { replaceExisting = false, runSetup = false, report = noop } = {}) {
+  let ok = true;
   const src = path.join(ctx.skillsDir, name);
   if (!fs.existsSync(src)) {
     report('error', `Skill "${name}" not found.`);
@@ -71,8 +150,9 @@ export function installSkill(ctx, name, { replaceExisting = false, report = noop
     copyDir(src, dest);
     report('success', `skill: ${name} ${replaceExisting ? 'updated' : 'installed'} → ${target.root}/skills/${name}/`);
     registerBundledAgents(ctx, target, src, report);
+    if (runSetup && !runSkillSetup(name, target, dest, report)) ok = false;
   }
-  return true;
+  return ok;
 }
 
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "td-ai-tools",
-  "version": "1.1.10",
+  "version": "1.1.12",
   "description": "Install agent skills and packs into your project",
   "type": "module",
   "scripts": {

package/skills/README.md

@@ -15,11 +15,11 @@
 - `pull-request`: Generates well-structured GitHub pull request descriptions for Shopify theme development teams by analyzing…
 - `pull-request-statamic`: Generates GitHub pull request descriptions for Statamic and Laravel development by analyzing git diffs and…
 - `record-changes`: Update `docs/changes.md` by summarizing the current branch against the primary development branch.
-- `scry`: Single-site visual regression workflow for comparing a live URL against a preview/staging URL in Playwright…
-- `shopify-cli`: Shopify CLI workflows for theme development. Use when the user needs to run or explain Shopify theme commands,…
+- `shopify-cli`: Shopify CLI workflows for theme development.
 - `stylesheet-migration`: Migrate Shopify Liquid `{% stylesheet %}` blocks into theme CSS assets using bundled Python scripts.
 - `td-js-vanilla-rules`: Theory Digital vanilla JavaScript standards for Shopify theme work.
 - `td-review`: Run parallel code review agents on a PR (including TD theme compliance) and produce a synthesized findings…
+- `visual-regression`: Single-site visual regression workflow for comparing a live URL against a preview/staging URL in Playwright…
 
 ## Skill Structure Convention
 - `<skill-name>/SKILL.md`

package/skills/{scry → visual-regression}/SKILL.md

@@ -1,13 +1,14 @@
 ---
-name: scry
-version: 1.0.0
+name: visual-regression
+version: 2.0.0
 description: Single-site visual regression workflow for comparing a live URL against a preview/staging URL in Playwright UI mode. Use when the user provides or asks to compare two links, one live/baseline link and one preview link, wants fresh or reused live snapshots, or needs an ad hoc visual check that stores generated paths and baselines inside the skill folder rather than adding a normal CI site.
 ---
 
-# Scry
+# Visual Regression
 
-Use `scry` for temporary visual regression checks between one live site and one preview site.
+Use `visual-regression` for temporary visual regression checks between one live site and one preview site.
 It keeps state in this skill folder so the workflow is separate from the repo's regular `sites/` configs.
+Prefer over-analyzing with thorough crawls to tightly scoped reviews.
 
 ## Workflow
 
@@ -16,7 +17,7 @@ It keeps state in this skill folder so the workflow is separate from the repo's
 3. Run the bundled script from the repository root:
 
 ```bash
-python3 .agents/skills/scry/scripts/scry.py <live-url> <preview-url> --refresh
+python3 .agents/skills/visual-regression/scripts/visual_regression.py <live-url> <preview-url> --refresh
 ```
 
 Use `--reuse` instead of `--refresh` when the user wants to compare against existing baselines.
@@ -24,11 +25,11 @@ The script opens Playwright UI mode by default.
 
 ## What The Script Does
 
-- Writes `.agents/skills/scry/config.json` with the latest live and preview URLs.
-- On first run, generates `.agents/skills/scry/paths.json` from the live URL's sitemap.
-- Reuses `.agents/skills/scry/paths.json` on later runs unless `--regenerate-paths` is passed.
-- Stores live baseline screenshots in `.agents/skills/scry/baselines/`.
-- Generates runtime Playwright files under `.agents/skills/scry/runtime/`.
+- Writes `.agents/skills/visual-regression/config.json` with the latest live and preview URLs.
+- On first run, generates `.agents/skills/visual-regression/paths.json` from the live URL's sitemap.
+- Reuses `.agents/skills/visual-regression/paths.json` on later runs unless `--regenerate-paths` is passed.
+- Stores live baseline screenshots in `.agents/skills/visual-regression/baselines/`.
+- Generates runtime Playwright files under `.agents/skills/visual-regression/runtime/`.
 - Runs `npx playwright test --ui` against the generated runtime config.
 
 If sitemap generation fails on the first run, the script falls back to a homepage-only `paths.json`.
@@ -39,25 +40,25 @@ After generation, inspect `paths.json` and trim duplicate template paths when th
 Refresh baselines and open UI comparison:
 
 ```bash
-python3 .agents/skills/scry/scripts/scry.py https://example.com https://preview.example.com --refresh
+python3 .agents/skills/visual-regression/scripts/visual_regression.py https://example.com https://preview.example.com --refresh
 ```
 
 Reuse existing baselines:
 
 ```bash
-python3 .agents/skills/scry/scripts/scry.py https://example.com https://preview.example.com --reuse
+python3 .agents/skills/visual-regression/scripts/visual_regression.py https://example.com https://preview.example.com --reuse
 ```
 
 Regenerate paths from the live sitemap:
 
 ```bash
-python3 .agents/skills/scry/scripts/scry.py https://example.com https://preview.example.com --refresh --regenerate-paths
+python3 .agents/skills/visual-regression/scripts/visual_regression.py https://example.com https://preview.example.com --refresh --regenerate-paths
 ```
 
 Run headless instead of UI mode:
 
 ```bash
-python3 .agents/skills/scry/scripts/scry.py https://example.com https://preview.example.com --refresh --no-ui
+python3 .agents/skills/visual-regression/scripts/visual_regression.py https://example.com https://preview.example.com --refresh --no-ui
 ```
 
 Useful options:
@@ -75,25 +76,25 @@ Some theme repositories do not have a local `package.json`, `node_modules/`, or
 Create the temporary install directory:
 
 ```bash
-mkdir -p /tmp/scry-playwright
+mkdir -p /tmp/visual-regression-playwright
 ```
 
 Install Playwright Test there:
 
 ```bash
-npm install --prefix /tmp/scry-playwright @playwright/test
+npm install --prefix /tmp/visual-regression-playwright @playwright/test
 ```
 
 Install the browser needed for the run:
 
 ```bash
-/tmp/scry-playwright/node_modules/.bin/playwright install chromium
+/tmp/visual-regression-playwright/node_modules/.bin/playwright install chromium
 ```
 
-Generate the scry config and paths from the repository root:
+Generate the visual regression config and paths from the repository root:
 
 ```bash
-python3 .agents/skills/scry/scripts/scry.py "$LIVE_URL" "$PREVIEW_URL" --refresh --regenerate-paths --no-ui --browser=chromium
+python3 .agents/skills/visual-regression/scripts/visual_regression.py "$LIVE_URL" "$PREVIEW_URL" --refresh --regenerate-paths --no-ui --browser=chromium
 ```
 
 If `paths.json` is large, inspect it and trim duplicate template paths before the full comparison.
@@ -101,13 +102,13 @@ If `paths.json` is large, inspect it and trim duplicate template paths before th
 Run the generated suite with the temporary Playwright install:
 
 ```bash
-env NODE_PATH=/tmp/scry-playwright/node_modules \
-  SCRY_CONFIG="$PWD/.agents/skills/scry/config.json" \
-  SCRY_PATHS="$PWD/.agents/skills/scry/paths.json" \
-  SCRY_BASELINE_DIR="$PWD/.agents/skills/scry/baselines" \
-  SCRY_REFRESH=1 \
-  /tmp/scry-playwright/node_modules/.bin/playwright test \
-  --config "$PWD/.agents/skills/scry/runtime/playwright.config.cjs"
+env NODE_PATH=/tmp/visual-regression-playwright/node_modules \
+  VISUAL_REGRESSION_CONFIG="$PWD/.agents/skills/visual-regression/config.json" \
+  VISUAL_REGRESSION_PATHS="$PWD/.agents/skills/visual-regression/paths.json" \
+  VISUAL_REGRESSION_BASELINE_DIR="$PWD/.agents/skills/visual-regression/baselines" \
+  VISUAL_REGRESSION_REFRESH=1 \
+  /tmp/visual-regression-playwright/node_modules/.bin/playwright test \
+  --config "$PWD/.agents/skills/visual-regression/runtime/playwright.config.cjs"
 ```
 
 ## Cleanup
@@ -121,7 +122,7 @@ rm -rf playwright-report test-results
 Remove the temporary dependency install:
 
 ```bash
-rm -rf /tmp/scry-playwright
+rm -rf /tmp/visual-regression-playwright
 ```
 
 Only remove downloaded Playwright browsers when intentionally resetting local tooling:
@@ -130,12 +131,12 @@ Only remove downloaded Playwright browsers when intentionally resetting local to
 rm -rf ~/.cache/ms-playwright
 ```
 
-Keep these scry artifacts unless intentionally resetting the comparison state:
+Keep these visual regression artifacts unless intentionally resetting the comparison state:
 
-- `.agents/skills/scry/config.json`
-- `.agents/skills/scry/paths.json`
-- `.agents/skills/scry/runtime/`
-- `.agents/skills/scry/baselines/`
+- `.agents/skills/visual-regression/config.json`
+- `.agents/skills/visual-regression/paths.json`
+- `.agents/skills/visual-regression/runtime/`
+- `.agents/skills/visual-regression/baselines/`
 
 ## Playwright CLI
 

package/skills/visual-regression/agents/openai.yaml

@@ -0,0 +1,4 @@
+interface:
+  display_name: "Visual Regression"
+  short_description: "Compare live and preview sites visually."
+  default_prompt: "Use $visual-regression to compare a live site URL against a preview URL."

package/skills/{scry/scripts/scry.py → visual-regression/scripts/visual_regression.py}

@@ -1,5 +1,5 @@
 #!/usr/bin/env python3
-"""Ad hoc visual regression runner for the scry skill.
+"""Ad hoc visual regression runner for the visual-regression skill.
 
 The Python entrypoint owns URL/path discovery and writes the runtime Playwright
 files used for comparison. Playwright still runs through `npx playwright test`
@@ -39,9 +39,9 @@ PLAYWRIGHT_SPEC = r'''const { test, expect } = require("@playwright/test");
 const fs = require("fs");
 const path = require("path");
 
-const config = JSON.parse(fs.readFileSync(process.env.SCRY_CONFIG, "utf8"));
-const paths = JSON.parse(fs.readFileSync(process.env.SCRY_PATHS, "utf8"));
-const refresh = process.env.SCRY_REFRESH === "1";
+const config = JSON.parse(fs.readFileSync(process.env.VISUAL_REGRESSION_CONFIG, "utf8"));
+const paths = JSON.parse(fs.readFileSync(process.env.VISUAL_REGRESSION_PATHS, "utf8"));
+const refresh = process.env.VISUAL_REGRESSION_REFRESH === "1";
 
 function joinUrl(baseUrl, pagePath) {
   const base = new URL(baseUrl);
@@ -179,7 +179,7 @@ for (const pathEntry of paths) {
 
 def parse_args(argv: list[str] | None = None) -> argparse.Namespace:
     parser = argparse.ArgumentParser(
-        prog="scry.py",
+        prog="visual_regression.py",
         formatter_class=argparse.RawDescriptionHelpFormatter,
         description=(
             "Compare a live URL against a preview URL with generated "
@@ -292,7 +292,7 @@ def fetch_xml(url: str) -> str:
     request = urllib.request.Request(
         url,
         headers={
-            "User-Agent": "ScryVisualRegression/1.0",
+            "User-Agent": "VisualRegression/1.0",
             "Accept": "application/xml, text/xml, */*",
         },
     )
@@ -417,7 +417,7 @@ def resolve_refresh_choice(args: argparse.Namespace) -> bool:
     if isinstance(args.refresh, bool):
         return args.refresh
     if not sys.stdin.isatty():
-        raise RuntimeError("Choose --refresh or --reuse. In non-interactive runs, scry will not guess.")
+        raise RuntimeError("Choose --refresh or --reuse. In non-interactive runs, visual regression will not guess.")
     answer = input("Refresh live baselines before comparing? [Y/n] ")
     return not re.match(r"^n(o)?$", answer.strip(), re.I)
 
@@ -433,7 +433,7 @@ def browser_project(browser: str) -> str:
 def write_runtime_files(config: dict[str, Any]) -> Path:
     RUNTIME_DIR.mkdir(parents=True, exist_ok=True)
 
-    spec_path = RUNTIME_DIR / "scry.spec.cjs"
+    spec_path = RUNTIME_DIR / "visual-regression.spec.cjs"
     playwright_config_path = RUNTIME_DIR / "playwright.config.cjs"
 
     spec_path.write_text(PLAYWRIGHT_SPEC, encoding="utf-8")
@@ -472,16 +472,16 @@ def run_playwright(playwright_config_path: Path, ui: bool, refresh: bool) -> int
     env = os.environ.copy()
     env.update(
         {
-            "SCRY_CONFIG": str(CONFIG_FILE),
-            "SCRY_PATHS": str(PATHS_FILE),
-            "SCRY_BASELINE_DIR": str(BASELINE_DIR),
-            "SCRY_REFRESH": "1" if refresh else "0",
+            "VISUAL_REGRESSION_CONFIG": str(CONFIG_FILE),
+            "VISUAL_REGRESSION_PATHS": str(PATHS_FILE),
+            "VISUAL_REGRESSION_BASELINE_DIR": str(BASELINE_DIR),
+            "VISUAL_REGRESSION_REFRESH": "1" if refresh else "0",
         }
     )
     try:
         completed = subprocess.run(args, env=env)
     except FileNotFoundError as exc:
-        raise RuntimeError("npx was not found; install Node.js/npm before running scry.") from exc
+        raise RuntimeError("npx was not found; install Node.js/npm before running visual regression.") from exc
     return completed.returncode
 
 
@@ -522,8 +522,8 @@ def main(argv: list[str] | None = None) -> int:
     write_json(CONFIG_FILE, config)
     playwright_config_path = write_runtime_files(config)
 
-    print(f"Scry paths: {PATHS_FILE} ({len(paths)} paths)")
-    print(f"Scry baselines: {BASELINE_DIR}")
+    print(f"Visual regression paths: {PATHS_FILE} ({len(paths)} paths)")
+    print(f"Visual regression baselines: {BASELINE_DIR}")
     print(
         "Refreshing live baselines during this run."
         if refresh
@@ -539,5 +539,5 @@ if __name__ == "__main__":
     except KeyboardInterrupt:
         sys.exit(130)
     except Exception as exc:
-        print(f"scry: {exc}", file=sys.stderr)
+        print(f"visual-regression: {exc}", file=sys.stderr)
         sys.exit(1)

package/skills/{scry/tests/test_scry.py → visual-regression/tests/test_visual_regression.py}

@@ -1,6 +1,6 @@
-"""Tests for scry.py helper behavior.
+"""Tests for visual_regression.py helper behavior.
 
-Run: python3 skills/scry/tests/test_scry.py
+Run: python3 skills/visual-regression/tests/test_visual_regression.py
 """
 
 from __future__ import annotations
@@ -10,20 +10,20 @@ import sys
 from pathlib import Path
 
 
-_SCRIPT = Path(__file__).resolve().parent.parent / "scripts" / "scry.py"
-_spec = importlib.util.spec_from_file_location("scry", _SCRIPT)
+_SCRIPT = Path(__file__).resolve().parent.parent / "scripts" / "visual_regression.py"
+_spec = importlib.util.spec_from_file_location("visual_regression", _SCRIPT)
 assert _spec and _spec.loader
-scry = importlib.util.module_from_spec(_spec)
-_spec.loader.exec_module(scry)
+visual_regression = importlib.util.module_from_spec(_spec)
+_spec.loader.exec_module(visual_regression)
 
 
 def test_normalize_url_adds_origin_slash():
-    assert scry.normalize_url("https://example.com", "live URL") == "https://example.com/"
+    assert visual_regression.normalize_url("https://example.com", "live URL") == "https://example.com/"
 
 
 def test_normalize_url_rejects_non_http():
     try:
-        scry.normalize_url("ftp://example.com", "live URL")
+        visual_regression.normalize_url("ftp://example.com", "live URL")
     except ValueError as exc:
         assert "http or https" in str(exc)
     else:
@@ -31,14 +31,14 @@ def test_normalize_url_rejects_non_http():
 
 
 def test_normalize_url_path_and_name():
-    assert scry.normalize_url_path("about/team/") == "/about/team"
-    assert scry.normalize_url_path("https://example.com/products/widget/") == "/products/widget"
-    assert scry.path_name("/") == "homepage"
-    assert scry.path_name("/products/widget!") == "products-widget"
+    assert visual_regression.normalize_url_path("about/team/") == "/about/team"
+    assert visual_regression.normalize_url_path("https://example.com/products/widget/") == "/products/widget"
+    assert visual_regression.path_name("/") == "homepage"
+    assert visual_regression.path_name("/products/widget!") == "products-widget"
 
 
 def test_unique_sorted_paths_dedupes_by_path():
-    paths = scry.unique_sorted_paths(
+    paths = visual_regression.unique_sorted_paths(
         [
             {"name": "z", "path": "/about/"},
             {"name": "a", "path": "/"},
@@ -63,19 +63,19 @@ def test_parse_sitemap_index_recurses_and_filters_origin():
             <url><loc>https://external.example.com/ignore/</loc></url>
           </urlset>""",
     }
-    original_fetch_xml = scry.fetch_xml
+    original_fetch_xml = visual_regression.fetch_xml
     try:
-        scry.fetch_xml = lambda url: fixtures[url]
-        assert scry.parse_sitemap(
+        visual_regression.fetch_xml = lambda url: fixtures[url]
+        assert visual_regression.parse_sitemap(
             "https://example.com/sitemap.xml",
             "https://example.com/",
         ) == [{"name": "about", "path": "/about"}]
     finally:
-        scry.fetch_xml = original_fetch_xml
+        visual_regression.fetch_xml = original_fetch_xml
 
 
 def test_parse_args_defaults_browsers():
-    args = scry.parse_args(["https://a.test", "https://b.test", "--reuse"])
+    args = visual_regression.parse_args(["https://a.test", "https://b.test", "--reuse"])
     assert args.refresh is False
     assert args.browsers == ["chromium", "firefox", "webkit"]
 

package/skills/scry/agents/openai.yaml

@@ -1,4 +0,0 @@
-interface:
-  display_name: "Scry"
-  short_description: "Compare live and preview sites visually."
-  default_prompt: "Use $scry to compare a live site URL against a preview URL."