@cloverleaf/reference-impl 0.5.0 → 0.5.1

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "cloverleaf",
   "description": "Cloverleaf reference implementation — Claude Code skills for task scaffolding and the Delivery pipeline (implementer, documenter, reviewer, UI reviewer with multi-viewport visual diff, QA, merge).",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "author": {
     "name": "Renato D'Arrigo",
     "email": "renato.darrigo@gmail.com"

package/VERSION

@@ -1 +1 @@
-0.4.1
+0.5.1

package/dist/plan.mjs

@@ -11,6 +11,7 @@ export function loadPlan(repoRoot, id) {
 }
 export function savePlan(repoRoot, plan) {
     validateOrThrow('https://cloverleaf.example/schemas/plan.schema.json', plan);
+    mkdirSync(plansDir(repoRoot), { recursive: true });
     const path = join(plansDir(repoRoot), `${plan.id}.json`);
     writeFileSync(path, JSON.stringify(plan, null, 2) + '\n');
 }

package/dist/rfc.mjs

@@ -1,4 +1,4 @@
-import { readFileSync, writeFileSync, existsSync } from 'node:fs';
+import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'node:fs';
 import { join } from 'node:path';
 import { rfcsDir } from './paths.mjs';
 import { validateOrThrow } from './validate.mjs';
@@ -11,6 +11,7 @@ export function loadRfc(repoRoot, id) {
 }
 export function saveRfc(repoRoot, rfc) {
     validateOrThrow('https://cloverleaf.example/schemas/rfc.schema.json', rfc);
+    mkdirSync(rfcsDir(repoRoot), { recursive: true });
     const path = join(rfcsDir(repoRoot), `${rfc.id}.json`);
     writeFileSync(path, JSON.stringify(rfc, null, 2) + '\n');
 }

package/dist/spike.mjs

@@ -1,4 +1,4 @@
-import { readFileSync, writeFileSync, existsSync } from 'node:fs';
+import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'node:fs';
 import { join } from 'node:path';
 import { spikesDir } from './paths.mjs';
 import { validateOrThrow } from './validate.mjs';
@@ -11,6 +11,7 @@ export function loadSpike(repoRoot, id) {
 }
 export function saveSpike(repoRoot, spike) {
     validateOrThrow('https://cloverleaf.example/schemas/spike.schema.json', spike);
+    mkdirSync(spikesDir(repoRoot), { recursive: true });
     const path = join(spikesDir(repoRoot), `${spike.id}.json`);
     writeFileSync(path, JSON.stringify(spike, null, 2) + '\n');
 }

package/dist/task.mjs

@@ -1,4 +1,4 @@
-import { readFileSync, writeFileSync, existsSync } from 'node:fs';
+import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'node:fs';
 import { join } from 'node:path';
 import { tasksDir, projectsDir } from './paths.mjs';
 import { validateOrThrow } from './validate.mjs';
@@ -11,6 +11,7 @@ export function loadTask(repoRoot, taskId) {
 }
 export function saveTask(repoRoot, task) {
     validateOrThrow('https://cloverleaf.example/schemas/task.schema.json', task);
+    mkdirSync(tasksDir(repoRoot), { recursive: true });
     const path = join(tasksDir(repoRoot), `${task.id}.json`);
     writeFileSync(path, JSON.stringify(task, null, 2) + '\n');
 }

package/lib/plan.ts

@@ -35,6 +35,7 @@ export function loadPlan(repoRoot: string, id: string): PlanDoc {
 
 export function savePlan(repoRoot: string, plan: PlanDoc): void {
   validateOrThrow('https://cloverleaf.example/schemas/plan.schema.json', plan);
+  mkdirSync(plansDir(repoRoot), { recursive: true });
   const path = join(plansDir(repoRoot), `${plan.id}.json`);
   writeFileSync(path, JSON.stringify(plan, null, 2) + '\n');
 }

package/lib/rfc.ts

@@ -1,4 +1,4 @@
-import { readFileSync, writeFileSync, existsSync } from 'node:fs';
+import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'node:fs';
 import { join } from 'node:path';
 import { rfcsDir } from './paths.js';
 import { validateOrThrow } from './validate.js';
@@ -27,6 +27,7 @@ export function loadRfc(repoRoot: string, id: string): RfcDoc {
 
 export function saveRfc(repoRoot: string, rfc: RfcDoc): void {
   validateOrThrow('https://cloverleaf.example/schemas/rfc.schema.json', rfc);
+  mkdirSync(rfcsDir(repoRoot), { recursive: true });
   const path = join(rfcsDir(repoRoot), `${rfc.id}.json`);
   writeFileSync(path, JSON.stringify(rfc, null, 2) + '\n');
 }

package/lib/spike.ts

@@ -1,4 +1,4 @@
-import { readFileSync, writeFileSync, existsSync } from 'node:fs';
+import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'node:fs';
 import { join } from 'node:path';
 import { spikesDir } from './paths.js';
 import { validateOrThrow } from './validate.js';
@@ -27,6 +27,7 @@ export function loadSpike(repoRoot: string, id: string): SpikeDoc {
 
 export function saveSpike(repoRoot: string, spike: SpikeDoc): void {
   validateOrThrow('https://cloverleaf.example/schemas/spike.schema.json', spike);
+  mkdirSync(spikesDir(repoRoot), { recursive: true });
   const path = join(spikesDir(repoRoot), `${spike.id}.json`);
   writeFileSync(path, JSON.stringify(spike, null, 2) + '\n');
 }

package/lib/task.ts

@@ -1,4 +1,4 @@
-import { readFileSync, writeFileSync, existsSync } from 'node:fs';
+import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'node:fs';
 import { join } from 'node:path';
 import { tasksDir, projectsDir } from './paths.js';
 import type { Task as SMTask } from '@cloverleaf/standard/validators/index.js';
@@ -33,6 +33,7 @@ export function loadTask(repoRoot: string, taskId: string): TaskDoc {
 
 export function saveTask(repoRoot: string, task: TaskDoc): void {
   validateOrThrow('https://cloverleaf.example/schemas/task.schema.json', task);
+  mkdirSync(tasksDir(repoRoot), { recursive: true });
   const path = join(tasksDir(repoRoot), `${task.id}.json`);
   writeFileSync(path, JSON.stringify(task, null, 2) + '\n');
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cloverleaf/reference-impl",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "Reference implementation of the Cloverleaf methodology as Claude Code skills. Implements the Tight Loop (Implementer + Reviewer).",
   "type": "module",
   "license": "MIT",

package/skills/cloverleaf-discover/SKILL.md

@@ -32,14 +32,14 @@ Each sub-skill runs from `main`. Between steps, confirm branch is `main` before
    [ "$(git rev-parse --abbrev-ref HEAD)" = "main" ] || { echo "Run /cloverleaf-discover from main" >&2; exit 1; }
    ```
 
-3. **Create the RFC** — inline `/cloverleaf-new-rfc $BRIEF_FILE` steps. Capture the printed RFC ID as `$RFC_ID`.
+3. **Create the RFC** — invoke `/cloverleaf-new-rfc $BRIEF_FILE`. Capture the printed RFC ID as `$RFC_ID`.
 
    Initialise revise loop counter:
    ```
    revise_loops = 0
    ```
 
-4. **Draft RFC (Researcher draftRfc)** — inline `/cloverleaf-draft-rfc $RFC_ID` steps. Researcher runs with a per-invocation bounce budget (3 bounces inside draft-rfc).
+4. **Draft RFC (Researcher draftRfc)** — invoke `/cloverleaf-draft-rfc $RFC_ID`. Researcher runs with a per-invocation bounce budget (3 bounces inside draft-rfc).
    - On bounce exhaustion (draft-rfc exits non-zero): dump state to `.cloverleaf/runs/$RFC_ID/discover-crash.json` and halt.
    - Reload the RFC. Its new status is either `spike-in-flight` (unknowns non-empty) or `planning` (unknowns empty).
 
@@ -48,7 +48,7 @@ Each sub-skill runs from `main`. Between steps, confirm branch is `main` before
    - If `spike-in-flight`: run every pending spike linked to this RFC:
      ```bash
      for each spike in .cloverleaf/spikes/*.json where parent_rfc.id === $RFC_ID AND status === "pending":
-       inline /cloverleaf-spike <SPIKE_ID> steps.
+       invoke /cloverleaf-spike <SPIKE_ID>.
      ```
 
      If `/cloverleaf-spike` exits non-zero or leaves the spike in a non-`completed` status after returning, treat that as bounce exhaustion: dump state to `.cloverleaf/runs/$RFC_ID/discover-crash.json` (including the spike ID) and halt. The orchestrator does NOT resume partially-run spike trees — if any spike is unresolved, the whole Discovery halts.
@@ -74,11 +74,11 @@ Each sub-skill runs from `main`. Between steps, confirm branch is `main` before
    ```
 
    Parse input. On:
-   - `approve` → inline `/cloverleaf-gate $RFC_ID approve` steps. Continue to step 7.
-   - `reject [reason]` → inline `/cloverleaf-gate $RFC_ID reject [reason]` steps. Exit with summary.
-   - `revise [reason]` → inline `/cloverleaf-gate $RFC_ID revise [reason]` steps. `revise_loops += 1`. If `revise_loops >= MAX_REVISE_LOOPS`, dump state and halt. Else loop back to step 4.
+   - `approve` → invoke `/cloverleaf-gate $RFC_ID approve`. Continue to step 7.
+   - `reject [reason]` → invoke `/cloverleaf-gate $RFC_ID reject [reason]`. Exit with summary.
+   - `revise [reason]` → invoke `/cloverleaf-gate $RFC_ID revise [reason]`. `revise_loops += 1`. If `revise_loops >= MAX_REVISE_LOOPS`, dump state and halt. Else loop back to step 4.
 
-7. **Plan breakdown** — inline `/cloverleaf-breakdown $RFC_ID` steps. Per-invocation bounce budget (3 bounces inside breakdown). Capture `$PLAN_ID`. On bounce exhaustion: dump state and halt.
+7. **Plan breakdown** — invoke `/cloverleaf-breakdown $RFC_ID`. Per-invocation bounce budget (3 bounces inside breakdown). Capture `$PLAN_ID`. On bounce exhaustion: dump state and halt.
 
 8. **Gate: task_batch_gate** — Plan is already in `gate-pending` (set by breakdown). Prompt:
    ```
@@ -86,8 +86,8 @@ Each sub-skill runs from `main`. Between steps, confirm branch is `main` before
    ```
 
    On:
-   - `approve` → inline `/cloverleaf-gate $PLAN_ID approve` steps. Continue.
-   - `reject [reason]` → inline `/cloverleaf-gate $PLAN_ID reject [reason]` steps. Exit with summary.
+   - `approve` → invoke `/cloverleaf-gate $PLAN_ID approve`. Continue.
+   - `reject [reason]` → invoke `/cloverleaf-gate $PLAN_ID reject [reason]`. Exit with summary.
 
 9. **Materialise tasks**:
    ```bash
@@ -123,7 +123,7 @@ Each sub-skill runs from `main`. Between steps, confirm branch is `main` before
     Run first root via /cloverleaf-run now? [y/N] > _
     ```
 
-    On `y` or `yes` (case-insensitive): inline `/cloverleaf-run <FIRST_ROOT>` steps. On anything else: exit with summary.
+    On `y` or `yes` (case-insensitive): invoke `/cloverleaf-run <FIRST_ROOT>`. On anything else: exit with summary.
 
 12. **Exit summary**:
     - RFC ID + final status
@@ -137,3 +137,4 @@ Each sub-skill runs from `main`. Between steps, confirm branch is `main` before
 - v0.5 prompts only for the FIRST DAG root. Multi-root concurrent Delivery is v0.6 scope.
 - All bounces and revise loops halt cleanly with a state dump at `.cloverleaf/runs/<RFC_ID>/discover-crash.json`. No partial work is left in an inconsistent state — the gate_decision events and work-item status are always coherent.
 - The readline prompts are compatible with the user's session-bridging tool for automated dogfooding.
+- "Invoke `/cloverleaf-X`" means spawn the sub-skill via the Skill tool — do NOT re-enter `/cloverleaf-discover` recursively. Sub-skills run in-process (same session) and return control to this orchestrator.

package/skills/cloverleaf-new-rfc/SKILL.md

@@ -47,8 +47,8 @@ cat > "$TMPFILE" <<EOF
   "id": "$RFC_ID",
   "status": "drafting",
   "owner": { "kind": "agent", "id": "researcher" },
-  "title": $(echo "$FIRST_LINE" | jq -Rs .),
-  "problem": $(echo "$BRIEF_CONTENT" | jq -Rs .),
+  "title": $(printf '%s' "$FIRST_LINE" | jq -Rs .),
+  "problem": $(printf '%s' "$BRIEF_CONTENT" | jq -Rs .),
   "solution": "TBD — to be populated by /cloverleaf-draft-rfc.",
   "unknowns": [],
   "acceptance_criteria": ["RFC body populated by researcher agent"],