maxsimcli 4.0.1 → 4.0.2
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/dist/assets/CHANGELOG.md +7 -0
- package/dist/assets/templates/skills/batch-worktree/SKILL.md +38 -85
- package/dist/assets/templates/skills/brainstorming/SKILL.md +44 -114
- package/dist/assets/templates/skills/code-review/SKILL.md +43 -71
- package/dist/assets/templates/skills/memory-management/SKILL.md +36 -100
- package/dist/assets/templates/skills/roadmap-writing/SKILL.md +39 -73
- package/dist/assets/templates/skills/sdd/SKILL.md +36 -85
- package/dist/assets/templates/skills/simplify/SKILL.md +96 -139
- package/dist/assets/templates/skills/systematic-debugging/SKILL.md +41 -74
- package/dist/assets/templates/skills/tdd/SKILL.md +32 -65
- package/dist/assets/templates/skills/using-maxsim/SKILL.md +26 -39
- package/dist/assets/templates/skills/verification-before-completion/SKILL.md +37 -56
- package/package.json +1 -1
|
@@ -1,36 +1,33 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: using-maxsim
|
|
3
|
-
description: Entry skill that establishes MAXSIM workflow rules — triggers before any action to route work through the correct MAXSIM commands, skills, and agents
|
|
4
3
|
alwaysApply: true
|
|
4
|
+
description: >-
|
|
5
|
+
Routes all work through MAXSIM's spec-driven workflow: checks for planning
|
|
6
|
+
directory, determines active phase, and dispatches to the correct MAXSIM
|
|
7
|
+
command. Use when starting any work session, resuming work, or when unsure
|
|
8
|
+
which MAXSIM command to run.
|
|
5
9
|
---
|
|
6
10
|
|
|
7
11
|
# Using MAXSIM
|
|
8
12
|
|
|
9
|
-
MAXSIM is a spec-driven development system. Work flows through phases, plans, and tasks
|
|
13
|
+
MAXSIM is a spec-driven development system. Work flows through phases, plans, and tasks -- not ad-hoc coding.
|
|
10
14
|
|
|
11
|
-
**
|
|
12
|
-
|
|
13
|
-
## The Iron Law
|
|
14
|
-
|
|
15
|
-
<HARD-GATE>
|
|
16
|
-
NO IMPLEMENTATION WITHOUT A PLAN.
|
|
17
|
-
If there is no .planning/ directory, run `/maxsim:init` first.
|
|
15
|
+
**HARD GATE -- No implementation without a plan.**
|
|
16
|
+
If there is no `.planning/` directory, run `/maxsim:init` first.
|
|
18
17
|
If there is no current phase, run `/maxsim:plan-phase` first.
|
|
19
18
|
If there is no PLAN.md for the current phase, run `/maxsim:plan-phase` first.
|
|
20
19
|
If there IS a plan, run `/maxsim:execute-phase` to execute it.
|
|
21
|
-
Skipping the workflow is a violation — not a shortcut.
|
|
22
|
-
</HARD-GATE>
|
|
23
20
|
|
|
24
|
-
##
|
|
21
|
+
## Process
|
|
25
22
|
|
|
26
|
-
|
|
23
|
+
Before starting any task:
|
|
27
24
|
|
|
28
|
-
1. **Check for `.planning/` directory**
|
|
29
|
-
2. **Check STATE.md**
|
|
30
|
-
3. **Check current phase**
|
|
31
|
-
4. **Route to the correct command** based on the
|
|
25
|
+
1. **Check for `.planning/` directory** -- if missing, initialize with `/maxsim:init`
|
|
26
|
+
2. **Check STATE.md** -- resume from last checkpoint if one exists
|
|
27
|
+
3. **Check current phase** -- determine what phase is active in ROADMAP.md
|
|
28
|
+
4. **Route to the correct command** based on the routing table below
|
|
32
29
|
|
|
33
|
-
|
|
30
|
+
### Routing Table
|
|
34
31
|
|
|
35
32
|
| Situation | Route To |
|
|
36
33
|
|-----------|----------|
|
|
@@ -39,18 +36,18 @@ This skill applies to ALL work sessions. Before starting any task:
|
|
|
39
36
|
| Active phase has no PLAN.md | `/maxsim:plan-phase` |
|
|
40
37
|
| Active phase has PLAN.md, not started | `/maxsim:execute-phase` |
|
|
41
38
|
| Checkpoint exists in STATE.md | `/maxsim:resume-work` |
|
|
42
|
-
| Bug found during execution | `/maxsim:debug`
|
|
43
|
-
| Phase complete, needs verification | `/maxsim:verify-phase`
|
|
39
|
+
| Bug found during execution | `/maxsim:debug` |
|
|
40
|
+
| Phase complete, needs verification | `/maxsim:verify-phase` |
|
|
44
41
|
| Quick standalone task | `/maxsim:quick` |
|
|
45
42
|
| User asks for help | `/maxsim:help` |
|
|
46
43
|
|
|
47
|
-
|
|
44
|
+
### Available Skills
|
|
48
45
|
|
|
49
46
|
Skills are behavioral rules that activate automatically based on context:
|
|
50
47
|
|
|
51
48
|
| Skill | Triggers When |
|
|
52
49
|
|-------|---------------|
|
|
53
|
-
| `using-maxsim` | Always (alwaysApply)
|
|
50
|
+
| `using-maxsim` | Always (alwaysApply) -- entry point for all MAXSIM work |
|
|
54
51
|
| `systematic-debugging` | Any bug, test failure, or unexpected behavior encountered |
|
|
55
52
|
| `tdd` | Implementing any feature or bug fix (write test first) |
|
|
56
53
|
| `verification-before-completion` | Before claiming any work is complete or passing |
|
|
@@ -60,7 +57,7 @@ Skills are behavioral rules that activate automatically based on context:
|
|
|
60
57
|
| `simplify` | When reviewing and cleaning up code changes |
|
|
61
58
|
| `code-review` | When reviewing implementation quality |
|
|
62
59
|
|
|
63
|
-
|
|
60
|
+
### Available Agents
|
|
64
61
|
|
|
65
62
|
Agents are specialized subagent prompts spawned by MAXSIM commands:
|
|
66
63
|
|
|
@@ -80,17 +77,7 @@ Agents are specialized subagent prompts spawned by MAXSIM commands:
|
|
|
80
77
|
| `maxsim-codebase-mapper` | Maps codebase structure | `/maxsim:init` |
|
|
81
78
|
| `maxsim-integration-checker` | Checks integration points | `/maxsim:verify-phase` |
|
|
82
79
|
|
|
83
|
-
## Common
|
|
84
|
-
|
|
85
|
-
| Excuse | Why It Violates the Rule |
|
|
86
|
-
|--------|--------------------------|
|
|
87
|
-
| "It's just a small fix" | Small fixes have context and consequences. Use `/maxsim:quick`. |
|
|
88
|
-
| "I know what to do, I don't need a plan" | Plans catch what you miss. The plan is the checkpoint. |
|
|
89
|
-
| "MAXSIM overhead is too much for this" | `/maxsim:quick` exists for lightweight tasks. Use it. |
|
|
90
|
-
| "I'll plan it in my head" | Plans in your head die with context. Write them down. |
|
|
91
|
-
| "The user said 'just do it'" | Route through `/maxsim:quick` — it is fast and still tracked. |
|
|
92
|
-
|
|
93
|
-
## Red Flags — STOP If You Catch Yourself:
|
|
80
|
+
## Common Pitfalls
|
|
94
81
|
|
|
95
82
|
- Writing implementation code without a PLAN.md
|
|
96
83
|
- Skipping `/maxsim:init` because "the project is simple"
|
|
@@ -99,22 +86,22 @@ Agents are specialized subagent prompts spawned by MAXSIM commands:
|
|
|
99
86
|
- Making architectural decisions without documenting them in STATE.md
|
|
100
87
|
- Finishing work without running verification
|
|
101
88
|
|
|
102
|
-
**If any
|
|
89
|
+
**If any of these occur: stop, check the routing table, follow the workflow.**
|
|
103
90
|
|
|
104
|
-
## Verification
|
|
91
|
+
## Verification
|
|
105
92
|
|
|
106
93
|
Before ending any work session:
|
|
107
94
|
|
|
108
95
|
- [ ] All work was routed through MAXSIM commands (not ad-hoc)
|
|
109
96
|
- [ ] STATE.md reflects current progress and decisions
|
|
110
|
-
- [ ] Any bugs encountered were debugged systematically
|
|
97
|
+
- [ ] Any bugs encountered were debugged systematically
|
|
111
98
|
- [ ] Tests were written before implementation (TDD)
|
|
112
99
|
- [ ] Completion claims have verification evidence
|
|
113
100
|
- [ ] Recurring patterns or errors were saved to memory
|
|
114
101
|
|
|
115
|
-
## Integration
|
|
102
|
+
## MAXSIM Integration
|
|
116
103
|
|
|
117
104
|
When a project has a `CLAUDE.md`, both apply:
|
|
118
105
|
- `CLAUDE.md` defines project-specific conventions (language, tools, style)
|
|
119
106
|
- MAXSIM skills define workflow rules (how work is structured and verified)
|
|
120
|
-
- If they conflict, `CLAUDE.md`
|
|
107
|
+
- If they conflict, `CLAUDE.md` takes priority for code style; MAXSIM takes priority for workflow structure
|
|
@@ -1,36 +1,28 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: verification-before-completion
|
|
3
|
-
description:
|
|
3
|
+
description: >-
|
|
4
|
+
Requires running verification commands and reading actual output before making
|
|
5
|
+
any completion claims. Use when claiming work is done, tests pass, builds
|
|
6
|
+
succeed, or bugs are fixed. Prevents false completion claims.
|
|
4
7
|
---
|
|
5
8
|
|
|
6
9
|
# Verification Before Completion
|
|
7
10
|
|
|
8
|
-
|
|
11
|
+
Evidence before claims, always.
|
|
9
12
|
|
|
10
|
-
**
|
|
13
|
+
**HARD GATE -- No completion claims without fresh verification evidence. If you have not run the verification command in this turn, you cannot claim it passes. "Should work" is not evidence. "I'm confident" is not evidence.**
|
|
11
14
|
|
|
12
|
-
##
|
|
15
|
+
## Process
|
|
13
16
|
|
|
14
|
-
|
|
15
|
-
NO COMPLETION CLAIMS WITHOUT FRESH VERIFICATION EVIDENCE.
|
|
16
|
-
If you have not run the verification command in this turn, you CANNOT claim it passes.
|
|
17
|
-
"Should work" is not evidence. "I'm confident" is not evidence.
|
|
18
|
-
Violating this rule is a violation — not a special case.
|
|
19
|
-
</HARD-GATE>
|
|
17
|
+
Before claiming any status or marking a task done:
|
|
20
18
|
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
4. **VERIFY:** Does the output actually confirm the claim?
|
|
29
|
-
- If NO: State the actual status with evidence
|
|
30
|
-
- If YES: State the claim WITH the evidence
|
|
31
|
-
5. **CLAIM:** Only now may you assert completion
|
|
32
|
-
|
|
33
|
-
**Skip any step = lying, not verifying.**
|
|
19
|
+
1. **IDENTIFY** -- What command proves this claim?
|
|
20
|
+
2. **RUN** -- Execute the full command fresh in this turn (not a previous run)
|
|
21
|
+
3. **READ** -- Read the full output, check the exit code, count failures
|
|
22
|
+
4. **VERIFY** -- Does the output actually confirm the claim?
|
|
23
|
+
- If NO: state the actual status with evidence
|
|
24
|
+
- If YES: state the claim with the evidence
|
|
25
|
+
5. **CLAIM** -- Only now may you assert completion
|
|
34
26
|
|
|
35
27
|
### Evidence Block Format
|
|
36
28
|
|
|
@@ -43,35 +35,11 @@ OUTPUT: [relevant excerpt of actual output]
|
|
|
43
35
|
VERDICT: PASS | FAIL
|
|
44
36
|
```
|
|
45
37
|
|
|
46
|
-
This format is required for task completion claims in MAXSIM plan execution. It is
|
|
47
|
-
|
|
48
|
-
## Common Rationalizations — REJECT THESE
|
|
49
|
-
|
|
50
|
-
| Excuse | Why It Violates the Rule |
|
|
51
|
-
|--------|--------------------------|
|
|
52
|
-
| "Should work now" | "Should" is not evidence. RUN the command. |
|
|
53
|
-
| "I'm confident in the logic" | Confidence is not evidence. Run it. |
|
|
54
|
-
| "The linter passed" | Linter passing does not mean tests pass or build succeeds. |
|
|
55
|
-
| "Just this once" | NO EXCEPTIONS. This is the rule, not a guideline. |
|
|
56
|
-
| "I only changed one line" | One line can break everything. Verify. |
|
|
57
|
-
| "The subagent reported success" | Trust test output and VCS diffs, not agent reports. |
|
|
58
|
-
| "Partial check is enough" | Partial proves nothing about the unchecked parts. |
|
|
59
|
-
|
|
60
|
-
## Red Flags — STOP If You Catch Yourself:
|
|
38
|
+
This format is required for task completion claims in MAXSIM plan execution. It is not required for intermediate status updates.
|
|
61
39
|
|
|
62
|
-
|
|
63
|
-
- Expressing satisfaction ("Great!", "Perfect!", "Done!") before running verification
|
|
64
|
-
- About to commit or push without running the test/build command in THIS turn
|
|
65
|
-
- Trusting a subagent's completion report without independent verification
|
|
66
|
-
- Thinking "the last run was clean, I only changed one line"
|
|
67
|
-
- About to mark a MAXSIM task as done without running the `<verify>` block
|
|
68
|
-
- Relying on a previous turn's test output as current evidence
|
|
40
|
+
### What Counts as Verification
|
|
69
41
|
|
|
70
|
-
|
|
71
|
-
|
|
72
|
-
## What Counts as Verification
|
|
73
|
-
|
|
74
|
-
| Claim | Requires | NOT Sufficient |
|
|
42
|
+
| Claim | Requires | Not Sufficient |
|
|
75
43
|
|-------|----------|----------------|
|
|
76
44
|
| "Tests pass" | Test command output showing 0 failures | Previous run, "should pass", partial run |
|
|
77
45
|
| "Build succeeds" | Build command with exit code 0 | Linter passing, "logs look clean" |
|
|
@@ -79,7 +47,19 @@ This format is required for task completion claims in MAXSIM plan execution. It
|
|
|
79
47
|
| "Task is complete" | All done criteria checked with evidence | "I implemented everything in the plan" |
|
|
80
48
|
| "No regressions" | Full test suite passing | "I only changed one file" |
|
|
81
49
|
|
|
82
|
-
##
|
|
50
|
+
## Common Pitfalls
|
|
51
|
+
|
|
52
|
+
| Excuse | Why It Fails |
|
|
53
|
+
|--------|-------------|
|
|
54
|
+
| "Should work now" | "Should" is not evidence. Run the command. |
|
|
55
|
+
| "I'm confident in the logic" | Confidence is not evidence. Run it. |
|
|
56
|
+
| "The linter passed" | Linter passing does not mean tests pass or build succeeds. |
|
|
57
|
+
| "I only changed one line" | One line can break everything. Verify. |
|
|
58
|
+
| "The subagent reported success" | Trust test output and VCS diffs, not agent reports. |
|
|
59
|
+
|
|
60
|
+
Stop if you catch yourself using "should", "probably", or "looks good" about unverified work, or expressing satisfaction before running verification.
|
|
61
|
+
|
|
62
|
+
## Verification
|
|
83
63
|
|
|
84
64
|
Before marking any work as complete:
|
|
85
65
|
|
|
@@ -91,12 +71,13 @@ Before marking any work as complete:
|
|
|
91
71
|
- [ ] No "should", "probably", or "seems to" in your completion statement
|
|
92
72
|
- [ ] Evidence block produced for the task completion claim
|
|
93
73
|
|
|
94
|
-
##
|
|
74
|
+
## MAXSIM Integration
|
|
75
|
+
|
|
76
|
+
The executor's task commit protocol requires verification before committing:
|
|
95
77
|
|
|
96
|
-
|
|
97
|
-
|
|
98
|
-
2. Confirm the `<done>` criteria are met with evidence
|
|
78
|
+
1. Run the task's verify block (automated checks)
|
|
79
|
+
2. Confirm the done criteria are met with evidence
|
|
99
80
|
3. Produce an evidence block for the task completion
|
|
100
81
|
4. Only then: stage files and commit
|
|
101
82
|
|
|
102
|
-
The verifier agent independently re-checks all claims
|
|
83
|
+
The verifier agent independently re-checks all claims -- do not assume the verifier will catch what you missed.
|
package/package.json
CHANGED