opendevbrowser 0.0.12 → 0.0.16

package/skills/opendevbrowser-best-practices/assets/templates/cdp-forward-envelope.json

@@ -0,0 +1,11 @@
+{
+  "id": 1,
+  "method": "forwardCDPCommand",
+  "params": {
+    "method": "Runtime.evaluate",
+    "params": {
+      "expression": "document.title"
+    },
+    "sessionId": "optional-flat-session-id"
+  }
+}

package/skills/opendevbrowser-best-practices/assets/templates/mode-flag-matrix.json

@@ -0,0 +1,56 @@
+{
+  "schemaVersion": "1.0",
+  "modes": [
+    {
+      "name": "managed",
+      "launch": "npx opendevbrowser launch --no-extension",
+      "statusProbe": "npx opendevbrowser status --session-id <session-id>",
+      "requiredFlags": [
+        "--no-extension"
+      ],
+      "statusExpectations": {
+        "mode": "managed"
+      }
+    },
+    {
+      "name": "extension",
+      "launch": "npx opendevbrowser launch --extension-only --wait-for-extension",
+      "statusProbe": "npx opendevbrowser status --daemon",
+      "requiredFlags": [
+        "--extension-only",
+        "--wait-for-extension"
+      ],
+      "statusExpectations": {
+        "extensionConnected": true,
+        "extensionHandshakeComplete": true,
+        "opsConnected": true
+      }
+    },
+    {
+      "name": "extension-legacy",
+      "launch": "npx opendevbrowser launch --extension-only --extension-legacy --wait-for-extension",
+      "statusProbe": "npx opendevbrowser status --daemon",
+      "requiredFlags": [
+        "--extension-only",
+        "--extension-legacy",
+        "--wait-for-extension"
+      ],
+      "statusExpectations": {
+        "extensionConnected": true,
+        "extensionHandshakeComplete": true,
+        "cdpConnected": true
+      }
+    },
+    {
+      "name": "cdpConnect",
+      "connect": "npx opendevbrowser connect --ws-endpoint ws://127.0.0.1:9222/devtools/browser/<id>",
+      "statusProbe": "npx opendevbrowser status --session-id <session-id>",
+      "requiredFlags": [
+        "--ws-endpoint"
+      ],
+      "statusExpectations": {
+        "mode": "cdpConnect"
+      }
+    }
+  ]
+}

package/skills/opendevbrowser-best-practices/assets/templates/ops-request-envelope.json

@@ -0,0 +1,9 @@
+{
+  "type": "ops_request",
+  "requestId": "req-session-status-001",
+  "clientId": "ops-client-1",
+  "opsSessionId": "ops-session-123",
+  "leaseId": "lease-123",
+  "command": "session.status",
+  "payload": {}
+}

package/skills/opendevbrowser-best-practices/assets/templates/robustness-checklist.json

@@ -0,0 +1,79 @@
+{
+  "schemaVersion": "1.0",
+  "skill": "",
+  "run_id": "",
+  "issues": [
+    {
+      "id": "ISSUE-01",
+      "status": "pending",
+      "evidence": [],
+      "notes": ""
+    },
+    {
+      "id": "ISSUE-02",
+      "status": "pending",
+      "evidence": [],
+      "notes": ""
+    },
+    {
+      "id": "ISSUE-03",
+      "status": "pending",
+      "evidence": [],
+      "notes": ""
+    },
+    {
+      "id": "ISSUE-04",
+      "status": "pending",
+      "evidence": [],
+      "notes": ""
+    },
+    {
+      "id": "ISSUE-05",
+      "status": "pending",
+      "evidence": [],
+      "notes": ""
+    },
+    {
+      "id": "ISSUE-06",
+      "status": "pending",
+      "evidence": [],
+      "notes": ""
+    },
+    {
+      "id": "ISSUE-07",
+      "status": "pending",
+      "evidence": [],
+      "notes": ""
+    },
+    {
+      "id": "ISSUE-08",
+      "status": "pending",
+      "evidence": [],
+      "notes": ""
+    },
+    {
+      "id": "ISSUE-09",
+      "status": "pending",
+      "evidence": [],
+      "notes": ""
+    },
+    {
+      "id": "ISSUE-10",
+      "status": "pending",
+      "evidence": [],
+      "notes": ""
+    },
+    {
+      "id": "ISSUE-11",
+      "status": "pending",
+      "evidence": [],
+      "notes": ""
+    },
+    {
+      "id": "ISSUE-12",
+      "status": "pending",
+      "evidence": [],
+      "notes": ""
+    }
+  ]
+}

