Conversation
- Send ask_user.rejected event to UI when server-side timeout occurs - Ensures UI moves timed-out questions to failed notifications and closes wizard - Fix unused callback parameters in IPC bridge for lint compliance
- Added `askUserTimeout` to `Preferences` interface and schema (default: 5 minutes) - Added Timeout Settings section to Advanced Settings modal - Updated `mcp-bridge` to sync timeout configuration with the main process - Refactored `mcp-bridge` retry logic to prevent stuck wizard on timeout - Updated `ipc` to handle dynamic timeout configuration for `ask_user` requests
- Remove ensureSingleListener enforcement to allow per-instance listeners - Restore correct event routing for multiple open workspaces
- Implement expand/collapse functionality for failed notification cards. - Add detailed views for 'question' and 'permission' notification types. - Display patterns and resources for permission requests. - Display full question text and options for questions. - Update CSS for expandable sections and internal elements.
- Redesign failed question panel with Markdown support and flexible sizing - Fix Electron IPC double registration error by adding cleanup logic - Fix CLI port mismatch by using correct dev server URL - Fix MCP listener race condition in preload script
…re and adjust table column formatting.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR significantly enhances the stability and user experience of CodeNomad. It introduces a configurable timeout for
ask_user, redesigns the currently rigid "Failed Notification" panel to support Markdown and flexible sizing, and includes critical bug fixes for Electron IPC, CLI port management, and MCP multi-instance stability.📦 Release Contents
1. ⏱️ Configurable Ask User Timeout
Key Changes:
askUserTimeoutpreference (default: 300s / 5m).ipc.tsandmcp-bridge.tsto dynamically sync the timeout value from the UI renderer to the main process/MCP server.Commits:
feat(ui/settings): implement configurable ask_user tool timeoutdocs: add configurable timeout instructions to README2. 🎨 Failed Notification Panel Redesign
Key Changes:
fit-content(up to the window width) to accommodate long lines or code blocks without aggressive wrapping.Commits:
feat(ui,electron): redesign notification panel and fix IPC/CLI issuesfeat(ui): add expandable detail view to failed notification panel3. 🛠️ Critical Stability & Bug Fixes
Key Changes:
setupCliIPC.localhost:3000, causing failures when the renderer port shifted (e.g., to 3001). Now dynamically readsVITE_DEV_SERVER_URL.ask_userlisteners for all instances. Now uses specific listener removal.Commits:
feat(ui,electron): redesign notification panel and fix IPC/CLI issuesfix(ui): resolve ask_user routing regression in multi-instancefix(mcp-server): notify UI on ask_user timeout to prevent stuck wizard📋 Checklist