@fro.bot/systematic 2.0.1 → 2.0.3

package/skills/setup/SKILL.md

@@ -8,26 +8,20 @@ disable-model-invocation: true
 
 ## Interaction Method
 
-If `question` is available, use it for all prompts below.
+Ask the user each question below using the platform's blocking question tool (e.g., `question` in OpenCode, `request_user_input` in Codex, `ask_user` in Gemini). If no structured question tool is available, present each question as a numbered list and wait for a reply before proceeding. For multiSelect questions, accept comma-separated numbers (e.g. `1, 3`). Never skip or auto-configure.
 
-If not, present each question as a numbered list and wait for a reply before proceeding to the next step. For multiSelect questions, accept comma-separated numbers (e.g. `1, 3`). Never skip or auto-configure.
-
-Interactive setup for `systematic.local.md` — configures which agents run during `/ce:review` and `/ce:work`.
+Interactive setup for `systematic.local.md` — configures which agents run during `ce:review` and `ce:work`.
 
 ## Step 1: Check Existing Config
 
-Read `systematic.local.md` in the project root. If it exists, display current settings summary and use question:
+Read `systematic.local.md` in the project root. If it exists, display current settings and ask:
 
 ```
-question: "Settings file already exists. What would you like to do?"
-header: "Config"
-options:
-  - label: "Reconfigure"
-    description: "Run the interactive setup again from scratch"
-  - label: "View current"
-    description: "Show the file contents, then stop"
-  - label: "Cancel"
-    description: "Keep current settings"
+Settings file already exists. What would you like to do?
+
+1. Reconfigure - Run the interactive setup again from scratch
+2. View current - Show the file contents, then stop
+3. Cancel - Keep current settings
 ```
 
 If "View current": read and display the file, then stop.
@@ -47,16 +41,13 @@ test -f requirements.txt && echo "python" || \
 echo "general"
 ```
 
-Use question:
+Ask:
 
 ```
-question: "Detected {type} project. How would you like to configure?"
-header: "Setup"
-options:
-  - label: "Auto-configure (Recommended)"
-    description: "Use smart defaults for {type}. Done in one click."
-  - label: "Customize"
-    description: "Choose stack, focus areas, and review depth."
+Detected {type} project. How would you like to configure?
+
+1. Auto-configure (Recommended) - Use smart defaults for {type}. Done in one click.
+2. Customize - Choose stack, focus areas, and review depth.
 ```
 
 ### If Auto-configure → Skip to Step 4 with defaults:
@@ -73,50 +64,35 @@ options:
 **a. Stack** — confirm or override:
 
 ```
-question: "Which stack should we optimize for?"
-header: "Stack"
-options:
-  - label: "{detected_type} (Recommended)"
-    description: "Auto-detected from project files"
-  - label: "Rails"
-    description: "Ruby on Rails — adds DHH-style and Rails-specific reviewers"
-  - label: "Python"
-    description: "Python — adds Pythonic pattern reviewer"
-  - label: "TypeScript"
-    description: "TypeScript — adds type safety reviewer"
+Which stack should we optimize for?
+
+1. {detected_type} (Recommended) - Auto-detected from project files
+2. Rails - Ruby on Rails, adds DHH-style and Rails-specific reviewers
+3. Python - Adds Pythonic pattern reviewer
+4. TypeScript - Adds type safety reviewer
 ```
 
 Only show options that differ from the detected type.
 
-**b. Focus areas** — multiSelect:
+**b. Focus areas** — multiSelect (user picks one or more):
 
 ```
-question: "Which review areas matter most?"
-header: "Focus"
-multiSelect: true
-options:
-  - label: "Security"
-    description: "Vulnerability scanning, auth, input validation (security-sentinel)"
-  - label: "Performance"
-    description: "N+1 queries, memory leaks, complexity (performance-oracle)"
-  - label: "Architecture"
-    description: "Design patterns, SOLID, separation of concerns (architecture-strategist)"
-  - label: "Code simplicity"
-    description: "Over-engineering, YAGNI violations (code-simplicity-reviewer)"
+Which review areas matter most? (comma-separated, e.g. 1, 3)
+
+1. Security - Vulnerability scanning, auth, input validation (security-sentinel)
+2. Performance - N+1 queries, memory leaks, complexity (performance-oracle)
+3. Architecture - Design patterns, SOLID, separation of concerns (architecture-strategist)
+4. Code simplicity - Over-engineering, YAGNI violations (code-simplicity-reviewer)
 ```
 
 **c. Depth:**
 
 ```
