feat: add newTerminal option for executing commands in fresh terminal each time

Button clicks always reused same terminal, preventing parallel execution or output comparison.
Example: clicking npm test while running would rerun in same terminal, losing previous output.

Added branch logic to create new terminal without Map storage when newTerminal: true.
Existing behavior (reuse) preserved as default false for backward compatibility.

fix #245

Author: kubrickcode

package.json

@@ -246,6 +246,11 @@
                 "type": "boolean",
                 "default": false,
                 "description": "%config.buttons.insertOnly.description%"
+              },
+              "newTerminal": {
+                "type": "boolean",
+                "default": false,
+                "description": "%config.buttons.newTerminal.description%"
               }
             },
             "required": [

package.nls.json

@@ -21,6 +21,7 @@
   "config.buttons.executeAll.description": "Execute all commands in group simultaneously",
   "config.buttons.group.description": "Sub-commands for grouped buttons (supports infinite nesting)",
   "config.buttons.insertOnly.description": "Insert command into terminal without executing (user must press Enter manually)",
+  "config.buttons.newTerminal.description": "Always create a new terminal for each execution instead of reusing existing one",
   "config.buttons.name.description": "Button label (supports $(icon-name) syntax)",
   "config.buttons.shortcut.description": "Single key shortcut for quick access (e.g. 'q', '1', 'F1')",
   "config.buttons.terminalName.description": "Custom terminal name",

package.nls.ko.json

@@ -21,6 +21,7 @@
   "config.buttons.executeAll.description": "그룹의 모든 명령 동시 실행",
   "config.buttons.group.description": "그룹 버튼의 하위 명령 (무한 중첩 지원)",
   "config.buttons.insertOnly.description": "명령을 실행하지 않고 터미널에만 입력 (사용자가 직접 Enter 입력 필요)",
+  "config.buttons.newTerminal.description": "기존 터미널을 재사용하지 않고 매번 새 터미널에서 실행",
   "config.buttons.name.description": "버튼 라벨 ($(icon-name) 구문 지원)",
   "config.buttons.shortcut.description": "빠른 접근을 위한 단일 키 단축키 (예: 'q', '1', 'F1')",
   "config.buttons.terminalName.description": "사용자 정의 터미널 이름",

src/internal/adapters.ts

@@ -17,7 +17,8 @@ export type TerminalExecutor = (
   useVsCodeApi?: boolean,
   terminalName?: string,
   buttonName?: string,
-  buttonRef?: object
+  buttonRef?: object,
+  newTerminal?: boolean
 ) => void;
 
 export type ConfigReader = {

src/internal/command-executor.spec.ts

@@ -557,7 +557,8 @@ describe("command-executor", () => {
           command: "echo test",
           id: "test-terminal-1",
           name: "Test Button",
-        })
+        }),
+        false
       );
     });
 
@@ -582,7 +583,8 @@ describe("command-executor", () => {
           id: "test-vscode-api",
           name: "Test Button",
           useVsCodeApi: true,
-        })
+        }),
+        false
       );
     });
 
@@ -607,7 +609,8 @@ describe("command-executor", () => {
           id: "test-terminal-name",
           name: "Test Button",
           terminalName: "Custom Terminal",
-        })
+        }),
+        false
       );
     });
 
@@ -634,7 +637,8 @@ describe("command-executor", () => {
           name: "Test Button",
           terminalName: "Custom Terminal",
           useVsCodeApi: true,
-        })
+        }),
+        false
       );
     });
 
@@ -663,6 +667,52 @@ describe("command-executor", () => {
 
       expect(mockTerminalExecutor).not.toHaveBeenCalled();
     });
