@sjawhar/opencode-legion-envoy 1.49.0 → 1.49.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/dist/src/server.js +1 -0
- package/package.json +1 -1
- package/skills/legion-worker/SKILL.md +16 -3
package/dist/src/server.js
CHANGED
|
@@ -13686,6 +13686,7 @@ var MessageEventPayloadSchema = object({
|
|
|
13686
13686
|
target: string2().nullish(),
|
|
13687
13687
|
in_reply_to: string2().nullish(),
|
|
13688
13688
|
reply_body: string2().optional(),
|
|
13689
|
+
thread_target: string2().optional(),
|
|
13689
13690
|
author: object({ kind: string2(), id: string2() }).optional()
|
|
13690
13691
|
});
|
|
13691
13692
|
var DispatchTargetedResourceIDSchema = uuid2();
|
package/package.json
CHANGED
|
@@ -265,8 +265,8 @@ later phase keeps it current rather than replacing it:
|
|
|
265
265
|
resolved <thread URL>
|
|
266
266
|
left open <thread URL> — newest reply by <login> is not an acceptance
|
|
267
267
|
|
|
268
|
-
**Thermo:**
|
|
269
|
-
<verdict>. (omitted entirely on a docs-only PR — no
|
|
268
|
+
**Thermo:** `ce-simplify-code` once at <head-sha>: <0 applied | applied → new head <sha>>; thermonuclear pair at the final head <sha>:
|
|
269
|
+
<verdict>. (omitted entirely on a docs-only PR — there is no code for either pass, so neither runs)
|
|
270
270
|
|
|
271
271
|
**E2E (implementer):** <surface> — ran `<command or run id>`, observed <result>, at head <sha>.
|
|
272
272
|
Negative control: <deliberately broken input> → <refusal or failure observed>.
|
|
@@ -367,9 +367,22 @@ this proof.
|
|
|
367
367
|
`E2E` line's head to the new SHA with
|
|
368
368
|
`rebase re-check <old-sha> → <new-sha>: fingerprint unchanged, bare gates only`; the
|
|
369
369
|
real-surface verification is not repeated. Different: a full test round.
|
|
370
|
+
- **The implementer runs `ce-simplify-code` once per pull request, after the last review round
|
|
371
|
+
closes and before the reviewer's final pass, when the diff touches runtime code; a docs-only
|
|
372
|
+
diff gets none.** It is scoped to the pull request's own diff, at the head where the last review
|
|
373
|
+
round closed: nothing applied leaves that head final; applied → the applied head is the final
|
|
374
|
+
head: CI runs on it, the pair runs once on it, and the E2E proof re-runs on it for the surface
|
|
375
|
+
the simplify diff touched (Sami, 2026-09-13: test on the real surface before merging, no
|
|
376
|
+
shortcuts — a refactor that "preserves behaviour" is a claim until it is executed). That cost is
|
|
377
|
+
why 0-applied is the expected outcome and a pass that applies is spent sparingly. At the applied
|
|
378
|
+
head the implementer re-cites the `CI` line and re-runs its own proof into `E2E (implementer)`,
|
|
379
|
+
and the tester re-runs its proof for the touched surface into `E2E (tester)`, before the
|
|
380
|
+
reviewer's final pass. Simplify is the last code change; the pair is the last review. Record it
|
|
381
|
+
in the `Thermo` line.
|
|
370
382
|
- The reviewer verifies the `CI`, `Threads`, and `E2E` facts against GitHub directly —
|
|
371
383
|
never from a handoff — then runs `task(agent="thermonuclear-deep-review")` and
|
|
372
|
-
`task(agent="thermonuclear-code-quality")` once at that head
|
|
384
|
+
`task(agent="thermonuclear-code-quality")` once at that head — the head the implementer's
|
|
385
|
+
simplify pass left final — and records the verdict.
|
|
373
386
|
Approval is refused while either `E2E (implementer)` or `E2E (tester)` is missing: `REQUEST_CHANGES` naming the missing line.
|
|
374
387
|
Skip the `Thermo` line entirely on a docs-only PR. Submit **one review per round** —
|
|
375
388
|
`REQUEST_CHANGES` when any correctness finding stands, otherwise `COMMENT` while the head
|