@sjawhar/opencode-legion-envoy 4.1.1 → 5.0.0

package/dist/src/server.js

@@ -14394,168 +14394,6 @@ var EnvelopeSchema = exports_external.object({
 });
 var MINTED_DEDUPE_KEY_PATTERN = "^(?:envoy\\.role\\.forward\\.)?(?:publish|agent\\.[^.]+)\\.(?:[0-9a-f]{32}|[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12})$";
 var mintedDedupeKey = new RegExp(MINTED_DEDUPE_KEY_PATTERN);
-// ../contracts/src/handoff-schema.ts
-var HANDOFF_SCHEMA_VERSION = 1;
-var HANDOFF_PHASES = ["architect", "plan", "implement", "test", "review"];
-var PLAN_REVIEW_MAX_ROUNDS = 3;
-var PLAN_REVIEW_VERDICTS = ["approved", "rejected", "failed"];
-var isoTimestamp = exports_external.string().regex(/^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}/);
-var handoffPhase = exports_external.enum(HANDOFF_PHASES);
-var nonEmpty = exports_external.string().trim().min(1);
-var proofSchema = exports_external.object({
-  criterion: nonEmpty,
-  surface: nonEmpty,
-  command: nonEmpty,
-  observed: nonEmpty,
-  headSha: nonEmpty,
-  negativeControl: nonEmpty
-}).passthrough();
-var routingHintsSchema = exports_external.object({
-  skipArchitect: exports_external.boolean().optional(),
-  complexity: exports_external.enum(["trivial", "small", "medium", "large"]).optional(),
-  estimatedImplementers: exports_external.number().optional()
-}).passthrough().optional();
-var baseHandoffSchema = exports_external.object({
-  schemaVersion: exports_external.literal(HANDOFF_SCHEMA_VERSION),
-  phase: handoffPhase,
-  completed: isoTimestamp,
-  learningsInjected: exports_external.array(exports_external.string()).optional(),
-  learningsHelpful: exports_external.array(exports_external.string()).optional()
-}).passthrough();
-var architectSchema = baseHandoffSchema.extend({
-  phase: exports_external.literal("architect"),
-  scope: exports_external.enum(["trivial", "small", "medium", "large"]).optional(),
-  components: exports_external.array(exports_external.string()).optional(),
-  subIssues: exports_external.array(exports_external.string()).optional(),
-  routingHints: routingHintsSchema,
-  concerns: exports_external.array(exports_external.string()).optional()
-});
-var requiredSkillsSchema = exports_external.object({
-  implement: exports_external.array(exports_external.string()).optional(),
-  test: exports_external.array(exports_external.string()).optional(),
-  review: exports_external.array(exports_external.string()).optional()
-}).passthrough().optional();
-var gapAnalysisSchema = exports_external.object({
-  findings: exports_external.array(exports_external.object({ finding: exports_external.string(), answer: exports_external.string() }).passthrough()).optional(),
-  error: exports_external.string().optional()
-}).passthrough().optional();
-var planReviewSchema = exports_external.object({
-  verdict: exports_external.enum(PLAN_REVIEW_VERDICTS),
-  rounds: exports_external.number(),
-  remainingIssues: exports_external.array(exports_external.object({ issue: exports_external.string(), evidence: exports_external.string() }).passthrough()).optional(),
-  error: exports_external.string().optional()
-}).passthrough().optional();
-var planSchema = baseHandoffSchema.extend({
-  phase: exports_external.literal("plan"),
-  taskCount: exports_external.number().optional(),
-  independentTasks: exports_external.number().optional(),
-  routingHints: routingHintsSchema,
-  concerns: exports_external.array(exports_external.string()).optional(),
-  workflowRecommendation: exports_external.string().optional(),
-  requiredSkills: requiredSkillsSchema,
-  gapAnalysis: gapAnalysisSchema,
-  planReview: planReviewSchema
-});
-var implementSchema = baseHandoffSchema.extend({
-  phase: exports_external.literal("implement"),
-  filesChanged: exports_external.array(exports_external.string()).optional(),
-  proof: exports_external.array(proofSchema).min(1),
-  trickyParts: exports_external.array(exports_external.string()).optional(),
-  deviations: exports_external.array(exports_external.string()).optional(),
-  openQuestions: exports_external.array(exports_external.string()).optional(),
-  subPlanningNeeded: exports_external.boolean().optional(),
-  discoveredComplexity: exports_external.array(exports_external.string()).optional(),
-  suggestedSubWorkers: exports_external.number().optional()
-});
-var testSchema = baseHandoffSchema.extend({
-  phase: exports_external.literal("test"),
-  passed: exports_external.number().optional(),
-  failed: exports_external.number().optional(),
-  failures: exports_external.array(exports_external.object({ criterion: exports_external.string(), evidence: exports_external.string() }).passthrough()).optional(),
-  implementerProof: exports_external.object({ verdict: exports_external.enum(["verified", "rejected"]), how: nonEmpty }).passthrough(),
-  proof: exports_external.array(proofSchema).min(1).optional(),
-  documentationFeedback: exports_external.string().optional(),
-  observations: exports_external.array(exports_external.string()).optional()
-}).refine((handoff) => (handoff.failures?.length ?? 0) > 0 || (handoff.failed ?? 0) > 0 || (handoff.proof?.length ?? 0) > 0, {
-  path: ["proof"],
-  message: "a passing test handoff needs the tester's own production-like proof"
-}).refine((handoff) => (handoff.failed ?? 0) === 0 || (handoff.failures?.length ?? 0) > 0, {
-  path: ["failures"],
-  message: "a test handoff that reports failed > 0 records at least one failure"
-}).refine((handoff) => handoff.implementerProof.verdict !== "rejected" || (handoff.failures?.length ?? 0) > 0, {
-  path: ["failures"],
-  message: "a rejected implementer proof is a recorded failure"
-});
-var reviewSchema = baseHandoffSchema.extend({
-  phase: exports_external.literal("review"),
-  critical: exports_external.number().optional(),
-  important: exports_external.number().optional(),
-  minor: exports_external.number().optional(),
-  verdict: exports_external.enum(["approved", "changes_requested"]).optional(),
-  keyFindings: exports_external.array(exports_external.object({ severity: exports_external.string(), file: exports_external.string(), description: exports_external.string() }).passthrough()).optional()
-});
-var nonEmptySkillList = exports_external.array(nonEmpty).min(1);
-var recorded = (shape, whatToRecord) => exports_external.object(shape, {
-  error: (issue2) => issue2.input === undefined ? `missing \u2014 record ${whatToRecord}` : undefined
-}).passthrough();
-var gapAnalysisWriteSchema = recorded({
-  findings: exports_external.array(exports_external.object({ finding: nonEmpty, answer: nonEmpty }).passthrough()).optional(),
-  error: nonEmpty.optional()
-}, "the gap analyst's `findings`, each with how the plan answers it (`[]` when it found none), or its failed call's `error`").refine((analysis) => analysis.findings === undefined !== (analysis.error === undefined), {
-  message: "record exactly one of `findings` or the failed call's `error`"
-});
-var planReviewWriteSchema = recorded({
-  verdict: exports_external.enum(PLAN_REVIEW_VERDICTS),
-  rounds: exports_external.number().int().min(1).max(PLAN_REVIEW_MAX_ROUNDS),
-  remainingIssues: exports_external.array(exports_external.object({ issue: nonEmpty, evidence: nonEmpty }).passthrough()).optional(),
-  error: nonEmpty.optional()
-}, "the plan review's `verdict` and `rounds`, with `remainingIssues` when it was rejected or `error` when a review's call failed").superRefine((review, ctx) => {
-  const remaining = review.remainingIssues?.length ?? 0;
-  if (review.verdict === "rejected" && remaining === 0) {
-    ctx.addIssue({
-      code: "custom",
-      path: ["remainingIssues"],
-      message: "a rejected review records the blocking issues its last round named"
-    });
-  }
-  if (review.verdict === "rejected" && review.rounds < PLAN_REVIEW_MAX_ROUNDS) {
-    ctx.addIssue({
-      code: "custom",
-      path: ["rounds"],
-      message: `a review still rejecting after ${review.rounds} of ${PLAN_REVIEW_MAX_ROUNDS} rounds is revised and reviewed again, not recorded`
-    });
-  }
-  if (review.verdict === "approved" && remaining > 0) {
-    ctx.addIssue({
-      code: "custom",
-      path: ["remainingIssues"],
-      message: "an approved review leaves no blocking issue standing"
-    });
-  }
-  if (review.verdict === "failed" !== (review.error !== undefined)) {
-    ctx.addIssue({
-      code: "custom",
-      path: ["error"],
-      message: "a failed review records its call's error, and only a failed review does"
-    });
-  }
-});
-var planWriteSchema = planSchema.extend({
-  requiredSkills: exports_external.object({
-    implement: nonEmptySkillList,
-    test: nonEmptySkillList,
-    review: nonEmptySkillList
-  }).passthrough(),
-  gapAnalysis: gapAnalysisWriteSchema,
-  planReview: planReviewWriteSchema
-});
-var phaseHandoffSchema = exports_external.discriminatedUnion("phase", [
-  architectSchema,
-  planSchema,
-  implementSchema,
-  testSchema,
-  reviewSchema
-]);
 // ../contracts/src/subject.ts
 var AGENT_TOPIC_PREFIX = "notifications.agent.";
 var DISPATCH_TOPIC_PREFIX = "notifications.dispatch.";
