Add Chat - Custom AI Assistant demo jQuery - #35025
Conversation
16adianay
changed the title
- Add aria-label to CheckBoxes - Add etalons - Fux quotes
There was a problem hiding this comment.
🟡 Changes recommended
Unrelated prompts, compound filters, hidden columns, and timed-out Smart Paste requests currently produce incorrect or unsafe behavior.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Adds a jQuery Chat demo that routes AI prompts to Form and DataGrid operations.
Changes:
- Adds the custom AI assistant UI and OpenAI integration.
- Implements form and grid command routing.
- Registers demo metadata, CSP permissions, and lint configuration.
File summaries
| File | Description |
|---|---|
apps/demos/utils/server/csp-server.js |
Allows required AI service and module sources. |
apps/demos/menuMeta.json |
Registers the new demo. |
apps/demos/eslint.config.mjs |
Configures lint exceptions for shared globals. |
apps/demos/Demos/Chat/CustomAIAssistant/jQuery/styles.css |
Styles the assistant, form, and grid. |
apps/demos/Demos/Chat/CustomAIAssistant/jQuery/index.js |
Initializes the demo UI and interactions. |
apps/demos/Demos/Chat/CustomAIAssistant/jQuery/index.html |
Defines assets and page structure. |
apps/demos/Demos/Chat/CustomAIAssistant/jQuery/grid-commands.js |
Implements AI-driven grid commands. |
apps/demos/Demos/Chat/CustomAIAssistant/jQuery/form-commands.js |
Implements form clearing and Smart Paste. |
apps/demos/Demos/Chat/CustomAIAssistant/jQuery/data.js |
Provides configuration and sample data. |
apps/demos/Demos/Chat/CustomAIAssistant/jQuery/chat-router.js |
Classifies and routes user requests. |
apps/demos/Demos/Chat/CustomAIAssistant/jQuery/ai-service.js |
Configures AI requests and retry behavior. |
Review details
- Files reviewed: 11/13 changed files
- Comments generated: 6
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
dmlvr
left a comment
dmlvr
left a comment
There was a problem hiding this comment.
Our demos currently support only a strictly defined set of files:
- index.html
- styles.css
- index.js
- data.js
This is related to the logic of automatic codepen creation.
Please, move data from other .js files to index.js.
dmlvr
left a comment
dmlvr
left a comment
There was a problem hiding this comment.
minor: do we need support a11y in this demo? For example I cannot focus FAB button from keyboadr.
The FAB button does not meet any accessibility requirements and currently are not accessible according to docs: |
- Move JS to index.js - Improve readability of long texts - DataGrid filters can be combined now - The updated form fields are shown in a message from AI assistant
No description provided.