package/skills/opendevbrowser-best-practices/assets/templates/surface-audit-checklist.json

@@ -0,0 +1,24 @@
+{
+  "schemaVersion": "1.0",
+  "expectedCounts": {
+    "cliCommands": 50,
+    "tools": 44,
+    "opsCommands": 36
+  },
+  "docsMustCover": [
+    "README.md",
+    "docs/CLI.md",
+    "docs/ARCHITECTURE.md",
+    "docs/SURFACE_REFERENCE.md",
+    "AGENTS.md",
+    "skills/opendevbrowser-best-practices/SKILL.md"
+  ],
+  "checks": [
+    "CLI command names documented by category",
+    "Tool names documented by category",
+    "Ops command namespaces documented",
+    "CDP envelope contract documented",
+    "Global and command-specific flags documented",
+    "Mode semantics documented"
+  ]
+}

package/skills/opendevbrowser-best-practices/scripts/odb-workflow.sh

@@ -0,0 +1,144 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+print_help() {
+  cat <<'EOF'
+OpenDevBrowser workflow router
+
+Usage:
+  ./skills/opendevbrowser-best-practices/scripts/odb-workflow.sh <workflow>
+
+Workflows:
+  provider-search
+  provider-crawl
+  parallel-multipage-safe
+  qa-debug
+  social-readonly-check
+  parity-check
+  surface-audit
+  ops-channel-check
+  cdp-channel-check
+  mode-flag-matrix
+  robustness-audit
+  list
+EOF
+}
+
+workflow="${1:-list}"
+
+case "$workflow" in
+  list)
+    print_help
+    ;;
+  provider-search)
+    cat <<'EOF'
+opendevbrowser_launch noExtension=true
+opendevbrowser_goto sessionId="<session-id>" url="<provider-search-url>"
+opendevbrowser_wait sessionId="<session-id>" until="networkidle"
+opendevbrowser_snapshot sessionId="<session-id>" format="actionables"
+opendevbrowser_network_poll sessionId="<session-id>" max=50
+EOF
+    ;;
+  provider-crawl)
+    cat <<'EOF'
+opendevbrowser_launch noExtension=true
+opendevbrowser_goto sessionId="<session-id>" url="<seed-url>"
+opendevbrowser_wait sessionId="<session-id>" until="networkidle"
+opendevbrowser_snapshot sessionId="<session-id>" format="actionables"
+opendevbrowser_scroll sessionId="<session-id>" dy=1000
+opendevbrowser_wait sessionId="<session-id>" until="networkidle"
+EOF
+    ;;
+  parallel-multipage-safe)
+    cat <<'EOF'
+# Reliable parallel pattern (as-is):
+# - Worker A owns session-a
+# - Worker B owns session-b
+# - Keep each session serial (snapshot -> action -> snapshot)
+# - Do NOT interleave independent target-use streams in one session
+# - Managed + persisted profiles: use unique profile paths per session
+
+opendevbrowser_launch noExtension=true
+opendevbrowser_launch noExtension=true
+
+# Worker A loop
+opendevbrowser_goto sessionId="<session-a>" url="<url-a1>"
+opendevbrowser_wait sessionId="<session-a>" until="networkidle"
+opendevbrowser_snapshot sessionId="<session-a>" format="actionables"
+
+# Worker B loop
+opendevbrowser_goto sessionId="<session-b>" url="<url-b1>"
+opendevbrowser_wait sessionId="<session-b>" until="networkidle"
+opendevbrowser_snapshot sessionId="<session-b>" format="actionables"
+EOF
+    ;;
+  qa-debug)
+    cat <<'EOF'
+opendevbrowser_snapshot sessionId="<session-id>" format="outline"
+opendevbrowser_console_poll sessionId="<session-id>" max=100
+opendevbrowser_network_poll sessionId="<session-id>" max=100
+opendevbrowser_screenshot sessionId="<session-id>"
+EOF
+    ;;
+  social-readonly-check)
+    cat <<'EOF'
+# Read-only social validation (default, no posting).
+opendevbrowser_snapshot sessionId="<session-id>" format="actionables"
+opendevbrowser_debug_trace_snapshot sessionId="<session-id>" maxEvents=50
+opendevbrowser_network_poll sessionId="<session-id>" max=100
+EOF
+    ;;
+  parity-check)
+    cat <<'EOF'
+npm run test -- tests/parity-matrix.test.ts
+npm run test -- tests/tools.test.ts tests/daemon-commands.integration.test.ts
+./skills/opendevbrowser-best-practices/scripts/validate-skill-assets.sh
+EOF
+    ;;
+  surface-audit)
+    cat <<'EOF'
+cat docs/SURFACE_REFERENCE.md
+cat skills/opendevbrowser-best-practices/artifacts/command-channel-reference.md
+cat skills/opendevbrowser-best-practices/assets/templates/surface-audit-checklist.json
+./skills/opendevbrowser-best-practices/scripts/validate-skill-assets.sh
+EOF
+    ;;
+  ops-channel-check)
+    cat <<'EOF'
+npx opendevbrowser serve
+npx opendevbrowser launch --extension-only --wait-for-extension --output-format json
+npx opendevbrowser status --daemon --output-format json
+# Verify opsConnected=true and extensionHandshakeComplete=true
+cat skills/opendevbrowser-best-practices/assets/templates/ops-request-envelope.json
+EOF
+    ;;
+  cdp-channel-check)
+    cat <<'EOF'
+npx opendevbrowser serve
+npx opendevbrowser launch --extension-only --extension-legacy --wait-for-extension --output-format json
+npx opendevbrowser status --daemon --output-format json
+# Verify cdpConnected=true while legacy session is active
+cat skills/opendevbrowser-best-practices/assets/templates/cdp-forward-envelope.json
+EOF
+    ;;
+  mode-flag-matrix)
+    cat <<'EOF'
+cat skills/opendevbrowser-best-practices/assets/templates/mode-flag-matrix.json
+npx opendevbrowser launch --no-extension --output-format json
+npx opendevbrowser launch --extension-only --wait-for-extension --output-format json
+npx opendevbrowser launch --extension-only --extension-legacy --wait-for-extension --output-format json
+EOF
+    ;;
+  robustness-audit)
+    cat <<'EOF'
+cat skills/opendevbrowser-best-practices/artifacts/browser-agent-known-issues-matrix.md
+cat skills/opendevbrowser-best-practices/assets/templates/robustness-checklist.json
+./skills/opendevbrowser-best-practices/scripts/run-robustness-audit.sh
+EOF
+    ;;
+  *)
+    echo "Unknown workflow: $workflow" >&2
+    print_help
+    exit 2
+    ;;
+esac

