Grids AI Assistant: Add API Descriptions Part 2#8767
Grids AI Assistant: Add API Descriptions Part 2#8767arman-boyakhchyan wants to merge 16 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
This PR extends the DevExtreme Grids AI Assistant documentation by adding/relocating API type descriptions (commands, filter expression objects, AI Assistant customization callbacks) and wiring the AI Assistant toolbar button into the shared grids toolbar include.
Changes:
- Added documentation pages for predefined grid commands/command names and filter-expression object shapes (Common Types + DataGrid-specific Types).
- Added detailed API descriptions and multi-framework examples for AI Assistant
customizeResponseTitle/customizeResponseText, plus relatedchat/popupconfiguration notes. - Removed obsolete placeholder “NewTopics” stubs after migrating content to the canonical API reference locations.
Reviewed changes
Copilot reviewed 36 out of 37 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
| includes/grids-toolbar.md | Adds aiAssistantButton to the predefined toolbar controls table. |
| api-reference/NewTopics/SummaryCommandTotalItem/SummaryCommandTotalItem.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/NewTopics/SummaryCommandGroupItem/SummaryCommandGroupItem.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/NewTopics/PredefinedCommands/PredefinedCommands.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/NewTopics/PredefinedCommandNames/PredefinedCommandNames.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/NewTopics/NegatedFilterExprObj/NegatedFilterExprObj.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/NewTopics/FilterExprObj/FilterExprObj.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/NewTopics/DataGridPredefinedCommands/DataGridPredefinedCommands.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/NewTopics/DataGridPredefinedCommandNames/DataGridPredefinedCommandNames.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/NewTopics/DataGridCommandInfo/DataGridCommandInfo.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/NewTopics/CompositeKeyPair/CompositeKeyPair.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/NewTopics/CommandInfo/CommandInfo.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/NewTopics/CombinedFilterExprObj/CombinedFilterExprObj.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/NewTopics/BasicFilterExprObj/BasicFilterExprObj.md | Removes placeholder stub (topic migrated to canonical location). |
| api-reference/40 Common Types/15 grids/ResponseStatusTexts/ResponseStatusTexts.md | Adds short + long descriptions for response status text object. |
| api-reference/40 Common Types/15 grids/ResponseStatus.md | Adds short description for AI response status enum. |
| api-reference/40 Common Types/15 grids/PredefinedCommands/PredefinedCommands.md | Adds documentation for predefined commands and their argument shapes. |
| api-reference/40 Common Types/15 grids/PredefinedCommandNames/PredefinedCommandNames.md | Adds documentation listing available predefined command names. |
| api-reference/40 Common Types/15 grids/NegatedFilterExprObj/NegatedFilterExprObj.md | Documents the negated filter expression object shape. |
| api-reference/40 Common Types/15 grids/FilterExprObj/FilterExprObj.md | Documents the union type for grid filter expressions. |
| api-reference/40 Common Types/15 grids/CompositeKeyPair/CompositeKeyPair.md | Documents composite key field/value pair shape. |
| api-reference/40 Common Types/15 grids/CommandInfo/CommandInfo.md | Documents command metadata object fields (name, args). |
| api-reference/40 Common Types/15 grids/CombinedFilterExprObj/CombinedFilterExprObj.md | Documents combined (AND/OR) filter expression object shape. |
| api-reference/40 Common Types/15 grids/BasicFilterExprObj/BasicFilterExprObj.md | Documents basic filter expression object shape. |
| api-reference/40 Common Types/15 grids/AIAssistant/popup.md | Expands the “do not recommend” popup options list. |
| api-reference/40 Common Types/15 grids/AIAssistant/customizeResponseTitle.md | Adds description + multi-framework examples for response title customization. |
| api-reference/40 Common Types/15 grids/AIAssistant/customizeResponseText.md | Adds description + multi-framework examples for per-command response texts. |
| api-reference/40 Common Types/15 grids/AIAssistant/chat.md | Expands the “do not recommend” chat options list. |
| api-reference/10 UI Components/dxDataGrid/9 Types/SummaryCommandTotalItem/SummaryCommandTotalItem.md | Adds DataGrid-specific type doc for summary-command total items. |
| api-reference/10 UI Components/dxDataGrid/9 Types/SummaryCommandGroupItem/SummaryCommandGroupItem.md | Adds DataGrid-specific type doc for summary-command group items. |
| api-reference/10 UI Components/dxDataGrid/9 Types/DataGridPredefinedCommands/DataGridPredefinedCommands.md | Adds DataGrid-specific predefined commands and argument shapes (includes summary/grouping). |
| api-reference/10 UI Components/dxDataGrid/9 Types/DataGridPredefinedCommandNames/DataGridPredefinedCommandNames.md | Adds DataGrid-specific command names list. |
| api-reference/10 UI Components/dxDataGrid/9 Types/DataGridCommandInfo/DataGridCommandInfo.md | Adds DataGrid-specific command info type doc. |
| api-reference/10 UI Components/dxDataGrid/1 Configuration/aiAssistant/customizeResponseTitle.md | Adds DataGrid option doc for response title customization with examples. |
| api-reference/10 UI Components/dxDataGrid/1 Configuration/aiAssistant/customizeResponseText.md | Adds DataGrid option doc for response text customization with examples. |
| api-reference/10 UI Components/dxDataGrid/1 Configuration/aiAssistant/aiAssistant.md | Updates AI Assistant option short description and inheritance metadata. |
Comments suppressed due to low confidence (4)
api-reference/40 Common Types/15 grids/AIAssistant/customizeResponseText.md:140
- The React example defines
customizeResponseText, but the component is configured withcustomizeResponseTitle={customizeResponseTitle}. This looks like a copy/paste error and will not work when copied; passcustomizeResponseTextinstead.
function App() {
return (
<{WidgetName}>
<AIAssistant
enabled={true}
customizeResponseTitle={customizeResponseTitle}
/>
api-reference/10 UI Components/dxDataGrid/1 Configuration/aiAssistant/customizeResponseText.md:140
- The React example defines
customizeResponseText, but the component is configured withcustomizeResponseTitle={customizeResponseTitle}. This looks like a copy/paste error and will not work when copied; passcustomizeResponseTextinstead.
function App() {
return (
<{WidgetName}>
<AIAssistant
enabled={true}
customizeResponseTitle={customizeResponseTitle}
/>
api-reference/40 Common Types/15 grids/ResponseStatus.md:12
- The updated
shortDescriptionis helpful, but the main description section still contains the placeholder<!-- Description goes here -->. Replace this placeholder with an actual description (or remove the section if this type intentionally has no long description).
api-reference/10 UI Components/dxDataGrid/1 Configuration/aiAssistant/aiAssistant.md:11 - This option now has a
shortDescription, but the main description section still contains the placeholder<!-- Description goes here -->. Replace this placeholder with an actual description (or remove it) to avoid publishing placeholder text.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 36 out of 37 changed files in this pull request and generated 10 comments.
Comments suppressed due to low confidence (2)
api-reference/40 Common Types/15 grids/ResponseStatus.md:12
- This topic still contains the placeholder
<!-- Description goes here -->in the main description section. Please replace it with the actual description (or remove the section) so the published API reference does not expose placeholders.
api-reference/10 UI Components/dxDataGrid/1 Configuration/aiAssistant/aiAssistant.md:11 - This topic still contains the placeholder
<!-- Description goes here -->in the main description section. Please replace it with an actual description (or remove the section) so the published API reference does not expose placeholders.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 36 out of 37 changed files in this pull request and generated 11 comments.
Comments suppressed due to low confidence (1)
api-reference/40 Common Types/15 grids/ResponseStatus.md:13
- The long description section still contains the
<!-- Description goes here -->placeholder. Please replace it with an actual description of thesuccess/failurevalues (or remove the placeholder if no long description is intended) so the page is not published with template text.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 38 out of 39 changed files in this pull request and generated 6 comments.
Comments suppressed due to low confidence (6)
api-reference/40 Common Types/15 grids/ResponseStatus.md:13
- The topic still contains the placeholder "" in the main description section. Please replace it with an actual description or remove the empty section to avoid publishing placeholder text.
api-reference/10 UI Components/dxDataGrid/1 Configuration/aiAssistant/aiAssistant.md:11 - The topic still contains the placeholder "" in the main description section. Please replace it with actual content (or remove the empty section) so this option page does not ship with template text.
api-reference/40 Common Types/15 grids/AIAssistant/customizeResponseTitle.md:216 - The file ends with a standalone "---" separator. Most API reference topics end with content (or a See Also block) rather than a trailing section delimiter. Please remove the trailing delimiter (or add the missing content after it) to avoid rendering an empty section/horizontal rule.
};
---
api-reference/40 Common Types/15 grids/AIAssistant/customizeResponseText.md:146
- The file ends with a standalone "---" separator. Most API reference topics end with content (or a See Also block) rather than a trailing section delimiter. Please remove the trailing delimiter (or add the missing content after it) to avoid rendering an empty section/horizontal rule.
);
};
---
api-reference/10 UI Components/dxDataGrid/1 Configuration/aiAssistant/customizeResponseTitle.md:216
- The file ends with a standalone "---" separator. Most API reference topics end with content (or a See Also block) rather than a trailing section delimiter. Please remove the trailing delimiter (or add the missing content after it) to avoid rendering an empty section/horizontal rule.
);
};
---
api-reference/10 UI Components/dxDataGrid/1 Configuration/aiAssistant/customizeResponseText.md:146
- The file ends with a standalone "---" separator. Most API reference topics end with content (or a See Also block) rather than a trailing section delimiter. Please remove the trailing delimiter (or add the missing content after it) to avoid rendering an empty section/horizontal rule.
);
};
---
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 38 out of 39 changed files in this pull request and generated 5 comments.
Comments suppressed due to low confidence (2)
api-reference/40 Common Types/15 grids/ResponseStatus.md:13
- The long description section still contains the placeholder "". Replace it with an actual description (for example, what "success" and "failure" indicate and where this type is used).
api-reference/10 UI Components/dxDataGrid/1 Configuration/aiAssistant/aiAssistant.md:11 - The long description is still a placeholder (""). Since this option is being documented, add a real description (for example, prerequisites like aiIntegration/enabled, and what the assistant UI does).
Co-authored-by: Vladimir Abadzhev <vladimira@devexpress.com>
| - AI Columns | ||
| - AI Assistant | ||
|
|
||
| To activate AI functionality, configure **aiIntegration** and specify {WidgetName} properties as follows: |
There was a problem hiding this comment.
To activate AI functionality sounds a bit vague. Any AI functionality? Specific AI functionality? All AI functionality?
There was a problem hiding this comment.
We can add clarity as follows:
| To activate AI functionality, configure **aiIntegration** and specify {WidgetName} properties as follows: | |
| To activate these AI features, configure **aiIntegration** and specify {WidgetName} properties as follows: |
| --- | ||
| --- | ||
| ##### shortDescription | ||
| A field/value pair used in composite DataGrid/TreeList keys. |
There was a problem hiding this comment.
| A field/value pair used in composite DataGrid/TreeList keys. | |
| A field/value pair used in composite DataGrid or TreeList keys. |
There was a problem hiding this comment.
Perhaps "and" is more appropriate here, as both DataGrid and TreeList can have composite keys.
| A field/value pair used in composite DataGrid/TreeList keys. | |
| A field/value pair used in composite DataGrid and TreeList keys. |
There was a problem hiding this comment.
Yes, but you don't need 'and' to imply that since '/' means 'or', you've got keys and it is clear it applies to both DataGrid and TreeList because without it, it would just be composite DataGrid + TreeList keys that do not make sense
|
|
||
| --- | ||
| <!-- Description goes here --> No newline at end of file | ||
| This object can contain two string values: `success` and `failure`. No newline at end of file |
There was a problem hiding this comment.
| This object can contain two string values: `success` and `failure`. | |
| This object can contain either `success` or `failure`. |
I don't think it can contain both at the same time
There was a problem hiding this comment.
This is an object type, and it can contain both at the same time. This type is used in customizeResponseText callbacks (both described in this PR), where the command's status is not returned. As such, customers need to define both if they wish to specify custom messages for both success and failure statuses.
There was a problem hiding this comment.
| This object can contain two string values: `success` and `failure`. | |
| This object can contain both `success` and `failure`. |
Co-authored-by: dirkpieterse <dirk.pieterse@devexpress.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 39 out of 40 changed files in this pull request and generated 3 comments.
Comments suppressed due to low confidence (2)
api-reference/10 UI Components/dxDataGrid/1 Configuration/aiAssistant/aiAssistant.md:8
- This topic still leaves the long description as the placeholder text, so the generated API page will expose unfinished documentation. Replace the placeholder with the actual aiAssistant description or remove the empty long-description section if it should inherit all content.
api-reference/40 Common Types/15 grids/ResponseStatus.md:10 - The short description is filled in, but the topic still contains the placeholder long description below it. Please replace the placeholder with a real description or remove that section so unfinished documentation is not generated.
| - Set **aiAssistant**.[enabled]({basewidgetpath}/Configuration/aiAssistant/#enabled) to `true` to activate the AI Assistant. | ||
|
|
||
| This object stores shared AI options for the {WidgetName} AI Assistant and all AI columns within the component. To configure AI options specific to a column, define **columns[]**.**ai**.[aiIntegration](/api-reference/40%20Common%20Types/15%20grids/ColumnAIOptions/aiIntegration.md '{basewidgetpath}/Configuration/columns/ai/#aiIntegration'). To configure AI options specific to the AI Assistant, define **aiAssistant**.[aiIntegration]({basewidgetpath}/Configuration/aiAssistant/#aiIntegration). |
No description provided.