@xylabs/ts-scripts-yarn3 7.4.24 → 7.4.25

package/dist/xy/xyParseOptions.mjs

@@ -1,8 +1,48 @@
 // src/xy/xyParseOptions.ts
 import yargs from "yargs";
 import { hideBin } from "yargs/helpers";
+
+// src/lib/tryRunLocalScript.ts
+import { spawnSync } from "child_process";
+import { readFileSync } from "fs";
+import PATH from "path";
+import chalk from "chalk";
+function tryRunLocalScript(commandName) {
+  if (process.env.XY_LOCAL_SCRIPT === "1") return void 0;
+  const rootPkgPath = PATH.resolve(process.cwd(), "package.json");
+  let rootPkg;
+  try {
+    rootPkg = JSON.parse(readFileSync(rootPkgPath, "utf8"));
+  } catch {
+    return void 0;
+  }
+  if (!rootPkg.scripts?.[commandName]) return void 0;
+  console.log(chalk.blue(`Delegating "${commandName}" to local script`));
+  const result = spawnSync("yarn", ["run", commandName], {
+    cwd: process.cwd(),
+    encoding: "utf8",
+    env: {
+      FORCE_COLOR: "3",
+      ...process.env,
+      XY_LOCAL_SCRIPT: "1"
+    },
+    shell: true,
+    stdio: "inherit"
+  });
+  return result.status ?? 1;
+}
+
+// src/xy/xyParseOptions.ts
 var xyParseOptions = () => {
-  return yargs(hideBin(process.argv)).scriptName("yarn xy").option("jobs", {
+  return yargs(hideBin(process.argv)).scriptName("yarn xy").middleware((argv) => {
+    const commandName = argv._[0];
+    if (commandName && argv._.length <= 1) {
+      const result = tryRunLocalScript(commandName);
+      if (result !== void 0) {
+        process.exit(result);
+      }
+    }
+  }, true).option("jobs", {
     alias: "j",
     default: void 0,
     description: "Max parallel jobs",

package/dist/xy/xyParseOptions.mjs.map

@@ -1 +1 @@
-{"version":3,"sources":["../../src/xy/xyParseOptions.ts"],"sourcesContent":["import type { Argv } from 'yargs'\nimport yargs from 'yargs'\n// eslint-disable-next-line import-x/no-internal-modules\nimport { hideBin } from 'yargs/helpers'\n\nexport const xyParseOptions = (): Argv => {\n  return yargs(hideBin(process.argv))\n    .scriptName('yarn xy')\n    .option('jobs', {\n      alias: 'j',\n      default: undefined,\n      description: 'Max parallel jobs',\n      type: 'number',\n    })\n    .option('verbose', {\n      alias: 'v',\n      default: false,\n      description: 'Run with verbose logging',\n      type: 'boolean',\n    })\n    .option('incremental', {\n      alias: 'i',\n      default: false,\n      description: 'Attempt to perform the action only on changed packages',\n      type: 'boolean',\n    })\n    .option('profile', {\n      alias: 'p',\n      default: false,\n      description: 'Profile action',\n      type: 'boolean',\n    })\n}\n"],"mappings":";AACA,OAAO,WAAW;AAElB,SAAS,eAAe;AAEjB,IAAM,iBAAiB,MAAY;AACxC,SAAO,MAAM,QAAQ,QAAQ,IAAI,CAAC,EAC/B,WAAW,SAAS,EACpB,OAAO,QAAQ;AAAA,IACd,OAAO;AAAA,IACP,SAAS;AAAA,IACT,aAAa;AAAA,IACb,MAAM;AAAA,EACR,CAAC,EACA,OAAO,WAAW;AAAA,IACjB,OAAO;AAAA,IACP,SAAS;AAAA,IACT,aAAa;AAAA,IACb,MAAM;AAAA,EACR,CAAC,EACA,OAAO,eAAe;AAAA,IACrB,OAAO;AAAA,IACP,SAAS;AAAA,IACT,aAAa;AAAA,IACb,MAAM;AAAA,EACR,CAAC,EACA,OAAO,WAAW;AAAA,IACjB,OAAO;AAAA,IACP,SAAS;AAAA,IACT,aAAa;AAAA,IACb,MAAM;AAAA,EACR,CAAC;AACL;","names":[]}
+{"version":3,"sources":["../../src/xy/xyParseOptions.ts","../../src/lib/tryRunLocalScript.ts"],"sourcesContent":["import type { Argv } from 'yargs'\nimport yargs from 'yargs'\n// eslint-disable-next-line import-x/no-internal-modules\nimport { hideBin } from 'yargs/helpers'\n\nimport { tryRunLocalScript } from '../lib/index.ts'\n\nexport const xyParseOptions = (): Argv => {\n  return yargs(hideBin(process.argv))\n    .scriptName('yarn xy')\n    .middleware((argv) => {\n      const commandName = argv._[0] as string\n      if (commandName && argv._.length <= 1) {\n        const result = tryRunLocalScript(commandName)\n        if (result !== undefined) {\n          process.exit(result)\n        }\n      }\n    }, true)\n    .option('jobs', {\n      alias: 'j',\n      default: undefined,\n      description: 'Max parallel jobs',\n      type: 'number',\n    })\n    .option('verbose', {\n      alias: 'v',\n      default: false,\n      description: 'Run with verbose logging',\n      type: 'boolean',\n    })\n    .option('incremental', {\n      alias: 'i',\n      default: false,\n      description: 'Attempt to perform the action only on changed packages',\n      type: 'boolean',\n    })\n    .option('profile', {\n      alias: 'p',\n      default: false,\n      description: 'Profile action',\n      type: 'boolean',\n    })\n}\n","import { spawnSync } from 'node:child_process'\nimport { readFileSync } from 'node:fs'\nimport PATH from 'node:path'\n\nimport chalk from 'chalk'\n\n/**\n * Attempts to run a local root package.json script instead of the built-in\n * xy command. Returns the exit code if the override ran, or undefined if\n * no override applies (no matching script or recursion guard).\n */\nexport function tryRunLocalScript(commandName: string): number | undefined {\n  // Prevent infinite recursion if the local script calls yarn xy\n  if (process.env.XY_LOCAL_SCRIPT === '1') return undefined\n\n  // Check if root package.json has a matching script\n  const rootPkgPath = PATH.resolve(process.cwd(), 'package.json')\n  let rootPkg: { scripts?: Record<string, string> }\n  try {\n    rootPkg = JSON.parse(readFileSync(rootPkgPath, 'utf8'))\n  } catch {\n    return undefined\n  }\n\n  if (!rootPkg.scripts?.[commandName]) return undefined\n\n  console.log(chalk.blue(`Delegating \"${commandName}\" to local script`))\n  const result = spawnSync('yarn', ['run', commandName], {\n    cwd: process.cwd(),\n    encoding: 'utf8',\n    env: {\n      FORCE_COLOR: '3', ...process.env, XY_LOCAL_SCRIPT: '1',\n    },\n    shell: true,\n    stdio: 'inherit',\n  })\n  return result.status ?? 1\n}\n"],"mappings":";AACA,OAAO,WAAW;AAElB,SAAS,eAAe;;;ACHxB,SAAS,iBAAiB;AAC1B,SAAS,oBAAoB;AAC7B,OAAO,UAAU;AAEjB,OAAO,WAAW;AAOX,SAAS,kBAAkB,aAAyC;AAEzE,MAAI,QAAQ,IAAI,oBAAoB,IAAK,QAAO;AAGhD,QAAM,cAAc,KAAK,QAAQ,QAAQ,IAAI,GAAG,cAAc;AAC9D,MAAI;AACJ,MAAI;AACF,cAAU,KAAK,MAAM,aAAa,aAAa,MAAM,CAAC;AAAA,EACxD,QAAQ;AACN,WAAO;AAAA,EACT;AAEA,MAAI,CAAC,QAAQ,UAAU,WAAW,EAAG,QAAO;AAE5C,UAAQ,IAAI,MAAM,KAAK,eAAe,WAAW,mBAAmB,CAAC;AACrE,QAAM,SAAS,UAAU,QAAQ,CAAC,OAAO,WAAW,GAAG;AAAA,IACrD,KAAK,QAAQ,IAAI;AAAA,IACjB,UAAU;AAAA,IACV,KAAK;AAAA,MACH,aAAa;AAAA,MAAK,GAAG,QAAQ;AAAA,MAAK,iBAAiB;AAAA,IACrD;AAAA,IACA,OAAO;AAAA,IACP,OAAO;AAAA,EACT,CAAC;AACD,SAAO,OAAO,UAAU;AAC1B;;;AD9BO,IAAM,iBAAiB,MAAY;AACxC,SAAO,MAAM,QAAQ,QAAQ,IAAI,CAAC,EAC/B,WAAW,SAAS,EACpB,WAAW,CAAC,SAAS;AACpB,UAAM,cAAc,KAAK,EAAE,CAAC;AAC5B,QAAI,eAAe,KAAK,EAAE,UAAU,GAAG;AACrC,YAAM,SAAS,kBAAkB,WAAW;AAC5C,UAAI,WAAW,QAAW;AACxB,gBAAQ,KAAK,MAAM;AAAA,MACrB;AAAA,IACF;AAAA,EACF,GAAG,IAAI,EACN,OAAO,QAAQ;AAAA,IACd,OAAO;AAAA,IACP,SAAS;AAAA,IACT,aAAa;AAAA,IACb,MAAM;AAAA,EACR,CAAC,EACA,OAAO,WAAW;AAAA,IACjB,OAAO;AAAA,IACP,SAAS;AAAA,IACT,aAAa;AAAA,IACb,MAAM;AAAA,EACR,CAAC,EACA,OAAO,eAAe;AAAA,IACrB,OAAO;AAAA,IACP,SAAS;AAAA,IACT,aAAa;AAAA,IACb,MAAM;AAAA,EACR,CAAC,EACA,OAAO,WAAW;AAAA,IACjB,OAAO;AAAA,IACP,SAAS;AAAA,IACT,aAAa;AAAA,IACb,MAAM;AAAA,EACR,CAAC;AACL;","names":[]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xylabs/ts-scripts-yarn3",
-  "version": "7.4.24",
+  "version": "7.4.25",
   "description": "TypeScript project scripts",
   "keywords": [
     "xylabs",
@@ -67,37 +67,37 @@
     "cosmiconfig-typescript-loader": "~6.2.0",
     "cpy": "~13.2.1",
     "deepmerge": "~4.3.1",
-    "dependency-cruiser": "~17.3.8",
-    "eslint": "^10.0.3",
+    "dependency-cruiser": "~17.3.9",
+    "eslint": "^10.1.0",
     "find-up": "~8.0.0",
-    "get-tsconfig": "~4.13.6",
+    "get-tsconfig": "~4.13.7",
     "glob": "~13.0.6",
     "license-checker": "~25.0.1",
     "parse-git-config": "~3.0.0",
     "picomatch": "~4.0.3",
     "publint": "~0.3.18",
-    "rollup": "~4.59.0",
-    "rollup-plugin-dts": "~6.3.0",
+    "rollup": "~4.60.0",
+    "rollup-plugin-dts": "~6.4.1",
     "rollup-plugin-node-externals": "~8.1.2",
     "sort-package-json": "~3.6.1",
     "tsc-prog": "~2.3.0",
     "tsup": "~8.5.1",
-    "typedoc": "~0.28.17",
+    "typedoc": "~0.28.18",
     "types-package-json": "~2.0.39",
     "yargs": "~18.0.0"
   },
   "devDependencies": {
     "@types/eslint": "~9.6.1",
     "@types/license-checker": "~25.0.6",
-    "@types/node": "~25.4.0",
+    "@types/node": "~25.5.0",
     "@types/parse-git-config": "~3.0.4",
     "@types/picomatch": "~4.0.2",
     "@types/yargs": "~17.0.35",
-    "@xylabs/tsconfig": "~7.4.24",
-    "esbuild": "0.27.3",
+    "@xylabs/tsconfig": "~7.4.25",
+    "esbuild": "0.27.4",
     "types-package-json": "~2.0.39",
-    "typescript": "~5.9.3",
-    "vitest": "^4.0.18"
+    "typescript": "^5.9.3",
+    "vitest": "^4.1.1"
   },
   "peerDependencies": {
     "typescript": "~5"

package/templates/claude/skills/xylabs-e2e-setup/SKILL.md

@@ -104,7 +104,7 @@ export default defineConfig({
 })
 ```
 ### `packages/e2e/pages/BasePage.ts`
-The `goto` base method navigates and waits for `networkidle`. Subclasses override `goto()` with
+The `goto` base method navigates and waits for `domcontentloaded`. Subclasses override `goto()` with
 no arguments and call `super.goto('/<path>')`.
 ```typescript
 import type { Page } from '@playwright/test'
@@ -115,7 +115,7 @@ export class BasePage {
   }
   async goto(path: string): Promise<void> {
     await this.page.goto(path)
-    await this.page.waitForLoadState('networkidle')
+    await this.page.waitForLoadState('domcontentloaded')
   }
 }
 ```
@@ -180,17 +180,31 @@ into `packages/e2e`. Use the e2e package name you derived in Step 1.
 ```
 Only add scripts that don't already exist.
 ---
-## Step 4 — Install Playwright browsers and CLI tool
+## Step 4 — Update root `.gitignore`
+Playwright generates several output directories that should never be committed. Read the root
+`.gitignore` and append the following block **only if the entries are not already present**:
+```gitignore
+# Playwright
+/packages/e2e/test-results/
+/packages/e2e/playwright-report/
+/packages/e2e/blob-report/
+/packages/e2e/.playwright/
+```
+Do not create a new `.gitignore` if one doesn't exist — if there is no root `.gitignore`, skip
+this step and note it to the user.
+
+---
+## Step 5 — Install Playwright browsers and CLI tool
 After scaffolding, run the following two commands via the Bash tool:
 
-**4a — Install browser binaries:**
+**5a — Install browser binaries:**
 ```bash
 yarn workspace <derived-e2e-name> playwright install
 ```
 This downloads the Chromium, Firefox, and WebKit browser binaries. Without this step the tests
 will fail immediately with a "browser not found" error.
 
-**4b — Install the `@playwright/cli` package and its skills:**
+**5b — Install the `@playwright/cli` package and its skills:**
 ```bash
 yarn workspace <derived-e2e-name> add @playwright/cli@latest && yarn workspace <derived-e2e-name> playwright-cli install --skills
 ```
@@ -204,7 +218,6 @@ the setup.
 ---
 ## Conventions to preserve
 - All files use ESM (`import`/`export`), never CommonJS.
-- Import paths include the `.js` extension (e.g. `'./BasePage.js'`) — required for ESM + `moduleResolution: bundler`.
 - Playwright bracket notation for env vars: `process.env['CI']`, not `process.env.CI` — avoids TypeScript strict-mode complaints.
 - `strict: true` in `tsconfig.json`.
 - No `dist/` output — `noEmit: true` because Playwright compiles on the fly via its own ESM loader.