Merge pull request #64 from mcintyre94/side-chat-feature-for-wisp-05453dfb

Add Quick Actions feature (Quick Chat + Bash tabs)

Author: mcintyre94

.claude/worktrees/pr-33-worktree-per-chat

@@ -0,0 +1 @@
+Subproject commit 7386251ed81d42d9ec2ba9a042308e2973efbc30

.claude/worktrees/remove-chevrons

@@ -0,0 +1 @@
+Subproject commit 239e7199768530a4490c8f74dfad0ec37910e7e4

QUICK_ACTIONS_PLAN.md

@@ -0,0 +1,120 @@
+# Quick Actions Feature Plan
+
+## Overview
+
+Quick Actions is a lightweight interaction layer that sits alongside the main chat, grouping two features: **Side Chat** (ask Claude a question) and **Bash** (run a shell command). Both are ephemeral, single-shot, and don't interrupt the main chat session.
+
+Replaces and supersedes the current side chat implementation on this branch.
+
+---
+
+## Entry Points
+
+### 1. Sprite-level (context menu)
+- Long-press / right-click on a Sprite in the dashboard or detail view
+- No chat context — just the Sprite and its default working directory (`/home/sprite/project`)
+- Service prefix: `wisp-quick-{UUID}` (separate from main chat services)
+
+### 2. Chat-level (toolbar button)
+- Toolbar button in the Chat view navigation bar
+- Has full chat context: session ID, working directory (may be a worktree path)
+- Same service prefix: `wisp-quick-{UUID}`
+
+---
+
+## Tabs
+
+### Side Chat
+
+**From Sprite (no session):**
+```
+claude -p --output-format stream-json \
+  --disallowedTools "Bash,Write,Edit,MultiEdit,WebSearch,WebFetch" \
+  --model MODEL \
+  --max-turns 3 \
+  'QUESTION'
+```
+- Read, Glob, Grep tools available — useful for asking about files without a full chat
+- Fresh session, no prior context
+
+**From Chat (with session):**
+- Same command, adds `--resume SESSION_ID`
+- Side chat Q&A is appended to main session history — benign, doesn't affect the current stream, and gives Claude useful context on future resumes
+
+**Why `--disallowedTools` instead of `--tools ""`:**
+Allowing read-only tools (Read, Glob, Grep) makes the feature meaningfully more useful, especially from the Sprite entry point where there's no prior session context. `--max-turns 3` bounds latency.
+
+**UI:**
+- Streaming markdown response (`.wisp` theme)
+- `ThinkingShimmerView` while waiting for first tokens
+- Single input bar, keyboard auto-focused on open
+- Multiple questions supported per sheet session (each resumes same session ID)
+
+---
+
+### Bash
+
+**From Sprite and Chat:**
+- Runs command via exec on the Sprite
+- Shows stdout + stderr in a terminal-styled output area (monospace, dark background)
+
+**From Chat only:**
+- "Insert into chat" button appears after command completes
+- Formats output as ` $ command\n{output} ` in a code block, prepends to chat input field, dismisses sheet
+- User adds context and sends normally
+
+**Keyboard:**
+- `.keyboardType(.asciiCapable)` — no emoji, cleaner for shell input
+- `.autocorrectionDisabled()` + `.textInputAutocapitalization(.never)`
+- Custom keyboard accessory bar with commonly painful-to-type characters: `` / - | > ~ ` $ & * . ``
+
+**UI:**
+- Monospace font on both input and output
+- Dark background output area (terminal aesthetic)
+- Streaming output as the command runs (if using service exec)
+
+---
+
+## Architecture
+
+### New files
+```
+Wisp/
+├── ViewModels/
+│   ├── QuickActionsViewModel.swift   # Owns context (sprite, session, workingDir), coordinates tabs
+│   └── BashQuickViewModel.swift      # Exec, output collection, "insert into chat" callback
+└── Views/
+    └── QuickActions/
+        ├── QuickActionsView.swift    # Tab container sheet (Side Chat | Bash)
+        ├── SideChatView.swift        # Refactored from current SideChatView
+        └── BashQuickView.swift       # Bash tab UI, keyboard accessory bar
+```
+
+### Modified files
+- `ChatView.swift` — swap `onSideChat` → `onQuickActions` toolbar button; present `QuickActionsView`
+- `ChatInputBar.swift` — remove `onSideChat`, add `onQuickActions` (or remove entirely — toolbar button may be sufficient)
+- `SpriteDetailView.swift` / dashboard — add Quick Actions to Sprite context menu
+- `SideChatViewModel.swift` — update `--tools ""` → `--disallowedTools ...`, add `--max-turns 3`
+
+### Removed
+- `ChatInputBar.onSideChat` (replaced by toolbar button)
+- Current `SideChatView` and its sheet wiring in `ChatView` (replaced by `QuickActionsView`)
+
+---
+
+## Service Cleanup
+- Both tabs use `wisp-quick-{UUID}` service names
+- Services deleted on completion or sheet dismiss (cancel in-flight tasks)
+- Main chat services (`wisp-claude-{UUID}`) are unaffected
+
+---
+
+## Open Questions
+
+1. **Bash exec mechanism** — `runExec()` (WebSocket, simpler, full output on completion) vs `streamService()` (streaming output as it runs). Streaming feels better UX-wise for longer commands; `runExec()` is simpler. Worth deciding before building.
+
+2. **Bash command history** — remember recent commands within a session? Nice-to-have, skip for MVP.
+
+3. **Tab default** — should the sheet remember which tab was last open, or always default to Side Chat?
+
+4. **Side chat from Sprite with no Claude token** — show an error state or hide the side chat tab entirely?

Wisp.xcodeproj/project.pbxproj

@@ -38,6 +38,14 @@
 		A21BF0D8D910819DD5591E88 /* ExecSession.swift in Sources */ = {isa = PBXBuildFile; fileRef = 2554BB4DE8EC315CA48767BF /* ExecSession.swift */; };
 		A610B5413A578D4AA424595D /* ToolUseCardView.swift in Sources */ = {isa = PBXBuildFile; fileRef = 8A278B4716095105A7024CF8 /* ToolUseCardView.swift */; };
 		A92BE24BEAF80FF31B4ACA00 /* CheckpointsViewModel.swift in Sources */ = {isa = PBXBuildFile; fileRef = D0AD6D912964E32FC8685EA3 /* CheckpointsViewModel.swift */; };
+		AA10BB20CC30DD40EE50FF01 /* QuickChatViewModel.swift in Sources */ = {isa = PBXBuildFile; fileRef = AA10BB20CC30DD40EE50FF02 /* QuickChatViewModel.swift */; };
+		AA3100BB00CC00DD00EE0001 /* QuickActionsViewModel.swift in Sources */ = {isa = PBXBuildFile; fileRef = AA3100BB00CC00DD00EE0002 /* QuickActionsViewModel.swift */; };
+		AA3200BB00CC00DD00EE0001 /* BashQuickViewModel.swift in Sources */ = {isa = PBXBuildFile; fileRef = AA3200BB00CC00DD00EE0002 /* BashQuickViewModel.swift */; };
+		AA3300BB00CC00DD00EE0001 /* QuickActionsView.swift in Sources */ = {isa = PBXBuildFile; fileRef = AA3300BB00CC00DD00EE0002 /* QuickActionsView.swift */; };
+		AA3400BB00CC00DD00EE0001 /* QuickChatView.swift in Sources */ = {isa = PBXBuildFile; fileRef = AA3400BB00CC00DD00EE0002 /* QuickChatView.swift */; };
+		AA3500BB00CC00DD00EE0001 /* BashQuickView.swift in Sources */ = {isa = PBXBuildFile; fileRef = AA3500BB00CC00DD00EE0002 /* BashQuickView.swift */; };
+		AA3700BB00CC00DD00EE0001 /* BashQuickViewModelTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = AA3700BB00CC00DD00EE0002 /* BashQuickViewModelTests.swift */; };
+		FE9876543210ABCDEF987601 /* QuickChatViewModelTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = FE9876543210ABCDEF987602 /* QuickChatViewModelTests.swift */; };
 		AA00BB11CC22DD3300000001 /* WispAskCard.swift in Sources */ = {isa = PBXBuildFile; fileRef = AA00BB11CC22DD3300000002 /* WispAskCard.swift */; };
 		AA11BB22CC33DD44EE55FF02 /* GitHubDeviceFlowClient.swift in Sources */ = {isa = PBXBuildFile; fileRef = AA11BB22CC33DD44EE55FF01 /* GitHubDeviceFlowClient.swift */; };
 		AA11BB22CC33DD44EE55FF04 /* GitHubDeviceFlowView.swift in Sources */ = {isa = PBXBuildFile; fileRef = AA11BB22CC33DD44EE55FF03 /* GitHubDeviceFlowView.swift */; };
@@ -234,6 +242,14 @@
 		F7A8B9C0D1E2F30415263749 /* PlanCardView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = PlanCardView.swift; sourceTree = "<group>"; };
 		FF11AA22BB33CC44DD55EE01 /* ChatViewModelHelpersTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ChatViewModelHelpersTests.swift; sourceTree = "<group>"; };
 		FF11AA22BB33CC44DD55EE03 /* ChatViewModelTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ChatViewModelTests.swift; sourceTree = "<group>"; };
+		AA10BB20CC30DD40EE50FF02 /* QuickChatViewModel.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = QuickChatViewModel.swift; sourceTree = "<group>"; };
+		AA3100BB00CC00DD00EE0002 /* QuickActionsViewModel.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = QuickActionsViewModel.swift; sourceTree = "<group>"; };
+		AA3200BB00CC00DD00EE0002 /* BashQuickViewModel.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = BashQuickViewModel.swift; sourceTree = "<group>"; };
+		AA3300BB00CC00DD00EE0002 /* QuickActionsView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = QuickActionsView.swift; sourceTree = "<group>"; };
+		AA3400BB00CC00DD00EE0002 /* QuickChatView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = QuickChatView.swift; sourceTree = "<group>"; };
+		AA3500BB00CC00DD00EE0002 /* BashQuickView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = BashQuickView.swift; sourceTree = "<group>"; };
+		AA3700BB00CC00DD00EE0002 /* BashQuickViewModelTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = BashQuickViewModelTests.swift; sourceTree = "<group>"; };
+		FE9876543210ABCDEF987602 /* QuickChatViewModelTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = QuickChatViewModelTests.swift; sourceTree = "<group>"; };
 /* End PBXFileReference section */
 
 /* Begin PBXFrameworksBuildPhase section */
@@ -407,6 +423,9 @@
 				EF49F7E0F4AC02FDF496D379 /* DashboardViewModel.swift */,
 				C3D4E5F6A7B80920415C6D7E /* SpriteChatListViewModel.swift */,
 				AC9BFD0F0AC39C136C9E828A /* SpriteOverviewViewModel.swift */,
+				AA3200BB00CC00DD00EE0002 /* BashQuickViewModel.swift */,
+				AA10BB20CC30DD40EE50FF02 /* QuickChatViewModel.swift */,
+				AA3100BB00CC00DD00EE0002 /* QuickActionsViewModel.swift */,
 			);
 			path = ViewModels;
 			sourceTree = "<group>";
@@ -451,6 +470,8 @@
 				DD11EE22FF334455AA66BB03 /* ServiceTypesTests.swift */,
 				EE22FF334455AA66BB770001 /* ExecSessionURLTests.swift */,
 				CC11DD22EE33FF4400110005 /* FileEntryTests.swift */,
+				AA3700BB00CC00DD00EE0002 /* BashQuickViewModelTests.swift */,
+				FE9876543210ABCDEF987602 /* QuickChatViewModelTests.swift */,
 			);
 			path = WispTests;
 			sourceTree = "<group>";
@@ -463,11 +484,22 @@
 				902EF50E1CB6723AF748F886 /* Dashboard */,
 				A5E41B8F9DACBEC637F8192A /* Settings */,
 				90D05FE8D8DC0D59A74BAD85 /* SpriteDetail */,
+				AA3600BB00CC00DD00EE0001 /* QuickActions */,
 				A7B8C9D0E1F2A3B4C5D6E7F8 /* WebView */,
 			);
 			path = Views;
 			sourceTree = "<group>";
 		};
+		AA3600BB00CC00DD00EE0001 /* QuickActions */ = {
+			isa = PBXGroup;
+			children = (
+				AA3300BB00CC00DD00EE0002 /* QuickActionsView.swift */,
+				AA3400BB00CC00DD00EE0002 /* QuickChatView.swift */,
+				AA3500BB00CC00DD00EE0002 /* BashQuickView.swift */,
+			);
+			path = QuickActions;
+			sourceTree = "<group>";
+		};
 		B22C752D93810C377FF981DD /* Chat */ = {
 			isa = PBXGroup;
 			children = (
@@ -746,6 +778,12 @@
 				C4D5E6F7A8B90A41637E8FA2 /* CheckpointMarkerView.swift in Sources */,
 				E1F2A3B4C5D6E7F800000014 /* UnifiedDiffView.swift in Sources */,
 				C575C7232C5FCA1A3F18A9D8 /* WispApp.swift in Sources */,
+				AA3500BB00CC00DD00EE0001 /* BashQuickView.swift in Sources */,
+				AA3200BB00CC00DD00EE0001 /* BashQuickViewModel.swift in Sources */,
+				AA3300BB00CC00DD00EE0001 /* QuickActionsView.swift in Sources */,
+				AA3100BB00CC00DD00EE0001 /* QuickActionsViewModel.swift in Sources */,
+				AA3400BB00CC00DD00EE0001 /* QuickChatView.swift in Sources */,
+				AA10BB20CC30DD40EE50FF01 /* QuickChatViewModel.swift in Sources */,
 				E1F2A3B4C5D6E7F800000012 /* WispCodeHighlighter.swift in Sources */,
 				E1F2A3B4C5D6E7F800000013 /* WispMarkdownTheme.swift in Sources */,
 			);
@@ -774,6 +812,8 @@
 				E7F8A9B0C1D2E3F405A6B7C2 /* ChatNameTests.swift in Sources */,
 				0DFF41BB14AC3922E95DF781 /* WorktreeTests.swift in Sources */,
 				CC11DD22EE33FF4400110006 /* FileEntryTests.swift in Sources */,
+				AA3700BB00CC00DD00EE0001 /* BashQuickViewModelTests.swift in Sources */,
+				FE9876543210ABCDEF987601 /* QuickChatViewModelTests.swift in Sources */,
 			);
 			runOnlyForDeploymentPostprocessing = 0;
 		};

Wisp/Utilities/Extensions.swift

@@ -1,4 +1,15 @@
 import Foundation
+#if canImport(UIKit)
+import UIKit
+#endif
+
+var isRunningOnMac: Bool {
+    #if targetEnvironment(macCatalyst)
+    true
+    #else
+    ProcessInfo.processInfo.isiOSAppOnMac
+    #endif
+}
 
 extension JSONDecoder {
     static func apiDecoder() -> JSONDecoder {

Wisp/ViewModels/BashQuickViewModel.swift

@@ -0,0 +1,95 @@
+import Foundation
+import os
+
+private let logger = Logger(subsystem: "com.wisp.app", category: "BashQuick")
+
+@Observable
+@MainActor
+final class BashQuickViewModel {
+    let spriteName: String
+    let workingDirectory: String
+
+    var command = ""
+    private(set) var output = ""
+    private(set) var isRunning = false
+    private(set) var error: String?
+    private(set) var lastCommand = ""
+
+    private var streamTask: Task<Void, Never>?
+
+    init(spriteName: String, workingDirectory: String) {
+        self.spriteName = spriteName
+        self.workingDirectory = workingDirectory
+    }
+
+    func send(apiClient: SpritesAPIClient) {
+        let cmd = command.trimmingCharacters(in: .whitespacesAndNewlines)
+        guard !cmd.isEmpty, !isRunning else { return }
+
+        lastCommand = cmd
+        command = ""
+        output = ""
+        error = nil
+        isRunning = true
+
+        streamTask = Task {
+            await executeCommand(cmd, apiClient: apiClient)
+        }
+    }
+
+    func cancel(apiClient: SpritesAPIClient) {
+        streamTask?.cancel()
+        streamTask = nil
+        isRunning = false
+    }
+
+    func insertFormatted() -> String {
+        BashQuickViewModel.formatInsert(command: lastCommand, output: output)
+    }
+
+    static func formatInsert(command: String, output: String) -> String {
+        "```\n$ \(command)\n\(output.trimmingCharacters(in: .newlines))\n```"
+    }
+
+    // MARK: - Private
+
+    private func executeCommand(_ cmd: String, apiClient: SpritesAPIClient) async {
+        let fullCommand = "cd \(workingDirectory) 2>/dev/null || true; \(cmd)"
+        let serviceName = "wisp-quick-\(UUID().uuidString.prefix(8).lowercased())"
+        let config = ServiceRequest(cmd: "bash", args: ["-c", fullCommand], needs: nil, httpPort: nil)
+        let stream = apiClient.streamService(spriteName: spriteName, serviceName: serviceName, config: config)
+
+        do {
+            streamLoop: for try await event in stream {
+                guard !Task.isCancelled else { break streamLoop }
+                switch event.type {
+                case .stdout, .stderr:
+                    if let text = event.data {
+                        output += text
+                    }
+                case .error:
+                    if output.isEmpty {
+                        error = event.data ?? "Service error"
+                    }
+                case .complete:
+                    break streamLoop
+                default:
+                    break
+                }
+            }
+        } catch {
+            if !Task.isCancelled {
+                self.error = "Connection error"
+                logger.error("Bash quick stream error: \(error.localizedDescription)")
+            }
+        }
+
+        Task {
+            try? await apiClient.deleteService(spriteName: spriteName, serviceName: serviceName)
+        }
+
+        if !Task.isCancelled {
+            isRunning = false
+        }
+    }
+}

Wisp/ViewModels/ChatViewModel.swift

@@ -54,7 +54,7 @@ final class ChatViewModel {
     var isLoadingHistory = false
 
     private var serviceName: String
-    private var sessionId: String?
+    private(set) var sessionId: String?
     var workingDirectory: String
     private(set) var worktreePath: String?
     /// True when any chat for this sprite has had a worktree created, indicating the

Wisp/ViewModels/QuickActionsViewModel.swift

@@ -0,0 +1,28 @@
+import Foundation
+
+@Observable
+@MainActor
+final class QuickActionsViewModel: Identifiable {
+    let id = UUID()
+    let spriteName: String
+    let sessionId: String?
+    let workingDirectory: String
+
+    let quickChatViewModel: QuickChatViewModel
+    let bashViewModel: BashQuickViewModel
+
+    init(spriteName: String, sessionId: String?, workingDirectory: String) {
+        self.spriteName = spriteName
+        self.sessionId = sessionId
+        self.workingDirectory = workingDirectory
+        self.quickChatViewModel = QuickChatViewModel(
+            spriteName: spriteName,
+            sessionId: sessionId,
+            workingDirectory: workingDirectory
+        )
+        self.bashViewModel = BashQuickViewModel(
+            spriteName: spriteName,
+            workingDirectory: workingDirectory
+        )
+    }
+}