Skip to content

Comments

Modified README.md to include all listed tools#146

Open
Skeftical wants to merge 1 commit intooracle:mainfrom
Skeftical:145-fix-readme-issue
Open

Modified README.md to include all listed tools#146
Skeftical wants to merge 1 commit intooracle:mainfrom
Skeftical:145-fix-readme-issue

Conversation

@Skeftical
Copy link
Member

Description

Modified README.md to include all tools implemented in server.py under “MySQL AI and MySQL HeatWave ML and GenAI Tools”.

Added tools:

  • ask_ml_rag_vector_store — Retrieve segments from the default vector store (skip_generate)
  • ask_ml_rag_innodb — Retrieve segments from InnoDB tables using specified segment and embedding columns
  • retrieve_relevant_schema_information — Retrieve relevant schemas and tables (DDL) for a natural language question

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Feb 13, 2026
4. `ragify_column(connection_id, table, input_col, embedding_col)`: Embed text into a VECTOR column
5. `list_vector_store_files_local(connection_id)`: List available files in `secure_file_priv`
6. `load_vector_store_local(connection_id, file_path)`: Load documents from local filesystem
7. `load_vector_store_oci(connection_id, namespace, bucket, prefix, schema, table)`: Load documents from OCI Object Storage
Copy link
Member

Choose a reason for hiding this comment

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

unrelated to this change, but the schema for this tool doesn't appear to match the code. From server.py:

@mcp.tool()
def load_vector_store_oci(
    connection_id: str,
    namespace: str,
    bucket_name: str,
    document_prefix: str,
) -> str:

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks for highlighting this. I have made a follow-up modification to reflect this.

Signed-off-by: fsavva <fotis.savva@oracle.com>
@Skeftical Skeftical force-pushed the 145-fix-readme-issue branch from 9e889ff to 553ad34 Compare February 18, 2026 10:29
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.

3 participants