@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.0",
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 isolated commits with `jj -R "$LEGION_WORKSPACE" new` and reviewable ones with
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
- (`JJ_USER`/`JJ_EMAIL` and the Git author/committer variables, set by the daemon when it opened
170
- the pane; the daemon also re-authors the workspace's working copy for your role at each
171
- assignment, since `jj split`/`jj describe` keep its author). Never set or override
172
- `user.name`/`user.email` in any jj or Git scope — not `jj config set`, not `--config`, not
173
- `git config`: `--config` outranks the pane environment and would put the wrong App back on your
174
- commits, and the repository-scoped jj config is one file shared by every issue workspace of the
175
- clone. Legion has two GitHub Apps, not one per role: your role's App is the **implement** App
176
- if you are the implementer or the merger, and the **review** App if you are the planner, tester,
177
- reviewer, or an architect (in Legion's own deployment, `legion-implementer[bot]` and
178
- `legion-reviewer[bot]`). A planner's commits authored by the review App are right. Before a push,
179
- check
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
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@sjawhar/pi-legion",
3
- "version": "10.2.0",
3
+ "version": "10.2.2",
4
4
  "license": "Apache-2.0",
5
5
  "type": "module",
6
6
  "omp": {