Skip to content

Fix : Test connection issue for Anthropic, Cohere, and Mistral - #18839

Open
ganesh8056 wants to merge 3 commits into
lts-3.16from
fix/ai-plugins-testconnection-fix
Open

Fix : Test connection issue for Anthropic, Cohere, and Mistral#18839
ganesh8056 wants to merge 3 commits into
lts-3.16from
fix/ai-plugins-testconnection-fix

Conversation

@ganesh8056

@ganesh8056 ganesh8056 commented Nov 3, 2026

Copy link
Copy Markdown
Collaborator

This pull request updates the connection test logic for Anthropic, Cohere, and Mistral plugin services to use model listing endpoints instead of sending chat messages. This approach provides a more reliable and lightweight way to validate API keys and connections, and it also improves error handling for failed connections.

Connection validation improvements:

  • marketplace/plugins/anthropic/lib/index.ts: Changed the Anthropic connection test to call the /v1/models REST API directly (instead of sending a chat message) to validate the API key. Improved error handling to provide clearer error messages and details when the connection fails.
  • marketplace/plugins/cohere/lib/index.ts: Updated the Cohere connection test to use cohere.models.list for validation instead of sending a chat message.
  • marketplace/plugins/mistral_ai/lib/index.ts: Modified the Mistral connection test to use client.models.list() for validation, removing the need to send a chat completion request and simplifying the logic.

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Security Audit Report

Node module vulnerabilities summary (total across frontend, server, marketplace, plugins, root):
πŸ”΄ Critical: 18
🟠 High: 149
🟑 Moderate: 324

Critical packages:

  • frontend: handlebars,jspdf,maplibre-gl,plotly.js,protobufjs,proxy-addr,shell-quote,storybook,tar,websocket-driver
  • server: @simple-git/argv-parser,handlebars,proxy-addr,simple-git
  • marketplace: handlebars,tar,websocket-driver
  • plugins: handlebars

ℹ️ Runs automatically on every PR update (also triggerable via the check-vulnerability label). Critical/High should be fixed before merge; Moderate can be tracked separately. Counts are from npm audit β€” per-workspace JSON artifacts are on the summary page.

@Mekhla-Asopa Mekhla-Asopa added the create-ee-lts-review-app uses ./docker/LTS/ee/ee-preview.Dockerfile label Nov 3, 2026
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Security Audit Report

Node module vulnerabilities summary (total across frontend, server, marketplace, plugins, root):
πŸ”΄ Critical: 0
🟠 High: 0
🟑 Moderate: 0

Critical packages:
None πŸŽ‰

ℹ️ Runs automatically on every PR update (also triggerable via the check-vulnerability label). Critical/High should be fixed before merge; Moderate can be tracked separately. Counts are from npm audit β€” per-workspace JSON artifacts are on the summary page.

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

πŸš€ EE LTS Review App Deployed!

Resource Link
App URL https://tooljet-ee-lts-pr-18839.onrender.com
Render Dashboard https://dashboard.render.com/web/srv-dbko70rncjis73et3iu0

Deployed using DockerHub-based pipeline - LTS Edition

@github-actions github-actions Bot added active-ee-lts-review-app For lts (github action build and deploy on render)) and removed create-ee-lts-review-app uses ./docker/LTS/ee/ee-preview.Dockerfile labels Nov 3, 2026
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Security Audit Report

Node module vulnerabilities summary (total across frontend, server, marketplace, plugins, root):
πŸ”΄ Critical: 0
🟠 High: 0
🟑 Moderate: 0

Critical packages:
None πŸŽ‰

ℹ️ Runs automatically on every PR update (also triggerable via the check-vulnerability label). Critical/High should be fixed before merge; Moderate can be tracked separately. Counts are from npm audit β€” per-workspace JSON artifacts are on the summary page.

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Marketplace Plugin added to stage bucket

πŸ” View Deployment Logs & Summary

