@sjawhar/pi-legion-envoy 7.8.1 → 7.8.2

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/legion.js CHANGED
@@ -17555,7 +17555,7 @@ function endInjectedUserTurns(sessionID) {
17555
17555
  // package.json
17556
17556
  var package_default = {
17557
17557
  name: "@sjawhar/pi-legion-envoy",
17558
- version: "7.8.1",
17558
+ version: "7.8.2",
17559
17559
  license: "Apache-2.0",
17560
17560
  type: "module",
17561
17561
  omp: {
@@ -339,6 +339,10 @@ line), the full definition of a proof, what the tester verifies, and the simplif
339
339
  The merger writes no handoff and pushes nothing, so this gate does not apply to it
340
340
  (`packages/daemon/internal/prompts/roles/merger.md`).
341
341
 
342
+ A planner picking up after the issue moves back from implementing starts fresh on the remote
343
+ tip — `jj -R "$LEGION_WORKSPACE" new legion/<KEY>@origin` — since the implementer's unpushed
344
+ commits are no longer in the chain (*Every role pushes its own commits*, below).
345
+
342
346
  Write the phase-specific handoff: call the `legion` tool with `op: "handoff_write"`, `phase: "<p>"`,
343
347
  and `data`: a JSON object of the phase-specific fields only. It runs `legion handoff write` in
344
348
  `$LEGION_WORKSPACE` and returns its output.
@@ -379,11 +383,16 @@ cd -- "$LEGION_WORKSPACE" && legion push
379
383
 
380
384
  It pushes `@-` through the provisioned credential helper, which authenticates as your role's App
381
385
  (`appauth.AppRoleFor` in `packages/daemon/internal/appauth/identity.go`). Your system prompt says
382
- which pushes skip CI and which commit-message keywords it refuses. Its ancestry check refuses
386
+ which pushes skip CI and which commit-message keywords it refuses. While the head commit carries
387
+ `skip-checks: true`, a pull-request body edit alone starts no GitHub workflow run, so a check that
388
+ re-judges the body re-runs on that head only after a later push; on a code head the same edit
389
+ re-runs it at once. Its ancestry check refuses
383
390
  unless `@-` descends from `legion/<KEY>@origin` (or the branch is not on GitHub yet): every issue
384
391
  workspace shares one clone, so another role's push moves `legion/<KEY>@origin` here at once, and
385
392
  a push that did not descend from it would move the remote branch sideways onto your commit and
386
- drop theirs.
393
+ drop theirs. A handoff commit never sits on an implementer's unpushed chain: when the issue moves
394
+ back to planning, the implementer's unpushed commits stay off the bookmark until the implementer
395
+ returns, and the planner writes its handoff on the remote tip.
387
396
 
388
397
  Before any rewrite of a commit you already pushed — a `jj squash --into` one, or any other
389
398
  rewrite — read *Rewriting pushed commits* in
@@ -392,7 +401,7 @@ is built on that commit and records the pushed tip `legion push` reads, the only
392
401
  push lets the remote branch move off its current tip.
393
402
 
394
403
  Before the push, check ancestry and identity as above: the chain carries every earlier phase's
395
- commits, and pushing them with yours is expected. A refusal, and a push the remote rejects, is a
404
+ pushed commits, and pushing them with yours is expected. A refusal, and a push the remote rejects, is a
396
405
  report to the architect with the output, never a force-push. The merger makes no commit and
397
406
  pushes nothing.
398
407
 
@@ -129,6 +129,9 @@ and the tester's proof below are both this proof.
129
129
  `skill://legion-worker`): the `Fast-follow:` line holds naming, duplication, or wording cleanup
130
130
  only. A base frozen for others to stack on is never rewritten (*Rewriting pushed commits* in
131
131
  `skill://legion-worker/references/conflicts-and-rewrites.md`); the `Chain` line records it.
132
+ - **GitHub refuses a pull request body over 65,536 characters.** Each verification round links its
133
+ evidence (the run, the comment) rather than inlining it once the body passes about 48,000
134
+ characters; the `Production` line's record always links.
132
135
 
133
136
  ## When no surface reaches the changed path
134
137
 
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@sjawhar/pi-legion-envoy",
3
- "version": "7.8.1",
3
+ "version": "7.8.2",
4
4
  "license": "Apache-2.0",
5
5
  "type": "module",
6
6
  "omp": {