oh-my-opencode 4.9.2 → 4.11.0

package/dist/skills/ast-grep/tests/smoke.ps1

@@ -0,0 +1,123 @@
+#Requires -Version 5.1
+# Smoke test for the ast-grep skill on Windows (PowerShell 5.1+).
+$ErrorActionPreference = 'Stop'
+
+$ScriptDir = Split-Path -Parent $MyInvocation.MyCommand.Path
+$SkillDir  = Split-Path -Parent $ScriptDir
+$Helper    = Join-Path $SkillDir 'scripts/ast_grep_helper.py'
+$Python    = if (Get-Command py -ErrorAction SilentlyContinue) { 'py' } else { 'python' }
+
+$Output = Join-Path $env:TEMP ("ast-grep-skill-smoke-" + [guid]::NewGuid().ToString('N').Substring(0,8))
+New-Item -ItemType Directory -Path $Output -Force | Out-Null
+
+function Pass([string]$msg) { Write-Host "PASS: $msg" }
+function Fail([string]$msg) { Write-Host "FAIL: $msg" -ForegroundColor Red; Remove-Item -Recurse -Force $Output -ErrorAction SilentlyContinue; exit 1 }
+
+function Run([string[]]$Args) {
+    $stdoutFile = Join-Path $Output ("out-" + [guid]::NewGuid().ToString('N').Substring(0,8) + ".txt")
+    $proc = Start-Process -FilePath $Python -ArgumentList (@($Helper) + $Args) -NoNewWindow -PassThru -Wait -RedirectStandardOutput $stdoutFile -RedirectStandardError "$stdoutFile.err"
+    $stdout = if (Test-Path $stdoutFile) { Get-Content $stdoutFile -Raw } else { '' }
+    $stderr = if (Test-Path "$stdoutFile.err") { Get-Content "$stdoutFile.err" -Raw } else { '' }
+    return [pscustomobject]@{
+        ExitCode = $proc.ExitCode
+        Stdout = $stdout
+        Stderr = $stderr
+        Combined = "$stdout`n$stderr"
+    }
+}
+
+try {
+    # 1. --version
+    $r = Run @('--version')
+    if ($r.Combined -notmatch 'ast-grep-helper') { Fail '--version output missing' }
+    Pass '--version'
+
+    # 2. langs (must list >=25)
+    $r = Run @('langs')
+    $langCount = ($r.Stdout -split "`n" | Where-Object { $_ -match '^  [a-z]' }).Count
+    if ($langCount -lt 25) { Fail "langs listed only $langCount (expected >=25)" }
+    Pass 'langs lists at least 25 languages'
+
+    # 3. regex misuse: \w+
+    $r = Run @('validate', '\w+', '--lang', 'ts')
+    if ($r.ExitCode -ne 2) { Fail "validate '\w+' should exit 2, got $($r.ExitCode)" }
+    if ($r.Combined -notmatch 'regex') { Fail "validate '\w+' should mention regex" }
+    Pass 'validate detects \w regex misuse'
+
+    # 4. valid pattern
+    $r = Run @('validate', 'console.log($MSG)', '--lang', 'ts')
+    if ($r.ExitCode -ne 0) { Fail "validate 'console.log(`$MSG)' should exit 0, got $($r.ExitCode)" }
+    Pass 'validate accepts plausible pattern'
+
+    # 5. Python trailing colon
+    $r = Run @('validate', 'def $F($$$):', '--lang', 'py')
+    if ($r.ExitCode -ne 2) { Fail "validate 'def `$F(`$`$`$):' should exit 2, got $($r.ExitCode)" }
+    if ($r.Combined -notmatch 'colon|trailing') { Fail 'validate should mention trailing colon' }
+    Pass 'validate detects Python trailing colon'
+
+    # 6. Incomplete TS function
+    $r = Run @('validate', 'function $N', '--lang', 'ts')
+    if ($r.ExitCode -ne 2) { Fail "validate 'function `$N' should exit 2, got $($r.ExitCode)" }
+    if ($r.Combined -notmatch 'incomplete|params|body') { Fail 'validate should hint about params/body' }
+    Pass 'validate detects incomplete TS function'
+
+    # 7. Alternation pipe
+    $r = Run @('validate', 'foo|bar', '--lang', 'ts')
+    if ($r.ExitCode -ne 2) { Fail "validate 'foo|bar' should exit 2, got $($r.ExitCode)" }
+    if ($r.Combined -notmatch 'alternation|regex') { Fail 'validate should mention alternation' }
+    Pass 'validate detects literal | alternation'
+
+    # 8. doctor
+    $r = Run @('doctor')
+    if ($r.Combined -notmatch 'ast-grep-helper') { Fail 'doctor missing helper version line' }
+    Pass 'doctor produces output'
+
+    # 9. search w/o binary
+    $r = Run @('-q', 'search', 'foo()', '--lang', 'ts', 'C:/nonexistent-path-xyzzy')
+    switch ($r.ExitCode) {
+        { $_ -in 0,1,4 } { Pass "search runs (rc=$($r.ExitCode), ast-grep available)" }
+        3 {
+            if ($r.Combined -notmatch 'install') { Fail 'search rc=3 should print install hint' }
+            Pass 'search without binary prints install hint'
+        }
+        default { Fail "search returned unexpected rc=$($r.ExitCode): $($r.Combined)" }
+    }
+
+    # 10. install.ps1 syntax (parse-check via PowerShell tokenizer)
+    $tokens = $null
+    $errors = $null
+    [System.Management.Automation.Language.Parser]::ParseFile(
+        (Join-Path $SkillDir 'install.ps1'), [ref]$tokens, [ref]$errors) | Out-Null
+    if ($errors -and $errors.Count -gt 0) { Fail "install.ps1 has parse errors: $($errors -join '; ')" }
+    Pass 'install.ps1 parses cleanly'
+
+    # 11. SKILL.md frontmatter
+    $skill = Get-Content (Join-Path $SkillDir 'SKILL.md') -Raw
+    if (-not $skill.StartsWith("---`n") -and -not $skill.StartsWith("---`r`n")) {
+        Fail 'SKILL.md must start with YAML frontmatter'
+    }
+    $endIdx = $skill.IndexOf("`n---`n", 4)
+    if ($endIdx -lt 0) { $endIdx = $skill.IndexOf("`r`n---`r`n", 4) }
+    if ($endIdx -lt 0) { Fail 'SKILL.md missing closing ---' }
+    $fm = $skill.Substring(4, $endIdx - 4)
+    if ($fm -notmatch '(?m)^name:\s*ast-grep\s*$') { Fail 'frontmatter missing name: ast-grep' }
+    if ($fm -notmatch '(?m)^description:') { Fail 'frontmatter missing description' }
+    Pass 'SKILL.md frontmatter shape'
+
+    # 12. All required reference files exist
+    $required = @(
+        'references/install.md', 'references/patterns.md', 'references/pitfalls.md',
+        'references/recipes.md', 'references/cli.md', 'references/yaml-rules.md',
+        'references/sgconfig.md'
+    )
+    foreach ($f in $required) {
+        if (-not (Test-Path (Join-Path $SkillDir $f))) { Fail "missing $f" }
+    }
+    Pass 'all references present'
+
+    Write-Host ''
+    Write-Host 'all smoke tests passed'
+}
+finally {
+    Remove-Item -Recurse -Force $Output -ErrorAction SilentlyContinue
+}

