opencode-thrum-pro 0.1.14 → 0.1.15
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/assets/skills/coordinator-interacting-with-the-owner/SKILL.md +1 -1
- package/assets/skills/coordinator-merging-code/SKILL.md +2 -2
- package/assets/skills/implementer-receiving-dispatch/SKILL.md +3 -3
- package/assets/skills/implementer-status-and-handoff/SKILL.md +4 -4
- package/assets/skills/persistent-watcher-archetype/resources/roster-watch.sh +2 -2
- package/assets/skills/verify-against-plan/SKILL.md +1 -1
- package/package.json +1 -1
|
@@ -131,7 +131,7 @@ a rule gets obeyed.** Same situation, both ways.
|
|
|
131
131
|
❌ *"<orchestrator> just finished reviewing <implementer>'s branch. The review found two
|
|
132
132
|
minor issues: an unused import in redact.go and a missing test case for the
|
|
133
133
|
empty-string path. Both were fixed and the branch is now clean. I'm going to
|
|
134
|
-
merge it into
|
|
134
|
+
merge it into the shared integration branch now and then check on the next epic..."*
|
|
135
135
|
✅ Nothing to the pane. If the agent needs an ack, `thrum reply <msg_id>
|
|
136
136
|
--stdin <<'EOF'` with `merged, see <merge-target>` as the body — the pane stays
|
|
137
137
|
silent.
|
|
@@ -379,7 +379,7 @@ agreeing (`rev-parse origin/<branch>` and `ls-remote`). **Assert the branch TIP*
|
|
|
379
379
|
agent's memory across restarts. **Never `git add -f`.**
|
|
380
380
|
4. **Delete the merged branch AND drop its queue bundle — one ritual, at merge
|
|
381
381
|
time**, after the push is verified. `feature/*` and `fix/*` are one-shot —
|
|
382
|
-
delete freely. **`agent/*`, `salvage/*`,
|
|
382
|
+
delete freely. **`agent/*`, `salvage/*`, the documentation deployment branch and `release/*` are
|
|
383
383
|
ongoing or archival — do NOT delete.** Then, in the same turn: `thrum queue
|
|
384
384
|
done <bundle>` + `thrum queue drop <bundle>` for the bundle that tracked this
|
|
385
385
|
branch, and `bd close <id>` any beads this merge closed. A merge is not
|
|
@@ -430,7 +430,7 @@ hides staleness.
|
|
|
430
430
|
- `go test … | tail`, or `$?` read after any pipe.
|
|
431
431
|
- Pushing without guarding on the merge's exit status.
|
|
432
432
|
- Declaring a class closed on a count you did not re-derive.
|
|
433
|
-
- Deleting `agent/*`, `salvage/*`,
|
|
433
|
+
- Deleting `agent/*`, `salvage/*`, the documentation deployment branch or `release/*`.
|
|
434
434
|
- Pushing a merge without dropping the queue bundle that tracked it.
|
|
435
435
|
|
|
436
436
|
## See also
|
|
@@ -105,9 +105,9 @@ the `implementer-maintaining-memory` skill — it references the
|
|
|
105
105
|
|
|
106
106
|
Immediately after sending the dispatch ACK, write `agent_status="working"` to
|
|
107
107
|
your local identity file. This is the Pattern D self-write that makes
|
|
108
|
-
`agent_status` carry signal across the fleet
|
|
109
|
-
|
|
110
|
-
|
|
108
|
+
`agent_status` carry signal across the fleet; the coordinator context monitoring
|
|
109
|
+
sweep uses it to flag agents that go tmux-quiet despite claiming `working`
|
|
110
|
+
(STUCK-WORKING classification).
|
|
111
111
|
|
|
112
112
|
```bash
|
|
113
113
|
# Step 1: ACK the dispatch within 2 minutes
|
|
@@ -67,8 +67,8 @@ Report the result as one line:
|
|
|
67
67
|
Branch pushed: <branch-name> @ <sha> (git ls-remote origin refs/heads/<branch-name> confirms)
|
|
68
68
|
```
|
|
69
69
|
|
|
70
|
-
**Never push
|
|
71
|
-
those.
|
|
70
|
+
**Never push protected integration or release branches** — the coordinator
|
|
71
|
+
merges those.
|
|
72
72
|
|
|
73
73
|
Never report DONE with the push status silently omitted — an orchestrator
|
|
74
74
|
that removes a worktree assuming "DONE implies pushed" on a project that
|
|
@@ -121,8 +121,8 @@ When reporting DONE to the coordinator, write `agent_status="idle"` to your
|
|
|
121
121
|
local identity file. This closes the `working→idle` transition opened by the ACK
|
|
122
122
|
template's `set-status working` (sibling skill
|
|
123
123
|
`implementer-receiving-dispatch`). The sweep script reads this field to
|
|
124
|
-
distinguish "agent finished cleanly" from "agent stuck claiming to work"
|
|
125
|
-
|
|
124
|
+
distinguish "agent finished cleanly" from "agent stuck claiming to work";
|
|
125
|
+
the coordinator context monitoring sweep uses it as one signal in its tier ladder.
|
|
126
126
|
|
|
127
127
|
```bash
|
|
128
128
|
# Step 1: Report DONE to coordinator
|
|
@@ -6,8 +6,8 @@
|
|
|
6
6
|
# never-merged personal-branch scripts (`origin/agent/watcher-primary` and
|
|
7
7
|
# `origin/agent/brainstorm-steward`, reconciled against a full byte-level
|
|
8
8
|
# diff account of both) into one canonical, git-tracked script.
|
|
9
|
-
# Neither branch is an ancestor of
|
|
10
|
-
# either design lands in the shared tree.
|
|
9
|
+
# Neither branch is an ancestor of the shared integration branch; this is
|
|
10
|
+
# the first time either design lands in the shared tree.
|
|
11
11
|
#
|
|
12
12
|
# EXECUTION MODEL: single-pass / cron-driven by default (ported from
|
|
13
13
|
# brainstorm-steward, the LATER of the two designs, Sep 10 vs Sep 6). Its
|
|
@@ -99,7 +99,7 @@ clear error — do not proceed with partial inputs.
|
|
|
99
99
|
4. **Authored-against drift check (informational; does not block on missing stamp).**
|
|
100
100
|
Look for a stamp of this form near the top of the plan:
|
|
101
101
|
|
|
102
|
-
Literal example: `**Authored-against:** \`a1b2c3d4e5\` target:
|
|
102
|
+
Literal example: `**Authored-against:** \`a1b2c3d4e5\` target: \`<merge-target>\``
|
|
103
103
|
|
|
104
104
|
Regex: `^\*\*Authored-against:\*\* \`([0-9a-f]+)\` target: \`([^\`]+)\``
|
|
105
105
|
(sha → capture group 1, merge_target → capture group 2)
|