@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
|
@@ -1,138 +1,145 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: exploratory-qa
|
|
3
|
-
description:
|
|
3
|
+
description: First-time-user exploratory QA walkthrough for web apps that FEEDS THE LIFECYCLE. Use when asked to experience an app the way a brand-new human user would — landing cold on the home page and clicking through to find anything confusing, broken, or hard to understand (machine-style labels, slow or unclear loads, cramped or cut-off UI, inconsistent/non-standard UX, awkward scroll behavior, unclear affordances) across all breakpoints. Instead of writing a report file, it files every finding as a tracked work item via lisa:tracker-write (bugs and usability/UX issues). A `ready` parameter controls whether those tickets are created build-ready (auto-picked-up by lisa:intake) or left in the backlog for human triage (default). For gaps in the automated Playwright test suite, use the e2e-coverage-gaps skill instead.
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Exploratory QA
|
|
7
7
|
|
|
8
8
|
## Overview
|
|
9
9
|
|
|
10
|
-
|
|
10
|
+
Experience the app the way a **brand-new human user** would: land cold on the home page with no prior
|
|
11
|
+
knowledge, then click through and actually try to use it — just like a real person. The goal is to
|
|
12
|
+
surface anything **confusing, broken, or hard to understand**, and to do so at **every breakpoint**.
|
|
13
|
+
|
|
14
|
+
This is a usability/experience pass, **not** a test-coverage audit. It does not look at the Playwright
|
|
15
|
+
suite or hunt for coverage gaps — for that, use the `e2e-coverage-gaps` skill. Here, every finding is
|
|
16
|
+
filed as a tracked work item so it enters the Lisa lifecycle — no static report file.
|
|
11
17
|
|
|
12
18
|
## Parameters
|
|
13
19
|
|
|
14
|
-
- **`target-url | env`** (first positional) — what to
|
|
15
|
-
- **`ready=true|false`** — the build-ready state for the
|
|
20
|
+
- **`target-url | env`** (first positional) — what to explore.
|
|
21
|
+
- **`ready=true|false`** — the build-ready state for the tickets this pass creates.
|
|
16
22
|
- `ready=true` → created build-ready, so `lisa:intake` / the build-intake scanner auto-picks them up.
|
|
17
|
-
- `ready=false` (**default**) → created in the backlog (not build-ready) for a human to review and
|
|
18
|
-
|
|
19
|
-
|
|
20
|
-
`ready` maps directly to the `build_ready` write-control input on `lisa:tracker-write`.
|
|
23
|
+
- `ready=false` (**default**) → created in the backlog (not build-ready) for a human to review and
|
|
24
|
+
promote into the queue.
|
|
21
25
|
|
|
22
26
|
## Core Workflow
|
|
23
27
|
|
|
24
28
|
### 1. Establish Scope
|
|
25
29
|
|
|
26
|
-
- Identify the target environment, account type, browser requirement, and the `ready` flag
|
|
27
|
-
|
|
28
|
-
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
-
|
|
49
|
-
|
|
50
|
-
|
|
51
|
-
|
|
52
|
-
|
|
53
|
-
|
|
54
|
-
|
|
55
|
-
- **
|
|
56
|
-
|
|
57
|
-
-
|
|
58
|
-
- **Load
|
|
59
|
-
|
|
60
|
-
- **
|
|
61
|
-
|
|
62
|
-
- **
|
|
30
|
+
- Identify the target environment, account type, and browser requirement, and read the `ready` flag
|
|
31
|
+
(default `false`).
|
|
32
|
+
- **Confirm the tracker is configured.** Findings 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 exploratory QA can file
|
|
35
|
+
findings — do not silently fall back to a report file.
|
|
36
|
+
- If credentials, tenant, or seed data are missing and cannot be discovered safely, ask one concise
|
|
37
|
+
clarifying question.
|
|
38
|
+
- Treat production-like environments conservatively. Do not mutate production data unless the user
|
|
39
|
+
explicitly approves it. Prefer a test user, dev/staging environment, or isolated seeded account.
|
|
40
|
+
|
|
41
|
+
### 2. Arrive Cold
|
|
42
|
+
|
|
43
|
+
- Start at the home/landing page with **no prior knowledge of the app**. Do **not** pre-read the
|
|
44
|
+
codebase to learn the intended flows — discover them the way a user would, by looking and clicking.
|
|
45
|
+
- Form a first impression: is it obvious what this app is, what to do first, and where to go next?
|
|
46
|
+
|
|
47
|
+
### 3. Use It Like a Human
|
|
48
|
+
|
|
49
|
+
Click through the visible paths and actually attempt real tasks — a first-time user explores, makes
|
|
50
|
+
mistakes, and tries the obvious thing. Cover at least these dimensions unless the user narrows scope:
|
|
51
|
+
|
|
52
|
+
- **Comprehension & labeling:** machine-style or developer labels shown to users (raw IDs, enum keys,
|
|
53
|
+
`snake_case`, `null`/`undefined`, untranslated i18n keys), jargon, unclear button/menu names, icons
|
|
54
|
+
with no discernible meaning.
|
|
55
|
+
- **Navigation clarity:** is it obvious how to get somewhere and back? Dead ends, hidden entry points,
|
|
56
|
+
surprising redirects, broken links, no clear "home".
|
|
57
|
+
- **Visual/layout quality:** cut-off or truncated text, overlap, cramped/crowded density, offscreen or
|
|
58
|
+
unreachable controls, accidental horizontal scroll, awkward empty space.
|
|
59
|
+
- **Consistency / standard UX:** components, spacing, button styles, terminology, and interaction
|
|
60
|
+
patterns should be consistent across the app and follow common conventions. Flag anything
|
|
61
|
+
non-standard or that differs screen-to-screen.
|
|
62
|
+
- **Load & responsiveness:** long or unclear load times, blank screens, spinners / `Loading...` /
|
|
63
|
+
`Connecting...` with no progress, anything that feels slow or janky.
|
|
64
|
+
- **Scroll behavior:** unexpected scroll position, scroll jumps, nested or locked scroll, sticky
|
|
65
|
+
elements that cover content, content that cannot be reached.
|
|
66
|
+
- **Behavior correctness:** does the obvious action do what a user expects? Confusing errors, silent
|
|
67
|
+
failures, disabled controls with no explanation, state that does not persist.
|
|
68
|
+
- **Affordance clarity:** can the user tell what is clickable, required, in-progress, or complete?
|
|
69
|
+
|
|
70
|
+
### 4. Cover All Breakpoints
|
|
71
|
+
|
|
72
|
+
- Discover breakpoints from the app (design tokens, CSS, responsive layout changes) when possible; if
|
|
73
|
+
unknown, use a practical baseline: phone, tablet/narrow, desktop, plus any app-specific cutoff.
|
|
74
|
+
- At each breakpoint, walk the key paths again and confirm the experience holds: expected
|
|
75
|
+
shell/navigation variant, critical controls visible and reachable, no unintended horizontal
|
|
76
|
+
overflow, intentional scroll containers still usable, nothing cut off or crowded.
|
|
77
|
+
|
|
78
|
+
### 5. Watch Load & Latency
|
|
79
|
+
|
|
80
|
+
- Notice time to first meaningful content and time spent in blank/loading/spinner/connecting states.
|
|
81
|
+
- A page can be technically interactive but still visually incomplete — note that.
|
|
82
|
+
- Treat long waits without clear progress, error, retry, or cancellation as findings. Use practical
|
|
83
|
+
labels (noticeable, slow, unacceptable) and include observed durations when available.
|
|
63
84
|
|
|
64
85
|
## Mutation Discipline
|
|
65
86
|
|
|
66
|
-
|
|
67
|
-
|
|
68
|
-
- Prefer high-value user workflows: create/edit/delete records, lists, boards, tags, notes, comments, scenarios, uploads, messages, settings, invitations, or assignments.
|
|
69
|
-
- Use unique names with a clear prefix such as `qa-`, `pw-`, or `codex-`.
|
|
70
|
-
- Before mutating, identify the cleanup path. After mutating, make a best effort to clean up through the UI, then verify cleanup.
|
|
71
|
-
- If UI cleanup is unavailable, file that as a product/test gap (a finding — see below). Use documented API cleanup only if appropriate for the project and account.
|
|
72
|
-
- Record all mutations performed, cleanup attempts, and residue left behind.
|
|
73
|
-
- Avoid destructive bulk actions unless the user explicitly asks or the test account is clearly disposable.
|
|
74
|
-
|
|
75
|
-
## Breakpoint Strategy
|
|
76
|
-
|
|
77
|
-
- Discover breakpoints from the codebase, design tokens, CSS, docs, or Playwright constants when possible.
|
|
78
|
-
- Test each named breakpoint and the boundary on both sides of important cutoffs.
|
|
79
|
-
- If breakpoints are unknown, use a practical baseline: phone, tablet/narrow, desktop, and any app-specific cutoff discovered during research.
|
|
80
|
-
- For each breakpoint, assert both user-visible behavior and automation-relevant state:
|
|
81
|
-
- Expected shell/navigation variant.
|
|
82
|
-
- Route-specific loaded content.
|
|
83
|
-
- Critical controls visible and reachable.
|
|
84
|
-
- No unintended document-level horizontal overflow.
|
|
85
|
-
- Intentional scroll containers remain usable.
|
|
86
|
-
- Hidden/off-canvas UI is not exposed as active content.
|
|
87
|
+
A real first-time user creates, edits, and deletes things — exercise those flows when the environment
|
|
88
|
+
is safe.
|
|
87
89
|
|
|
88
|
-
|
|
89
|
-
|
|
90
|
-
|
|
91
|
-
|
|
92
|
-
-
|
|
93
|
-
- Note when a page is technically interactive but still visually incomplete.
|
|
94
|
-
- Treat long waits without clear progress, error, retry, or cancellation as bugs or test gaps.
|
|
95
|
-
- Do not overfit exact milliseconds unless the project has defined budgets. Use practical labels such as noticeable, slow, or unacceptable and include observed durations when available.
|
|
90
|
+
- Use unique names with a clear prefix such as `qa-` or `codex-`.
|
|
91
|
+
- Before mutating, identify the cleanup path. After mutating, make a best effort to clean up through
|
|
92
|
+
the UI, then verify cleanup. If UI cleanup is unavailable, that itself is a usability finding.
|
|
93
|
+
- Avoid destructive bulk actions unless the user explicitly asks or the account is clearly disposable.
|
|
94
|
+
- Record all mutations performed, cleanup attempts, and any residue left behind.
|
|
96
95
|
|
|
97
96
|
## Filing findings as tracked work
|
|
98
97
|
|
|
99
|
-
This skill does **not** write a report file. Every finding becomes a **leaf work item** created via
|
|
98
|
+
This skill does **not** write a report file. Every finding becomes a **leaf work item** created via
|
|
99
|
+
`lisa:tracker-write` (the vendor-neutral writer — it dispatches to the configured tracker and runs the
|
|
100
|
+
validation gate; never call a vendor `*-write-*` skill directly). Map each finding to a type:
|
|
100
101
|
|
|
101
102
|
| Finding | `issue_type` | `build_ready` |
|
|
102
103
|
|---------|--------------|---------------|
|
|
103
|
-
| User-visible **bug** | `Bug` | the `ready` flag (default `false`) |
|
|
104
|
-
| **Usability / UX issue**
|
|
105
|
-
| **Missing Playwright test** (coverage gap) | `Task` | **`true` (always)** |
|
|
106
|
-
|
|
107
|
-
Each finding is a flat leaf (no children), so `build_ready` applies directly. Pass it explicitly on every create — `build_ready: <ready flag>` for bugs and suggestions, `build_ready: true` for missing tests.
|
|
104
|
+
| User-visible **bug** (broken behavior) | `Bug` | the `ready` flag (default `false`) |
|
|
105
|
+
| **Usability / UX / clarity issue** | `Improvement` | the `ready` flag (default `false`) |
|
|
108
106
|
|
|
109
|
-
Each
|
|
107
|
+
Each finding is a flat leaf (no children), so `build_ready` applies directly — pass it explicitly on
|
|
108
|
+
every create. Each ticket MUST be a complete spec (the validator rejects thin tickets):
|
|
110
109
|
|
|
111
110
|
- **Three-audience description** (context / business value, technical approach, stakeholder impact).
|
|
112
|
-
- **For a bug:** exact reproduction steps, observed-vs-expected, the env / account / breakpoint it
|
|
113
|
-
|
|
114
|
-
- **For a
|
|
115
|
-
|
|
111
|
+
- **For a bug:** exact reproduction steps, observed-vs-expected, the env / account / breakpoint it
|
|
112
|
+
occurred at, and console/network evidence.
|
|
113
|
+
- **For a usability issue:** the observed friction (what was confusing, cramped, inconsistent, or hard
|
|
114
|
+
to understand), who it affects, **where** (route + breakpoint), and the proposed improvement.
|
|
115
|
+
- **Gherkin acceptance criteria** describing the fixed behavior.
|
|
116
116
|
|
|
117
117
|
### Idempotency — don't spam duplicates
|
|
118
118
|
|
|
119
|
-
Re-running a
|
|
119
|
+
Re-running a pass must not refile the same finding. Before creating a ticket, search the tracker for an
|
|
120
|
+
**open** ticket carrying a stable marker `[lisa-exploratory-qa] <finding-key>` in its body (the
|
|
121
|
+
`<finding-key>` is a stable slug of surface + symptom, e.g. `settings-modal/horizontal-overflow@tablet`).
|
|
122
|
+
If one exists, reference/update it instead of creating a duplicate; only create when none exists.
|
|
123
|
+
**Match by the marker, never by title** (titles get edited). A *closed* prior ticket does not suppress a
|
|
124
|
+
new one — a recurrence after a fix is a genuine regression.
|
|
120
125
|
|
|
121
126
|
## Output
|
|
122
127
|
|
|
123
128
|
No report file. Emit a concise in-session summary:
|
|
124
129
|
|
|
125
130
|
- **Scope:** target URL/env, browser/tool, account type, build/version if visible, date.
|
|
126
|
-
- **
|
|
127
|
-
- **Findings filed**, bucketed by type — bugs, usability
|
|
131
|
+
- **First impression:** could a new user tell what the app is and what to do first?
|
|
132
|
+
- **Findings filed**, bucketed by type — bugs, usability/clarity issues — each with its **created or
|
|
133
|
+
referenced ticket ref** and its **build-ready state** (`ready` vs `triage/backlog`).
|
|
128
134
|
- **Observed but not filed:** anything noticed but intentionally not ticketed, with why.
|
|
129
135
|
|
|
130
136
|
## Quality Bar
|
|
131
137
|
|
|
132
|
-
-
|
|
133
|
-
|
|
134
|
-
- Prefer concrete, reproducible findings. Every ticket must stand alone for an implementer who was not
|
|
138
|
+
- Explore as a true first-time user — judge clarity, not whether you (who can read the code) can figure
|
|
139
|
+
it out.
|
|
140
|
+
- Prefer concrete, reproducible findings. Every ticket must stand alone for an implementer who was not
|
|
141
|
+
in the session.
|
|
135
142
|
- Do not claim cleanup succeeded unless verified.
|
|
136
|
-
-
|
|
137
|
-
-
|
|
143
|
+
- File tickets per the `ready` flag (default: backlog for human triage).
|
|
144
|
+
- This skill is about the human experience only — route automated-coverage gaps to `e2e-coverage-gaps`.
|
|
138
145
|
- Preserve unrelated repo changes.
|
|
@@ -1,4 +1,4 @@
|
|
|
1
1
|
display_name: "Exploratory QA"
|
|
2
|
-
short_description: "
|
|
2
|
+
short_description: "First-time-user exploratory QA walkthrough for web apps that FEEDS THE LIFECYCLE"
|
|
3
3
|
default_prompt:
|
|
4
|
-
- "Use $exploratory-qa:
|
|
4
|
+
- "Use $exploratory-qa: First-time-user exploratory QA walkthrough for web apps that FEEDS THE LIFECYCLE."
|
|
@@ -23,6 +23,18 @@ Repository mode: **{{mode}}**.
|
|
|
23
23
|
- `{{wikiRoot}}/staff/` — digital-staff role pages (optional).
|
|
24
24
|
- Synthesis categories: {{categories}}.
|
|
25
25
|
|
|
26
|
+
## Answering questions (query-first)
|
|
27
|
+
When you need to answer a question about this project — its code, architecture, decisions, domain
|
|
28
|
+
concepts, conventions, or documentation — reach for the wiki **first**: run `/query "<question>"`
|
|
29
|
+
(Codex: `$lisa-wiki-query`) before ad-hoc code search, grep, or a web lookup. The wiki is the
|
|
30
|
+
curated, cited source of truth for this project, so a query is usually faster and more accurate than
|
|
31
|
+
re-deriving the answer from raw files.
|
|
32
|
+
|
|
33
|
+
This is the **primary** method, not the only one. If `/query` returns nothing relevant, the answer
|
|
34
|
+
is not in the wiki yet, or the question is about live/uncommitted state, fall back to reading the
|
|
35
|
+
code, browsing `{{wikiRoot}}/index.md`, or other tools — and consider `/ingest` to capture any
|
|
36
|
+
durable knowledge you had to reconstruct so the next query succeeds.
|
|
37
|
+
|
|
26
38
|
## Ordered ingestion pipeline (never reorder)
|
|
27
39
|
`source note → synthesis → index → log → verify → state → commit/PR`.
|
|
28
40
|
State advances **only** after source notes + synthesis + index + log + verification all pass.
|
|
@@ -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
|