package/dist/skills/ast-grep/tests/smoke.sh

@@ -0,0 +1,212 @@
+#!/usr/bin/env bash
+# Smoke test for the ast-grep skill on POSIX (macOS / Linux / WSL / Git Bash).
+#
+# Tests:
+#   1. helper --version
+#   2. helper langs                              (lists 25 languages)
+#   3. helper validate '\w+' --lang ts           (must exit 2 with hint)
+#   4. helper validate 'console.log($MSG)'       (must exit 0 plausible)
+#   5. helper validate 'def $F($$$):' --lang py  (must exit 2 - trailing colon)
+#   6. helper validate 'function $N' --lang ts   (must exit 2 - incomplete)
+#   7. helper validate 'foo|bar' --lang ts       (must exit 2 - alternation)
+#   8. helper doctor                             (informational; tolerates no-binary)
+#   9. helper search w/o binary => exit 3 with install hint
+#  10. install.sh --help                         (parses)
+#  11. SKILL.md frontmatter shape                (canonical)
+#
+# Runs against the helper using ONLY stdlib python3 - no ast-grep needed.
+
+set -euo pipefail
+
+SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
+SKILL_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
+HELPER="python3 $SKILL_DIR/scripts/ast_grep_helper.py"
+
+OUTPUT_DIR="$(mktemp -d -t ast-grep-skill-smoke-XXXXXX)"
+trap 'rm -rf "$OUTPUT_DIR"' EXIT
+
+fail() { echo "FAIL: $*" >&2; exit 1; }
+pass() { echo "PASS: $*"; }
+
+omo_runtime_slug() {
+  case "$(uname -s)" in
+    Darwin) local os_slug="darwin" ;;
+    MINGW*|MSYS*|CYGWIN*) local os_slug="win32" ;;
+    *) local os_slug="linux" ;;
+  esac
+
+  case "$(uname -m | tr '[:upper:]' '[:lower:]')" in
+    arm64|aarch64) local arch_slug="arm64" ;;
+    *) local arch_slug="x64" ;;
+  esac
+
+  printf '%s-%s' "$os_slug" "$arch_slug"
+}
+
+fake_sg() {
+  local target="$1"
+  mkdir -p "$(dirname "$target")"
+  cat > "$target" <<'SH'
+#!/usr/bin/env bash
+if [ "${1:-}" = "--version" ]; then
+  printf 'ast-grep 0.43.0 fake\n'
+else
+  printf 'fake ast-grep\n'
+fi
+SH
+  chmod +x "$target"
+}
+
+# 1. --version
+$HELPER --version | grep -q "ast-grep-helper" || fail "--version output missing"
+pass "--version"
+
+# 2. langs (must list 25)
+LANG_LINES=$($HELPER langs | grep -E '^  [a-z]' | wc -l | tr -d ' ')
+[ "$LANG_LINES" -ge 25 ] || fail "langs listed only $LANG_LINES (expected >=25)"
+pass "langs lists at least 25 languages"
+
+# 3. regex misuse: \w+
+set +e
+$HELPER validate '\w+' --lang ts > "$OUTPUT_DIR/v1.out" 2>&1
+RC=$?
+set -e
+[ $RC -eq 2 ] || fail "validate '\\w+' should exit 2, got $RC (output: $(cat $OUTPUT_DIR/v1.out))"
+grep -qi 'regex' "$OUTPUT_DIR/v1.out" || fail "validate '\\w+' should mention regex (output: $(cat $OUTPUT_DIR/v1.out))"
+pass "validate detects \\w regex misuse"
+
+# 4. valid pattern
+set +e
+$HELPER validate 'console.log($MSG)' --lang ts > "$OUTPUT_DIR/v2.out" 2>&1
+RC=$?
+set -e
+[ $RC -eq 0 ] || fail "validate 'console.log(\$MSG)' should exit 0, got $RC (output: $(cat $OUTPUT_DIR/v2.out))"
+pass "validate accepts plausible pattern"
+
+# 5. Python trailing colon
+set +e
+$HELPER validate 'def $F($$$):' --lang py > "$OUTPUT_DIR/v3.out" 2>&1
+RC=$?
+set -e
+[ $RC -eq 2 ] || fail "validate 'def \$F(\$\$\$):' should exit 2, got $RC"
+grep -qi 'colon\|trailing' "$OUTPUT_DIR/v3.out" || fail "validate should mention trailing colon"
+pass "validate detects Python trailing colon"
+
+# 6. Incomplete TS function
+set +e
+$HELPER validate 'function $N' --lang ts > "$OUTPUT_DIR/v4.out" 2>&1
+RC=$?
+set -e
+[ $RC -eq 2 ] || fail "validate 'function \$N' should exit 2, got $RC"
+grep -qi 'incomplete\|params\|body' "$OUTPUT_DIR/v4.out" || fail "validate should hint about params/body"
+pass "validate detects incomplete TS function"
+
+# 7. Alternation pipe
+set +e
+$HELPER validate 'foo|bar' --lang ts > "$OUTPUT_DIR/v5.out" 2>&1
+RC=$?
+set -e
+[ $RC -eq 2 ] || fail "validate 'foo|bar' should exit 2, got $RC"
+grep -qi 'alternation\|regex' "$OUTPUT_DIR/v5.out" || fail "validate should mention alternation"
+pass "validate detects literal | alternation"
+
+# 8. doctor (informational)
+$HELPER doctor > "$OUTPUT_DIR/doc.out" 2>&1 || true
+grep -q "ast-grep-helper" "$OUTPUT_DIR/doc.out" || fail "doctor missing helper version line"
+pass "doctor produces output"
+
+# Given OMO_AST_GREP_SG_PATH points at a fake executable.
+# When doctor resolves ast-grep, then it reports that exact path.
+FAKE_ENV_SG="$OUTPUT_DIR/fake-env/sg"
+fake_sg "$FAKE_ENV_SG"
+OMO_AST_GREP_SG_PATH="$FAKE_ENV_SG" $HELPER doctor > "$OUTPUT_DIR/omo-env.out" 2>&1
+grep -Fq "ast-grep binary: $FAKE_ENV_SG" "$OUTPUT_DIR/omo-env.out" || fail "OMO_AST_GREP_SG_PATH was not preferred: $(cat "$OUTPUT_DIR/omo-env.out")"
+pass "OMO_AST_GREP_SG_PATH resolves first"
+
+# Given HOME has an OMO runtime sg executable.
+# When doctor resolves ast-grep, then it reports the HOME runtime path.
+RUNTIME_HOME="$OUTPUT_DIR/home"
+RUNTIME_SLUG="$(omo_runtime_slug)"
+RUNTIME_BIN="sg"
+case "$RUNTIME_SLUG" in
+  win32-*) RUNTIME_BIN="sg.exe" ;;
+esac
+FAKE_RUNTIME_SG="$RUNTIME_HOME/.omo/runtime/ast-grep/$RUNTIME_SLUG/$RUNTIME_BIN"
+fake_sg "$FAKE_RUNTIME_SG"
+HOME="$RUNTIME_HOME" CODEX_HOME= OMO_AST_GREP_SG_PATH= $HELPER doctor > "$OUTPUT_DIR/omo-runtime.out" 2>&1
+grep -Fq "ast-grep binary: $FAKE_RUNTIME_SG" "$OUTPUT_DIR/omo-runtime.out" || fail "OMO HOME runtime was not resolved: $(cat "$OUTPUT_DIR/omo-runtime.out")"
+pass "OMO HOME runtime resolves before standalone fallback"
+
+# 9. search w/o binary => either runs successfully (binary found) OR exits 3 with hint
+# The CI may or may not have ast-grep installed; both are valid.
+set +e
+$HELPER -q search 'foo()' --lang ts /nonexistent-path-xyzzy > "$OUTPUT_DIR/s1.out" 2>&1
+RC=$?
+set -e
+case "$RC" in
+  0|1|4)
+    # Binary present but no matches OR sg returned non-fatal error - both fine
+    pass "search runs (rc=$RC, ast-grep available)"
+    ;;
+  3)
+    grep -qi 'install' "$OUTPUT_DIR/s1.out" || fail "search rc=3 should print install hint"
+    pass "search without binary prints install hint"
+    ;;
+  *)
+    fail "search returned unexpected rc=$RC: $(cat $OUTPUT_DIR/s1.out)"
+    ;;
+esac
+
+# 10. install.sh syntax check + --help
+bash -n "$SKILL_DIR/install.sh" || fail "install.sh has syntax errors"
+$SKILL_DIR/install.sh --help > "$OUTPUT_DIR/inst.out" 2>&1 || true
+grep -qi 'install' "$OUTPUT_DIR/inst.out" || fail "install.sh --help missing keyword 'install'"
+pass "install.sh syntax + --help"
+
+# 11. SKILL.md frontmatter
+python3 - <<PY
+import re, sys
+src = open("$SKILL_DIR/SKILL.md").read()
+assert src.startswith("---\n"), "SKILL.md must start with YAML frontmatter"
+end = src.find("\n---\n", 4)
+assert end > 0, "SKILL.md missing closing ---"
+fm = src[4:end]
+assert re.search(r"^name:\s*ast-grep\s*$", fm, re.M), "frontmatter missing name: ast-grep"
+assert re.search(r"^description:", fm, re.M), "frontmatter missing description"
+PY
+pass "SKILL.md frontmatter shape"
+
+# 12. All required reference files exist
+for f in references/install.md references/patterns.md references/pitfalls.md \
+         references/recipes.md references/cli.md references/yaml-rules.md \
+         references/sgconfig.md; do
+  test -f "$SKILL_DIR/$f" || fail "missing $f"
+done
+pass "all references present"
+
+python3 - "$SKILL_DIR" <<'PY' || fail "Korean characters found in skill content"
+import pathlib, re, sys
+root = pathlib.Path(sys.argv[1])
+hangul = re.compile(r"[\uac00-\ud7a3]")
+targets = [root / "SKILL.md", root / "README.md"]
+for d in ("references", "scripts", "tests", ".github"):
+    targets.extend((root / d).rglob("*"))
+targets.extend([root / "install.sh", root / "install.ps1"])
+hits = 0
+for p in targets:
+    if not p.is_file():
+        continue
+    try:
+        text = p.read_text(encoding="utf-8")
+    except (UnicodeDecodeError, OSError):
+        continue
+    for n, line in enumerate(text.splitlines(), 1):
+        if hangul.search(line):
+            print(f"{p}:{n}: {line.rstrip()}")
+            hits += 1
+sys.exit(1 if hits else 0)
+PY
+pass "no Korean in skill content"
+
+echo ""
+echo "all smoke tests passed"

