fix: Include read file path in tool title (#158)

Author: benbrandt

src/CodexToolCallMapper.ts

@@ -202,7 +202,7 @@ function createCommandActionEvent(
             toolCallId: id,
             status: acpStatus,
             kind: "read",
-            title: "Read file",
+            title: `Read file '${commandAction.path}'`,
             locations: [{ path: commandAction.path }],
         };
     } else if (commandAction.type === "search") {

src/__tests__/CodexACPAgent/command-action-events.test.ts

@@ -89,6 +89,42 @@ describe('CodexEventHandler - command action events', () => {
         );
     });
 
+    it('should include the path in read file command titles', async () => {
+        const readFileNotification: ServerNotification = {
+            method: 'item/started',
+            params: {
+                threadId: sessionId,
+                turnId: 'turn-1',
+                item: {
+                    type: 'commandExecution',
+                    id: 'command-read-file',
+                    command: 'sed -n "1,80p" /test/project/src/index.ts',
+                    cwd: '/test/project',
+                    processId: null,
+                    source: 'agent',
+                    status: 'inProgress',
+                    commandActions: [
+                        {
+                            type: 'read',
+                            command: 'sed -n "1,80p" /test/project/src/index.ts',
+                            name: 'sed',
+                            path: '/test/project/src/index.ts',
+                        },
+                    ],
+                    aggregatedOutput: null,
+                    exitCode: null,
+                    durationMs: null,
+                },
+            },
+        };
+
+        await setupPromptAndSendNotifications(mockFixture, sessionId, sessionState, [readFileNotification]);
+
+        await expect(mockFixture.getAcpConnectionDump([])).toMatchFileSnapshot(
+            'data/command-read-file-with-path.json'
+        );
+    });
+
     it('should handle search command with query and path', async () => {
         const searchNotification: ServerNotification = {
             method: 'item/started',

src/__tests__/CodexACPAgent/data/command-read-file-with-path.json

@@ -0,0 +1,20 @@
+{
+  "method": "sessionUpdate",
+  "args": [
+    {
+      "sessionId": "test-session-id",
+      "update": {
+        "sessionUpdate": "tool_call",
+        "toolCallId": "command-read-file",
+        "status": "in_progress",
+        "kind": "read",
+        "title": "Read file '/test/project/src/index.ts'",
+        "locations": [
+          {
+            "path": "/test/project/src/index.ts"
+          }
+        ]
+      }
+    }
+  ]
+}