From 9200fc41a6e2833789ea3e7f1e49712533cfac74 Mon Sep 17 00:00:00 2001 From: erangi-ar <111747955+erangi-ar@users.noreply.github.com> Date: Fri, 28 Nov 2025 11:20:54 +0530 Subject: [PATCH] QA bug fixes (#174) * partialy completes prompt refiner * integrate prompt refiner with llm_config_module * fixed ruff lint issues * complete prompt refiner, chunk retriver and reranker * remove unnesessary comments * updated .gitignore * Remove data_sets from tracking * update .gitignore file * complete vault setup and response generator * remove ignore comment * removed old modules * fixed merge conflicts * Vault Authentication token handling (#154) (#70) * partialy completes prompt refiner * integrate prompt refiner with llm_config_module * fixed ruff lint issues * complete prompt refiner, chunk retriver and reranker * remove unnesessary comments * updated .gitignore * Remove data_sets from tracking * update .gitignore file * complete vault setup and response generator * remove ignore comment * removed old modules * fixed merge conflicts * added initial setup for the vector indexer * initial llm orchestration service update with context generation * added new endpoints * vector indexer with contextual retrieval * fixed requested changes * fixed issue * initial diff identifier setup * uncommment docker compose file * added test endpoint for orchestrate service * fixed ruff linting issue * Rag 103 budget related schema changes (#41) * Refactor llm_connections table: update budget tracking fields and reorder columns * Add budget threshold fields and logic to LLM connection management * Enhance budget management: update budget status logic, adjust thresholds, and improve form handling for LLM connections * resolve pr comments & refactoring * rename commonUtils --------- * Rag 93 update connection status (#47) * Refactor llm_connections table: update budget tracking fields and reorder columns * Add budget threshold fields and logic to LLM connection management * Enhance budget management: update budget status logic, adjust thresholds, and improve form handling for LLM connections * resolve pr comments & refactoring * rename commonUtils * Implement LLM connection status update functionality with API integration and UI enhancements --------- * Rag 99 production llm connections logic (#46) * Refactor llm_connections table: update budget tracking fields and reorder columns * Add budget threshold fields and logic to LLM connection management * Enhance budget management: update budget status logic, adjust thresholds, and improve form handling for LLM connections * resolve pr comments & refactoring * rename commonUtils * Add production connection retrieval and update related components * Implement LLM connection environment update and enhance connection management logic --------- * Rag 119 endpoint to update used budget (#42) * Refactor llm_connections table: update budget tracking fields and reorder columns * Add budget threshold fields and logic to LLM connection management * Enhance budget management: update budget status logic, adjust thresholds, and improve form handling for LLM connections * resolve pr comments & refactoring * Add functionality to update used budget for LLM connections with validation and response handling * Implement budget threshold checks and connection deactivation logic in update process * resolve pr comments --------- * Rag 113 warning and termination banners (#43) * Refactor llm_connections table: update budget tracking fields and reorder columns * Add budget threshold fields and logic to LLM connection management * Enhance budget management: update budget status logic, adjust thresholds, and improve form handling for LLM connections * resolve pr comments & refactoring * Add budget status check and update BudgetBanner component * rename commonUtils * resove pr comments --------- * rag-105-reset-used-budget-cron-job (#44) * Refactor llm_connections table: update budget tracking fields and reorder columns * Add budget threshold fields and logic to LLM connection management * Enhance budget management: update budget status logic, adjust thresholds, and improve form handling for LLM connections * resolve pr comments & refactoring * Add cron job to reset used budget * rename commonUtils * resolve pr comments * Remove trailing slash from vault/agent-out in .gitignore --------- * Rag 101 budget check functionality (#45) * Refactor llm_connections table: update budget tracking fields and reorder columns * Add budget threshold fields and logic to LLM connection management * Enhance budget management: update budget status logic, adjust thresholds, and improve form handling for LLM connections * resolve pr comments & refactoring * rename commonUtils * budget check functionality --------- * gui running on 3003 issue fixed * gui running on 3003 issue fixed (#50) * added get-configuration.sqpl and updated llmconnections.ts * Add SQL query to retrieve configuration values * Hashicorp key saving (#51) * gui running on 3003 issue fixed * Add SQL query to retrieve configuration values --------- * Remove REACT_APP_NOTIFICATION_NODE_URL variable Removed REACT_APP_NOTIFICATION_NODE_URL environment variable. * added initil diff identifier functionality * test phase1 * Refactor inference and connection handling in YAML and TypeScript files * fixes (#52) * gui running on 3003 issue fixed * Add SQL query to retrieve configuration values * Refactor inference and connection handling in YAML and TypeScript files --------- * Add entry point script for Vector Indexer with command line interface * fix (#53) * gui running on 3003 issue fixed * Add SQL query to retrieve configuration values * Refactor inference and connection handling in YAML and TypeScript files * Add entry point script for Vector Indexer with command line interface --------- * diff fixes * uncomment llm orchestration service in docker compose file * complete vector indexer * Add YAML configurations and scripts for managing vault secrets * Add vault secret management functions and endpoints for LLM connections * Add Test Production LLM page with messaging functionality and styles * fixed issue * fixed merge conflicts * fixed issue * fixed issue * updated with requested chnages * fixed test ui endpoint request responses schema issue * fixed dvc path issue * added dspy optimization * filters fixed * refactor: restructure llm_connections table for improved configuration and tracking * feat: enhance LLM connection handling with AWS and Azure embedding credentials * fixed issues * refactor: remove redundant Azure and AWS credential assignments in vault secret functions * fixed issue * intial vault setup script * complete vault authentication handling * review requested change fix * fixed issues according to the pr review * fixed issues in docker compose file relevent to pr review --------- Co-authored-by: Charith Nuwan Bimsara <59943919+nuwangeek@users.noreply.github.com> Co-authored-by: erangi-ar * Enhance LLM connection update SQL and improve responsive design in LLMConnectionForm * temp revert env update logic --------- Co-authored-by: Charith Nuwan Bimsara <59943919+nuwangeek@users.noreply.github.com> Co-authored-by: nuwangeek Co-authored-by: erangi-ar --- .../rag-search/POST/update-llm-connection.sql | 18 ++--- .../LLMConnectionForm/LLMConnectionForm.scss | 76 +++++++++++++++++-- 2 files changed, 79 insertions(+), 15 deletions(-) diff --git a/DSL/Resql/rag-search/POST/update-llm-connection.sql b/DSL/Resql/rag-search/POST/update-llm-connection.sql index e4fa4fd..3fa7bc6 100644 --- a/DSL/Resql/rag-search/POST/update-llm-connection.sql +++ b/DSL/Resql/rag-search/POST/update-llm-connection.sql @@ -25,19 +25,19 @@ SET embedding_target_uri = :embedding_target_uri, embedding_azure_api_key = :embedding_azure_api_key WHERE id = :connection_id -RETURNING - id, +RETURNING + id, connection_name, - llm_platform, - llm_model, - embedding_platform, - embedding_model, - monthly_budget, + llm_platform, + llm_model, + embedding_platform, + embedding_model, + monthly_budget, warn_budget_threshold, stop_budget_threshold, disconnect_on_budget_exceed, - environment, - connection_status, + environment, + connection_status, created_at, deployment_name, target_uri, diff --git a/GUI/src/components/molecules/LLMConnectionForm/LLMConnectionForm.scss b/GUI/src/components/molecules/LLMConnectionForm/LLMConnectionForm.scss index 571d801..c999f4a 100644 --- a/GUI/src/components/molecules/LLMConnectionForm/LLMConnectionForm.scss +++ b/GUI/src/components/molecules/LLMConnectionForm/LLMConnectionForm.scss @@ -90,15 +90,54 @@ .flex-grid { display: flex; gap: 12px; + flex-wrap: wrap; + + button { + flex: 0 1 auto; + min-width: 80px; + max-width: 100%; + } } // Responsive design - @media (max-width: 768px) { - padding: 16px; - + // Very small screens - wrap buttons (inline buttons with wrapping) + @media (max-width: 480px) { + padding: 8px; + + .form-section { + padding: 12px; + margin-bottom: 20px; + } + + .form-footer { + margin-top: 20px; + padding-top: 12px; + } + + .flex-grid { + + flex-wrap: wrap; + gap: 8px; + justify-content: flex-end; + + button { + flex: 0 1 auto; + + min-width: 60px; + max-width: calc(50% - 4px); + padding: 8px 12px; + font-size: 13px; + } + } + } + + // Small screens - mobile + @media (min-width: 481px) and (max-width: 768px) { + padding: 12px; + .form-section { - padding: 16px; - margin-bottom: 24px; + padding: 14px; + margin-bottom: 22px; } .radio-options { @@ -109,9 +148,34 @@ padding: 6px 10px; } + .form-footer { + margin-top: 24px; + padding-top: 16px; + } + + .flex-grid { + flex-direction: column-reverse; + gap: 12px; + + button { + width: 100%; + min-width: unset; + } + } + } + + // Medium screens - tablet + @media (min-width: 769px) and (max-width: 1024px) { .flex-grid { - flex-direction: column; gap: 8px; + + button { + flex: 1 1 auto; + min-width: 70px; + max-width: 200px; + font-size: 14px; + padding: 8px 12px; + } } } }