@sjawhar/pi-legion 10.2.0 → 10.2.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
|
@@ -16148,7 +16148,7 @@ function onEnvoyRoleRegained(listener) {
|
|
|
16148
16148
|
// package.json
|
|
16149
16149
|
var package_default = {
|
|
16150
16150
|
name: "@sjawhar/pi-legion",
|
|
16151
|
-
version: "10.2.
|
|
16151
|
+
version: "10.2.2",
|
|
16152
16152
|
license: "Apache-2.0",
|
|
16153
16153
|
type: "module",
|
|
16154
16154
|
omp: {
|
|
@@ -69,6 +69,12 @@ Do not start retro before step 2 or skip it because the change seems mechanical;
|
|
|
69
69
|
|
|
70
70
|
## Durable outputs
|
|
71
71
|
|
|
72
|
+
Before your turn's first commit, run `jj -R "$LEGION_WORKSPACE" log -r @ --no-graph -T 'empty'`.
|
|
73
|
+
When it prints `true`, run `jj -R "$LEGION_WORKSPACE" new @-`; otherwise `@` holds only your own
|
|
74
|
+
resumed edits, so run `jj -R "$LEGION_WORKSPACE" new`. Never `jj describe` or `jj split` the
|
|
75
|
+
working-copy commit you found at the turn's start; `skill://legion-worker` explains the
|
|
76
|
+
shared-workspace authorship rule.
|
|
77
|
+
|
|
72
78
|
Write the integrated learning as one or more discoverable documents under `docs/solutions/`.
|
|
73
79
|
Organize by reusable topic rather than by pull request, but never edit a document you did not
|
|
74
80
|
write this retro — not its frontmatter, not its body. Two trees' retros can land within the same
|
|
@@ -158,25 +158,25 @@ An issue that renames a repository, package, or URL across the codebase also fol
|
|
|
158
158
|
`skill://legion-worker/references/systematic-rename.md`; one that deletes code, a command, or
|
|
159
159
|
documentation follows `skill://legion-worker/references/cleanup-deletion.md`.
|
|
160
160
|
|
|
161
|
-
Create
|
|
162
|
-
`jj -R "$LEGION_WORKSPACE" split -m "<message>" <paths…>`, each holding only your phase's paths.
|
|
161
|
+
Create reviewable commits with `jj -R "$LEGION_WORKSPACE" split -m "<message>" <paths…>`, each holding only your phase's paths.
|
|
163
162
|
|
|
164
163
|
Commit attribution is automatic: the extension exports a `JJ_CONFIG` overlay when your
|
|
165
164
|
session starts, so every jj commit you make carries an `Omp-Session: <this-session-id>`
|
|
166
165
|
trailer with no action from you. Do not add attribution trailers by hand.
|
|
167
166
|
|
|
168
|
-
Your pane's environment already supplies your phase's author and committer identity
|
|
169
|
-
|
|
170
|
-
|
|
171
|
-
|
|
172
|
-
|
|
173
|
-
|
|
174
|
-
|
|
175
|
-
|
|
176
|
-
|
|
177
|
-
|
|
178
|
-
|
|
179
|
-
|
|
167
|
+
Your pane's environment already supplies your phase's author and committer identity (`JJ_USER`/`JJ_EMAIL` and the Git author/committer variables, set by the daemon when it opened the pane).
|
|
168
|
+
Before your turn's first commit, run `jj -R "$LEGION_WORKSPACE" log -r @ --no-graph -T 'empty'`.
|
|
169
|
+
If it prints `true`, run `jj -R "$LEGION_WORKSPACE" new @-`; if it prints `false`, `@` holds only
|
|
170
|
+
your own resumed edits, so run `jj -R "$LEGION_WORKSPACE" new`. Never `jj describe` or `jj split`
|
|
171
|
+
the working-copy commit you found at the start of your turn: it can carry the previous role's
|
|
172
|
+
author, and those commands keep it. Never set or override `user.name`/`user.email`
|
|
173
|
+
in any jj or Git scope — not `jj config set`, not `--config`, not `git config`: `--config` outranks
|
|
174
|
+
the pane environment and would put the wrong App back on your commits, and the repository-scoped jj
|
|
175
|
+
config is one file shared by every issue workspace of the clone. Legion has two GitHub Apps, not
|
|
176
|
+
one per role: your role's App is the **implement** App if you are the implementer or the merger,
|
|
177
|
+
and the **review** App if you are the planner, tester, reviewer, or an architect (in Legion's own
|
|
178
|
+
deployment, `legion-implementer[bot]` and `legion-reviewer[bot]`). A planner's commits authored
|
|
179
|
+
by the review App are right. Before a push, check
|
|
180
180
|
`jj -R "$LEGION_WORKSPACE" log -r 'main@origin..@' -T 'author.email() ++ " | " ++ committer.email() ++ " " ++ description.first_line() ++ "\n"'`
|
|
181
181
|
shows your role's App in both columns **on every commit you made** — not on the whole list:
|
|
182
182
|
earlier phases' commits are legitimately authored by their own role's App. Their *committer* is
|