package/skills/opendevbrowser-best-practices/scripts/run-robustness-audit.sh

@@ -0,0 +1,83 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+root="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
+skills_root="$(cd "$root/.." && pwd)"
+matrix_ref="../opendevbrowser-best-practices/artifacts/browser-agent-known-issues-matrix.md"
+
+usage() {
+  cat <<'HELP'
+Run robustness audit checks across opendevbrowser workflow skill packs.
+
+Usage:
+  ./skills/opendevbrowser-best-practices/scripts/run-robustness-audit.sh [skill-name]
+
+Examples:
+  ./skills/opendevbrowser-best-practices/scripts/run-robustness-audit.sh
+  ./skills/opendevbrowser-best-practices/scripts/run-robustness-audit.sh opendevbrowser-shopping
+HELP
+}
+
+skills=(
+  "opendevbrowser-login-automation:ISSUE-01 ISSUE-02 ISSUE-03 ISSUE-04 ISSUE-05 ISSUE-06 ISSUE-07"
+  "opendevbrowser-form-testing:ISSUE-01 ISSUE-02 ISSUE-05 ISSUE-06 ISSUE-08"
+  "opendevbrowser-data-extraction:ISSUE-01 ISSUE-06 ISSUE-08 ISSUE-09 ISSUE-10"
+  "opendevbrowser-shopping:ISSUE-06 ISSUE-09 ISSUE-10 ISSUE-11 ISSUE-12"
+  "opendevbrowser-product-presentation-asset:ISSUE-10 ISSUE-11 ISSUE-12"
+  "opendevbrowser-research:ISSUE-06 ISSUE-09 ISSUE-10 ISSUE-12"
+)
+
+status=0
+target="${1:-all}"
+
+if [[ "$target" == "-h" || "$target" == "--help" ]]; then
+  usage
+  exit 0
+fi
+
+found_target=0
+for entry in "${skills[@]}"; do
+  skill="${entry%%:*}"
+  expected="${entry#*:}"
+  if [[ "$target" != "all" && "$skill" != "$target" ]]; then
+    continue
+  fi
+  found_target=1
+
+  skill_file="$skills_root/$skill/SKILL.md"
+  if [[ ! -f "$skill_file" ]]; then
+    echo "Missing SKILL.md: $skill_file" >&2
+    status=1
+    continue
+  fi
+
+  if ! grep -Fq "Robustness Coverage (Known-Issue Matrix)" "$skill_file"; then
+    echo "[$skill] missing robustness coverage section" >&2
+    status=1
+  fi
+
+  if ! grep -Fq "$matrix_ref" "$skill_file"; then
+    echo "[$skill] missing matrix reference: $matrix_ref" >&2
+    status=1
+  fi
+
+  for issue_id in $expected; do
+    if ! grep -Fq "$issue_id" "$skill_file"; then
+      echo "[$skill] missing issue id: $issue_id" >&2
+      status=1
+    fi
+  done
+
+done
+
+if [[ "$target" != "all" && $found_target -eq 0 ]]; then
+  echo "Unknown skill: $target" >&2
+  usage
+  exit 2
+fi
+
+if [[ $status -ne 0 ]]; then
+  exit $status
+fi
+
+echo "Robustness audit checks passed for ${target}."

