@sjawhar/opencode-legion-envoy 5.6.5 → 5.7.1

package/dist/src/server.js

@@ -15449,6 +15449,32 @@ function renderAdvice(tool, key, advice, opts) {
   }
   return lines;
 }
+function renderSuggestions(suggestions, configUrl) {
+  if (suggestions === undefined)
+    return [];
+  if (suggestions.missing !== undefined) {
+    return [`Related-item search was skipped: ${suggestions.missing}.`];
+  }
+  const lines = [];
+  if (suggestions.related.length > 0) {
+    lines.push("Possibly related, found by search:");
+    for (const item of suggestions.related) {
+      lines.push(`- ${suggestionLabel(item)} \u2192 ${new URL(item.href, configUrl).toString()}`);
+    }
+  }
+  if (suggestions.decision !== undefined) {
+    const item = suggestions.decision;
+    const when = item.answered_at === undefined ? "" : ` on ${item.answered_at}`;
+    lines.push(`A past decision may already answer this: ${suggestionLabel(item)}, answered by ${item.answered_by}${when} \u2192 ${new URL(item.href, configUrl).toString()}`);
+  }
+  return lines;
+}
+function suggestionLabel(item) {
+  if (item.owner.kind === "issue") {
+    return `${item.owner.key} [${item.owner.status}] ${item.owner.title}`;
+  }
+  return `${item.owner.name} (${item.owner.project}/${item.owner.slug})`;
+}
 function documentResultDetails(artifact) {
   return {
     project: artifact.project,
@@ -16626,7 +16652,8 @@ async function executeDispatchTool(input) {
           ...renderAdvice(input.tool, created.key, created.advice, {
             isPrimarySpec: spec !== undefined
           }),
-          ...componentGuidance === undefined ? [] : [componentGuidance]
+          ...componentGuidance === undefined ? [] : [componentGuidance],
+          ...renderSuggestions(created.advice?.suggestions, configUrl)
         ];
         return {
           text: [
@@ -16906,7 +16933,10 @@ async function executeDispatchTool(input) {
       };
       const ask = resolved?.owner.kind === "project" ? await client.artifactAsk(resolved.artifact.id, askInput) : await client.ask(issue2(), askInput);
       const askOwner = ask.issue_key !== null ? issueTopic(ask.issue_key) : resolved === undefined ? issueTopic(issue2()) : documentTopic(resolved.artifact);
-      const adviceLines = renderAdvice(input.tool, askOwner.label, ask.advice, {});
+      const adviceLines = [
+        ...renderAdvice(input.tool, askOwner.label, ask.advice, {}),
+        ...renderSuggestions(ask.advice?.suggestions, configUrl)
+      ];
       return {
         text: [
           `Asked ${ask.id} on ${askOwner.label} (urgency ${ask.urgency}): ${ask.question}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sjawhar/opencode-legion-envoy",
-  "version": "5.6.5",
+  "version": "5.7.1",
   "license": "Apache-2.0",
   "type": "module",
   "main": "dist/src/server.js",

package/skills/dispatch-first/SKILL.md

@@ -27,6 +27,10 @@ parent's children and the issue's `Components:` line show where the rest of that
 
 ## What to do with what you find
 
+Filing an issue or opening an ask also searches for you: the result's `advice.suggestions` names
+likely duplicates or a decision that may already settle it. Treat a hit there the same as one you
+found yourself.
+
 - **The work is already tracked: extend that issue.** Put the finding on it (a comment, a message,
   or an edit to its spec) instead of filing another. File a new issue only when no hit covers the
   work, and cite the nearest one you ruled out (`dispatch://KEY`).

package/skills/legion-architect/SKILL.md

@@ -250,8 +250,10 @@ The daemon keeps this order from its fixed table; you start none of its steps:
 
 1. tester green and review cycles complete;
 2. on a clean review, the reviewer approves the head by SHA, and the daemon moves the issue to
-   `retro`. No role pushes a `.legion/` deletion: the approved head still carries `.legion/`, and
-   the operator removes it from the default branch in a follow-up pull request after the merge;
+   `retro`. No role pushes a `.legion/` deletion: the approved head still carries `.legion/`. The
+   daemon strips whatever `.legion/` main still carries from the next issue's branch before any of
+   its roles start, so that tree's own merge carries the removal onto the default branch; no
+   operator sweep follows;
 3. retro commits its learnings under `docs/solutions/` on top of the approved head; that
    commit does not void the approval and never returns the tree to the tester or reviewer;
 4. the merger verifies the current head is the reviewer-approved head plus only commits that

package/skills/legion-retro/SKILL.md

@@ -19,7 +19,9 @@ retrospective's durable output.
 
 1. Tester green and all code-review cycles finish.
 2. The reviewer approves the head. It still carries `.legion/`: no role removes it before the
-   merge, and the operator removes it from the default branch after the merge.
+   merge. The daemon strips whatever `.legion/` main still carries from the next issue's branch
+   before any of its roles start, so that tree's own merge carries the removal onto the default
+   branch; no operator sweep follows.
 3. Run this retro: commit durable learnings to `docs/solutions/` and post the retro message on
    the Dispatch issue.
    Retro writes **no `.legion` file**, so it never changes the approved head's handoffs.

package/skills/legion-worker/SKILL.md

@@ -397,9 +397,11 @@ pushes nothing.
 
 Do not report phase completion until the write, existence check, handoff commit, and push
 succeed. This is the committed copy the next phase reads after revival. No phase removes
-`.legion/`: the reviewer approves a head that carries it, and the operator removes it from the
-default branch after the merge. Retro and the post-merge production check write no
-`.legion/<phase>.json`, commit no handoff, and report with `handoff_complete` alone (below).
+`.legion/`: the reviewer approves a head that carries it. The daemon strips any `.legion/` still on
+main from the next issue's branch before any of its roles start (dispatch://LEGION-565), so that
+tree's own merge carries the removal onto the default branch; no operator sweep follows. Retro and
+the post-merge production check write no `.legion/<phase>.json`, commit no handoff, and report with
+`handoff_complete` alone (below).
 
 ## Completion: report to the architect, then stay
 

package/skills/legion-worker/references/merge-gate.md

@@ -6,11 +6,13 @@ path it cites is in sjawhar/legion.
 
 The order, in full: the tester's evidence green → the reviewer's approval of the head →
 retro → the merger's READY → the human merge → the implementer's production check. No role
-removes `.legion/` before the merge: the approved head carries it, and the operator removes it
-from the default branch after the merge. After the approval, only retro's `docs/solutions/`
-commit leaves it standing on its own (*Retro*, below). A conflict-forced merge goes back to the
-reviewer for a confirmation or a new round, as the fingerprint decides (*The reviewer*, below,
-and `skill://legion-worker/references/conflicts-and-rewrites.md`), and any other change voids it.
+removes `.legion/` before the merge: the approved head carries it. The daemon strips whatever
+`.legion/` main still carries from the next issue's branch before any of its roles start
+(dispatch://LEGION-565), so that tree's own merge carries the removal onto the default branch; no
+operator sweep follows. After the approval, only retro's `docs/solutions/` commit leaves it
+standing on its own (*Retro*, below). A conflict-forced merge goes back to the reviewer for a
+confirmation or a new round, as the fingerprint decides (*The reviewer*, below, and
+`skill://legion-worker/references/conflicts-and-rewrites.md`), and any other change voids it.
 
 ## The reviewer