+
+    it("should call terminalExecutor with newTerminal true", () => {
+      const mockTerminalExecutor = vi.fn();
+      const button: ButtonConfig = {
+        command: "echo test",
+        id: "test-new-terminal",
+        name: "Test Button",
+        newTerminal: true,
+      };
+
+      executeTerminalCommand(button, mockTerminalExecutor);
+
+      expect(mockTerminalExecutor).toHaveBeenCalledWith(
+        "echo test",
+        false,
+        undefined,
+        "Test Button",
+        expect.objectContaining({
+          command: "echo test",
+          id: "test-new-terminal",
+          name: "Test Button",
+          newTerminal: true,
+        }),
+        true
+      );
+    });
+
+    it("should call terminalExecutor with newTerminal false when not specified", () => {
+      const mockTerminalExecutor = vi.fn();
+      const button: ButtonConfig = {
+        command: "echo test",
+        id: "test-default-terminal",
+        name: "Test Button",
+      };
+
+      executeTerminalCommand(button, mockTerminalExecutor);
+
+      expect(mockTerminalExecutor).toHaveBeenCalledWith(
+        "echo test",
+        false,
+        undefined,
+        "Test Button",
+        expect.anything(),
+        false
+      );
+    });
   });
 
   describe("executeCommandsRecursively", () => {
@@ -697,23 +747,26 @@ describe("command-executor", () => {
         false,
         undefined,
         "Command 1[0]",
-        expect.anything()
+        expect.anything(),
+        false
       );
       expect(mockTerminalExecutor).toHaveBeenNthCalledWith(
         2,
         "echo test2",
         true,
         undefined,
         "Command 2[1]",
-        expect.anything()
+        expect.anything(),
+        false
       );
       expect(mockTerminalExecutor).toHaveBeenNthCalledWith(
         3,
         "echo test3",
         false,
         "Custom Terminal",
         "Command 3[2]",
-        expect.anything()
+        expect.anything(),
+        false
       );
     });
 
@@ -749,15 +802,17 @@ describe("command-executor", () => {
         false,
         undefined,
         "Group Command[0]>Child 1[0]",
-        expect.anything()
+        expect.anything(),
+        false
       );
       expect(mockTerminalExecutor).toHaveBeenNthCalledWith(
         2,
         "echo child2",
         true,
         undefined,
         "Group Command[0]>Child 2[1]",
-        expect.anything()
+        expect.anything(),
+        false
       );
     });
 
@@ -821,15 +876,17 @@ describe("command-executor", () => {
         false,
         undefined,
         "Level 1 Group[0]>Level 2 Group[0]>Level 3 Command[0]",
-        expect.anything()
+        expect.anything(),
+        false
       );
       expect(mockTerminalExecutor).toHaveBeenNthCalledWith(
         2,
         "echo level2",
         false,
         undefined,
         "Level 1 Group[0]>Level 2 Command[1]",
-        expect.anything()
+        expect.anything(),
+        false
       );
     });
 
@@ -856,7 +913,8 @@ describe("command-executor", () => {
         false,
         undefined,
         "Valid Command[0]",
-        expect.anything()
+        expect.anything(),
+        false
       );
     });
 
@@ -883,7 +941,8 @@ describe("command-executor", () => {
         false,
         undefined,
         "Valid Command[0]",
-        expect.anything()
+        expect.anything(),
+        false
       );
     });
 
@@ -944,15 +1003,17 @@ describe("command-executor", () => {
         false,
         undefined,
         "Regular Command[0]",
-        expect.anything()
+        expect.anything(),
+        false
       );
       expect(mockTerminalExecutor).toHaveBeenNthCalledWith(
         2,
         "echo child",
         false,
         undefined,
         "Group with executeAll[1]>Child Command[0]",
-        expect.anything()
+        expect.anything(),
+        false
       );
     });
 
@@ -1011,23 +1072,26 @@ describe("command-executor", () => {
         false,
         undefined,
         "Root Group[0]>Branch 1[0]>Leaf 1[0]",
-        expect.anything()
+        expect.anything(),
+        false
       );
       expect(mockTerminalExecutor).toHaveBeenNthCalledWith(
         2,
         "echo leaf2",
         false,
         undefined,
         "Root Group[0]>Branch 1[0]>Leaf 2[1]",
-        expect.anything()
+        expect.anything(),
+        false
       );
       expect(mockTerminalExecutor).toHaveBeenNthCalledWith(
         3,
         "echo direct",
         false,
         undefined,
         "Root Group[0]>Direct Command[2]",
-        expect.anything()
+        expect.anything(),
+        false
       );
     });
 
