Skip to content

Conversation

@ryoppippi
Copy link
Member

@ryoppippi ryoppippi commented Nov 18, 2025

Removes implementation details (BM25 algorithm references) from the Meta Tools README section.

Users don't need to know about internal search algorithms to use meta tools effectively. This keeps the README focused on user-facing functionality and aligns with our documentation site.

Remove reference to "Orama's BM25 algorithm" from the Meta Tools section
to keep documentation focused on user-facing functionality rather than
internal implementation details.

This change:
- Removes mention of specific algorithms (BM25, Orama)
- Keeps the focus on what users can do (discover and execute tools)
- Aligns with the documentation in agents/typescript/meta-tools.mdx
- Maintains consistency with SDK documentation standards

Users don't need to know about the underlying search algorithm to
effectively use meta tools. The implementation details can change
over time without affecting the user-facing API.
Copilot AI review requested due to automatic review settings November 18, 2025 14:45
@ryoppippi ryoppippi requested a review from a team as a code owner November 18, 2025 14:45
@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 18, 2025

Open in StackBlitz

npm i https://pkg.pr.new/StackOneHQ/stackone-ai-node/@stackone/ai@138

commit: b1ca20b

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes internal implementation details from the Meta Tools section in the README, making the documentation more user-focused and maintainable.

  • Removed specific references to Orama and BM25 algorithm from the Meta Tools documentation
  • Maintains focus on user-facing functionality (tool discovery and execution capabilities)
  • Aligns with documentation best practices by avoiding implementation details that may change

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Related: ENG-11260
@ryoppippi ryoppippi enabled auto-merge (squash) November 18, 2025 15:04
Copy link
Contributor

@NicolasBelissent NicolasBelissent left a comment

Choose a reason for hiding this comment

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

LGTM

@ryoppippi ryoppippi merged commit 99ebcc2 into main Nov 18, 2025
6 of 7 checks passed
@ryoppippi ryoppippi deleted the docs/remove-meta-tools-implementation-details branch November 18, 2025 16:37
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.

3 participants