@agentshouse/kit 0.2.1-ce145e9aadbf6 → 0.2.1-cf6326dfa5c3e
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/README.md +32 -31
- package/dist/acp.js +46 -103
- package/dist/agents.js +151 -41
- package/dist/api.js +34 -2
- package/dist/app-git.js +48 -0
- package/dist/bootstrap.json +4 -4
- package/dist/bridge.js +36 -57
- package/dist/chatgpt.js +32 -0
- package/dist/clis.js +126 -69
- package/dist/codex-main.js +58 -0
- package/dist/conversations.js +457 -277
- package/dist/copies.js +198 -124
- package/dist/end.js +23 -0
- package/dist/enrol-main.js +7 -1
- package/dist/events.js +525 -0
- package/dist/files.js +22 -2
- package/dist/git.js +10 -8
- package/dist/home.js +96 -7
- package/dist/house-cli.js +291 -0
- package/dist/house-main.js +7 -159
- package/dist/inputs.js +63 -0
- package/dist/instructions.js +10 -32
- package/dist/kit-cli.js +68 -0
- package/dist/kit-main.js +8 -19
- package/dist/login.js +118 -103
- package/dist/logout.js +14 -13
- package/dist/parts.js +82 -0
- package/dist/placement.js +30 -0
- package/dist/refusals.js +12 -0
- package/dist/relay.js +12 -2
- package/dist/resident.js +91 -35
- package/dist/scope.js +203 -0
- package/dist/secret-input.js +3 -2
- package/dist/service-main.js +9 -0
- package/dist/service.js +55 -0
- package/dist/shell.js +3 -0
- package/dist/sign-in.js +34 -28
- package/dist/skills.js +81 -12
- package/dist/stream.js +87 -11
- package/dist/transcription.js +57 -9
- package/dist/translate.js +2 -8
- package/dist/update.js +50 -29
- package/dist/wire.js +6 -0
- package/dist/work.js +346 -0
- package/how-we-work.md +13 -3
- package/package.json +3 -2
- package/skills/ask-matt/SKILL.md +17 -11
- package/skills/ask-matt/agents/openai.yaml +1 -0
- package/skills/code-review/SKILL.md +5 -5
- package/skills/codebase-design/DESIGN-IT-TWICE.md +1 -1
- package/skills/codebase-design/SKILL.md +0 -2
- package/skills/diagnosing-bugs/SKILL.md +2 -4
- package/skills/domain-modeling/{CONTEXT-FORMAT.md → GLOSSARY-FORMAT.md} +10 -10
- package/skills/domain-modeling/SKILL.md +12 -14
- package/skills/grill-me/SKILL.md +1 -2
- package/skills/grill-me/agents/openai.yaml +1 -0
- package/skills/grill-with-docs/SKILL.md +1 -2
- package/skills/grill-with-docs/agents/openai.yaml +1 -0
- package/skills/grilling/SKILL.md +2 -2
- package/skills/handoff/SKILL.md +2 -3
- package/skills/handoff/agents/openai.yaml +1 -0
- package/skills/implement/SKILL.md +5 -4
- package/skills/implement/agents/openai.yaml +1 -0
- package/skills/implement-spec/SKILL.md +40 -0
- package/skills/implement-spec/agents/openai.yaml +5 -0
- package/skills/improve-codebase-architecture/SKILL.md +6 -7
- package/skills/improve-codebase-architecture/agents/openai.yaml +1 -0
- package/skills/pr/CREDITS.md +3 -0
- package/skills/pr/LICENSE +21 -0
- package/skills/pr/SKILL.md +170 -0
- package/skills/pr/agents/openai.yaml +3 -0
- package/skills/prototype/SKILL.md +0 -2
- package/skills/research/SKILL.md +0 -2
- package/skills/retro/LICENSE +21 -0
- package/skills/retro/SKILL.md +44 -0
- package/skills/retro/agents/openai.yaml +5 -0
- package/skills/tdd/SKILL.md +2 -4
- package/skills/teach/SKILL.md +3 -4
- package/skills/teach/agents/openai.yaml +1 -0
- package/skills/to-questionnaire/SKILL.md +1 -2
- package/skills/to-questionnaire/agents/openai.yaml +1 -0
- package/skills/to-spec/SKILL.md +1 -2
- package/skills/to-spec/agents/openai.yaml +1 -0
- package/skills/to-tickets/SKILL.md +3 -4
- package/skills/to-tickets/agents/openai.yaml +1 -0
- package/skills/triage/AGENT-BRIEF.md +207 -0
- package/skills/triage/LICENSE +21 -0
- package/skills/triage/OUT-OF-SCOPE.md +105 -0
- package/skills/triage/SKILL.md +112 -0
- package/skills/triage/agents/openai.yaml +5 -0
- package/skills/wait-what/SKILL.md +2 -3
- package/skills/wait-what/agents/openai.yaml +1 -0
- package/skills/wayfinder/SKILL.md +5 -6
- package/skills/wayfinder/agents/openai.yaml +1 -0
- package/skills/wizard/LICENSE +21 -0
- package/skills/wizard/SKILL.md +44 -0
- package/skills/wizard/agents/openai.yaml +3 -0
- package/skills/wizard/template.sh +223 -0
- package/skills/writing-for-agents/SKILL.md +0 -2
- package/dist/blocks.js +0 -51
- package/skills/resolving-merge-conflicts/SKILL.md +0 -16
- package/skills/resolving-merge-conflicts/agents/openai.yaml +0 -3
- /package/skills/{resolving-merge-conflicts → implement-spec}/LICENSE +0 -0
|
@@ -1,10 +1,9 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: to-questionnaire
|
|
3
3
|
description: Turn a decision you can't fully answer into a questionnaire for someone else to fill in.
|
|
4
|
+
disable-model-invocation: true
|
|
4
5
|
---
|
|
5
6
|
|
|
6
|
-
<!-- Vendored from mattpocock/skills at commit c55ee46073ed923f86ce59a5eb3b6d895095d1b7 (skills/productivity/to-questionnaire). MIT licensed — see LICENSE in this directory. -->
|
|
7
|
-
|
|
8
7
|
Turn something the user can't answer alone into a **questionnaire**: a Markdown document they hand to one person to fill in async, or fill out together over a meeting. The recipient holds knowledge the user lacks; the questionnaire pulls it out of them.
|
|
9
8
|
|
|
10
9
|
**Grill the send, not the subject.** Interview the user only about the _send_, which they can always answer: who it goes to, and what they need back. The questions in the document then target the **gap** between what the recipient knows and what the user needs.
|
package/skills/to-spec/SKILL.md
CHANGED
|
@@ -1,10 +1,9 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: to-spec
|
|
3
3
|
description: "Turn the current conversation into a spec and publish it to the project issue tracker: no interview, just synthesis of what you've already discussed."
|
|
4
|
+
disable-model-invocation: true
|
|
4
5
|
---
|
|
5
6
|
|
|
6
|
-
<!-- Vendored from mattpocock/skills at commit c55ee46073ed923f86ce59a5eb3b6d895095d1b7 (skills/engineering/to-spec). MIT licensed — see LICENSE in this directory. -->
|
|
7
|
-
|
|
8
7
|
This skill takes the current conversation context and codebase understanding and produces a spec. Do NOT interview the user; just synthesize what you already know.
|
|
9
8
|
|
|
10
9
|
The issue tracker and triage label vocabulary should have been provided to you. If not, tell the user to run `/setup-matt-pocock-skills`.
|
|
@@ -1,10 +1,9 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: to-tickets
|
|
3
3
|
description: Break a plan, spec, or the current conversation into a set of tracer-bullet tickets, each declaring its blocking edges, published to the configured tracker (edges as text in one file per ticket locally, or native blocking links on a real tracker).
|
|
4
|
+
disable-model-invocation: true
|
|
4
5
|
---
|
|
5
6
|
|
|
6
|
-
<!-- Vendored from mattpocock/skills at commit c55ee46073ed923f86ce59a5eb3b6d895095d1b7 (skills/engineering/to-tickets). MIT licensed — see LICENSE in this directory. -->
|
|
7
|
-
|
|
8
7
|
# To Tickets
|
|
9
8
|
|
|
10
9
|
Break a plan, spec, or conversation into a set of **tickets**: tracer-bullet vertical slices, each declaring the tickets that **block** it.
|
|
@@ -61,7 +60,7 @@ Iterate until the user approves the breakdown.
|
|
|
61
60
|
Publish the approved tickets. **How** depends on the tracker `/setup-matt-pocock-skills` configured; the tickets are the same either way, only the shape of the blocking edges changes:
|
|
62
61
|
|
|
63
62
|
- **Local files** → write one file per ticket under `.scratch/<feature-slug>/issues/<NN>-<slug>.md`, numbered from `01` in dependency order (blockers first). Each file's "Blocked by" lists the numbers/titles it depends on. Use the per-ticket file template below: one ticket per file, never a single combined file.
|
|
64
|
-
- **A real issue tracker (GitHub, Linear, …)** → publish one issue per ticket in dependency order (blockers first) so each ticket's blocking edges can reference real identifiers. Use the platform's native blocking
|
|
63
|
+
- **A real issue tracker (GitHub, Linear, …)** → publish one issue per ticket in dependency order (blockers first) so each ticket's blocking edges can reference real identifiers. Use the platform's native blocking relationship where it has one; otherwise set each ticket's "Blocked by" to the blocking issues. If the source was an existing issue, make each ticket its sub-issue (tracker doc's operation). Apply the `ready-for-agent` triage label unless instructed otherwise; the tickets are agent-grabbable by construction.
|
|
65
64
|
|
|
66
65
|
Work the **frontier**: any ticket whose blockers are all done. For a purely linear chain that means top to bottom.
|
|
67
66
|
|
|
@@ -99,7 +98,7 @@ The end-to-end behaviour this ticket makes work, from the user's perspective, no
|
|
|
99
98
|
|
|
100
99
|
## Blocked by
|
|
101
100
|
|
|
102
|
-
- A reference to each blocking ticket, or "None (can start immediately)".
|
|
101
|
+
- A reference to each blocking ticket, or "None (can start immediately)". Omit this section when blockers were set as native edges.
|
|
103
102
|
|
|
104
103
|
</issue-template>
|
|
105
104
|
|
|
@@ -0,0 +1,207 @@
|
|
|
1
|
+
# Writing Agent Briefs
|
|
2
|
+
|
|
3
|
+
An agent brief is a structured comment posted on a GitHub issue or PR when it moves to `ready-for-agent`. It is the authoritative specification that an AFK agent will work from. The original body and discussion are context: the agent brief is the contract.
|
|
4
|
+
|
|
5
|
+
The brief states **what the agent should do**, which stretches to both surfaces: for an issue, that's building the change from nothing; for a PR, it's what's left to do *to the existing diff*: finish it, close gaps, address review points. Same principles either way; the PR example below shows the difference.
|
|
6
|
+
|
|
7
|
+
## Principles
|
|
8
|
+
|
|
9
|
+
### Durability over precision
|
|
10
|
+
|
|
11
|
+
The issue may sit in `ready-for-agent` for days or weeks. The codebase will change in the meantime. Write the brief so it stays useful even as files are renamed, moved, or refactored.
|
|
12
|
+
|
|
13
|
+
- **Do** describe interfaces, types, and behavioral contracts
|
|
14
|
+
- **Do** name specific types, function signatures, or config shapes that the agent should look for or modify
|
|
15
|
+
- **Don't** reference file paths: they go stale
|
|
16
|
+
- **Don't** reference line numbers
|
|
17
|
+
- **Don't** assume the current implementation structure will remain the same
|
|
18
|
+
|
|
19
|
+
### Behavioral, not procedural
|
|
20
|
+
|
|
21
|
+
Describe **what** the system should do, not **how** to implement it. The agent will explore the codebase fresh and make its own implementation decisions.
|
|
22
|
+
|
|
23
|
+
- **Good:** "The `SkillConfig` type should accept an optional `schedule` field of type `CronExpression`"
|
|
24
|
+
- **Bad:** "Open src/types/skill.ts and add a schedule field on line 42"
|
|
25
|
+
- **Good:** "When a user runs `/triage` with no arguments, they should see a summary of issues needing attention"
|
|
26
|
+
- **Bad:** "Add a switch statement in the main handler function"
|
|
27
|
+
|
|
28
|
+
### Complete acceptance criteria
|
|
29
|
+
|
|
30
|
+
The agent needs to know when it's done. Every agent brief must have concrete, testable acceptance criteria. Each criterion should be independently verifiable.
|
|
31
|
+
|
|
32
|
+
- **Good:** "Running `gh issue list --label needs-triage` returns issues that have been through initial classification"
|
|
33
|
+
- **Bad:** "Triage should work correctly"
|
|
34
|
+
|
|
35
|
+
### Explicit scope boundaries
|
|
36
|
+
|
|
37
|
+
State what is out of scope. This prevents the agent from gold-plating or making assumptions about adjacent features.
|
|
38
|
+
|
|
39
|
+
## Template
|
|
40
|
+
|
|
41
|
+
```markdown
|
|
42
|
+
## Agent Brief
|
|
43
|
+
|
|
44
|
+
**Category:** bug / enhancement
|
|
45
|
+
**Summary:** one-line description of what needs to happen
|
|
46
|
+
|
|
47
|
+
**Current behavior:**
|
|
48
|
+
Describe what happens now. For bugs, this is the broken behavior.
|
|
49
|
+
For enhancements, this is the status quo the feature builds on.
|
|
50
|
+
|
|
51
|
+
**Desired behavior:**
|
|
52
|
+
Describe what should happen after the agent's work is complete.
|
|
53
|
+
Be specific about edge cases and error conditions.
|
|
54
|
+
|
|
55
|
+
**Key interfaces:**
|
|
56
|
+
- `TypeName`: what needs to change and why
|
|
57
|
+
- `functionName()` return type: what it currently returns vs what it should return
|
|
58
|
+
- Config shape: any new configuration options needed
|
|
59
|
+
|
|
60
|
+
**Acceptance criteria:**
|
|
61
|
+
- [ ] Specific, testable criterion 1
|
|
62
|
+
- [ ] Specific, testable criterion 2
|
|
63
|
+
- [ ] Specific, testable criterion 3
|
|
64
|
+
|
|
65
|
+
**Out of scope:**
|
|
66
|
+
- Thing that should NOT be changed or addressed in this issue
|
|
67
|
+
- Adjacent feature that might seem related but is separate
|
|
68
|
+
```
|
|
69
|
+
|
|
70
|
+
## Examples
|
|
71
|
+
|
|
72
|
+
### Good agent brief (bug)
|
|
73
|
+
|
|
74
|
+
```markdown
|
|
75
|
+
## Agent Brief
|
|
76
|
+
|
|
77
|
+
**Category:** bug
|
|
78
|
+
**Summary:** Skill description truncation drops mid-word, producing broken output
|
|
79
|
+
|
|
80
|
+
**Current behavior:**
|
|
81
|
+
When a skill description exceeds 1024 characters, it is truncated at exactly
|
|
82
|
+
1024 characters regardless of word boundaries. This produces descriptions
|
|
83
|
+
that end mid-word (e.g. "Use when the user wants to confi").
|
|
84
|
+
|
|
85
|
+
**Desired behavior:**
|
|
86
|
+
Truncation should break at the last word boundary before 1024 characters
|
|
87
|
+
and append "..." to indicate truncation.
|
|
88
|
+
|
|
89
|
+
**Key interfaces:**
|
|
90
|
+
- The `SkillMetadata` type's `description` field: no type change needed,
|
|
91
|
+
but the validation/processing logic that populates it needs to respect
|
|
92
|
+
word boundaries
|
|
93
|
+
- Any function that reads SKILL.md frontmatter and extracts the description
|
|
94
|
+
|
|
95
|
+
**Acceptance criteria:**
|
|
96
|
+
- [ ] Descriptions under 1024 chars are unchanged
|
|
97
|
+
- [ ] Descriptions over 1024 chars are truncated at the last word boundary
|
|
98
|
+
before 1024 chars
|
|
99
|
+
- [ ] Truncated descriptions end with "..."
|
|
100
|
+
- [ ] The total length including "..." does not exceed 1024 chars
|
|
101
|
+
|
|
102
|
+
**Out of scope:**
|
|
103
|
+
- Changing the 1024 char limit itself
|
|
104
|
+
- Multi-line description support
|
|
105
|
+
```
|
|
106
|
+
|
|
107
|
+
### Good agent brief (enhancement)
|
|
108
|
+
|
|
109
|
+
```markdown
|
|
110
|
+
## Agent Brief
|
|
111
|
+
|
|
112
|
+
**Category:** enhancement
|
|
113
|
+
**Summary:** Add `.out-of-scope/` directory support for tracking rejected feature requests
|
|
114
|
+
|
|
115
|
+
**Current behavior:**
|
|
116
|
+
When a feature request is rejected, the issue is closed with a `wontfix` label
|
|
117
|
+
and a comment. There is no persistent record of the decision or reasoning.
|
|
118
|
+
Future similar requests require the maintainer to recall or search for the
|
|
119
|
+
prior discussion.
|
|
120
|
+
|
|
121
|
+
**Desired behavior:**
|
|
122
|
+
Rejected feature requests should be documented in `.out-of-scope/<concept>.md`
|
|
123
|
+
files that capture the decision, reasoning, and links to all issues that
|
|
124
|
+
requested the feature. When triaging new issues, these files should be
|
|
125
|
+
checked for matches.
|
|
126
|
+
|
|
127
|
+
**Key interfaces:**
|
|
128
|
+
- Markdown file format in `.out-of-scope/`: each file should have a
|
|
129
|
+
`# Concept Name` heading, a `**Decision:**` line, a `**Reason:**` line,
|
|
130
|
+
and a `**Prior requests:**` list with issue links
|
|
131
|
+
- The triage workflow should read all `.out-of-scope/*.md` files early
|
|
132
|
+
and match incoming issues against them by concept similarity
|
|
133
|
+
|
|
134
|
+
**Acceptance criteria:**
|
|
135
|
+
- [ ] Closing a feature as wontfix creates/updates a file in `.out-of-scope/`
|
|
136
|
+
- [ ] The file includes the decision, reasoning, and link to the closed issue
|
|
137
|
+
- [ ] If a matching `.out-of-scope/` file already exists, the new issue is
|
|
138
|
+
appended to its "Prior requests" list rather than creating a duplicate
|
|
139
|
+
- [ ] During triage, existing `.out-of-scope/` files are checked and surfaced
|
|
140
|
+
when a new issue matches a prior rejection
|
|
141
|
+
|
|
142
|
+
**Out of scope:**
|
|
143
|
+
- Automated matching (human confirms the match)
|
|
144
|
+
- Reopening previously rejected features
|
|
145
|
+
- Bug reports (only enhancement rejections go to `.out-of-scope/`)
|
|
146
|
+
```
|
|
147
|
+
|
|
148
|
+
### Good agent brief (PR)
|
|
149
|
+
|
|
150
|
+
For a PR, "Current behavior" describes the state of the diff, and the brief asks the agent to finish or fix it rather than build from scratch.
|
|
151
|
+
|
|
152
|
+
```markdown
|
|
153
|
+
## Agent Brief
|
|
154
|
+
|
|
155
|
+
**Category:** enhancement
|
|
156
|
+
**Summary:** Finish the contributor's `--json` output flag for `triage list`
|
|
157
|
+
|
|
158
|
+
**Current behavior:**
|
|
159
|
+
The PR adds a `--json` flag that serializes the issue list to JSON. The happy
|
|
160
|
+
path works and the diff matches the project's command structure. Two gaps
|
|
161
|
+
remain: errors are still printed as human text (not JSON), and the new flag has
|
|
162
|
+
no test coverage.
|
|
163
|
+
|
|
164
|
+
**Desired behavior:**
|
|
165
|
+
With `--json`, all output (including errors) is well-formed JSON on stdout,
|
|
166
|
+
and the command's exit codes are unchanged. The existing human-readable output
|
|
167
|
+
is untouched when the flag is absent.
|
|
168
|
+
|
|
169
|
+
**Key interfaces:**
|
|
170
|
+
- The command's error path should emit `{ "error": string }` under `--json`
|
|
171
|
+
instead of the plain-text error
|
|
172
|
+
- Reuse the existing serializer the PR already added; don't introduce a second
|
|
173
|
+
|
|
174
|
+
**Acceptance criteria:**
|
|
175
|
+
- [ ] `triage list --json` emits valid JSON for both success and error cases
|
|
176
|
+
- [ ] Exit codes match the non-JSON command
|
|
177
|
+
- [ ] A test covers the `--json` success output and one error case
|
|
178
|
+
- [ ] Default (non-JSON) output is byte-for-byte unchanged
|
|
179
|
+
|
|
180
|
+
**Out of scope:**
|
|
181
|
+
- Adding `--json` to any other command
|
|
182
|
+
- Changing the JSON shape of the success payload the PR already defined
|
|
183
|
+
```
|
|
184
|
+
|
|
185
|
+
### Bad agent brief
|
|
186
|
+
|
|
187
|
+
```markdown
|
|
188
|
+
## Agent Brief
|
|
189
|
+
|
|
190
|
+
**Summary:** Fix the triage bug
|
|
191
|
+
|
|
192
|
+
**What to do:**
|
|
193
|
+
The triage thing is broken. Look at the main file and fix it.
|
|
194
|
+
The function around line 150 has the issue.
|
|
195
|
+
|
|
196
|
+
**Files to change:**
|
|
197
|
+
- src/triage/handler.ts (line 150)
|
|
198
|
+
- src/types.ts (line 42)
|
|
199
|
+
```
|
|
200
|
+
|
|
201
|
+
This is bad because:
|
|
202
|
+
- No category
|
|
203
|
+
- Vague description ("the triage thing is broken")
|
|
204
|
+
- References file paths and line numbers that will go stale
|
|
205
|
+
- No acceptance criteria
|
|
206
|
+
- No scope boundaries
|
|
207
|
+
- No description of current vs desired behavior
|
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
MIT License
|
|
2
|
+
|
|
3
|
+
Copyright (c) 2026 Matt Pocock
|
|
4
|
+
|
|
5
|
+
Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
6
|
+
of this software and associated documentation files (the "Software"), to deal
|
|
7
|
+
in the Software without restriction, including without limitation the rights
|
|
8
|
+
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
9
|
+
copies of the Software, and to permit persons to whom the Software is
|
|
10
|
+
furnished to do so, subject to the following conditions:
|
|
11
|
+
|
|
12
|
+
The above copyright notice and this permission notice shall be included in all
|
|
13
|
+
copies or substantial portions of the Software.
|
|
14
|
+
|
|
15
|
+
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
16
|
+
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
17
|
+
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
18
|
+
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
19
|
+
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
20
|
+
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
|
|
21
|
+
SOFTWARE.
|
|
@@ -0,0 +1,105 @@
|
|
|
1
|
+
# Out-of-Scope Knowledge Base
|
|
2
|
+
|
|
3
|
+
The `.out-of-scope/` directory in a repo stores persistent records of rejected feature requests. It serves two purposes:
|
|
4
|
+
|
|
5
|
+
1. **Institutional memory**: why a feature was rejected, so the reasoning isn't lost when the issue is closed
|
|
6
|
+
2. **Deduplication**: when a new issue comes in that matches a prior rejection, the skill can surface the previous decision instead of re-litigating it
|
|
7
|
+
|
|
8
|
+
## Directory structure
|
|
9
|
+
|
|
10
|
+
```
|
|
11
|
+
.out-of-scope/
|
|
12
|
+
├── dark-mode.md
|
|
13
|
+
├── plugin-system.md
|
|
14
|
+
└── graphql-api.md
|
|
15
|
+
```
|
|
16
|
+
|
|
17
|
+
One file per **concept**, not per issue. Multiple issues requesting the same thing are grouped under one file.
|
|
18
|
+
|
|
19
|
+
## File format
|
|
20
|
+
|
|
21
|
+
The file should be written in a relaxed, readable style, more like a short design document than a database entry. Use paragraphs, code samples, and examples to make the reasoning clear and useful to someone encountering it for the first time.
|
|
22
|
+
|
|
23
|
+
```markdown
|
|
24
|
+
# Dark Mode
|
|
25
|
+
|
|
26
|
+
This project does not support dark mode or user-facing theming.
|
|
27
|
+
|
|
28
|
+
## Why this is out of scope
|
|
29
|
+
|
|
30
|
+
The rendering pipeline assumes a single color palette defined in
|
|
31
|
+
`ThemeConfig`. Supporting multiple themes would require:
|
|
32
|
+
|
|
33
|
+
- A theme context provider wrapping the entire component tree
|
|
34
|
+
- Per-component theme-aware style resolution
|
|
35
|
+
- A persistence layer for user theme preferences
|
|
36
|
+
|
|
37
|
+
This is a significant architectural change that doesn't align with the
|
|
38
|
+
project's focus on content authoring. Theming is a concern for downstream
|
|
39
|
+
consumers who embed or redistribute the output.
|
|
40
|
+
|
|
41
|
+
```ts
|
|
42
|
+
// The current ThemeConfig interface is not designed for runtime switching:
|
|
43
|
+
interface ThemeConfig {
|
|
44
|
+
colors: ColorPalette; // single palette, resolved at build time
|
|
45
|
+
fonts: FontStack;
|
|
46
|
+
}
|
|
47
|
+
```
|
|
48
|
+
|
|
49
|
+
## Prior requests
|
|
50
|
+
|
|
51
|
+
- #42: "Add dark mode support"
|
|
52
|
+
- #87: "Night theme for accessibility"
|
|
53
|
+
- #134: "Dark theme option"
|
|
54
|
+
```
|
|
55
|
+
|
|
56
|
+
### Naming the file
|
|
57
|
+
|
|
58
|
+
Use a short, descriptive kebab-case name for the concept: `dark-mode.md`, `plugin-system.md`, `graphql-api.md`. The name should be recognizable enough that someone browsing the directory understands what was rejected without opening the file.
|
|
59
|
+
|
|
60
|
+
### Writing the reason
|
|
61
|
+
|
|
62
|
+
The reason should be substantive: not "we don't want this" but why. Good reasons reference:
|
|
63
|
+
|
|
64
|
+
- Project scope or philosophy ("This project focuses on X; theming is a downstream concern")
|
|
65
|
+
- Technical constraints ("Supporting this would require Y, which conflicts with our Z architecture")
|
|
66
|
+
- Strategic decisions ("We chose to use A instead of B because...")
|
|
67
|
+
|
|
68
|
+
The reason should be durable. Avoid referencing temporary circumstances ("we're too busy right now"); those aren't real rejections, they're deferrals.
|
|
69
|
+
|
|
70
|
+
## When to check `.out-of-scope/`
|
|
71
|
+
|
|
72
|
+
During triage (Step 1: Gather context), read all files in `.out-of-scope/`. When evaluating a new issue:
|
|
73
|
+
|
|
74
|
+
- Check if the request matches an existing out-of-scope concept
|
|
75
|
+
- Matching is by concept similarity, not keyword: "night theme" matches `dark-mode.md`
|
|
76
|
+
- If there's a match, surface it to the maintainer: "This is similar to `.out-of-scope/dark-mode.md`. We rejected this before because [reason]. Do you still feel the same way?"
|
|
77
|
+
|
|
78
|
+
The maintainer may:
|
|
79
|
+
|
|
80
|
+
- **Confirm**: the new issue gets added to the existing file's "Prior requests" list, then closed
|
|
81
|
+
- **Reconsider**: the out-of-scope file gets deleted or updated, and the issue proceeds through normal triage
|
|
82
|
+
- **Disagree**: the issues are related but distinct, proceed with normal triage
|
|
83
|
+
|
|
84
|
+
## When to write to `.out-of-scope/`
|
|
85
|
+
|
|
86
|
+
Only when an **enhancement** (not a bug) is *rejected* as `wontfix`. This applies to enhancement PRs exactly as it does to issues: a rejected PR is recorded here so the same request doesn't return as fresh code.
|
|
87
|
+
|
|
88
|
+
Do **not** write here when something is closed as `wontfix` because it's **already implemented**. That's a built feature, not a rejected one; recording it would poison the dedup checks with false rejections. Instead, the closing comment points to where the feature already lives.
|
|
89
|
+
|
|
90
|
+
The flow:
|
|
91
|
+
|
|
92
|
+
1. Maintainer decides a feature request is out of scope
|
|
93
|
+
2. Check if a matching `.out-of-scope/` file already exists
|
|
94
|
+
3. If yes: append the new issue to the "Prior requests" list
|
|
95
|
+
4. If no: create a new file with the concept name, decision, reason, and first prior request
|
|
96
|
+
5. Post a comment on the issue explaining the decision and mentioning the `.out-of-scope/` file
|
|
97
|
+
6. Close the issue with the `wontfix` label
|
|
98
|
+
|
|
99
|
+
## Updating or removing out-of-scope files
|
|
100
|
+
|
|
101
|
+
If the maintainer changes their mind about a previously rejected concept:
|
|
102
|
+
|
|
103
|
+
- Delete the `.out-of-scope/` file
|
|
104
|
+
- The skill does not need to reopen old issues; they're historical records
|
|
105
|
+
- The new issue that triggered the reconsideration proceeds through normal triage
|
|
@@ -0,0 +1,112 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: triage
|
|
3
|
+
description: Move issues and external PRs through a state machine of triage roles, categorise, verify, grill if needed, and write agent-ready briefs.
|
|
4
|
+
disable-model-invocation: true
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
# Triage
|
|
8
|
+
|
|
9
|
+
Move issues on the project issue tracker through a small state machine of triage roles.
|
|
10
|
+
|
|
11
|
+
If this repo treats external pull requests as a request surface (see the issue-tracker config), triage covers them too: **a PR is an issue with attached code**, using the same roles, same states, and same machine, with a few deltas marked "for a PR" below. Resolve a bare `#42` to an issue or PR per the tracker config.
|
|
12
|
+
|
|
13
|
+
Every comment or issue posted to the issue tracker during triage **must** start with this disclaimer:
|
|
14
|
+
|
|
15
|
+
```
|
|
16
|
+
> *This was generated by AI during triage.*
|
|
17
|
+
```
|
|
18
|
+
|
|
19
|
+
## Reference docs
|
|
20
|
+
|
|
21
|
+
- [AGENT-BRIEF.md](AGENT-BRIEF.md): how to write durable agent briefs
|
|
22
|
+
- [OUT-OF-SCOPE.md](OUT-OF-SCOPE.md): how the `.out-of-scope/` knowledge base works
|
|
23
|
+
|
|
24
|
+
## Roles
|
|
25
|
+
|
|
26
|
+
Two **category** roles:
|
|
27
|
+
|
|
28
|
+
- `bug`: something is broken
|
|
29
|
+
- `enhancement`: new feature or improvement
|
|
30
|
+
|
|
31
|
+
Five **state** roles:
|
|
32
|
+
|
|
33
|
+
- `needs-triage`: maintainer needs to evaluate
|
|
34
|
+
- `needs-info`: waiting on reporter for more information
|
|
35
|
+
- `ready-for-agent`: fully specified, ready for an AFK agent
|
|
36
|
+
- `ready-for-human`: needs human implementation
|
|
37
|
+
- `wontfix`: will not be actioned
|
|
38
|
+
|
|
39
|
+
For a PR, the same states read against the attached code: `ready-for-agent` means a brief is attached and an agent should take the next step on the diff; `ready-for-human` means it's ready for a human to merge.
|
|
40
|
+
|
|
41
|
+
Every triaged issue should carry exactly one category role and one state role. If state roles conflict, flag it and ask the maintainer before doing anything else.
|
|
42
|
+
|
|
43
|
+
These are canonical role names. The actual label strings used in the issue tracker may differ. The mapping should have been provided to you. If not, tell the user to run `/setup-matt-pocock-skills`.
|
|
44
|
+
|
|
45
|
+
State transitions: an unlabeled issue normally goes to `needs-triage` first; from there it moves to `needs-info`, `ready-for-agent`, `ready-for-human`, or `wontfix`. `needs-info` returns to `needs-triage` once the reporter replies. The maintainer can override at any time; flag transitions that look unusual and ask before proceeding.
|
|
46
|
+
|
|
47
|
+
## Invocation
|
|
48
|
+
|
|
49
|
+
The maintainer invokes `/triage` and describes what they want in natural language. Interpret the request and act. Examples:
|
|
50
|
+
|
|
51
|
+
- "Show me anything that needs my attention"
|
|
52
|
+
- "Let's look at #42" (issue or PR)
|
|
53
|
+
- "Move #42 to ready-for-agent"
|
|
54
|
+
- "What's ready for agents to pick up?"
|
|
55
|
+
|
|
56
|
+
## Show what needs attention
|
|
57
|
+
|
|
58
|
+
Query the issue tracker and present three buckets, oldest first:
|
|
59
|
+
|
|
60
|
+
1. **Unlabeled**: never triaged.
|
|
61
|
+
2. **`needs-triage`**: evaluation in progress.
|
|
62
|
+
3. **`needs-info` with reporter activity since the last triage notes**: needs re-evaluation.
|
|
63
|
+
|
|
64
|
+
When PRs are in scope, include external PRs in these buckets and tag each line `[PR]` or `[issue]`. Discovery surfaces only *external* PRs (the tracker config defines who counts as external), so a collaborator's in-flight PR is not triage work. This filter is discovery-only; an explicitly named PR is always triaged regardless of author.
|
|
65
|
+
|
|
66
|
+
Show counts and a one-line summary per item. Let the maintainer pick.
|
|
67
|
+
|
|
68
|
+
## Triage a specific issue or PR
|
|
69
|
+
|
|
70
|
+
1. **Gather context.** Read the full issue or PR (body, comments, labels, author, dates; for a PR, the diff too). Parse any prior triage notes so you don't re-ask resolved questions. Explore the codebase using the project's domain glossary, respecting ADRs in the area. Run two checks against the codebase: (a) **redundancy**: search for an existing implementation of the requested behavior by domain concept (not just the request's wording), and report where you looked. If found, it's an already-implemented `wontfix` (step 5). (b) **prior rejection**: read `.out-of-scope/*.md` and surface any that resembles this request.
|
|
71
|
+
|
|
72
|
+
2. **Recommend.** Tell the maintainer your category and state recommendation with reasoning, plus a brief codebase summary relevant to the request (including whether it's already implemented). Wait for direction.
|
|
73
|
+
|
|
74
|
+
3. **Verify the claim.** Before any grilling, check that the claim holds up. For a bug, reproduce it from the reporter's steps. For a PR, confirm the diff does what it claims: check it out, run the relevant tests or commands. Report what happened: confirmed (with code path), failed, or insufficient detail (a strong `needs-info` signal). A confirmed verification makes a much stronger agent brief.
|
|
75
|
+
|
|
76
|
+
4. **Grill (if needed).** If the request needs fleshing out, call the Skill tool twice, for "grilling" and "domain-modeling", and grill it into shape a round of questions at a time, sharpening domain terms and updating `GLOSSARY.md`/ADRs inline as decisions land.
|
|
77
|
+
|
|
78
|
+
5. **Apply the outcome:**
|
|
79
|
+
- `ready-for-agent`: post an agent brief comment ([AGENT-BRIEF.md](AGENT-BRIEF.md)).
|
|
80
|
+
- `ready-for-human`: same structure as an agent brief, but note why it can't be delegated (judgment calls, external access, design decisions, manual testing).
|
|
81
|
+
- `needs-info`: post triage notes (template below).
|
|
82
|
+
- For `wontfix`, close the issue, with the comment depending on *why*:
|
|
83
|
+
- **Already implemented**: the change already exists in the codebase. Point to where it lives; do **not** write to `.out-of-scope/` (that KB is for *rejected* requests, not built ones).
|
|
84
|
+
- **Rejected (bug)**: give a polite explanation, then close.
|
|
85
|
+
- **Rejected (enhancement)**: write to `.out-of-scope/`, link to it from a comment, then close ([OUT-OF-SCOPE.md](OUT-OF-SCOPE.md)).
|
|
86
|
+
- `needs-triage`: apply the role. Optional comment if there's partial progress.
|
|
87
|
+
|
|
88
|
+
## Quick state override
|
|
89
|
+
|
|
90
|
+
If the maintainer says "move #42 to ready-for-agent", trust them and apply the role directly. Confirm what you're about to do (role changes, comment, close), then act. Skip grilling. If moving to `ready-for-agent` without a grilling session, ask whether they want to write an agent brief.
|
|
91
|
+
|
|
92
|
+
## Needs-info template
|
|
93
|
+
|
|
94
|
+
```markdown
|
|
95
|
+
## Triage Notes
|
|
96
|
+
|
|
97
|
+
**What we've established so far:**
|
|
98
|
+
|
|
99
|
+
- point 1
|
|
100
|
+
- point 2
|
|
101
|
+
|
|
102
|
+
**What we still need from you (@reporter):**
|
|
103
|
+
|
|
104
|
+
- question 1
|
|
105
|
+
- question 2
|
|
106
|
+
```
|
|
107
|
+
|
|
108
|
+
Capture everything resolved during grilling under "established so far" so the work isn't lost. Questions must be specific and actionable, not "please provide more info".
|
|
109
|
+
|
|
110
|
+
## Resuming a previous session
|
|
111
|
+
|
|
112
|
+
If prior triage notes exist on the issue or PR, read them, check whether the reporter has answered any outstanding questions, and present an updated picture before continuing. Don't re-ask resolved questions.
|
|
@@ -1,8 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: wait-what
|
|
3
3
|
description: "Stop. That last message did not land: re-pitch it."
|
|
4
|
+
disable-model-invocation: true
|
|
4
5
|
---
|
|
5
6
|
|
|
6
|
-
|
|
7
|
-
|
|
8
|
-
Wait, I don't understand where you've got to here. Re-pitch that: give me a little bit of context, talk in ASD-STE100 Simplified Technical English, and use the ubiquitous language from `CONTEXT.md` (follow `CONTEXT-MAP.md` to the right one if the repo has more than one).
|
|
7
|
+
Wait, I don't understand where you've got to here. Re-pitch that: give me a little bit of context, talk in ASD-STE100 Simplified Technical English, and use the ubiquitous language from `GLOSSARY.md` (follow `GLOSSARY-MAP.md` to the right one if the repo has more than one).
|
|
@@ -1,10 +1,9 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: wayfinder
|
|
3
3
|
description: Plan a huge chunk of work (more than one agent session can hold) as a shared map of decision tickets on your issue tracker, and resolve them one at a time until the way to the destination is clear.
|
|
4
|
+
disable-model-invocation: true
|
|
4
5
|
---
|
|
5
6
|
|
|
6
|
-
<!-- Vendored from mattpocock/skills at commit c55ee46073ed923f86ce59a5eb3b6d895095d1b7 (skills/engineering/wayfinder). MIT licensed — see LICENSE in this directory. -->
|
|
7
|
-
|
|
8
7
|
A loose idea has arrived, too big for one agent session, and wrapped in fog: the way from here to the **destination** isn't visible yet. Wayfinding is about finding that way, not charging at the destination. This skill charts the way as a **shared map** on the repo's issue tracker, then works its **decision tickets** (questions whose resolution is a decision, not slices of a build to execute) one at a time until the route is clear.
|
|
9
8
|
|
|
10
9
|
The destination varies per effort, and naming it is the first act of charting: it shapes every ticket. It might be a spec to hand off and iterate on, a decision to lock before planning starts, or a change made in place like a data-structure migration. The map is domain-agnostic: engineering work, course content, whatever fits the shape.
|
|
@@ -63,7 +62,7 @@ Each ticket is a **child issue** of the map; the tracker's issue id is its ident
|
|
|
63
62
|
<the decision or investigation this ticket resolves>
|
|
64
63
|
```
|
|
65
64
|
|
|
66
|
-
Each ticket carries a `wayfinder:<type>` label, one of `research`, `prototype`, `grilling`, `task` (see [Ticket Types](#ticket-types)).
|
|
65
|
+
Each ticket carries a `wayfinder:<type>` label, one of `research`, `prototype`, `grilling`, `task` (see [Ticket Types](#ticket-types)). `wayfinder:` labels are the only labels a map and its tickets carry, never a triage label like `ready-for-agent`: they are decisions, not implementation work.
|
|
67
66
|
|
|
68
67
|
A session **claims** a ticket by assigning it to the dev driving the map, **first**, before any work, so concurrent sessions skip it. That assignee _is_ the claim: an open, unassigned ticket is unclaimed.
|
|
69
68
|
|
|
@@ -112,8 +111,8 @@ User invokes with a loose idea.
|
|
|
112
111
|
1. **Name the destination.** Call the Skill tool twice, for "grilling" and "domain-modeling", to pin down what this map is finding its way to: the spec, decision, or change. The destination fixes the scope, so it's settled first.
|
|
113
112
|
2. **Map the frontier.** Grill again, **breadth-first** this time: fan out across the whole space rather than deep on any one thread, surfacing the open decisions and the first steps takeable now. **If this surfaces no fog** (the way to the destination is already clear, the whole journey small enough for one session), you don't need a map. Stop and ask the user how they'd like to proceed.
|
|
114
113
|
3. **Create the map** (label `wayfinder:map`): Destination and Notes filled in, Decisions-so-far empty, the fog sketched into **Not yet specified**.
|
|
115
|
-
4. **Create the tickets you can specify now** as child issues of the map, then wire blocking edges in a **second pass** (issues need ids before they can reference each other). Wiring sorts them into the frontier and the blocked; everything you can't yet specify stays in the fog: the **Not yet specified** section.
|
|
116
|
-
5. **Fire the research subagents.** For each `research` ticket you just created, spin up a subagent that calls the Skill tool with "research" to resolve it in parallel, capturing its findings on a throwaway `research/<name>` branch with a context pointer from the ticket.
|
|
114
|
+
4. **Create the tickets you can specify now** as child issues of the map, then wire blocking edges in a **second pass** (issues need ids before they can reference each other). Write cross-references in that pass too, with real ids: a placeholder `#<n>` auto-links to an unrelated issue. Wiring sorts them into the frontier and the blocked; everything you can't yet specify stays in the fog: the **Not yet specified** section.
|
|
115
|
+
5. **Fire the research subagents.** For each `research` ticket you just created, spin up a subagent that calls the Skill tool with "research" to resolve it in parallel, capturing its findings on a throwaway `research/<name>` branch with a context pointer from the ticket. Push the branch but open no PR: it is never merged.
|
|
117
116
|
6. Stop: charting is one session's work; it hand-resolves nothing.
|
|
118
117
|
|
|
119
118
|
### Work through the map
|
|
@@ -122,7 +121,7 @@ User invokes with a map (URL or number). A ticket is **optional**: without one,
|
|
|
122
121
|
|
|
123
122
|
1. Load the **map**: the low-res view, not every ticket body.
|
|
124
123
|
2. Choose the ticket. If the user named one, use it. Otherwise take the first frontier ticket in order. **Claim it**: assign it to yourself before any work.
|
|
125
|
-
3. Resolve it. **Zoom as needed**: fetch the full body of any related or closed ticket on demand; call the Skill tool for whichever skills the `## Notes` block names. If in doubt, call the Skill tool twice, for "grilling" and "domain-modeling".
|
|
124
|
+
3. Resolve it as the type its `wayfinder:<type>` label names (see [Ticket Types](#ticket-types)). Read the label, not just the body: the body never states the type. **Zoom as needed**: fetch the full body of any related or closed ticket on demand; call the Skill tool for whichever skills the `## Notes` block names. If in doubt, call the Skill tool twice, for "grilling" and "domain-modeling".
|
|
126
125
|
4. Record the resolution: post the answer as a **resolution comment**, **close** the issue, and **append a context pointer** to the map's Decisions-so-far.
|
|
127
126
|
5. Add newly-surfaced tickets (create-then-wire); graduate any fog the answer has made specifiable, clearing each graduated patch from **Not yet specified** so it lives only as its new ticket. If the answer reveals that a ticket (this one or another) sits beyond the destination, **rule it out of scope** rather than resolving it on the route. If the decision invalidates other parts of the map, update or delete those tickets.
|
|
128
127
|
|
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
MIT License
|
|
2
|
+
|
|
3
|
+
Copyright (c) 2026 Matt Pocock
|
|
4
|
+
|
|
5
|
+
Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
6
|
+
of this software and associated documentation files (the "Software"), to deal
|
|
7
|
+
in the Software without restriction, including without limitation the rights
|
|
8
|
+
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
9
|
+
copies of the Software, and to permit persons to whom the Software is
|
|
10
|
+
furnished to do so, subject to the following conditions:
|
|
11
|
+
|
|
12
|
+
The above copyright notice and this permission notice shall be included in all
|
|
13
|
+
copies or substantial portions of the Software.
|
|
14
|
+
|
|
15
|
+
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
16
|
+
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
17
|
+
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
18
|
+
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
19
|
+
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
20
|
+
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
|
|
21
|
+
SOFTWARE.
|