@@ -1049,7 +1113,8 @@ describe("command-executor", () => {
         false,
         undefined,
         "Test Command[0]",
-        expect.objectContaining({ insertOnly: true })
+        expect.objectContaining({ insertOnly: true }),
+        false
       );
     });
 
@@ -1086,15 +1151,64 @@ describe("command-executor", () => {
         false,
         undefined,
         "Dev Tools[0]>Docker Shell[0]",
-        expect.objectContaining({ insertOnly: true })
+        expect.objectContaining({ insertOnly: true }),
+        false
       );
       expect(mockTerminalExecutor).toHaveBeenNthCalledWith(
         2,
         "git status",
         false,
         undefined,
         "Dev Tools[0]>Git Status[1]",
-        expect.objectContaining({ insertOnly: false })
+        expect.objectContaining({ insertOnly: false }),
+        false
+      );
+    });
+
+    it("should pass newTerminal flag through buttonRef in nested groups", () => {
+      const mockTerminalExecutor = vi.fn();
+      const commands: ButtonConfig[] = [
+        {
+          executeAll: true,
+          group: [
+            {
+              command: "npm run build",
+              id: "build-cmd",
+              name: "Build",
+              newTerminal: true,
+            },
+            {
+              command: "npm run test",
+              id: "test-cmd",
+              name: "Test",
+              newTerminal: false,
+            },
+          ],
+          id: "ci-group",
+          name: "CI Commands",
+        },
+      ];
+
+      executeCommandsRecursively(commands, mockTerminalExecutor);
+
+      expect(mockTerminalExecutor).toHaveBeenCalledTimes(2);
+      expect(mockTerminalExecutor).toHaveBeenNthCalledWith(
+        1,
+        "npm run build",
+        false,
+        undefined,
+        "CI Commands[0]>Build[0]",
+        expect.objectContaining({ newTerminal: true }),
+        true
+      );
+      expect(mockTerminalExecutor).toHaveBeenNthCalledWith(
+        2,
+        "npm run test",
+        false,
+        undefined,
+        "CI Commands[0]>Test[1]",
+        expect.objectContaining({ newTerminal: false }),
+        false
       );
     });
   });
@@ -1172,7 +1286,8 @@ describe("command-executor", () => {
         false,
         undefined,
         "Test 1",
-        expect.objectContaining({ shortcut: "a" })
+        expect.objectContaining({ shortcut: "a" }),
+        false
       );
     });
 
@@ -1223,7 +1338,8 @@ describe("command-executor", () => {
         false,
         undefined,
         "Test 2",
-        expect.objectContaining({ shortcut: "b" })
+        expect.objectContaining({ shortcut: "b" }),
+        false
       );
     });
 

src/internal/command-executor.ts

@@ -140,7 +140,8 @@ export const executeTerminalCommand = (
     button.useVsCodeApi || false,
     button.terminalName,
     button.name,
-    button
+    button,
+    button.newTerminal || false
   );
 
   eventBus?.emit("button:executed", { button, success: true });
@@ -208,7 +209,14 @@ export const executeCommandsRecursively = (
     if (cmd.group && cmd.executeAll) {
       executeCommandsRecursively(cmd.group, terminalExecutor, buttonId, eventBus);
     } else if (cmd.command) {
-      terminalExecutor(cmd.command, cmd.useVsCodeApi || false, cmd.terminalName, buttonId, cmd);
+      terminalExecutor(
+        cmd.command,
+        cmd.useVsCodeApi || false,
+        cmd.terminalName,
+        buttonId,
+        cmd,
+        cmd.newTerminal || false
+      );
       eventBus?.emit("button:executed", { button: cmd, success: true });
     }
   });