Skip to content

feat: add embedding support for parasail provider - #7096

Open
sammaji wants to merge 1 commit into
09-09-test_embeddings_e2e_coverage_for_multimodal_embeddingsfrom
09-11-feat_adds_embeddings_support_for_parasail
Open

feat: add embedding support for parasail provider#7096
sammaji wants to merge 1 commit into
09-09-test_embeddings_e2e_coverage_for_multimodal_embeddingsfrom
09-11-feat_adds_embeddings_support_for_parasail

Conversation

@sammaji

@sammaji sammaji commented Sep 11, 2026

Copy link
Copy Markdown
Member

Summary

Enables embedding support for the Parasail provider by wiring up Parasail's OpenAI-compatible /v1/embeddings endpoint instead of returning an UnsupportedOperationError.

Changes

  • Replaced the stub Embedding implementation in the Parasail provider with a call to openai.HandleOpenAIEmbeddingRequest, using Parasail's /v1/embeddings endpoint with bearer auth.
  • Updated the Parasail integration test to use parasail-bge-m3 as the embedding model and enabled the Embedding test scenario.
  • Updated the provider capability table in the overview docs to mark Parasail embeddings as supported.
  • Updated the Parasail-specific docs to document the embeddings endpoint, supported input formats, and model addressing (Parasail ID or HuggingFace ID), and removed embeddings from the unsupported features list.

Type of change

  • Bug fix
  • Feature
  • Refactor
  • Documentation
  • Chore/CI

Affected areas

  • Core (Go)
  • Transports (HTTP)
  • Providers/Integrations
  • Plugins
  • UI (React)
  • Docs

How to test

go test ./core/providers/parasail/...

The Embedding scenario will run against the Parasail API using the parasail-bge-m3 model. Confirm a valid BifrostEmbeddingResponse is returned without error.

Breaking changes

  • Yes
  • No

Related issues

Security considerations

No new secrets or auth mechanisms introduced. The existing bearer token flow used by other Parasail endpoints is reused as-is.

Checklist

  • I read docs/contributing/README.md and followed the guidelines
  • I added/updated tests where appropriate
  • I updated documentation where needed
  • I verified builds succeed (Go and UI)
  • I verified the CI pipeline passes locally if applicable

@coderabbitai

coderabbitai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 2c9f5da9-0cbd-4dcc-a0db-a6e141146fd2

📥 Commits

Reviewing files that changed from the base of the PR and between 341ae96 and 246bcec.

📒 Files selected for processing (5)
  • core/providers/parasail/parasail.go
  • core/providers/parasail/parasail_test.go
  • docs/providers/supported-providers/overview.mdx
  • docs/providers/supported-providers/parasail.mdx
  • ui/lib/constants/logs.ts

Included review availability: 7 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 10 reviews per hour.


📝 Summary

Summary by CodeRabbit

  • New Features

    • Parasail now supports embedding requests through the OpenAI-compatible /v1/embeddings endpoint.
    • Embedding requests support bearer authentication, configured headers, supported input formats, encoding options, and Parasail embedding models.
    • Parasail is recognized as an embedding-capable provider in the application.
  • Documentation

    • Updated provider support listings and Parasail documentation to reflect embedding support and accepted request options.
  • Tests

    • Added embedding scenarios using the Parasail embedding model.

Walkthrough

Parasail now supports embedding requests through its OpenAI-compatible endpoint. Tests enable embedding scenarios, documentation lists the supported operation, and UI provider constants recognize Parasail as embedding-capable.

Changes

Parasail embeddings

Layer / File(s) Summary
Embedding endpoint and validation
core/providers/parasail/parasail.go, core/providers/parasail/parasail_test.go
Parasail forwards embedding requests through the shared OpenAI handler. Tests configure parasail-bge-m3 and enable embedding scenarios.
Support metadata and documentation
docs/providers/supported-providers/*, ui/lib/constants/logs.ts
Support matrices and Parasail documentation list embeddings as supported. UI constants include Parasail among embedding providers.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: pratham-mishra04

Merge Risk: ⚪ Minimal · up to 246bc

The Parasail embedding support changes are consistent across the provider implementation, tests, documentation, and UI capability metadata.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 3 files. (2 skipped: 2 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely states the main change: adding embedding support for the Parasail provider.
Description check ✅ Passed The description covers the purpose, implementation changes, tests, affected areas, breaking changes, security, documentation, and checklist status. Related issues are not provided, and full Go/UI buil…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 3 files. (2 skipped: 2 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 09-11-feat_adds_embeddings_support_for_parasail

Comment @coderabbitai help to get the list of available commands.

@sammaji
sammaji force-pushed the 09-11-feat_adds_embeddings_support_for_parasail branch from 318482e to 246bcec Compare September 11, 2026 17:38
@sammaji
sammaji marked this pull request as ready for review September 11, 2026 17:41
Copilot AI lite review requested due to automatic review settings September 11, 2026 17:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants