@andwhy/e2e-pipeline 0.14.0 → 0.14.2

package/dist/bin/init.js

@@ -120,9 +120,11 @@ const run = async () => {
     }
     const createdWorkflow = result.created.some((f) => f.includes(`.github${path.sep}workflows`));
     if (createdWorkflow) {
+        const heading = result.replacedWorkflows.length > 0
+            ? "\nGitHub workflows replaced with callers of the pipeline's reusable workflows. They\nuse the repository secrets the old ones used"
+            : "\nGitHub workflows created. They need repository secrets before they run";
         console.log([
-            "\nGitHub workflows created. They need repository secrets before they run",
-            "(values in 1Password):",
+            `${heading} (values in 1Password):`,
             "  CHECKLY_API_KEY, CHECKLY_ACCOUNT_ID      — deploy monitors, auto-heal",
             "  VERCEL_AUTOMATION_BYPASS_SECRET          — PR tests through Vercel",
             "                                             deployment protection",

package/dist/utils/copy.d.ts

@@ -4,6 +4,8 @@ interface CopyResult {
     regenerated: string[];
     /** Pre-plugin copies of package files deleted from the project (--force). */
     removed: string[];
+    /** Workflows --update-workflows replaced (a subset of created). */
+    replacedWorkflows: string[];
     /** Things the scaffold cannot fix itself and the user must act on. */
     warnings: string[];
 }

package/dist/utils/copy.js

@@ -1,3 +1,4 @@
+import { execSync } from "node:child_process";
 import path from "node:path";
 import fs from "fs-extra";
 // The agents, the skill and the always-on rules ship as the `e2e-pipeline`
@@ -130,6 +131,8 @@ const writeWorkflows = async (githubSrc, projectRoot, updateWorkflows, result) =
         await fs.ensureDir(path.dirname(destPath));
         await fs.copy(srcPath, destPath);
         result.created.push(destPath);
+        if (exists)
+            result.replacedWorkflows.push(destPath);
     }
     if (stale.length > 0) {
         result.warnings.push(`Workflows from before 0.14.0: ${stale.join(", ")}. They still work, but changes to them ` +
@@ -254,12 +257,55 @@ export const findSpecsOutsideE2e = async (projectRoot) => {
     await walk(testsRoot);
     return [...found].sort();
 };
+// A repo that ignores `.claude/*` (a common pattern from before the plugin,
+// keeping settings.local.json and session files out of git) silently drops
+// the one file the plugin needs: nobody else gets the plugin entry, and
+// nothing in the diff shows it is missing. Git decides: `check-ignore`
+// applies every rule the repo has, so a negation is appended only when the
+// file really is ignored, at the end of .gitignore where it wins over the
+// rule above it. Outside a git repo there is nothing to do.
+const SETTINGS_REL = ".claude/settings.json";
+const isGitIgnored = (projectRoot, rel) => {
+    try {
+        execSync(`git check-ignore -q -- ${rel}`, {
+            cwd: projectRoot,
+            stdio: "ignore",
+        });
+        return true;
+    }
+    catch {
+        // Exit 1: not ignored. Exit 128: not a git repository. Either way, the
+        // file is not being dropped by an ignore rule.
+        return false;
+    }
+};
+const unignoreClaudeSettings = async (projectRoot, result) => {
+    if (!isGitIgnored(projectRoot, SETTINGS_REL))
+        return;
+    const gitignorePath = path.join(projectRoot, ".gitignore");
+    const existing = (await fs.pathExists(gitignorePath))
+        ? await fs.readFile(gitignorePath, "utf-8")
+        : "";
+    const separator = existing.length > 0 && !existing.endsWith("\n") ? "\n" : "";
+    await fs.appendFile(gitignorePath, `${separator}\n# The e2e-pipeline plugin entry is committed (settings.local.json stays ignored)\n!${SETTINGS_REL}\n`);
+    if (!result.created.includes(gitignorePath))
+        result.created.push(gitignorePath);
+    if (isGitIgnored(projectRoot, SETTINGS_REL)) {
+        result.warnings.push(`.gitignore still ignores ${SETTINGS_REL} after appending \`!${SETTINGS_REL}\` ` +
+            `(a rule ignores the .claude directory itself, which no negation can undo). ` +
+            `Change that rule to \`.claude/*\` so the plugin entry can be committed.`);
+        return;
+    }
+    result.warnings.push(`.gitignore ignored ${SETTINGS_REL}, which carries the ${PLUGIN_ID} plugin entry every ` +
+        `collaborator needs. Appended \`!${SETTINGS_REL}\`; commit both files.`);
+};
 export const scaffold = async (templatesRoot, projectRoot, { force = false, fileFormat = "ts", resetConfig = false, updateWorkflows = false, } = {}) => {
     const result = {
         created: [],
         skipped: [],
         regenerated: [],
         removed: [],
+        replacedWorkflows: [],
         warnings: [],
     };
     const configExt = fileFormat === "mjs" ? "mjs" : "ts";
@@ -308,7 +354,9 @@ export const scaffold = async (templatesRoot, projectRoot, { force = false, file
         const srcPath = path.join(sharedSrc, src);
         const destPath = path.join(projectRoot, dest);
         const exists = await fs.pathExists(destPath);
-        if (exists && resetConfig && src.endsWith(`.config.template.${configExt}`)) {
+        if (exists &&
+            resetConfig &&
+            src.endsWith(`.config.template.${configExt}`)) {
             // checkly.config.new.ts, not checkly.config.ts.new — keeping the real
             // extension means prettier can format it and the editor highlights it,
             // so the diff against your config shows content rather than style.
@@ -332,5 +380,6 @@ export const scaffold = async (templatesRoot, projectRoot, { force = false, file
     // walks the user through adding them.
     await writeWorkflows(path.join(templatesRoot, "github"), projectRoot, updateWorkflows, result);
     await appendGitignore(projectRoot, result);
+    await unignoreClaudeSettings(projectRoot, result);
     return result;
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@andwhy/e2e-pipeline",
-  "version": "0.14.0",
+  "version": "0.14.2",
   "description": "Scaffolds the full e2e testing pipeline: agent-assisted Playwright tests, CI runs against Vercel previews, and Checkly monitors that follow production deploys",
   "type": "module",
   "license": "MIT",
@@ -23,7 +23,8 @@
     "build": "tsc",
     "build:plugin": "node scripts/build-plugin.mjs",
     "check:plugin": "node scripts/build-plugin.mjs --check",
-    "prepublishOnly": "npm run build && npm run check:plugin"
+    "prepublishOnly": "npm run build && npm run check:plugin",
+    "skill:zip": "node scripts/zip-skill.mjs"
   },
   "dependencies": {
     "fs-extra": "^11.3.0",

package/plugin/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "e2e-pipeline",
   "displayName": "&why e2e pipeline",
-  "version": "0.14.0",
+  "version": "0.14.2",
   "description": "Agent-assisted Playwright e2e tests: planner, generator and healer agents plus the playwright-test-pipeline skill that drives Plan → Generate → Heal, Checkly preview validation and deployment.",
   "author": {
     "name": "&why",

package/plugin/skills/playwright-test-pipeline/SKILL.md

@@ -6,6 +6,8 @@ description: Guides the full workflow for creating and maintaining Playwright
   deployment. Use when the user wants to create, extend, or fix e2e/Playwright
   tests, asks to "test" a page/feature/flow, or asks how the test pipeline
   works.
+metadata:
+  version: "0.14.2"
 ---
 
 # Playwright Test Pipeline
@@ -500,7 +502,7 @@ Set up once per project:
 6. **One Slack Workflow Builder rule** routes this project's messages from
    the shared e2e channel to the project channel. All notifications post to
    the e2e channel tagged `[e2e:<repo-name>]`; the rule matches the **full
-   bracketed tag** — never a short name like `gbk`, which collides with
+   bracketed tag** — never a short name like `shop`, which collides with
    other text — and sends the message to the project channel. The sent
    message's template is just the message-text variable (plus the
    link-to-original variable when available, so threads happen on the