package/skills/opendevbrowser-best-practices/scripts/validate-skill-assets.sh

@@ -0,0 +1,93 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+skill_root="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
+skill_file="$skill_root/SKILL.md"
+
+required_paths=(
+  "artifacts/provider-workflows.md"
+  "artifacts/parity-gates.md"
+  "artifacts/debug-trace-playbook.md"
+  "artifacts/fingerprint-tiers.md"
+  "artifacts/macro-workflows.md"
+  "artifacts/browser-agent-known-issues-matrix.md"
+  "artifacts/command-channel-reference.md"
+  "assets/templates/mode-flag-matrix.json"
+  "assets/templates/ops-request-envelope.json"
+  "assets/templates/cdp-forward-envelope.json"
+  "assets/templates/robustness-checklist.json"
+  "assets/templates/surface-audit-checklist.json"
+  "scripts/odb-workflow.sh"
+  "scripts/run-robustness-audit.sh"
+  "scripts/validate-skill-assets.sh"
+)
+
+json_templates=(
+  "assets/templates/mode-flag-matrix.json"
+  "assets/templates/ops-request-envelope.json"
+  "assets/templates/cdp-forward-envelope.json"
+  "assets/templates/robustness-checklist.json"
+  "assets/templates/surface-audit-checklist.json"
+)
+
+command_ref_markers=(
+  'CLI commands: `55`'
+  'Plugin tools: `48`'
+  '`/ops` command names: `38`'
+  'docs/SURFACE_REFERENCE.md'
+  'opencode'
+  'codex'
+  'claudecode'
+  'ampcli'
+)
+
+status=0
+
+for rel_path in "${required_paths[@]}"; do
+  full_path="$skill_root/$rel_path"
+  if [[ ! -f "$full_path" ]]; then
+    echo "Missing required asset: $rel_path" >&2
+    status=1
+  fi
+done
+
+for rel_path in "scripts/odb-workflow.sh" "scripts/run-robustness-audit.sh" "scripts/validate-skill-assets.sh"; do
+  full_path="$skill_root/$rel_path"
+  if [[ -f "$full_path" && ! -x "$full_path" ]]; then
+    echo "Script is not executable: $rel_path" >&2
+    status=1
+  fi
+done
+
+for rel_path in "${json_templates[@]}"; do
+  full_path="$skill_root/$rel_path"
+  if [[ -f "$full_path" ]]; then
+    if ! node -e 'const fs=require("fs"); JSON.parse(fs.readFileSync(process.argv[1], "utf8"));' "$full_path" >/dev/null 2>&1; then
+      echo "Invalid JSON template: $rel_path" >&2
+      status=1
+    fi
+  fi
+done
+
+for rel_path in "${required_paths[@]}"; do
+  if ! grep -Fq "$rel_path" "$skill_file"; then
+    echo "SKILL.md missing reference: $rel_path" >&2
+    status=1
+  fi
+done
+
+command_ref_path="$skill_root/artifacts/command-channel-reference.md"
+if [[ -f "$command_ref_path" ]]; then
+  for marker in "${command_ref_markers[@]}"; do
+    if ! grep -Fq "$marker" "$command_ref_path"; then
+      echo "Command/channel reference missing marker: $marker" >&2
+      status=1
+    fi
+  done
+fi
+
+if [[ $status -ne 0 ]]; then
+  exit $status
+fi
+
+echo "Skill assets validated: ${#required_paths[@]} files referenced/present, ${#json_templates[@]} JSON templates parsed."

