Skip to content

docs: improve tool descriptions with review comment summary instructions#172

Merged
ichoosetoaccept merged 1 commit intomainfrom
02-17-docs-improve-tool-descriptions-with-review-comment-summary-instructions
Feb 17, 2026
Merged

docs: improve tool descriptions with review comment summary instructions#172
ichoosetoaccept merged 1 commit intomainfrom
02-17-docs-improve-tool-descriptions-with-review-comment-summary-instructions

Conversation

@ichoosetoaccept
Copy link
Member

@ichoosetoaccept ichoosetoaccept commented Feb 17, 2026

Enhance MCP tool descriptions for list_review_comments, list_stack_review_comments, and summarize_review_status with explicit instructions on how agents should present review data to users.

What changed

Tool docstrings now instruct agents to:

  1. Group comments by file for readability
  2. Classify each by severity (🔴 Bug, 🚩 Flagged, 🟡 Warning, 📝 Info)
  3. Highlight stale threads that can likely be resolved
  4. Show unresolved count and severity breakdown as a quick summary

This is purely a tool description change — no logic changes. The MCP tool description is the natural place to instruct agents on expected behavior.

Fixes #67

@greptile-apps
Copy link

greptile-apps bot commented Feb 17, 2026

Greptile Summary

Enhanced MCP tool docstrings for list_review_comments, list_stack_review_comments, and summarize_review_status with explicit instructions on how AI agents should present review data to users. The changes add structured guidance for grouping comments by file, classifying by severity using emojis (🔴 Bug, 🚩 Flagged, 🟡 Warning, 📝 Info), highlighting stale threads, and showing summary statistics.

Key changes:

  • Added presentation guidelines to tool docstrings instructing agents on output formatting
  • Emoji severity indicators match existing codebase conventions (config.py, models.py, reviewers/devin.py)
  • Removed obsolete .codereviewbuddy.toml file (should have been deleted in PR feat!: env-var-first config, remove TOML config file #162)

This is documentation-only - no logic changes to tool implementations.

Confidence Score: 5/5

  • Safe to merge - documentation-only changes with no logic modifications
  • This PR only modifies docstrings to add presentation instructions for AI agents and removes an obsolete config file. The emoji severity mappings are consistent with existing code in config.py, models.py, and reviewers/devin.py. No runtime behavior is changed.
  • No files require special attention

Important Files Changed

Filename Overview
.codereviewbuddy.toml Removed obsolete TOML config file (replaced by env-var config in PR #162)
src/codereviewbuddy/server.py Enhanced tool docstrings with presentation instructions for review comment summaries

Last reviewed commit: 5573bb0

@ichoosetoaccept ichoosetoaccept force-pushed the 02-17-docs-improve-tool-descriptions-with-review-comment-summary-instructions branch from b908414 to 91149bf Compare February 17, 2026 10:10
@ichoosetoaccept ichoosetoaccept force-pushed the 02-17-feat-check-env-cli-command-for-config-validation branch from f9cb677 to 26242ca Compare February 17, 2026 10:10
@ichoosetoaccept ichoosetoaccept force-pushed the 02-17-docs-improve-tool-descriptions-with-review-comment-summary-instructions branch from 91149bf to f2d854c Compare February 17, 2026 11:17
Copy link
Member Author

ichoosetoaccept commented Feb 17, 2026

Merge activity

  • Feb 17, 11:17 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Feb 17, 11:24 AM UTC: Graphite rebased this pull request as part of a merge.
  • Feb 17, 11:25 AM UTC: @ichoosetoaccept merged this pull request with Graphite.

@ichoosetoaccept ichoosetoaccept changed the base branch from 02-17-feat-check-env-cli-command-for-config-validation to graphite-base/172 February 17, 2026 11:22
@ichoosetoaccept ichoosetoaccept changed the base branch from graphite-base/172 to main February 17, 2026 11:23
@ichoosetoaccept ichoosetoaccept force-pushed the 02-17-docs-improve-tool-descriptions-with-review-comment-summary-instructions branch from f2d854c to 5573bb0 Compare February 17, 2026 11:24
@ichoosetoaccept ichoosetoaccept merged commit 99a5d34 into main Feb 17, 2026
7 checks passed
@ichoosetoaccept ichoosetoaccept deleted the 02-17-docs-improve-tool-descriptions-with-review-comment-summary-instructions branch February 17, 2026 11:25
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.

feat: include review comment summary with importance classification in tool descriptions

1 participant