feat: little things - shortcuts, worktrees, unread filter, CLI read-only

src/__tests__/cli/commands/send.test.ts

@@ -81,7 +81,8 @@ describe('send command', () => {
 			'claude-code',
 			'/path/to/project',
 			'Hello world',
-			undefined
+			undefined,
+			{ readOnlyMode: undefined }
 		);
 		expect(consoleSpy).toHaveBeenCalledTimes(1);
 
@@ -128,7 +129,8 @@ describe('send command', () => {
 			'claude-code',
 			'/path/to/project',
 			'Continue from before',
-			'session-xyz-789'
+			'session-xyz-789',
+			{ readOnlyMode: undefined }
 		);
 
 		const output = JSON.parse(consoleSpy.mock.calls[0][0]);
@@ -153,7 +155,8 @@ describe('send command', () => {
 			'claude-code',
 			'/custom/project/path',
 			'Do something',
-			undefined
+			undefined,
+			{ readOnlyMode: undefined }
 		);
 	});
 
@@ -173,7 +176,30 @@ describe('send command', () => {
 
 		expect(detectCodex).toHaveBeenCalled();
 		expect(detectClaude).not.toHaveBeenCalled();
-		expect(spawnAgent).toHaveBeenCalledWith('codex', expect.any(String), 'Use codex', undefined);
+		expect(spawnAgent).toHaveBeenCalledWith('codex', expect.any(String), 'Use codex', undefined, {
+			readOnlyMode: undefined,
+		});
+	});
+
+	it('should pass readOnlyMode when --read-only flag is set', async () => {
+		vi.mocked(resolveAgentId).mockReturnValue('agent-abc-123');
+		vi.mocked(getSessionById).mockReturnValue(mockAgent());
+		vi.mocked(detectClaude).mockResolvedValue({ available: true, path: '/usr/bin/claude' });
+		vi.mocked(spawnAgent).mockResolvedValue({
+			success: true,
+			response: 'Read-only response',
+			agentSessionId: 'session-ro',
+		});
+
+		await send('agent-abc', 'Analyze this code', { readOnly: true });
+
+		expect(spawnAgent).toHaveBeenCalledWith(
+			'claude-code',
+			'/path/to/project',
+			'Analyze this code',
+			undefined,
+			{ readOnlyMode: true }
+		);
 	});
 
 	it('should exit with error when agent ID is not found', async () => {

src/__tests__/cli/services/agent-spawner.test.ts

@@ -1075,6 +1075,42 @@ Some text with [x] in it that's not a checkbox
 			}
 		});
 
+		it('should include read-only args for Claude when readOnlyMode is true', async () => {
+			const resultPromise = spawnAgent('claude-code', '/project', 'prompt', undefined, {
+				readOnlyMode: true,
+			});
+
+			await new Promise((resolve) => setTimeout(resolve, 0));
+
+			const [, args] = mockSpawn.mock.calls[0];
+			// Should include Claude's read-only args from centralized definitions
+			expect(args).toContain('--permission-mode');
+			expect(args).toContain('plan');
+			// Should still have base args
+			expect(args).toContain('--print');
+			expect(args).toContain('--dangerously-skip-permissions');
+
+			mockStdout.emit('data', Buffer.from('{"type":"result","result":"Done"}\n'));
+			mockChild.emit('close', 0);
+			await resultPromise;
+		});
+
+		it('should not include read-only args when readOnlyMode is false', async () => {
+			const resultPromise = spawnAgent('claude-code', '/project', 'prompt', undefined, {
+				readOnlyMode: false,
+			});
+
+			await new Promise((resolve) => setTimeout(resolve, 0));
+
+			const [, args] = mockSpawn.mock.calls[0];
+			expect(args).not.toContain('--permission-mode');
+			expect(args).not.toContain('plan');
+
+			mockStdout.emit('data', Buffer.from('{"type":"result","result":"Done"}\n'));
+			mockChild.emit('close', 0);
+			await resultPromise;
+		});
+
 		it('should generate unique session-id for each spawn', async () => {
 			// First spawn
 			const promise1 = spawnAgent('claude-code', '/project', 'prompt1');

src/__tests__/renderer/components/QuickActionsModal.test.tsx

@@ -1615,4 +1615,79 @@ describe('QuickActionsModal', () => {
 			expect(screen.queryByText('Context: Send to Agent')).not.toBeInTheDocument();
 		});
 	});
