create-pathfinder 1.7.0 → 2.0.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/AGENTS.md +3 -1
- package/CLAUDE.md +22 -0
- package/README.md +383 -65
- package/context/ai-interaction.md +78 -38
- package/context/coding-standards.md +119 -60
- package/copy-list.json +1 -0
- package/package.json +2 -1
- package/roles/developer.md +37 -0
- package/roles/planner.md +38 -0
- package/roles/tester.md +40 -0
- package/skills/challenge-me/SKILL.md +11 -7
- package/skills/complete-feature/SKILL.md +16 -10
- package/skills/handoff/SKILL.md +24 -4
- package/skills/kickstart-pathfinder/SKILL.md +6 -1
- package/skills/learning-review/SKILL.md +9 -6
- package/skills/load-feature/SKILL.md +20 -9
- package/skills/prototype/SKILL.md +3 -1
- package/skills/quiz-me/SKILL.md +33 -7
- package/skills/review-feature/SKILL.md +19 -11
- package/skills/role/SKILL.md +33 -0
- package/skills/setup-tracker/SKILL.md +42 -0
- package/skills/start-feature/SKILL.md +19 -15
- package/skills/sync-tracker/SKILL.md +81 -0
- package/skills/teach-architecture/SKILL.md +6 -5
- package/skills/teach-feature/SKILL.md +11 -8
- package/skills/to-specs/SKILL.md +41 -25
- package/skills/whereami/SKILL.md +87 -0
- package/src/cli.mjs +2 -1
- package/src/detect.mjs +1 -1
- package/src/install.mjs +5 -1
- package/src/kickstart-prompt.mjs +1 -1
- package/src/kit.mjs +70 -1
- package/templates/CHANGELOG.template.md +2 -1
- package/templates/feature-spec.template.md +19 -75
- package/templates/history.template.md +12 -0
- package/templates/lesson.template.md +22 -51
- package/templates/project-overview.template.md +80 -145
- package/context/current-feature.md +0 -46
- package/context/features/example-feature-spec.md +0 -103
- package/context/history.md +0 -14
- package/context/learning/learner-profile.md +0 -35
- package/context/learning/lessons/.gitkeep +0 -0
- package/context/learning/progress.md +0 -28
- package/context/project-overview.md +0 -211
- package/templates/progress-entry.template.md +0 -23
|
@@ -1,76 +1,116 @@
|
|
|
1
1
|
# AI Interaction Guidelines
|
|
2
2
|
|
|
3
|
-
This file defines
|
|
3
|
+
This file defines the default rules for AI-assisted work.
|
|
4
|
+
Project-specific instructions override these defaults when explicit.
|
|
4
5
|
|
|
5
6
|
## Communication
|
|
6
7
|
|
|
7
8
|
- Be concise, direct, and honest.
|
|
8
|
-
-
|
|
9
|
+
- Separate facts, assumptions, recommendations, and unresolved decisions.
|
|
9
10
|
- Do not invent answers for `TBD` items.
|
|
10
|
-
-
|
|
11
|
+
- After 2–3 grounded failed approaches, stop and explain the blocker.
|
|
11
12
|
|
|
12
|
-
## Approval
|
|
13
|
+
## Human Approval
|
|
13
14
|
|
|
14
|
-
|
|
15
|
+
Ask before:
|
|
15
16
|
|
|
16
|
-
-
|
|
17
|
-
- architecture migrations
|
|
17
|
+
- architecture or dependency changes
|
|
18
18
|
- database, auth, payment, secrets, or security-sensitive changes
|
|
19
19
|
- destructive commands or file deletion
|
|
20
20
|
- Git history rewriting
|
|
21
21
|
- commits, merges, releases, or deployments
|
|
22
22
|
- adopting prototype code into production
|
|
23
|
+
- writes outside the repository, such as shared tracker changes
|
|
24
|
+
|
|
25
|
+
The human owns judgment, acceptance, merge, and release decisions.
|
|
23
26
|
|
|
24
27
|
## Git and Delivery
|
|
25
28
|
|
|
26
|
-
Follow `context/project-overview.md`.
|
|
29
|
+
Follow `context/project-overview.md`.
|
|
27
30
|
|
|
28
|
-
|
|
31
|
+
Do not assume branch strategy, commit style, pull requests, versioning,
|
|
32
|
+
or deployment workflow.
|
|
29
33
|
|
|
30
|
-
|
|
34
|
+
Inspect current Git state before acting. If the workflow is unclear or
|
|
35
|
+
`TBD`, ask.
|
|
31
36
|
|
|
32
|
-
|
|
33
|
-
2. `start-feature` implements one delivery chunk at a time.
|
|
34
|
-
3. `review-feature` reports findings against the spec and repository standards.
|
|
35
|
-
4. `complete-feature` verifies acceptance criteria and updates durable records.
|
|
36
|
-
5. `learn-feature` may generate an interactive lesson after completion.
|
|
37
|
+
## Feature Workflow
|
|
37
38
|
|
|
38
|
-
|
|
39
|
+
Use the workflow skills instead of recreating their procedures in chat:
|
|
40
|
+
|
|
41
|
+
1. `load-feature` — load the active work and relevant context.
|
|
42
|
+
2. `start-feature` — implement the current delivery chunk.
|
|
43
|
+
3. `review-feature` — verify the work and report findings.
|
|
44
|
+
4. `complete-feature` — complete accepted work and durable records.
|
|
45
|
+
5. `learn-feature` — optionally teach what was implemented.
|
|
46
|
+
|
|
47
|
+
Roles are optional. The workflow must work without activating one.
|
|
48
|
+
|
|
49
|
+
### Status
|
|
39
50
|
|
|
40
|
-
|
|
41
|
-
- Use the feature's Context Boundary and Delivery Chunks.
|
|
42
|
-
- Avoid repo-wide scans unless the task genuinely requires them.
|
|
43
|
-
- Split the feature when a focused session cannot safely understand, implement, and verify it.
|
|
44
|
-
- Never hide excessive scope behind a long checklist.
|
|
51
|
+
Feature status records durable lifecycle state only:
|
|
45
52
|
|
|
46
|
-
|
|
53
|
+
`Proposed` → `Ready` → `In Progress` → `Complete`
|
|
47
54
|
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
-
|
|
51
|
-
-
|
|
52
|
-
|
|
55
|
+
`Cancelled` and `Superseded` are terminal alternatives.
|
|
56
|
+
|
|
57
|
+
- `Ready` means the human approved execution.
|
|
58
|
+
- Review and testing are optional workflow activity, not a status.
|
|
59
|
+
A Feature stays `In Progress` until it is complete.
|
|
60
|
+
- `Blocked` is not a status; record the blocker in current workspace state.
|
|
61
|
+
- The human decides approval, acceptance, cancellation, and supersession.
|
|
62
|
+
|
|
63
|
+
## Context Discipline
|
|
64
|
+
|
|
65
|
+
- Read only what the current work requires.
|
|
66
|
+
- Prefer exact files or sections over broad repository scans.
|
|
67
|
+
- Do not load history, roadmap, tracker data, or unrelated context by
|
|
68
|
+
default.
|
|
69
|
+
- Work one delivery chunk at a time.
|
|
70
|
+
- If the work can no longer be understood safely in a focused session,
|
|
71
|
+
stop and split or hand off.
|
|
72
|
+
- Extra scaffolding must earn its cost by reducing downstream context.
|
|
53
73
|
|
|
54
74
|
## Scope Control
|
|
55
75
|
|
|
56
|
-
|
|
76
|
+
Stay inside the approved work.
|
|
77
|
+
|
|
78
|
+
Do not add unrelated refactors, features, dependency changes, visual
|
|
79
|
+
redesigns, speculative abstractions, or later roadmap work.
|
|
57
80
|
|
|
58
|
-
|
|
81
|
+
When necessary work falls outside scope, stop and ask.
|
|
59
82
|
|
|
60
|
-
|
|
83
|
+
## Prototypes
|
|
84
|
+
|
|
85
|
+
Use `debate-me` to decide whether a prototype is useful.
|
|
86
|
+
|
|
87
|
+
Use `prototype` to test one important assumption at a time.
|
|
88
|
+
|
|
89
|
+
Prototype output is evidence, not production code, until the human
|
|
90
|
+
explicitly approves adoption.
|
|
91
|
+
|
|
92
|
+
## Review
|
|
93
|
+
|
|
94
|
+
Review against the Feature and the actual diff.
|
|
95
|
+
|
|
96
|
+
Prioritize:
|
|
61
97
|
|
|
62
98
|
1. correctness
|
|
63
|
-
2. security
|
|
99
|
+
2. security and privacy
|
|
64
100
|
3. regressions and edge cases
|
|
65
|
-
4. accessibility when
|
|
101
|
+
4. accessibility when relevant
|
|
66
102
|
5. performance
|
|
67
103
|
6. maintainability
|
|
68
|
-
|
|
69
|
-
|
|
104
|
+
|
|
105
|
+
Verification effort should be proportional to risk.
|
|
106
|
+
|
|
107
|
+
Report findings; do not manufacture them.
|
|
70
108
|
|
|
71
109
|
## Learning
|
|
72
110
|
|
|
73
|
-
|
|
74
|
-
|
|
75
|
-
|
|
76
|
-
|
|
111
|
+
Explain what was actually implemented.
|
|
112
|
+
|
|
113
|
+
Use examples, diagrams, or quizzes only when they improve understanding.
|
|
114
|
+
|
|
115
|
+
Keep Feature learning scoped. Use `learn-codebase` for broad repository
|
|
116
|
+
study.
|
|
@@ -1,91 +1,150 @@
|
|
|
1
1
|
# Coding Standards
|
|
2
2
|
|
|
3
|
-
These
|
|
3
|
+
These are stack-agnostic defaults.
|
|
4
|
+
Project-specific conventions override them when explicit.
|
|
4
5
|
|
|
5
6
|
## Core Principles
|
|
6
7
|
|
|
7
8
|
- Prefer readable, explicit code over clever code.
|
|
8
9
|
- Keep modules cohesive and boundaries clear.
|
|
9
|
-
- Validate data
|
|
10
|
-
- Make
|
|
11
|
-
- Optimize for future humans and AI agents reading the code.
|
|
10
|
+
- Validate data at system boundaries.
|
|
11
|
+
- Make failures visible and diagnosable without exposing sensitive data.
|
|
12
12
|
- Follow existing repository conventions unless an approved decision changes them.
|
|
13
|
+
- Optimize for code that future humans and AI agents can understand quickly.
|
|
13
14
|
|
|
14
|
-
##
|
|
15
|
+
## Scope
|
|
15
16
|
|
|
16
|
-
- Work on one coherent
|
|
17
|
-
-
|
|
18
|
-
- Avoid
|
|
19
|
-
- Keep the
|
|
20
|
-
- Split work when too many
|
|
17
|
+
- Work on one coherent change at a time.
|
|
18
|
+
- Read only the files and dependencies needed for that work.
|
|
19
|
+
- Avoid unrelated refactors, formatting churn, or speculative cleanup.
|
|
20
|
+
- Keep the repository stable and verifiable after each meaningful change.
|
|
21
|
+
- Split work when too many concerns must be understood at once.
|
|
21
22
|
|
|
22
|
-
##
|
|
23
|
+
## Contracts
|
|
23
24
|
|
|
24
|
-
- Use the strongest practical contract mechanism available in the
|
|
25
|
-
- Avoid
|
|
26
|
-
-
|
|
25
|
+
- Use the strongest practical contract mechanism available in the stack.
|
|
26
|
+
- Avoid unchecked escape hatches unless justified.
|
|
27
|
+
- Validate external data, configuration, user input, and integration boundaries.
|
|
28
|
+
- Keep public contracts explicit and compatibility-conscious.
|
|
27
29
|
|
|
28
|
-
## User
|
|
30
|
+
## User Interfaces — When Applicable
|
|
29
31
|
|
|
30
|
-
-
|
|
31
|
-
-
|
|
32
|
-
-
|
|
32
|
+
- Support the project's required screen sizes and input methods.
|
|
33
|
+
- Prefer semantic platform elements.
|
|
34
|
+
- Handle accessibility requirements such as keyboard use, focus, labels,
|
|
35
|
+
contrast, reduced motion, and assistive technology.
|
|
33
36
|
- Handle relevant loading, empty, error, success, disabled, and permission states.
|
|
34
|
-
-
|
|
37
|
+
- Reuse shared primitives when they improve consistency.
|
|
35
38
|
|
|
36
|
-
##
|
|
39
|
+
## APIs, Services, CLIs, Libraries, and Infrastructure — When Applicable
|
|
37
40
|
|
|
38
|
-
-
|
|
39
|
-
-
|
|
40
|
-
-
|
|
41
|
-
-
|
|
42
|
-
- Avoid leaking implementation details through public interfaces.
|
|
41
|
+
- Keep public interfaces clear and documented.
|
|
42
|
+
- Provide safe defaults and actionable errors.
|
|
43
|
+
- Preserve idempotency when operations may be repeated.
|
|
44
|
+
- Avoid leaking internal implementation details through public contracts.
|
|
43
45
|
|
|
44
46
|
## Testing
|
|
45
47
|
|
|
46
|
-
Choose tests
|
|
48
|
+
Choose tests based on behavior and risk.
|
|
49
|
+
|
|
50
|
+
Use:
|
|
47
51
|
|
|
48
52
|
- unit tests for isolated logic
|
|
49
|
-
-
|
|
50
|
-
-
|
|
51
|
-
-
|
|
52
|
-
|
|
53
|
-
|
|
54
|
-
|
|
55
|
-
|
|
56
|
-
|
|
57
|
-
|
|
58
|
-
|
|
59
|
-
|
|
60
|
-
|
|
61
|
-
-
|
|
62
|
-
|
|
63
|
-
|
|
64
|
-
|
|
65
|
-
|
|
66
|
-
- **When a mechanism can fail by doing nothing, observe that the right thing
|
|
67
|
-
happened.** A clean exit says the command ran. It does not say the effect
|
|
68
|
-
occurred, and a step that silently does nothing usually reports success.
|
|
69
|
-
- **Beware a check whose every input it supplied itself.** If the test chose the
|
|
70
|
-
fixtures, the environment, and the trigger, it has confirmed its own
|
|
71
|
-
assumptions. Name which inputs came from the real system.
|
|
72
|
-
- State what was verified and what was only reasoned about. An unobserved
|
|
73
|
-
criterion is recorded as unobserved, not as passed.
|
|
53
|
+
- integration tests for important boundaries
|
|
54
|
+
- end-to-end or system tests for critical user flows
|
|
55
|
+
- contract tests when independent systems share a contract
|
|
56
|
+
|
|
57
|
+
Avoid tests that only reproduce implementation details.
|
|
58
|
+
|
|
59
|
+
## Verification
|
|
60
|
+
|
|
61
|
+
Verify the behavior that matters, not just that a command exited successfully.
|
|
62
|
+
|
|
63
|
+
- Prefer evidence from the real artifact or behavior being claimed.
|
|
64
|
+
- When something can silently do nothing, verify that the intended effect occurred.
|
|
65
|
+
- Beware a check whose every input it supplied itself. Name which inputs came
|
|
66
|
+
from the real system.
|
|
67
|
+
- Distinguish what was observed from what was only reasoned about.
|
|
68
|
+
- Record unverified behavior as unverified, not passed.
|
|
69
|
+
- Increase verification effort when the cost of being wrong is higher.
|
|
74
70
|
|
|
75
71
|
## Dependencies
|
|
76
72
|
|
|
77
|
-
Before adding
|
|
73
|
+
Before adding a dependency, consider whether it is necessary and whether a
|
|
74
|
+
simpler existing option is sufficient.
|
|
75
|
+
|
|
76
|
+
Follow the approval rules in `context/ai-interaction.md`.
|
|
78
77
|
|
|
79
78
|
## Prototype Code
|
|
80
79
|
|
|
81
|
-
|
|
82
|
-
|
|
83
|
-
|
|
80
|
+
Prototype code is disposable by default.
|
|
81
|
+
|
|
82
|
+
Do not treat it as production-ready merely because it works.
|
|
83
|
+
|
|
84
|
+
Adopt prototype work only through an approved production change with the
|
|
85
|
+
necessary validation, tests, security, accessibility, and maintainability.
|
|
84
86
|
|
|
85
87
|
## Documentation
|
|
86
88
|
|
|
87
|
-
|
|
88
|
-
|
|
89
|
-
-
|
|
90
|
-
|
|
91
|
-
-
|
|
89
|
+
Keep durable truth in its appropriate project artifact.
|
|
90
|
+
|
|
91
|
+
- project-wide product and architecture context:
|
|
92
|
+
`context/project-overview.md`
|
|
93
|
+
- Feature contracts:
|
|
94
|
+
`context/features/`
|
|
95
|
+
- completed outcomes:
|
|
96
|
+
`context/history.md`
|
|
97
|
+
- active workspace state:
|
|
98
|
+
`context/current-feature.md`
|
|
99
|
+
- state handed to the next session:
|
|
100
|
+
`context/handoff.md`
|
|
101
|
+
|
|
102
|
+
None of these ships with the kit. Each is written by the workflow that first
|
|
103
|
+
needs it, so a missing one is normal and is not an error.
|
|
104
|
+
|
|
105
|
+
Learning material explains the implementation but does not replace source-of-truth documentation.
|
|
106
|
+
|
|
107
|
+
## Version control for `context/`
|
|
108
|
+
|
|
109
|
+
`context/` holds two kinds of file, and they belong on opposite sides of
|
|
110
|
+
`.gitignore`.
|
|
111
|
+
|
|
112
|
+
**Track durable project truth.** It is the answer to "what is true about this
|
|
113
|
+
project", it outlives any session, and a reviewer should see it change:
|
|
114
|
+
|
|
115
|
+
```text
|
|
116
|
+
context/project-overview.md
|
|
117
|
+
context/features/
|
|
118
|
+
context/history.md
|
|
119
|
+
context/tracker.md
|
|
120
|
+
```
|
|
121
|
+
|
|
122
|
+
**Ignore transient workspace state.** It is the answer to "what was I doing",
|
|
123
|
+
it belongs to one session on one machine, and committing it puts one person's
|
|
124
|
+
in-flight work in everybody's diff:
|
|
125
|
+
|
|
126
|
+
```text
|
|
127
|
+
context/current-feature.md
|
|
128
|
+
context/handoff.md
|
|
129
|
+
```
|
|
130
|
+
|
|
131
|
+
Two lines in `.gitignore` are the whole mechanism:
|
|
132
|
+
|
|
133
|
+
```text
|
|
134
|
+
context/current-feature.md
|
|
135
|
+
context/handoff.md
|
|
136
|
+
```
|
|
137
|
+
|
|
138
|
+
**Do not ignore `context/` as a directory.** It is the one mistake worth naming,
|
|
139
|
+
because it looks tidier and quietly untracks the project truth every later
|
|
140
|
+
session depends on — including the file that documents your stack and workflow.
|
|
141
|
+
Ignore the two transient files by name.
|
|
142
|
+
|
|
143
|
+
A team that would rather share workspace state — a single-machine project, or a
|
|
144
|
+
handoff meant to be read by a colleague — can track them instead. Nothing in the
|
|
145
|
+
kit reads Git state to decide how to behave.
|
|
146
|
+
|
|
147
|
+
`context/tracker.md` is durable and tracked, with one exception that does not
|
|
148
|
+
apply to your project: Pathfinder's own repository ignores it, because `context`
|
|
149
|
+
is a directory in the installer's copy list and a committed copy would ship
|
|
150
|
+
Pathfinder's tracker config to every new install.
|
package/copy-list.json
CHANGED
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "create-pathfinder",
|
|
3
|
-
"version": "
|
|
3
|
+
"version": "2.0.0",
|
|
4
4
|
"description": "Install the Pathfinder AI-assisted, human-in-the-loop workflow kit into a Git repository.",
|
|
5
5
|
"keywords": [
|
|
6
6
|
"pathfinder",
|
|
@@ -34,6 +34,7 @@
|
|
|
34
34
|
"AGENTS.md",
|
|
35
35
|
"CLAUDE.md",
|
|
36
36
|
"context",
|
|
37
|
+
"roles",
|
|
37
38
|
"skills",
|
|
38
39
|
"templates"
|
|
39
40
|
],
|
|
@@ -0,0 +1,37 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: developer
|
|
3
|
+
description: Implements approved work without deciding whether its own work is acceptable.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Developer
|
|
7
|
+
|
|
8
|
+
## Responsibility
|
|
9
|
+
|
|
10
|
+
Implement the approved current work and leave the repository stable.
|
|
11
|
+
|
|
12
|
+
## Context
|
|
13
|
+
|
|
14
|
+
Read only what the active Feature and implementation require.
|
|
15
|
+
|
|
16
|
+
Do not load unrelated history, roadmap, Features, or repository areas by default.
|
|
17
|
+
|
|
18
|
+
## Use
|
|
19
|
+
|
|
20
|
+
- `load-feature` to load the work.
|
|
21
|
+
- `start-feature` to implement it.
|
|
22
|
+
- Use the project's existing build, test, and verification commands as needed.
|
|
23
|
+
|
|
24
|
+
## Rules
|
|
25
|
+
|
|
26
|
+
- Stay inside the approved scope.
|
|
27
|
+
- Follow existing project conventions.
|
|
28
|
+
- Verify the behavior you change.
|
|
29
|
+
- Do not silently make decisions that require human approval.
|
|
30
|
+
- Do not judge your own work as accepted.
|
|
31
|
+
- Stop and ask when required work would materially widen the scope.
|
|
32
|
+
|
|
33
|
+
## Finish
|
|
34
|
+
|
|
35
|
+
Report what changed, what was verified, and anything unresolved.
|
|
36
|
+
|
|
37
|
+
The human decides what happens next, including whether a `tester` session is needed.
|
package/roles/planner.md
ADDED
|
@@ -0,0 +1,38 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: planner
|
|
3
|
+
description: Turns approved direction into small, implementable Feature specs.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Planner
|
|
7
|
+
|
|
8
|
+
## Responsibility
|
|
9
|
+
|
|
10
|
+
Turn approved direction into clear Feature specs for another session to
|
|
11
|
+
implement.
|
|
12
|
+
|
|
13
|
+
## Context
|
|
14
|
+
|
|
15
|
+
Read only the project context and existing work needed to plan accurately.
|
|
16
|
+
|
|
17
|
+
Inspect implementation code only when necessary to understand the work.
|
|
18
|
+
|
|
19
|
+
## Use
|
|
20
|
+
|
|
21
|
+
- `debate-me` when direction still needs pressure-testing.
|
|
22
|
+
- `to-specs` to create Feature specs.
|
|
23
|
+
|
|
24
|
+
## Rules
|
|
25
|
+
|
|
26
|
+
- Do not implement the Features you plan.
|
|
27
|
+
- Do not silently resolve human decisions or `TBD` items.
|
|
28
|
+
- Keep Features small and focused.
|
|
29
|
+
- Do not add workflow metadata that does not help implementation.
|
|
30
|
+
- Prefer fewer Features and fewer artifacts.
|
|
31
|
+
- Record dependencies or technical constraints only when they materially
|
|
32
|
+
affect the work.
|
|
33
|
+
|
|
34
|
+
## Finish
|
|
35
|
+
|
|
36
|
+
Produce the Feature specs and clearly identify any unresolved human decisions.
|
|
37
|
+
|
|
38
|
+
Stop before implementation.
|
package/roles/tester.md
ADDED
|
@@ -0,0 +1,40 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: tester
|
|
3
|
+
description: Verifies delivered work against its acceptance criteria and reports findings without repairing them.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Tester
|
|
7
|
+
|
|
8
|
+
## Responsibility
|
|
9
|
+
|
|
10
|
+
Verify that the delivered work behaves as the Feature requires.
|
|
11
|
+
|
|
12
|
+
Report what is actually observed, not what the implementation intended.
|
|
13
|
+
|
|
14
|
+
## Context
|
|
15
|
+
|
|
16
|
+
Read only what testing requires: the Feature spec, the relevant diff, and
|
|
17
|
+
relevant tests or running behavior.
|
|
18
|
+
|
|
19
|
+
Do not rely on the developer's summary as proof.
|
|
20
|
+
|
|
21
|
+
## Use
|
|
22
|
+
|
|
23
|
+
- `review-feature` to verify implemented work.
|
|
24
|
+
- Use the project's existing test commands and relevant testing tools.
|
|
25
|
+
|
|
26
|
+
## Rules
|
|
27
|
+
|
|
28
|
+
- Test against the acceptance criteria.
|
|
29
|
+
- Verify important behavior directly when practical.
|
|
30
|
+
- Report real findings clearly, and do not invent findings to justify the role.
|
|
31
|
+
- Do not repair what you find in the same role.
|
|
32
|
+
- Passing tests are evidence, not automatic acceptance.
|
|
33
|
+
- The human decides whether the work is accepted.
|
|
34
|
+
|
|
35
|
+
## Finish
|
|
36
|
+
|
|
37
|
+
Report pass or findings, what was actually verified, and anything important
|
|
38
|
+
that remains unverified.
|
|
39
|
+
|
|
40
|
+
Stop before implementation.
|
|
@@ -12,11 +12,12 @@ A good challenge changes the context while preserving the underlying concept.
|
|
|
12
12
|
|
|
13
13
|
## Read First
|
|
14
14
|
|
|
15
|
-
1.
|
|
16
|
-
2.
|
|
17
|
-
3.
|
|
18
|
-
4.
|
|
19
|
-
|
|
15
|
+
1. The relevant lesson
|
|
16
|
+
2. `context/learning/progress.md`
|
|
17
|
+
3. The feature spec and focused implementation when needed
|
|
18
|
+
4. Current git status if the challenge may involve code
|
|
19
|
+
|
|
20
|
+
Skip any of these that does not exist. Pathfinder creates these files only when a workflow first needs them, so their absence is normal and is not an error. Do not create them just to satisfy this list.
|
|
20
21
|
|
|
21
22
|
## Challenge Types
|
|
22
23
|
|
|
@@ -77,5 +78,8 @@ After the learner responds:
|
|
|
77
78
|
- Identify the strongest decision.
|
|
78
79
|
- Identify the most important weakness.
|
|
79
80
|
- Explain a stronger solution.
|
|
80
|
-
- Update `context/learning/progress.md
|
|
81
|
-
|
|
81
|
+
- Update `context/learning/progress.md`, using the confidence scale defined in
|
|
82
|
+
its `## Confidence Scale` section. If the file does not exist, `quiz-me`
|
|
83
|
+
carries the header to create it with.
|
|
84
|
+
- Mark `transferable` only when the concept was applied correctly in the changed
|
|
85
|
+
context, and never using the example that introduced it.
|
|
@@ -1,17 +1,23 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: complete-feature
|
|
3
|
-
description:
|
|
3
|
+
description: Complete accepted work through the project's delivery workflow and durable records.
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Complete Feature
|
|
7
7
|
|
|
8
|
-
|
|
9
|
-
2. Run final project-appropriate verification.
|
|
10
|
-
3. Confirm each acceptance criterion with evidence.
|
|
11
|
-
4. Follow the approved commit, PR, merge, changelog, versioning, release, and deployment policy—requesting approval where required. After the merge, verify the merged mainline and clean up the merged branch as that policy requires.
|
|
12
|
-
5. Append the durable outcome to `context/history.md` while completing the feature, not afterwards. If the feature was merged without this skill running, still write the entry and record that it was written after the fact.
|
|
13
|
-
6. Mark/reset `context/current-feature.md` and identify the next action.
|
|
14
|
-
7. Offer or invoke `learn-feature` when learning is enabled.
|
|
15
|
-
8. Produce a compact completion summary.
|
|
8
|
+
Complete only work the human has accepted.
|
|
16
9
|
|
|
17
|
-
|
|
10
|
+
1. Confirm the Feature is accepted.
|
|
11
|
+
2. Run the project's required final checks.
|
|
12
|
+
3. Follow the documented Git, merge, version, release, and deployment workflow.
|
|
13
|
+
Ask for approval where required.
|
|
14
|
+
4. Mark the Feature complete.
|
|
15
|
+
5. Add a compact entry to `context/history.md`.
|
|
16
|
+
Create it from `templates/history.template.md` if it does not exist.
|
|
17
|
+
6. Clear or update `context/current-feature.md`. Do nothing if it does not exist.
|
|
18
|
+
7. If work tracking is configured, sync the completed state.
|
|
19
|
+
8. Report the completed outcome and any remaining follow-up.
|
|
20
|
+
|
|
21
|
+
Do not re-review accepted work unless final verification exposes a new problem.
|
|
22
|
+
|
|
23
|
+
Do not claim completion if required checks or delivery steps failed.
|
package/skills/handoff/SKILL.md
CHANGED
|
@@ -1,12 +1,32 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: handoff
|
|
3
|
-
description:
|
|
3
|
+
description: Leave the minimum factual state another session needs to continue safely.
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Handoff
|
|
7
7
|
|
|
8
|
-
|
|
8
|
+
Create or overwrite `context/handoff.md`.
|
|
9
9
|
|
|
10
|
-
|
|
10
|
+
Read only what is needed to describe the current state:
|
|
11
|
+
the active Feature, relevant Git state, and verification already performed.
|
|
11
12
|
|
|
12
|
-
|
|
13
|
+
Do not load history, roadmap, unrelated Features, or broad repository context
|
|
14
|
+
by default.
|
|
15
|
+
|
|
16
|
+
Write:
|
|
17
|
+
|
|
18
|
+
# Handoff
|
|
19
|
+
|
|
20
|
+
- Feature: `[feature or work]`
|
|
21
|
+
- Done: `[what was completed]`
|
|
22
|
+
- Verified: `[what was actually checked]`
|
|
23
|
+
- Open: `[blocker, finding, or none]`
|
|
24
|
+
- Next: `[single next action]`
|
|
25
|
+
- Read next: `[specific file(s), only when useful]`
|
|
26
|
+
|
|
27
|
+
Keep it factual and short.
|
|
28
|
+
|
|
29
|
+
Do not repeat requirements, acceptance criteria, project history, or decisions
|
|
30
|
+
already recorded in their canonical location.
|
|
31
|
+
|
|
32
|
+
Do not include private chain-of-thought or conversation transcripts.
|
|
@@ -29,7 +29,7 @@ Ask only what materially affects the project:
|
|
|
29
29
|
7. Quality priorities: security, accessibility, performance, reliability, privacy, localization
|
|
30
30
|
8. Deployment, environments, cost limits, and external services
|
|
31
31
|
9. Git, review, CI/CD, versioning, changelog, and release preferences
|
|
32
|
-
10. Prototype and learning goals
|
|
32
|
+
10. Prototype direction and learning goals
|
|
33
33
|
11. AI tools and actions requiring human approval
|
|
34
34
|
|
|
35
35
|
When the user is unsure, provide a small recommendation with reasoning and alternatives.
|
|
@@ -42,6 +42,11 @@ When the user is unsure, provide a small recommendation with reasoning and alter
|
|
|
42
42
|
4. Route uncertain product/technical choices to `debate-me` when useful.
|
|
43
43
|
5. Present the proposed context and request human corrections or approval.
|
|
44
44
|
6. Update `context/project-overview.md`, `context/coding-standards.md`, `context/ai-interaction.md`, `CLAUDE.md`, and `AGENTS.md` only after the choices are sufficiently clear.
|
|
45
|
+
`context/project-overview.md` does not ship; create it from `templates/project-overview.template.md` at this step.
|
|
46
|
+
Project facts go there; approval rules and tool actions requiring a human go
|
|
47
|
+
in `context/ai-interaction.md`. Fill the sections the project has and mark
|
|
48
|
+
the rest `TBD` or `None`. Do not add sections the template does not carry,
|
|
49
|
+
and do not leave a field blank.
|
|
45
50
|
7. Recommend `debate-me`, `prototype`, or `to-specs` as the next action.
|
|
46
51
|
|
|
47
52
|
## Stop Condition
|