Skip to content

fix: add missing descriptions to tool parameter schema properties#9087

Merged
tomivirkki merged 2 commits intomainfrom
fix/missing-chart-ai-tool-descriptions
Apr 10, 2026
Merged

fix: add missing descriptions to tool parameter schema properties#9087
tomivirkki merged 2 commits intomainfrom
fix/missing-chart-ai-tool-descriptions

Conversation

@tomivirkki
Copy link
Copy Markdown
Member

Summary

  • Add descriptions to all tool parameter schema properties in ChartAITools that were previously missing them, improving the quality of LLM tool-calling by providing clear semantics for each property

🤖 Generated with Claude Code

@tomivirkki tomivirkki marked this pull request as draft April 9, 2026 14:21
@tomivirkki tomivirkki force-pushed the fix/missing-chart-ai-tool-descriptions branch from e9ca4b8 to 2d83107 Compare April 9, 2026 14:33
Base automatically changed from feat/chart-ai-controller-per-series-configuration to main April 9, 2026 15:17
@tomivirkki tomivirkki force-pushed the fix/missing-chart-ai-tool-descriptions branch from 2d83107 to 7eba482 Compare April 9, 2026 16:31
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tomivirkki tomivirkki force-pushed the fix/missing-chart-ai-tool-descriptions branch from 7eba482 to 0ae4ffa Compare April 10, 2026 06:43
@tomivirkki tomivirkki marked this pull request as ready for review April 10, 2026 06:44
@tomivirkki tomivirkki requested a review from ugur-vaadin April 10, 2026 06:44
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
12.3% Duplication on New Code (required ≤ 10%)

See analysis details on SonarQube Cloud

@tomivirkki tomivirkki added this pull request to the merge queue Apr 10, 2026
Merged via the queue into main with commit 2e911ee Apr 10, 2026
8 of 9 checks passed
@tomivirkki tomivirkki deleted the fix/missing-chart-ai-tool-descriptions branch April 10, 2026 09:32
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.

2 participants