deepflow 0.1.90 → 0.1.91

package/src/commands/df/execute.md

@@ -101,24 +101,37 @@ Context ≥50% → checkpoint and exit. Before spawning: `TaskUpdate(status: "in
 
 ### 5.5. RATCHET CHECK
 
-Run health checks in worktree after each agent completes.
+Run `node bin/ratchet.js` in the worktree directory after each agent completes:
+```bash
+node bin/ratchet.js --worktree ${WORKTREE_PATH} --snapshot .deepflow/auto-snapshot.txt
+```
+
+The script handles all health checks internally and outputs structured JSON:
+```json
+{"status": "PASS"|"FAIL"|"SALVAGEABLE", "reason": "...", "details": "..."}
+```
+
+**Exit codes:** 0 = PASS, 1 = FAIL (script already ran `git revert HEAD --no-edit`), 2 = SALVAGEABLE (lint/typecheck only; build+tests passed).
+
+**You MUST NOT inspect, classify, or reinterpret test failures. FAIL means revert. No exceptions.**
 
-| File | Build | Test | Typecheck | Lint |
-|------|-------|------|-----------|------|
-| `package.json` | `npm run build` | `npm test` | `npx tsc --noEmit` | `npm run lint` |
-| `pyproject.toml` | — | `pytest` | `mypy .` | `ruff check .` |
-| `Cargo.toml` | `cargo build` | `cargo test` | — | `cargo clippy` |
-| `go.mod` | `go build ./...` | `go test ./...` | — | `go vet ./...` |
+**Prohibited actions during ratchet:**
+- No `git stash` or `git checkout` for investigation purposes
+- No inline edits to pre-existing test files
+- No reading raw test output to decide what "really" failed
 
-Run Build → Test → Typecheck → Lint (stop on first failure). Ratchet uses ONLY pre-existing tests from `.deepflow/auto-snapshot.txt`.
+**Broken-tests policy:** Updating pre-existing tests requires a separate dedicated task in PLAN.md with explicit justification — never inline during execution.
+
+**Orchestrator response by exit code:**
+- **Exit 0 (PASS):** Commit stands. Proceed to §5.6 wave test agent.
+- **Exit 1 (FAIL):** Script already reverted. Set `TaskUpdate(status: "pending")`. Report: `"✗ T{n}: reverted"`.
+- **Exit 2 (SALVAGEABLE):** Spawn `Agent(model="haiku")` to fix lint/typecheck issues. Re-run `node bin/ratchet.js`. If still non-zero → revert both commits, set status pending.
 
 **Edit scope validation:** `git diff HEAD~1 --name-only` vs allowed globs. Violation → revert, report.
 **Impact completeness:** diff vs Impact callers/duplicates. Gap → advisory warning (no revert).
 
 **Metric gate (Optimize only):** Run `eval "${metric_command}"` with cwd=`${WORKTREE_PATH}` (never `cd && eval`). Parse float (non-numeric → revert). Compare using `direction`+`min_improvement_threshold`. Both ratchet AND metric must pass → keep. Ratchet pass + metric stagnant → revert. Secondary metrics: regression > `regression_threshold` (5%) → WARNING in auto-report.md (no revert).
 
-**Output truncation:** Success → suppress. Build fail → last 15 lines. Test fail → names + last 20 lines. Typecheck/lint → count + first 5 errors.
-
 **Token tracking result (on pass):** Read `end_percentage`. Sum token fields from `.deepflow/token-history.jsonl` between start/end timestamps (awk ISO 8601 compare). Write to `.deepflow/results/T{N}.yaml`:
 ```yaml
 tokens:
