Merge pull request #8 from dakl/feat/terminal-emulator

feat: replace message UI with PTY terminal emulator

Author: dakl

electron-builder.yml

@@ -11,6 +11,7 @@ asarUnpack:
   - "**/node_modules/better-sqlite3/**"
   - "**/node_modules/bindings/**"
   - "**/node_modules/file-uri-to-path/**"
+  - "**/node_modules/node-pty/**"
 publish:
   provider: github
   owner: dakl

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "codez",
-  "version": "0.2.1",
+  "version": "2.1.0",
   "productName": "Codez",
   "description": "A macOS desktop app for managing AI coding agent sessions across git worktrees.",
   "author": "Daniel Klevebring",
@@ -27,6 +27,7 @@
   },
   "dependencies": {
     "better-sqlite3": "^12.6.2",
+    "node-pty": "^1.0.0",
     "react-markdown": "^10.1.0",
     "rehype-highlight": "^7.0.2",
     "remark-gfm": "^4.0.1"
@@ -40,6 +41,8 @@
     "@types/react": "^19.2.14",
     "@types/react-dom": "^19.2.3",
     "@vitejs/plugin-react": "^5.1.4",
+    "@xterm/addon-fit": "^0.10.0",
+    "@xterm/xterm": "^5.5.0",
     "concurrently": "^8.2.2",
     "electron": "^40.4.1",
     "electron-builder": "^26.7.0",

package.json

@@ -1,7 +1,6 @@
 import { describe, expect, it } from "vitest";
 import { createAdapter } from "./agent-registry";
 import { ClaudeAdapter } from "./claude-adapter";
-import { MistralAdapter } from "./mistral-adapter";
 
 describe("createAdapter", () => {
   it("returns a ClaudeAdapter for agent type 'claude'", () => {
@@ -13,15 +12,6 @@ describe("createAdapter", () => {
     expect(adapter).toBeInstanceOf(ClaudeAdapter);
   });
 
-  it("returns a MistralAdapter for agent type 'mistral'", () => {
-    const adapter = createAdapter({
-      agentType: "mistral",
-      sessionId: "sess-1",
-      worktreePath: "/tmp/wt",
-    });
-    expect(adapter).toBeInstanceOf(MistralAdapter);
-  });
-
   it("throws for unknown agent type", () => {
     expect(() =>
       createAdapter({
@@ -42,18 +32,4 @@ describe("createAdapter", () => {
     const args = adapter.buildStartArgs("test");
     expect(args).toContain("my-session");
   });
-
-  it("Mistral adapter builds correct arguments", () => {
-    const adapter = createAdapter({
-      agentType: "mistral",
-      sessionId: "test-session",
-      worktreePath: "/test/worktree",
-    });
-    const args = adapter.buildStartArgs("hello world");
-    expect(args).toContain("-p");
-    expect(args.some((arg) => arg.includes("hello world"))).toBe(true);
-    // streaming is now a positional argument after -p
-    expect(args).toContain("streaming");
-    // Note: vibe doesn't support --session-id for new sessions like claude does
-  });
 });

resources/icon.png

@@ -1,33 +1,20 @@
 import type { AgentType } from "../../shared/agent-types.js";
 import { ClaudeAdapter } from "./claude-adapter.js";
-import { MistralAdapter } from "./mistral-adapter.js";
 
 interface CreateAdapterOptions {
   agentType: AgentType;
   sessionId: string;
   worktreePath: string;
-  allowedTools?: string[];
   additionalDirs?: string[];
-  permissionMode?: string;
 }
 
-export function createAdapter(options: CreateAdapterOptions): ClaudeAdapter | MistralAdapter {
+export function createAdapter(options: CreateAdapterOptions): ClaudeAdapter {
   switch (options.agentType) {
     case "claude":
       return new ClaudeAdapter({
         sessionId: options.sessionId,
         worktreePath: options.worktreePath,
-        allowedTools: options.allowedTools,
         additionalDirs: options.additionalDirs,
-        permissionMode: options.permissionMode as "default" | "acceptEdits" | "bypassPermissions" | "plan",
-      });
-    case "mistral":
-      return new MistralAdapter({
-        sessionId: options.sessionId,
-        worktreePath: options.worktreePath,
-        allowedTools: options.allowedTools,
-        additionalDirs: options.additionalDirs,
-        permissionMode: options.permissionMode as "default" | "acceptEdits" | "bypassPermissions" | "plan",
       });
     default:
       throw new Error(`No adapter available for agent type: ${options.agentType}`);

src/__fixtures__/claude-stream-permission.ndjson

@@ -44,42 +44,6 @@ describe("ClaudeAdapter", () => {
       expect(args).not.toContain("--resume");
       expect(args).not.toContain("--continue");
     });
-
-    it("includes --allowedTools when allowedTools are provided", () => {
-      const adapterWithTools = new ClaudeAdapter({
-        sessionId: "test-session-1",
-        worktreePath: "/tmp/worktree",
-        allowedTools: ["Bash(git:*)", "Edit", "mcp__papershelf__*"],
-      });
-      const args = adapterWithTools.buildStartArgs("Hello");
-      expect(args).toContain("--allowedTools");
-      const toolsIndex = args.indexOf("--allowedTools");
-      expect(args[toolsIndex + 1]).toBe("Bash(git:*)");
-      expect(args[toolsIndex + 2]).toBe("Edit");
-      expect(args[toolsIndex + 3]).toBe("mcp__papershelf__*");
-    });
-
-    it("omits --allowedTools when none provided", () => {
-      const args = adapter.buildStartArgs("Hello");
-      expect(args).not.toContain("--allowedTools");
-    });
-
-    it("includes --permission-mode when not default", () => {
-      const adapterWithPerms = new ClaudeAdapter({
-        sessionId: "test-session-1",
-        worktreePath: "/tmp/worktree",
-        permissionMode: "acceptEdits",
-      });
-      const args = adapterWithPerms.buildStartArgs("Hello");
-      expect(args).toContain("--permission-mode");
-      const modeIndex = args.indexOf("--permission-mode");
-      expect(args[modeIndex + 1]).toBe("acceptEdits");
-    });
-
-    it("omits --permission-mode when default", () => {
-      const args = adapter.buildStartArgs("Hello");
-      expect(args).not.toContain("--permission-mode");
-    });
   });
 
   describe("buildResumeArgs", () => {
@@ -103,20 +67,6 @@ describe("ClaudeAdapter", () => {
       const args = adapter.buildResumeArgs("Continue");
       expect(args).not.toContain("--session-id");
     });
-
-    it("includes --allowedTools on resume when provided", () => {
-      const adapterWithTools = new ClaudeAdapter({
-        sessionId: "test-session-1",
-        worktreePath: "/tmp/worktree",
-        allowedTools: ["Edit", "Read"],
-      });
-      adapterWithTools.setAgentSessionId("claude-session-abc");
-      const args = adapterWithTools.buildResumeArgs("Continue");
-      expect(args).toContain("--allowedTools");
-      const toolsIndex = args.indexOf("--allowedTools");
-      expect(args[toolsIndex + 1]).toBe("Edit");
-      expect(args[toolsIndex + 2]).toBe("Read");
-    });
   });
 
   describe("parseLine", () => {
@@ -364,32 +314,6 @@ describe("ClaudeAdapter", () => {
       expect(event).toBeNull();
     });
 
-    it("maps control_request with can_use_tool to permission_request event", () => {
-      const event = adapter.parseLine({
-        type: "control_request",
-        request_id: "req_1_abc123",
-        request: {
-          subtype: "can_use_tool",
-          tool_name: "Bash",
-          input: { command: "git commit -m 'fix'" },
-        },
-      });
-      expect(event).not.toBeNull();
-      expect(event?.type).toBe("permission_request");
-      expect(event?.data.requestId).toBe("req_1_abc123");
-      expect(event?.data.toolName).toBe("Bash");
-      expect(event?.data.toolInput).toEqual({ command: "git commit -m 'fix'" });
-    });
-
-    it("returns null for control_request with unknown subtype", () => {
-      const event = adapter.parseLine({
-        type: "control_request",
-        request_id: "req_2",
-        request: { subtype: "unknown_subtype" },
-      });
-      expect(event).toBeNull();
-    });
-
     it("returns null for non-delta stream events we don't map", () => {
       const event = adapter.parseLine({
         type: "stream_event",
@@ -424,22 +348,5 @@ describe("ClaudeAdapter", () => {
       expect(types).toContain("message_complete");
       expect(types).toContain("session_end");
     });
-
-    it("parses permission request fixture into expected events", () => {
-      const lines = loadFixtureEvents("claude-stream-permission.ndjson");
-      const events = adapter.parseLines(lines);
-
-      const types = events.map((e) => e.type);
-      expect(types).toContain("session_start");
-      expect(types).toContain("tool_use_start");
-      expect(types).toContain("permission_request");
-      expect(types).toContain("tool_result");
-      expect(types).toContain("message_complete");
-      expect(types).toContain("session_end");
-
-      const permEvent = events.find((e) => e.type === "permission_request");
-      expect(permEvent.data.toolName).toBe("Bash");
-      expect(permEvent.data.requestId).toBe("req_1_abc");
-    });
   });
 });