Skip to content

Commit e529d85

Browse files
smypmsaclaude
andcommitted
chore: remove local .md file references from work-item skill
Make PR descriptions self-contained — reviewers should not need to read local docs to understand context. Remove hard dependency on docs/work-items.md and docs/implementation-progress.md from the finalize stage. Update Step 0 to resolve items from conversation context instead of a specific file. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent bd0bd6b commit e529d85

File tree

3 files changed

+23
-56
lines changed

3 files changed

+23
-56
lines changed

.claude/skills/work-item/SKILL.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,21 @@
11
---
22
name: work-item
3-
description: Work item pipeline for pumpfun-cli — investigate, plan, implement (TDD), verify, finalize as PR. MUST use this skill whenever the user wants to start, tackle, pick up, implement, or work on a work item from docs/work-items.md. Triggers on any mention of work items by number ("item 5", "item #5", "#5"), by name ("pre-trade balance validation", "quote command", "health check"), or by sequence ("next work item", "next item on the board", "next undone item", "next P1 item", "whatever is next"). Also triggers on phrases like "let's work on", "start implementing", "tackle item", "knock out", "pick up", "can we do item" when referring to the work items board. Does NOT trigger for: running tests, reviewing PRs, listing remaining items, fixing bugs, or explaining code.
3+
description: Work item pipeline for pumpfun-cli — investigate, plan, implement (TDD), verify, finalize as PR. MUST use this skill whenever the user wants to start, tackle, pick up, implement, or work on a work item. Triggers on any mention of work items by number ("item 5", "item #5", "#5", "B-06"), by name ("pre-trade balance validation", "quote command", "health check"), or by sequence ("next work item", "next item on the board", "next undone item", "next P1 item", "whatever is next"). Also triggers on phrases like "let's work on", "start implementing", "tackle item", "knock out", "pick up", "can we do item" when referring to work items. Does NOT trigger for: running tests, reviewing PRs, listing remaining items, fixing bugs, or explaining code.
44
---
55

66
# Work Item Pipeline
77

8-
Automate work item execution from `docs/work-items.md` through a 5-stage gated pipeline.
8+
Automate work item execution through a 5-stage gated pipeline.
99

1010
## Invocation
1111

1212
`/work-item <number>` or `/work-item` (picks next undone item).
1313

1414
## Step 0: Resolve the Target Item
1515

16-
1. Read `docs/work-items.md`
17-
2. If a number was provided, find the item with that number (e.g., `### 5.` or `### #5`)
18-
3. If no number was provided, find the lowest-numbered item NOT marked with `~~` strikethrough or `✅ Done`, following the "Recommended Implementation Order" section
16+
1. Resolve the work item from context — the user may provide a number, ID (e.g., "B-06"), title, or description. The item may come from a backlog, a conversation, or a direct request.
17+
2. If a specific item was identified, extract its details.
18+
3. If no specific item was given, ask the user which item to work on.
1919
4. Extract:
2020
- `ITEM_NUMBER`: the item number
2121
- `ITEM_TITLE`: the short title (e.g., "Pre-trade balance validation")
@@ -122,7 +122,7 @@ Automate work item execution from `docs/work-items.md` through a 5-stage gated p
122122

123123
| Failure | Response |
124124
|---------|----------|
125-
| Item not found in work-items.md | Report error, list available undone items |
125+
| Item not found or unclear | Report error, ask the user to clarify |
126126
| Item already marked Done | Report it's done, ask user to pick another |
127127
| Stage 1 finds item not feasible | Present findings, recommend abort |
128128
| Stage 3 cannot get tests green | Present blockers after 2 attempts, offer re-run/abort/manual |

.claude/skills/work-item/prompts/1-investigate.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ Investigate the current codebase and CLI behavior to understand the gap this wor
3838
- **Unit** (always): mocked tests in `tests/test_core/`, `tests/test_commands/`, `tests/test_protocol/`
3939
- **Surfpool** (if touches RPC/transaction logic): integration tests in `tests/test_surfpool/`
4040
- **Mainnet** (rare — only if surfpool cannot cover it): e2e via `./scripts/mainnet-test.sh`
41-
7. **Check for combinable items.** Read `docs/work-items.md` and check if any other undone item shares >50% of the same code path. If so, recommend combining.
41+
7. **Check for combinable items.** Check `docs/backlog.md` (if it exists) or ask the user if any other planned item shares >50% of the same code path. If so, recommend combining.
4242

4343
## Output Format
4444

Lines changed: 16 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Stage 5: Finalize Work Item #{{ITEM_NUMBER}} — {{ITEM_TITLE}}
22

