Resolves issue #443 where meta+enter keyboard shortcut only worked from
the prompt textarea. Now works consistently from title, directory, and
prompt fields by moving keyboard handling to the modal level.
- Add global meta+enter handler at SessionLauncher modal level
- Remove input-specific keyboard handler from CommandInput textarea
- Update visual hints to show shortcut on Launch button
- Clean up footer hints for consistency
When continuing a session with HTTP MCP servers, the X-Session-ID header
must be updated to reflect the new child session ID rather than inheriting
the parent's session ID. This ensures proper session tracking across
continued sessions.
Rename the method for better clarity and move correlation logic to execute
immediately after approval creation for all approval types, not just auto-approved
- Update OpenAPI spec to support both stdio and HTTP MCP server configurations
- Add type, url, and headers fields for HTTP servers alongside existing stdio fields
- Update mapper to handle optional fields for both server types
- Remove 5-minute timeout from MCP approval handler for better session resumption
- Configure WUI to use HTTP MCP server endpoint instead of stdio npx command
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
Enable the 'i' key to close the ToolResultModal for a true toggle experience.
This allows users to navigate sessions with one hand - j/k for scrolling
and 'i' to toggle the modal open/closed, without reaching for ESC.
- Add 'i' hotkey handler to close modal when open
- Update footer hint to show "i or ESC to close"
- Replace Unix socket RPC calls with REST API calls to /api/v1/sessions
- Remove unused imports (net, bufio, rpc, claudecode)
- Use standard HTTP client for session creation requests
- Maintain same test behavior and assertions
Add comprehensive integration test that verifies tool_use_id is properly
set in the approvals table when Claude Code triggers approvals through
the MCP HTTP endpoint. The test:
- Launches Claude sessions via daemon RPC with HTTP MCP configuration
- Verifies approvals are created with tool_use_id populated
- Tests both single and parallel approval scenarios
- Confirms tool_use_id enables direct correlation without race conditions
- Keep query persistence feature from upstream
- Preserve title field functionality from feature branch
- Remove redundant query field from config object
NB - SOME TEST IS STALLING OUT RIGHT NOW! Needs a timeout/failure or something.
- Add console.debug logging in WUI to track tool_use_id in approval events
- Update CLAUDE.md documentation for test commands
- Extend claudecode-go types to support HTTP MCP servers with headers
- Update launch command to use HTTP MCP endpoint at /api/v1/mcp
- Remove stdio MCP server implementation from hlyr
- Clean up index.ts to remove MCP server initialization
- Add event-driven approval resolution to MCP server
- Implement automatic X-Session-ID header injection for HTTP MCP servers
- Update store to handle HTTP MCP configuration alongside stdio
- Remove MCP stub handler as full implementation is complete
- Add golang/mock dependency for test mocks
- Update integration tests to account for MCP event listener
- Use updateSession instead of updateSessionOptimistic in Layout when receiving server data
- Fix stale closure issue in meta+y hotkey by reading current state directly from store
- Ensures bypass permissions toggle always uses the most current session state
Prevents loss of draft prompts when users accidentally close the launcher
by saving the query text to localStorage on every keystroke. The draft
is restored when the launcher reopens and cleared only after successful
session launch.
- Save draft to localStorage on every query change
- Restore draft when launcher reopens (via 'c' shortcut or Cmd+K)
- Clear draft only after successful session creation
- Preserve draft on errors for retry attempts
- Follow existing ResponseInput localStorage pattern for consistency
Enables quick submission of the bypass permissions dialog using meta+Enter (Cmd+Enter on Mac, Ctrl+Enter on Windows/Linux). The shortcut respects validation rules and only submits when the form is valid. Visual indicator shows the keyboard shortcut next to the submit button when enabled.
- Test Claude's transmission of custom headers including X-Session-ID
- Verify approval creation with tool_use_id in Phase 4 implementation
- Test auto-approval behavior with dangerously_skip_permissions flag
- Confirm session correlation works correctly via HTTP headers
- Add CreateApprovalWithToolUseID method to approval manager
- Extract session ID from X-Session-ID header in MCP requests
- Handle auto-approval for dangerously_skip_permissions flag
- Fix approval creation to include comment field in SQLite store
- Wire up MCP server with approval manager in daemon HTTP server
- Kept debugLogSnapshotFailure function and logger import uncommented
- Function invocation remains commented in catch block
- Added proper ESLint and TypeScript ignore comments for unused function
- Cleaner approach that keeps debug code readily available
The Claude API returns permission_denials as an array of objects with
tool_name, tool_use_id, and tool_input fields, but the SDK expected
a simple string array. This type mismatch caused JSON unmarshaling
failures that incorrectly marked successful sessions as FAILED.
- Add PermissionDenial struct and PermissionDenials wrapper type
- Implement custom UnmarshalJSON to handle both object and string formats
- Update StreamEvent and Result structs to use the new type
- Add missing PermissionDenials field copy in Result builder
- Add comprehensive tests for all permission denial formats
Fixes sessions being incorrectly marked as FAILED when they contain
permission denials but complete successfully (e.g., session C5E8CF80
which created PR #430 successfully but showed as failed).
- Commented out entire debug function as block comment
- Logger import also commented to avoid unused import
- Clean approach that preserves debug code for future use
- All checks now pass (lint, typecheck, format)
- Created debugLogSnapshotFailure helper function with original logging logic
- Commented out just the function invocation instead of entire code block
- Kept logger import active for potential future use
- Cleaner approach that preserves debug capability without code clutter
- Hide 'Brainrot Mode' until user types 'brain' in search
- Show context-aware 'Archive'/'Unarchive' label based on session state
- Improves UI clarity and reduces clutter in command palette
Replace stub MCP handler with full implementation using mark3labs/mcp-go v0.37.0
library. The new server provides complete MCP protocol support with proper
JSON-RPC handling, tool schema generation with annotations, and session ID
extraction from headers.
Key changes:
- Add mark3labs/mcp-go dependency for MCP protocol implementation
- Create MCPServer with request_approval tool and auto-deny mode
- Update daemon to use new MCP server instead of stub
- Add proper context key type to satisfy linter requirements
- Extract session IDs from X-Session-ID and Mcp-Session-Id headers
- Add comprehensive integration tests for schema validation and edge cases
The server is configured as stateless for Phase 3, with approval manager
integration coming in Phase 4. Auto-deny mode (MCP_AUTO_DENY_ALL=true)
enables predictable testing without requiring approval infrastructure.
Tested with curl commands and integration test suite (40 tests passing).
- Rename TestMigration12_ToolUseID to TestMigration14_ToolUseID to match actual migration number
- Fix defer Close() to ignore error per linter recommendation