-question: "How thorough should reviews be?"
-header: "Depth"
-options:
-  - label: "Thorough (Recommended)"
-    description: "Stack reviewers + all selected focus agents."
-  - label: "Fast"
-    description: "Stack reviewers + code simplicity only. Less context, quicker."
-  - label: "Comprehensive"
-    description: "All above + git history, data integrity, agent-native checks."
+How thorough should reviews be?
+
+1. Thorough (Recommended) - Stack reviewers + all selected focus agents.
+2. Fast - Stack reviewers + code simplicity only. Less context, quicker.
+3. Comprehensive - All above + git history, data integrity, agent-native checks.
 ```
 
 ## Step 4: Build Agent List and Write File
@@ -151,7 +127,7 @@ plan_review_agents: [{computed plan agent list}]
 # Review Context
 
 Add project-specific review instructions here.
-These notes are passed to all review agents during /ce:review and /ce:work.
+These notes are passed to all review agents during ce:review and ce:work.
 
 Examples:
 - "We use Turbo Frames heavily — check for frame-busting issues"
@@ -172,3 +148,4 @@ Agents:       {count} configured
 Tip: Edit the "Review Context" section to add project-specific instructions.
      Re-run this setup anytime to reconfigure.
 ```
+

package/skills/slfg/SKILL.md

@@ -9,26 +9,26 @@ Swarm-enabled LFG. Run these steps in order, parallelizing where indicated. Do n
 
 ## Sequential Phase
 
-1. **Optional:** If the `ralph-wiggum` skill is available, run `/ralph-wiggum:ralph-loop "finish all slash commands" --completion-promise "DONE"`. If not available or it fails, skip and continue to step 2 immediately.
-2. `/systematic:ce-plan $ARGUMENTS`
+1. **Optional:** If the `ralph-loop` skill is available, run `/ralph-loop:ralph-loop "finish all slash commands" --completion-promise "DONE"`. If not available or it fails, skip and continue to step 2 immediately.
+2. `/ce:plan $ARGUMENTS`
 3. **Conditionally** run `/systematic:deepen-plan`
    - Run the `deepen-plan` workflow only if the plan is `Standard` or `Deep`, touches a high-risk area (auth, security, payments, migrations, external APIs, significant rollout concerns), or still has obvious confidence gaps in decisions, sequencing, system-wide impact, risks, or verification
    - If you run the `deepen-plan` workflow, confirm the plan was deepened or explicitly judged sufficiently grounded before moving on
    - If you skip it, note why and continue to step 4
-4. `/systematic:ce-work` — **Use swarm mode**: Make a Task list and launch an army of agent swarm subagents to build the plan
+4. `/ce:work` — **Use swarm mode**: Make a Task list and launch an army of agent swarm subagents to build the plan
 
 ## Parallel Phase
 
 After work completes, launch steps 5 and 6 as **parallel swarm agents** (both only need code to be written):
 
-5. `/systematic:ce-review` — spawn as background Task agent
+5. `/ce:review` — spawn as background Task agent
 6. `/systematic:test-browser` — spawn as background Task agent
 
 Wait for both to complete before continuing.
 
 ## Finalize Phase
 
-7. `/systematic:resolve_todo_parallel` — resolve any findings from the review
+7. `/systematic:resolve-todo-parallel` — resolve findings, compound on learnings, clean up completed todos
 8. `/systematic:feature-video` — record the final walkthrough and add to PR
 9. Output `<promise>DONE</promise>` when video is in PR
 

package/skills/test-browser/SKILL.md

@@ -4,56 +4,45 @@ description: Run browser tests on pages affected by current PR or branch
 argument-hint: '[PR number, branch name, ''current'', or --port PORT]'
 ---
 
-# Browser Test Command
+# Browser Test Skill
 
