Merge pull request #334 from affaan-m/codex/release-1.8.0-core

feat: v1.8.0 harness release (core reliability + parity + new commands)

scripts/claw.js

@@ -144,10 +144,11 @@ function searchSessions(query, dir) {
   const q = String(query || '').toLowerCase().trim();
   if (!q) return [];
 
-  const sessions = listSessions(dir);
+  const sessionDir = dir || getClawDir();
+  const sessions = listSessions(sessionDir);
   const results = [];
   for (const name of sessions) {
-    const p = getSessionPath(name);
+    const p = path.join(sessionDir, `${name}.md`);
     const content = loadHistory(p);
     if (!content) continue;
 
@@ -212,12 +213,12 @@ function exportSession(filePath, format, outputPath) {
   return { ok: false, message: `Unsupported export format: ${format}` };
 }
 
-function branchSession(currentSessionPath, newSessionName) {
+function branchSession(currentSessionPath, newSessionName, targetDir = getClawDir()) {
   if (!isValidSessionName(newSessionName)) {
     return { ok: false, message: `Invalid branch session name: ${newSessionName}` };
   }
 
-  const target = getSessionPath(newSessionName);
+  const target = path.join(targetDir, `${newSessionName}.md`);
   fs.mkdirSync(path.dirname(target), { recursive: true });
 
   const content = loadHistory(currentSessionPath);

tests/integration/hooks.test.js

@@ -100,13 +100,29 @@ function runHookWithInput(scriptPath, input = {}, env = {}, timeoutMs = 10000) {
 function runHookCommand(command, input = {}, env = {}, timeoutMs = 10000) {
   return new Promise((resolve, reject) => {
     const isWindows = process.platform === 'win32';
-    const shell = isWindows ? 'cmd' : 'bash';
-    const shellArgs = isWindows ? ['/d', '/s', '/c', command] : ['-lc', command];
+    const mergedEnv = { ...process.env, CLAUDE_PLUGIN_ROOT: REPO_ROOT, ...env };
+    const resolvedCommand = command.replace(
+      /\$\{([A-Z_][A-Z0-9_]*)\}/g,
+      (_, name) => String(mergedEnv[name] || '')
+    );
 
-    const proc = spawn(shell, shellArgs, {
-      env: { ...process.env, CLAUDE_PLUGIN_ROOT: REPO_ROOT, ...env },
-      stdio: ['pipe', 'pipe', 'pipe']
-    });
+    const nodeMatch = resolvedCommand.match(/^node\s+"([^"]+)"\s*(.*)$/);
+    const useDirectNodeSpawn = Boolean(nodeMatch);
+    const shell = isWindows ? 'cmd' : 'bash';
+    const shellArgs = isWindows ? ['/d', '/s', '/c', resolvedCommand] : ['-lc', resolvedCommand];
+    const nodeArgs = nodeMatch
+      ? [
+          nodeMatch[1],
+          ...Array.from(
+            nodeMatch[2].matchAll(/"([^"]*)"|(\S+)/g),
+            m => m[1] !== undefined ? m[1] : m[2]
+          )
+        ]
+      : [];
+
+    const proc = useDirectNodeSpawn
+      ? spawn('node', nodeArgs, { env: mergedEnv, stdio: ['pipe', 'pipe', 'pipe'] })
+      : spawn(shell, shellArgs, { env: mergedEnv, stdio: ['pipe', 'pipe', 'pipe'] });
 
     let stdout = '';
     let stderr = '';

tests/scripts/claw.test.js

@@ -252,35 +252,31 @@ function runTests() {
 
   if (test('searchSessions finds query in saved session', () => {
     const tmpDir = makeTmpDir();
-    const originalHome = process.env.HOME;
-    process.env.HOME = tmpDir;
     try {
-      const sessionPath = path.join(tmpDir, '.claude', 'claw', 'alpha.md');
-      fs.mkdirSync(path.dirname(sessionPath), { recursive: true });
+      const clawDir = path.join(tmpDir, '.claude', 'claw');
+      const sessionPath = path.join(clawDir, 'alpha.md');
+      fs.mkdirSync(clawDir, { recursive: true });
       appendTurn(sessionPath, 'User', 'Need oauth migration');
-      const results = searchSessions('oauth');
+      const results = searchSessions('oauth', clawDir);
       assert.strictEqual(results.length, 1);
       assert.strictEqual(results[0].session, 'alpha');
     } finally {
-      process.env.HOME = originalHome;
       fs.rmSync(tmpDir, { recursive: true, force: true });
     }
   })) passed++; else failed++;
 
   if (test('branchSession copies history into new branch session', () => {
     const tmpDir = makeTmpDir();
-    const originalHome = process.env.HOME;
-    process.env.HOME = tmpDir;
     try {
-      const source = path.join(tmpDir, '.claude', 'claw', 'base.md');
-      fs.mkdirSync(path.dirname(source), { recursive: true });
+      const clawDir = path.join(tmpDir, '.claude', 'claw');
+      const source = path.join(clawDir, 'base.md');
+      fs.mkdirSync(clawDir, { recursive: true });
       appendTurn(source, 'User', 'base content');
-      const result = branchSession(source, 'feature-branch');
+      const result = branchSession(source, 'feature-branch', clawDir);
       assert.strictEqual(result.ok, true);
       const branched = fs.readFileSync(result.path, 'utf8');
       assert.ok(branched.includes('base content'));
     } finally {
-      process.env.HOME = originalHome;
       fs.rmSync(tmpDir, { recursive: true, force: true });
     }
   })) passed++; else failed++;