Codex/feat saved prior sql shortcut#22
Merged
Merged
Conversation
…gent flow - Introduced the ResolveSavedPriorSqlNode to evaluate and reuse saved SQL queries. - Updated agent step metadata to include the new node in the analysis stage. - Enhanced LangGraph runtime to incorporate the new node, allowing for conditional execution based on prior SQL resolution. - Modified state management to track saved prior SQL resolutions and fallback scenarios. - Updated delivery contract to capture saved prior SQL evidence for improved tracking and diagnostics. - Implemented service for capturing and managing saved prior SQL records, enhancing overall SQL handling in the conversation module.
- Updated the FormatAnswerNode to provide a more detailed analysis summary, including metrics and coverage information. - Introduced new methods for building and normalizing delivery artifacts, improving the handling of chart and validation data. - Enhanced the ChatDeliveryEnrichmentService to integrate ChartBI artifacts and manage risk tags effectively. - Modified the DeliveryContractMapper to support artifact overrides and additional risk tags, ensuring better delivery semantics. - Improved the SQL inline panel and assistant message components to display artifact information, enhancing user experience. - Added comprehensive tests to validate the new delivery contract features and ensure compatibility with existing flows.
- Added support for specifying allowed directories for SQLite file paths via the `SQLITE_ALLOWED_DIRS` environment variable. - Refactored `AppConfigService` to include a method for resolving configured paths, improving path handling for SQLite and datasource uploads. - Updated `SqliteExecutorService` to utilize the new path resolution logic, ensuring correct execution of SQL queries based on datasource configuration. - Enhanced `DatasourceService` to validate SQLite paths against allowed directories, preventing unauthorized access to filesystem locations. - Improved frontend wizard for SQLite datasource creation, enforcing absolute path requirements and providing user feedback on path validity. - Added integration tests to validate the new SQLite handling features and ensure compliance with allowed directory restrictions.
- Added `LLM_STREAM_TIMEOUT_MS` to the configuration, allowing for customizable stream timeout settings. - Updated `AppConfigService` to include a method for retrieving the new stream timeout value with fallback logic. - Enabled `AGENT_PLANNING_SCAFFOLD_ENABLED` by default to improve agent planning capabilities. - Enhanced `ChatController` and related services to support the new actor parameter for better context handling in message processing. - Improved regex patterns in semantic evaluators to recognize count-style phrasing and customer synonyms, enhancing slot-filling accuracy. - Updated frontend components to reflect new design elements and improve user experience in chat interactions. - Added unit tests to validate the new features and ensure robust functionality across the application.
- Updated `SqlGenerationService` to include handling for the new error code `LLM_TOOL_CALL_EXECUTION_FAILED`, improving error management during tool execution. - Enhanced `SqlPromptBuilder` with additional semantic guardrails to prevent querying schema system tables unless explicitly requested by the user, ensuring stricter adherence to user intent. - Added unit tests to validate the new fallback behavior in SQL generation when tool execution fails, ensuring robustness in the conversation agent's functionality.
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
Frontend Checklist
style={{...}}）Validation
pnpm --filter @text2sql/frontend lintpnpm --filter @text2sql/frontend testpnpm --filter @text2sql/frontend buildPost-Deploy Monitoring & Validation