@tritard/waterbrother 0.9.5 → 0.9.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tritard/waterbrother",
-  "version": "0.9.5",
+  "version": "0.9.7",
   "description": "Waterbrother: Grok-powered coding CLI with local tools, sessions, operator modes, and approval controls",
   "type": "module",
   "bin": {

package/src/cli.js

@@ -6207,43 +6207,57 @@ async function promptLoop(agent, session, context) {
     }
 
     if (line === "/experiment" || line.startsWith("/experiment ")) {
-      const goalArg = line.replace("/experiment", "").trim();
+      // Parse: /experiment <goal> --metric "command" --attempts N --time M
+      const rawArgs = line.replace("/experiment", "").trim();
+      if (!rawArgs) {
+        console.log('Usage: /experiment <goal> --metric "npm test" --attempts 5 --time 30');
+        console.log('  --metric   command to measure (required)');
+        console.log('  --attempts max attempts, 0=infinite (default: 0)');
+        console.log('  --time     time limit in minutes (default: 60)');
+        continue;
+      }
+
+      // Extract flags — parse --metric value (everything until next -- flag or end)
+      const attemptsMatch = rawArgs.match(/--attempts\s+(\d+)/);
+      const timeMatch = rawArgs.match(/--time\s+(\d+)/);
+      // Remove --attempts and --time first so --metric can grab the rest
+      let cleaned = rawArgs.replace(/--attempts\s+\d+/g, "").replace(/--time\s+\d+/g, "");
+      let metricCmd = "";
+      const metricIdx = cleaned.indexOf("--metric");
+      if (metricIdx !== -1) {
+        const afterMetric = cleaned.slice(metricIdx + 8).trim();
+        // If quoted, take the quoted content; otherwise take everything until end
+        if (afterMetric.startsWith('"')) {
+          const endQuote = afterMetric.indexOf('"', 1);
+          metricCmd = endQuote > 0 ? afterMetric.slice(1, endQuote) : afterMetric.slice(1);
+        } else {
+          metricCmd = afterMetric;
+        }
+        cleaned = cleaned.slice(0, metricIdx).trim();
+      }
+      const goalArg = cleaned.trim();
+
       if (!goalArg) {
-        console.log("Usage: /experiment <goal>  (e.g. /experiment speed up the test suite)");
+        console.log("experiment needs a goal");
         continue;
       }
 
-      // Build charter
       const charter = parseCharterFromGoal(goalArg);
 
-      // Ask for metric command
-      if (!charter.metric.command) {
-        try {
-          const metricCmd = await promptLine("metric command (e.g. npm test, python bench.py): ", { input: process.stdin, output: process.stdout });
-          charter.metric.command = metricCmd.trim();
-        } catch {
-          console.log("experiment canceled");
-          continue;
-        }
-        if (!charter.metric.command) {
-          console.log("no metric command — experiment canceled");
-          continue;
-        }
+      if (metricCmd) {
+        charter.metric.command = metricCmd.trim();
+      }
+      if (attemptsMatch) {
+        charter.budget.maxAttempts = parseInt(attemptsMatch[1], 10);
+      }
+      if (timeMatch) {
+        charter.budget.maxMinutes = parseInt(timeMatch[1], 10);
       }
 
-      // Ask for attempt budget (0 = infinite, runs until Ctrl+C or time limit)
-      try {
-        const attemptsStr = await promptLine("max attempts (0 = run until interrupted) [0]: ", { input: process.stdin, output: process.stdout });
-        const parsed = parseInt(attemptsStr.trim(), 10);
-        if (parsed > 0) charter.budget.maxAttempts = parsed;
-      } catch {}
-
-      // Ask for time budget
-      try {
-        const timeStr = await promptLine(`time limit in minutes [${charter.budget.maxMinutes}]: `, { input: process.stdin, output: process.stdout });
-        const parsed = parseInt(timeStr.trim(), 10);
-        if (parsed > 0) charter.budget.maxMinutes = parsed;
-      } catch {}
+      if (!charter.metric.command) {
+        console.log('experiment needs a metric: /experiment <goal> --metric "npm test"');
+        continue;
+      }
 
       const isInfinite = !charter.budget.maxAttempts || charter.budget.maxAttempts <= 0;
       console.log(`────────────────────────────────────────────────────────────`);

package/src/experiment.js

@@ -63,12 +63,12 @@ export async function runMetric({ command, extract, cwd }) {
     let stdout, stderr;
 
     if (isWin) {
-      const result = await execFileAsync(command, [], { ...execOpts, shell: true });
+      // Use PowerShell on Windows for access to proper commands
+      const result = await execFileAsync("powershell.exe", ["-NoProfile", "-Command", command], execOpts);
       stdout = String(result.stdout || "");
       stderr = String(result.stderr || "");
     } else {
-      const parts = command.split(/\s+/);
-      const result = await execFileAsync(parts[0], parts.slice(1), execOpts);
+      const result = await execFileAsync("/bin/sh", ["-c", command], execOpts);
       stdout = String(result.stdout || "");
       stderr = String(result.stderr || "");
     }