Skip to content

Simplify AI panel query correction prompts to respond with SQL only#1634

Merged
Artuomka merged 1 commit into
mainfrom
backend-simplify-ai-panel-correction-prompt
Feb 25, 2026
Merged

Simplify AI panel query correction prompts to respond with SQL only#1634
Artuomka merged 1 commit into
mainfrom
backend-simplify-ai-panel-correction-prompt

Conversation

@gugu

@gugu gugu commented Feb 25, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 simplifies AI prompt instructions for SQL query correction in two use cases. The changes make the prompts more concise while maintaining all essential requirements, and reorganize imports for better readability.

Changes:

  • Simplified query correction prompt instructions from verbose multi-line format to concise single-line format
  • Added "if possible" qualifier to optimization instructions to make them less demanding
  • Reorganized import statements in both files for better grouping

Reviewed changes

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

File Description
generate-table-dashboard-with-ai.use.case.ts Reorganized imports and simplified query correction prompt from 4-line IMPORTANT section to concise single-line instruction
generate-panel-position-with-ai.use.case.ts Reorganized imports and simplified query correction prompt consistently with the first file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Artuomka Artuomka enabled auto-merge February 25, 2026 19:31
@Artuomka Artuomka merged commit ec10fc4 into main Feb 25, 2026
21 of 23 checks passed
@Artuomka Artuomka deleted the backend-simplify-ai-panel-correction-prompt branch February 25, 2026 19:38
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