FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(mcp): modify ask_codebase prompt to require explicit call out by msukkari · Pull Request #995 · sourcebot-dev/sourcebot · GitHub

/ sourcebot Public

fix(mcp): modify ask_codebase prompt to require explicit call out - #995

Merged
msukkari merged 3 commits into
mainfrom
msukkari/ask_subagent_gate
Mar 11, 2026
ConversationCommits (3)ChecksFiles changed
Merged

fix(mcp): modify ask_codebase prompt to require explicit call out#995
msukkari merged 3 commits into
mainfrom
msukkari/ask_subagent_gate

Conversation

msukkari commented Mar 11, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

As ask_codebase is still experimental, let's require the prompt to explicitly ask for this tool call to get it invoked

Summary by CodeRabbit

Release Notes

  • Documentation

    • Updated documentation to clarify that the ask_codebase tool requires explicit invocation in MCP.

  • Chores

    • Reorganized internal tool descriptions for improved maintainability.

msukkari requested a review from brendan-kellam March 11, 2026 22:43

This comment has been minimized.

Sign in to view

coderabbitai Bot commented Mar 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info ⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 88df5104-bbf5-4125-a8a9-a16169e90c78

📥 Commits

Reviewing files that changed from the base of the PR and between f0cef5e and 6616158.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • packages/web/src/features/mcp/server.ts

Walkthrough

Documents that the ask_codebase tool must be invoked explicitly in MCP and centralizes MCP tool description strings into a single TOOL_DESCRIPTIONS object used by multiple tool registrations.

Changes

Cohort / File(s) Summary
Changelog
CHANGELOG.md		 Added Unreleased → Changed entry requiring explicit invocation of the ask_codebase tool in MCP.
MCP Tool Descriptions
packages/web/src/features/mcp/server.ts		 Introduced a centralized TOOL_DESCRIPTIONS object and updated tool registrations (search_code, list_commits, list_repos, read_file, list_tree, list_language_models, ask_codebase) to use descriptions from this object instead of inline strings.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • brendan-kellam
✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch msukkari/ask_subagent_gate

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

msukkari merged commit aaf0b1e into main Mar 11, 2026
6 checks passed
github-actions Bot mentioned this pull request Mar 11, 2026
Open
This was referenced Mar 17, 2026
Merged
Merged
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

brendan-kellam brendan-kellam approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL