Skip to content

Conversation

Spycsh
Copy link
Collaborator

@Spycsh Spycsh commented Apr 29, 2025

Description

Enable OPEA microservices to start as MCP servers.

Finished

  • microservice level seamless conversion to MCP with register_microservice
  • support TOOL/PROMPT/RESOURCE func type
  • simple example with asr whisper
  • add tool/prompt/resource to existing MCP server
  • add dummy MCP server test
  • add specific test case for asr with mcp client

Issues

1832

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)
  • Others (enhancement, documentation, validation, etc.)

Dependencies

mcp

Tests

ut

Copy link

codecov bot commented Apr 29, 2025

Codecov Report

Attention: Patch coverage is 81.48148% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
comps/cores/mega/micro_service.py 77.27% 5 Missing ⚠️
Files with missing lines Coverage Δ
comps/cores/mega/constants.py 97.26% <100.00%> (+0.15%) ⬆️
comps/cores/mega/micro_service.py 78.70% <77.27%> (-1.08%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Spycsh Spycsh requested a review from chensuyue as a code owner April 29, 2025 09:25
@joshuayao joshuayao linked an issue May 7, 2025 that may be closed by this pull request
@lvliang-intel lvliang-intel changed the title Enable OPEA microservices to start as MCP servers FEAT: Enable OPEA microservices to start as MCP servers May 11, 2025
@Spycsh Spycsh requested a review from rbrugaro as a code owner May 12, 2025 02:50
@lvliang-intel lvliang-intel merged commit 28a2820 into opea-project:main May 13, 2025
18 checks passed
alexsin368 pushed a commit to alexsin368/GenAIComps that referenced this pull request May 15, 2025
yinghu5 added a commit that referenced this pull request May 16, 2025
* add support for remote server

Signed-off-by: alexsin368 <[email protected]>

* add steps to enable remote server

Signed-off-by: alexsin368 <[email protected]>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* remove use_remote_service

Signed-off-by: alexsin368 <[email protected]>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* add OpenAI models instructions, fix format of commands

Signed-off-by: alexsin368 <[email protected]>

* simplify ChatOpenAI instantiation

Signed-off-by: alexsin368 <[email protected]>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Revert "simplify ChatOpenAI instantiation"

This reverts commit b7c4acf.

* add back check and logic for llm_engine, set openai_key argument

Signed-off-by: alexsin368 <[email protected]>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Provide ARCH option for lvm-video-llama image build (#1630)

Signed-off-by: ZePan110 <[email protected]>
Signed-off-by: alexsin368 <[email protected]>

* Add sglang microservice for supporting llama4 model (#1640)

Signed-off-by: Ye, Xinyu <[email protected]>
Co-authored-by: Lv,Liang1 <[email protected]>
Signed-off-by: alexsin368 <[email protected]>

* Remove invalid codeowner. (#1642)

Signed-off-by: ZePan110 <[email protected]>
Signed-off-by: alexsin368 <[email protected]>

* add support for remote server

Signed-off-by: alexsin368 <[email protected]>

* add steps to enable remote server

Signed-off-by: alexsin368 <[email protected]>

* remove use_remote_service

Signed-off-by: alexsin368 <[email protected]>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Signed-off-by: alexsin368 <[email protected]>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Signed-off-by: alexsin368 <[email protected]>

* bug fix for chunk_size and overlap cause error in dataprep ingestion (#1643)

* bug fix for dataingest url

Signed-off-by: Mustafa <[email protected]>

* add validation function

Signed-off-by: Mustafa <[email protected]>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* validation update

Signed-off-by: Mustafa <[email protected]>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* update validation function

Signed-off-by: Mustafa <[email protected]>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Signed-off-by: Mustafa <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Signed-off-by: alexsin368 <[email protected]>

* MariaDB Vector integrations for retriever & dataprep services (#1645)

* Add MariaDB Vector third-party service

MariaDB Vector was introduced since MariaDB Server 11.7

Signed-off-by: Razvan-Liviu Varzaru <[email protected]>

* Add retriever MariaDB Vector integration

Signed-off-by: Razvan-Liviu Varzaru <[email protected]>

* Add dataprep MariaDB Vector integration

Signed-off-by: Razvan-Liviu Varzaru <[email protected]>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Fix CI failures

- md5 is used for the primary key not as a security hash
- fixed mariadb readme headers

Signed-off-by: Razvan-Liviu Varzaru <[email protected]>

---------

Signed-off-by: Razvan-Liviu Varzaru <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Signed-off-by: alexsin368 <[email protected]>

* update PR reviewers (#1651)

Signed-off-by: chensuyue <[email protected]>
Signed-off-by: alexsin368 <[email protected]>

* Expand test matrix, find all tests use 3rd party Dockerfiles (#1676)

* Expand test matrix, find all tests use 3rd party Dockerfiles

Signed-off-by: chensuyue <[email protected]>
Signed-off-by: alexsin368 <[email protected]>

* fix the typo of README.md Comp (#1679)

Update README.md for first entry of OPEA

Signed-off-by: alexsin368 <[email protected]>

* Fix request handle timeout issue (#1687)

Signed-off-by: lvliang-intel <[email protected]>
Signed-off-by: alexsin368 <[email protected]>

* FEAT: Enable OPEA microservices to start as MCP servers (#1635)

Signed-off-by: alexsin368 <[email protected]>

* Fix huggingface_hub API upgrade issue (#1691)

* Fix huggingfacehub API upgrade issue

Signed-off-by: lvliang-intel <[email protected]>
Signed-off-by: alexsin368 <[email protected]>

* add OpenAI models instructions, fix format of commands

Signed-off-by: alexsin368 <[email protected]>

* Fix dataprep opensearch ingest issue (#1697)

Signed-off-by: lvliang-intel <[email protected]>
Signed-off-by: alexsin368 <[email protected]>

* Fix embedding issue with ArangoDB due to deprecated HuggingFace API (#1694)

Signed-off-by: lvliang-intel <[email protected]>
Signed-off-by: alexsin368 <[email protected]>

* simplify ChatOpenAI instantiation

Signed-off-by: alexsin368 <[email protected]>

* Revert "simplify ChatOpenAI instantiation"

This reverts commit b7c4acf.

Signed-off-by: alexsin368 <[email protected]>

* add back check and logic for llm_engine, set openai_key argument

Signed-off-by: alexsin368 <[email protected]>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Signed-off-by: alexsin368 <[email protected]>
Signed-off-by: ZePan110 <[email protected]>
Signed-off-by: Ye, Xinyu <[email protected]>
Signed-off-by: Mustafa <[email protected]>
Signed-off-by: Razvan-Liviu Varzaru <[email protected]>
Signed-off-by: chensuyue <[email protected]>
Signed-off-by: lvliang-intel <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ying Hu <[email protected]>
Co-authored-by: ZePan110 <[email protected]>
Co-authored-by: Liang Lv <[email protected]>
Co-authored-by: Mustafa <[email protected]>
Co-authored-by: Razvan Liviu Varzaru <[email protected]>
Co-authored-by: chen, suyue <[email protected]>
Co-authored-by: Spycsh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] ASR MCP Support
4 participants