Lap Tran
bc7daf8590
- **Restore 82 completed tasks** from tasks/complete/ back to tasks/ top level (all <7 days old per the cleanup policy; premature bulk archive was fixed). - **Dashboard: fix scroll-reset on auto-refresh** — renderBoard rebuilds the board via innerHTML every 2s, destroying each column-body's scrollTop. Now snapshots column-body scrollTop + board.scrollLeft + view.scrollTop before rebuild and restores after (matched by PHASE_GROUPS index). - **Dashboard UI additions** (pre-existing unstaged work): approval section cards, transition buttons, inline artifact editor (textarea for writing missing SPEC/VERDICT/etc from the detail modal). - **Bind ornith as Implement model** — config.md: Model explicit to omlx/Ornith-1.0-35B-4bit-mlx, context window 32768. Interactive autopilot already used ornith via opencode default; now explicit. - **Fix cleanup stub** — automaton-cleanup.sh had a stale --project arg pointing at a pytest temp dir (test isolation leak). Rewired to point at ~/.automaton. - **Fix plist-isolation test** — test asserted host plist doesn't exist, but a real install creates it. Now snapshots mtime before run, asserts unchanged after (only a write during the test counts as bleed). - **New Playwright smoke test** (tests/test_dashboard_ui.py) — 2 tests: board renders tasks, column scroll survives auto-refresh tick. Verified the test fails without the scroll fix (scrollTop resets to 0). Skipped via importorskip when playwright is absent (main CI stays green). - **Clarify SI loop scope in README** — new-project onboarding section documents the framework-scoped self-improvement loop and options (leave/pause/create project loop). - **CHANGELOG** documents all changes including the known model-divergence gap (mde tasks marked complete but per-role model binding was never implemented).
44 lines
2.0 KiB
Markdown
44 lines
2.0 KiB
Markdown
# DOC_REVIEW: add-goal-mode
|
|
|
|
Reviewed doc impact for task `add-goal-mode`.
|
|
|
|
## Doc edits in this task
|
|
|
|
### 1. `design/loops/technical.md`
|
|
Schema section (section 2) already updated with `work_source` and `acceptance_criteria` fields. Self-improvement example (section 9) already references `work_source: {kind: "audit"}`. No further changes needed.
|
|
|
|
### 2. `design/loops/functional.md`
|
|
Already documents `work_source` and `acceptance_criteria` in the loop.json field list (lines 96-97). No change needed.
|
|
|
|
### 3. `templates/loops/ci-triage/loop.json`
|
|
Updated with explicit `"work_source": {"kind": "single"}` and `"acceptance_criteria": [...]`. Matches SPEC R7. No further change.
|
|
|
|
### 4. `AGENTS.md`
|
|
The "State Enforcement -- Loops (v1)" section mentions `--check-gate` and the runner. No new CLI surface in this task (the `--goal` flag is deferred to v1.1 per SPEC Non-Goals). No change needed.
|
|
|
|
### 5. `README.md`
|
|
The loop engineering section already references work sources at a high level. The specific `work_source.kind` values (`single`, `audit`, `backlog`) are implementation details documented in `design/loops/`. No change needed for v1.
|
|
|
|
### 6. `CHANGELOG.md`
|
|
Add an `[unreleased]` entry for goal-mode work sources, verifier tokens, and audit --json. **Action:** apply.
|
|
|
|
### 7. `prompts/`
|
|
No loop prompts land in this task (deferred to task 6 per SPEC Non-Goals). No change.
|
|
|
|
### 8. `contracts/harness-integration.md`
|
|
No new harness integration surface in this task. No change.
|
|
|
|
## Code-doc consistency check
|
|
|
|
- `technical.md` section 2 schema: `work_source.kind` values match the `_FIND_WORK_DISPATCH` keys (`single`, `audit`, `backlog`). PASS
|
|
- `technical.md` section 2 schema: `acceptance_criteria` described as "string OR list" matches `_acceptance_criteria_text` implementation. PASS
|
|
- `functional.md` line 96-97: `work_source` shape matches implementation. PASS
|
|
- `ci-triage/loop.json`: template fields match schema docs. PASS
|
|
|
|
## Summary
|
|
|
|
Doc edits in this task:
|
|
- `CHANGELOG.md`: new `[unreleased]` entry.
|
|
|
|
No code-doc mismatches found. READY for referee.
|