@@ -131,10 +144,6 @@ tokens:
 ```
 Omit if context.json/token-history.jsonl/awk unavailable. Never fail ratchet for tracking errors.
 
-**Evaluate:** All pass → commit stands. Failure → partial salvage:
-1. Lint/typecheck-only (build+tests passed): spawn `Agent(model="haiku")` to fix. Re-ratchet. Fail → revert both.
-2. Build/test failure → `git revert HEAD --no-edit` (no salvage).
-
 ### 5.6. WAVE TEST AGENT
 
 <!-- AC-8: After wave ratchet passes, Opus test agent spawns and writes unit tests -->
@@ -147,8 +156,11 @@ Omit if context.json/token-history.jsonl/awk unavailable. Never fail ratchet for
 
 **Flow:**
 1. Capture the implementation diff: `git -C ${WORKTREE_PATH} diff HEAD~1` → store as `IMPL_DIFF`.
-2. Spawn `Agent(model="opus")` with Wave Test prompt (§6). `run_in_background=true`. End turn, wait.
-3. On notification:
+2. Gather dedup context:
+   - Read `.deepflow/auto-snapshot.txt` → store full file list as `SNAPSHOT_FILES`.
+   - Extract existing test function names: `grep -h 'describe\|it(\|test(\|def test_\|func Test' $(cat .deepflow/auto-snapshot.txt) 2>/dev/null | head -50` → store as `EXISTING_TEST_NAMES`.
+3. Spawn `Agent(model="opus")` with Wave Test prompt (§6), passing `SNAPSHOT_FILES` and `EXISTING_TEST_NAMES`. `run_in_background=true`. End turn, wait.
+4. On notification:
    a. Run ratchet check (§5.5) — all new + pre-existing tests must pass.
    b. **Tests pass** → commit stands. **Re-snapshot** immediately so wave N+1 ratchet includes wave N tests:
       ```bash
@@ -167,7 +179,7 @@ Omit if context.json/token-history.jsonl/awk unavailable. Never fail ratchet for
           {failure_feedback}
           Fix the issues above. Do NOT repeat the same mistakes.
           ```
-        - On implementer notification: ratchet check (§5.5). Passed → goto step 1 (spawn test agent again). Failed → same retry logic.
+        - On implementer notification: ratchet check (§5.5). Passed → goto step 2 (gather dedup context, spawn test agent again). Failed → same retry logic.
       - If `attempt_count >= 3`:
         - Revert ALL commits back to pre-task state: `git -C ${WORKTREE_PATH} reset --hard {pre_task_commit}`
         - `TaskUpdate(status: "pending")`
@@ -279,9 +291,16 @@ Implementation diff:
 Files changed: {changed_files}
 Existing test patterns: {test_file_examples from auto-snapshot.txt, first 3}
 
+Pre-existing test files (from auto-snapshot.txt):
+{SNAPSHOT_FILES}
+
+Existing test function names (do NOT duplicate these):
+{EXISTING_TEST_NAMES}
+
 --- END ---
 Write thorough unit tests covering: happy paths, edge cases, error handling.
 Follow existing test conventions in the codebase.
+Do not duplicate tests for functionality already covered by the existing tests listed above.
 Commit as: test({spec}): wave-{N} unit tests
 Do NOT modify implementation files. ONLY add/edit test files.
 Last line of your response MUST be: TASK_STATUS:pass or TASK_STATUS:fail

package/templates/config-template.yaml

@@ -95,6 +95,30 @@ quality:
   # Timeout in seconds to wait for the dev server to become ready (default: 30)
   browser_timeout: 30
 
+# Ratchet configuration for /df:verify health gate
+# Ratchet snapshots baseline metrics (tests passing, coverage, type checks) before execution
+# and ensures subsequent runs don't regress. These overrides control which commands ratchet monitors.
+ratchet:
+  # Override auto-detected build command for ratchet health checks
+  # If empty, ratchet will detect from indicator files (package.json scripts, Cargo.toml, etc.)
+  # Examples: "npm run build", "cargo build", "go build ./..."
+  build_command: ""
+
+  # Override auto-detected test command for ratchet health checks
+  # If empty, ratchet will detect from indicator files
+  # Examples: "npm test", "pytest", "go test ./...", "cargo test"
+  test_command: ""
+
+  # Override auto-detected typecheck command for ratchet health checks
+  # If empty, ratchet will detect from indicator files (e.g., "tsc --noEmit" for TypeScript)
+  # Examples: "tsc --noEmit", "mypy .", "cargo check"
+  typecheck_command: ""
+
+  # Override auto-detected lint command for ratchet health checks
+  # If empty, ratchet will detect from indicator files
+  # Examples: "eslint .", "flake8", "cargo clippy"
+  lint_command: ""
+
 # deepflow-dashboard team mode settings
 # dashboard_url: URL of the shared team server for POST ingestion
 # Leave blank (or omit) to use local-only mode (no data is pushed)