@codyswann/lisa 2.171.6 → 2.173.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/package.json +1 -1
- package/plugins/lisa/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa/skills/github-build-intake/SKILL.md +4 -0
- package/plugins/lisa/skills/jira-build-intake/SKILL.md +1 -0
- package/plugins/lisa/skills/linear-build-intake/SKILL.md +2 -0
- package/plugins/lisa/skills/repair-intake/SKILL.md +33 -11
- package/plugins/lisa-agy/plugin.json +1 -1
- package/plugins/lisa-agy/skills/github-build-intake/SKILL.md +4 -0
- package/plugins/lisa-agy/skills/jira-build-intake/SKILL.md +1 -0
- package/plugins/lisa-agy/skills/linear-build-intake/SKILL.md +2 -0
- 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/github-build-intake/SKILL.md +4 -0
- package/plugins/lisa-copilot/skills/jira-build-intake/SKILL.md +1 -0
- package/plugins/lisa-copilot/skills/linear-build-intake/SKILL.md +2 -0
- 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/github-build-intake/SKILL.md +4 -0
- package/plugins/lisa-cursor/skills/jira-build-intake/SKILL.md +1 -0
- package/plugins/lisa-cursor/skills/linear-build-intake/SKILL.md +2 -0
- 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/skills/github-build-intake/SKILL.md +4 -0
- package/plugins/src/base/skills/jira-build-intake/SKILL.md +1 -0
- package/plugins/src/base/skills/linear-build-intake/SKILL.md +2 -0
- 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.0",
|
|
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": {
|
|
@@ -245,6 +245,10 @@ A blocker is active if it is open and has no cleared status label. Treat `status
|
|
|
245
245
|
|
|
246
246
|
```bash
|
|
247
247
|
gh issue edit <number> --repo <org>/<repo> --remove-label "$READY" --add-label "$CLAIMED"
|
|
248
|
+
# Assign to the authenticated user ONLY when the issue is currently unassigned (attributable claim;
|
|
249
|
+
# do not pile a second assignee onto an issue that already has an owner):
|
|
250
|
+
gh issue view <number> --repo <org>/<repo> --json assignees -q '.assignees | length' # → if 0:
|
|
251
|
+
gh issue edit <number> --repo <org>/<repo> --add-assignee "@me"
|
|
248
252
|
gh issue comment <number> --repo <org>/<repo> --body "[claude-build-intake] Claimed by Claude. Starting build."
|
|
249
253
|
```
|
|
250
254
|
|
|
@@ -177,6 +177,7 @@ This gate never blocks a legitimate flat Task/Bug: those have no open children a
|
|
|
177
177
|
#### 3b. Claim
|
|
178
178
|
|
|
179
179
|
Transition the ticket from `$READY` to `$CLAIMED` by invoking `lisa:atlassian-access` `operation: transition key: <TICKET> to: "$CLAIMED"`.
|
|
180
|
+
- **Assign to the authenticated user when the ticket is unassigned.** A claim must be attributable. If the ticket has no assignee, assign it to the authenticated account — prefer acli `--assignee @me` (resolves server-side to the authenticated user, which avoids the federated-`accountId` mis-assignment), or `write-ticket` with the `accountId` from the `/rest/api/3/myself` identity probe the access skill already documents. Leave an already-assigned ticket's assignee untouched — never reassign work that already has an owner.
|
|
180
181
|
- Post a `[claude-build-intake]` comment via `lisa:atlassian-access` `operation: comment key: <TICKET> body: "Claimed by Claude. Starting build."`
|
|
181
182
|
- This is the idempotency lock — a re-entrant cycle's `Status = $READY` filter will not see this ticket again.
|
|
182
183
|
|
|
@@ -186,6 +186,8 @@ This gate never blocks a legitimate flat Task/Bug: those have no open children a
|
|
|
186
186
|
|
|
187
187
|
Update labels via `mcp__linear-server__save_issue`: remove `$READY`, add `$CLAIMED`. Resolve label IDs via `list_issue_labels` (create `$CLAIMED` if missing).
|
|
188
188
|
|
|
189
|
+
**Assign to the authenticated user when the Issue is unassigned.** A claim must be attributable. If the Issue has no assignee, set its `assigneeId` to the authenticated viewer (resolve the viewer's id via the Linear MCP identity — e.g. `get_user` for the current actor) through `mcp__linear-server__save_issue`. Leave an already-assigned Issue's assignee untouched — never reassign work that already has an owner.
|
|
190
|
+
|
|
189
191
|
Post a `[claude-build-intake]` comment via `save_comment`: `"Claimed by Claude. Starting build."`
|
|
190
192
|
|
|
191
193
|
This is the idempotency lock — a re-entrant cycle's `label: $READY` filter will not see this Issue again.
|
|
@@ -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
|
|
@@ -245,6 +245,10 @@ A blocker is active if it is open and has no cleared status label. Treat `status
|
|
|
245
245
|
|
|
246
246
|
```bash
|
|
247
247
|
gh issue edit <number> --repo <org>/<repo> --remove-label "$READY" --add-label "$CLAIMED"
|
|
248
|
+
# Assign to the authenticated user ONLY when the issue is currently unassigned (attributable claim;
|
|
249
|
+
# do not pile a second assignee onto an issue that already has an owner):
|
|
250
|
+
gh issue view <number> --repo <org>/<repo> --json assignees -q '.assignees | length' # → if 0:
|
|
251
|
+
gh issue edit <number> --repo <org>/<repo> --add-assignee "@me"
|
|
248
252
|
gh issue comment <number> --repo <org>/<repo> --body "[claude-build-intake] Claimed by Claude. Starting build."
|
|
249
253
|
```
|
|
250
254
|
|
|
@@ -177,6 +177,7 @@ This gate never blocks a legitimate flat Task/Bug: those have no open children a
|
|
|
177
177
|
#### 3b. Claim
|
|
178
178
|
|
|
179
179
|
Transition the ticket from `$READY` to `$CLAIMED` by invoking `lisa:atlassian-access` `operation: transition key: <TICKET> to: "$CLAIMED"`.
|
|
180
|
+
- **Assign to the authenticated user when the ticket is unassigned.** A claim must be attributable. If the ticket has no assignee, assign it to the authenticated account — prefer acli `--assignee @me` (resolves server-side to the authenticated user, which avoids the federated-`accountId` mis-assignment), or `write-ticket` with the `accountId` from the `/rest/api/3/myself` identity probe the access skill already documents. Leave an already-assigned ticket's assignee untouched — never reassign work that already has an owner.
|
|
180
181
|
- Post a `[claude-build-intake]` comment via `lisa:atlassian-access` `operation: comment key: <TICKET> body: "Claimed by Claude. Starting build."`
|
|
181
182
|
- This is the idempotency lock — a re-entrant cycle's `Status = $READY` filter will not see this ticket again.
|
|
182
183
|
|
|
@@ -186,6 +186,8 @@ This gate never blocks a legitimate flat Task/Bug: those have no open children a
|
|
|
186
186
|
|
|
187
187
|
Update labels via `mcp__linear-server__save_issue`: remove `$READY`, add `$CLAIMED`. Resolve label IDs via `list_issue_labels` (create `$CLAIMED` if missing).
|
|
188
188
|
|
|
189
|
+
**Assign to the authenticated user when the Issue is unassigned.** A claim must be attributable. If the Issue has no assignee, set its `assigneeId` to the authenticated viewer (resolve the viewer's id via the Linear MCP identity — e.g. `get_user` for the current actor) through `mcp__linear-server__save_issue`. Leave an already-assigned Issue's assignee untouched — never reassign work that already has an owner.
|
|
190
|
+
|
|
189
191
|
Post a `[claude-build-intake]` comment via `save_comment`: `"Claimed by Claude. Starting build."`
|
|
190
192
|
|
|
191
193
|
This is the idempotency lock — a re-entrant cycle's `label: $READY` filter will not see this Issue again.
|
|
@@ -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
|
|
@@ -245,6 +245,10 @@ A blocker is active if it is open and has no cleared status label. Treat `status
|
|
|
245
245
|
|
|
246
246
|
```bash
|
|
247
247
|
gh issue edit <number> --repo <org>/<repo> --remove-label "$READY" --add-label "$CLAIMED"
|
|
248
|
+
# Assign to the authenticated user ONLY when the issue is currently unassigned (attributable claim;
|
|
249
|
+
# do not pile a second assignee onto an issue that already has an owner):
|
|
250
|
+
gh issue view <number> --repo <org>/<repo> --json assignees -q '.assignees | length' # → if 0:
|
|
251
|
+
gh issue edit <number> --repo <org>/<repo> --add-assignee "@me"
|
|
248
252
|
gh issue comment <number> --repo <org>/<repo> --body "[claude-build-intake] Claimed by Claude. Starting build."
|
|
249
253
|
```
|
|
250
254
|
|
|
@@ -177,6 +177,7 @@ This gate never blocks a legitimate flat Task/Bug: those have no open children a
|
|
|
177
177
|
#### 3b. Claim
|
|
178
178
|
|
|
179
179
|
Transition the ticket from `$READY` to `$CLAIMED` by invoking `lisa:atlassian-access` `operation: transition key: <TICKET> to: "$CLAIMED"`.
|
|
180
|
+
- **Assign to the authenticated user when the ticket is unassigned.** A claim must be attributable. If the ticket has no assignee, assign it to the authenticated account — prefer acli `--assignee @me` (resolves server-side to the authenticated user, which avoids the federated-`accountId` mis-assignment), or `write-ticket` with the `accountId` from the `/rest/api/3/myself` identity probe the access skill already documents. Leave an already-assigned ticket's assignee untouched — never reassign work that already has an owner.
|
|
180
181
|
- Post a `[claude-build-intake]` comment via `lisa:atlassian-access` `operation: comment key: <TICKET> body: "Claimed by Claude. Starting build."`
|
|
181
182
|
- This is the idempotency lock — a re-entrant cycle's `Status = $READY` filter will not see this ticket again.
|
|
182
183
|
|
|
@@ -186,6 +186,8 @@ This gate never blocks a legitimate flat Task/Bug: those have no open children a
|
|
|
186
186
|
|
|
187
187
|
Update labels via `mcp__linear-server__save_issue`: remove `$READY`, add `$CLAIMED`. Resolve label IDs via `list_issue_labels` (create `$CLAIMED` if missing).
|
|
188
188
|
|
|
189
|
+
**Assign to the authenticated user when the Issue is unassigned.** A claim must be attributable. If the Issue has no assignee, set its `assigneeId` to the authenticated viewer (resolve the viewer's id via the Linear MCP identity — e.g. `get_user` for the current actor) through `mcp__linear-server__save_issue`. Leave an already-assigned Issue's assignee untouched — never reassign work that already has an owner.
|
|
190
|
+
|
|
189
191
|
Post a `[claude-build-intake]` comment via `save_comment`: `"Claimed by Claude. Starting build."`
|
|
190
192
|
|
|
191
193
|
This is the idempotency lock — a re-entrant cycle's `label: $READY` filter will not see this Issue again.
|
|
@@ -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
|
|
@@ -245,6 +245,10 @@ A blocker is active if it is open and has no cleared status label. Treat `status
|
|
|
245
245
|
|
|
246
246
|
```bash
|
|
247
247
|
gh issue edit <number> --repo <org>/<repo> --remove-label "$READY" --add-label "$CLAIMED"
|
|
248
|
+
# Assign to the authenticated user ONLY when the issue is currently unassigned (attributable claim;
|
|
249
|
+
# do not pile a second assignee onto an issue that already has an owner):
|
|
250
|
+
gh issue view <number> --repo <org>/<repo> --json assignees -q '.assignees | length' # → if 0:
|
|
251
|
+
gh issue edit <number> --repo <org>/<repo> --add-assignee "@me"
|
|
248
252
|
gh issue comment <number> --repo <org>/<repo> --body "[claude-build-intake] Claimed by Claude. Starting build."
|
|
249
253
|
```
|
|
250
254
|
|
|
@@ -177,6 +177,7 @@ This gate never blocks a legitimate flat Task/Bug: those have no open children a
|
|
|
177
177
|
#### 3b. Claim
|
|
178
178
|
|
|
179
179
|
Transition the ticket from `$READY` to `$CLAIMED` by invoking `lisa:atlassian-access` `operation: transition key: <TICKET> to: "$CLAIMED"`.
|
|
180
|
+
- **Assign to the authenticated user when the ticket is unassigned.** A claim must be attributable. If the ticket has no assignee, assign it to the authenticated account — prefer acli `--assignee @me` (resolves server-side to the authenticated user, which avoids the federated-`accountId` mis-assignment), or `write-ticket` with the `accountId` from the `/rest/api/3/myself` identity probe the access skill already documents. Leave an already-assigned ticket's assignee untouched — never reassign work that already has an owner.
|
|
180
181
|
- Post a `[claude-build-intake]` comment via `lisa:atlassian-access` `operation: comment key: <TICKET> body: "Claimed by Claude. Starting build."`
|
|
181
182
|
- This is the idempotency lock — a re-entrant cycle's `Status = $READY` filter will not see this ticket again.
|
|
182
183
|
|
|
@@ -186,6 +186,8 @@ This gate never blocks a legitimate flat Task/Bug: those have no open children a
|
|
|
186
186
|
|
|
187
187
|
Update labels via `mcp__linear-server__save_issue`: remove `$READY`, add `$CLAIMED`. Resolve label IDs via `list_issue_labels` (create `$CLAIMED` if missing).
|
|
188
188
|
|
|
189
|
+
**Assign to the authenticated user when the Issue is unassigned.** A claim must be attributable. If the Issue has no assignee, set its `assigneeId` to the authenticated viewer (resolve the viewer's id via the Linear MCP identity — e.g. `get_user` for the current actor) through `mcp__linear-server__save_issue`. Leave an already-assigned Issue's assignee untouched — never reassign work that already has an owner.
|
|
190
|
+
|
|
189
191
|
Post a `[claude-build-intake]` comment via `save_comment`: `"Claimed by Claude. Starting build."`
|
|
190
192
|
|
|
191
193
|
This is the idempotency lock — a re-entrant cycle's `label: $READY` filter will not see this Issue again.
|
|
@@ -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.0",
|
|
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.0",
|
|
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.0",
|
|
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.0",
|
|
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.0",
|
|
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"
|
|
@@ -245,6 +245,10 @@ A blocker is active if it is open and has no cleared status label. Treat `status
|
|
|
245
245
|
|
|
246
246
|
```bash
|
|
247
247
|
gh issue edit <number> --repo <org>/<repo> --remove-label "$READY" --add-label "$CLAIMED"
|
|
248
|
+
# Assign to the authenticated user ONLY when the issue is currently unassigned (attributable claim;
|
|
249
|
+
# do not pile a second assignee onto an issue that already has an owner):
|
|
250
|
+
gh issue view <number> --repo <org>/<repo> --json assignees -q '.assignees | length' # → if 0:
|
|
251
|
+
gh issue edit <number> --repo <org>/<repo> --add-assignee "@me"
|
|
248
252
|
gh issue comment <number> --repo <org>/<repo> --body "[claude-build-intake] Claimed by Claude. Starting build."
|
|
249
253
|
```
|
|
250
254
|
|
|
@@ -177,6 +177,7 @@ This gate never blocks a legitimate flat Task/Bug: those have no open children a
|
|
|
177
177
|
#### 3b. Claim
|
|
178
178
|
|
|
179
179
|
Transition the ticket from `$READY` to `$CLAIMED` by invoking `lisa:atlassian-access` `operation: transition key: <TICKET> to: "$CLAIMED"`.
|
|
180
|
+
- **Assign to the authenticated user when the ticket is unassigned.** A claim must be attributable. If the ticket has no assignee, assign it to the authenticated account — prefer acli `--assignee @me` (resolves server-side to the authenticated user, which avoids the federated-`accountId` mis-assignment), or `write-ticket` with the `accountId` from the `/rest/api/3/myself` identity probe the access skill already documents. Leave an already-assigned ticket's assignee untouched — never reassign work that already has an owner.
|
|
180
181
|
- Post a `[claude-build-intake]` comment via `lisa:atlassian-access` `operation: comment key: <TICKET> body: "Claimed by Claude. Starting build."`
|
|
181
182
|
- This is the idempotency lock — a re-entrant cycle's `Status = $READY` filter will not see this ticket again.
|
|
182
183
|
|
|
@@ -186,6 +186,8 @@ This gate never blocks a legitimate flat Task/Bug: those have no open children a
|
|
|
186
186
|
|
|
187
187
|
Update labels via `mcp__linear-server__save_issue`: remove `$READY`, add `$CLAIMED`. Resolve label IDs via `list_issue_labels` (create `$CLAIMED` if missing).
|
|
188
188
|
|
|
189
|
+
**Assign to the authenticated user when the Issue is unassigned.** A claim must be attributable. If the Issue has no assignee, set its `assigneeId` to the authenticated viewer (resolve the viewer's id via the Linear MCP identity — e.g. `get_user` for the current actor) through `mcp__linear-server__save_issue`. Leave an already-assigned Issue's assignee untouched — never reassign work that already has an owner.
|
|
190
|
+
|
|
189
191
|
Post a `[claude-build-intake]` comment via `save_comment`: `"Claimed by Claude. Starting build."`
|
|
190
192
|
|
|
191
193
|
This is the idempotency lock — a re-entrant cycle's `label: $READY` filter will not see this Issue again.
|
|
@@ -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
|