-<command_purpose>Run end-to-end browser tests on pages affected by a PR or branch changes using agent-browser CLI.</command_purpose>
+Run end-to-end browser tests on pages affected by a PR or branch changes using the `agent-browser` CLI.
 
-## CRITICAL: Use agent-browser CLI Only
+## Use `agent-browser` Only For Browser Automation
 
-**DO NOT use Chrome MCP tools (mcp__claude-in-chrome__*).**
+This workflow uses the `agent-browser` CLI exclusively. Do not use any alternative browser automation system, browser MCP integration, or built-in browser-control tool. If the platform offers multiple ways to control a browser, always choose `agent-browser`.
 
-This command uses the `agent-browser` CLI exclusively. The agent-browser CLI is a Bash-based tool from Vercel that runs headless Chromium. It is NOT the same as Chrome browser automation via MCP.
+Use `agent-browser` for: opening pages, clicking elements, filling forms, taking screenshots, and scraping rendered content.
 
-If you find yourself calling `mcp__claude-in-chrome__*` tools, STOP. Use `agent-browser` Bash commands instead.
-
-## Introduction
-
-<role>QA Engineer specializing in browser-based end-to-end testing</role>
-
-This command tests affected pages in a real browser, catching issues that unit tests miss:
-- JavaScript integration bugs
-- CSS/layout regressions
-- User workflow breakages
-- Console errors
+Platform-specific hints:
+- In OpenCode, do not use Chrome MCP tools (`mcp__claude-in-chrome__*`).
+- In Codex, do not substitute unrelated browsing tools.
 
 ## Prerequisites
 
-<requirements>
 - Local development server running (e.g., `bin/dev`, `rails server`, `npm run dev`)
-- agent-browser CLI installed (see Setup below)
+- `agent-browser` CLI installed (see Setup below)
 - Git repository with changes to test
-</requirements>
 
 ## Setup
 
-**Check installation:**
 ```bash
 command -v agent-browser >/dev/null 2>&1 && echo "Installed" || echo "NOT INSTALLED"
 ```
 
-**Install if needed:**
+Install if needed:
 ```bash
 npm install -g agent-browser
-agent-browser install  # Downloads Chromium (~160MB)
+agent-browser install
 ```
 
 See the `agent-browser` skill for detailed usage.
 
-## Main Tasks
+## Workflow
 
-### 0. Verify agent-browser Installation
+### 1. Verify Installation
 
-Before starting ANY browser testing, verify agent-browser is installed:
+Before starting, verify `agent-browser` is available:
 
 ```bash
 command -v agent-browser >/dev/null 2>&1 && echo "Ready" || (echo "Installing..." && npm install -g agent-browser && agent-browser install)
@@ -61,27 +50,20 @@ command -v agent-browser >/dev/null 2>&1 && echo "Ready" || (echo "Installing...
 
 If installation fails, inform the user and stop.
 
-### 1. Ask Browser Mode
-
-<ask_browser_mode>
-
-Before starting tests, ask user if they want to watch the browser:
-
-Use question with:
-- Question: "Do you want to watch the browser tests run?"
-- Options:
-  1. **Headed (watch)** - Opens visible browser window so you can see tests run
-  2. **Headless (faster)** - Runs in background, faster but invisible
+### 2. Ask Browser Mode
 
-Store the choice and use `--headed` flag when user selects "Headed".
+Ask the user whether to run headed or headless (using the platform's question tool — e.g., `question` in OpenCode, `request_user_input` in Codex, `ask_user` in Gemini — or present options and wait for a reply):
 
-</ask_browser_mode>
+```
+Do you want to watch the browser tests run?
 
-### 2. Determine Test Scope
+1. Headed (watch) - Opens visible browser window so you can see tests run
+2. Headless (faster) - Runs in background, faster but invisible
+```
 
-<test_target> $ARGUMENTS </test_target>
+Store the choice and use the `--headed` flag when the user selects option 1.
 
-<determine_scope>
+### 3. Determine Test Scope
 
 **If PR number provided:**
 ```bash
