-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Rename MCP server tool name: database_* to rtdb_* #9139
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll acknowledge your comments with a 👀 emoji and then get to work. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! I will automatically address your feedback. For any comments you don't want me to act on, just include (aside). For security, I will only act on instructions from the user who triggered this task for this pull request. |
There was a problem hiding this 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 renames the MCP server tool names from database_*
to rtdb_*
to better reflect that these tools are specifically for Firebase Realtime Database functionality.
- Updates the feature name from
database
tortdb
across type definitions, mappings, and configuration files - Renames exported tool collections and directory references to use the new
rtdb
naming convention - Updates documentation to reflect the new tool names (
rtdb_get_data
,rtdb_set_data
,rtdb_get_rules
,rtdb_validate_rules
)
Reviewed Changes
Copilot reviewed 6 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
src/mcp/util.ts | Updates feature API mapping from database to rtdb |
src/mcp/types.ts | Changes feature type definition from database to rtdb |
src/mcp/tools/rtdb/index.ts | Renames exported tool collection from realtimeDatabaseTools to rtdbTools |
src/mcp/tools/index.ts | Updates import and tool mapping to use new rtdb naming |
src/mcp/prompts/index.ts | Updates prompts mapping from database to rtdb |
src/mcp/README.md | Updates documentation table with new tool names and feature group |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
this generally lgtm. We should fire up the MCP inspector to make sure this works e2e also cc @nguymichael just so ur aware about this rename |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG.
Please address merge conflicts and try it out with Gemini CLI or MCP inspector.
Don't forget to run |
This change renames the MCP server tool name from
database_*
tortdb_*
.Specifically, the following tools have been renamed:
database_get_data
->rtdb_get_data
database_set_data
->rtdb_set_data
database_get_rules
->rtdb_get_rules
database_validate_rules
->rtdb_validate_rules
The directory
src/mcp/tools/database
has been renamed tosrc/mcp/tools/rtdb
.The feature name has been changed from
database
tortdb
insrc/mcp/types.ts
,src/mcp/prompts/index.ts
, andsrc/mcp/util.ts
.The
README.md
has been updated to reflect the new tool names and the new feature group name (rtdb
).PR created automatically by Jules for task 826937673060723026