Skip to content

Top-level bullets for AI in the docs#10693

Merged
m0mus merged 3 commits into
helidon-io:mainfrom
m0mus:ai_docs
Sep 24, 2025
Merged

Top-level bullets for AI in the docs#10693
m0mus merged 3 commits into
helidon-io:mainfrom
m0mus:ai_docs

Conversation

@m0mus

@m0mus m0mus commented Sep 19, 2025

Copy link
Copy Markdown
Contributor

Resolves #10550

Adds MCP Server docs

@oracle-contributor-agreement oracle-contributor-agreement Bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Sep 19, 2025
Comment thread docs/src/main/asciidoc/includes/ai/mcp.adoc Outdated
Comment thread docs/src/main/asciidoc/includes/ai/mcp.adoc Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Under General Concepts > Creating Components

  1. Change "Supported LangChain4j components are automatically created and registered in service registry if exists the corresponding configuration and enabled property is set to true." to
    "Supported LangChain4j components are automatically created and registered in service registry if the corresponding configuration exists and enabled property is set to true."

  2. Change "For example, adding the following lines to application.yaml cause of automatic creating and registering OpenAiChatModel, which later can be injected in other classes of your application:" to

"For example, adding the following lines to application.yaml causes the automatic creation and registration OpenAiChatModel, which later can be injected in other classes of your application:"

Under Supplier Factory

"This method is not limited to LangChain4j and suitable for creating and registering other classes." This should be documented somewhere else.

"By default, the default or unnamed Helidon bean is created. To create a named service, you need to add @Service.Named("name") annotation."

How does a user use the newly named service? It would be good to provide a link to an example here.

Under Injecting Components:

"You can inject Helidon services into another Helidon services using constructor injection like this:"

Typo. Please change: "You can inject a Helidon service into another Helidon service using constructor injection like this:"

The generic example should be in the same location as for supplier factory. Here, keep a concrete example for LangChain4j.

Under Creating AI Service:
"It’s a pure LangChain4j component.". What does being pure mean? are there other types of components?

MCP Client should go to the MCP page.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The purpose of this PR is moving AI docs to the root location and add MCP page. Changing LC4j docs is out scope. Please create another issue for it and assign it to me.

Comment thread docs/src/main/asciidoc/mp/ai/mcp.adoc Outdated

@hyder hyder left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

comments inline

@hyder hyder left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved. I'll create an issue for the other one.

@m0mus m0mus merged commit 726eebd into helidon-io:main Sep 24, 2025
113 of 129 checks passed
romain-grecourt pushed a commit to al3xandru/helidon that referenced this pull request Sep 24, 2025
* Top-level bullets for AI in the docs

Resolves helidon-io#10550

Adds MCP Server docs

* Addressing comments

* MCP Client section added
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move AI Documentation to top level

3 participants