/addvalue — Find and ship real value
Purpose
Review a target carefully (remote Git repo, local folder, library, CLI, UI,
docs site, or product surface) and produce a ranked plan of improvements that
actually help users or maintainers. Then optionally implement the top item.
This is not “open 20 tiny PRs.” Prefer one hard, reviewable change over many shallow ones.
Triggers
/addvalue [path|owner/repo|.]
- “add value”, “how can this be improved”, “inspiring updates”, “what should we build”
- Track B contribution hunting when quality > volume
Partners (optional)
| Partner |
When |
How |
| This agent (default) |
Always |
Full review + implement |
| Ollama Cloud GLM |
User says /addvalue --ollama or /ollama mid-pass |
Load ollama skill; one high-reasoning plan call; execute or merge with your plan |
| Local Ollama |
User says /addvalue --local or /local |
Load local skill; use as second opinion only |
| agy / agent CLIs |
User names a CLI (agy, hermes, project scripts) |
Prefer project engines over inventing parallel tools; pass paths not huge file dumps |
Paid/cloud partners: honor the user’s Paid Models — Ask First rule. /ollama is standing approval for one GLM planner call on that turn. Ask before other paid APIs.
Hard rules
- Scope lock — name the target and one primary outcome before editing.
- Read before invent — README, CONTRIBUTING, AI_POLICY, recent issues/PRs, architecture entrypoints. No drive-by.
- Value > vanity — reject star-farm, typo-only, comment-churn, duplicate open PRs.
- Hard + inspirational bar (non-negotiable) — Prefer work a sharp maintainer would not clear in an afternoon. Reject “correct but small” DX/docs/pattern-rebase fixes unless the user explicitly lowers the bar. Winning work should be either:
- Hard: races, protocol/semantics bugs, cross-layer design, subtle correctness with non-obvious tests, or
- Inspirational: a capability users invent workarounds for; something that changes what the tool can do, not just how you debug it.
- Human gate for upstream — for third-party GitHub PRs, stop at draft +
Take OWNER/REPO#N unless already approved.
- Reproduce / accept — bugs need a failing test or exact repro; features need acceptance checks from issues/docs/users.
- Verify with evidence — project’s own test/lint/build; report commands + exit codes.
- No fake done — ledger or session note: what changed, how verified, what remains.
Explicit rejects (fail the bar)
- Explain/debug CLIs for existing behavior
- Single-file pattern/rebase fixes with clear recipes
- Good-first-issue labeled work unless it hides a deep bug
- Anything a competent maintainer would ship between meetings
- Competing open PRs / already-assigned maintainer work you cannot uniquely advance
Workflow
Copy and track:
ADDVALUE Progress:
- [ ] 1. Target + intent
- [ ] 2. Map the system
- [ ] 3. Signal scan (issues, users, friction)
- [ ] 4. Opportunity bank + score
- [ ] 5. Pick ONE winner
- [ ] 6. Plan (smallest correct change)
- [ ] 7. Implement + verify (if in scope)
- [ ] 8. Report
1. Target + intent
Identify:
| Field |
Example |
| Target |
pypa/hatch, ~/projects/foo, . |
| Kind |
library / app / CLI / docs / monorepo / playbook |
| Intent |
contribute upstream / improve own product / invent roadmap |
| Constraint |
max 1 PR, no UI, Windows-only, etc. |
If target missing, default to current workspace root.
2. Map the system
Spend tokens on structure, not trivia:
- Entry points (CLI,
main, routes, pyproject scripts)
- Core domain modules (where value lives)
- Test/verify path (
hatch test, pytest, npm test, …)
- Contribution/AI policy if present
- Recent release notes / changelog tone
For remote repos: gh repo view, gh issue list, gh pr list, clone only when implementing.
For local / non-git projects: same mapping via filesystem + run scripts; VCS optional.
3. Signal scan
Gather friction from:
- Open issues (bugs and enhancements; “help wanted”; milestone)
- Closed-but-painful issues / “won’t fix” that aging may reopen
- Docs vs reality (README steps that fail)
- Error paths, Windows/macOS gaps, silent failures
- Competitor / sibling tools (what users already expect)
- Your own dry-run of the happy path
Optional partner pass: ask Ollama Cloud (/ollama) for a ranked opportunity list given only paths + issue titles (not full dumps).
4. Opportunity bank + score
Write 5–12 candidates. Score each 1–5 on:
| Axis |
High score means |
| User impact |
Pain or desire is real and frequent |
| Leverage |
One change unlocks many workflows |
| Fit |
Matches project style and maintainer taste |
| Feasibility |
Clear repro/acceptance; you can verify |
| Novelty |
Not already PR’d / not trivial |
| Hardness |
Would take a sharp maintainer >1 day, or needs deep domain/design |
| Inspiration |
Users will say “finally” — new capability, not polish |
| Risk |
(invert) low chance of breaking core contracts |
Winner score = impact + leverage + fit + feasibility + novelty + hardness + inspiration − risk.
Floor: reject candidates with hardness < 4 and inspiration < 4 unless the user asked for a small win.
Discard: duplicates of open PRs, policy violations, scope bombs, “rewrite the world.”
5. Pick ONE winner
State in one paragraph:
- Problem / missing capability
- Who benefits
- Why this beats the runners-up
- Out of scope for this pass
6. Plan
Minimum plan:
- Repro or acceptance checks
- Files likely touched
- Smallest design that matches existing patterns
- Verify commands
- Upstream gate (if any)
Use Plan mode when trade-offs are large; otherwise implement.
7. Implement
- Smallest diff; match project style
- Tests first when fixing bugs
- No
Co-authored-by LLM on third-party PRs
- Disclose AI if the project’s CONTRIBUTING requires it
- Stop before
gh pr create unless user said Take OWNER/REPO#N
8. Report
## ADDVALUE: <target>
**Winner:** <title> (<issue/url or local>)
**Why:** <one line>
### Also considered
- ...
### Done / next
- Implemented: ...
- Verified: `<cmd>` → exit N
- Blocked / gate: ...
Domains the review must consider
Do not only hunt bugs. Explicitly scan:
| Lens |
Questions |
| Correctness |
Silent data loss? Wrong defaults? Race? |
| Features |
What’s half-built? What’s documented but missing? |
| UX / DX |
Confusing errors? Missing --help examples? Slow feedback? |
| Perf |
Hot paths? Accidental O(n²)? |
| Security / safety |
Trust boundaries? Path traversal? Secret leakage? |
| Platform |
Windows path/newline/env gaps? |
| Docs |
Lies, missing recipes, no migration notes |
| Inspiration |
What would make a power user say “finally”? |
Anti-patterns
- Mass AI PRs / typo farms / readme churn for graph greens
- Picking an issue that already has an active PR
- “Improvements” that fight the maintainer’s stated design
- Partner model dumps that you paste as the PR body unread
- Claiming value without verify evidence
- Opening a third upstream PR while two are already waiting (unless user overrides)
Compose with other modes
| Need |
Use |
| Contract before multi-session work |
/goal |
| Autonomous until criteria |
/goal-loop |
| Quality polish after implement |
/better |
| Many unique variants |
/scale |
| Cloud planner partner |
/ollama inside or after /addvalue |
| Recurring re-scan |
/loop 1d /addvalue . |
Progressive disclosure
- Scoring templates + output schemas → reference.md
- Worked examples → examples.md
Installation paths
- Skill: this folder's
SKILL.md
- Command: optional
/addvalue command file in the host agent
1---2name: addvalue3description: Deep-reviews a GitHub repo, local project, CLI, docs, or product for high-leverage bugs, features, UX gaps, and inspiring upgrades—not typo farms. Use when the user says /addvalue, add value, find improvements, or what to build next. Do not use for a /better polish pass, a reproduce-then-fix hunt (bug-hunter), or mass good-first-issue PRs.4---5
6# /addvalue — Find and ship real value
7
8## Purpose
9
10Review a **target** carefully (remote Git repo, local folder, library, CLI, UI,
11docs site, or product surface) and produce a ranked plan of improvements that
12actually help users or maintainers. Then optionally implement the top item.
13
14This is **not** “open 20 tiny PRs.” Prefer one hard, reviewable change over many shallow ones.
15
16## Triggers
17
18- `/addvalue [path|owner/repo|.]`
19- “add value”, “how can this be improved”, “inspiring updates”, “what should we build”
20- Track B contribution hunting when quality > volume
21
22## Partners (optional)
23
24| Partner | When | How |
25|---------|------|-----|
26| **This agent (default)** | Always | Full review + implement |
27| **Ollama Cloud GLM** | User says `/addvalue --ollama` or `/ollama` mid-pass | Load `ollama` skill; one high-reasoning plan call; execute or merge with your plan |
28| **Local Ollama** | User says `/addvalue --local` or `/local` | Load local skill; use as second opinion only |
29| **agy / agent CLIs** | User names a CLI (`agy`, `hermes`, project scripts) | Prefer project engines over inventing parallel tools; pass paths not huge file dumps |
30
31Paid/cloud partners: honor the user’s **Paid Models — Ask First** rule. `/ollama` is standing approval for one GLM planner call on that turn. Ask before other paid APIs.
32
33## Hard rules
34
351. **Scope lock** — name the target and one primary outcome before editing.
362. **Read before invent** — README, CONTRIBUTING, AI_POLICY, recent issues/PRs, architecture entrypoints. No drive-by.
373. **Value > vanity** — reject star-farm, typo-only, comment-churn, duplicate open PRs.
384. **Hard + inspirational bar (non-negotiable)** — Prefer work a sharp maintainer would **not** clear in an afternoon. Reject “correct but small” DX/docs/pattern-rebase fixes unless the user explicitly lowers the bar. Winning work should be either:
39 - **Hard:** races, protocol/semantics bugs, cross-layer design, subtle correctness with non-obvious tests, or
40 - **Inspirational:** a capability users invent workarounds for; something that changes what the tool can do, not just how you debug it.
415. **Human gate for upstream** — for third-party GitHub PRs, stop at draft + `Take OWNER/REPO#N` unless already approved.
426. **Reproduce / accept** — bugs need a failing test or exact repro; features need acceptance checks from issues/docs/users.
437. **Verify with evidence** — project’s own test/lint/build; report commands + exit codes.
448. **No fake done** — ledger or session note: what changed, how verified, what remains.
45
46### Explicit rejects (fail the bar)
47
48- Explain/debug CLIs for existing behavior
49- Single-file pattern/rebase fixes with clear recipes
50- Good-first-issue labeled work unless it hides a deep bug
51- Anything a competent maintainer would ship between meetings
52- Competing open PRs / already-assigned maintainer work you cannot uniquely advance
53
54## Workflow
55
56Copy and track:
57
58```
59ADDVALUE Progress:
60- [ ] 1. Target + intent
61- [ ] 2. Map the system
62- [ ] 3. Signal scan (issues, users, friction)
63- [ ] 4. Opportunity bank + score
64- [ ] 5. Pick ONE winner
65- [ ] 6. Plan (smallest correct change)
66- [ ] 7. Implement + verify (if in scope)
67- [ ] 8. Report
68```
69
70### 1. Target + intent
71
72Identify:
73
74| Field | Example |
75|-------|---------|
76| Target | `pypa/hatch`, `~/projects/foo`, `.` |
77| Kind | library / app / CLI / docs / monorepo / playbook |
78| Intent | contribute upstream / improve own product / invent roadmap |
79| Constraint | max 1 PR, no UI, Windows-only, etc. |
80
81If target missing, default to **current workspace root**.
82
83### 2. Map the system
84
85Spend tokens on structure, not trivia:
86
87- Entry points (CLI, `main`, routes, `pyproject` scripts)
88- Core domain modules (where value lives)
89- Test/verify path (`hatch test`, `pytest`, `npm test`, …)
90- Contribution/AI policy if present
91- Recent release notes / changelog tone
92
93For remote repos: `gh repo view`, `gh issue list`, `gh pr list`, clone only when implementing.
94
95For **local / non-git** projects: same mapping via filesystem + run scripts; VCS optional.
96
97### 3. Signal scan
98
99Gather friction from:
100
101- Open issues (bugs **and** enhancements; “help wanted”; milestone)
102- Closed-but-painful issues / “won’t fix” that aging may reopen
103- Docs vs reality (README steps that fail)
104- Error paths, Windows/macOS gaps, silent failures
105- Competitor / sibling tools (what users already expect)
106- Your own dry-run of the happy path
107
108Optional partner pass: ask Ollama Cloud (`/ollama`) for a **ranked opportunity list** given only paths + issue titles (not full dumps).
109
110### 4. Opportunity bank + score
111
112Write 5–12 candidates. Score each 1–5 on:
113
114| Axis | High score means |
115|------|------------------|
116| **User impact** | Pain or desire is real and frequent |
117| **Leverage** | One change unlocks many workflows |
118| **Fit** | Matches project style and maintainer taste |
119| **Feasibility** | Clear repro/acceptance; you can verify |
120| **Novelty** | Not already PR’d / not trivial |
121| **Hardness** | Would take a sharp maintainer >1 day, or needs deep domain/design |
122| **Inspiration** | Users will say “finally” — new capability, not polish |
123| **Risk** | (invert) low chance of breaking core contracts |
124
125**Winner score** = impact + leverage + fit + feasibility + novelty + hardness + inspiration − risk.
126
127**Floor:** reject candidates with hardness < 4 **and** inspiration < 4 unless the user asked for a small win.
128
129Discard: duplicates of open PRs, policy violations, scope bombs, “rewrite the world.”
130
131### 5. Pick ONE winner
132
133State in one paragraph:
134
135- Problem / missing capability
136- Who benefits
137- Why this beats the runners-up
138- Out of scope for this pass
139
140### 6. Plan
141
142Minimum plan:
143
1441. Repro or acceptance checks
1452. Files likely touched
1463. Smallest design that matches existing patterns
1474. Verify commands
1485. Upstream gate (if any)
149
150Use Plan mode when trade-offs are large; otherwise implement.
151
152### 7. Implement
153
154- Smallest diff; match project style
155- Tests first when fixing bugs
156- No `Co-authored-by` LLM on third-party PRs
157- Disclose AI if the project’s CONTRIBUTING requires it
158- Stop before `gh pr create` unless user said `Take OWNER/REPO#N`
159
160### 8. Report
161
162```markdown
163## ADDVALUE: <target>
164
165**Winner:** <title> (<issue/url or local>)
166**Why:** <one line>
167
168### Also considered
169- ...
170
171### Done / next
172- Implemented: ...
173- Verified: `<cmd>` → exit N
174- Blocked / gate: ...
175```
176
177## Domains the review must consider
178
179Do not only hunt bugs. Explicitly scan:
180
181| Lens | Questions |
182|------|-----------|
183| **Correctness** | Silent data loss? Wrong defaults? Race? |
184| **Features** | What’s half-built? What’s documented but missing? |
185| **UX / DX** | Confusing errors? Missing `--help` examples? Slow feedback? |
186| **Perf** | Hot paths? Accidental O(n²)? |
187| **Security / safety** | Trust boundaries? Path traversal? Secret leakage? |
188| **Platform** | Windows path/newline/env gaps? |
189| **Docs** | Lies, missing recipes, no migration notes |
190| **Inspiration** | What would make a power user say “finally”? |
191
192## Anti-patterns
193
194- Mass AI PRs / typo farms / readme churn for graph greens
195- Picking an issue that already has an active PR
196- “Improvements” that fight the maintainer’s stated design
197- Partner model dumps that you paste as the PR body unread
198- Claiming value without verify evidence
199- Opening a third upstream PR while two are already waiting (unless user overrides)
200
201## Compose with other modes
202
203| Need | Use |
204|------|-----|
205| Contract before multi-session work | `/goal` |
206| Autonomous until criteria | `/goal-loop` |
207| Quality polish after implement | `/better` |
208| Many unique variants | `/scale` |
209| Cloud planner partner | `/ollama` inside or after `/addvalue` |
210| Recurring re-scan | `/loop 1d /addvalue .` |
211
212## Progressive disclosure
213
214- Scoring templates + output schemas → [reference.md](reference.md)
215- Worked examples → [examples.md](examples.md)
216
217## Installation paths
218
219- Skill: this folder's `SKILL.md`
220- Command: optional `/addvalue` command file in the host agent