opencode-codeops 2.1.1 → 2.2.0

package/scripts/codeops_plan_migrate.py

@@ -17,9 +17,21 @@ from dataclasses import dataclass
 from pathlib import Path
 
 try:
-    from scripts.codeops_plan import BLOCKED_REASON_RE, IMPLEMENTS_RE, parse_implements, parse_tasks
+    from scripts.codeops_plan import (
+        BLOCKED_REASON_RE,
+        IMPLEMENTS_RE,
+        has_checkbox_lines,
+        parse_implements,
+        parse_tasks,
+    )
 except ModuleNotFoundError:  # Direct execution adds scripts/, not the repository root, to sys.path.
-    from codeops_plan import BLOCKED_REASON_RE, IMPLEMENTS_RE, parse_implements, parse_tasks
+    from codeops_plan import (
+        BLOCKED_REASON_RE,
+        IMPLEMENTS_RE,
+        has_checkbox_lines,
+        parse_implements,
+        parse_tasks,
+    )
 
 
 TARGET_ID_RE = re.compile(
@@ -231,7 +243,9 @@ def inspect_migration(codeops_root: Path) -> Migration:
                 continue
             execution_text = execution.read_text(encoding="utf-8")
             tasks = parse_tasks(execution_text)
-            if not tasks:
+            # Presence stays checkbox-based (id-agnostic) so legacy id-less
+            # checklists keep migrating; only real tasks are checked further.
+            if not has_checkbox_lines(execution_text):
                 problems.append(f"{execution}: contains no execution tasks")
             for task in tasks:
                 if task.marker == "!" and not BLOCKED_REASON_RE.search(task.text):

package/skills/exec-plan/SKILL.md

@@ -172,6 +172,11 @@ For each task, in order:
    the Master Progress Checklist — see the protocol's dual-format detection) and bump the Progress
    counter / Last Updated stamp as soon as implementation finishes (crash-safe), promote it to
    `[x]` only after its verification passes. A task never shows `[x]` with a failing verify.
+
+   - **Report progress** — when the `codeops_progress` tool is available — at the transition
+     points in [execution-protocol.md](execution-protocol.md) §Progress Reporting; a failed or
+     missing report never blocks any step.
+
 3. **Verify** — run your project's verify command (from the project's AGENTS.md, or detected
    project conventions), output captured per the protocol's **Verify-output capture rule**
    (PASS one-liner; on failure the last 50 log lines + log path). Pass → promote `[~]` → `[x]`;

package/skills/exec-plan/execution-protocol.md

@@ -444,6 +444,35 @@ otherwise still `[~]` — with the progress counter and Last Updated stamp curre
 
 ---
 
+## Progress Reporting (fail-soft)
+
+While a plan executes, the agent may report live progress through the `codeops_progress` tool so
+the sidebar shows the current run. Reporting is an observability aid only: the Markdown execution
+plan remains the single progress source of truth, and a missing tool or a failed call never
+blocks, delays, or changes any verify, commit, or plan update — when the tool is unavailable,
+execution proceeds silently.
+
+Report at these moments; `plan` is the plan folder name, and every call carries `plan` and
+`activity` (absent optional fields are carried by the run state):
+
+| Moment | Report payload |
+| ------ | -------------- |
+| Run start (start point determined) | `{ plan, phase, task, activity: "implementing" }` |
+| Phase start (subsequent phases) | `{ plan, phase, activity: "starting" }` |
+| Task implemented (after the `[~]` mark) | `{ plan, task, activity: "implementing" }` |
+| Verification (before running verify) | `{ plan, activity: "verifying" }` |
+| Task verified (after `[x]` + the progress-bar display) | `{ plan, verified, total, activity: "implementing" }` — counts from that display |
+| Blocker (after the `[!]` mark) | `{ plan, activity: "blocked", detail: "<short reason>" }` |
+| Delegation (around an executor/reviewer dispatch) | `{ plan, activity: "delegating", detail: "<role>" }` |
+| Review step (post-phase quality loop) | `{ plan, activity: "reviewing" }` |
+| Waiting (before pausing for an explicit user decision) | `{ plan, activity: "waiting" }` |
+| Run done (all tasks complete) | `{ plan, activity: "done" }` |
+
+One call per listed moment, no other calls (noise control). A reporting failure or a host without
+the tool never blocks, delays, or changes any step.
+
+---
+
 ## Step 3: Session Wrap-Up
 
 1. Complete the current task before stopping.

package/plugin/temp-lifecycle.spec.test.mjs

@@ -1,55 +0,0 @@
-/**
- * Specification tests for the scratch-directory lifecycle.
- *
- * The plugin exports one scratch directory (`$CODEOPS_TMPDIR`) per plugin
- * runtime, and every session served by that runtime shares it. Deleting the
- * directory when a plugin instance unloads destroys in-flight scratch files of
- * sessions that are still running, so the contract is: the plugin never
- * deletes the shared directory; the age-based sweep reclaims abandoned
- * directories instead.
- *
- * This is the regression oracle for that contract: a failure here means the
- * repository reintroduced early deletion, never that the test is wrong.
- *
- * @module temp-lifecycle.spec.test
- */
-
-import assert from "node:assert/strict"
-import { readFileSync } from "node:fs"
-import { dirname, join, resolve } from "node:path"
-import { describe, it } from "node:test"
-import { fileURLToPath } from "node:url"
-
-const ROOT = resolve(dirname(fileURLToPath(import.meta.url)), "..")
-
-/**
- * Read a repository file as UTF-8 text.
- * @param {string} relativePath - Path relative to the repository root.
- * @returns {string} The file contents.
- */
-function read(relativePath) {
-  return readFileSync(join(ROOT, relativePath), "utf8")
-}
-
-describe("scratch-directory lifecycle", () => {
-  it("keeps the plugin free of scratch-directory deletion", () => {
-    const plugin = read("plugin/index.ts")
-    assert.doesNotMatch(plugin, /removeSessionTmpDir/)
-    assert.match(plugin, /cleanStaleTmpDirs\(\)/)
-    assert.match(plugin, /controller\.abort\(\)/)
-  })
-
-  it("keeps the temp hygiene module free of exact-directory deletion", () => {
-    assert.doesNotMatch(read("bin/lib/tmp-hygiene.mjs"), /removeSessionTmpDir/)
-    assert.doesNotMatch(read("bin/lib/tmp-hygiene.d.mts"), /removeSessionTmpDir/)
-  })
-
-  it("documents the age-based sweep as the automatic cleanup", () => {
-    const hygiene = read("_shared/workspace-hygiene.md")
-    assert.doesNotMatch(hygiene, /removes that runtime's\s+directory when the plugin unloads/i)
-    assert.match(hygiene, /seven days/i)
-    const readme = read("README.md")
-    assert.doesNotMatch(readme, /removes the\s+directory when it unloads/i)
-    assert.match(readme, /sweeps\s+directories abandoned/i)
-  })
-})