caroushell 0.1.16 → 0.1.18

package/dist/history-suggester.js

@@ -11,6 +11,7 @@ class HistorySuggester {
     constructor(filePath) {
         this.prefix = "⌛";
         this.items = [];
+        this.filteredItems = [];
         this.maxItems = 1000;
         const home = process.env.HOME || process.env.USERPROFILE || process.cwd();
         // Default path: ~/.caroushell/history
@@ -24,9 +25,11 @@ class HistorySuggester {
         try {
             const data = await fs_1.promises.readFile(this.filePath, "utf8");
             this.items = this.parseHistory(data);
+            this.filteredItems = this.items;
         }
         catch {
             this.items = [];
+            this.filteredItems = [];
         }
     }
     async add(command) {
@@ -45,29 +48,26 @@ class HistorySuggester {
         await fs_1.promises.appendFile(this.filePath, this.serializeHistoryEntry(command), "utf8");
     }
     latest() {
-        return this.items;
+        return this.filteredItems;
     }
     async refreshSuggestions(carousel, maxDisplayed) {
         const input = carousel.getCurrentRow();
-        let results;
-        if (!input) {
-            // this.items 0 index is newest
-            results = this.items;
-        }
-        else {
+        let suggestedItems = this.items;
+        if (input) {
+            // filter by input substring
             const q = input.toLowerCase();
-            const matched = [];
+            suggestedItems = [];
             // iterate from newest to oldest so we skip older duplicates
             const seen = new Set();
             for (let i = 0; i < this.items.length; i++) {
                 const it = this.items[i];
                 if (it.toLowerCase().includes(q) && !seen.has(it)) {
                     seen.add(it);
-                    matched.push(it);
+                    suggestedItems.push(it);
                 }
             }
-            results = matched;
         }
+        this.filteredItems = suggestedItems;
         carousel.render();
     }
     descriptionForAi() {

package/dist/spawner.js

@@ -3,9 +3,10 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.runUserCommand = runUserCommand;
 const child_process_1 = require("child_process");
 const process_1 = require("process");
+const logs_1 = require("./logs");
 const isWin = process.platform === "win32";
 const shellBinary = isWin ? "cmd.exe" : "/bin/bash";
-const shellArgs = isWin ? ["/c"] : ["-lc"];
+const shellArgs = isWin ? ["/d", "/s", "/c"] : ["-lc"];
 const builtInCommands = {
     cd: async (args) => {
         if (args.length === 1) {
@@ -47,6 +48,7 @@ function expandVars(input) {
     return out;
 }
 async function runUserCommand(command) {
+    (0, logs_1.logLine)(`Running command: ${command}`);
     const trimmed = command.trim();
     if (!trimmed)
         return false;
@@ -54,11 +56,11 @@ async function runUserCommand(command) {
     if (typeof args[0] === "string" && builtInCommands[args[0]]) {
         return await builtInCommands[args[0]](args);
     }
-    // "shell: true" to prevent the bug of `echo "asdf"` outputting
-    // \"Asdf\" instead of "Asdf"
+    // "windowsVerbatimArguments: true" to prevent the bug of `echo "asdf"` outputting
+    // \"asdf\" instead of "asdf". I wonder why node defaults to quoting args on windows.
     const proc = (0, child_process_1.spawn)(shellBinary, [...shellArgs, command], {
         stdio: "inherit",
-        // shell: true,
+        windowsVerbatimArguments: true,
     });
     await new Promise((resolve, reject) => {
         proc.on("error", reject);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "caroushell",
-  "version": "0.1.16",
+  "version": "0.1.18",
   "description": "Terminal carousel that suggests commands from history, config, and AI.",
   "type": "commonjs",
   "main": "dist/main.js",