@@ -98,11 +80,7 @@ git diff --name-only main...HEAD
 git diff --name-only main...[branch]
 ```
 
-</determine_scope>
-
-### 3. Map Files to Routes
-
-<file_to_route_mapping>
+### 4. Map Files to Routes
 
 Map changed files to testable routes:
 
@@ -120,45 +98,23 @@ Map changed files to testable routes:
 
 Build a list of URLs to test based on the mapping.
 
-</file_to_route_mapping>
-
-### 4. Detect Dev Server Port
-
-<detect_port>
+### 5. Detect Dev Server Port
 
-Determine the dev server port using this priority order:
+Determine the dev server port using this priority:
 
-**Priority 1: Explicit argument**
-If the user passed a port number (e.g., `/test-browser 5000` or `/test-browser --port 5000`), use that port directly.
+1. **Explicit argument** — if the user passed `--port 5000`, use that directly
+2. **Project instructions** — check `AGENTS.md`, `AGENTS.md`, or other instruction files for port references
+3. **package.json** — check dev/start scripts for `--port` flags
+4. **Environment files** — check `.env`, `.env.local`, `.env.development` for `PORT=`
+5. **Default** — fall back to `3000`
 
-**Priority 2: AGENTS.md / project instructions**
 ```bash
-# Check AGENTS.md for port references
-grep -Eio '(port\s*[:=]\s*|localhost:)([0-9]{4,5})' AGENTS.md 2>/dev/null | grep -Eo '[0-9]{4,5}' | head -1
-```
-
-**Priority 3: package.json scripts**
-```bash
-# Check dev/start scripts for --port flags
-grep -Eo '\-\-port[= ]+[0-9]{4,5}' package.json 2>/dev/null | grep -Eo '[0-9]{4,5}' | head -1
-```
-
-**Priority 4: Environment files**
-```bash
-# Check .env, .env.local, .env.development for PORT=
-grep -h '^PORT=' .env .env.local .env.development 2>/dev/null | tail -1 | cut -d= -f2
-```
-
-**Priority 5: Default fallback**
-If none of the above yields a port, default to `3000`.
-
-Store the result in a `PORT` variable for use in all subsequent steps.
-
-```bash
-# Combined detection (run this)
 PORT="${EXPLICIT_PORT:-}"
 if [ -z "$PORT" ]; then
   PORT=$(grep -Eio '(port\s*[:=]\s*|localhost:)([0-9]{4,5})' AGENTS.md 2>/dev/null | grep -Eo '[0-9]{4,5}' | head -1)
+  if [ -z "$PORT" ]; then
+    PORT=$(grep -Eio '(port\s*[:=]\s*|localhost:)([0-9]{4,5})' AGENTS.md 2>/dev/null | grep -Eo '[0-9]{4,5}' | head -1)
+  fi
 fi
 if [ -z "$PORT" ]; then
   PORT=$(grep -Eo '\-\-port[= ]+[0-9]{4,5}' package.json 2>/dev/null | grep -Eo '[0-9]{4,5}' | head -1)
@@ -170,77 +126,64 @@ PORT="${PORT:-3000}"
 echo "Using dev server port: $PORT"
 ```
 
-</detect_port>
-
-### 5. Verify Server is Running
-
-<check_server>
-
-Before testing, verify the local server is accessible using the detected port:
+### 6. Verify Server is Running
 
 ```bash
 agent-browser open http://localhost:${PORT}
 agent-browser snapshot -i
 ```
 
-If server is not running, inform user:
-```markdown
-**Server not running on port ${PORT}**
+If the server is not running, inform the user:
+
+```
+Server not running on port ${PORT}
 
 Please start your development server:
 - Rails: `bin/dev` or `rails server`
 - Node/Next.js: `npm run dev`
-- Custom port: `/test-browser --port <your-port>`
+- Custom port: run this skill again with `--port <your-port>`
 
-Then run `/test-browser` again.
+Then re-run this skill.
 ```
 
-</check_server>
-
-### 6. Test Each Affected Page
+### 7. Test Each Affected Page
 
-<test_pages>
+For each affected route:
 
-For each affected route, use agent-browser CLI commands (NOT Chrome MCP):
-
-**Step 1: Navigate and capture snapshot**
+**Navigate and capture snapshot:**
 ```bash
 agent-browser open "http://localhost:${PORT}/[route]"
 agent-browser snapshot -i
 ```
 
