@codyswann/lisa 2.110.0 → 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/github-write-prd/SKILL.md +9 -5
- package/plugins/lisa/skills/prd-source-write/SKILL.md +17 -0
- package/plugins/lisa/skills/project-ideation/SKILL.md +7 -1
- package/plugins/lisa/skills/repair-intake/SKILL.md +86 -9
- package/plugins/lisa/skills/research/SKILL.md +11 -2
- 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/github-write-prd/SKILL.md +9 -5
- package/plugins/src/base/skills/prd-source-write/SKILL.md +17 -0
- package/plugins/src/base/skills/project-ideation/SKILL.md +7 -1
- package/plugins/src/base/skills/repair-intake/SKILL.md +86 -9
- package/plugins/src/base/skills/research/SKILL.md +11 -2
- 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`)
|
|
@@ -10,7 +10,8 @@ Create (or update) a PRD issue in the configured source repo. Invoked by `lisa:p
|
|
|
10
10
|
when `source = github`; do not call directly from a vendor-neutral caller.
|
|
11
11
|
|
|
12
12
|
`$ARGUMENTS` carries the `lisa:prd-source-write` spec: `title`, `body` (full PRD markdown),
|
|
13
|
-
`initial_role` (`draft` | `ready`, default `draft`), `dedupe_key`, `marker`, optional `source_ref
|
|
13
|
+
`initial_role` (`draft` | `ready`, default `draft`), `dedupe_key`, `marker`, optional `source_ref`,
|
|
14
|
+
and optional `ideation_ledger_payload` from `lisa:project-ideation` via `lisa:research`.
|
|
14
15
|
|
|
15
16
|
## Phase 1 — Resolve repo and PRD lifecycle labels
|
|
16
17
|
|
|
@@ -64,10 +65,13 @@ section, preserve it verbatim unless the caller intentionally supplied an update
|
|
|
64
65
|
use the shared `usage-accounting` serializer/merge path rather than hand-editing ledger rows.
|
|
65
66
|
|
|
66
67
|
**Exploratory ideation run ledger (both paths).** When the write was initiated by
|
|
67
|
-
`lisa:project-ideation
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
|
|
68
|
+
`lisa:project-ideation`, carries a project-ideation marker, or includes
|
|
69
|
+
`ideation_ledger_payload`, persist a managed `## Exploratory Ideation Run Ledger` section in the PRD
|
|
70
|
+
body. Prefer the managed section over a comment so the PRD itself remains the operator's source of
|
|
71
|
+
truth; use a managed comment only if the body cannot be updated. Populate the fields from
|
|
72
|
+
`ideation_ledger_payload` when present, falling back to `marker`, `initial_role`, repo config, and
|
|
73
|
+
runtime metadata only for missing fields. Keep one managed section by replacing the content between
|
|
74
|
+
stable markers:
|
|
71
75
|
|
|
72
76
|
```markdown
|
|
73
77
|
## Exploratory Ideation Run Ledger
|
|
@@ -27,6 +27,17 @@ dedupe_key: "<stable-key>" # e.g. project-ideation's idea key
|
|
|
27
27
|
marker: "[lisa-project-ideation] idea=<stable-key>" # embedded in the PRD body for dedupe
|
|
28
28
|
origin: { tool: project-ideation | research | manual }
|
|
29
29
|
source_ref: "<optional existing PRD ref to force an update>"
|
|
30
|
+
ideation_ledger_payload: # optional; forwarded unchanged to the vendor writer
|
|
31
|
+
selected_marker: "<same value as marker>"
|
|
32
|
+
automation_id: "<Codex/Claude automation id or unavailable>"
|
|
33
|
+
automation_memory_path: "<path or unavailable>"
|
|
34
|
+
repo: "<org>/<repo or detected repo identity>"
|
|
35
|
+
prd_ready: true|false
|
|
36
|
+
persona_names: ["<derived persona name>"]
|
|
37
|
+
persona_evidence_refs: ["<file/doc/table/release ref>"]
|
|
38
|
+
selected_idea: "<selected idea title/key>"
|
|
39
|
+
rejected_overlap_candidates: ["<issue refs/titles considered and rejected>"]
|
|
40
|
+
expected_empirical_verification_artifact: "<artifact ref or unavailable>"
|
|
30
41
|
```
|
|
31
42
|
|
|
32
43
|
`initial_role` semantics are uniform across vendors (the role STRINGS resolve per vendor from
|
|
@@ -72,6 +83,10 @@ prior PRD-source-write behavior to preserve, so omitted means `draft`.
|
|
|
72
83
|
the applied role (`draft`/`ready`), the dedupe marker, and whether it was created or reused.
|
|
73
84
|
Downstream callers (research, project-ideation) parse this — do not paraphrase.
|
|
74
85
|
|
|
86
|
+
When `ideation_ledger_payload` is present, this shim still does not render or interpret it. Forward
|
|
87
|
+
the object verbatim to the selected vendor writer so source-specific rendering remains behind the
|
|
88
|
+
configured writer and the dispatch layer never bypasses source selection.
|
|
89
|
+
|
|
75
90
|
## Rules
|
|
76
91
|
|
|
77
92
|
- Never bypass dispatch — a vendor-neutral caller calling a `*-write-prd` skill directly defeats the
|
|
@@ -80,5 +95,7 @@ prior PRD-source-write behavior to preserve, so omitted means `draft`.
|
|
|
80
95
|
and fallback behavior belongs in the vendor writers and follows the `usage-accounting` contract.
|
|
81
96
|
- Never accept a source outside `{notion, confluence, github, linear}`. `jira` and `file` fail loudly.
|
|
82
97
|
- Never mutate the spec between layers. The vendor writers define their own create/dedupe contract.
|
|
98
|
+
- Never drop, rename, or vendor-render `ideation_ledger_payload`; it is a pass-through payload for
|
|
99
|
+
the configured writer.
|
|
83
100
|
- Never invent a PRD lifecycle role string — resolve every role from `config-resolution` per vendor.
|
|
84
101
|
- Idempotency is the vendor writer's job (marker search before create); this shim only routes.
|
|
@@ -155,7 +155,13 @@ For each idea in the creation set, invoke `/lisa:research` with:
|
|
|
155
155
|
grounding and the empirical verification plan),
|
|
156
156
|
- `prd_ready` (this run's flag — `lisa:research` maps it to draft vs prd-ready),
|
|
157
157
|
- a stable **dedupe marker** (see below) so a re-run references the existing PRD instead of creating
|
|
158
|
-
a duplicate
|
|
158
|
+
a duplicate,
|
|
159
|
+
- a structured `ideation_ledger_payload` handoff containing the selected marker, automation id and
|
|
160
|
+
memory path when available, persona names, persona evidence references, rejected overlap
|
|
161
|
+
candidates, repo identity, `prd_ready`, selected idea title/key, and the expected empirical
|
|
162
|
+
verification artifact. This payload is the only ideation-run metadata channel between
|
|
163
|
+
`project-ideation`, `research`, `prd-source-write`, and the vendor writer; keep GitHub-specific
|
|
164
|
+
rendering out of this skill.
|
|
159
165
|
|
|
160
166
|
`lisa:research` synthesizes the PRD and creates it in the configured source via
|
|
161
167
|
`lisa:prd-source-write`. `project-ideation` never writes to the source directly — it delegates, so
|
|
@@ -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
|
|
|
@@ -16,6 +16,12 @@ Produce a PRD for the problem in `$ARGUMENTS`, then create it in the configured
|
|
|
16
16
|
- `dedupe_key` / `marker` (optional) — a stable dedupe marker (e.g. supplied by
|
|
17
17
|
`lisa:project-ideation`) embedded in the created PRD so re-runs reference the existing PRD rather
|
|
18
18
|
than creating a duplicate.
|
|
19
|
+
- `ideation_ledger_payload` (optional, required when invoked by `lisa:project-ideation`) — a
|
|
20
|
+
structured metadata object to forward unchanged to `lisa:prd-source-write`. It carries the
|
|
21
|
+
selected marker, automation id/path when available, persona names, persona evidence references,
|
|
22
|
+
rejected overlap candidates, repo identity, `prd_ready`, selected idea title/key, and expected
|
|
23
|
+
empirical verification artifact. `research` may use these fields to inform the PRD body, but must
|
|
24
|
+
not discard, rename, or vendor-render them.
|
|
19
25
|
|
|
20
26
|
## Orchestration: agent team
|
|
21
27
|
|
|
@@ -55,5 +61,8 @@ source — there is no loose document artifact.** Before handing the synthesized
|
|
|
55
61
|
`lisa:usage-accounting` so the PRD body carries the canonical `## Lisa Usage` ledger from creation
|
|
56
62
|
time onward. If the runtime does not expose trustworthy usage, the direct entry must still be
|
|
57
63
|
written with `source: unavailable` and nullable token/cost fields rather than silently omitting the
|
|
58
|
-
Research row.
|
|
59
|
-
|
|
64
|
+
Research row. If the call includes `ideation_ledger_payload`, pass that object through in the
|
|
65
|
+
`lisa:prd-source-write` spec unchanged alongside `marker`, `dedupe_key`, and `initial_role`; this is
|
|
66
|
+
the vendor-neutral handoff that lets the configured writer render an auditable run ledger without
|
|
67
|
+
`research` bypassing source selection. A `source` must be configured; if it is not, stop and report
|
|
68
|
+
it rather than emitting a document. The Plan flow consumes the created PRD next.
|
|
@@ -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.
|