td-ai-tools 1.1.10 → 1.1.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "td-ai-tools",
-  "version": "1.1.10",
+  "version": "1.1.11",
   "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."