@codyswann/lisa 2.110.1 → 2.111.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/commands/repair-intake.md +2 -2
- package/plugins/lisa/rules/config-resolution.md +2 -2
- package/plugins/lisa/skills/repair-intake/SKILL.md +86 -9
- package/plugins/lisa-cdk/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-expo/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-expo/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-expo/commands/e2e-coverage-gaps.md +7 -0
- package/plugins/lisa-expo/commands/exploratory-qa.md +2 -2
- package/plugins/lisa-expo/skills/e2e-coverage-gaps/SKILL.md +105 -0
- package/plugins/lisa-expo/skills/e2e-coverage-gaps/agents/openai.yaml +4 -0
- package/plugins/lisa-expo/skills/exploratory-qa/SKILL.md +100 -93
- package/plugins/lisa-expo/skills/exploratory-qa/agents/openai.yaml +2 -2
- 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/commands/e2e-coverage-gaps.md +7 -0
- package/plugins/lisa-harper-fabric/commands/exploratory-qa.md +2 -2
- package/plugins/lisa-harper-fabric/skills/e2e-coverage-gaps/SKILL.md +105 -0
- package/plugins/lisa-harper-fabric/skills/e2e-coverage-gaps/agents/openai.yaml +4 -0
- package/plugins/lisa-harper-fabric/skills/exploratory-qa/SKILL.md +100 -93
- package/plugins/lisa-harper-fabric/skills/exploratory-qa/agents/openai.yaml +2 -2
- package/plugins/lisa-nestjs/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs/.codex-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-rails/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-rails/commands/e2e-coverage-gaps.md +7 -0
- package/plugins/lisa-rails/commands/exploratory-qa.md +2 -2
- package/plugins/lisa-rails/skills/e2e-coverage-gaps/SKILL.md +105 -0
- package/plugins/lisa-rails/skills/e2e-coverage-gaps/agents/openai.yaml +4 -0
- package/plugins/lisa-rails/skills/exploratory-qa/SKILL.md +100 -93
- package/plugins/lisa-rails/skills/exploratory-qa/agents/openai.yaml +2 -2
- package/plugins/lisa-typescript/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript/.codex-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/templates/llm-wiki-contract.md +12 -0
- package/plugins/src/base/commands/repair-intake.md +2 -2
- package/plugins/src/base/rules/config-resolution.md +2 -2
- package/plugins/src/base/skills/repair-intake/SKILL.md +86 -9
- package/plugins/src/expo/commands/e2e-coverage-gaps.md +7 -0
- package/plugins/src/expo/commands/exploratory-qa.md +2 -2
- package/plugins/src/expo/skills/e2e-coverage-gaps/SKILL.md +105 -0
- package/plugins/src/expo/skills/exploratory-qa/SKILL.md +100 -93
- package/plugins/src/harper-fabric/commands/e2e-coverage-gaps.md +7 -0
- package/plugins/src/harper-fabric/commands/exploratory-qa.md +2 -2
- package/plugins/src/harper-fabric/skills/e2e-coverage-gaps/SKILL.md +105 -0
- package/plugins/src/harper-fabric/skills/exploratory-qa/SKILL.md +100 -93
- package/plugins/src/rails/commands/e2e-coverage-gaps.md +7 -0
- package/plugins/src/rails/commands/exploratory-qa.md +2 -2
- package/plugins/src/rails/skills/e2e-coverage-gaps/SKILL.md +105 -0
- package/plugins/src/rails/skills/exploratory-qa/SKILL.md +100 -93
- package/plugins/src/wiki/templates/llm-wiki-contract.md +12 -0
package/package.json
CHANGED
|
@@ -82,7 +82,7 @@
|
|
|
82
82
|
"lodash": ">=4.18.1"
|
|
83
83
|
},
|
|
84
84
|
"name": "@codyswann/lisa",
|
|
85
|
-
"version": "2.
|
|
85
|
+
"version": "2.111.0",
|
|
86
86
|
"description": "Claude Code governance framework that applies guardrails, guidance, and automated enforcement to projects",
|
|
87
87
|
"main": "dist/index.js",
|
|
88
88
|
"exports": {
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
|
-
description: "Repair counterpart to /lisa:intake. Vendor-agnostic batch scanner that finds stuck or half-closed work — items left in `blocked`, stalled in an in-progress role (build `claimed`, PRD `in_review`), terminal-labeled items still natively open, and rollups whose children are all terminal — across the same queues /lisa:intake serves (Notion / Confluence / Linear / GitHub PRDs; JIRA / GitHub / Linear build issues). Repairs every materially actionable candidate inside the `max_candidates` cap: resumes stalled in-progress work in place, re-validates blocked PRDs, re-dispatches blocked build items whose blockers have cleared, performs terminal native closure, and closes out completed rollups. Cron-safe and bounded; default GitHub intake_mode is both and default max_candidates is 100."
|
|
3
|
-
argument-hint: "<Notion-PRD-database-URL | Confluence-space-URL | Confluence-parent-page-URL | Linear-workspace-URL | Linear-team-URL | GitHub-repo-URL | org/repo | JIRA-project-key | JQL-filter> [intake_mode=prd|build|both] [stale_after=
|
|
2
|
+
description: "Repair counterpart to /lisa:intake. Vendor-agnostic batch scanner that finds stuck or half-closed work — items left in `blocked`, stalled in an in-progress role (build `claimed`, PRD `in_review`), terminal-labeled items still natively open, and rollups whose children are all terminal — across the same queues /lisa:intake serves (Notion / Confluence / Linear / GitHub PRDs; JIRA / GitHub / Linear build issues). Repairs every materially actionable candidate inside the `max_candidates` cap: resumes stalled in-progress work in place — but for a stalled build it first diagnoses the PR/deploy state and, if the PR cannot merge (conflict, rebase, failing checks, unaddressed CodeRabbit/changes-requested) or a deploy failed, files a build-ready fix ticket and moves the item to `blocked` (blocked by it) instead of re-dispatching — re-validates blocked PRDs, re-dispatches blocked build items whose blockers have cleared, performs terminal native closure, and closes out completed rollups. Cron-safe and bounded; default GitHub intake_mode is both and default max_candidates is 100."
|
|
3
|
+
argument-hint: "<Notion-PRD-database-URL | Confluence-space-URL | Confluence-parent-page-URL | Linear-workspace-URL | Linear-team-URL | GitHub-repo-URL | org/repo | JIRA-project-key | JQL-filter> [intake_mode=prd|build|both] [stale_after=2h] [max_candidates=100] [force=true]"
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
Use the /lisa:repair-intake skill to scan the queue for stuck or half-closed items and repair every materially actionable candidate inside `max_candidates` (default 100). For GitHub queues, default `intake_mode` to `both` when the caller omits it. $ARGUMENTS
|
|
@@ -147,7 +147,7 @@ fi
|
|
|
147
147
|
"intake": {
|
|
148
148
|
"assignee": "<vendor-user-id-or-login>",
|
|
149
149
|
"repair": {
|
|
150
|
-
"staleAfterHours":
|
|
150
|
+
"staleAfterHours": 2,
|
|
151
151
|
"maxCandidates": 100
|
|
152
152
|
}
|
|
153
153
|
}
|
|
@@ -299,7 +299,7 @@ documented defaults, so existing projects need no config change.
|
|
|
299
299
|
|
|
300
300
|
| Key | Required | Default | Notes |
|
|
301
301
|
|-----|----------|---------|-------|
|
|
302
|
-
| `intake.repair.staleAfterHours` | no | `
|
|
302
|
+
| `intake.repair.staleAfterHours` | no | `2` | How long an in-progress item (build `claimed`, PRD `in_review`) may show no observable activity before repair-intake treats it as stalled and resumes it. `blocked` items are judged on blocker/answer state, not this threshold. Overridable per-run via `stale_after=<dur>` in `$ARGUMENTS` (which always wins). The same value is the default backoff window for loop-prevention notes. |
|
|
303
303
|
| `intake.repair.maxCandidates` | no | `100` | Upper bound on how many stuck items repair-intake enumerates while searching for the first actionable one. Bounds scan cost. Overridable per-run via `max_candidates=<n>`. |
|
|
304
304
|
|
|
305
305
|
### Intake assignee filter (`intake.assignee`)
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: repair-intake
|
|
3
|
-
description: "Vendor-agnostic repair scanner — the recovery counterpart to lisa:intake. Where intake claims `ready` work, repair-intake finds work that got stuck or was left half-closed: items left in `blocked`, stalled in an in-progress role (build `claimed`, PRD `in_review`), terminal-labeled items that are still natively open, and rollup/container items whose children are all terminal but whose parent is not closed out. Scans the same queues lisa:intake serves (Notion / Confluence / Linear / GitHub PRD databases; JIRA / GitHub / Linear build queues), enumerates candidates up to `max_candidates`, and repairs every materially actionable one in that bounded set: resumes stalled in-progress work IN PLACE (build → the vendor agent + the scanner's post-agent transition; PRD → the source `*-to-tracker` dry-run validate→route pipeline), re-validates blocked PRDs when new clarifying answers exist, re-dispatches blocked build items whose `is blocked by` dependencies have since closed, performs terminal native closure for terminal-labeled items, and closes rollups whose associated child work is fully terminal. Idempotent, loop-protected via a [lisa-repair-intake] marker + state fingerprint + backoff. Never mutates product-owned states (`draft`, `verified`) and never touches `ready` items. Designed as a /schedule cron target running alongside lisa:intake."
|
|
3
|
+
description: "Vendor-agnostic repair scanner — the recovery counterpart to lisa:intake. Where intake claims `ready` work, repair-intake finds work that got stuck or was left half-closed: items left in `blocked`, stalled in an in-progress role (build `claimed`, PRD `in_review`), terminal-labeled items that are still natively open, and rollup/container items whose children are all terminal but whose parent is not closed out. Scans the same queues lisa:intake serves (Notion / Confluence / Linear / GitHub PRD databases; JIRA / GitHub / Linear build queues), enumerates candidates up to `max_candidates`, and repairs every materially actionable one in that bounded set: resumes stalled in-progress work IN PLACE (build → the vendor agent + the scanner's post-agent transition; PRD → the source `*-to-tracker` dry-run validate→route pipeline) — but for a stalled build it first diagnoses the PR/deploy state and, if the PR cannot merge (conflict, rebase-required, failing checks, unaddressed CodeRabbit/changes-requested) or a deploy failed, files a build-ready leaf fix ticket and moves the item to `blocked` (blocked by that ticket) rather than re-dispatching, re-validates blocked PRDs when new clarifying answers exist, re-dispatches blocked build items whose `is blocked by` dependencies have since closed, performs terminal native closure for terminal-labeled items, and closes rollups whose associated child work is fully terminal. Idempotent, loop-protected via a [lisa-repair-intake] marker + state fingerprint + backoff. Never mutates product-owned states (`draft`, `verified`) and never touches `ready` items. Designed as a /schedule cron target running alongside lisa:intake."
|
|
4
4
|
allowed-tools: ["Skill", "Bash", "Read", "Write", "Edit", "mcp__linear-server__list_teams", "mcp__linear-server__list_projects", "mcp__linear-server__get_project", "mcp__linear-server__save_project", "mcp__linear-server__list_project_labels", "mcp__linear-server__list_issues", "mcp__linear-server__get_issue", "mcp__linear-server__save_issue", "mcp__linear-server__list_comments", "mcp__linear-server__save_comment", "mcp__linear-server__list_issue_labels", "mcp__linear-server__create_issue_label"]
|
|
5
5
|
---
|
|
6
6
|
|
|
@@ -14,7 +14,11 @@ close-out** roles and moves work *unstuck* or *fully closed*:
|
|
|
14
14
|
`in_review`) whose processing cycle died. It is technically "being worked" but nothing is
|
|
15
15
|
happening, so it sits ignored forever. (The vendor PRD intakes explicitly leave an errored PRD
|
|
16
16
|
in `in_review` "for the human to investigate from there" — that orphan is exactly what this
|
|
17
|
-
skill recovers.)
|
|
17
|
+
skill recovers.) For a stalled **build**, repair-intake first diagnoses *why* it stalled by
|
|
18
|
+
inspecting its PRs and deploys: if the PR cannot merge (conflict / rebase-required / failing
|
|
19
|
+
checks / unaddressed CodeRabbit or `CHANGES_REQUESTED` review) or a deploy failed, it files a
|
|
20
|
+
build-ready leaf fix ticket and moves the item to `blocked` (blocked by that ticket) instead of
|
|
21
|
+
blindly re-dispatching the agent — which would just churn against an unmergeable PR.
|
|
18
22
|
- **Recoverable blocked** — an item in `blocked` whose blocker may now be gone: an
|
|
19
23
|
`is blocked by` dependency has since closed, clarifying questions have been answered, or
|
|
20
24
|
research/waiting resolves the ambiguity that stopped it.
|
|
@@ -35,14 +39,14 @@ lifecycle state with provider-native closure and rollup state.
|
|
|
35
39
|
## Public contract
|
|
36
40
|
|
|
37
41
|
```text
|
|
38
|
-
/lisa:repair-intake <queue> [intake_mode=prd|build|both] [stale_after=
|
|
42
|
+
/lisa:repair-intake <queue> [intake_mode=prd|build|both] [stale_after=2h] [max_candidates=100] [force=true]
|
|
39
43
|
```
|
|
40
44
|
|
|
41
45
|
| Token | Meaning | Default |
|
|
42
46
|
|-------|---------|---------|
|
|
43
47
|
| `<queue>` | Same queue identifier `lisa:intake` accepts (see Source dispatch). Required. | — |
|
|
44
48
|
| `intake_mode` | `prd` \| `build` \| `both`. Only meaningful for a GitHub `org/repo` (or bare `github`) that hosts both PRD and build label namespaces. `both` is unique to repair — a repair sweep usefully covers both lifecycles in one schedule. Absent → `both` when both namespaces exist, else whichever lifecycle exists. | `both` for dual GitHub queues; otherwise infer |
|
|
45
|
-
| `stale_after` | How long with no observable activity before an in-progress item counts as stalled. Accepts `24h`, `90m`, `2d`, or `0` (treat any in-progress item as stalled — manual recovery, also the only way to resume work on a provider that exposes no reliable timestamp). Overrides config. | `
|
|
49
|
+
| `stale_after` | How long with no observable activity before an in-progress item counts as stalled. Accepts `24h`, `90m`, `2d`, or `0` (treat any in-progress item as stalled — manual recovery, also the only way to resume work on a provider that exposes no reliable timestamp). Overrides config. | `2h` |
|
|
46
50
|
| `max_candidates` | Cap on how many stuck/close-out candidates to enumerate and evaluate. Repair every materially actionable candidate within this bounded set, then stop. Overrides config. | `100` |
|
|
47
51
|
| `force` | `true` bypasses the loop-prevention backoff window (so a manual re-run re-attempts items even if their fingerprint is unchanged). It does **not** change the staleness rule — use `stale_after=0` for that. | `false` |
|
|
48
52
|
|
|
@@ -193,7 +197,7 @@ staleness — their repairability is judged on current blocker/answer state, not
|
|
|
193
197
|
1. `$ARGUMENTS` `stale_after=<dur>` (one-off override) — always wins. Parse `Nh` / `Nm` / `Nd` /
|
|
194
198
|
`0` into hours.
|
|
195
199
|
2. `.lisa.config.json` `intake.repair.staleAfterHours` (durable project default).
|
|
196
|
-
3. Built-in default: **
|
|
200
|
+
3. Built-in default: **2 hours**.
|
|
197
201
|
|
|
198
202
|
`stale_after=0` means "treat any in-progress item as stalled" — a manual full-recovery lever,
|
|
199
203
|
and the only way to resume work on a provider that exposes no reliable activity timestamp.
|
|
@@ -214,6 +218,14 @@ If ANY of these is newer than the threshold, the item is **active** → record i
|
|
|
214
218
|
skip it (read-only). For build `claimed`, an open PR with recent commits/checks is active. For
|
|
215
219
|
PRD `in_review`, a recent comment or page edit is active.
|
|
216
220
|
|
|
221
|
+
Count only **forward-progress** signals as keep-alive: new commits, a review that was just
|
|
222
|
+
requested or posted, an in-progress/queued check run, a fresh progress comment. A **settled
|
|
223
|
+
blocker state** — a failing/errored check run, `CONFLICTING` mergeability, a `CHANGES_REQUESTED`
|
|
224
|
+
review, an unaddressed CodeRabbit/reviewer change request, or a failed deployment — is NOT
|
|
225
|
+
keep-alive activity: it does not reset the staleness clock. The clock runs from the last genuine
|
|
226
|
+
progress event, so a PR that has been sitting failed/conflicted/awaiting-changes for longer than
|
|
227
|
+
`stale_after` counts as stalled and is diagnosed below.
|
|
228
|
+
|
|
217
229
|
If a provider cannot expose any reliable timestamp, do **not** auto-resume its in-progress
|
|
218
230
|
items unless the caller passed `stale_after=0`. (Dependency-cleared `blocked` repair still
|
|
219
231
|
proceeds — it is judged on blocker state, not time.)
|
|
@@ -225,10 +237,22 @@ Apply per candidate. Continue through the ordered list until every candidate ins
|
|
|
225
237
|
native close/archive/complete, re-dispatch, or refreshed note), be recorded read-only, or be
|
|
226
238
|
recorded under Errors. Do not stop after the first write; the cap is the batch boundary.
|
|
227
239
|
|
|
228
|
-
### Build `claimed` (stalled in-progress) → resume in place
|
|
240
|
+
### Build `claimed` (stalled in-progress) → diagnose blocker, else resume in place
|
|
241
|
+
|
|
242
|
+
After the staleness gate passes, **first diagnose why it stalled** by inspecting the item's PRs and
|
|
243
|
+
deploys (see "Stuck-cause diagnosis" below). A stalled build usually stalled for a concrete external
|
|
244
|
+
reason, and re-dispatching the agent at it will not fix a PR that cannot merge or a deploy that
|
|
245
|
+
failed — it just churns.
|
|
229
246
|
|
|
230
|
-
|
|
231
|
-
|
|
247
|
+
0. **Diagnose PR & deploy blockers.** If a real external blocker is found (PR cannot merge — merge
|
|
248
|
+
conflict / rebase-required / failing checks / `CHANGES_REQUESTED` / unaddressed CodeRabbit; or a
|
|
249
|
+
failed deploy), **do not dispatch the agent**. Instead file a build-ready leaf fix ticket for the
|
|
250
|
+
blocker, move this item `claimed → blocked` with an `is blocked by` link to that ticket, and
|
|
251
|
+
record it. The existing "Build `blocked` → unblock if cleared" path resumes this item on a later
|
|
252
|
+
cycle once the fix ticket is terminal — a self-healing loop. Skip the resume steps below.
|
|
253
|
+
|
|
254
|
+
If no external blocker is found, the work simply died mid-flight — run the **same per-item sequence
|
|
255
|
+
the vendor build-intake runs**, skipping the claim transition (the item is already `claimed`):
|
|
232
256
|
|
|
233
257
|
1. Dispatch the item to the vendor agent — `lisa:jira-agent` / `lisa:github-agent` /
|
|
234
258
|
`lisa:linear-agent` (matching the queue's tracker) — with the item ref. This resumes the work
|
|
@@ -246,6 +270,59 @@ skipping the claim transition (the item is already `claimed`):
|
|
|
246
270
|
> partially-built item look freshly human-approved to the next `lisa:intake` claim, and forces a
|
|
247
271
|
> two-cycle recovery. Resume in place.
|
|
248
272
|
|
|
273
|
+
#### Stuck-cause diagnosis: PR & deploy blockers
|
|
274
|
+
|
|
275
|
+
Run this for every stalled `claimed` build item **before** considering an agent re-dispatch. The
|
|
276
|
+
goal is to distinguish "work died mid-flight, just resume it" from "work is blocked on a concrete
|
|
277
|
+
external state that resuming the agent cannot fix."
|
|
278
|
+
|
|
279
|
+
**1. Find the associated PR(s) and deploy(s).** From the item's linked PRs (GitHub: remote/dev
|
|
280
|
+
links and `gh pr list --search <issue-ref>`; JIRA: dev-status / remote links; Linear: attachments
|
|
281
|
+
and git-branch links) and the deploy(s) for the resulting merge (the env-keyed `deploy.branches`
|
|
282
|
+
mapping from `config-resolution`). Read each PR with the vendor's native state, e.g. GitHub
|
|
283
|
+
`gh pr view <n> --json mergeable,mergeStateStatus,reviewDecision,statusCheckRollup,comments,reviews`.
|
|
284
|
+
|
|
285
|
+
**2. Classify as a blocker.** Treat any of these as a real external blocker:
|
|
286
|
+
|
|
287
|
+
- **Merge conflict / rebase required** — `mergeable = CONFLICTING`, or `mergeStateStatus` in
|
|
288
|
+
`DIRTY` / `BEHIND`.
|
|
289
|
+
- **Failing required checks** — `statusCheckRollup` has a `FAILURE`/`ERROR`/`TIMED_OUT` conclusion,
|
|
290
|
+
or `mergeStateStatus = UNSTABLE`/`BLOCKED` due to checks.
|
|
291
|
+
- **Change requests outstanding** — `reviewDecision = CHANGES_REQUESTED`, or unresolved CodeRabbit
|
|
292
|
+
(or other reviewer) comments that request changes and have not been addressed by a newer commit.
|
|
293
|
+
- **Branch-protection / approvals blocked** — `mergeStateStatus = BLOCKED` for a reason other than
|
|
294
|
+
a transient check still running.
|
|
295
|
+
- **Failed deploy** — the deployment for the item's merge/branch reports a failed/errored status
|
|
296
|
+
(failed deploy workflow run, failed deployment status, or the project's deploy check is red).
|
|
297
|
+
|
|
298
|
+
A check that is still **queued/in progress**, or a `CLEAN`/`HAS_HOOKS` mergeable PR with no
|
|
299
|
+
outstanding change request, is **not** a blocker — that is normal in-flight state. (Such a PR with
|
|
300
|
+
recent check/commit activity would already have been caught as `active` by the staleness gate.)
|
|
301
|
+
|
|
302
|
+
**3. On a blocker found → file a leaf fix ticket + block the item.**
|
|
303
|
+
|
|
304
|
+
1. **File one build-ready leaf fix ticket** per distinct blocker via `lisa:tracker-write` (the
|
|
305
|
+
vendor-neutral leaf writer + validation gate; never a vendor `*-write-*` skill directly),
|
|
306
|
+
`issue_type: Bug` for a failing-check/conflict/failed-deploy, `Task` for review-feedback
|
|
307
|
+
follow-up, `build_ready: true` so it auto-builds. The ticket MUST name: the blocked item + its
|
|
308
|
+
PR/deploy URL, the exact blocker (conflict / which checks failed with their logs link / which
|
|
309
|
+
change requests / which deploy run), three-audience description, and Gherkin acceptance criteria
|
|
310
|
+
for "PR is mergeable / deploy is green."
|
|
311
|
+
2. **Transition the stalled item `claimed → blocked`** and add an **`is blocked by`** link to the
|
|
312
|
+
new fix ticket (vendor-native: JIRA issue link `is blocked by`; GitHub/Linear `Blocked by:` line
|
|
313
|
+
+ label). Post a `[lisa-repair-intake]` note naming what it is blocked by and why.
|
|
314
|
+
3. **Record it** as a repair write. Do **not** dispatch the vendor agent for this item this cycle.
|
|
315
|
+
|
|
316
|
+
The item now sits in `blocked`; once the fix ticket reaches a terminal state, the **Build
|
|
317
|
+
`blocked` → unblock if cleared** path (next section) detects the cleared `is blocked by`
|
|
318
|
+
dependency and resumes the original in place — a self-healing loop.
|
|
319
|
+
|
|
320
|
+
**Idempotency.** Before filing, check for an **open** fix ticket already carrying the marker
|
|
321
|
+
`[lisa-repair-intake] blocker:<item-ref>/<blocker-key>` (blocker-key is a stable slug of the
|
|
322
|
+
blocker, e.g. `pr-1234/merge-conflict` or `pr-1234/checks-failing`). If one exists, reference it
|
|
323
|
+
and ensure the `is blocked by` link is present rather than creating a duplicate. Honor the backoff
|
|
324
|
+
window and state fingerprint (Loop prevention) so re-runs over the same unchanged blocker are no-ops.
|
|
325
|
+
|
|
249
326
|
### Build `blocked` → re-evaluate, unblock if cleared
|
|
250
327
|
|
|
251
328
|
1. Read the block reason and dependencies (see Dependency clearing).
|
|
@@ -399,7 +476,7 @@ cron tick.
|
|
|
399
476
|
- Before writing a note or re-attempting a `blocked` item, compute the current fingerprint. If
|
|
400
477
|
an identical fingerprint was already posted within the **backoff window**, skip the item
|
|
401
478
|
silently (record as `still_blocked` / `active`, no write).
|
|
402
|
-
- Backoff window default = `stale_after` (
|
|
479
|
+
- Backoff window default = `stale_after` (2h). `force=true` bypasses backoff for a manual run.
|
|
403
480
|
- A *changed* fingerprint (new blocker state, new answers, new verdict) always warrants a fresh
|
|
404
481
|
note + re-attempt — backoff suppresses only no-op repeats.
|
|
405
482
|
|
|
@@ -0,0 +1,7 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: "Explore gaps in the automated Playwright/e2e suite: inventory the app's routes and existing tests, find routes with no coverage or flows tested only on the happy path (missing error, permission, empty, loading, and edge cases), confirm each in the running app, and file one build-ready missing-test ticket per gap via lisa:tracker-write. The optional ready flag (default true) controls build-ready vs backlog. For human usability issues, use exploratory-qa instead."
|
|
3
|
+
allowed-tools: ["Skill"]
|
|
4
|
+
argument-hint: "[target-url | env] [ready=true|false]"
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
Use the /lisa-expo:e2e-coverage-gaps skill to inventory the app's routes and the existing Playwright suite, find uncovered and happy-path-only paths, confirm each gap in the running app, and file one build-ready missing-test ticket per gap via lisa:tracker-write (build-ready per the ready flag, default true). For human usability/experience findings, use /lisa-expo:exploratory-qa. $ARGUMENTS
|
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
|
-
description: "Run a
|
|
2
|
+
description: "Run a first-time-user exploratory QA walkthrough: experience the app like a brand-new human user, clicking through to find anything confusing, broken, or hard to understand (machine-style labels, slow or unclear loads, cramped or cut-off UI, inconsistent UX, awkward scroll behavior) across all breakpoints, and file each finding (bug or usability issue) as a tracked work item via lisa:tracker-write. The optional ready flag marks tickets build-ready (auto-picked-up by lisa:intake) or leaves them in the backlog for human triage (default). For gaps in the automated Playwright suite, use e2e-coverage-gaps instead."
|
|
3
3
|
allowed-tools: ["Skill"]
|
|
4
4
|
argument-hint: "[target-url | env] [ready=true|false]"
|
|
5
5
|
---
|
|
6
6
|
|
|
7
|
-
Use the /lisa-expo:exploratory-qa skill to
|
|
7
|
+
Use the /lisa-expo:exploratory-qa skill to experience the app like a brand-new first-time user — landing cold on the home page and clicking through to find anything confusing, broken, or hard to understand across all breakpoints — and file each finding (bugs, usability/clarity issues) as a tracked work item via lisa:tracker-write, build-ready or in triage per the ready flag (default: triage). For automated Playwright coverage gaps, use /lisa-expo:e2e-coverage-gaps. $ARGUMENTS
|
|
@@ -0,0 +1,105 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: e2e-coverage-gaps
|
|
3
|
+
description: Playwright/e2e coverage-gap explorer that FEEDS THE LIFECYCLE. Use when asked to find paths the automated end-to-end suite does NOT cover — routes with no test at all, or flows that are only happy-path tested (missing error, validation, permission, empty, loading, and edge states). It inventories the app's routes and the existing Playwright tests, explores the running app to confirm each uncovered or under-covered path is real and reachable, and files one build-ready missing-test ticket per gap via lisa:tracker-write. For human usability/experience issues (confusing, cramped, or broken UI), use the exploratory-qa skill instead.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# E2E Coverage Gaps
|
|
7
|
+
|
|
8
|
+
## Overview
|
|
9
|
+
|
|
10
|
+
Find where the automated end-to-end (Playwright) suite is **blind**: routes with no test at all, and
|
|
11
|
+
flows that only assert the **happy path** while ignoring error, permission, empty, loading, and edge
|
|
12
|
+
cases. Inventory the app's routes and the existing tests, explore the running app to confirm each gap
|
|
13
|
+
is real and reachable, then file each gap as a **build-ready missing-test work item** so it enters the
|
|
14
|
+
Lisa lifecycle.
|
|
15
|
+
|
|
16
|
+
This skill is purely about **automated-coverage gaps**. It does not judge whether the UI is confusing
|
|
17
|
+
or pretty — for human usability findings, use the `exploratory-qa` skill.
|
|
18
|
+
|
|
19
|
+
## Parameters
|
|
20
|
+
|
|
21
|
+
- **`target-url | env`** (first positional) — the app to inventory and explore.
|
|
22
|
+
- **`ready=true|false`** — build-ready state for the missing-test tickets (**default `true`**). Adding
|
|
23
|
+
missing coverage is safe to queue, so these default to build-ready; pass `ready=false` to leave them
|
|
24
|
+
in the backlog for human triage.
|
|
25
|
+
|
|
26
|
+
## Core Workflow
|
|
27
|
+
|
|
28
|
+
### 1. Establish Scope
|
|
29
|
+
|
|
30
|
+
- Identify the target environment, account type, and browser requirement, and read the `ready` flag
|
|
31
|
+
(default `true`).
|
|
32
|
+
- **Confirm the tracker is configured.** Gaps are filed as tickets, so read `tracker` from
|
|
33
|
+
`.lisa.config.json` (local overrides global). If it is unset, stop and report that the tracker must
|
|
34
|
+
be configured (via `/lisa:setup:jira` / `:github` / `:linear`) before gaps can be filed.
|
|
35
|
+
|
|
36
|
+
### 2. Inventory App Routes
|
|
37
|
+
|
|
38
|
+
- Enumerate every route/screen from the project's routing source — filesystem routes, the router
|
|
39
|
+
config, navigation definitions, or a generated sitemap.
|
|
40
|
+
- For each route capture: its path + params, the auth/role it requires, and the primary user actions
|
|
41
|
+
reachable from it (forms, mutations, filters, flows).
|
|
42
|
+
|
|
43
|
+
### 3. Inventory Existing Playwright Coverage
|
|
44
|
+
|
|
45
|
+
- Inspect the Playwright config, auth/setup projects, fixtures, constants, selectors, spec directories,
|
|
46
|
+
skipped/TODO/flaky tests, retries, and viewport/device projects.
|
|
47
|
+
- For each spec, record which route(s)/flow(s) it exercises and whether it asserts **behavior** or just
|
|
48
|
+
**presence**.
|
|
49
|
+
- Use existing test helpers/selectors when exploring — they reveal intended flows and stable hooks.
|
|
50
|
+
|
|
51
|
+
### 4. Compute the Gap Matrix
|
|
52
|
+
|
|
53
|
+
Map routes/flows against existing coverage and classify each gap:
|
|
54
|
+
|
|
55
|
+
- **Uncovered route/flow:** no test touches it at all.
|
|
56
|
+
- **Happy-path-only:** the success case is tested but the non-happy paths are not — missing
|
|
57
|
+
error/validation, permission/denied, empty/zero-state, loading/slow, and boundary/edge scenarios.
|
|
58
|
+
- **Assertion-thin:** a test exists but asserts presence rather than behavior/outcome.
|
|
59
|
+
- **Skipped/TODO/flaky:** coverage exists but is disabled or unreliable.
|
|
60
|
+
- **Breakpoint gap:** a flow is only tested at a single viewport when behavior changes across
|
|
61
|
+
breakpoints.
|
|
62
|
+
|
|
63
|
+
### 5. Explore to Confirm
|
|
64
|
+
|
|
65
|
+
- Navigate each candidate gap in the running app to confirm it is real, reachable, and worth a test.
|
|
66
|
+
Discard gaps that aren't actually reachable or are intentionally out of scope.
|
|
67
|
+
|
|
68
|
+
### 6. File One Ticket Per Gap
|
|
69
|
+
|
|
70
|
+
Each confirmed gap becomes a leaf **missing-test** work item created via `lisa:tracker-write` (the
|
|
71
|
+
vendor-neutral writer — it dispatches to the configured tracker and runs the validation gate; never
|
|
72
|
+
call a vendor `*-write-*` skill directly), `issue_type: Task`, **build-ready per the `ready` flag
|
|
73
|
+
(default `true`)**. Pass `build_ready` explicitly on every create. Each ticket MUST specify:
|
|
74
|
+
|
|
75
|
+
- The **route/flow** and the exact **user behavior the test must assert**.
|
|
76
|
+
- **Which scenario is missing** (uncovered vs which non-happy path: error / validation / permission /
|
|
77
|
+
empty / loading / edge / breakpoint).
|
|
78
|
+
- The **stable selector / fixture / flow** to use — concrete and automatable.
|
|
79
|
+
- **Three-audience description** + **Gherkin acceptance criteria** describing the test to add.
|
|
80
|
+
|
|
81
|
+
### Idempotency — don't spam duplicates
|
|
82
|
+
|
|
83
|
+
Before creating a ticket, search the tracker for an **open** ticket carrying a stable marker
|
|
84
|
+
`[lisa-e2e-coverage-gaps] <gap-key>` in its body (the `<gap-key>` is a stable slug of route + missing
|
|
85
|
+
scenario, e.g. `checkout/payment-declined` or `dashboard/empty-state@mobile`). If one exists, reference
|
|
86
|
+
it instead of duplicating. **Match by the marker, never by title.** A *closed* prior ticket does not
|
|
87
|
+
suppress a genuine new gap.
|
|
88
|
+
|
|
89
|
+
## Output
|
|
90
|
+
|
|
91
|
+
No report file. Emit a concise in-session summary:
|
|
92
|
+
|
|
93
|
+
- **Route inventory:** count of routes/screens discovered.
|
|
94
|
+
- **Existing coverage:** strengths and thin areas (skipped/flaky, assertion-thin, single-viewport).
|
|
95
|
+
- **Gap matrix:** uncovered vs happy-path-only vs breakpoint gaps.
|
|
96
|
+
- **Tickets filed:** each gap with its created/referenced ticket ref and build-ready state.
|
|
97
|
+
|
|
98
|
+
## Quality Bar
|
|
99
|
+
|
|
100
|
+
- Distinguish **no coverage** from **happy-path-only** — both are gaps, but the ticket must say which.
|
|
101
|
+
- Every ticket must stand alone for an implementer who was not in the session and must be concretely
|
|
102
|
+
automatable (named route, behavior, and stable hook).
|
|
103
|
+
- Do not refile gaps already covered or already ticketed (marker check).
|
|
104
|
+
- This skill is about automated coverage only — route human usability issues to `exploratory-qa`.
|
|
105
|
+
- Preserve unrelated repo changes.
|