This change addresses potential grey/blank screen or crashing issues in the webview by:
1. **Adding an ErrorBoundary:** I introduced an `ErrorBoundary` component at the root of `webview-ui/src/App.tsx`. This component catches JavaScript rendering errors in its children, displaying a user-friendly fallback UI instead of a blank or broken screen. It also logs the error to the console for easier debugging.
2. **Implementing Loading/Timeout Logic:** I modified `webview-ui/src/App.tsx` to:
* Display a "Loading..." message while the initial extension state (`didHydrateState`) is being processed.
* Implement a 10-second timeout. If state hydration doesn't complete within this period, an error message is shown, guiding you to try reloading or checking for updates. This provides better feedback than a persistent blank screen.
3. **Adding Unit Tests:**
* I created tests for the `ErrorBoundary` to ensure it correctly catches errors and renders the fallback UI or child components as expected.
* I created tests for the `App.tsx` loading and timeout logic to verify that loading indicators and error messages are displayed appropriately under different hydration scenarios.
These changes aim to make the webview more resilient to rendering errors and provide better feedback to you during initialization, improving the overall user experience and aiding in diagnosing future issues.
* fix: Missing or inconsistent syntax highlighting across UI components
- Change file listings to use 'shellsession' for terminal-like highlighting
- Use 'markdown' for code definitions and instructions
- Add file extension-based language detection for new files
- Ensure consistent 'diff' highlighting for all diff content
- Use 'xml' language for error messages
- Make language property required in CodeAccordian
- Set default fallback to 'txt' instead of undefined
Fixes: #3655
Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org>
* chore: make language property required in CodeBlock
- Updated CodeBlockProps interface to make language property required
- Updated mock implementation to match the interface change
- Ensured CodeAccordian always provides a fallback language value
Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org>
---------
Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org>
Co-authored-by: Eric Wheeler <roo-code@z.ewheeler.org>
* [Condense] Add a button to condense the task context
* wip
* wip
* wip
* bring back delete size
* account for the system prompt in the context
* update tests to use systemPrompt
* add type
* translations
* nit
* update tests
* filter to the current task
* nit
* refactor
* nit
* non interactive option
* simplify chat summary UI
* changeset
* nit
* fix check-types
* throw
* feat(settings): Introduce the "auto-approve request count" feature from Cline
This is the first minor UI feature I've added, so please let me know if I'm missing anything! (translations, organization, etc!)
Please see commits for details
introduce allowedMaxRequests to globalSettingsSchema
update ExtensionState and its context with allowedMaxRequests
implement UI for setting max requests in AutoApproveMenu component
prompt user when auto-approval limit is reached with i18n support
increment consecutiveAutoApprovedRequestsCount and reset upon user approval
add translations for auto-approved request limit reached prompt in multiple languages
add new UI for "auto_approval_max_req_reached" in ChatRowContent
display prompt with title, description, and button for user action
🔧 chore(gitignore): add .idea to .gitignore to exclude IDE-specific files
- remove .idea/workspace.xml to clean up repository
* 🔧 chore(gitignore): add IDE configuration files to ignore list
- add .idea directory to ignore JetBrains IDE configurations
* 🌐 i18n(chat): add translation keys for api request limit
- introduce translation keys for "title" and "unlimited" in multiple languages
- update description for api request limit in various languages
* 🌐 i18n(chat): migrate auto-approved request limit translations
- move translations from common.json to chat.json across locales
- update component to use Trans for dynamic text rendering
* Update the UI for setting max requests
* Hide the auto-approve limit warning once clicked
---------
Co-authored-by: Matt Rubens <mrubens@users.noreply.github.com>
