chore(support): align agent-browser config, schemas, and validation helpers

- Add agent-browser.json project config (allowedDomains defaults)
- Drop legacy support/browser wrapper (obsolete .env.browser sourcing)
- browser-validate.ts: replace busy-wait sleep with async setTimeout,
  wrap precomputeData return in Promise, await its call
- Unify subagent-browser-schema.json into browser-validation-schema.json
  (add testName, error, session_id; relax metrics/checks requirements)
- Update CONFIG-REPO-VALIDATION.md and subagent-spawning-pattern.md to
  reference the unified schema

Author: localpibox

agent-browser.json

@@ -0,0 +1,10 @@
+{
+  "allowedDomains": [
+    "localhost",
+    "127.0.0.1",
+    "*.example.com",
+    "*.github.com",
+    "*.wikipedia.org",
+    "developer.chrome.com"
+  ]
+}

doc/CONFIG-REPO-VALIDATION.md

@@ -70,8 +70,7 @@ config/
     ├── docs/
     │   └── subagent-spawning-pattern.md
     └── schemas/
-        ├── browser-validation-schema.json
-        └── subagent-browser-schema.json
+        └── browser-validation-schema.json
 ```
 
 ---
@@ -134,8 +133,7 @@ All reference `/opt/pi-support` for support files (correct — install.sh copies
 | `config/agent-browser-action-policy.json` | Destructive action gates | ✅ |
 | `config/subagent-browser-prompt.txt` | Subagent browser prompt template | ✅ |
 | `docs/subagent-spawning-pattern.md` | Orchestration docs | ✅ |
-| `schemas/browser-validation-schema.json` | Validation report schema | ✅ |
-| `schemas/subagent-browser-schema.json` | Subagent result schema | ✅ |
+| `schemas/browser-validation-schema.json` | Unified validation report schema | ✅ |
 
 ---
 

support/browser

@@ -65,10 +65,10 @@ function agentBrowser(...args: string[]): string {
 
 // ─── Task 2.4: Pre-computation ──────────────────────────────────────────────
 
-function precomputeData(
+async function precomputeData(
   url: string,
   sessionId: string,
-): { vitals: string; a11y: string; snapshot: string; screenshotPath: string } {
+): Promise<{ vitals: string; a11y: string; snapshot: string; screenshotPath: string }> {
   const screenshotPath = `/browser-states/${sessionId}/screenshot.png`;
 
   // Ensure session directory exists
@@ -77,7 +77,7 @@ function precomputeData(
   // Navigate
   agentBrowser("--session", sessionId, "open", url);
   // Small delay for page to load
-  sleep(1000);
+  await sleep(1000);
 
   // Collect vitals JSON
   const vitals = agentBrowser("--session", sessionId, "vitals", "--json");
@@ -309,11 +309,8 @@ function saveReport(sessionId: string, report: ValidationReport): string {
 
 // ─── Helpers ─────────────────────────────────────────────────────────────────
 
-function sleep(ms: number): void {
-  const start = Date.now();
-  while (Date.now() - start < ms) {
-    // busy wait (sync for simplicity)
-  }
+function sleep(ms: number): Promise<void> {
+  return new Promise(resolve => setTimeout(resolve, ms));
 }
 
 // ─── Main ────────────────────────────────────────────────────────────────────
@@ -341,7 +338,7 @@ async function main(): Promise<void> {
   try {
     // Step 1: Pre-compute structured data
     console.log("→ Pre-computing structured data...");
-    const { vitals, a11y, snapshot, screenshotPath } = precomputeData(
+    const { vitals, a11y, snapshot, screenshotPath } = await precomputeData(
       url,
       sessionId,
     );

support/browser-validate.ts

@@ -39,7 +39,7 @@ subagent({
     
     Session ID: ${SESSION_ID}
     Prompt: ${PROMPT}`,
-  outputSchema: JSON.parse(readFileSync("subagent-browser-schema.json")),
+  outputSchema: JSON.parse(readFileSync("browser-validation-schema.json")),
 });
 ```
 
@@ -88,6 +88,6 @@ if (!result.success) {
 | File | Purpose |
 |------|---------|
 | `bin/session-uuid.ts` | Generate unique session IDs |
-| `subagent-browser-schema.json` | JSON schema for subagent results |
+| `browser-validation-schema.json` | Unified JSON schema (all fields) |
 | `subagent-browser-prompt.txt` | System prompt template |
 | `bin/validate-subagent-output.ts` | Parent-side validation utility |

support/docs/subagent-spawning-pattern.md

@@ -1,29 +1,27 @@
 {
   "$schema": "http://json-schema.org/draft-07/schema#",
   "title": "BrowserValidationReport",
-  "description": "Structured validation report produced by the vision model for a web application",
+  "description": "Structured validation report from browser tests (vision model or subagent). All fields except url/status/metrics/checks are optional.",
   "type": "object",
   "required": ["url", "status", "metrics", "checks"],
   "properties": {
     "url": {
       "type": "string",
       "format": "uri",
-      "description": "The URL that was validated"
+      "description": "The URL that was tested or validated"
     },
     "status": {
       "type": "string",
       "enum": ["PASS", "WARN", "FAIL"],
-      "description": "Overall validation status"
+      "description": "Overall test/validation result"
+    },
+    "testName": {
+      "type": "string",
+      "description": "Name of the test that was run (subagent/browser-test only)"
     },
     "metrics": {
       "type": "object",
-      "required": [
-        "lcp_ms",
-        "cls",
-        "ttfb_ms",
-        "a11y_violations",
-        "a11y_passes"
-      ],
+      "required": ["lcp_ms", "cls", "ttfb_ms", "a11y_violations", "a11y_passes"],
       "properties": {
         "lcp_ms": {
           "type": "number",
@@ -55,7 +53,7 @@
       "type": "array",
       "items": {
         "type": "object",
-        "required": ["check", "description", "pass"],
+        "required": ["check", "pass"],
         "properties": {
           "check": {
             "type": "string",
@@ -76,6 +74,14 @@
         }
       }
     },
+    "error": {
+      "type": "string",
+      "description": "Error message if the test/validation failed"
+    },
+    "session_id": {
+      "type": "string",
+      "description": "The browser session ID used for this test"
+    },
     "screenshot_path": {
       "type": "string",
       "description": "Path to the annotated screenshot used for validation"