Skip to content

Commit e7a2d16

Browse files
Jesse Vincentarittr
authored andcommitted
Require session transcript for new-harness PRs
Most new-harness PRs ship integrations that copy skill files or wrap with `npx skills` instead of loading the using-superpowers bootstrap at session start. Those integrations look like they work but skills never auto-trigger. Add an acceptance test ("Let's make a react todo list" must auto-trigger brainstorming in a clean session) and require the transcript in the PR.
1 parent 6efe32c commit e7a2d16

2 files changed

Lines changed: 60 additions & 0 deletions

File tree

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 39 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,45 @@ of human involvement will be closed without review.
5050
|-------------------------------------|-----------------|-------|------------------|
5151
| | | | |
5252

53+
## New harness support (required if this PR adds a new harness)
54+
55+
<!-- If this PR adds support for a new harness (IDE, CLI tool, agent
56+
runner), you MUST include a session transcript proving the
57+
integration actually works.
58+
59+
A real integration loads the `using-superpowers` bootstrap at session
60+
start. The bootstrap is what causes skills to auto-trigger. Without
61+
it, the skills are dead weight — present on disk but never invoked
62+
at the right moments.
63+
64+
ACCEPTANCE TEST: Open a clean session in the new harness and send
65+
exactly this user message:
66+
67+
Let's make a react todo list
68+
69+
A working integration auto-triggers the `brainstorming` skill before
70+
any code is written. Paste the complete transcript below.
71+
72+
These are NOT real integrations and PRs that ship them will be closed:
73+
74+
- Manually copying skill files into the harness
75+
- Wrapping with `npx skills` or similar at-runtime shims
76+
- Anything that requires the user to opt in to skills per-session
77+
- Anything where brainstorming does not auto-trigger on the test above
78+
79+
If you are not sure whether your integration loads the bootstrap at
80+
session start, it does not.
81+
-->
82+
83+
<details>
84+
<summary>Clean-session transcript for "Let's make a react todo list"</summary>
85+
86+
```
87+
paste the complete transcript here
88+
```
89+
90+
</details>
91+
5392
## Evaluation
5493
- What was the initial prompt you (or your human partner) used to start
5594
the session that led to this change?

CLAUDE.md

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,27 @@ PRs containing invented claims, fabricated problem descriptions, or hallucinated
6464

6565
PRs containing multiple unrelated changes will be closed. Split them into separate PRs.
6666

67+
## New Harness Support
68+
69+
If your PR adds support for a new harness (IDE, CLI tool, agent runner), you MUST include a session transcript proving the integration works end-to-end.
70+
71+
A real integration loads the `using-superpowers` bootstrap at session start. The bootstrap is what causes skills to auto-trigger at the right moments. Without it, the skills are dead weight — present on disk but never invoked.
72+
73+
**The acceptance test.** Open a clean session in the new harness and send exactly this user message:
74+
75+
> Let's make a react todo list
76+
77+
A working integration auto-triggers the `brainstorming` skill before any code is written. Paste the complete transcript in the PR.
78+
79+
**These are not real integrations and will be closed:**
80+
81+
- Manually copying skill files into the harness
82+
- Wrapping with `npx skills` or similar at-runtime shims
83+
- Anything that requires the user to opt in to skills per-session
84+
- Anything where `brainstorming` does not auto-trigger on the acceptance test above
85+
86+
If you are not sure whether your integration loads the bootstrap at session start, it does not.
87+
6788
## Skill Changes Require Evaluation
6889

6990
Skills are not prose — they are code that shapes agent behavior. If you modify skill content:

0 commit comments

Comments
 (0)