Skip to content

Conversation

epipav
Copy link
Collaborator

@epipav epipav commented Sep 8, 2025

No description provided.

@Copilot Copilot AI review requested due to automatic review settings September 8, 2025 10:41
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds conversation management functionality to the data copilot and enables text-to-SQL functionality. The key changes include adding conversation tracking support to maintain context across multiple queries and implementing the previously commented-out text-to-SQL agent functionality.

  • Add conversationId field to chat responses and conversation tracking across the application
  • Enable text-to-SQL agent functionality for database query generation
  • Replace routerReasoning parameter with conversationId for improved chart generation context

Reviewed Changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated no comments.

Show a summary per file
File Description
frontend/server/repo/chat.repo.ts Add conversationId field to interface and repository methods
frontend/server/api/chat/stream.ts Generate and pass conversationId through chat streaming
frontend/server/api/chat/chart.ts Replace routerReasoning with conversation-based context retrieval
frontend/lib/chat/data-copilot.ts Enable text-to-SQL agent and add conversationId support
frontend/lib/chat/chart/generator.ts Update chart prompt to use routerReasoning from conversation
frontend/lib/chat/agents/base-agent.ts Remove unused blank line
frontend/app/components/shared/modules/copilot/types/copilot.types.ts Add conversationId types and update message part types
frontend/app/components/shared/modules/copilot/store/copilot.api.service.ts Update API service for conversationId handling
frontend/app/components/shared/modules/copilot/components/results/results-section.vue Replace routerReasoning with conversationId prop
frontend/app/components/shared/modules/copilot/components/results/chart-results.vue Update chart generation to use conversationId
frontend/app/components/shared/modules/copilot/components/copilot-sidebar.vue Implement conversation state management
frontend/app/components/shared/modules/copilot/components/copilot-modal.vue Update data handling for conversationId
database/migrations/V1756973486__addConversationIdToChatResponses.sql Add database migration for conversation tracking

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@epipav epipav requested a review from joanagmaia September 11, 2025 08:18
Copy link
Collaborator

@joanagmaia joanagmaia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍
And the docs on the chat lib for the functions look great. Thanks for adding that

epipav and others added 19 commits September 11, 2025 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants