@cloverleaf/reference-impl 0.5.4 → 0.5.5

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.4",
+  "version": "0.5.5",
   "author": {
     "name": "Renato D'Arrigo",
     "email": "renato.darrigo@gmail.com"

package/VERSION

@@ -1 +1 @@
-0.5.4
+0.5.5

package/dist/prep-worktree.mjs

@@ -1,4 +1,4 @@
-import { cpSync, existsSync } from 'node:fs';
+import { cpSync, existsSync, rmSync } from 'node:fs';
 import { execSync } from 'node:child_process';
 import { join } from 'node:path';
 /**
@@ -41,10 +41,23 @@ export function prepWorktree(mainRoot, worktreePath) {
     const wtRefImplNm = join(worktreePath, 'reference-impl', 'node_modules');
     // verbatimSymlinks keeps relative symlink targets byte-identical, so the @cloverleaf/standard
     // link in reference-impl/node_modules/ resolves against the worktree after copy.
-    cpSync(mainStandardNm, wtStandardNm, { recursive: true, verbatimSymlinks: true });
-    cpSync(mainRefImplNm, wtRefImplNm, { recursive: true, verbatimSymlinks: true });
+    //
+    // primeCopy wipes the destination before cpSync. Two reasons:
+    //   1. Idempotence: a partial prior run (or a re-invocation after a test failure) may
+    //      leave partial state; we must not trip on it.
+    //   2. cpSync with verbatimSymlinks: true does not reliably overwrite an existing
+    //      symlink at the destination even with force: true (CLV-20 Reviewer repro was
+    //      EEXIST on vite/node_modules/.bin on second invocation).
+    primeCopy(mainStandardNm, wtStandardNm);
+    primeCopy(mainRefImplNm, wtRefImplNm);
     execSync('npm run build', {
         cwd: join(worktreePath, 'standard'),
         stdio: 'pipe',
     });
 }
+function primeCopy(src, dst) {
+    if (existsSync(dst)) {
+        rmSync(dst, { recursive: true, force: true });
+    }
+    cpSync(src, dst, { recursive: true, verbatimSymlinks: true });
+}

package/lib/prep-worktree.ts

@@ -1,4 +1,4 @@
-import { cpSync, existsSync } from 'node:fs';
+import { cpSync, existsSync, rmSync } from 'node:fs';
 import { execSync } from 'node:child_process';
 import { join } from 'node:path';
 
@@ -45,11 +45,25 @@ export function prepWorktree(mainRoot: string, worktreePath: string): void {
 
   // verbatimSymlinks keeps relative symlink targets byte-identical, so the @cloverleaf/standard
   // link in reference-impl/node_modules/ resolves against the worktree after copy.
-  cpSync(mainStandardNm, wtStandardNm, { recursive: true, verbatimSymlinks: true });
-  cpSync(mainRefImplNm, wtRefImplNm, { recursive: true, verbatimSymlinks: true });
+  //
+  // primeCopy wipes the destination before cpSync. Two reasons:
+  //   1. Idempotence: a partial prior run (or a re-invocation after a test failure) may
+  //      leave partial state; we must not trip on it.
+  //   2. cpSync with verbatimSymlinks: true does not reliably overwrite an existing
+  //      symlink at the destination even with force: true (CLV-20 Reviewer repro was
+  //      EEXIST on vite/node_modules/.bin on second invocation).
+  primeCopy(mainStandardNm, wtStandardNm);
+  primeCopy(mainRefImplNm, wtRefImplNm);
 
   execSync('npm run build', {
     cwd: join(worktreePath, 'standard'),
     stdio: 'pipe',
   });
 }
+
+function primeCopy(src: string, dst: string): void {
+  if (existsSync(dst)) {
+    rmSync(dst, { recursive: true, force: true });
+  }
+  cpSync(src, dst, { recursive: true, verbatimSymlinks: true });
+}

package/package.json

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