package/dist/skills/debugging/SKILL.md

@@ -0,0 +1,116 @@
+---
+name: debugging
+description: "MUST USE for any real runtime debugging across ANY language or binary — crashes, silent failures, wrong responses, stuck processes, memory leaks, async misbehavior, unexplained timing, reverse engineering. Runs a hypothesis-driven loop: form ≥3 hypotheses, investigate in parallel, after 2 failed rounds spawn Oracles from orthogonal angles, confirm root cause, lock with a failing test, fix minimally, QA by actually USING the system, scrub artifacts. The actual HOW lives in `references/` — READ THEM. Triggers: 'debug this', 'why is X not working', 'hanging', 'attach a debugger', 'reverse engineer', 'pwndbg', 'gdb', 'lldb', 'node inspect', 'tsx debug', 'pdb', 'dlv', 'delve', 'rust-gdb', 'set a breakpoint', 'context window exploded', 'why is the response empty', 'attach the debugger', 'debug it', 'why is this happening', 'trace this bug', 'reproduce and fix', 'silent failure', 'HTTP 200 but empty', 'why did it stop', 'inspect the binary', 'reverse engineering', 'playwright'."
+---
+
+# Debugging
+
+You are a hypothesis-driven debugger. Two disciplines apply regardless of language, runtime, or whether you have source:
+
+1. **Runtime truth beats code reading.** Every claim about why the bug happens must come from observed state — never from a plausible story spun from reading code.
+2. **Leave no trace.** Debugging creates artifacts. Every artifact is journaled and removed before you call the task done.
+
+The rest of this file is a map. **The knowledge is in `references/`.** This file cannot teach you how to debug — it can only tell you which reference will, for your exact situation.
+
+---
+
+# 🚨 READ THE REFERENCES. THIS IS NOT OPTIONAL.
+
+> **This skill is intentionally small.** Ninety percent of what you need to know lives in `references/`. If you skim this file and start working without opening the references, you will reattach a debugger the wrong way, miss a silent-failure pattern you've never seen before, waste an hour on a source-map gotcha, or invent a worse version of a tool that already solves your problem.
+>
+> **Every reference below is mandatory when its scenario applies.** "I know this language" is not an exemption. The references exist because every runtime and every specialist tool has at least one gotcha that silently wastes hours, and you will not know which gotcha until you read the file.
+>
+> **The gate rule**: before you run a command from a given reference's domain, you must have read that reference in this session. Re-reading across sessions is cheap. Guessing is expensive.
+
+---
+
+## Runtime Setup — MANDATORY READING BEFORE ATTACHING
+
+The methodology is language-agnostic. The commands to launch, attach, breakpoint, and inspect are not. **Open the matching reference before Phase 0. Not during. Not after.**
+
+| Your runtime is… | Open this before attaching anything | Non-negotiable because… |
+|---|---|---|
+| Python (CPython, pytest, asyncio, Django, FastAPI) | 📖 **[references/runtimes/python.md](references/runtimes/python.md)** | pdb vs ipdb vs debugpy vs pytest --pdb all have different attach semantics. Async code needs special breakpoint handling. Wrappers like `poetry run` swallow flags. |
+| Node.js / tsx / ts-node / Bun / Deno (running source) | 📖 **[references/runtimes/node.md](references/runtimes/node.md)** | `tsx` + `node inspect` CLI has a **silent source-map failure** — breakpoints by line number do not fire. You will not notice unless you read this first. |
+| Rust (cargo, tokio, panics) | 📖 **[references/runtimes/rust.md](references/runtimes/rust.md)** | Release builds strip symbols. Tokio tasks need `tokio-console`. The borrow checker makes `dbg!` the faster tool most of the time. |
+| Go (goroutines, dlv, pprof, race) | 📖 **[references/runtimes/go.md](references/runtimes/go.md)** | Goroutine leaks and recovered panics are silent by default. `dlv` has a specific port convention. `go test -race` is the first thing to run, not the last. |
+| Native binary / stripped C/C++ / no source | 📖 **[references/runtimes/native-binary.md](references/runtimes/native-binary.md)** | The workflow (triage → dynamic → static → scripted repro) is counterintuitive if you've never done it. `strings -n 8` silently drops short interpolations like `${x}` — read bytes directly for any extraction that matters. macOS adds SIP / Mach-O / lldb specifics that don't apply on Linux. |
+| **Bundled-app binary** (Bun SEA, Node SEA, Deno compile, pkg, nexe, Electron, Tauri, PyInstaller) | 📖 **[references/runtimes/bundled-js-binary.md](references/runtimes/bundled-js-binary.md)** | These look like Mach-O / ELF but their *high-level* source is recoverable with the right per-bundler tool — Ghidra is overkill. Source-format reality varies: Bun/pkg/nexe/Electron-asar are usually plaintext; Node SEA with code-cache, PyInstaller `.pyc`, and Deno eszip need extra tooling; Tauri's Rust core still needs native-binary.md. Workflow: identify bundler → locate bundle → extract with the bundler-specific tool → grep. |
+
+**If you cannot honestly say you just opened the reference for your runtime, open it now.**
+
+> 🚨 **Native binary vs bundled binary — check before committing**: `file ./target` calls them both Mach-O / ELF. The 30-second discriminator is `du -h ./target` (50 MB+ suspect bundled) plus `strings -n 12 ./target | rg -iE 'bun|node_modules|webpack|esbuild|deno|pkg/lib|electron|pyinstaller|nexe|NODE_SEA_FUSE|tauri'`. If hits → bundled-js-binary.md. If clean → native-binary.md.
+
+---
+
+## Specialist Tools — ACTIVELY USE WHEN THE SCENARIO FITS
+
+These are not "optional extras". They are the correct tool in their domain, and anything else is slower and less reliable. **If the bug fits the domain, you MUST use the tool. Read the reference first to know how.**
+
+| Tool | Use when | Reference |
+|---|---|---|
+| **Playwright CLI** | Any browser-served web UI bug. Any flow that requires clicking/typing/navigating. Any "works locally, breaks in prod" where the browser or viewport is the variable. **For Phase 8 QA of any browser product, you MUST drive a real browser via Playwright — not curl, not imagination.** | 📖 **[references/tools/playwright-cli.md](references/tools/playwright-cli.md)** |
+| **Ghidra** | Any binary without trustworthy source — third-party closed libs, malware, vendored binaries whose behavior contradicts docs, CTF, firmware. **Use Ghidra's decompiler before `strings`/`objdump` guessing. It turns machine code into readable C.** | 📖 **[references/tools/ghidra.md](references/tools/ghidra.md)** |
+| **pwndbg** | Any native binary debugging session. It is GDB with the useful views (registers, stack, disasm, heap) always visible. **If you'd reach for plain `gdb`, reach for `pwndbg` instead — it is strictly a superset.** | 📖 **[references/tools/pwndbg.md](references/tools/pwndbg.md)** |
+| **pwntools** | Any time you need a reproducible interaction with a binary or network service — crafted payloads, exploit automation, fuzz harness, CTF scripting. | 📖 **[references/tools/pwntools.md](references/tools/pwntools.md)** |
+
+**Failing to use these tools in their domain is a process failure, not a stylistic choice.** If the bug is in a browser and you did Phase 8 without Playwright, you are doing it wrong. If the bug is in a stripped binary and you read hex with `xxd`, you are doing it wrong. The references tell you how. Read them.
+
+---
+
+## The Phase Loop — READ THE REFERENCE FOR THE PHASE YOU ARE ENTERING
+
+Each phase has exactly one reference. Read it as you enter the phase — not in advance, not from memory. The references are self-contained and short.
+
+| # | Phase | 📖 Open this when entering |
+|---|---|---|
+| 0 | **Environment assessment** — know the runtime, ports, symbols, env vars, watchers before attaching | [references/methodology/00-setup.md](references/methodology/00-setup.md) |
+| 1 | **Journal setup** — single `.debug-journal.md` tracks every artifact for guaranteed revert | [references/methodology/00-setup.md](references/methodology/00-setup.md) |
+| 2 | **Hypothesis formation** — minimum three, across orthogonal axes, each with distinguishing evidence | [references/methodology/02-investigate.md](references/methodology/02-investigate.md) |
+| 3 | **Parallel investigation** — team mode `debug-squad` when enabled, async subagents otherwise | [references/methodology/02-investigate.md](references/methodology/02-investigate.md) |
+| 4 | **Oracle Triple** — after 2 consecutive failed rounds, spawn three Oracles with orthogonal framings and synthesize | [references/methodology/04-oracle-triple.md](references/methodology/04-oracle-triple.md) |
+| 5 | **User decision escalation** — only when evidence exhausted and the call has policy implications | [references/methodology/05-escalate.md](references/methodology/05-escalate.md) |
+| 6 | **Root cause confirmation** — confirmed only when toggling the suspected cause toggles the bug | [references/methodology/06-fix.md](references/methodology/06-fix.md) |
+| 7 | **TDD fix** — red test first, minimal green, no scope expansion | [references/methodology/06-fix.md](references/methodology/06-fix.md) |
+| 8 | **Manual QA** — actually use the system (tmux for CLI, Playwright for browser, real curl for API, real repro for binary) | [references/methodology/08-qa.md](references/methodology/08-qa.md) |
+| 9 | **Cleanup** — walk the journal, revert every artifact, verify `git diff` shows only fix + test | [references/methodology/09-cleanup.md](references/methodology/09-cleanup.md) |
+| 10 | **Final verification** — four evidence gates before declaring done | [references/methodology/09-cleanup.md](references/methodology/09-cleanup.md) |
+
+**Phase references are short by design.** Reading one takes a minute. Skipping one costs an hour.
+
+### Cross-cutting methodology references
+
+These are not phases — read them when the situation calls for them:
+
+| Situation | Reference |
+|---|---|
+| You cannot run the actual operation (paid API, blocked network, missing hardware) but still need runtime evidence | 📖 **[references/methodology/partial-runtime-evidence.md](references/methodology/partial-runtime-evidence.md)** |
+| You're about to declare an extraction / audit / reverse-engineering task done and want a skeptical pass | 📖 **[references/methodology/partial-runtime-evidence.md#verification-oracle-pattern-for-non-debug-tasks](references/methodology/partial-runtime-evidence.md#verification-oracle-pattern-for-non-debug-tasks)** (Verification Oracle is *not* the same as Oracle Triple — read the file) |
+
+---
+
+## Non-Negotiable Safety Invariants
+
+<safety>
+1. **Runtime state is the only source of truth.** A hypothesis without an observed value is a guess. Do not fix guesses.
+2. **Every debug artifact is journaled before it is created.** Journal-then-modify, not modify-then-remember-maybe.
+3. **Never ship a fix without a failing-first test.** Red→green transition required, or the fix is unverified.
+4. **Never declare done on type-check/compile alone.** Types catch declaration bugs. Only running the actual user scenario catches the actual user bug.
+5. **Never ask the user a question that runtime evidence can already answer.** Escalation is for genuine ambiguity.
+6. **Never silently swallow errors while debugging.** If the system swallows errors, that is often the bug itself. Make them loud temporarily; restore at cleanup.
+7. **Never `git commit` from inside this skill.** Commits belong to `/git-master` after the user confirms the fix.
+8. **Never attach without having read the runtime reference.** The gate rule.
+</safety>
+
+---
+
+## What to Do Right Now
+
+1. Read the user's bug description.
+2. Identify the runtime.
+3. **Open `references/runtimes/<runtime>.md`.** Read it.
+4. Identify which specialist tools apply. **Open each matching `references/tools/*.md`.** Read them.
+5. Open `references/methodology/00-setup.md` and start Phase 0.
+6. Follow the phase loop. Read each methodology reference as you enter the phase.
+
+**The references are the skill. This file is an index.**

