@sjawhar/pi-legion-envoy 5.17.7 → 5.18.0
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 +1 -1
- package/dist/skills/legion-worker/SKILL.md +21 -2
- package/package.json +1 -1
package/dist/legion.js
CHANGED
|
@@ -33478,7 +33478,7 @@ import { logger } from "@oh-my-pi/pi-utils";
|
|
|
33478
33478
|
// package.json
|
|
33479
33479
|
var package_default = {
|
|
33480
33480
|
name: "@sjawhar/pi-legion-envoy",
|
|
33481
|
-
version: "5.
|
|
33481
|
+
version: "5.18.0",
|
|
33482
33482
|
type: "module",
|
|
33483
33483
|
omp: {
|
|
33484
33484
|
extensions: [
|
|
@@ -628,12 +628,31 @@ cd -- "$LEGION_WORKSPACE" && \
|
|
|
628
628
|
**Every role pushes its own commits.** After the handoff commit — and, for the tester, the red
|
|
629
629
|
tests it wrote — advance the issue bookmark and push it with the provisioned credential helper,
|
|
630
630
|
which authenticates as your role's App (`appRoleForLegionRole` in
|
|
631
|
-
`packages/daemon/src/daemon/github-apps.ts`).
|
|
631
|
+
`packages/daemon/src/daemon/github-apps.ts`).
|
|
632
|
+
|
|
633
|
+
**Under the Go daemon, every push is `legion push`,** run from bash in your workspace in place of
|
|
634
|
+
the commands below. It runs this same procedure on `@-`: the ancestry check, against the remote
|
|
635
|
+
branch or the tip you recorded before rewriting pushed commits (below), then the bookmark and the
|
|
636
|
+
push. It also decides whether the push skips CI. A push skips CI only when none of its commits
|
|
637
|
+
touches anything but handoffs whose phase guarantees a later push: the planner's
|
|
638
|
+
`.legion/plan.json`, the tester's `.legion/test.json`, and a reviewer's `.legion/review.json` whose
|
|
639
|
+
`verdict` is `"changes_requested"`. Its head then ends with GitHub's `skip-checks: true` trailer,
|
|
640
|
+
and the Go daemon carries the code head's verdict to it. Every other push runs CI in full. Never
|
|
641
|
+
add or remove that trailer yourself, never write one of GitHub's bracket keywords (`[skip ci]`,
|
|
642
|
+
`[ci skip]`, `[no ci]`, `[skip actions]`, `[actions skip]`) into a commit message, and never push
|
|
643
|
+
the issue branch with the commands below under the Go daemon: a hand-run push of a handoff that
|
|
644
|
+
could skip CI runs it in full, and a hand-added trailer or keyword on any other push skips CI on a
|
|
645
|
+
head a human may merge. `legion push` refuses a head whose message carries a keyword and pushes
|
|
646
|
+
nothing until you take it out. Under the TypeScript daemon, whose `legion` has no `push` command,
|
|
647
|
+
run the commands below yourself.
|
|
648
|
+
|
|
649
|
+
`-r @-` puts the bookmark on the commit you just
|
|
632
650
|
split off: the working copy left above it has no description, and `jj git push` refuses a
|
|
633
651
|
commit without one. `--allow-backwards` is for that local step alone: after a split the bookmark
|
|
634
652
|
can sit on the undescribed working copy above `@-`. `--bookmark` also publishes the locally
|
|
635
653
|
provisioned bookmark on its first push — a bookmark not yet tracking a remote one is tracked
|
|
636
|
-
automatically. This is the one push procedure
|
|
654
|
+
automatically. This is the one push procedure, run as `legion push` under the Go daemon and by
|
|
655
|
+
hand under the TypeScript daemon; every push of the issue branch uses it:
|
|
637
656
|
|
|
638
657
|
```bash
|
|
639
658
|
cd -- "$LEGION_WORKSPACE" && \
|