Update columns_view description to clarify it includes up to six most important columns#1635
Conversation
… important columns
There was a problem hiding this comment.
Pull request overview
Updates the AI table-settings generation prompt to clarify that columns_view should contain only a small set of the most important columns, rather than every column.
Changes:
- Refined the
columns_viewdescription to “Up to six most important columns in preferred display order”.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| 2. search_fields: Columns that should be searchable (text fields like name, email, title) | ||
| 3. readonly_fields: Columns that should not be editable (like auto_increment, timestamps) | ||
| 4. columns_view: All columns in preferred display order | ||
| 4. columns_view: Up to six most important columns in preferred display order |
There was a problem hiding this comment.
This prompt text now says columns_view should be limited ("Up to six most important columns"), but there is another table-settings prompt in backend/src/ai-core/tools/prompts.ts that still states "All columns". Keeping two divergent prompt definitions for the same concept is likely to cause inconsistent AI outputs depending on which prompt is used; consider updating the other prompt as well or consolidating to a single shared prompt builder to avoid future drift.
| 4. columns_view: Up to six most important columns in preferred display order | |
| 4. columns_view: All columns in preferred display order |
No description provided.