+
+	describe('Create Worktree action', () => {
+		it('shows Create Worktree action for git repo sessions with callback', () => {
+			const onQuickCreateWorktree = vi.fn();
+			const props = createDefaultProps({
+				sessions: [createMockSession({ isGitRepo: true })],
+				onQuickCreateWorktree,
+			});
+			render(<QuickActionsModal {...props} />);
+
+			expect(screen.getByText('Create Worktree')).toBeInTheDocument();
+		});
+
+		it('calls onQuickCreateWorktree with active session and closes modal', () => {
+			const onQuickCreateWorktree = vi.fn();
+			const session = createMockSession({ isGitRepo: true });
+			const props = createDefaultProps({
+				sessions: [session],
+				onQuickCreateWorktree,
+			});
+			render(<QuickActionsModal {...props} />);
+
+			fireEvent.click(screen.getByText('Create Worktree'));
+
+			expect(onQuickCreateWorktree).toHaveBeenCalledWith(session);
+			expect(props.setQuickActionOpen).toHaveBeenCalledWith(false);
+		});
+
+		it('resolves to parent session when active session is a worktree child', () => {
+			const onQuickCreateWorktree = vi.fn();
+			const parentSession = createMockSession({
+				id: 'parent-1',
+				name: 'Parent',
+				isGitRepo: true,
+			});
+			const childSession = createMockSession({
+				id: 'child-1',
+				name: 'Child',
+				isGitRepo: true,
+				parentSessionId: 'parent-1',
+				worktreeBranch: 'feature-1',
+			});
+			const props = createDefaultProps({
+				sessions: [parentSession, childSession],
+				activeSessionId: 'child-1',
+				onQuickCreateWorktree,
+			});
+			render(<QuickActionsModal {...props} />);
+
+			fireEvent.click(screen.getByText('Create Worktree'));
+
+			// Should resolve to parent, not the child
+			expect(onQuickCreateWorktree).toHaveBeenCalledWith(parentSession);
+		});
+
+		it('does not show Create Worktree when session is not a git repo', () => {
+			const onQuickCreateWorktree = vi.fn();
+			const props = createDefaultProps({
+				sessions: [createMockSession({ isGitRepo: false })],
+				onQuickCreateWorktree,
+			});
+			render(<QuickActionsModal {...props} />);
+
+			expect(screen.queryByText('Create Worktree')).not.toBeInTheDocument();
+		});
+
+		it('does not show Create Worktree when callback is not provided', () => {
+			const props = createDefaultProps({
+				sessions: [createMockSession({ isGitRepo: true })],
+			});
+			render(<QuickActionsModal {...props} />);
+
+			expect(screen.queryByText('Create Worktree')).not.toBeInTheDocument();
+		});
+	});
 });

src/__tests__/renderer/hooks/useFileExplorerEffects.test.ts

@@ -423,6 +423,7 @@ describe('useFileExplorerEffects', () => {
 			renderHook(() => useFileExplorerEffects(deps));
 
 			expect(flattenTree).toHaveBeenCalledWith(tree, new Set(['src']));
+			expect(useFileExplorerStore.getState().filteredFileTree).toEqual(tree);
 			expect(useFileExplorerStore.getState().flatFileList).toEqual(flatResult);
 		});
 
@@ -437,6 +438,7 @@ describe('useFileExplorerEffects', () => {
 			const deps = createDeps();
 			renderHook(() => useFileExplorerEffects(deps));
 
+			expect(useFileExplorerStore.getState().filteredFileTree).toEqual([]);
 			expect(useFileExplorerStore.getState().flatFileList).toEqual([]);
 		});