@sjawhar/opencode-legion-envoy 1.49.0 → 1.49.2

package/dist/src/server.js

@@ -13686,6 +13686,7 @@ var MessageEventPayloadSchema = object({
   target: string2().nullish(),
   in_reply_to: string2().nullish(),
   reply_body: string2().optional(),
+  thread_target: string2().optional(),
   author: object({ kind: string2(), id: string2() }).optional()
 });
 var DispatchTargetedResourceIDSchema = uuid2();
@@ -13751,6 +13752,29 @@ var AskFollowerEventPayloadSchema = object({
   session_id: string2().optional(),
   by: object({ kind: string2(), id: string2().optional() }).passthrough().optional()
 });
+// ../contracts/src/dispatch-href.ts
+function itemFromSearch(search) {
+  const params = new URLSearchParams(search);
+  const ask = params.get("ask");
+  const comment = params.get("comment");
+  if (ask === null && comment === null) {
+    return;
+  }
+  if (ask !== null && comment !== null) {
+    return null;
+  }
+  const raw = ask ?? comment ?? "";
+  if (raw === "" || raw.includes("/")) {
+    return null;
+  }
+  let id;
+  try {
+    id = decodeURIComponent(raw);
+  } catch {
+    return null;
+  }
+  return ask === null ? { id, kind: "comment" } : { id, kind: "ask" };
+}
 // ../contracts/src/dispatch-snippet.ts
 var HTML_ENTITIES = [
   ["&lt;", "<"],
@@ -15828,6 +15852,13 @@ function dispatchRefFromUrl(value, serverUrl) {
   if (issuePage) {
     const [, key, page, artifact, ask, comment, message] = issuePage;
     const version = versionOf("v");
+    if (page === "spec" || artifact !== undefined) {
+      const item = itemFromSearch(url.search);
+      if (item === null)
+        return;
+      if (item !== undefined)
+        return `dispatch://${key}/${item.kind}/${item.id}`;
+    }
     if (page === "spec" && version !== "")
       return `dispatch://${key}/artifact/spec${version}`;
     if (page !== undefined)
@@ -15848,13 +15879,10 @@ function dispatchRefFromUrl(value, serverUrl) {
     return;
   const [, project, slug] = documentPage;
   const document = `dispatch://${project}/artifact/${decodeURIComponent(slug ?? "")}${versionOf("version")}`;
-  const ask = url.searchParams.get("ask");
-  if (ask !== null)
-    return `${document}/ask/${ask}`;
-  const comment = url.searchParams.get("comment");
-  if (comment !== null)
-    return `${document}/comment/${comment}`;
-  return document;
+  const item = itemFromSearch(url.search);
+  if (item === null)
+    return;
+  return item === undefined ? document : `${document}/${item.kind}/${item.id}`;
 }
 var refGrammarProblem = "ref must be a valid dispatch:// reference such as dispatch://KEY-1, " + "dispatch://KEY-1/ask/<uuid>, dispatch://KEY-1/comment/<uuid>, " + "dispatch://KEY-1/message/<uuid>, dispatch://KEY-1/artifact/<slug>, or " + "dispatch://PROJECT/artifact/<document-ref> (an artifact id, slug, or filename); " + "a dashboard URL on this Dispatch server is accepted too";
 var ownerRequiredProblem = "issue is required; supply issue or set LEGION_ISSUE";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sjawhar/opencode-legion-envoy",
-  "version": "1.49.0",
+  "version": "1.49.2",
   "type": "module",
   "main": "dist/src/server.js",
   "exports": {

package/skills/legion-worker/SKILL.md

@@ -265,8 +265,8 @@ later phase keeps it current rather than replacing it:
 resolved <thread URL>
 left open <thread URL> — newest reply by <login> is not an acceptance
 
-**Thermo:** thermonuclear-deep-review + thermonuclear-code-quality run once at <head-sha>:
-<verdict>. (omitted entirely on a docs-only PR — no thermo pass runs)
+**Thermo:** `ce-simplify-code` once at <head-sha>: <0 applied | applied → new head <sha>>; thermonuclear pair at the final head <sha>:
+<verdict>. (omitted entirely on a docs-only PR — there is no code for either pass, so neither runs)
 
 **E2E (implementer):** <surface> — ran `<command or run id>`, observed <result>, at head <sha>.
 Negative control: <deliberately broken input> → <refusal or failure observed>.
@@ -367,9 +367,22 @@ this proof.
   `E2E` line's head to the new SHA with
   `rebase re-check <old-sha> → <new-sha>: fingerprint unchanged, bare gates only`; the
   real-surface verification is not repeated. Different: a full test round.
+- **The implementer runs `ce-simplify-code` once per pull request, after the last review round
+  closes and before the reviewer's final pass, when the diff touches runtime code; a docs-only
+  diff gets none.** It is scoped to the pull request's own diff, at the head where the last review
+  round closed: nothing applied leaves that head final; applied → the applied head is the final
+  head: CI runs on it, the pair runs once on it, and the E2E proof re-runs on it for the surface
+  the simplify diff touched (Sami, 2026-09-13: test on the real surface before merging, no
+  shortcuts — a refactor that "preserves behaviour" is a claim until it is executed). That cost is
+  why 0-applied is the expected outcome and a pass that applies is spent sparingly. At the applied
+  head the implementer re-cites the `CI` line and re-runs its own proof into `E2E (implementer)`,
+  and the tester re-runs its proof for the touched surface into `E2E (tester)`, before the
+  reviewer's final pass. Simplify is the last code change; the pair is the last review. Record it
+  in the `Thermo` line.
 - The reviewer verifies the `CI`, `Threads`, and `E2E` facts against GitHub directly —
   never from a handoff — then runs `task(agent="thermonuclear-deep-review")` and
-  `task(agent="thermonuclear-code-quality")` once at that head and records the verdict.
+  `task(agent="thermonuclear-code-quality")` once at that head — the head the implementer's
+  simplify pass left final — and records the verdict.
   Approval is refused while either `E2E (implementer)` or `E2E (tester)` is missing: `REQUEST_CHANGES` naming the missing line.
   Skip the `Thermo` line entirely on a docs-only PR. Submit **one review per round** —
   `REQUEST_CHANGES` when any correctness finding stands, otherwise `COMMENT` while the head