━━━━━━━━━━━━━━━ UPLOAD SUMMARY ━━━━━━━━━━━━━━━━━
[8:09:29 AM] πŸŽ‰ All files uploaded successfully
[8:09:29 AM] βœ… Successfully uploaded: 633/633 files
[8:09:29 AM] ❌ Failed uploads: 0/633 files
[8:09:29 AM] β„Ή Total time: 3.3s

πŸ†• New files pushed (0):
  (none β€” every key already existed in the bucket)

✨ New plugins pushed (0):
  (none β€” no brand-new plugin in this run)

πŸ“ Plugins already present in the bucket (47):
  β€’ aftership
  β€’ anthropic
  β€’ asana
  β€’ authorizenet
  β€’ aws-bedrock
  β€’ aws-lambda
  β€’ awsredshift
  β€’ azurerepos
  β€’ clickup
  β€’ cohere
  β€’ confluence
  β€’ couchbase
  β€’ easypost
  β€’ engagespot
  β€’ fedex
  β€’ gemini
  β€’ github
  β€’ gmail
  β€’ googlecalendar
  β€’ harperdb
  β€’ hubspot
  β€’ hugging_face
  β€’ intercom
  β€’ jira
  β€’ microsoft_graph
  β€’ mistral_ai
  β€’ openai
  β€’ openrouter
  β€’ pinecone
  β€’ plivo
  β€’ pocketbase
  β€’ portkey
  β€’ presto
  β€’ prometheus
  β€’ qdrant
  β€’ quickbooks
  β€’ s3
  β€’ salesforce
  β€’ servicenow
  β€’ sharepoint
  β€’ spanner
  β€’ supabase
  β€’ textract
  β€’ typesafe
  β€’ ups
  β€’ weaviate
  β€’ xero

@Mekhla-Asopa Mekhla-Asopa added tested run-cypress Cypress E2E action labels Nov 3, 2026
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Security Audit Report

Node module vulnerabilities summary (total across frontend, server, marketplace, plugins, root):
πŸ”΄ Critical: 0
🟠 High: 0
🟑 Moderate: 0

Critical packages:
None πŸŽ‰

ℹ️ Runs automatically on every PR update (also triggerable via the check-vulnerability label). Critical/High should be fixed before merge; Moderate can be tracked separately. Counts are from npm audit β€” per-workspace JSON artifacts are on the summary page.

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Security Audit Report

Node module vulnerabilities summary (total across frontend, server, marketplace, plugins, root):
πŸ”΄ Critical: 0
🟠 High: 0
🟑 Moderate: 0

Critical packages:
None πŸŽ‰

ℹ️ Runs automatically on every PR update (also triggerable via the check-vulnerability label). Critical/High should be fixed before merge; Moderate can be tracked separately. Counts are from npm audit β€” per-workspace JSON artifacts are on the summary page.

@Mekhla-Asopa Mekhla-Asopa added run-cypress Cypress E2E action and removed run-cypress Cypress E2E action labels Nov 3, 2026
@adishM98 adishM98 mentioned this pull request Nov 3, 2026
Open
3 of 4 tasks
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

πŸ”„ EE LTS Review App Rebuilding

New commit 3eb03a4 pushed. Image updated and deployment triggered.

Using cached layers for faster build

@Mekhla-Asopa Mekhla-Asopa added run-cypress Cypress E2E action and removed run-cypress Cypress E2E action labels Nov 3, 2026
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

πŸ”„ EE LTS Review App Rebuilding

New commit 30ca0a8 pushed. Image updated and deployment triggered.

Using cached layers for faster build

@adishM98 adishM98 added the suspend-ee-lts-review-app For lts (github action build and deploy on render)) label Nov 4, 2026
@github-actions github-actions Bot removed the active-ee-lts-review-app For lts (github action build and deploy on render)) label Nov 4, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

marketplace-plugin-deployed run-cypress Cypress E2E action suspend-ee-lts-review-app For lts (github action build and deploy on render)) tested

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ganesh8056 @adishM98 @Mekhla-Asopa