-**Step 2: For headed mode (visual debugging)**
+**For headed mode:**
 ```bash
 agent-browser --headed open "http://localhost:${PORT}/[route]"
 agent-browser --headed snapshot -i
 ```
 
-**Step 3: Verify key elements**
+**Verify key elements:**
 - Use `agent-browser snapshot -i` to get interactive elements with refs
 - Page title/heading present
 - Primary content rendered
 - No error messages visible
 - Forms have expected fields
 
-**Step 4: Test critical interactions**
+**Test critical interactions:**
 ```bash
-agent-browser click @e1  # Use ref from snapshot
+agent-browser click @e1
 agent-browser snapshot -i
 ```
 
-**Step 5: Take screenshots**
+**Take screenshots:**
 ```bash
 agent-browser screenshot page-name.png
-agent-browser screenshot --full page-name-full.png  # Full page
+agent-browser screenshot --full page-name-full.png
 ```
 
-</test_pages>
-
-### 7. Human Verification (When Required)
-
-<human_verification>
+### 8. Human Verification (When Required)
 
-Pause for human input when testing touches:
+Pause for human input when testing touches flows that require external interaction:
 
 | Flow Type | What to Ask |
 |-----------|-------------|
@@ -250,11 +193,12 @@ Pause for human input when testing touches:
 | SMS | "Verify you received the SMS code" |
 | External APIs | "Confirm the [service] integration is working" |
 
-Use question:
-```markdown
-**Human Verification Needed**
+Ask the user (using the platform's question tool, or present numbered options and wait):
+
+```
+Human Verification Needed
 
-This test touches the [flow type]. Please:
+This test touches [flow type]. Please:
 1. [Action to take]
 2. [What to verify]
 
@@ -263,11 +207,7 @@ Did it work correctly?
 2. No - describe the issue
 ```
 
-</human_verification>
-
-### 8. Handle Failures
-
-<failure_handling>
+### 9. Handle Failures
 
 When a test fails:
 
@@ -275,40 +215,27 @@ When a test fails:
    - Screenshot the error state: `agent-browser screenshot error.png`
    - Note the exact reproduction steps
 
-2. **Ask user how to proceed:**
-   ```markdown
-   **Test Failed: [route]**
+2. **Ask the user how to proceed:**
+
+   ```
+   Test Failed: [route]
 
    Issue: [description]
    Console errors: [if any]
 
    How to proceed?
    1. Fix now - I'll help debug and fix
-   2. Create todo - Add to todos/ for later
+   2. Create todo - Add a todo for later (using the file-todos skill)
    3. Skip - Continue testing other pages
    ```
 
-3. **If "Fix now":**
-   - Investigate the issue
-   - Propose a fix
-   - Apply fix
-   - Re-run the failing test
-
-4. **If "Create todo":**
-   - Create `{id}-pending-p1-browser-test-{description}.md`
-   - Continue testing
-
-5. **If "Skip":**
-   - Log as skipped
-   - Continue testing
+3. **If "Fix now":** investigate, propose a fix, apply, re-run the failing test
+4. **If "Create todo":** load the `file-todos` skill and create a todo with priority p1 and description `browser-test-{description}`, continue
+5. **If "Skip":** log as skipped, continue
 
-</failure_handling>
+### 10. Test Summary
 
-### 9. Test Summary
-
-<test_summary>
-
-After all tests complete, present summary:
+After all tests complete, present a summary:
 
 ```markdown
 ## Browser Test Results
@@ -341,8 +268,6 @@ After all tests complete, present summary:
 ### Result: [PASS / FAIL / PARTIAL]
 ```
 
-</test_summary>
-
 ## Quick Usage Examples
 
 ```bash
@@ -361,8 +286,6 @@ After all tests complete, present summary:
 
 ## agent-browser CLI Reference
 
-**ALWAYS use these Bash commands. NEVER use mcp__claude-in-chrome__* tools.**
-
 ```bash
 # Navigation
 agent-browser open <url>           # Navigate to URL
@@ -391,3 +314,4 @@ agent-browser --headed click @e1       # Click in visible browser
 agent-browser wait @e1             # Wait for element
 agent-browser wait 2000            # Wait milliseconds
 ```
+