@williambeto/ai-workflow 2.9.0 → 2.9.1

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 ## [unreleased]
 
+## [2.9.1] - 2026-07-15
+
+### 🐛 Bug Fixes
+
+- Accept persisted Playwright snapshots from the scoped delivery as valid UI
+  evidence, regardless of their test-directory location
+- Recognize screenshots generated under `.evidence/visual/` and evaluate them
+  in the same `collect-evidence` execution that captured them
+- Avoid repeating `NOT_DELIVERED` as a second blocking diagnostic when an
+  earlier validation or quality gate already explains the failure
+
+### 🧪 Testing
+
+- Add regression coverage for Playwright snapshot discovery, generated visual
+  evidence ordering, missing required screenshots, and non-duplicated finalizer
+  diagnostics
+
 ## [2.9.0] - 2026-07-15
 
 ### 🚀 Features

package/bin/ai-workflow.js

@@ -34,7 +34,7 @@ import {
   QualityGuard,
   ValidationPlanner,
   parseDeliveryOutcomeClaim
-} from "../chunk-GLYX3HEY.js";
+} from "../chunk-PP42PB7M.js";
 import {
   DeliveryDecisionEngine
 } from "../chunk-H7GIKXFO.js";
@@ -3697,7 +3697,7 @@ var commandMap = {
     });
   },
   validate: async (_, flags) => {
-    const { runValidate } = await import("../validate-DDKN6ORS.js");
+    const { runValidate } = await import("../validate-7G3HBE2K.js");
     await runValidate({
       cwd: process.cwd(),
       taskSlug: flags.taskSlug,

package/{chunk-GLYX3HEY.js → chunk-PP42PB7M.js}

@@ -25,6 +25,9 @@ function parseStatusLine(line) {
 function isManagedChangePath(file = "") {
   return /(^|\/)(node_modules|vendor|dist|coverage|\.cache)(\/|$)/.test(normalizePath(file));
 }
+function isScreenshotPath(file = "") {
+  return /\.(png|jpe?g|webp)$/i.test(normalizePath(file));
+}
 async function exists(target) {
   try {
     await fs.access(target);
@@ -163,17 +166,22 @@ var QualityGuard = class {
   }
   async verifyUiEvidence() {
     if (!this.isUiTask()) return { status: "NOT_RUN", reason: "Non-UI task." };
-    const candidates = ["docs", "screenshots", "evidence"];
-    const found = [];
+    const candidates = ["docs", "screenshots", "evidence", ".evidence"];
+    const found = /* @__PURE__ */ new Set();
+    for (const file of this.getScopedChangedFiles()) {
+      const normalized = normalizePath(file);
+      if (!isScreenshotPath(normalized) || path.isAbsolute(normalized) || normalized.startsWith("../")) continue;
+      if (await exists(path.join(this.cwd, normalized))) found.add(normalized);
+    }
     for (const dir of candidates) {
       const root = path.join(this.cwd, dir);
       if (!await exists(root)) continue;
       const entries = await fs.readdir(root, { recursive: true }).catch(() => []);
       for (const entry of entries) {
-        if (/\.(png|jpe?g|webp)$/i.test(String(entry))) found.push(normalizePath(path.join(dir, String(entry))));
+        if (isScreenshotPath(String(entry))) found.add(normalizePath(path.join(dir, String(entry))));
       }
     }
-    if (found.length) return { status: "PASS", screenshots: found.slice(0, 20) };
+    if (found.size) return { status: "PASS", screenshots: [...found].sort().slice(0, 20) };
     if (this.evidencePolicy === "required") {
       return { status: "FAIL_QUALITY_GATE", reason: "Required evidence for a high-risk or release UI change does not include a persisted screenshot." };
     }
@@ -673,6 +681,26 @@ var EvidenceCollector = class {
     const implementationFilesPresent = qualityGuard.isImplementationTask();
     const executableBehavior = await qualityGuard.hasExecutableBehaviorChanges(this.deliveryChangedFiles || void 0);
     const results = tasks.map((task) => this.runTask(task));
+    let visualEvidence = null;
+    if (this.visualDist) {
+      console.log(`
+--- Capturing Visual Evidence ---`);
+      console.log(`Serving ${this.visualDist} on port ${this.port}...`);
+      try {
+        const verifier = new VisualVerifier({ cwd: this.cwd, visualDist: this.visualDist, port: this.port });
+        const screenshots = await verifier.captureScreenshots();
+        visualEvidence = {
+          desktop: path3.relative(this.cwd, screenshots[0]),
+          mobile: path3.relative(this.cwd, screenshots[1])
+        };
+        console.log(`Visual evidence captured successfully:`);
+        console.log(`- Desktop: ${visualEvidence.desktop}`);
+        console.log(`- Mobile: ${visualEvidence.mobile}`);
+      } catch (err) {
+        console.error(`Error capturing visual evidence: ${err.message}`);
+        throw err;
+      }
+    }
     const policyValidation = await qualityGuard.verify();
     const behaviorTests = results.filter((result) => result.kind === "test");
     const passingBehaviorTest = behaviorTests.some((result) => result.status === "PASS");
@@ -745,7 +773,8 @@ var EvidenceCollector = class {
       deliveryDecision: deliveryDecision.decision,
       claimedOutcome: this.claimedOutcome
     });
-    if (["NEEDS_CLARIFICATION", "NOT_DELIVERED"].includes(deliveryOutcome) && ["PASS", "PASS_WITH_NOTES"].includes(overallStatus)) {
+    const deliveryOutcomeBlocks = ["NEEDS_CLARIFICATION", "NOT_DELIVERED"].includes(deliveryOutcome) && ["PASS", "PASS_WITH_NOTES"].includes(overallStatus);
+    if (deliveryOutcomeBlocks) {
       overallStatus = "BLOCKED";
     }
     const limitations = [];
@@ -811,7 +840,7 @@ var EvidenceCollector = class {
         "satisfy the stated decision requirement before finalizing"
       ));
     }
-    if (["NEEDS_CLARIFICATION", "NOT_DELIVERED"].includes(deliveryOutcome)) {
+    if (deliveryOutcomeBlocks) {
       limitations.push(blockingDiagnostic(
         "delivery-outcome",
         `the resolved outcome is ${deliveryOutcome}`,
@@ -820,26 +849,6 @@ var EvidenceCollector = class {
       ));
     }
     const uniqueLimitations = [...new Set(limitations)];
-    let visualEvidence = null;
-    if (this.visualDist) {
-      console.log(`
---- Capturing Visual Evidence ---`);
-      console.log(`Serving ${this.visualDist} on port ${this.port}...`);
-      try {
-        const verifier = new VisualVerifier({ cwd: this.cwd, visualDist: this.visualDist, port: this.port });
-        const screenshots = await verifier.captureScreenshots();
-        visualEvidence = {
-          desktop: path3.relative(this.cwd, screenshots[0]),
-          mobile: path3.relative(this.cwd, screenshots[1])
-        };
-        console.log(`Visual evidence captured successfully:`);
-        console.log(`- Desktop: ${visualEvidence.desktop}`);
-        console.log(`- Mobile: ${visualEvidence.mobile}`);
-      } catch (err) {
-        console.error(`Error capturing visual evidence: ${err.message}`);
-        throw err;
-      }
-    }
     const evidence = {
       timestamp: (/* @__PURE__ */ new Date()).toISOString(),
       taskSlug: this.taskSlug,
@@ -1053,4 +1062,4 @@ export {
   EvidenceCollector,
   ValidationPlanner
 };
-//# sourceMappingURL=chunk-GLYX3HEY.js.map
+//# sourceMappingURL=chunk-PP42PB7M.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@williambeto/ai-workflow",
-  "version": "2.9.0",
+  "version": "2.9.1",
   "description": "AI Workflow Kit — OpenCode-first software delivery workflow with agents, commands, skills, validation, and evidence",
   "license": "MIT",
   "author": "José Willams",

package/{validate-DDKN6ORS.js → validate-7G3HBE2K.js}

@@ -3,7 +3,7 @@ import {
   QualityGuard,
   ValidationPlanner,
   VisualVerifier
-} from "./chunk-GLYX3HEY.js";
+} from "./chunk-PP42PB7M.js";
 import "./chunk-H7GIKXFO.js";
 import "./chunk-5WRI5ZAA.js";
 
@@ -86,4 +86,4 @@ async function runValidate({
 export {
   runValidate
 };
-//# sourceMappingURL=validate-DDKN6ORS.js.map
+//# sourceMappingURL=validate-7G3HBE2K.js.map