3-
You are finalizing the implementation of a work item for the pumpfun-cli project. Commit the changes, create a PR, and update documentation.
3+
You are finalizing the implementation of a work item for the pumpfun-cli project. Commit the changes and create a PR.
44

55
## Verification Results
66

@@ -12,62 +12,41 @@ You are finalizing the implementation of a work item for the pumpfun-cli project
1212

1313
## Steps
1414

15-
### 1. Update docs/work-items.md
15+
### 1. Stage and commit
1616

17-
Mark item #{{ITEM_NUMBER}} as done. Change the heading from:
18-
```
19-
### N. Title
20-
```
21-
to:
22-
```
23-
### N. ~~Title~~ ✅ Done — PR [#X](URL)
24-
```
25-
26-
The PR URL will be filled in after step 5. For now, use a placeholder `TBD`.
27-
28-
### 2. Update docs/implementation-progress.md
29-
30-
Add a new section for this work item with:
31-
- Task name and status (Done)
32-
- Files modified
33-
- Key details about the implementation
34-
- Number of new tests added
35-
36-
Follow the existing format in the file.
37-
38-
### 3. Stage and commit
39-
40-
Stage only the files that were changed during implementation plus the docs you just updated. Do NOT use `git add -A` or `git add .`.
17+
Stage only the files that were changed during implementation. Do NOT use `git add -A` or `git add .`. Do NOT stage files in `docs/` or `idl/`.
4118

4219
```bash
43-
git add <each file from FILES_CHANGED> docs/work-items.md docs/implementation-progress.md
20+
git add <each file from FILES_CHANGED>
4421
git status
4522
```
4623

47-
Verify no `.env`, `wallet.enc`, `idl/`, or credential files are staged.
24+
Verify no `.env`, `wallet.enc`, `idl/`, `docs/`, or credential files are staged.
4825

4926
Commit with conventional commit format:
5027
```bash
51-
git commit -m "feat: <short description of the feature>
28+
git commit -m "fix: <short description>
5229
53-
Implements work item #{{ITEM_NUMBER}} ({{ITEM_TITLE}}).
30+
<2-3 sentence explanation of what was done and why>
5431
5532
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>"
5633
```
5734

58-
### 4. Push the feature branch
35+
### 2. Push the feature branch
5936

6037
```bash
6138
git push -u origin HEAD
6239
```
6340

64-
### 5. Create PR
41+
### 3. Create PR
42+
43+
The PR description must be **self-contained** — a reviewer should understand the full context without needing to read any external files. Do NOT reference `docs/backlog.md`, `docs/work-items.md`, or any other local documentation files in the PR title or body.
6544

6645
```bash
67-
gh pr create --title "feat: {{ITEM_TITLE}}" --body "$(cat <<'EOF'
46+
gh pr create --title "fix: {{ITEM_TITLE}}" --body "$(cat <<'EOF'
6847
## Summary
6948
70-
Implements work item #{{ITEM_NUMBER}} from `docs/work-items.md`.
49+
<Explain the problem that existed and why it matters — a reviewer unfamiliar with the backlog should understand the context from this section alone.>
7150
7251
<1-3 bullet points describing what was added/changed>
7352
@@ -92,17 +71,7 @@ EOF
9271
)"
9372
```
9473

95-
### 6. Update work-items.md with PR link
96-
97-
Now that you have the PR URL, go back and update `docs/work-items.md` to replace the `TBD` placeholder with the actual PR link. Create a second commit (do NOT amend):
98-
99-
```bash
100-
git add docs/work-items.md
101-
git commit -m "docs: add PR link for work item #{{ITEM_NUMBER}}"
102-
git push
103-
```
104-
105-
### 7. Report
74+
### 4. Report
10675

10776
Output the PR URL.
10877

@@ -113,12 +82,10 @@ Output the PR URL.
11382
**Branch:** <branch name>
11483
**Title:** <PR title>
11584

116-
## Docs Updated
117-
- work-items.md: item #{{ITEM_NUMBER}} marked Done
118-
- implementation-progress.md: section added
119-
12085
## CONSTRAINTS
12186

12287
- Do NOT commit `.env`, `wallet.enc`, or credential files.
88+
- Do NOT stage or commit files in `docs/` or `idl/`.
12389
- Do NOT push to `main` directly. Always use feature branch + PR.
12490
- Do NOT merge the PR. That is the user's decision.
91+
- Do NOT reference local .md files (backlog, work-items, etc.) in the PR description. The PR must be self-contained.

0 commit comments

Comments
 (0)