feat(assistant): add error message display and enhance LLM configuration UI#261
Merged
feat(assistant): add error message display and enhance LLM configuration UI#261
Conversation
- Add isBuiltIn boolean field to Tool entity to distinguish built-in tools - Add proper decorators and validation to command field in Tool entity - Improve Puppeteer executable path detection with fallback to common Chrome/Chromium locations - Add existsSync check for browser executable validation - Standardize quote style to double quotes in screenshot handler - Update worker lockfile configuration These changes enable better tool categorization and more robust browser automation across different environments by auto-detecting Chrome/Chromium installations.
…ion UI - Add error message rendering in chat with AlertCircle icon - Display API URL in LLM manager components when available - Update Tool type with command and isBuiltIn fields - Improve error handling in assistant streaming hook - Add operationId support to API documentation decorators - Change API key input to text type for visibility
1 task
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
- Implement frontend components for chart rendering, Handlebars templating, report columns, and pages for listing, editing, and viewing reports - Add backend entities, DTOs, services, controllers, and PDF generation for security reports - Include Handlebars templates for executive, technical, developer, and infrastructure report types - Integrate with existing vulnerability data for report content population
Add comprehensive security reports functionality including UI components for listing, editing, and viewing reports, API endpoints for CRUD operations and PDF generation, routing for report pages, and backend module integration. Includes navigation updates to edit mode and type definitions for report content structures.
- Add new MCP tools: detail_asset, list_assets_in_target, detail_vuln, list_issues, detail_issue, list_tools, list_workers, job_manager - Introduce security_analyst prompt for expert security analysis persona - Extend MCP schemas with detailed parameter and output definitions - Update jobs registry service to support filtering by job status and workspace - Enhance existing tools with improved descriptions and metadata - Add new environment variables for assistant configuration - Update .gitignore with additional exclusions for searxng and .claude directories - Import new modules (Issues, Tools, Workers, JobsRegistry) into MCP module for expanded functionality
- Updated MCP prompt to include additional capabilities like listing tools and services - Enhanced Zod schemas with detailed descriptions for better API usability - Improved MCP tools with enriched output schemas, worker online status logic, and job management details - Reformatted statistic entity and refined tools service for consistency and accuracy
Refine the script skipping logic to use a Set of exact hostnames ('tailwindcss.com' and 'cdn.tailwindcss.com') instead of a suffix check, ensuring more precise filtering to avoid breaking root app borders.
Member
Author
|
/gemini review |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
l1ttps
requested changes
Jan 31, 2026
Member
l1ttps
left a comment
There was a problem hiding this comment.
Hey @ntthanh2603, thanks for this comprehensive PR!
This looks great overall - the new report generation system is a fantastic addition with the different templates (developer, executive, infrastructure, technical) and the enhanced LLM configuration UI is much improved. The error message display in the chat assistant is also a nice touch for better user experience.
I found a small typo that needs to be fixed: in console/src/components/common/page.tsx line 26, "w-ful" should be "w-full" in the className. This would cause a styling issue.
Other than that, the changes look solid with good error handling improvements and new security report capabilities. Great work on adding this substantial new functionality!
…cker compose - Add new assistant service with postgres, redis, and API integrations - Include searxng search engine service for enhanced search capabilities - Configure environment variables for assistant host, ports, and API URLs - Set up volumes for assistant logs, knowledge base, and searxng data - Update example.env with assistant configuration and MCP settings
Add comprehensive searxng settings.yml configuration file with default settings for search engines, plugins, UI configuration, and server settings. Also update .gitignore to properly exclude searxng directory.
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
4 participants
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.
No description provided.