package/skills/opendevbrowser-continuity-ledger/SKILL.md

@@ -1,45 +1,89 @@
 ---
 name: opendevbrowser-continuity-ledger
-description: Maintain an OpenDevBrowser continuity ledger in opendevbrowser_continuity.md for long-running tasks and resumable work.
-version: 1.0.0
+description: This skill should be used when the user asks to "track continuity", "resume a long task", "maintain CONTINUITY.md", or run multi-step work that may span context compaction.
+version: 1.1.0
 ---
 
 # OpenDevBrowser Continuity Ledger
 
-Use a lightweight ledger to keep long-running tasks on track across sessions or context compaction.
+Use this guide to maintain compaction-safe project state in `CONTINUITY.md`.
 
-## When to use
-- Multi-step work that spans several actions
-- Refactors, migrations, or release work
-- Investigations with multiple findings
-- Any task likely to resume later
+## When to Use This Skill
 
-## Ledger file
-- Always use `opendevbrowser_continuity.md` at the repo root.
-- Create it if it does not exist.
-- Keep it short and factual.
+Use this skill when any of these are true:
 
-## Exact template (copy as-is)
+- Work is multi-step or long-running.
+- Context compaction or session handoff is likely.
+- Multiple agents are coordinating the same task.
 
-```markdown
-# OpenDevBrowser Continuity Ledger
+Start with `opendevbrowser-best-practices` quick start for execution. Switch here once continuity tracking is required.
+
+## Ownership Rules
+
+Apply these rules exactly:
+
+- Allow only the main orchestrator agent to edit `CONTINUITY.md`.
+- Instruct sub-agents to never edit `CONTINUITY.md`.
+- Require sub-agents to append their outcomes to `sub_continuity.md`.
+- If `CONTINUITY.md` is modified incorrectly by another agent, restore it immediately and continue.
+
+## Start-of-Turn Protocol
+
+Run this sequence at the beginning of each turn:
+
+1. Read `CONTINUITY.md`.
+2. Read `sub_continuity.md`.
+3. Update `CONTINUITY.md` to reflect the current goal, constraints, decisions, and execution state.
+4. Proceed with implementation.
+
+If recall is incomplete, rebuild from visible context, mark gaps `UNCONFIRMED`, then continue.
+
+## Required Ledger Template
+
+Maintain these headings and sections:
 
+```markdown
 Goal (incl. success criteria):
 - Constraints/Assumptions:
 - Key decisions:
 - State:
   - Done:
   - Now:
-  - Next:
+  - Next: at least 4 next tasks/subtasks each with a brief description. must be detailed with a clear action item and expected outcome and files to be impacted
 - Open questions (UNCONFIRMED if needed):
+  - When you have open questions, do your research in the codebase (and on the internet for best practices) to understand the existing patterns and constraints. Choose answers that are consistent with the existing patterns and constraints and best-practice and research all synchronized into logical recommendations. You must research codebase + external sources first, state the recommended option with brief rationale, and explicitly list any items that still require user input.
 - Working set (files/ids/commands):
+- Key learnings: what worked; what didn't work, best approach identified for next time
 ```
 
-## Update rules
-1. At the start of a long task, read the ledger and refresh Goal/Now/Next.
-2. Update the ledger when goals, decisions, or progress state change.
-3. Record important tool outcomes briefly.
-4. If context is lost, rebuild the ledger from visible state and mark gaps as `UNCONFIRMED`.
+## Update Triggers
+
+Update `CONTINUITY.md` whenever one of these changes:
+
+- Goal or success criteria
+- Constraints or assumptions
+- Key decisions
+- Progress state (`Done`, `Now`, `Next`)
+- Important command/tool outcomes
+
+Keep entries factual and concise. Avoid transcript-style logging.
+
+## Handling Open Questions
+
+When uncertainty exists:
+
+1. Research codebase patterns first.
+2. Research external best practices where relevant.
+3. Recommend a preferred option with rationale.
+4. List only unresolved user-input decisions.
+5. Mark unknown facts as `UNCONFIRMED`.
+
+## Reply Pattern
+
+Start response messages with a short ledger snapshot:
+
+- Goal
+- Now/Next
+- Open questions + recommended option
 
-## Reply pattern
-Start replies with a short "Ledger Snapshot" (Goal + Now/Next + Open questions) when the ledger is in use.
+Print the full ledger only when it materially changes or when requested.