@@ -14570,283 +14408,6 @@ function dispatchDocumentSubject(project, slug, type) {
 function agentSubject(session) {
   return `${AGENT_TOPIC_PREFIX}${session}`;
 }
-
-// ../contracts/src/legion-roles.ts
-var LEGION_ROLES = [
-  "architect",
-  "planner",
-  "implementer",
-  "tester",
-  "reviewer",
-  "merger"
-];
-
-// ../contracts/src/legion-daemon-api.ts
-var nonEmptyString = exports_external.string().min(1);
-var appLogin = exports_external.string().regex(/^[^[\]]+\[bot\]$/);
-var legionRole = exports_external.enum(LEGION_ROLES);
-var requiredUnknown = exports_external.unknown().refine((value) => value !== undefined, {
-  message: "Required"
-});
-var architectCapability = exports_external.strictObject({
-  tree: nonEmptyString,
-  sessionId: nonEmptyString,
-  secret: nonEmptyString
-});
-var controllerIssue = exports_external.strictObject({
-  secret: nonEmptyString,
-  issue: nonEmptyString
-});
-var TREE_STATUSES = ["queued", "active", "lingering", "dead", "launch-failed", "closed"];
-var stateTmuxLocator = exports_external.strictObject({
-  runtime: exports_external.literal("tmux"),
-  tmuxSession: nonEmptyString,
-  tmuxWindowId: nonEmptyString,
-  tmuxPaneId: nonEmptyString.optional()
-});
-var stateK8sLocator = exports_external.strictObject({
-  runtime: exports_external.literal("kubernetes"),
-  namespace: nonEmptyString,
-  podName: nonEmptyString,
-  podUid: nonEmptyString,
-  pvcName: nonEmptyString
-});
-var stateLocator = exports_external.discriminatedUnion("runtime", [stateTmuxLocator, stateK8sLocator]);
-var stateTreeLocator = exports_external.discriminatedUnion("runtime", [
-  stateTmuxLocator.extend({ ompSessionFile: nonEmptyString.optional() }),
-  stateK8sLocator.extend({ ompSessionFile: nonEmptyString.optional() })
-]);
-var stateExternalControllerLocator = exports_external.strictObject({
-  runtime: exports_external.literal("kubernetes"),
-  external: exports_external.literal(true),
-  sessionId: nonEmptyString,
-  registeredAt: exports_external.number().int().nonnegative()
-});
-var stateIssue = exports_external.strictObject({
-  key: nonEmptyString,
-  title: exports_external.string(),
-  status: exports_external.enum(ISSUE_STATUSES).optional(),
-  children: exports_external.array(nonEmptyString),
-  parent: nonEmptyString.optional(),
-  lastAppliedSeq: exports_external.number().int().nonnegative().optional()
-});
-var stateWorkspaceLost = exports_external.strictObject({
-  at: nonEmptyString,
-  generation: exports_external.number().int().nonnegative(),
-  fromRef: nonEmptyString,
-  previousSessionId: nonEmptyString.optional()
-});
-var stateTree = exports_external.strictObject({
-  status: exports_external.enum(TREE_STATUSES),
-  generation: exports_external.number().int().nonnegative(),
-  launchFailures: exports_external.number().int().nonnegative(),
-  readyConfirmedAt: exports_external.number().optional(),
-  locator: stateTreeLocator.optional(),
-  workspaceLost: stateWorkspaceLost.optional()
-});
-var stateGate = exports_external.strictObject({
-  artifactId: nonEmptyString,
-  latestVersion: exports_external.number().int().positive(),
-  approvedVersion: exports_external.number().int().positive().optional()
-});
-var stateRole = exports_external.strictObject({
-  role: nonEmptyString,
-  issue: nonEmptyString.optional(),
-  generation: exports_external.number().int().nonnegative().optional(),
-  sessionId: nonEmptyString.optional(),
-  readyConfirmedAt: exports_external.number().optional(),
-  launchFailures: exports_external.number().int().nonnegative().optional(),
-  locator: stateLocator.optional(),
-  workspaceLost: stateWorkspaceLost.optional()
-});
-var stateQueuedWorkerIdentity = {
-  roleToken: nonEmptyString,
-  issue: nonEmptyString,
-  role: nonEmptyString
-};
-var stateQueuedWorker = exports_external.union([
-  exports_external.strictObject(stateQueuedWorkerIdentity),
-  exports_external.strictObject({
-    ...stateQueuedWorkerIdentity,
-    kind: exports_external.enum(["assignment", "catchup"]),
-    queuedAt: nonEmptyString
-  })
-]);
-var LegionDaemonApi = {
-  State: {
-    response: exports_external.strictObject({
-      project: nonEmptyString,
-      version: exports_external.number().int(),
-      issues: exports_external.record(exports_external.string(), stateIssue),
-      trees: exports_external.record(exports_external.string(), stateTree),
-      admission: exports_external.strictObject({
-        cap: exports_external.number().int().nonnegative(),
-        active: exports_external.array(nonEmptyString),
-        queue: exports_external.array(nonEmptyString)
-      }),
-      gates: exports_external.record(exports_external.string(), stateGate),
-      controllerLocator: exports_external.union([stateTreeLocator, stateExternalControllerLocator]).optional(),
-      roles: exports_external.record(exports_external.string(), stateRole),
-      controllerPendingNotices: exports_external.number().int().nonnegative(),
-      pendingStatusWrites: exports_external.array(nonEmptyString),
-      workerAdmission: exports_external.strictObject({ queue: exports_external.array(stateQueuedWorker) })
-    })
-  },
-  ControllerReady: {
-    request: exports_external.strictObject({
-      secret: nonEmptyString,
-      sessionId: nonEmptyString,
-      ompSessionFile: nonEmptyString.optional(),
-      pluginVersion: nonEmptyString
-    }),
-    response: exports_external.object({})
-  },
-  ControllerSecret: {
-    request: exports_external.strictObject({}),
-    response: exports_external.object({ secret: nonEmptyString })
-  },
-  ProcessStarted: {
-    request: exports_external.strictObject({
-      tree: nonEmptyString,
-      generation: exports_external.number().int(),
-      rootSessionId: nonEmptyString,
-      agentId: nonEmptyString,
-      bootToken: nonEmptyString,
-      ompSessionFile: nonEmptyString,
-      pluginVersion: nonEmptyString
-    }),
-    response: exports_external.object({
-      roleTokens: exports_external.record(exports_external.string(), exports_external.string()),
-      controlSubject: nonEmptyString,
-      gates: exports_external.object({
-        design: exports_external.enum(["root-issues", "off"])
-      }).optional(),
-      secret: nonEmptyString
-    })
-  },
-  ProcessReady: {
-    request: architectCapability.extend({ generation: exports_external.number().int() }),
-    response: exports_external.object({})
-  },
-  ProcessExit: {
-    request: architectCapability.extend({ generation: exports_external.number().int() }),
-    response: exports_external.object({})
-  },
-  WaveRelease: {
-    request: architectCapability.extend({ issues: exports_external.array(nonEmptyString).optional() }),
-    response: exports_external.object({ released: exports_external.array(nonEmptyString) })
-  },
-  Escalate: {
-    request: architectCapability.extend({
-      kind: exports_external.enum(["re-file", "capacity", "cross-tree"]),
-      context: requiredUnknown
-    }),
-    response: exports_external.object({})
-  },
-  ProvisioningCredential: {
-    request: architectCapability.extend({ issue: nonEmptyString }),
-    response: exports_external.object({ token: nonEmptyString })
-  },
-  WorkerStarted: {
-    request: exports_external.strictObject({
-      tree: nonEmptyString,
-      issue: nonEmptyString,
-      role: legionRole,
-      bootToken: nonEmptyString,
-      sessionId: nonEmptyString,
-      agentId: nonEmptyString,
-      ompSessionFile: nonEmptyString,
-      pluginVersion: nonEmptyString
-    }),
-    response: exports_external.object({
-      roleToken: nonEmptyString,
-      secret: nonEmptyString,
-      gitName: nonEmptyString,
-      gitEmail: nonEmptyString
-    })
-  },
-  WorkerReady: {
-    request: exports_external.strictObject({
-      tree: nonEmptyString,
-      issue: nonEmptyString,
-      role: legionRole,
-      sessionId: nonEmptyString,
-      generation: exports_external.number().int().nonnegative(),
-      secret: nonEmptyString
-    }),
-    response: exports_external.object({})
-  },
-  PhaseComplete: {
-    request: exports_external.strictObject({
-      grantId: nonEmptyString,
-      summary: nonEmptyString
-    }),
-    response: exports_external.object({})
-  },
-  SpawnWorker: {
-    request: architectCapability.extend({
-      issue: nonEmptyString,
-      role: legionRole,
-      task: nonEmptyString,
-      requestId: exports_external.uuid()
-    }),
-    response: exports_external.object({
-      status: exports_external.enum(["spawned", "resumed", "queued"]),
-      roleToken: nonEmptyString
-    })
-  },
-  WorkerSession: {
-    request: exports_external.strictObject({
-      sessionId: nonEmptyString,
-      recoveryToken: nonEmptyString
-    }),
-    response: exports_external.object({
-      tree: nonEmptyString,
-      issue: nonEmptyString,
-      role: legionRole,
-      secret: nonEmptyString
-    })
-  },
-  IssueStatus: {
-    request: controllerIssue.extend({
-      status: exports_external.enum(ISSUE_STATUSES),
-      tree: nonEmptyString.optional(),
-      sessionId: nonEmptyString.optional()
-    }),
-    response: exports_external.object({})
-  },
-  GatesRegister: {
-    request: architectCapability.extend({
-      issue: nonEmptyString,
-      artifactId: exports_external.uuid(),
-      version: exports_external.number().int().positive()
-    }),
-    response: exports_external.object({})
-  },
-  Grant: {
-    request: exports_external.union([
-      exports_external.strictObject({
-        sessionId: nonEmptyString,
-        secret: nonEmptyString,
-        tree: nonEmptyString,
-        issue: nonEmptyString
-      }),
-      exports_external.strictObject({ sessionId: nonEmptyString, secret: nonEmptyString })
-    ]),
-    response: exports_external.object({ grantId: nonEmptyString, expiresAt: nonEmptyString })
-  },
-  GitHubToken: {
-    request: exports_external.strictObject({ grantId: nonEmptyString }),
-    response: exports_external.object({
-      token: nonEmptyString,
-      appLogin: exports_external.string().endsWith("[bot]"),
-      legionAppLogins: exports_external.object({ implement: appLogin, review: appLogin }).optional()
-    })
-  },
-  GitCredential: {
-    request: exports_external.strictObject({ grantId: nonEmptyString })
-  }
-};
 // ../contracts/src/repo.ts
 function canonicalRepo(owner, repo) {
   return `${owner.trim().toLowerCase()}/${repo.trim().toLowerCase().replace(/\.git$/, "")}`;

package/package.json

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

package/skills/legion-worker/SKILL.md

@@ -125,12 +125,10 @@ committed predecessor handoffs in lifecycle order from `$LEGION_WORKSPACE/.legio
 4. `test.json`
 5. `review.json`
 
-Read only files that precede the assigned phase. Every handoff is validated when it is read:
-`validatePhaseHandoff` (`packages/contracts/src/handoff-schema.ts`) checks the
-file, and the ledger (`packages/daemon/src/handoff/ledger.ts`) treats a file that
-fails validation as missing.
-Undeclared fields pass validation untouched and reach the next worker; a declared field of the
-wrong type fails the whole file, so the `legion` tool's `handoff_read` returns null for that phase.
+Read only files that precede the assigned phase. Each was held to its phase's rules when it was
+written (`handoff_write`, in the completion gate below): fields the phase does not declare passed
+untouched and reach the next worker. The `legion` tool's `handoff_read` returns each file as it
+stands in the workspace.
 Write the phase-specific fields the next phase and the architect need, consistent with what
 predecessor phases already wrote. The durable copy lives in
 `$LEGION_WORKSPACE/.legion/<phase>.json`. If a committed handoff conflicts with memory or a prior

package/skills/legion-worker/references/review-threads.md

@@ -13,7 +13,8 @@ reviewer on every re-review, the merger before READY. Every path it cites is in
   `Accepted:` — the opener's own follow-up included — leaves the thread open, because resolution
   considers only the newest comment). The review App can reply on a thread but cannot resolve it:
   GitHub grants resolving a review thread to the pull request's author, and the implementer opens
-  every Legion pull request (`packages/daemon/src/daemon/AGENTS.md`, GitHub Apps).
+  every Legion pull request (`docs/site/src/content/docs/legion/running-legion.md`, "The two
+  GitHub Apps").
   When `LEGION_GRANT_FILE` or `LEGION_GRANT` is set, use `legion threads resolve --pr <number> --repo <owner>/<repo>`.
   When neither is set, add `--gh` to that command, which applies the fallback's rule below through
   your own `gh`; where no `legion` command is installed, use `gh api graphql` with the session's