@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.
|
|
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.
|
|
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
|
|