package/dist/skills/debugging/references/methodology/00-setup.md

@@ -0,0 +1,108 @@
+# Phase 0 + 1 — Environment Assessment & Journal Setup
+
+Before a debugger touches anything, you need a map of what's running and a ledger of what you'll touch. Skipping either phase is how debug sessions turn into "why is my repo dirty a week later" sessions.
+
+---
+
+## Phase 0 — Environment Assessment
+
+Map the ground truth before you attach. Attaching the wrong way wastes the first hour.
+
+### 1. Identify the runtime
+
+Read the actual manifest file, don't guess from extensions:
+
+- Python → `pyproject.toml`, `requirements*.txt`, `setup.py`, `uv.lock`, `.python-version`
+- Node → `package.json` (check `scripts`, check `engines`, check `type: module`)
+- Rust → `Cargo.toml`, `rust-toolchain*`
+- Go → `go.mod`, `go.sum`
+- Native / mixed → `Makefile`, `CMakeLists.txt`, the binary itself (`file <path>`)
+
+### 2. Load the matching runtime reference
+
+The moment you know the runtime, open `references/runtimes/<runtime>.md`. The commands in this phase (and every phase after) are runtime-specific. The shape of the answers is the same; the commands are not.
+
+### 3. Gather observable environment state
+
+The shape of the answers you need (commands in the runtime reference):
+
+| Question | Why it matters |
+|---|---|
+| What binary/interpreter/runtime actually launches the process? | Determines debugger flag plumbing. Wrappers (`tsx`, `poetry run`, `cargo run`, `bun`, supervisor scripts) change how flags propagate. |
+| Is there already a debug-relevant port in use, or another instance of the service running? | Either attach to it or kill it deliberately — never silently compete. |
+| Are symbols / source maps / debug info present and correct? | This determines whether breakpoints land on the right lines. Compiled-but-not-debug builds, stripped binaries, and incomplete source maps all silently misplace breakpoints. |
+| Does the code path require env vars, config files, or auth tokens to reach the bug? | Missing env often produces early-return paths that masquerade as the bug itself. |
+| Is there an existing failing test or known repro? | Prefer amplifying an existing repro over inventing one. |
+| Are watchers (file watchers, hot reloaders, supervisors) going to restart the process mid-session? | If yes, turn them off before attaching. Restarts drop inspector connections and invalidate breakpoints. |
+
+### 4. Gate check
+
+If any answer is "I'm not sure", you are not ready for Phase 1. Investigate until certain. Guessing here cascades into false-positive hypotheses in Phase 2.
+
+---
+
+## Phase 1 — Journal Setup
+
+Open **one** journal file at the project root: `.debug-journal.md`. Single source of truth for every artifact this skill creates. The contract with the user that you can undo everything.
+
+### Exclude from git (don't pollute the committed ignore list)
+
+```bash
+grep -qx '.debug-journal.md' .git/info/exclude || echo '.debug-journal.md' >> .git/info/exclude
+```
+
+`.git/info/exclude` is per-clone and not committed — perfect for local-session artifacts.
+
+### Journal template
+
+```markdown
+# Debug Journal — <short bug name>
+Started: <ISO timestamp>
+Goal: <one-sentence user request>
+
+## Environment snapshot (Phase 0)
+- Runtime: <language + version + launcher>
+- Entry: <command that starts the process>
+- Ports / sockets: <app=..., debugger=..., etc>
+- Git HEAD: <sha>, working tree clean? <yes/no>
+- References read: <list the files from references/ you loaded — proves you did the gate>
+
+## Hypotheses
+1. [STATUS] <hypothesis> — distinguishing evidence: <what would confirm/refute> — if true, fix is: <two words>
+2. ...
+
+## Failed hypothesis round counter
+- Round 1: <result>
+- Round 2: <result>
+<!-- At 2 consecutive failures, invoke Oracle Triple (see 04-oracle-triple.md). -->
+
+## Artifacts to revert
+<!-- Every temp edit, tmux session, fixture, env override, saved debugger session goes here
+     BEFORE it is created. The rule is journal-then-modify. -->
+- [ ] `src/foo.py` — added `breakpoint()` on 2 lines. Revert: `git checkout src/foo.py`
+- [ ] tmux session `debug-server`. Kill: `tmux kill-session -t debug-server`
+- [ ] `/tmp/debug-payload.json`. Remove: `rm /tmp/debug-payload.json`
+- [ ] env var in current shell: `FOO_BASE_URL=...`. Unset when done.
+- [ ] GDB session save: `~/ghidra-projects/scratch.gzf`. Remove if not promoting.
+
+## Findings
+<!-- Append observed values here with timestamp. Verbatim only, no paraphrasing. -->
+
+## Oracle Triple (if invoked)
+<!-- One subsection per Oracle round, with the synthesized new hypothesis set. -->
+
+## Final fix
+<!-- File paths + test path. Filled during Phase 7. -->
+```
+
+### The journal-then-modify rule
+
+Before any modification to the repo, shell, or system state, append to "Artifacts to revert" first. This one discipline is what prevents debug sessions from becoming git cleanup sessions.
+
+If you catch yourself about to run a command that creates a file, opens a port, or modifies source — stop, journal the intended artifact with its revert command, then run the command. Not the other way around.
+
+### Why a single journal (not scattered TODO comments)
+
+- One `git checkout`, one `rm`, one `tmux kill-session` list — simple Phase 9 walk.
+- Survives interruptions. If you get pulled away mid-session, the next agent (or you later) can continue or revert without guessing.
+- Prevents the most common failure: leaving `console.log`/`print()`/`dbg!` scattered across the tree.