@sjawhar/opencode-legion-envoy 5.7.0 → 5.8.0
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/dist/src/server.js
CHANGED
|
@@ -13792,6 +13792,7 @@ function actorLabel(actor, titles) {
|
|
|
13792
13792
|
var AGENT_STREAM_LIMITS = {
|
|
13793
13793
|
partChars: 16000,
|
|
13794
13794
|
toolChars: 8000,
|
|
13795
|
+
modelChars: 200,
|
|
13795
13796
|
historyMessages: 200,
|
|
13796
13797
|
historyBytes: 512 * 1024,
|
|
13797
13798
|
snapshotIntervalMs: 100
|
package/package.json
CHANGED
|
@@ -250,8 +250,10 @@ The daemon keeps this order from its fixed table; you start none of its steps:
|
|
|
250
250
|
|
|
251
251
|
1. tester green and review cycles complete;
|
|
252
252
|
2. on a clean review, the reviewer approves the head by SHA, and the daemon moves the issue to
|
|
253
|
-
`retro`. No role pushes a `.legion/` deletion: the approved head still carries `.legion
|
|
254
|
-
|
|
253
|
+
`retro`. No role pushes a `.legion/` deletion: the approved head still carries `.legion/`. The
|
|
254
|
+
daemon strips whatever `.legion/` main still carries from the next issue's branch before any of
|
|
255
|
+
its roles start, so that tree's own merge carries the removal onto the default branch; no
|
|
256
|
+
operator sweep follows;
|
|
255
257
|
3. retro commits its learnings under `docs/solutions/` on top of the approved head; that
|
|
256
258
|
commit does not void the approval and never returns the tree to the tester or reviewer;
|
|
257
259
|
4. the merger verifies the current head is the reviewer-approved head plus only commits that
|
|
@@ -19,7 +19,9 @@ retrospective's durable output.
|
|
|
19
19
|
|
|
20
20
|
1. Tester green and all code-review cycles finish.
|
|
21
21
|
2. The reviewer approves the head. It still carries `.legion/`: no role removes it before the
|
|
22
|
-
merge
|
|
22
|
+
merge. The daemon strips whatever `.legion/` main still carries from the next issue's branch
|
|
23
|
+
before any of its roles start, so that tree's own merge carries the removal onto the default
|
|
24
|
+
branch; no operator sweep follows.
|
|
23
25
|
3. Run this retro: commit durable learnings to `docs/solutions/` and post the retro message on
|
|
24
26
|
the Dispatch issue.
|
|
25
27
|
Retro writes **no `.legion` file**, so it never changes the approved head's handoffs.
|
|
@@ -397,9 +397,11 @@ pushes nothing.
|
|
|
397
397
|
|
|
398
398
|
Do not report phase completion until the write, existence check, handoff commit, and push
|
|
399
399
|
succeed. This is the committed copy the next phase reads after revival. No phase removes
|
|
400
|
-
`.legion/`: the reviewer approves a head that carries it
|
|
401
|
-
|
|
402
|
-
|
|
400
|
+
`.legion/`: the reviewer approves a head that carries it. The daemon strips any `.legion/` still on
|
|
401
|
+
main from the next issue's branch before any of its roles start (dispatch://LEGION-565), so that
|
|
402
|
+
tree's own merge carries the removal onto the default branch; no operator sweep follows. Retro and
|
|
403
|
+
the post-merge production check write no `.legion/<phase>.json`, commit no handoff, and report with
|
|
404
|
+
`handoff_complete` alone (below).
|
|
403
405
|
|
|
404
406
|
## Completion: report to the architect, then stay
|
|
405
407
|
|
|
@@ -6,11 +6,13 @@ path it cites is in sjawhar/legion.
|
|
|
6
6
|
|
|
7
7
|
The order, in full: the tester's evidence green → the reviewer's approval of the head →
|
|
8
8
|
retro → the merger's READY → the human merge → the implementer's production check. No role
|
|
9
|
-
removes `.legion/` before the merge: the approved head carries it
|
|
10
|
-
|
|
11
|
-
|
|
12
|
-
|
|
13
|
-
|
|
9
|
+
removes `.legion/` before the merge: the approved head carries it. The daemon strips whatever
|
|
10
|
+
`.legion/` main still carries from the next issue's branch before any of its roles start
|
|
11
|
+
(dispatch://LEGION-565), so that tree's own merge carries the removal onto the default branch; no
|
|
12
|
+
operator sweep follows. After the approval, only retro's `docs/solutions/` commit leaves it
|
|
13
|
+
standing on its own (*Retro*, below). A conflict-forced merge goes back to the reviewer for a
|
|
14
|
+
confirmation or a new round, as the fingerprint decides (*The reviewer*, below, and
|
|
15
|
+
`skill://legion-worker/references/conflicts-and-rewrites.md`), and any other change voids it.
|
|
14
16
|
|
|
15
17
|
## The reviewer
|
|
16
18
|
|