create-harness-vibe-coding 0.8.3 → 0.8.5

package/templates/common/scripts/wf-remove.mjs

@@ -1,11 +1,11 @@
 #!/usr/bin/env node
 /**
- * wf-remove.mjs — Safely remove Harness framework files.
+ * wf-remove.mjs �?Safely remove Harness framework files.
  *
  * Classifies all Harness-owned files into:
- *   SAFE     — framework files matching stored checksums → auto-remove
- *   MODIFIED — framework files edited by user → MUST confirm
- *   USER     — user data files → NEVER remove
+ *   SAFE     �?framework files matching stored checksums �?auto-remove
+ *   MODIFIED �?framework files edited by user �?MUST confirm
+ *   USER     �?user data files �?NEVER remove
  *
  * Usage:
  *   node Harness/scripts/wf-remove.mjs                    # dry-run: show plan
@@ -23,12 +23,12 @@ import { resolve, dirname, join, sep } from 'path';
 import { fileURLToPath } from 'url';
 import { createInterface } from 'readline';
 
-const __dirname = dirname(fileURLToPath(import.meta.url));
+const SCRIPT_FILE = fileURLToPath(import.meta.url);
+const __dirname = dirname(SCRIPT_FILE);
+const SCRIPT_ROOT = resolve(__dirname, '..', '..');
 const ROOT = process.env.WF_ROOT
   ? resolve(process.env.WF_ROOT)
-  : (existsSync(resolve(process.cwd(), 'Harness', '.harness-version'))
-    ? process.cwd()
-    : resolve(__dirname, '..', '..'));
+  : SCRIPT_ROOT;
 const VERSION_FILE = resolve(ROOT, 'Harness', '.harness-version');
 
 // ── Classification ─────────────────────────────────────────────────
@@ -45,7 +45,7 @@ const HARNESS_PREFIXES = [
   'tests/.gitkeep',
 ];
 
-/** Files the user owns — NEVER remove. Consistent with wf-update-check PRESERVE_PATTERNS. */
+/** Files the user owns �?NEVER remove. Consistent with wf-update-check PRESERVE_PATTERNS. */
 const USER_DATA_PATTERNS = [
   /^Harness\/PROGRESS\.md$/,
   /^Harness\/tasks\//,
@@ -97,17 +97,71 @@ function isPurgeableHarnessData(file, { keepTasks }) {
   if (keepTasks && /^Harness\/tasks\//.test(file) && !isFrameworkTaskFile(file)) return false;
   return PURGEABLE_HARNESS_DATA_PATTERNS.some(p => p.test(file));
 }
-
-/** Framework files that should always exist (not part of removal). */
-const KEEP_FRAMEWORK = new Set([
-  'Harness/.harness-version',
-]);
-
-/** Directories to clean up if empty after file removal. */
-const CLEANUP_DIRS = [
+
+/** Framework files that should always exist (not part of removal). */
+const KEEP_FRAMEWORK = new Set([
+  'Harness/.harness-version',
+]);
+
+const BUILT_IN_AGENT_NAMES = [
+  'architect-manager',
+  'architect',
+  'context-master',
+  'debugger',
+  'docs-researcher',
+  'explore-manager',
+  'implement-manager',
+  'implementer',
+  'memory-master',
+  'planner',
+  'researcher',
+  'review-manager',
+  'reviewer',
+  'tdd-guide',
+  'test-writer',
+  'verifier',
+];
+
+const BUILT_IN_SKILL_NAMES = [
+  'browser-e2e',
+  'github-pr-review',
+  'python-backend',
+  'subagent-orchestrator',
+  'tdd',
+  'ts-react-frontend',
+  'ui-ux-review',
+  'wf',
+  'wf-auto',
+  'wf-auto-spark',
+  'wf-browser',
+  'wf-learn',
+  'wf-max',
+  'wf-readme',
+  'wf-remove',
+  'wf-review',
+  'wf-update',
+];
+
+const KNOWN_FRAMEWORK_FILES = new Set([
+  ...BUILT_IN_AGENT_NAMES.map(name => `.claude/agents/${name}.md`),
+  ...BUILT_IN_SKILL_NAMES.flatMap(name => [
+    `.claude/skills/${name}/SKILL.md`,
+    `.agents/skills/${name}/SKILL.md`,
+  ]),
+  '.claude/commands/wf-help.md',
+  '.claude/rules/ecc/common.md',
+]);
+
+function isKnownFrameworkFile(file) {
+  return KNOWN_FRAMEWORK_FILES.has(file);
+}
+
+/** Directories to clean up if empty after file removal. */
+const CLEANUP_DIRS = [
   '.claude/agents',
   '.claude/skills/wf-auto',
   '.claude/skills/wf-browser',
+  '.claude/skills/tdd',
   '.claude/skills/wf',
   '.claude/skills/wf-learn',
   '.claude/skills/wf-max',
@@ -118,6 +172,7 @@ const CLEANUP_DIRS = [
   '.claude/skills/subagent-orchestrator',
   '.agents/skills/wf-auto',
   '.agents/skills/wf-browser',
+  '.agents/skills/tdd',
   '.agents/skills/wf',
   '.agents/skills/wf-learn',
   '.agents/skills/wf-max',
@@ -249,7 +304,7 @@ function removeEmptyDirs(startDir) {
 
 async function askUser(question) {
   if (!process.stdin.isTTY) {
-    console.log('   (non-interactive — defaulting to KEEP)');
+    console.log('   (non-interactive �?defaulting to KEEP)');
     return 'k';
   }
   const rl = createInterface({ input: process.stdin, output: process.stdout });
@@ -298,10 +353,13 @@ async function main() {
     addFilesUnder(allFiles, dir, scanIssues);
   }
 
-  // Add framework files that might not be in checksums (newer additions)
+  // Add framework files that might not be in checksums (newer additions)
   const extraPatterns = [
+    ...KNOWN_FRAMEWORK_FILES,
     '.claude/skills/wf/SKILL.md',
     '.agents/skills/wf/SKILL.md',
+    '.claude/skills/tdd/SKILL.md',
+    '.agents/skills/tdd/SKILL.md',
     '.claude/skills/wf-auto/SKILL.md',
     '.agents/skills/wf-auto/SKILL.md',
     'Harness/WF-AUTO.md',
@@ -319,9 +377,9 @@ async function main() {
   }
 
   // 2. Classify every file
-  const safe = [];     // SAFE — matches checksum, auto-remove
-  const modified = []; // MODIFIED — user edited, must confirm
-  const user = [];     // USER — never remove
+  const safe = [];     // SAFE �?matches checksum, auto-remove
+  const modified = []; // MODIFIED �?user edited, must confirm
+  const user = [];     // USER �?never remove
   const skipped = [];  // File not on disk, traversal rejected, or framework-keep
   const purge = [];    // Explicitly requested Harness user-data removal
   skipped.push(...scanIssues);
@@ -343,13 +401,13 @@ async function main() {
         });
         continue;
       }
-      user.push({ file, reason: 'user data — NEVER removed' });
+      user.push({ file, reason: 'user data �?NEVER removed' });
       continue;
     }
 
     // Only allow deletion of files under harness-owned prefixes
     if (!isHarnessOwned(canonical)) {
-      user.push({ file, reason: 'outside harness prefix — NEVER removed' });
+      user.push({ file, reason: 'outside harness prefix �?NEVER removed' });
       continue;
     }
 
@@ -362,31 +420,40 @@ async function main() {
       skipped.push({ file, reason: 'not on disk' });
       continue;
     }
-
-    const currentHash = sha256File(diskPath);
-    const storedHash = storedChecksums[file];
-
-    if (!storedHash) {
-      modified.push({ file, currentHash, storedHash: 'none', reason: 'not in checksums' });
-    } else if (currentHash === storedHash) {
-      safe.push({ file, currentHash, storedHash });
-    } else {
+
+    const currentHash = sha256File(diskPath);
+    const storedHash = storedChecksums[file];
+
+    if (!storedHash) {
+      if (isKnownFrameworkFile(canonical)) {
+        safe.push({
+          file,
+          currentHash,
+          storedHash: currentHash,
+          reason: 'known framework file missing from legacy checksums',
+        });
+      } else {
+        modified.push({ file, currentHash, storedHash: 'none', reason: 'not in checksums' });
+      }
+    } else if (currentHash === storedHash) {
+      safe.push({ file, currentHash, storedHash });
+    } else {
       modified.push({ file, currentHash, storedHash, reason: 'user modified' });
     }
   }
 
-  // Also check for .claude/settings.json (might have user hooks)
+    // Also check for .claude/settings.json (might have user permissions)
   const settingsFile = resolve(ROOT, '.claude', 'settings.json');
   if (existsSync(settingsFile)) {
     const settingsHash = sha256File(settingsFile);
     const storedSettingsHash = storedChecksums['.claude/settings.json'];
     if (storedSettingsHash && settingsHash !== storedSettingsHash) {
-      // User modified settings — move to modified
+      // User modified settings �?move to modified
       // Remove from safe if it was there
       const idx = safe.findIndex(s => s.file === '.claude/settings.json');
       if (idx >= 0) {
         safe.splice(idx, 1);
-        modified.push({ file: '.claude/settings.json', currentHash: settingsHash, storedHash: storedSettingsHash, reason: 'user modified settings (may contain hooks/permissions)' });
+        modified.push({ file: '.claude/settings.json', currentHash: settingsHash, storedHash: storedSettingsHash, reason: 'user modified settings (may contain permissions)' });
       }
     }
   }
@@ -425,16 +492,16 @@ async function main() {
     return;
   }
 
-  console.log('\n🧹 WF-REMOVE — Harness framework removal plan\n');
+  console.log('\n🧹 WF-REMOVE �?Harness framework removal plan\n');
 
   if (safe.length > 0) {
-    console.log(`✅ SAFE (${safe.length} files — auto-remove, unmodified framework):`);
-    for (const s of safe) console.log(`   ✕ ${s.file}`);
+    console.log(`�?SAFE (${safe.length} files �?auto-remove, unmodified framework):`);
+    for (const s of safe) console.log(`   �?${s.file}`);
     console.log('');
   }
 
   if (modified.length > 0) {
-    console.log(`⚠ MODIFIED (${modified.length} files — REQUIRE CONFIRMATION):`);
+    console.log(`�?MODIFIED (${modified.length} files �?REQUIRE CONFIRMATION):`);
     for (const m of modified) {
       console.log(`   ? ${m.file}  [${m.reason}]`);
     }
@@ -442,8 +509,8 @@ async function main() {
   }
 
   if (user.length > 0) {
-    console.log(`🔒 USER DATA (${user.length} files — NEVER removed):`);
-    for (const u of user) console.log(`   ○ ${u.file}  [${u.reason}]`);
+    console.log(`🔒 USER DATA (${user.length} files �?NEVER removed):`);
+    for (const u of user) console.log(`   �?${u.file}  [${u.reason}]`);
     console.log('');
   }
 
@@ -460,17 +527,17 @@ async function main() {
     return;
   }
 
-  // 4. Apply — remove SAFE files automatically (rehash before unlink)
+  // 4. Apply �?remove SAFE files automatically (rehash before unlink)
   let safeRemoved = 0;
   let safeFailed = 0;
   for (const s of safe) {
     const diskPath = safePath(s.file);
-    if (!diskPath) { console.error(`   ✗ Traversal rejected: ${s.file}`); safeFailed++; continue; }
+    if (!diskPath) { console.error(`   �?Traversal rejected: ${s.file}`); safeFailed++; continue; }
     if (!existsSync(diskPath)) continue;
     // Re-verify hash hasn't changed since classification
     const currentHash = sha256File(diskPath);
     if (currentHash !== s.storedHash) {
-      console.log(`   ⊘ Skipped (modified since classification): ${s.file}`);
+      console.log(`   �?Skipped (modified since classification): ${s.file}`);
       safeFailed++;
       continue;
     }
@@ -478,13 +545,13 @@ async function main() {
       unlinkSync(diskPath);
       safeRemoved++;
     } catch (e) {
-      console.error(`   ✗ Failed to remove: ${s.file} — ${e.message}`);
+      console.error(`   �?Failed to remove: ${s.file} �?${e.message}`);
       safeFailed++;
     }
   }
-  console.log(`✓ Removed ${safeRemoved} safe files.`);
+  console.log(`�?Removed ${safeRemoved} safe files.`);
 
-  // 5. Handle MODIFIED files — prompt user
+  // 5. Handle MODIFIED files �?prompt user
   let modifiedRemoved = 0;
   let modifiedKept = 0;
   let modifiedFailed = 0;
@@ -504,7 +571,7 @@ async function main() {
     }
     if (yes) {
       // Non-interactive mode: skip all modified
-      console.log(`   ⊘ Skipped (modified): ${m.file}`);
+      console.log(`   �?Skipped (modified): ${m.file}`);
       modifiedKept++;
       continue;
     }
@@ -516,17 +583,17 @@ async function main() {
     const answer = await askUser('   Choose [d/k]: ');
     if (answer === 'd' || answer === 'delete') {
       const diskPath = safePath(m.file);
-      if (!diskPath) { console.error(`   ✗ Traversal rejected: ${m.file}`); modifiedFailed++; continue; }
+      if (!diskPath) { console.error(`   �?Traversal rejected: ${m.file}`); modifiedFailed++; continue; }
       try {
         unlinkSync(diskPath);
-        console.log(`   ✓ Deleted: ${m.file}`);
+        console.log(`   �?Deleted: ${m.file}`);
         modifiedRemoved++;
       } catch (e) {
-        console.error(`   ✗ Failed: ${m.file} — ${e.message}`);
+        console.error(`   �?Failed: ${m.file} �?${e.message}`);
         modifiedFailed++;
       }
     } else {
-      console.log(`   ⊘ Kept: ${m.file}`);
+      console.log(`   �?Kept: ${m.file}`);
       modifiedKept++;
     }
   }
@@ -587,7 +654,7 @@ async function main() {
       }
     } else if (!existsSync(harPath) || (existsSync(harPath) && readdirSync(harPath).filter(f => f !== '.harness-version').length === 0)) {
       unlinkSync(VERSION_FILE);
-      console.log('✓ Removed .harness-version (Harness directory empty).');
+      console.log('�?Removed .harness-version (Harness directory empty).');
     }
   }
 
@@ -606,7 +673,7 @@ async function main() {
     if (pruned > 0) {
       versionData.checksums = checksums;
       writeFileSync(VERSION_FILE, JSON.stringify(versionData, null, 2) + '\n', 'utf-8');
-      console.log(`✓ Pruned ${pruned} stale checksum(s) from .harness-version.`);
+      console.log(`�?Pruned ${pruned} stale checksum(s) from .harness-version.`);
     }
   }
 
@@ -620,7 +687,7 @@ async function main() {
         // Strip the harness section: everything from "## 1. Harness Binding" to the next "## "
         const cleaned = content.replace(/## 1\. Harness Binding[\s\S]*?(?=## 2\.)/, '');
         writeFileSync(claudePath, cleaned, 'utf-8');
-        console.log('✓ Stripped Harness binding section from CLAUDE.md.');
+        console.log('�?Stripped Harness binding section from CLAUDE.md.');
       }
     }
   }

package/templates/optional/skills/browser-e2e/Harness/workflows/browser-e2e.md

@@ -17,7 +17,7 @@ Browser evidence in this project follows the contract:
 2. **CLI mode is preferred for deterministic steps** — use `browser-use open/state/click/screenshot` for predictable flows
 3. **Agent mode is for dynamic exploration** — use Browser Use Agent API when the page structure is unknown or changing
 4. **Evidence goes to the task directory** — `Harness/tasks/<task-id>/evidence/*.png`
-5. **Stable UI selector contract** — Stable accessible labels/roles and stable test hooks such as `data-testid` are required for critical UI controls and states: inputs, buttons, filters, rows, empty/error/loading states.
+5. **Stable UI selector contract** — Stable accessible labels/roles and stable test selectors such as `data-testid` are required for critical UI controls and states: inputs, buttons, filters, rows, empty/error/loading states.
 
 ## Chrome DevTools / CDP / MCP Checklist
 
@@ -30,7 +30,14 @@ Browser evidence in this project follows the contract:
 - [ ] Capture screenshot, trace, video, or result artifact paths
 - [ ] Clean up any dev server or browser processes
 
-## Quick Install
+### Harness Bridge
+
+For frontend-backend flows, use `Harness/HARNESS_BRIDGE.md` to validate UI
+selectors, API payloads, seeded data, runtime state probes, and CDP/network
+traces. Browser validation must produce an AC-by-AC result matrix when the task
+has acceptance criteria.
+
+## Quick Install
 
 ```bash
 # One-time setup
@@ -58,7 +65,8 @@ If `browser-use` is not installed, fall back to:
 
 ## Integration Points
 
-- **WF mode**: when browser-visible changes are made, follow the evidence contract in `Harness/WF.md#Browser And API Evidence`
+- **WF mode**: when browser-visible changes are made, follow the evidence contract in `Harness/WF.md#Browser And API Evidence`
+- **Harness Bridge**: use `Harness/HARNESS_BRIDGE.md` for UI contract, API contract, seeded test data, runtime state probes, and network trace collection.
 - **wf-browser**: Claude Code uses `.claude/skills/wf-browser/SKILL.md`; Codex uses `.agents/skills/wf-browser/SKILL.md`. Both load this workflow.
 - **MEMORY.md**: registered as optional workflow skill
 - **README.md**: routing table row "Browser E2E testing or automation" → browser-e2e

package/templates/optional/skills/ts-react-frontend/Harness/workflows/ts-react-frontend.md

@@ -7,7 +7,7 @@
 - Real-browser smoke or screenshot evidence for user-visible changes.
 - Console/runtime error check result for the changed screen or flow.
 - Notes on responsive behavior and important interaction states.
-- Stable accessible labels/roles and stable test hooks such as `data-testid` are required for critical UI controls and states: inputs, buttons, filters, rows, empty/error/loading states.
+- Stable accessible labels/roles and stable test selectors such as `data-testid` are required for critical UI controls and states: inputs, buttons, filters, rows, empty/error/loading states.
 
 Typecheck, build, and unit tests are necessary signals but are not enough for user-visible React changes. Before claiming UI acceptance, load the app in a real browser by Playwright, Chrome DevTools/CDP, or a documented manual run and capture screenshot/trace/console evidence.
 For TS/React UI work, capture the selector contract in the feature doc before implementation so tests and manual checks can target stable selectors instead of component internals or brittle DOM paths.

package/templates/common/.codex/hooks.json

@@ -1,59 +0,0 @@
-{
-  "hooks": {
-    "SessionStart": [
-      {
-        "matcher": "",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node Harness/scripts/wf-mode-hook.mjs"
-          }
-        ]
-      }
-    ],
-    "UserPromptSubmit": [
-      {
-        "matcher": "",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node Harness/scripts/wf-mode-hook.mjs"
-          }
-        ]
-      }
-    ],
-    "PreToolUse": [
-      {
-        "matcher": "Edit|Write|MultiEdit|Bash",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node Harness/scripts/wf-mode-hook.mjs"
-          }
-        ]
-      }
-    ],
-    "PostToolUse": [
-      {
-        "matcher": "",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node Harness/scripts/wf-mode-hook.mjs"
-          }
-        ]
-      }
-    ],
-    "Stop": [
-      {
-        "matcher": "",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node Harness/scripts/wf-mode-hook.mjs"
-          }
-        ]
-      }
-    ]
-  }
-}