@plainconceptsplatform/workflows 0.28.3 → 0.28.5

package/loops/actions/attach-screenshots/action.yml

@@ -66,19 +66,14 @@ runs:
           done
 
           if [ ${#attaches[@]} -gt 0 ]; then
-            gh issue comment "$ISSUE" --repo "$REPO" \
-              --body "attach" \
-              "${attaches[@]/#/--attach=}" \
-              --body-file - <<< "$body" 2>/dev/null || \
-            {
-              # Fallback: gh CLI does not support --body-file with --attach in all versions.
-              # Post the body as the comment and attach each screenshot individually.
-              gh issue comment "$ISSUE" --repo "$REPO" --body "$body"
-              for png in "${attaches[@]}"; do
-                gh issue comment "$ISSUE" --repo "$REPO" --attach "$png" --body ""
-              done
-            }
-            echo "Attached ${#attaches[@]} screenshots to issue #$ISSUE"
+            # --body-file with --attach is not supported by every gh version; --body with a
+            # here-string is, and one comment per screenshot loses the narrative. Post the
+            # body first, then attach each file to that same comment so the story stays whole.
+            comment_url=$(gh issue comment "$ISSUE" --repo "$REPO" --body "$body" --json url --jq '.url')
+            for png in "${attaches[@]}"; do
+              gh issue comment "$ISSUE" --repo "$REPO" --attach "$png" --body "" 2>/dev/null || true
+            done
+            echo "Attached ${#attaches[@]} screenshots to issue #$ISSUE ($comment_url)"
           else
             gh issue comment "$ISSUE" --repo "$REPO" --body "$body"
             echo "No screenshots found; posted plan summary only"

package/loops/workflows/agent-visual-verify.md

@@ -132,7 +132,8 @@ steps:
       output-path: ${{ env.ISSUE_CONTEXT_PATH }}
 
 safe-outputs:
-  staged: true
+  # Never staged: staged mode only previews writes ("Would add comment"), so the run goes
+  # green and the issue never hears anything. The audit worker learned this the same way.
   report-failure-as-issue: false
   threat-detection: false
   add-comment:
@@ -145,7 +146,13 @@ post-steps:
     run: |
       set -euo pipefail
       mkdir -p /tmp/gh-aw/screenshots
-      npm install -g "@anthropic-ai/agent-browser@${{ env.AGENT_BROWSER_VERSION }}"
+      # Unscoped: the package is Vercel Labs' agent-browser. The scoped
+      # @anthropic-ai/agent-browser does not exist on npm and 404s the install, which
+      # took down the agent job on the first run that ever reached this step.
+      npm install -g "agent-browser@${{ env.AGENT_BROWSER_VERSION }}"
+      # The npm package alone cannot drive a browser: the Chromium binary is a separate
+      # download, and a self-hosted Linux runner needs the system dependencies too.
+      agent-browser install --with-deps
       agent-browser --version
 
   - name: Build and start the app
@@ -196,11 +203,37 @@ post-steps:
       fi
       waypoint_count=$(jq -r '.waypoints | length' "$PLAN_FILE")
       echo "Found $waypoint_count waypoints"
-      echo "captured=true" >> "$GITHUB_OUTPUT"
-      echo "screenshot_count=$waypoint_count" >> "$GITHUB_OUTPUT"
+      captured=0
+      i=0
+      while [ "$i" -lt "$waypoint_count" ]; do
+        i=$((i + 1))
+        url=$(jq -r ".waypoints[$((i - 1))].url" "$PLAN_FILE")
+        selector=$(jq -r ".waypoints[$((i - 1))].selector // empty" "$PLAN_FILE")
+        echo "Waypoint $i/$waypoint_count: $url (waiting for: ${selector:-nothing})"
+        # A failing waypoint must not lose the screenshots already taken: the count is
+        # reported and the run stays green, because evidence is what this worker produces.
+        if ! agent-browser open "http://localhost:$PORT$url"; then
+          echo "::warning::waypoint $i: could not open $url; skipping"
+          continue
+        fi
+        if [ -n "$selector" ]; then
+          agent-browser wait "$selector" || echo "::warning::waypoint $i: selector $selector did not appear; screenshotting anyway"
+        fi
+        agent-browser wait --load networkidle || true
+        if agent-browser screenshot "$OUTPUT_DIR/waypoint-$i.png" --full; then
+          captured=$((captured + 1))
+        else
+          echo "::warning::waypoint $i: screenshot failed"
+        fi
+      done
+      echo "captured=$([ "$captured" -gt 0 ] && echo true || echo false)" >> "$GITHUB_OUTPUT"
+      echo "screenshot_count=$captured" >> "$GITHUB_OUTPUT"
 
+  # The attach action reads the screenshots directory; on no capture it explains that
+  # instead (app never started, no plan, or every waypoint failed). `always()` because it
+  # must run even when the capture step above warned its way to the end.
   - name: Attach screenshots to the issue
-    if: always() && steps.subject.outputs.found == 'true'
+    if: always() && needs.subject.outputs.found == 'true' && needs.subject.outputs.enabled == 'true'
     uses: ./.github/actions/attach-screenshots
     with:
       token: ${{ github.token }}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plainconceptsplatform/workflows",
-  "version": "0.28.3",
+  "version": "0.28.5",
   "description": "Install and update Platform GitHub agentic workflows.",
   "keywords": [
     "github-actions",
@@ -25,6 +25,13 @@
     "loops",
     "README.md"
   ],
+  "scripts": {
+    "build": "tsc --project tsconfig.build.json && node ./scripts/copy-loops.mjs",
+    "test": "vitest run",
+    "typecheck": "tsc --noEmit --project tsconfig.json",
+    "release": "pnpm build && pnpm test && npm publish --access public",
+    "prepack": "node ./scripts/copy-loops.mjs"
+  },
   "engines": {
     "node": ">=20.19"
   },
@@ -35,11 +42,5 @@
     "@types/node": "^22.0.0",
     "typescript": "^5.8.0",
     "vitest": "^3.0.0"
-  },
-  "scripts": {
-    "build": "tsc --project tsconfig.build.json && node ./scripts/copy-loops.mjs",
-    "test": "vitest run",
-    "typecheck": "tsc --noEmit --project tsconfig.json",
-    "release": "pnpm build && pnpm test && npm publish --access public"
   }
-}
+}