caroushell 0.1.17 → 0.1.19

package/dist/history-suggester.js

@@ -10,6 +10,7 @@ const config_1 = require("./config");
 class HistorySuggester {
     constructor(filePath) {
         this.prefix = "⌛";
+        // `items` is the history as a list of commands, with the most recent at index 0
         this.items = [];
         this.filteredItems = [];
         this.maxItems = 1000;
@@ -71,18 +72,19 @@ class HistorySuggester {
         carousel.render();
     }
     descriptionForAi() {
+        // This goes into the prompt
         const lines = [];
         const maxHistoryLines = 20;
-        const start = Math.max(0, this.items.length - maxHistoryLines);
-        const end = this.items.length - 1;
-        const reverseSlice = this.items.slice(start, end).reverse();
-        if (reverseSlice.length > 0) {
-            lines.push(`The most recent command is: "${reverseSlice[0]}"`);
+        const start = 0;
+        const end = maxHistoryLines;
+        const newToOldSlice = this.items.slice(start, end);
+        if (newToOldSlice.length > 0) {
+            lines.push(`The most recent command is: "${newToOldSlice[0]}"`);
         }
-        if (reverseSlice.length > 1) {
+        if (newToOldSlice.length > 1) {
             lines.push("The most recent commands are (from recent to oldest):");
-            for (let i = 0; i < reverseSlice.length; i++) {
-                lines.push(`  ${i + 1}. ${reverseSlice[i]}`);
+            for (let i = 0; i < newToOldSlice.length; i++) {
+                lines.push(`  ${i + 1}. ${newToOldSlice[i]}`);
             }
         }
         return lines.join("\n");

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.17",
+  "version": "0.1.19",
   "description": "Terminal carousel that suggests commands from history, config, and AI.",
   "type": "commonjs",
   "main": "dist/main.js",
@@ -17,6 +17,7 @@
     "build": "tsc -p tsconfig.release.json",
     "prepare": "npm run build",
     "start": "node dist/main.js",
+    "test": "node --import tsx --test tests/history-suggester.test.ts",
     "test:generate": "tsx src/test-generate.ts",
     "lint": "eslint . --ext .ts",
     "release": "tsx scripts/release.ts"