@zalom/plastic 1.0.3 → 1.1.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/PLASTIC-reference.md +11 -11
- package/PLASTIC.md +7 -7
- package/README.md +47 -0
- package/agents/plastic-enforcer.md +3 -3
- package/agents/plastic-intent-curator.md +9 -10
- package/agents/plastic-spec-specialist.md +9 -6
- package/bin/plastic.js +2 -2
- package/package.json +1 -1
- package/scripts/dashboard.rb +1 -1
- package/scripts/doctor.rb +52 -4
- package/scripts/end-intent +330 -0
- package/scripts/hook-code-gate +1 -1
- package/scripts/hook-continue +1 -1
- package/scripts/hook-create-gate +2 -2
- package/scripts/hook-session-start +1 -1
- package/scripts/lib/bridge.rb +35 -3
- package/scripts/lib/installer_core.rb +8 -6
- package/scripts/lib/store_provisioning.rb +1 -1
- package/scripts/new-intent +1 -1
- package/scripts/{versions.rb → rollback.rb} +33 -41
- package/scripts/update.rb +1 -1
- package/skills/_active-intent-gate.md +1 -1
- package/skills/_decision-tables.md +50 -0
- package/skills/auto/SKILL.md +47 -64
- package/skills/auto/evals/evals.json +4 -4
- package/skills/auto/references/agent-architecture.md +2 -2
- package/skills/dashboard/SKILL.md +2 -2
- package/skills/doctor/SKILL.md +3 -3
- package/skills/install/SKILL.md +1 -1
- package/skills/{brainstorming → intent-brainstorming}/SKILL.md +43 -45
- package/skills/intent-brainstorming/evals/evals.json +67 -0
- package/skills/{brainstorming → intent-brainstorming}/references/design-principles.md +12 -17
- package/skills/{continuing → intent-continuing}/SKILL.md +3 -3
- package/skills/{continuing → intent-continuing}/evals/evals.json +24 -24
- package/skills/{creating-intent → intent-creating}/SKILL.md +3 -3
- package/skills/{creating-intent → intent-creating}/evals/evals.json +1 -1
- package/skills/{intent-discovery → intent-discovering}/SKILL.md +1 -1
- package/skills/intent-ending/SKILL.md +151 -0
- package/skills/intent-ending/evals/evals.json +74 -0
- package/skills/{executing-plan → intent-executing}/SKILL.md +75 -15
- package/skills/intent-executing/evals/evals.json +66 -0
- package/skills/{brainstorming-grill-me → intent-grilling}/SKILL.md +4 -4
- package/skills/{linking-intents → intent-linking}/SKILL.md +1 -1
- package/skills/{linking-intents → intent-linking}/evals/evals.json +1 -1
- package/skills/{lock → intent-locking}/SKILL.md +1 -1
- package/skills/{writing-plans → intent-planning}/SKILL.md +48 -20
- package/skills/intent-planning/evals/evals.json +66 -0
- package/skills/{writing-plans → intent-planning}/references/plan-format.md +31 -7
- package/skills/{research → intent-researching}/SKILL.md +2 -2
- package/skills/{research → intent-researching}/evals/evals.json +1 -1
- package/skills/{savepoint → intent-savepoint}/SKILL.md +1 -1
- package/skills/intent-speccing/SKILL.md +69 -0
- package/skills/intent-speccing/evals/evals.json +126 -0
- package/skills/intent-speccing/references/per-section-fill-rules.md +92 -0
- package/skills/intent-speccing/references/self-verify-checklist.md +38 -0
- package/skills/intent-starting/SKILL.md +2 -2
- package/skills/intent-starting/evals/evals.json +2 -2
- package/skills/{creating-project → project-creating}/SKILL.md +2 -2
- package/skills/releasing/SKILL.md +18 -14
- package/skills/{versions → rollback}/SKILL.md +30 -20
- package/skills/{creating-skills → skill-creating}/SKILL.md +2 -2
- package/skills/{creating-skills → skill-creating}/evals/evals.json +5 -5
- package/skills/{creating-skills → skill-creating}/references/progressive-disclosure.md +2 -2
- package/skills/{evaluating-skills → skill-evaluating}/SKILL.md +1 -1
- package/skills/{evaluating-skills → skill-evaluating}/evals/evals.json +3 -3
- package/skills/{intent-curator → store-curating}/SKILL.md +6 -7
- package/skills/{intent-curator → store-curating}/evals/evals.json +1 -1
- package/skills/{managing-index → store-indexing}/SKILL.md +6 -7
- package/skills/{managing-index → store-indexing}/evals/evals.json +1 -1
- package/skills/{add-project-store → store-provisioning}/SKILL.md +2 -2
- package/skills/tutorial/SKILL.md +65 -0
- package/skills/tutorial/evals/evals.json +186 -0
- package/skills/tutorial/references/track-1-guided.md +123 -0
- package/skills/tutorial/references/track-2-auto.md +95 -0
- package/skills/tutorial/references/track-3-projects-and-roadmaps.md +110 -0
- package/skills/update/SKILL.md +1 -1
- package/templates/agents.md +7 -7
- package/skills/brainstorming/evals/evals.json +0 -22
- /package/skills/{continuing → intent-continuing}/references/context-management.md +0 -0
- /package/skills/{creating-intent → intent-creating}/references/lifecycle.md +0 -0
- /package/skills/{creating-intent → intent-creating}/references/wikilinks.md +0 -0
- /package/skills/{executing-plan → intent-executing}/code-quality-reviewer-prompt.md +0 -0
- /package/skills/{executing-plan → intent-executing}/implementer-prompt.md +0 -0
- /package/skills/{executing-plan → intent-executing}/spec-reviewer-prompt.md +0 -0
- /package/skills/{linking-intents → intent-linking}/references/zettelkasten.md +0 -0
- /package/skills/{savepoint → intent-savepoint}/references/context-management.md +0 -0
- /package/skills/{creating-project → project-creating}/references/hubs-projects.md +0 -0
- /package/skills/{creating-project → project-creating}/references/project-scaffolding.md +0 -0
- /package/skills/{creating-skills → skill-creating}/references/agents.md +0 -0
- /package/skills/{creating-skills → skill-creating}/references/evals.md +0 -0
- /package/skills/{creating-skills → skill-creating}/references/hooks.md +0 -0
- /package/skills/{creating-skills → skill-creating}/references/scripts.md +0 -0
- /package/skills/{creating-skills → skill-creating}/references/skills.md +0 -0
- /package/skills/{creating-skills → skill-creating}/scripts/scaffold.rb +0 -0
- /package/skills/{evaluating-skills → skill-evaluating}/assets/eval-template.json +0 -0
- /package/skills/{evaluating-skills → skill-evaluating}/references/convention-checks.md +0 -0
- /package/skills/{evaluating-skills → skill-evaluating}/references/eval-methodology.md +0 -0
- /package/skills/{managing-index → store-indexing}/references/zettelkasten-linking.md +0 -0
|
@@ -0,0 +1,69 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: plastic-intent-speccing
|
|
3
|
+
description: >
|
|
4
|
+
Consolidate the enriched Why (`## Context`, `### Decisions`, `## Insights`) into `spec.md`
|
|
5
|
+
through a deterministic, judgment-free numbered sequence. Use when the user types the command
|
|
6
|
+
to turn the brainstorm into a spec, asks to "write the spec", "spec this intent", "consolidate
|
|
7
|
+
into spec.md", or the active intent is at Why with enough Decisions on record to close it out.
|
|
8
|
+
Also fires on an indirect request that never names spec.md, such as "turn what we just
|
|
9
|
+
discussed into the contract the planner builds from." Distinct from `plastic-intent-brainstorming`
|
|
10
|
+
(the exploration that produces the enriched Why, upstream of this skill) and
|
|
11
|
+
`plastic-intent-planning` (turns an existing spec.md into plan.md for How, downstream of this
|
|
12
|
+
skill).
|
|
13
|
+
user-invocable: true
|
|
14
|
+
---
|
|
15
|
+
|
|
16
|
+
# Intent Speccing
|
|
17
|
+
|
|
18
|
+
Consolidate the enriched Why into `spec.md` through eight fixed steps, one action per step. The
|
|
19
|
+
next action is never a judgment call: when a section cannot be filled from what is on record,
|
|
20
|
+
step 5 is to stop and ask, never invent.
|
|
21
|
+
|
|
22
|
+
## Precondition
|
|
23
|
+
|
|
24
|
+
Run only when the active intent is at Why (its `## Intent` and enriched `## Context` plus
|
|
25
|
+
`### Decisions` exist, `spec.md` does not yet exist or is being redone) and the user has asked to
|
|
26
|
+
consolidate that Why into the spec. If no intent is active, or the active intent is not at Why,
|
|
27
|
+
report that and stop; do not guess which intent is meant.
|
|
28
|
+
|
|
29
|
+
## The 8-step sequence
|
|
30
|
+
|
|
31
|
+
| Step | Action |
|
|
32
|
+
|---|---|
|
|
33
|
+
| 1 | Confirm the precondition above. |
|
|
34
|
+
| 2 | Read inputs in fixed order and build a ruling ledger: (a) `## Context` and `### Decisions`; (b) `## Insights` newest-last, so a later ruling supersedes an earlier conflicting one; (c) `resources/discovery--<slug>.md`; (d) any other `resources/*.md`. |
|
|
35
|
+
| 3 | Decide the tier: adopt the brainstorming `Tier:` recommendation from Decisions or Insights; otherwise apply the PLASTIC.md tiers rubric (S = single mechanism or file cluster, M = one subsystem, L = cross-cutting or novel design). |
|
|
36
|
+
| 4 | Fill the template section by section, all 8 sections in template order. Read `references/per-section-fill-rules.md` now, filling the template is the trigger. Write the tier as the literal top line above `# Spec:`. Encode every ruling into its matching section, later supersedes earlier on conflict. A collapsed single-line S/M section is complete and valid; do not pad it. |
|
|
37
|
+
| 5 | Gap rule: if any section cannot be filled from the ledger built in step 2, STOP and ask the user for the missing ruling. Never invent scope to fill a gap. |
|
|
38
|
+
| 6 | State the gate position (below) so the user knows what happens next. |
|
|
39
|
+
| 7 | Self-verify against the checklist. Read `references/self-verify-checklist.md` now, verifying before presenting is the trigger. Fix any failing check, then re-verify from the top. |
|
|
40
|
+
| 8 | Present `spec.md` for the user-review gate, then hand off to `plastic-intent-planning` for How. |
|
|
41
|
+
|
|
42
|
+
## Gate position (step 6)
|
|
43
|
+
|
|
44
|
+
The Why stage's deliverable is `spec.md`; the gate is satisfied the moment a complete, real
|
|
45
|
+
`spec.md` exists (this is the gates-by-name framing: gate-check enforces spec.md before plan.md,
|
|
46
|
+
not the Transition Gates table row). Writing `plan.md` is what opens the code gate for Exec, and
|
|
47
|
+
writing `plan.md` is not this skill's job, that is `plastic-intent-planning`. State this to the user
|
|
48
|
+
at step 6 so the handoff at step 8 is expected, not a surprise.
|
|
49
|
+
|
|
50
|
+
## Tier stamp (step 3, convention only)
|
|
51
|
+
|
|
52
|
+
Write `Tier: S|M|L` as the literal first line of the file, above the `# Spec:` heading. This line
|
|
53
|
+
is convention-only: the orchestrator and the planner read it, no gate and no doctor check
|
|
54
|
+
validates it.
|
|
55
|
+
|
|
56
|
+
## Completion report
|
|
57
|
+
|
|
58
|
+
State, in this order: which file was written (`spec.md`, new or rewritten), the stamped `Tier:`
|
|
59
|
+
value, the count of Acceptance Criteria produced (the surface the planner will cover), which
|
|
60
|
+
`## Insights` rulings superseded an earlier Decision (if any) and where each landed, and the
|
|
61
|
+
handoff target (`plastic-intent-planning`). If step 5 stopped for a missing ruling, report that
|
|
62
|
+
instead: which section, what is missing, and the question put to the user.
|
|
63
|
+
|
|
64
|
+
## References
|
|
65
|
+
|
|
66
|
+
| Trigger | Read |
|
|
67
|
+
|---|---|
|
|
68
|
+
| Filling the template (step 4) | `references/per-section-fill-rules.md` |
|
|
69
|
+
| Self-verifying before presenting (step 7) | `references/self-verify-checklist.md` |
|
|
@@ -0,0 +1,126 @@
|
|
|
1
|
+
{
|
|
2
|
+
"skill_name": "plastic-intent-speccing",
|
|
3
|
+
"notes": "Intent 163. Scopes: description triggering (1-4) and behavior (5-8: all-8-sections output, mandatory superseding-ruling case, STOP-and-ask gap rule, tabular alternatives with no em-dash). No dedicated Ruby test file backs this skill yet (natural-language guided command), so all assertions are result: expect-pass pending a real observed run, per the intent-starting evals.json convention.",
|
|
4
|
+
"evals": [
|
|
5
|
+
{
|
|
6
|
+
"id": 1,
|
|
7
|
+
"scope": "triggering",
|
|
8
|
+
"set": "train",
|
|
9
|
+
"prompt": "The active intent is at Why with Context and Decisions recorded. Consolidate the enriched Why into spec.md.",
|
|
10
|
+
"expected_output": "Activates plastic-intent-speccing (the user-typed Why-to-How consolidation command).",
|
|
11
|
+
"files": [],
|
|
12
|
+
"assertions": [
|
|
13
|
+
{
|
|
14
|
+
"type": "code",
|
|
15
|
+
"check": "router CHOICE == plastic-intent-speccing",
|
|
16
|
+
"result": "expect-pass"
|
|
17
|
+
}
|
|
18
|
+
]
|
|
19
|
+
},
|
|
20
|
+
{
|
|
21
|
+
"id": 2,
|
|
22
|
+
"scope": "triggering",
|
|
23
|
+
"set": "train",
|
|
24
|
+
"prompt": "Turn what we just discussed into the contract the planner builds from.",
|
|
25
|
+
"expected_output": "Activates plastic-intent-speccing; an indirect trigger that names neither the skill nor spec.md.",
|
|
26
|
+
"files": [],
|
|
27
|
+
"assertions": [
|
|
28
|
+
{
|
|
29
|
+
"type": "code",
|
|
30
|
+
"check": "router CHOICE == plastic-intent-speccing",
|
|
31
|
+
"result": "expect-pass"
|
|
32
|
+
}
|
|
33
|
+
]
|
|
34
|
+
},
|
|
35
|
+
{
|
|
36
|
+
"id": 3,
|
|
37
|
+
"scope": "triggering",
|
|
38
|
+
"set": "validation",
|
|
39
|
+
"prompt": "Brainstorm this intent.",
|
|
40
|
+
"expected_output": "Does NOT activate plastic-intent-speccing; activates plastic-intent-brainstorming (exploration that produces the enriched Why, upstream of speccing).",
|
|
41
|
+
"files": [],
|
|
42
|
+
"assertions": [
|
|
43
|
+
{
|
|
44
|
+
"type": "code",
|
|
45
|
+
"check": "router CHOICE != plastic-intent-speccing",
|
|
46
|
+
"result": "expect-pass"
|
|
47
|
+
}
|
|
48
|
+
]
|
|
49
|
+
},
|
|
50
|
+
{
|
|
51
|
+
"id": 4,
|
|
52
|
+
"scope": "triggering",
|
|
53
|
+
"set": "validation",
|
|
54
|
+
"prompt": "Write the plan.",
|
|
55
|
+
"expected_output": "Does NOT activate plastic-intent-speccing; activates plastic-intent-planning (turns an existing spec.md into plan.md for How, downstream of speccing).",
|
|
56
|
+
"files": [],
|
|
57
|
+
"assertions": [
|
|
58
|
+
{
|
|
59
|
+
"type": "code",
|
|
60
|
+
"check": "router CHOICE != plastic-intent-speccing",
|
|
61
|
+
"result": "expect-pass"
|
|
62
|
+
}
|
|
63
|
+
]
|
|
64
|
+
},
|
|
65
|
+
{
|
|
66
|
+
"id": 5,
|
|
67
|
+
"scope": "behavior",
|
|
68
|
+
"set": "train",
|
|
69
|
+
"prompt": "Intent Y is at Why: Context describes the problem and goal, and 5 Decisions are recorded resolving scope, approach, and one rejected alternative. Consolidate into spec.md.",
|
|
70
|
+
"expected_output": "Produces spec.md with Tier: S|M|L as the literal top line above the Spec heading; all 8 template sections present once, in template order (Problem, Goals, Non-Goals, Approach, Alternatives Considered, Decisions, Acceptance Criteria, Open Questions); no template placeholder text remains; every recorded Decision is encoded into its matching section.",
|
|
71
|
+
"files": [],
|
|
72
|
+
"assertions": [
|
|
73
|
+
{
|
|
74
|
+
"type": "human",
|
|
75
|
+
"check": "Tier line is literally the first line of the file; all 8 sections appear once each, in template order; no placeholder text; every Decision traces to a section",
|
|
76
|
+
"result": "expect-pass"
|
|
77
|
+
}
|
|
78
|
+
]
|
|
79
|
+
},
|
|
80
|
+
{
|
|
81
|
+
"id": 6,
|
|
82
|
+
"scope": "behavior",
|
|
83
|
+
"set": "train",
|
|
84
|
+
"prompt": "Intent Z has Decisions recording D4: ship the setting as a CLI flag. A later Insights entry, timestamped after D4, reads: superseding ruling, ship as a config-file setting instead of a CLI flag, per user correction. Consolidate into spec.md.",
|
|
85
|
+
"expected_output": "The produced spec Approach and Decisions sections encode the LATER ruling (config-file setting); the superseded earlier Decision (CLI flag) does not stand as the shipped design in any section. The Decisions section notes that the later Insight supersedes D4.",
|
|
86
|
+
"files": [],
|
|
87
|
+
"assertions": [
|
|
88
|
+
{
|
|
89
|
+
"type": "human",
|
|
90
|
+
"check": "Approach and Decisions state the config-file setting, not the CLI flag; no section still asserts the CLI-flag path as the shipped design",
|
|
91
|
+
"result": "expect-pass"
|
|
92
|
+
}
|
|
93
|
+
]
|
|
94
|
+
},
|
|
95
|
+
{
|
|
96
|
+
"id": 7,
|
|
97
|
+
"scope": "behavior",
|
|
98
|
+
"set": "validation",
|
|
99
|
+
"prompt": "Intent W is at Why. Context says the team wants error handling that is more resilient, but no Decision or Insight states which specific mechanism (retry, circuit breaker, or fallback) was chosen. Consolidate into spec.md.",
|
|
100
|
+
"expected_output": "Stops at step 5 (the gap rule) instead of inventing an Approach; asks the user which error-handling mechanism was decided, naming the missing ruling and the section it blocks.",
|
|
101
|
+
"files": [],
|
|
102
|
+
"assertions": [
|
|
103
|
+
{
|
|
104
|
+
"type": "human",
|
|
105
|
+
"check": "no invented Approach or Decisions content fills the gap; the agent asks for the missing ruling instead of guessing a default mechanism",
|
|
106
|
+
"result": "expect-pass"
|
|
107
|
+
}
|
|
108
|
+
]
|
|
109
|
+
},
|
|
110
|
+
{
|
|
111
|
+
"id": 8,
|
|
112
|
+
"scope": "behavior",
|
|
113
|
+
"set": "validation",
|
|
114
|
+
"prompt": "Intent V has Decisions recording 3 rejected alternatives, each with a one-line reason it lost. Consolidate into spec.md.",
|
|
115
|
+
"expected_output": "Alternatives Considered renders as a table (Alternative, Not chosen because), not the bullet-dash form shown in the template; the produced spec file contains no em-dashes or en-dashes anywhere in the file.",
|
|
116
|
+
"files": [],
|
|
117
|
+
"assertions": [
|
|
118
|
+
{
|
|
119
|
+
"type": "human",
|
|
120
|
+
"check": "Alternatives Considered is a two-column table with one row per rejected alternative; a full-file dash-glyph scan of the produced spec file finds none",
|
|
121
|
+
"result": "expect-pass"
|
|
122
|
+
}
|
|
123
|
+
]
|
|
124
|
+
}
|
|
125
|
+
]
|
|
126
|
+
}
|
|
@@ -0,0 +1,92 @@
|
|
|
1
|
+
# Per-Section Fill Rules
|
|
2
|
+
|
|
3
|
+
The single shared method for turning the enriched Why into `spec.md`. Both
|
|
4
|
+
`plastic-intent-speccing` (guided) and the `plastic-spec-specialist` agent (auto) point here;
|
|
5
|
+
neither restates these rules. One fill rule per template section, in template order.
|
|
6
|
+
|
|
7
|
+
Read the ruling ledger first (built in step 2 of the SKILL.md sequence): `## Context` plus
|
|
8
|
+
`### Decisions`, then `## Insights` newest-last (a later ruling supersedes an earlier
|
|
9
|
+
conflicting one), then `resources/discovery--<slug>.md`, then any other `resources/*.md`.
|
|
10
|
+
Every rule below names which part of that ledger feeds the section.
|
|
11
|
+
|
|
12
|
+
## 1. Problem
|
|
13
|
+
|
|
14
|
+
Source: the intent's `## Intent` line plus the problem narrative in `## Context`.
|
|
15
|
+
|
|
16
|
+
State the problem as a problem, not a solution: what is broken, missing, or costly today, for
|
|
17
|
+
whom. Do not describe the fix here, that belongs in Approach. If `## Insights` records a later
|
|
18
|
+
reframe of the problem (a scope correction, a retargeted root cause), the later framing wins.
|
|
19
|
+
|
|
20
|
+
## 2. Goals
|
|
21
|
+
|
|
22
|
+
Source: explicit goal statements in `## Context`, plus any `### Decisions` entry that commits to
|
|
23
|
+
delivering a specific outcome.
|
|
24
|
+
|
|
25
|
+
One bullet per goal, each a concrete, observable outcome the delivery must reach. Encode every
|
|
26
|
+
Decision that commits to an outcome as its own bullet, do not compress two Decisions into one
|
|
27
|
+
vague goal. A single-line collapsed Goals section (S/M tier) is complete if it names every
|
|
28
|
+
outcome; do not pad it with restated Problem text.
|
|
29
|
+
|
|
30
|
+
## 3. Non-Goals
|
|
31
|
+
|
|
32
|
+
Source: `### Decisions` entries that name something explicitly out of scope, and `## Insights`
|
|
33
|
+
rulings that narrowed scope after the initial brainstorm.
|
|
34
|
+
|
|
35
|
+
One bullet per excluded item, each naming what is out and, briefly, why (a sibling intent owns
|
|
36
|
+
it, a scope-reframe ruling cut it, it is a future intent). If a later Insight narrows scope
|
|
37
|
+
further than an earlier Decision, the Insight's narrower boundary is the one recorded.
|
|
38
|
+
|
|
39
|
+
## 4. Approach
|
|
40
|
+
|
|
41
|
+
Source: every `### Decisions` entry that describes HOW, resolved into one coherent narrative.
|
|
42
|
+
|
|
43
|
+
Write prose, not a decision list. Synthesize the chosen path so it reads as one design: what
|
|
44
|
+
gets built, in what shape, and how the pieces fit. Every Decision that shapes the approach must
|
|
45
|
+
be traceable to a sentence here, but do not restate each Decision by its Dn label, that
|
|
46
|
+
enumeration belongs in the Decisions section. Use a table only where the Approach itself compares
|
|
47
|
+
options inline (rare; usually that comparison belongs in Alternatives Considered instead).
|
|
48
|
+
|
|
49
|
+
## 5. Alternatives Considered
|
|
50
|
+
|
|
51
|
+
Source: `### Decisions` entries and `## Insights` rulings that name a rejected path and the
|
|
52
|
+
reason it lost.
|
|
53
|
+
|
|
54
|
+
Render as a table, not the template's bullet-dash form, so the reason column stays uniform and
|
|
55
|
+
free of dash-glyph punctuation:
|
|
56
|
+
|
|
57
|
+
| Alternative | Not chosen because |
|
|
58
|
+
|---|---|
|
|
59
|
+
| <rejected path> | <the ruling's stated reason, paraphrased> |
|
|
60
|
+
|
|
61
|
+
One row per rejected alternative. If two Decisions reject variants of the same alternative, merge
|
|
62
|
+
them into one row rather than duplicating it. Never use an em-dash or en-dash in the reason
|
|
63
|
+
column; write "because" or a colon instead.
|
|
64
|
+
|
|
65
|
+
## 6. Decisions
|
|
66
|
+
|
|
67
|
+
Source: `### Decisions` verbatim, in the order recorded, one entry per Decision.
|
|
68
|
+
|
|
69
|
+
One bullet per decision, each keeping its `Dn` label and its ruling in paraphrase or verbatim.
|
|
70
|
+
Do not drop a decision because it seems minor: every Decision must appear here even if its effect
|
|
71
|
+
on the shipped Approach or Goals is small. When `## Insights` records a later ruling that
|
|
72
|
+
supersedes an earlier Decision, add or amend the bullet to state the later ruling and note which
|
|
73
|
+
earlier Decision it supersedes.
|
|
74
|
+
|
|
75
|
+
## 7. Acceptance Criteria
|
|
76
|
+
|
|
77
|
+
Source: any criteria named directly in `### Decisions` or `## Insights`, plus one inferred
|
|
78
|
+
criterion per Goal that has no explicit criterion on record.
|
|
79
|
+
|
|
80
|
+
One checkbox bullet per criterion, each concretely checkable: a fact a reviewer can confirm true
|
|
81
|
+
or false by inspection, a command, or a file, never a vague quality judgment. Every Goal must map
|
|
82
|
+
to at least one criterion. A criterion that cannot be checked without more interpretation is not
|
|
83
|
+
done, rewrite it or ask (the gap rule, SKILL.md step 5).
|
|
84
|
+
|
|
85
|
+
## 8. Open Questions
|
|
86
|
+
|
|
87
|
+
Source: any question raised during Why that has no matching Decision or Insight resolving it.
|
|
88
|
+
|
|
89
|
+
List each unresolved question as its own bullet. If every question raised during Why has a
|
|
90
|
+
resolving Decision or Insight, write `None` and, directly under it, one line per resolved
|
|
91
|
+
question naming which Decision or Insight resolved it (so a reader can audit the resolution
|
|
92
|
+
instead of taking "None" on faith).
|
|
@@ -0,0 +1,38 @@
|
|
|
1
|
+
# Self-Verify Checklist
|
|
2
|
+
|
|
3
|
+
Run this before presenting spec.md (SKILL.md step 7). Ten checks: the four points of
|
|
4
|
+
brainstorming's proven Spec Self-Review, merged with the six binding output checks this skill
|
|
5
|
+
adds. Fix any failing check, then re-verify from the top; do not present a spec that fails one of
|
|
6
|
+
these.
|
|
7
|
+
|
|
8
|
+
## Brainstorming's four Spec Self-Review points
|
|
9
|
+
|
|
10
|
+
1. **Placeholder scan.** Any "TBD", "TODO", incomplete section, or vague requirement left in the
|
|
11
|
+
draft? Fix it before moving on.
|
|
12
|
+
2. **Internal consistency.** Do any two sections contradict each other? Does the Approach
|
|
13
|
+
actually match what Goals and Acceptance Criteria describe?
|
|
14
|
+
3. **Scope check.** Is this spec focused enough for a single implementation plan, or does the
|
|
15
|
+
Problem actually describe more than one independent piece of work that needs decomposing
|
|
16
|
+
first?
|
|
17
|
+
4. **Ambiguity check.** Could any requirement be read two different ways? If so, pick one
|
|
18
|
+
reading and rewrite the line so only that reading survives.
|
|
19
|
+
|
|
20
|
+
## The six binding output checks
|
|
21
|
+
|
|
22
|
+
5. **No template placeholder text remains.** No literal `<intent name>`, `<alternative>`, `...`,
|
|
23
|
+
sample bracket text, or other template filler from `templates/spec.md` survives anywhere in
|
|
24
|
+
the artifact.
|
|
25
|
+
6. **Tier stamp is the top line.** `Tier: S|M|L` is the literal first line of the file, above the
|
|
26
|
+
`# Spec:` heading, not folded into a section body.
|
|
27
|
+
7. **All 8 sections present, in template order.** Problem, Goals, Non-Goals, Approach,
|
|
28
|
+
Alternatives Considered, Decisions, Acceptance Criteria, Open Questions, each present once, in
|
|
29
|
+
that order, none merged into another.
|
|
30
|
+
8. **Every `## Insights` ruling is traceable to a spec line.** Walk the Insights log entry by
|
|
31
|
+
entry: each one lands in some section of the spec. Where two rulings conflict, the later one
|
|
32
|
+
(further down the log) is the one that landed, and the earlier one does not silently persist
|
|
33
|
+
in a different section.
|
|
34
|
+
9. **Acceptance criteria are concretely checkable.** Each Acceptance Criteria bullet reads as a
|
|
35
|
+
fact a reviewer can confirm true or false by inspection, a command, or a named file, not a
|
|
36
|
+
quality judgment a reviewer would have to interpret.
|
|
37
|
+
10. **No em-dashes or en-dashes anywhere in the artifact.** Scan the full file; replace any dash
|
|
38
|
+
glyph with a comma, period, parenthesis, or colon.
|
|
@@ -57,7 +57,7 @@ enforces it: without a held lock, mutating writes to this active intent's dir ar
|
|
|
57
57
|
4. **Dispatch What-stage discovery (under the lock).** Right after arming, when the intent
|
|
58
58
|
was just activated in step 1 (on a resume that already has
|
|
59
59
|
`resources/discovery--<slug>.md`, skip: discovery runs once per intent, at activation
|
|
60
|
-
only), dispatch the `plastic-intent-discovery` agent (see the `plastic-intent-
|
|
60
|
+
only), dispatch the `plastic-intent-discovery` agent (see the `plastic-intent-discovering`
|
|
61
61
|
skill), now that this session owns the lock, deposit authorized as the owner session. Resolve its
|
|
62
62
|
model explicitly and pass it at dispatch time (belt-and-braces): `read-config
|
|
63
63
|
agents.models.plastic-intent-discovery --project <repo>`. The agent runs QMD discovery
|
|
@@ -88,7 +88,7 @@ that message, never delete a lock file by hand.
|
|
|
88
88
|
Read `savepoint.md` and classify from the **last line** alone, then verify ONLY that line's
|
|
89
89
|
artifact is real (sentinel-aware via `Bridge.stage_file_present?`). On drift (the last line
|
|
90
90
|
disagrees with files on disk), rebuild the ledger from disk and note the correction. Do not
|
|
91
|
-
inline the rebuild; the `plastic-savepoint` skill owns it:
|
|
91
|
+
inline the rebuild; the `plastic-intent-savepoint` skill owns it:
|
|
92
92
|
```bash
|
|
93
93
|
ruby -r ~/.plastic/scripts/lib/bridge -e 'Bridge.rebuild_savepoint("<intent_dir>")'
|
|
94
94
|
```
|
|
@@ -40,7 +40,7 @@
|
|
|
40
40
|
"scope": "triggering",
|
|
41
41
|
"set": "validation",
|
|
42
42
|
"prompt": "continue",
|
|
43
|
-
"expected_output": "Does NOT activate plastic-intent-starting; bare 'continue' is plastic-continuing (no specific intent named).",
|
|
43
|
+
"expected_output": "Does NOT activate plastic-intent-starting; bare 'continue' is plastic-intent-continuing (no specific intent named).",
|
|
44
44
|
"files": [],
|
|
45
45
|
"assertions": [
|
|
46
46
|
{ "type": "code", "check": "router CHOICE != plastic-intent-starting", "result": "expect-pass" }
|
|
@@ -51,7 +51,7 @@
|
|
|
51
51
|
"scope": "triggering",
|
|
52
52
|
"set": "validation",
|
|
53
53
|
"prompt": "create a new intent for the uploader retry policy",
|
|
54
|
-
"expected_output": "Does NOT activate plastic-intent-starting; activates plastic-creating
|
|
54
|
+
"expected_output": "Does NOT activate plastic-intent-starting; activates plastic-intent-creating.",
|
|
55
55
|
"files": [],
|
|
56
56
|
"assertions": [
|
|
57
57
|
{ "type": "code", "check": "router CHOICE != plastic-intent-starting", "result": "expect-pass" }
|
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
---
|
|
2
|
-
name: plastic-creating
|
|
2
|
+
name: plastic-project-creating
|
|
3
3
|
description: >-
|
|
4
4
|
Create a new project from an implementation intent. Sets up project directory,
|
|
5
5
|
git init, AGENTS.md with founding intent decisions, plastic-install --local,
|
|
@@ -79,7 +79,7 @@ writes `INDEX.md` and `project.yml` only if missing. It is idempotent, so it is
|
|
|
79
79
|
safe even when the tactical mirror in step 5 already created the store directory.
|
|
80
80
|
Do not create the store with an inline `mkdir`; the provisioner is the only place
|
|
81
81
|
a store is made. For a project that is already registered but store-less, use the
|
|
82
|
-
`plastic-
|
|
82
|
+
`plastic-store-provisioning` skill instead.
|
|
83
83
|
|
|
84
84
|
### 8. Mark Global Intent(s) Completed
|
|
85
85
|
|
|
@@ -214,26 +214,30 @@ gh release edit <tag-name> --latest
|
|
|
214
214
|
|
|
215
215
|
### 8. Complete Active Intent
|
|
216
216
|
|
|
217
|
-
A release IS a delivery. The active intent that drove this work must be completed as part of the release process. This is NOT optional.
|
|
217
|
+
A release IS a delivery. The active intent that drove this work must be completed as part of the release process. This is NOT optional. The mechanical close (outcome/INDEX/savepoint/commit) is `plastic-intent-ending`'s job, not this skill's: run its backing script rather than restating that prose here.
|
|
218
218
|
|
|
219
|
-
1. Read `~/.plastic/INDEX.md`
|
|
219
|
+
1. Read `~/.plastic/INDEX.md` (or the project's INDEX.md) - find active intent(s) related to this release.
|
|
220
220
|
2. For each active intent being delivered:
|
|
221
|
-
a. Write `outcome.md`
|
|
222
|
-
b.
|
|
223
|
-
c.
|
|
224
|
-
|
|
225
|
-
|
|
226
|
-
|
|
221
|
+
a. Write a real `outcome.md` (never leave the scaffold placeholder), `disposition: delivered`, referencing the release tag.
|
|
222
|
+
b. Update `## Insights` with final observations.
|
|
223
|
+
c. Run the mechanical close (steps 1-4 of `plastic-intent-ending`): this stamps the intent file's `## Outcome` summary, moves the INDEX.md line to `## Completed` (dated today, with a rich entry description via `--index-note`), appends the savepoint `Done` bookend, and commits the store, all in one call:
|
|
224
|
+
```bash
|
|
225
|
+
ruby ~/.plastic/scripts/end-intent --store <store_path> --id <ID> --disposition delivered \
|
|
226
|
+
--outcome-summary "delivered in <tag-name>: <one-line summary>" \
|
|
227
|
+
--index-note "<tag-name>, <mode/tier>; <what shipped>; <suite result>"
|
|
228
|
+
```
|
|
229
|
+
d. Update clusters to show `_(completed)_` (the store-curating skill's job on its next pass).
|
|
227
230
|
|
|
228
231
|
**If no active intent exists for this release**, that itself is a problem - work happened outside the intent system. Log it and move on, but flag it.
|
|
229
232
|
|
|
230
233
|
### 9. Clean Up the Intent's Worktrees (merge-then-remove)
|
|
231
234
|
|
|
232
|
-
|
|
233
|
-
|
|
234
|
-
|
|
235
|
-
|
|
236
|
-
|
|
235
|
+
This is the release branch of `plastic-intent-ending`'s Step 5 disarm (`merge: true`), not a
|
|
236
|
+
separate concern: a release is the merge-then-remove path for the intent's worktrees (intent
|
|
237
|
+
73c3), so the intent's code branch is merged back into the default branch BEFORE the worktree
|
|
238
|
+
is removed. Drive it through `Worktree.finish` with `merge: true`, which merges the code
|
|
239
|
+
branch, then removes both worktrees (code + paired store), prunes both repos, and clears the
|
|
240
|
+
worktree block from the bridge:
|
|
237
241
|
|
|
238
242
|
```bash
|
|
239
243
|
ruby -r ~/.plastic/scripts/lib/worktree -r ~/.plastic/scripts/lib/bridge -e \
|
|
@@ -241,7 +245,7 @@ ruby -r ~/.plastic/scripts/lib/worktree -r ~/.plastic/scripts/lib/bridge -e \
|
|
|
241
245
|
```
|
|
242
246
|
|
|
243
247
|
(Uses `discover_bridge`, not a bare session-keyed `Bridge.read`, because a session can own more
|
|
244
|
-
than one live bridge now
|
|
248
|
+
than one live bridge now (intent 131) and `discover_bridge` resolves the right one for this cwd.)
|
|
245
249
|
|
|
246
250
|
Honor the worktree-cleanup rule: never leave an orphaned worktree, and run `git worktree
|
|
247
251
|
prune` in the affected repo if you hit a stale reference. For why this is the one place the
|
|
@@ -1,10 +1,10 @@
|
|
|
1
1
|
---
|
|
2
|
-
name: plastic-
|
|
2
|
+
name: plastic-rollback
|
|
3
3
|
description: Use when the user wants to see their Plastic version history or roll back to a previously-installed version after a bad release. Manages the local, append-only versions.json ledger and steps between versions the user has actually run. For moving to a brand-new release, use plastic-update instead.
|
|
4
4
|
user-invocable: true
|
|
5
5
|
---
|
|
6
6
|
|
|
7
|
-
# Plastic
|
|
7
|
+
# Plastic Rollback: local version time-machine
|
|
8
8
|
|
|
9
9
|
## When to Use
|
|
10
10
|
- "show plastic versions", "version history", "what versions have I run"
|
|
@@ -14,12 +14,18 @@ user-invocable: true
|
|
|
14
14
|
For upgrading to a **new** release, use `plastic-update`. This skill only navigates
|
|
15
15
|
versions you have **already installed**, the ones recorded in the ledger.
|
|
16
16
|
|
|
17
|
+
## Read-only by default
|
|
18
|
+
|
|
19
|
+
A flagless run only prints the version history table. It never switches, never prompts,
|
|
20
|
+
and never offers to keep going further back. Switching a version always needs an
|
|
21
|
+
explicit target, named with `--version`.
|
|
22
|
+
|
|
17
23
|
## Channel rule
|
|
18
24
|
|
|
19
|
-
`
|
|
25
|
+
`rollback` restores whichever build you name from the ledger, it does not take a channel
|
|
20
26
|
flag for the target. The pinned `<channel>` below is only the npx invocation itself:
|
|
21
27
|
derive it from `~/.plastic/VERSION` the same way as the other lifecycle skills,
|
|
22
|
-
`-alpha`
|
|
28
|
+
`-alpha` to `@alpha`, `-beta` to `@beta`, otherwise `@latest`.
|
|
23
29
|
|
|
24
30
|
## The ledger
|
|
25
31
|
|
|
@@ -37,45 +43,49 @@ troubleshooting record.
|
|
|
37
43
|
|
|
38
44
|
## Procedure
|
|
39
45
|
|
|
40
|
-
### Show history
|
|
46
|
+
### Show history (read-only, no switch)
|
|
41
47
|
|
|
42
48
|
```bash
|
|
43
|
-
npx -y @zalom/plastic@<channel>
|
|
49
|
+
npx -y @zalom/plastic@<channel> rollback
|
|
44
50
|
```
|
|
45
51
|
|
|
46
|
-
Prints the table with the currently-installed version marked.
|
|
47
|
-
|
|
52
|
+
Prints the table with the currently-installed version marked. Never switches, never
|
|
53
|
+
prompts, no matter what the last recorded action was.
|
|
48
54
|
|
|
49
|
-
###
|
|
55
|
+
### Switch to a specific version
|
|
50
56
|
|
|
51
57
|
```bash
|
|
52
|
-
npx -y @zalom/plastic@<channel>
|
|
53
|
-
npx -y @zalom/plastic@<channel> versions --downgrade --version 1.0.0-alpha.15 # to a specific run
|
|
58
|
+
npx -y @zalom/plastic@<channel> rollback --version 1.0.0-alpha.15
|
|
54
59
|
```
|
|
55
60
|
|
|
56
|
-
|
|
57
|
-
|
|
58
|
-
|
|
61
|
+
The only way to actually switch. The target must be a version you have actually run (the
|
|
62
|
+
ledger); the direction (upgrade or downgrade) is derived automatically by comparing the
|
|
63
|
+
target to the installed version.
|
|
59
64
|
|
|
60
|
-
|
|
65
|
+
`--downgrade --version V` and `--upgrade --version V` are accepted as explicit-target
|
|
66
|
+
synonyms of `--version V`, the direction flag is descriptive only:
|
|
61
67
|
|
|
62
68
|
```bash
|
|
63
|
-
npx -y @zalom/plastic@<channel>
|
|
69
|
+
npx -y @zalom/plastic@<channel> rollback --downgrade --version 1.0.0-alpha.15
|
|
70
|
+
npx -y @zalom/plastic@<channel> rollback --upgrade --version 1.0.0-alpha.18
|
|
64
71
|
```
|
|
65
72
|
|
|
73
|
+
A bare `--downgrade` or `--upgrade` with no `--version` is an error: it prints a message
|
|
74
|
+
asking for an explicit target and performs no switch.
|
|
75
|
+
|
|
66
76
|
### After any change
|
|
67
77
|
|
|
68
78
|
Run `plastic-doctor` to confirm health, then emit the reporting block and suggest
|
|
69
79
|
`/clear` so the session picks up the swapped conventions:
|
|
70
80
|
|
|
71
81
|
```
|
|
72
|
-
Plastic
|
|
73
|
-
Command: npx -y @zalom/plastic@<channel>
|
|
82
|
+
Plastic rollback (<channel>)
|
|
83
|
+
Command: npx -y @zalom/plastic@<channel> rollback <flags>
|
|
74
84
|
Version: <before> -> <after>
|
|
75
85
|
Doctor: <summary or "all clear">
|
|
76
86
|
```
|
|
77
87
|
|
|
78
88
|
## Notes
|
|
79
89
|
- The ledger is **never** edited or pruned, it is the audit trail.
|
|
80
|
-
-
|
|
81
|
-
the user rather than forcing the
|
|
90
|
+
- Switching cannot un-migrate a store-format change; if a warning appears, surface it to
|
|
91
|
+
the user rather than forcing the switch.
|
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
---
|
|
2
|
-
name: plastic-creating
|
|
2
|
+
name: plastic-skill-creating
|
|
3
3
|
description: >
|
|
4
4
|
Author or revise a Plastic skill, a subagent or Agent role file, or a
|
|
5
5
|
lifecycle hook with progressive disclosure. Use when creating or editing a
|
|
@@ -63,4 +63,4 @@ routes each authoring task to the reference that holds the depth.
|
|
|
63
63
|
- To start a new skill, agent, or hook from a born-slim file, run
|
|
64
64
|
`scripts/scaffold.rb`.
|
|
65
65
|
- To design, run, and grade evals in depth (paired runs, assertions after
|
|
66
|
-
observing, pass rates), use the `plastic-evaluating
|
|
66
|
+
observing, pass rates), use the `plastic-skill-evaluating` skill.
|
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
{
|
|
2
|
-
"skill_name": "plastic-creating
|
|
2
|
+
"skill_name": "plastic-skill-creating",
|
|
3
3
|
"evals": [
|
|
4
4
|
{
|
|
5
5
|
"id": 1,
|
|
@@ -45,19 +45,19 @@
|
|
|
45
45
|
{
|
|
46
46
|
"id": 5,
|
|
47
47
|
"prompt": "Run the evals on my skill and check the pass rate",
|
|
48
|
-
"expected_output": "The skill should NOT trigger. Running evals and reading pass rates is the job of plastic-evaluating
|
|
48
|
+
"expected_output": "The skill should NOT trigger. Running evals and reading pass rates is the job of plastic-skill-evaluating, not authoring. Near-miss negative: shares the words 'skill' and 'evals' but the operation is grading an existing skill, not creating or revising one.",
|
|
49
49
|
"files": [],
|
|
50
50
|
"assertions": [
|
|
51
|
-
"The skill does not activate (running and grading evals is plastic-evaluating
|
|
51
|
+
"The skill does not activate (running and grading evals is plastic-skill-evaluating, not authoring)."
|
|
52
52
|
]
|
|
53
53
|
},
|
|
54
54
|
{
|
|
55
55
|
"id": 6,
|
|
56
56
|
"prompt": "Create a new intent for the dashboard work",
|
|
57
|
-
"expected_output": "The skill should NOT trigger. Creating an intent is plastic-creating
|
|
57
|
+
"expected_output": "The skill should NOT trigger. Creating an intent is plastic-intent-creating. Near-miss negative: shares the verb 'create' but the object is an intent, not a skill, agent, or hook.",
|
|
58
58
|
"files": [],
|
|
59
59
|
"assertions": [
|
|
60
|
-
"The skill does not activate (creating an intent is plastic-creating
|
|
60
|
+
"The skill does not activate (creating an intent is plastic-intent-creating)."
|
|
61
61
|
]
|
|
62
62
|
},
|
|
63
63
|
{
|
|
@@ -125,8 +125,8 @@ agent reads as a dependency it loads on its own terms. [C8]
|
|
|
125
125
|
| Form | Effect |
|
|
126
126
|
|------|--------|
|
|
127
127
|
| `REQUIRED BACKGROUND: superpowers:test-driven-development` | Good. Names the dependency; the agent loads it when relevant. |
|
|
128
|
-
| `For eval depth, use plastic-evaluating
|
|
129
|
-
| `@skills/evaluating
|
|
128
|
+
| `For eval depth, use plastic-skill-evaluating.` | Good. Names the skill, leaves loading to the agent. |
|
|
129
|
+
| `@skills/skill-evaluating/SKILL.md` | Bad. `@`-path syntax force-loads the file immediately, defeating disclosure. |
|
|
130
130
|
|
|
131
131
|
The `@`-path form pulls the target into context the moment the line is read, so it spends
|
|
132
132
|
the budget the level split was built to protect. Name the skill and let the trigger
|
|
@@ -1,16 +1,16 @@
|
|
|
1
1
|
{
|
|
2
|
-
"skill_name": "evaluating
|
|
2
|
+
"skill_name": "skill-evaluating",
|
|
3
3
|
"evals": [
|
|
4
4
|
{
|
|
5
5
|
"id": 1,
|
|
6
|
-
"prompt": "I want to evaluate whether my creating
|
|
6
|
+
"prompt": "I want to evaluate whether my intent-creating skill follows Plastic conventions",
|
|
7
7
|
"expected_output": "The skill should activate and guide the user through convention compliance evaluation: choose eval scope, design test cases using convention-checks reference, run paired evals, write assertions after observing.",
|
|
8
8
|
"files": [],
|
|
9
9
|
"assertions": []
|
|
10
10
|
},
|
|
11
11
|
{
|
|
12
12
|
"id": 2,
|
|
13
|
-
"prompt": "Run evals on the brainstorming skill to see if the description triggers correctly",
|
|
13
|
+
"prompt": "Run evals on the intent-brainstorming skill to see if the description triggers correctly",
|
|
14
14
|
"expected_output": "The skill should activate and guide through description triggering evaluation: design ~20 queries with should-trigger and near-miss negatives, 60/40 train/validation split, compute trigger rates.",
|
|
15
15
|
"files": [],
|
|
16
16
|
"assertions": []
|