@codyswann/lisa 2.172.0 → 2.173.1
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/package.json +1 -1
- package/plugins/lisa/.claude-plugin/plugin.json +2 -2
- package/plugins/lisa/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa/hooks/enforce-team-first.sh +29 -14
- package/plugins/lisa/skills/repair-intake/SKILL.md +33 -11
- package/plugins/lisa-agy/plugin.json +1 -1
- package/plugins/lisa-agy/skills/repair-intake/SKILL.md +33 -11
- package/plugins/lisa-cdk/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk-agy/plugin.json +1 -1
- package/plugins/lisa-cdk-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-copilot/skills/repair-intake/SKILL.md +33 -11
- package/plugins/lisa-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cursor/skills/repair-intake/SKILL.md +33 -11
- package/plugins/lisa-expo/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-expo/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-expo-agy/plugin.json +1 -1
- package/plugins/lisa-expo-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-expo-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-agy/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs-agy/plugin.json +1 -1
- package/plugins/lisa-nestjs-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw-agy/plugin.json +1 -1
- package/plugins/lisa-openclaw-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser-agy/plugin.json +1 -1
- package/plugins/lisa-phaser-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-rails-agy/plugin.json +1 -1
- package/plugins/lisa-rails-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript-agy/plugin.json +1 -1
- package/plugins/lisa-typescript-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki-agy/plugin.json +1 -1
- package/plugins/lisa-wiki-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/src/base/.claude-plugin/plugin.json +1 -1
- package/plugins/src/base/hooks/enforce-team-first.sh +29 -14
- package/plugins/src/base/skills/repair-intake/SKILL.md +33 -11
package/package.json
CHANGED
|
@@ -91,7 +91,7 @@
|
|
|
91
91
|
"ws": ">=8.20.1"
|
|
92
92
|
},
|
|
93
93
|
"name": "@codyswann/lisa",
|
|
94
|
-
"version": "2.
|
|
94
|
+
"version": "2.173.1",
|
|
95
95
|
"description": "Claude Code governance framework that applies guardrails, guidance, and automated enforcement to projects",
|
|
96
96
|
"main": "dist/index.js",
|
|
97
97
|
"exports": {
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lisa",
|
|
3
|
-
"version": "2.
|
|
3
|
+
"version": "2.173.1",
|
|
4
4
|
"description": "Universal governance — agents, skills, commands, hooks, and rules for all projects",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Cody Swann"
|
|
@@ -64,7 +64,7 @@
|
|
|
64
64
|
]
|
|
65
65
|
},
|
|
66
66
|
{
|
|
67
|
-
"matcher": "TeamCreate",
|
|
67
|
+
"matcher": "TeamCreate|Agent",
|
|
68
68
|
"hooks": [
|
|
69
69
|
{
|
|
70
70
|
"type": "command",
|
|
@@ -9,10 +9,13 @@
|
|
|
9
9
|
# - UserPromptSubmit : detects /lisa:research|plan|implement|intake|debrief in the
|
|
10
10
|
# raw prompt and arms enforcement for the session
|
|
11
11
|
# - PreToolUse : detects the same skills via a `Skill` tool call,
|
|
12
|
-
# arms enforcement, and blocks bypass tool calls
|
|
13
|
-
#
|
|
14
|
-
#
|
|
15
|
-
# team
|
|
12
|
+
# arms enforcement, and blocks bypass tool calls until
|
|
13
|
+
# the team is established — i.e. until a TeamCreate
|
|
14
|
+
# (older Claude Code) or the first Agent spawn
|
|
15
|
+
# (implicit-team model, Claude Code >= 2.1.178) fires
|
|
16
|
+
# - PostToolUse : on a successful Claude TeamCreate OR a successful
|
|
17
|
+
# Agent spawn (implicit team), marks the session as
|
|
18
|
+
# team-established (lifts enforcement)
|
|
16
19
|
# - SubagentStart : marks the new subagent session as a teammate so
|
|
17
20
|
# it is exempt — teammates inherit the lead's team
|
|
18
21
|
# and must never create a second team (double-create
|
|
@@ -85,7 +88,12 @@ case "$HOOK_EVENT" in
|
|
|
85
88
|
|
|
86
89
|
PostToolUse)
|
|
87
90
|
TOOL_NAME=$(printf '%s' "$INPUT" | jq -r '.tool_name // empty' 2>/dev/null || true)
|
|
88
|
-
|
|
91
|
+
# A successful TeamCreate (older Claude Code) OR a successful Agent spawn
|
|
92
|
+
# (Claude Code >= 2.1.178, where TeamCreate/TeamDelete were removed in favor
|
|
93
|
+
# of the implicit-team model — the team forms automatically when the lead
|
|
94
|
+
# spawns its first teammate via the Agent tool) marks the session as
|
|
95
|
+
# team-established and lifts enforcement.
|
|
96
|
+
if [ "$TOOL_NAME" = "TeamCreate" ] || [ "$TOOL_NAME" = "Agent" ]; then
|
|
89
97
|
ERROR=$(printf '%s' "$INPUT" | jq -r '.tool_response.error // empty' 2>/dev/null || true)
|
|
90
98
|
IS_ERROR=$(printf '%s' "$INPUT" | jq -r '.tool_response.is_error // empty' 2>/dev/null || true)
|
|
91
99
|
if [ -z "$ERROR" ] && [ "$IS_ERROR" != "true" ]; then
|
|
@@ -136,9 +144,12 @@ if [ -f "$TEAM_FLAG" ]; then
|
|
|
136
144
|
exit 0
|
|
137
145
|
fi
|
|
138
146
|
|
|
139
|
-
# These
|
|
147
|
+
# These are the path forward; never block them.
|
|
148
|
+
# - ToolSearch / TeamCreate : the older explicit-team path (Claude Code < 2.1.178)
|
|
149
|
+
# - Agent : the implicit-team path (Claude Code >= 2.1.178) —
|
|
150
|
+
# spawning the first teammate IS what forms the team
|
|
140
151
|
case "$TOOL_NAME" in
|
|
141
|
-
ToolSearch|TeamCreate)
|
|
152
|
+
ToolSearch|TeamCreate|Agent)
|
|
142
153
|
exit 0
|
|
143
154
|
;;
|
|
144
155
|
esac
|
|
@@ -163,21 +174,25 @@ fi
|
|
|
163
174
|
ACTIVE_SKILL=$(cat "$SKILL_FLAG" 2>/dev/null || echo "lisa:???")
|
|
164
175
|
cat >&2 <<EOF
|
|
165
176
|
Blocked: this session invoked /${ACTIVE_SKILL}, which is an agent-team flow.
|
|
166
|
-
In Claude, before any other tool call, you must
|
|
177
|
+
In Claude, before any other tool call, you must establish the team by spawning
|
|
178
|
+
your first teammate:
|
|
167
179
|
|
|
168
|
-
|
|
169
|
-
|
|
180
|
+
Call the \`Agent\` tool with an appropriate \`subagent_type\`.
|
|
181
|
+
|
|
182
|
+
The team forms automatically the moment the lead spawns its first teammate —
|
|
183
|
+
this is the implicit-team model on Claude Code >= 2.1.178, where the explicit
|
|
184
|
+
\`TeamCreate\`/\`TeamDelete\` tools were removed. (On older Claude Code the
|
|
185
|
+
\`TeamCreate\` tool path still works and is also accepted.)
|
|
170
186
|
|
|
171
187
|
The current attempt to call \`${TOOL_NAME}\` is a team-bypass path. Reading
|
|
172
188
|
the ticket, exploring the code, fetching context — those are tasks for the
|
|
173
|
-
team you are about to
|
|
174
|
-
exists.
|
|
189
|
+
team you are about to spawn, not for the lead session before the team exists.
|
|
175
190
|
|
|
176
191
|
If you are running Lisa in a non-Claude harness, this Claude enforcement hook
|
|
177
192
|
should not be installed; follow the runtime-aware orchestration preamble in the
|
|
178
193
|
skill instead.
|
|
179
194
|
|
|
180
|
-
Re-read the orchestration preamble in /${ACTIVE_SKILL} and start
|
|
181
|
-
|
|
195
|
+
Re-read the orchestration preamble in /${ACTIVE_SKILL} and start by spawning a
|
|
196
|
+
teammate with the \`Agent\` tool.
|
|
182
197
|
EOF
|
|
183
198
|
exit 2
|
|
@@ -20,10 +20,13 @@ close-out** roles and moves work *unstuck* or *fully closed*:
|
|
|
20
20
|
after its agent returned) — no re-dispatch. A PR that is merely **behind its base** (`BEHIND`, no
|
|
21
21
|
conflict) is **re-synced in place** with `gh pr update-branch` so the already-enabled auto-merge can
|
|
22
22
|
finally land — a clean rebase needs no human, and leaving it stranded is the exact gap that lets an
|
|
23
|
-
auto-merge PR sit unmerged forever.
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
23
|
+
auto-merge PR sit unmerged forever. A **true merge conflict** is first given **one bounded in-place
|
|
24
|
+
re-dispatch** to the build agent — whose `drive-pr-to-merge` fix-mode loop resolves conflicts — because
|
|
25
|
+
a conflict, unlike a failing external check, is fixable by re-running the build; only a conflict that
|
|
26
|
+
survives that single attempt (or that the agent says needs design input) becomes a fix ticket. The
|
|
27
|
+
genuinely non-resolvable blockers — failing checks / unaddressed CodeRabbit or `CHANGES_REQUESTED`
|
|
28
|
+
review / a failed deploy — get a build-ready leaf fix ticket with the item moved to `blocked` (blocked
|
|
29
|
+
by that ticket) instead of blindly re-dispatching the agent, which would just churn against them.
|
|
27
30
|
- **Recoverable blocked** — an item in `blocked` whose blocker may now be gone. The blocker is
|
|
28
31
|
one of three classes, and repair re-checks **all** of them, not just dependencies: (a) an
|
|
29
32
|
`is blocked by` **dependency** has since closed; (b) a **validation / quality-gate self-block** —
|
|
@@ -316,12 +319,16 @@ a PR that cannot merge or a deploy that failed — it just churns.
|
|
|
316
319
|
Re-sync the branch in place so the already-enabled auto-merge can land (see diagnosis step 3).
|
|
317
320
|
Keep the item `claimed`; a later cycle confirms the merge and transitions. Do **not** file a fix
|
|
318
321
|
ticket for a clean rebase.
|
|
319
|
-
- **A
|
|
320
|
-
|
|
321
|
-
|
|
322
|
-
|
|
323
|
-
|
|
324
|
-
|
|
322
|
+
- **A true merge conflict** → **not** an immediate fix ticket. A conflict is fixable by re-running
|
|
323
|
+
the build, so attempt **one** in-place re-dispatch first (the resume sequence below; the vendor
|
|
324
|
+
agent re-enters `drive-pr-to-merge` fix mode, which resolves conflicts). Only a conflict that
|
|
325
|
+
survives that single attempt — the same conflicting head still `CONFLICTING` on a later cycle — or
|
|
326
|
+
that the agent reports needs design input, falls through to the fix-ticket path (diagnosis step 5).
|
|
327
|
+
- **A real external blocker re-running the build cannot fix** (failing checks / `CHANGES_REQUESTED` /
|
|
328
|
+
unaddressed CodeRabbit; or a failed deploy) → **do not dispatch the agent**. File a build-ready leaf
|
|
329
|
+
fix ticket for the blocker, move this item `claimed → blocked` with an `is blocked by` link to that
|
|
330
|
+
ticket, and record it. The existing "Build `blocked` → unblock if cleared" path resumes this item on
|
|
331
|
+
a later cycle once the fix ticket is terminal — a self-healing loop. Skip the resume steps below.
|
|
325
332
|
|
|
326
333
|
If the PR is healthy in-flight and no blocker is found, the work simply died mid-flight — run the **same per-item sequence
|
|
327
334
|
the vendor build-intake runs**, skipping the claim transition (the item is already `claimed`):
|
|
@@ -400,7 +407,9 @@ branch — operate on it the same way.
|
|
|
400
407
|
|
|
401
408
|
- **True merge conflict** — `mergeable = CONFLICTING` or `mergeStateStatus = DIRTY` (overlapping
|
|
402
409
|
changes a plain rebase cannot resolve), or `gh pr update-branch` (step 3) reported a conflict. A
|
|
403
|
-
merely `BEHIND` branch is **not** here — it was re-synced in step 3.
|
|
410
|
+
merely `BEHIND` branch is **not** here — it was re-synced in step 3. Unlike the other classes below, a
|
|
411
|
+
conflict is **resolvable by re-running the build**, so step 5 gives it one in-place re-dispatch before
|
|
412
|
+
filing — see its conflict-first rule.
|
|
404
413
|
- **Failing required checks** — `statusCheckRollup` has a `FAILURE`/`ERROR`/`TIMED_OUT` conclusion,
|
|
405
414
|
or `mergeStateStatus = UNSTABLE`/`BLOCKED` due to checks.
|
|
406
415
|
- **Change requests outstanding** — `reviewDecision = CHANGES_REQUESTED`, or unresolved CodeRabbit
|
|
@@ -416,6 +425,19 @@ recent check/commit activity would already have been caught as `active` by the s
|
|
|
416
425
|
|
|
417
426
|
**5. On a blocker found → file a leaf fix ticket + block the item.**
|
|
418
427
|
|
|
428
|
+
**Conflict-first exception (try to resolve before filing).** A *true merge conflict* — and only a
|
|
429
|
+
conflict, not failing checks, change requests, or a failed deploy — is fixable by re-running the build:
|
|
430
|
+
the vendor agent's `drive-pr-to-merge` fix-mode loop resolves conflicts. So before filing a fix ticket
|
|
431
|
+
for a conflict, give the item **one** in-place re-dispatch: run the resume-in-place sequence (steps 1–3
|
|
432
|
+
of the parent path above), which re-enters `drive-pr-to-merge` in fix mode against the existing PR.
|
|
433
|
+
Bound it to a single attempt per conflicting head — when you re-dispatch, post a `[lisa-repair-intake]
|
|
434
|
+
conflict-resolve-attempt: <item-ref>@<head-sha>` marker keyed on the PR head SHA. On a later cycle, if
|
|
435
|
+
that marker already exists for the **same** head SHA and the PR is still `CONFLICTING`, the attempt
|
|
436
|
+
failed: stop retrying and file the fix ticket below. File immediately (skip the attempt) if the agent
|
|
437
|
+
reports the conflict needs design input. Every other blocker class files the fix ticket with no
|
|
438
|
+
re-dispatch. Honor the backoff window / state fingerprint (Loop prevention) so the re-dispatch is never
|
|
439
|
+
re-issued against an unchanged conflicting head.
|
|
440
|
+
|
|
419
441
|
1. **File one build-ready leaf fix ticket** per distinct blocker via `lisa:tracker-write` (the
|
|
420
442
|
vendor-neutral leaf writer + validation gate; never a vendor `*-write-*` skill directly),
|
|
421
443
|
`issue_type: Bug` for a failing-check/conflict/failed-deploy, `Task` for review-feedback
|
|
@@ -20,10 +20,13 @@ close-out** roles and moves work *unstuck* or *fully closed*:
|
|
|
20
20
|
after its agent returned) — no re-dispatch. A PR that is merely **behind its base** (`BEHIND`, no
|
|
21
21
|
conflict) is **re-synced in place** with `gh pr update-branch` so the already-enabled auto-merge can
|
|
22
22
|
finally land — a clean rebase needs no human, and leaving it stranded is the exact gap that lets an
|
|
23
|
-
auto-merge PR sit unmerged forever.
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
23
|
+
auto-merge PR sit unmerged forever. A **true merge conflict** is first given **one bounded in-place
|
|
24
|
+
re-dispatch** to the build agent — whose `drive-pr-to-merge` fix-mode loop resolves conflicts — because
|
|
25
|
+
a conflict, unlike a failing external check, is fixable by re-running the build; only a conflict that
|
|
26
|
+
survives that single attempt (or that the agent says needs design input) becomes a fix ticket. The
|
|
27
|
+
genuinely non-resolvable blockers — failing checks / unaddressed CodeRabbit or `CHANGES_REQUESTED`
|
|
28
|
+
review / a failed deploy — get a build-ready leaf fix ticket with the item moved to `blocked` (blocked
|
|
29
|
+
by that ticket) instead of blindly re-dispatching the agent, which would just churn against them.
|
|
27
30
|
- **Recoverable blocked** — an item in `blocked` whose blocker may now be gone. The blocker is
|
|
28
31
|
one of three classes, and repair re-checks **all** of them, not just dependencies: (a) an
|
|
29
32
|
`is blocked by` **dependency** has since closed; (b) a **validation / quality-gate self-block** —
|
|
@@ -316,12 +319,16 @@ a PR that cannot merge or a deploy that failed — it just churns.
|
|
|
316
319
|
Re-sync the branch in place so the already-enabled auto-merge can land (see diagnosis step 3).
|
|
317
320
|
Keep the item `claimed`; a later cycle confirms the merge and transitions. Do **not** file a fix
|
|
318
321
|
ticket for a clean rebase.
|
|
319
|
-
- **A
|
|
320
|
-
|
|
321
|
-
|
|
322
|
-
|
|
323
|
-
|
|
324
|
-
|
|
322
|
+
- **A true merge conflict** → **not** an immediate fix ticket. A conflict is fixable by re-running
|
|
323
|
+
the build, so attempt **one** in-place re-dispatch first (the resume sequence below; the vendor
|
|
324
|
+
agent re-enters `drive-pr-to-merge` fix mode, which resolves conflicts). Only a conflict that
|
|
325
|
+
survives that single attempt — the same conflicting head still `CONFLICTING` on a later cycle — or
|
|
326
|
+
that the agent reports needs design input, falls through to the fix-ticket path (diagnosis step 5).
|
|
327
|
+
- **A real external blocker re-running the build cannot fix** (failing checks / `CHANGES_REQUESTED` /
|
|
328
|
+
unaddressed CodeRabbit; or a failed deploy) → **do not dispatch the agent**. File a build-ready leaf
|
|
329
|
+
fix ticket for the blocker, move this item `claimed → blocked` with an `is blocked by` link to that
|
|
330
|
+
ticket, and record it. The existing "Build `blocked` → unblock if cleared" path resumes this item on
|
|
331
|
+
a later cycle once the fix ticket is terminal — a self-healing loop. Skip the resume steps below.
|
|
325
332
|
|
|
326
333
|
If the PR is healthy in-flight and no blocker is found, the work simply died mid-flight — run the **same per-item sequence
|
|
327
334
|
the vendor build-intake runs**, skipping the claim transition (the item is already `claimed`):
|
|
@@ -400,7 +407,9 @@ branch — operate on it the same way.
|
|
|
400
407
|
|
|
401
408
|
- **True merge conflict** — `mergeable = CONFLICTING` or `mergeStateStatus = DIRTY` (overlapping
|
|
402
409
|
changes a plain rebase cannot resolve), or `gh pr update-branch` (step 3) reported a conflict. A
|
|
403
|
-
merely `BEHIND` branch is **not** here — it was re-synced in step 3.
|
|
410
|
+
merely `BEHIND` branch is **not** here — it was re-synced in step 3. Unlike the other classes below, a
|
|
411
|
+
conflict is **resolvable by re-running the build**, so step 5 gives it one in-place re-dispatch before
|
|
412
|
+
filing — see its conflict-first rule.
|
|
404
413
|
- **Failing required checks** — `statusCheckRollup` has a `FAILURE`/`ERROR`/`TIMED_OUT` conclusion,
|
|
405
414
|
or `mergeStateStatus = UNSTABLE`/`BLOCKED` due to checks.
|
|
406
415
|
- **Change requests outstanding** — `reviewDecision = CHANGES_REQUESTED`, or unresolved CodeRabbit
|
|
@@ -416,6 +425,19 @@ recent check/commit activity would already have been caught as `active` by the s
|
|
|
416
425
|
|
|
417
426
|
**5. On a blocker found → file a leaf fix ticket + block the item.**
|
|
418
427
|
|
|
428
|
+
**Conflict-first exception (try to resolve before filing).** A *true merge conflict* — and only a
|
|
429
|
+
conflict, not failing checks, change requests, or a failed deploy — is fixable by re-running the build:
|
|
430
|
+
the vendor agent's `drive-pr-to-merge` fix-mode loop resolves conflicts. So before filing a fix ticket
|
|
431
|
+
for a conflict, give the item **one** in-place re-dispatch: run the resume-in-place sequence (steps 1–3
|
|
432
|
+
of the parent path above), which re-enters `drive-pr-to-merge` in fix mode against the existing PR.
|
|
433
|
+
Bound it to a single attempt per conflicting head — when you re-dispatch, post a `[lisa-repair-intake]
|
|
434
|
+
conflict-resolve-attempt: <item-ref>@<head-sha>` marker keyed on the PR head SHA. On a later cycle, if
|
|
435
|
+
that marker already exists for the **same** head SHA and the PR is still `CONFLICTING`, the attempt
|
|
436
|
+
failed: stop retrying and file the fix ticket below. File immediately (skip the attempt) if the agent
|
|
437
|
+
reports the conflict needs design input. Every other blocker class files the fix ticket with no
|
|
438
|
+
re-dispatch. Honor the backoff window / state fingerprint (Loop prevention) so the re-dispatch is never
|
|
439
|
+
re-issued against an unchanged conflicting head.
|
|
440
|
+
|
|
419
441
|
1. **File one build-ready leaf fix ticket** per distinct blocker via `lisa:tracker-write` (the
|
|
420
442
|
vendor-neutral leaf writer + validation gate; never a vendor `*-write-*` skill directly),
|
|
421
443
|
`issue_type: Bug` for a failing-check/conflict/failed-deploy, `Task` for review-feedback
|
|
@@ -20,10 +20,13 @@ close-out** roles and moves work *unstuck* or *fully closed*:
|
|
|
20
20
|
after its agent returned) — no re-dispatch. A PR that is merely **behind its base** (`BEHIND`, no
|
|
21
21
|
conflict) is **re-synced in place** with `gh pr update-branch` so the already-enabled auto-merge can
|
|
22
22
|
finally land — a clean rebase needs no human, and leaving it stranded is the exact gap that lets an
|
|
23
|
-
auto-merge PR sit unmerged forever.
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
23
|
+
auto-merge PR sit unmerged forever. A **true merge conflict** is first given **one bounded in-place
|
|
24
|
+
re-dispatch** to the build agent — whose `drive-pr-to-merge` fix-mode loop resolves conflicts — because
|
|
25
|
+
a conflict, unlike a failing external check, is fixable by re-running the build; only a conflict that
|
|
26
|
+
survives that single attempt (or that the agent says needs design input) becomes a fix ticket. The
|
|
27
|
+
genuinely non-resolvable blockers — failing checks / unaddressed CodeRabbit or `CHANGES_REQUESTED`
|
|
28
|
+
review / a failed deploy — get a build-ready leaf fix ticket with the item moved to `blocked` (blocked
|
|
29
|
+
by that ticket) instead of blindly re-dispatching the agent, which would just churn against them.
|
|
27
30
|
- **Recoverable blocked** — an item in `blocked` whose blocker may now be gone. The blocker is
|
|
28
31
|
one of three classes, and repair re-checks **all** of them, not just dependencies: (a) an
|
|
29
32
|
`is blocked by` **dependency** has since closed; (b) a **validation / quality-gate self-block** —
|
|
@@ -316,12 +319,16 @@ a PR that cannot merge or a deploy that failed — it just churns.
|
|
|
316
319
|
Re-sync the branch in place so the already-enabled auto-merge can land (see diagnosis step 3).
|
|
317
320
|
Keep the item `claimed`; a later cycle confirms the merge and transitions. Do **not** file a fix
|
|
318
321
|
ticket for a clean rebase.
|
|
319
|
-
- **A
|
|
320
|
-
|
|
321
|
-
|
|
322
|
-
|
|
323
|
-
|
|
324
|
-
|
|
322
|
+
- **A true merge conflict** → **not** an immediate fix ticket. A conflict is fixable by re-running
|
|
323
|
+
the build, so attempt **one** in-place re-dispatch first (the resume sequence below; the vendor
|
|
324
|
+
agent re-enters `drive-pr-to-merge` fix mode, which resolves conflicts). Only a conflict that
|
|
325
|
+
survives that single attempt — the same conflicting head still `CONFLICTING` on a later cycle — or
|
|
326
|
+
that the agent reports needs design input, falls through to the fix-ticket path (diagnosis step 5).
|
|
327
|
+
- **A real external blocker re-running the build cannot fix** (failing checks / `CHANGES_REQUESTED` /
|
|
328
|
+
unaddressed CodeRabbit; or a failed deploy) → **do not dispatch the agent**. File a build-ready leaf
|
|
329
|
+
fix ticket for the blocker, move this item `claimed → blocked` with an `is blocked by` link to that
|
|
330
|
+
ticket, and record it. The existing "Build `blocked` → unblock if cleared" path resumes this item on
|
|
331
|
+
a later cycle once the fix ticket is terminal — a self-healing loop. Skip the resume steps below.
|
|
325
332
|
|
|
326
333
|
If the PR is healthy in-flight and no blocker is found, the work simply died mid-flight — run the **same per-item sequence
|
|
327
334
|
the vendor build-intake runs**, skipping the claim transition (the item is already `claimed`):
|
|
@@ -400,7 +407,9 @@ branch — operate on it the same way.
|
|
|
400
407
|
|
|
401
408
|
- **True merge conflict** — `mergeable = CONFLICTING` or `mergeStateStatus = DIRTY` (overlapping
|
|
402
409
|
changes a plain rebase cannot resolve), or `gh pr update-branch` (step 3) reported a conflict. A
|
|
403
|
-
merely `BEHIND` branch is **not** here — it was re-synced in step 3.
|
|
410
|
+
merely `BEHIND` branch is **not** here — it was re-synced in step 3. Unlike the other classes below, a
|
|
411
|
+
conflict is **resolvable by re-running the build**, so step 5 gives it one in-place re-dispatch before
|
|
412
|
+
filing — see its conflict-first rule.
|
|
404
413
|
- **Failing required checks** — `statusCheckRollup` has a `FAILURE`/`ERROR`/`TIMED_OUT` conclusion,
|
|
405
414
|
or `mergeStateStatus = UNSTABLE`/`BLOCKED` due to checks.
|
|
406
415
|
- **Change requests outstanding** — `reviewDecision = CHANGES_REQUESTED`, or unresolved CodeRabbit
|
|
@@ -416,6 +425,19 @@ recent check/commit activity would already have been caught as `active` by the s
|
|
|
416
425
|
|
|
417
426
|
**5. On a blocker found → file a leaf fix ticket + block the item.**
|
|
418
427
|
|
|
428
|
+
**Conflict-first exception (try to resolve before filing).** A *true merge conflict* — and only a
|
|
429
|
+
conflict, not failing checks, change requests, or a failed deploy — is fixable by re-running the build:
|
|
430
|
+
the vendor agent's `drive-pr-to-merge` fix-mode loop resolves conflicts. So before filing a fix ticket
|
|
431
|
+
for a conflict, give the item **one** in-place re-dispatch: run the resume-in-place sequence (steps 1–3
|
|
432
|
+
of the parent path above), which re-enters `drive-pr-to-merge` in fix mode against the existing PR.
|
|
433
|
+
Bound it to a single attempt per conflicting head — when you re-dispatch, post a `[lisa-repair-intake]
|
|
434
|
+
conflict-resolve-attempt: <item-ref>@<head-sha>` marker keyed on the PR head SHA. On a later cycle, if
|
|
435
|
+
that marker already exists for the **same** head SHA and the PR is still `CONFLICTING`, the attempt
|
|
436
|
+
failed: stop retrying and file the fix ticket below. File immediately (skip the attempt) if the agent
|
|
437
|
+
reports the conflict needs design input. Every other blocker class files the fix ticket with no
|
|
438
|
+
re-dispatch. Honor the backoff window / state fingerprint (Loop prevention) so the re-dispatch is never
|
|
439
|
+
re-issued against an unchanged conflicting head.
|
|
440
|
+
|
|
419
441
|
1. **File one build-ready leaf fix ticket** per distinct blocker via `lisa:tracker-write` (the
|
|
420
442
|
vendor-neutral leaf writer + validation gate; never a vendor `*-write-*` skill directly),
|
|
421
443
|
`issue_type: Bug` for a failing-check/conflict/failed-deploy, `Task` for review-feedback
|
|
@@ -20,10 +20,13 @@ close-out** roles and moves work *unstuck* or *fully closed*:
|
|
|
20
20
|
after its agent returned) — no re-dispatch. A PR that is merely **behind its base** (`BEHIND`, no
|
|
21
21
|
conflict) is **re-synced in place** with `gh pr update-branch` so the already-enabled auto-merge can
|
|
22
22
|
finally land — a clean rebase needs no human, and leaving it stranded is the exact gap that lets an
|
|
23
|
-
auto-merge PR sit unmerged forever.
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
23
|
+
auto-merge PR sit unmerged forever. A **true merge conflict** is first given **one bounded in-place
|
|
24
|
+
re-dispatch** to the build agent — whose `drive-pr-to-merge` fix-mode loop resolves conflicts — because
|
|
25
|
+
a conflict, unlike a failing external check, is fixable by re-running the build; only a conflict that
|
|
26
|
+
survives that single attempt (or that the agent says needs design input) becomes a fix ticket. The
|
|
27
|
+
genuinely non-resolvable blockers — failing checks / unaddressed CodeRabbit or `CHANGES_REQUESTED`
|
|
28
|
+
review / a failed deploy — get a build-ready leaf fix ticket with the item moved to `blocked` (blocked
|
|
29
|
+
by that ticket) instead of blindly re-dispatching the agent, which would just churn against them.
|
|
27
30
|
- **Recoverable blocked** — an item in `blocked` whose blocker may now be gone. The blocker is
|
|
28
31
|
one of three classes, and repair re-checks **all** of them, not just dependencies: (a) an
|
|
29
32
|
`is blocked by` **dependency** has since closed; (b) a **validation / quality-gate self-block** —
|
|
@@ -316,12 +319,16 @@ a PR that cannot merge or a deploy that failed — it just churns.
|
|
|
316
319
|
Re-sync the branch in place so the already-enabled auto-merge can land (see diagnosis step 3).
|
|
317
320
|
Keep the item `claimed`; a later cycle confirms the merge and transitions. Do **not** file a fix
|
|
318
321
|
ticket for a clean rebase.
|
|
319
|
-
- **A
|
|
320
|
-
|
|
321
|
-
|
|
322
|
-
|
|
323
|
-
|
|
324
|
-
|
|
322
|
+
- **A true merge conflict** → **not** an immediate fix ticket. A conflict is fixable by re-running
|
|
323
|
+
the build, so attempt **one** in-place re-dispatch first (the resume sequence below; the vendor
|
|
324
|
+
agent re-enters `drive-pr-to-merge` fix mode, which resolves conflicts). Only a conflict that
|
|
325
|
+
survives that single attempt — the same conflicting head still `CONFLICTING` on a later cycle — or
|
|
326
|
+
that the agent reports needs design input, falls through to the fix-ticket path (diagnosis step 5).
|
|
327
|
+
- **A real external blocker re-running the build cannot fix** (failing checks / `CHANGES_REQUESTED` /
|
|
328
|
+
unaddressed CodeRabbit; or a failed deploy) → **do not dispatch the agent**. File a build-ready leaf
|
|
329
|
+
fix ticket for the blocker, move this item `claimed → blocked` with an `is blocked by` link to that
|
|
330
|
+
ticket, and record it. The existing "Build `blocked` → unblock if cleared" path resumes this item on
|
|
331
|
+
a later cycle once the fix ticket is terminal — a self-healing loop. Skip the resume steps below.
|
|
325
332
|
|
|
326
333
|
If the PR is healthy in-flight and no blocker is found, the work simply died mid-flight — run the **same per-item sequence
|
|
327
334
|
the vendor build-intake runs**, skipping the claim transition (the item is already `claimed`):
|
|
@@ -400,7 +407,9 @@ branch — operate on it the same way.
|
|
|
400
407
|
|
|
401
408
|
- **True merge conflict** — `mergeable = CONFLICTING` or `mergeStateStatus = DIRTY` (overlapping
|
|
402
409
|
changes a plain rebase cannot resolve), or `gh pr update-branch` (step 3) reported a conflict. A
|
|
403
|
-
merely `BEHIND` branch is **not** here — it was re-synced in step 3.
|
|
410
|
+
merely `BEHIND` branch is **not** here — it was re-synced in step 3. Unlike the other classes below, a
|
|
411
|
+
conflict is **resolvable by re-running the build**, so step 5 gives it one in-place re-dispatch before
|
|
412
|
+
filing — see its conflict-first rule.
|
|
404
413
|
- **Failing required checks** — `statusCheckRollup` has a `FAILURE`/`ERROR`/`TIMED_OUT` conclusion,
|
|
405
414
|
or `mergeStateStatus = UNSTABLE`/`BLOCKED` due to checks.
|
|
406
415
|
- **Change requests outstanding** — `reviewDecision = CHANGES_REQUESTED`, or unresolved CodeRabbit
|
|
@@ -416,6 +425,19 @@ recent check/commit activity would already have been caught as `active` by the s
|
|
|
416
425
|
|
|
417
426
|
**5. On a blocker found → file a leaf fix ticket + block the item.**
|
|
418
427
|
|
|
428
|
+
**Conflict-first exception (try to resolve before filing).** A *true merge conflict* — and only a
|
|
429
|
+
conflict, not failing checks, change requests, or a failed deploy — is fixable by re-running the build:
|
|
430
|
+
the vendor agent's `drive-pr-to-merge` fix-mode loop resolves conflicts. So before filing a fix ticket
|
|
431
|
+
for a conflict, give the item **one** in-place re-dispatch: run the resume-in-place sequence (steps 1–3
|
|
432
|
+
of the parent path above), which re-enters `drive-pr-to-merge` in fix mode against the existing PR.
|
|
433
|
+
Bound it to a single attempt per conflicting head — when you re-dispatch, post a `[lisa-repair-intake]
|
|
434
|
+
conflict-resolve-attempt: <item-ref>@<head-sha>` marker keyed on the PR head SHA. On a later cycle, if
|
|
435
|
+
that marker already exists for the **same** head SHA and the PR is still `CONFLICTING`, the attempt
|
|
436
|
+
failed: stop retrying and file the fix ticket below. File immediately (skip the attempt) if the agent
|
|
437
|
+
reports the conflict needs design input. Every other blocker class files the fix ticket with no
|
|
438
|
+
re-dispatch. Honor the backoff window / state fingerprint (Loop prevention) so the re-dispatch is never
|
|
439
|
+
re-issued against an unchanged conflicting head.
|
|
440
|
+
|
|
419
441
|
1. **File one build-ready leaf fix ticket** per distinct blocker via `lisa:tracker-write` (the
|
|
420
442
|
vendor-neutral leaf writer + validation gate; never a vendor `*-write-*` skill directly),
|
|
421
443
|
`issue_type: Bug` for a failing-check/conflict/failed-deploy, `Task` for review-feedback
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lisa-openclaw",
|
|
3
|
-
"version": "2.
|
|
3
|
+
"version": "2.173.1",
|
|
4
4
|
"description": "Connect staff roles to Telegram or Slack via OpenClaw — facilitator/specialist hub-and-spoke routing and repo-coding topics, for Claude Code and Codex",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Cody Swann"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lisa-openclaw",
|
|
3
|
-
"version": "2.
|
|
3
|
+
"version": "2.173.1",
|
|
4
4
|
"description": "Connect staff roles to Telegram or Slack via OpenClaw — facilitator/specialist hub-and-spoke routing and repo-coding topics, across Claude and Codex.",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Cody Swann"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lisa-openclaw",
|
|
3
|
-
"version": "2.
|
|
3
|
+
"version": "2.173.1",
|
|
4
4
|
"description": "Connect staff roles to Telegram or Slack via OpenClaw — facilitator/specialist hub-and-spoke routing and repo-coding topics, for Claude Code and Codex",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Cody Swann"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lisa-openclaw",
|
|
3
|
-
"version": "2.
|
|
3
|
+
"version": "2.173.1",
|
|
4
4
|
"description": "Connect staff roles to Telegram or Slack via OpenClaw — facilitator/specialist hub-and-spoke routing and repo-coding topics, for Claude Code and Codex",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Cody Swann"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lisa-openclaw",
|
|
3
|
-
"version": "2.
|
|
3
|
+
"version": "2.173.1",
|
|
4
4
|
"description": "Connect staff roles to Telegram or Slack via OpenClaw — facilitator/specialist hub-and-spoke routing and repo-coding topics, for Claude Code and Codex",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Cody Swann"
|
|
@@ -9,10 +9,13 @@
|
|
|
9
9
|
# - UserPromptSubmit : detects /lisa:research|plan|implement|intake|debrief in the
|
|
10
10
|
# raw prompt and arms enforcement for the session
|
|
11
11
|
# - PreToolUse : detects the same skills via a `Skill` tool call,
|
|
12
|
-
# arms enforcement, and blocks bypass tool calls
|
|
13
|
-
#
|
|
14
|
-
#
|
|
15
|
-
# team
|
|
12
|
+
# arms enforcement, and blocks bypass tool calls until
|
|
13
|
+
# the team is established — i.e. until a TeamCreate
|
|
14
|
+
# (older Claude Code) or the first Agent spawn
|
|
15
|
+
# (implicit-team model, Claude Code >= 2.1.178) fires
|
|
16
|
+
# - PostToolUse : on a successful Claude TeamCreate OR a successful
|
|
17
|
+
# Agent spawn (implicit team), marks the session as
|
|
18
|
+
# team-established (lifts enforcement)
|
|
16
19
|
# - SubagentStart : marks the new subagent session as a teammate so
|
|
17
20
|
# it is exempt — teammates inherit the lead's team
|
|
18
21
|
# and must never create a second team (double-create
|
|
@@ -85,7 +88,12 @@ case "$HOOK_EVENT" in
|
|
|
85
88
|
|
|
86
89
|
PostToolUse)
|
|
87
90
|
TOOL_NAME=$(printf '%s' "$INPUT" | jq -r '.tool_name // empty' 2>/dev/null || true)
|
|
88
|
-
|
|
91
|
+
# A successful TeamCreate (older Claude Code) OR a successful Agent spawn
|
|
92
|
+
# (Claude Code >= 2.1.178, where TeamCreate/TeamDelete were removed in favor
|
|
93
|
+
# of the implicit-team model — the team forms automatically when the lead
|
|
94
|
+
# spawns its first teammate via the Agent tool) marks the session as
|
|
95
|
+
# team-established and lifts enforcement.
|
|
96
|
+
if [ "$TOOL_NAME" = "TeamCreate" ] || [ "$TOOL_NAME" = "Agent" ]; then
|
|
89
97
|
ERROR=$(printf '%s' "$INPUT" | jq -r '.tool_response.error // empty' 2>/dev/null || true)
|
|
90
98
|
IS_ERROR=$(printf '%s' "$INPUT" | jq -r '.tool_response.is_error // empty' 2>/dev/null || true)
|
|
91
99
|
if [ -z "$ERROR" ] && [ "$IS_ERROR" != "true" ]; then
|
|
@@ -136,9 +144,12 @@ if [ -f "$TEAM_FLAG" ]; then
|
|
|
136
144
|
exit 0
|
|
137
145
|
fi
|
|
138
146
|
|
|
139
|
-
# These
|
|
147
|
+
# These are the path forward; never block them.
|
|
148
|
+
# - ToolSearch / TeamCreate : the older explicit-team path (Claude Code < 2.1.178)
|
|
149
|
+
# - Agent : the implicit-team path (Claude Code >= 2.1.178) —
|
|
150
|
+
# spawning the first teammate IS what forms the team
|
|
140
151
|
case "$TOOL_NAME" in
|
|
141
|
-
ToolSearch|TeamCreate)
|
|
152
|
+
ToolSearch|TeamCreate|Agent)
|
|
142
153
|
exit 0
|
|
143
154
|
;;
|
|
144
155
|
esac
|
|
@@ -163,21 +174,25 @@ fi
|
|
|
163
174
|
ACTIVE_SKILL=$(cat "$SKILL_FLAG" 2>/dev/null || echo "lisa:???")
|
|
164
175
|
cat >&2 <<EOF
|
|
165
176
|
Blocked: this session invoked /${ACTIVE_SKILL}, which is an agent-team flow.
|
|
166
|
-
In Claude, before any other tool call, you must
|
|
177
|
+
In Claude, before any other tool call, you must establish the team by spawning
|
|
178
|
+
your first teammate:
|
|
167
179
|
|
|
168
|
-
|
|
169
|
-
|
|
180
|
+
Call the \`Agent\` tool with an appropriate \`subagent_type\`.
|
|
181
|
+
|
|
182
|
+
The team forms automatically the moment the lead spawns its first teammate —
|
|
183
|
+
this is the implicit-team model on Claude Code >= 2.1.178, where the explicit
|
|
184
|
+
\`TeamCreate\`/\`TeamDelete\` tools were removed. (On older Claude Code the
|
|
185
|
+
\`TeamCreate\` tool path still works and is also accepted.)
|
|
170
186
|
|
|
171
187
|
The current attempt to call \`${TOOL_NAME}\` is a team-bypass path. Reading
|
|
172
188
|
the ticket, exploring the code, fetching context — those are tasks for the
|
|
173
|
-
team you are about to
|
|
174
|
-
exists.
|
|
189
|
+
team you are about to spawn, not for the lead session before the team exists.
|
|
175
190
|
|
|
176
191
|
If you are running Lisa in a non-Claude harness, this Claude enforcement hook
|
|
177
192
|
should not be installed; follow the runtime-aware orchestration preamble in the
|
|
178
193
|
skill instead.
|
|
179
194
|
|
|
180
|
-
Re-read the orchestration preamble in /${ACTIVE_SKILL} and start
|
|
181
|
-
|
|
195
|
+
Re-read the orchestration preamble in /${ACTIVE_SKILL} and start by spawning a
|
|
196
|
+
teammate with the \`Agent\` tool.
|
|
182
197
|
EOF
|
|
183
198
|
exit 2
|
|
@@ -20,10 +20,13 @@ close-out** roles and moves work *unstuck* or *fully closed*:
|
|
|
20
20
|
after its agent returned) — no re-dispatch. A PR that is merely **behind its base** (`BEHIND`, no
|
|
21
21
|
conflict) is **re-synced in place** with `gh pr update-branch` so the already-enabled auto-merge can
|
|
22
22
|
finally land — a clean rebase needs no human, and leaving it stranded is the exact gap that lets an
|
|
23
|
-
auto-merge PR sit unmerged forever.
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
23
|
+
auto-merge PR sit unmerged forever. A **true merge conflict** is first given **one bounded in-place
|
|
24
|
+
re-dispatch** to the build agent — whose `drive-pr-to-merge` fix-mode loop resolves conflicts — because
|
|
25
|
+
a conflict, unlike a failing external check, is fixable by re-running the build; only a conflict that
|
|
26
|
+
survives that single attempt (or that the agent says needs design input) becomes a fix ticket. The
|
|
27
|
+
genuinely non-resolvable blockers — failing checks / unaddressed CodeRabbit or `CHANGES_REQUESTED`
|
|
28
|
+
review / a failed deploy — get a build-ready leaf fix ticket with the item moved to `blocked` (blocked
|
|
29
|
+
by that ticket) instead of blindly re-dispatching the agent, which would just churn against them.
|
|
27
30
|
- **Recoverable blocked** — an item in `blocked` whose blocker may now be gone. The blocker is
|
|
28
31
|
one of three classes, and repair re-checks **all** of them, not just dependencies: (a) an
|
|
29
32
|
`is blocked by` **dependency** has since closed; (b) a **validation / quality-gate self-block** —
|
|
@@ -316,12 +319,16 @@ a PR that cannot merge or a deploy that failed — it just churns.
|
|
|
316
319
|
Re-sync the branch in place so the already-enabled auto-merge can land (see diagnosis step 3).
|
|
317
320
|
Keep the item `claimed`; a later cycle confirms the merge and transitions. Do **not** file a fix
|
|
318
321
|
ticket for a clean rebase.
|
|
319
|
-
- **A
|
|
320
|
-
|
|
321
|
-
|
|
322
|
-
|
|
323
|
-
|
|
324
|
-
|
|
322
|
+
- **A true merge conflict** → **not** an immediate fix ticket. A conflict is fixable by re-running
|
|
323
|
+
the build, so attempt **one** in-place re-dispatch first (the resume sequence below; the vendor
|
|
324
|
+
agent re-enters `drive-pr-to-merge` fix mode, which resolves conflicts). Only a conflict that
|
|
325
|
+
survives that single attempt — the same conflicting head still `CONFLICTING` on a later cycle — or
|
|
326
|
+
that the agent reports needs design input, falls through to the fix-ticket path (diagnosis step 5).
|
|
327
|
+
- **A real external blocker re-running the build cannot fix** (failing checks / `CHANGES_REQUESTED` /
|
|
328
|
+
unaddressed CodeRabbit; or a failed deploy) → **do not dispatch the agent**. File a build-ready leaf
|
|
329
|
+
fix ticket for the blocker, move this item `claimed → blocked` with an `is blocked by` link to that
|
|
330
|
+
ticket, and record it. The existing "Build `blocked` → unblock if cleared" path resumes this item on
|
|
331
|
+
a later cycle once the fix ticket is terminal — a self-healing loop. Skip the resume steps below.
|
|
325
332
|
|
|
326
333
|
If the PR is healthy in-flight and no blocker is found, the work simply died mid-flight — run the **same per-item sequence
|
|
327
334
|
the vendor build-intake runs**, skipping the claim transition (the item is already `claimed`):
|
|
@@ -400,7 +407,9 @@ branch — operate on it the same way.
|
|
|
400
407
|
|
|
401
408
|
- **True merge conflict** — `mergeable = CONFLICTING` or `mergeStateStatus = DIRTY` (overlapping
|
|
402
409
|
changes a plain rebase cannot resolve), or `gh pr update-branch` (step 3) reported a conflict. A
|
|
403
|
-
merely `BEHIND` branch is **not** here — it was re-synced in step 3.
|
|
410
|
+
merely `BEHIND` branch is **not** here — it was re-synced in step 3. Unlike the other classes below, a
|
|
411
|
+
conflict is **resolvable by re-running the build**, so step 5 gives it one in-place re-dispatch before
|
|
412
|
+
filing — see its conflict-first rule.
|
|
404
413
|
- **Failing required checks** — `statusCheckRollup` has a `FAILURE`/`ERROR`/`TIMED_OUT` conclusion,
|
|
405
414
|
or `mergeStateStatus = UNSTABLE`/`BLOCKED` due to checks.
|
|
406
415
|
- **Change requests outstanding** — `reviewDecision = CHANGES_REQUESTED`, or unresolved CodeRabbit
|
|
@@ -416,6 +425,19 @@ recent check/commit activity would already have been caught as `active` by the s
|
|
|
416
425
|
|
|
417
426
|
**5. On a blocker found → file a leaf fix ticket + block the item.**
|
|
418
427
|
|
|
428
|
+
**Conflict-first exception (try to resolve before filing).** A *true merge conflict* — and only a
|
|
429
|
+
conflict, not failing checks, change requests, or a failed deploy — is fixable by re-running the build:
|
|
430
|
+
the vendor agent's `drive-pr-to-merge` fix-mode loop resolves conflicts. So before filing a fix ticket
|
|
431
|
+
for a conflict, give the item **one** in-place re-dispatch: run the resume-in-place sequence (steps 1–3
|
|
432
|
+
of the parent path above), which re-enters `drive-pr-to-merge` in fix mode against the existing PR.
|
|
433
|
+
Bound it to a single attempt per conflicting head — when you re-dispatch, post a `[lisa-repair-intake]
|
|
434
|
+
conflict-resolve-attempt: <item-ref>@<head-sha>` marker keyed on the PR head SHA. On a later cycle, if
|
|
435
|
+
that marker already exists for the **same** head SHA and the PR is still `CONFLICTING`, the attempt
|
|
436
|
+
failed: stop retrying and file the fix ticket below. File immediately (skip the attempt) if the agent
|
|
437
|
+
reports the conflict needs design input. Every other blocker class files the fix ticket with no
|
|
438
|
+
re-dispatch. Honor the backoff window / state fingerprint (Loop prevention) so the re-dispatch is never
|
|
439
|
+
re-issued against an unchanged conflicting head.
|
|
440
|
+
|
|
419
441
|
1. **File one build-ready leaf fix ticket** per distinct blocker via `lisa:tracker-write` (the
|
|
420
442
|
vendor-neutral leaf writer + validation gate; never a vendor `*-write-*` skill directly),
|
|
421
443
|
`issue_type: Bug` for a failing-check/conflict/failed-deploy, `Task` for review-feedback
|