[Project] AI Standardization - #239
Draft
akash9676 wants to merge 245 commits into
Draft
Conversation
…n across services
…PC, enabling request cancellation
…LiteLLMRPC, enabling request cancellation" This reverts commit a654e9e.
…e for request cancellation
Integrated LiteLLM RPC
feat(db): addding tables and schemas for AI integration
…ks for model creation and update
… and update UI for testing functionality
…king and visualization of AI requests
…age handling in AI requests
…d model actions, enhancing UI clarity
…role, and study options, including UI components for selection and expiry date management
…components for improved modularity and maintainability
…data binding for user, role, and study selections
…arallel limits and improve task management
MarinaJim
reviewed
Jul 24, 2026
| /** | ||
| * Builds the params object passed directly to LiteLLM's completion() call from a credential row and a model name. | ||
| * | ||
| * @param {{ provider?: string, apiKey?: string, apiBaseUrl?: string, apiVersion?: string }} credential |
Collaborator
There was a problem hiding this comment.
same as in previous files:
- We describe the parameters differently.
- No textual description for return
Check other functions and files for this too.
Collaborator
There was a problem hiding this comment.
it is the same JSdocs but modern and cleaner, but no harm in changing them as well. I did for my functions, update yours. @akash9676
| * @param {Object} options Trigger runtime options | ||
| * @returns {Object} | ||
| */ | ||
| function transactionOptions(options = {}) { |
Collaborator
There was a problem hiding this comment.
we usually just pass { transaction: options.transaction } when calling a model function (getbyid etc) instead of defining such a function
| const model = server.db.models[QUEUE_TABLE]; | ||
| if (!model) return null; | ||
|
|
||
| const item = await model.add({ |
Collaborator
There was a problem hiding this comment.
new lines look weird in this function
…Lab/CARE into project-1-ai-standardization
MRawhani
reviewed
Jul 27, 2026
| documentId, | ||
| studySessionId: null, | ||
| studyStepId: null, | ||
| key: buildStudyHookKey("nlpRequest", hook.name), |
Collaborator
There was a problem hiding this comment.
why nlpRequest here as key?
when we save to document data, we use the wrokflow service name.
Co-authored-by: Cursor <cursoragent@cursor.com>
- Added `getProviders` method in LiteLLMRPC to fetch supported provider slugs. - Integrated provider loading in AICredential component with loading state and error handling. - Updated chat service to include `getProviders` functionality. - Enhanced templateResolver.js to correct import paths. - Added server-side support for `getProviders` in the LiteLLM Python implementation.
- Changed the order of 'Triggers' to 6 and 'Settings' to 7 in the dashboard navigation group settings.
…component - Changed the display of trigger names to show "<Deleted Trigger>" for triggers that are no longer available. - Updated error summary to reflect the new trigger name handling for deleted triggers.
- Implemented a searchable mode for the select component, allowing users to filter options via a search input. - Updated the form documentation to include examples of the new searchable select feature. - Enhanced the AICredential component to utilize the searchable select for provider selection, improving user experience during provider loading.
…omponent - Updated the updateMapping method to allow only one table-based source for Skills, while enabling multiple sources for Hooks when using indexed placeholders.
- Simplified value binding in the Select component to ensure it directly uses option values. - Enhanced currentData handling to prevent unnecessary updates and ensure correct value resolution. - Updated InputMap component to use :model-value for better compatibility with the Select component's changes.
- Refactored fallback logic to ensure skills are loaded correctly when no live nodes are available. - Enhanced error handling during fallback loading to prevent crashes and ensure skills are populated. - Updated skill update logic to handle cases where skills may be empty, ensuring clients receive the correct updates.
- Updated the file system module import to use fs.promises for better handling of asynchronous operations. - This change improves code readability and aligns with modern JavaScript practices.
…Lab/CARE into project-1-ai-standardization
eyadmohamed01
self-requested a review
August 12, 2026 15:58
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.