@biffo/cli 0.274.0 → 0.275.1
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/_skeletons/plugin-template/AGENTS.md +36 -0
- package/_skeletons/plugin-template/web-admin/pnpm-lock.yaml +2914 -0
- package/_skeletons/plugin-template/web-admin/src/lib/api-core.test.ts +125 -0
- package/_skeletons/plugin-template/web-admin/src/lib/api-core.ts +37 -1
- package/_skeletons/sibling-template/AGENTS.md +36 -0
- package/package.json +1 -1
|
@@ -117,6 +117,42 @@ it deliberately and say so in a comment; never steal a fresh one.
|
|
|
117
117
|
- **Squash-merge, delete the branch, remove the worktree:**
|
|
118
118
|
`gh pr merge <N> --squash --delete-branch`.
|
|
119
119
|
|
|
120
|
+
### If you were dispatched by another agent, you do NOT wait and you do NOT merge
|
|
121
|
+
|
|
122
|
+
Everything above assumes the session doing the work is also the session that
|
|
123
|
+
will merge it. **A delegated subagent is not that session.** If an orchestrator
|
|
124
|
+
spawned you to do one unit of work, your finished state is:
|
|
125
|
+
|
|
126
|
+
> branch pushed → PR opened → **report back immediately**, with the PR number
|
|
127
|
+
> and what you changed.
|
|
128
|
+
|
|
129
|
+
Do **not** run `wait-for-checks`. Do **not** merge. The orchestrator watches CI
|
|
130
|
+
and merges, and sends you back if CI fails on your change.
|
|
131
|
+
|
|
132
|
+
**Why this is a rule and not a preference.** A subagent has no way to sleep. Told
|
|
133
|
+
to wait for CI, it stops, wakes, re-reads the same pending status, and stops
|
|
134
|
+
again — and every wake is a full context reload that reports nothing new. Measured
|
|
135
|
+
on 2026-08-11: one dispatch tranche produced **25+ wake-ups** returning only
|
|
136
|
+
"still waiting"; a single agent burned ~145k tokens re-reading a queued deploy
|
|
137
|
+
before it was killed. The orchestrator does the same wait for nothing extra: it
|
|
138
|
+
is already awake, and one loop covers every PR in the tranche at once instead of
|
|
139
|
+
N agents each paying a context reload for the same minutes.
|
|
140
|
+
|
|
141
|
+
**The instruction is the cause, so the fix lives here.** Both of those incidents
|
|
142
|
+
happened in sessions whose dispatch briefs ended "get CI green, then squash-merge"
|
|
143
|
+
— written by an orchestrator that _had already recorded the lesson_ and then
|
|
144
|
+
followed §5 above, which says exactly that. §5 is right for the session that owns
|
|
145
|
+
the work and wrong for a delegate, and until now nothing said so.
|
|
146
|
+
|
|
147
|
+
**Keep a delegate alive only when it still has a DECISION to make** — a real test
|
|
148
|
+
failure to diagnose, a design question to answer. "Wait, then merge" is not a
|
|
149
|
+
decision.
|
|
150
|
+
|
|
151
|
+
Orchestrators: end every dispatch brief with the finished state above, and reach
|
|
152
|
+
for `gh pr merge --auto` when the merge needs no judgement — then still verify,
|
|
153
|
+
because `--auto` waits for ever on a check that cannot re-evaluate itself (a
|
|
154
|
+
`Release Guards` closing-keyword failure needs a re-run, not a wait).
|
|
155
|
+
|
|
120
156
|
## 6. Push honestly, and verify the remote has your commit
|
|
121
157
|
|
|
122
158
|
- **Push with the exit status visible:** `git push origin HEAD; echo $?`. A pipe
|