Skip to content

Add Tools section to all toolset documentation pages#2044

Open
aantn wants to merge 2 commits into
masterfrom
claude/grafana-companion-tools-recommendation-gzId3
Open

Add Tools section to all toolset documentation pages#2044
aantn wants to merge 2 commits into
masterfrom
claude/grafana-companion-tools-recommendation-gzId3

Conversation

@aantn
Copy link
Copy Markdown
Collaborator

@aantn aantn commented May 14, 2026

Summary

This PR adds a standardized "Tools" section to all toolset documentation pages, providing users with a comprehensive reference of available tools for each integration. This improves discoverability and helps users understand what capabilities are exposed by each toolset.

Changes

  • Added Tools sections to 40+ toolset documentation files with tool name and description tables
  • Standardized format across all toolsets using the --8<-- "snippets/toolset_capabilities_intro.md" include for consistency
  • Tool documentation includes:
    • For native toolsets (Kubernetes, Prometheus, Grafana, etc.): Detailed tables with tool names and descriptions
    • For MCP-based toolsets (GitHub, Grafana MCP, Azure MCP, etc.): Summary of common capabilities with links to upstream documentation
    • For database toolsets: Generic tool patterns with templated names (e.g., <toolset_name>_query)
  • Added tip box to Grafana Dashboards documentation recommending related data source toolsets (Prometheus, Loki, Tempo)

Notable Implementation Details

  • MCP-based toolsets include a note that tools are proxied from upstream servers with links to authoritative documentation
  • Database toolsets document the naming convention for dynamically-named tools
  • All sections follow the same structure for consistency and ease of navigation
  • The Grafana Dashboards page includes a helpful tip about pairing with data source toolsets for complete functionality

https://claude.ai/code/session_01S9AGX5yyeiWcdP3G2XSVVm

Summary by CodeRabbit

  • Documentation
    • Added comprehensive "Tools" sections across 40+ toolset documentation files, standardizing how available tools and capabilities are presented.
    • Updated tool descriptions for clarity and consistency across integrations (AKS, ArgoCD, Azure, database, monitoring, and other toolsets).
    • New tool documented: ArgoCD's argocd_app_manifest_source_revision for multi-source application manifests.

Review Change Stack

…n tip

- Add a "Tools" table to each builtin toolset doc enumerating the tools
  HolmesGPT gets from the toolset, sourced from the toolset definitions
  in holmes/plugins/toolsets/.
- For MCP-server toolsets, summarize the capability categories and link
  to the upstream MCP server docs where available.
- In the Grafana Dashboards doc, add a top-of-page tip recommending users
  also enable Prometheus, Loki, and Tempo so HolmesGPT can query the data
  sources behind their dashboards.

Signed-off-by: Claude <noreply@anthropic.com>
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 14, 2026

Deploy Preview for holmes-docs failed. Why did it fail? →

Name Link
🔨 Latest commit 0296fc4
🔍 Latest deploy log https://app.netlify.com/projects/holmes-docs/deploys/6a063a7dfbd08f00086b498d

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 14, 2026

Warning

Rate limit exceeded

@aantn has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 1 minute and 34 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2fa199e6-e0c8-4bca-bd55-a626d057f241

📥 Commits

Reviewing files that changed from the base of the PR and between 3bf9930 and 0296fc4.

📒 Files selected for processing (28)
  • docs/data-sources/builtin-toolsets/aks-node-health.md
  • docs/data-sources/builtin-toolsets/aks.md
  • docs/data-sources/builtin-toolsets/argocd.md
  • docs/data-sources/builtin-toolsets/azure-sql.md
  • docs/data-sources/builtin-toolsets/cilium.md
  • docs/data-sources/builtin-toolsets/confluence-mcp.md
  • docs/data-sources/builtin-toolsets/connectivity-check.md
  • docs/data-sources/builtin-toolsets/github-mcp.md
  • docs/data-sources/builtin-toolsets/grafana-mcp.md
  • docs/data-sources/builtin-toolsets/grafanaloki.md
  • docs/data-sources/builtin-toolsets/grafanatempo.md
  • docs/data-sources/builtin-toolsets/helm.md
  • docs/data-sources/builtin-toolsets/inspektor-gadget.md
  • docs/data-sources/builtin-toolsets/internet.md
  • docs/data-sources/builtin-toolsets/kafka.md
  • docs/data-sources/builtin-toolsets/kubernetes-remediation-mcp.md
  • docs/data-sources/builtin-toolsets/mariadb-mcp.md
  • docs/data-sources/builtin-toolsets/mongodb-atlas.md
  • docs/data-sources/builtin-toolsets/newrelic.md
  • docs/data-sources/builtin-toolsets/notion.md
  • docs/data-sources/builtin-toolsets/prometheus.md
  • docs/data-sources/builtin-toolsets/rabbitmq.md
  • docs/data-sources/builtin-toolsets/robusta.md
  • docs/data-sources/builtin-toolsets/sentry-mcp.md
  • docs/data-sources/builtin-toolsets/servicenow.md
  • docs/data-sources/builtin-toolsets/slab.md
  • docs/data-sources/builtin-toolsets/splunk-mcp.md
  • docs/data-sources/builtin-toolsets/victoriametrics.md

Walkthrough

This pull request standardizes documentation for 47 HolmesGPT builtin-toolset integration pages by adding "## Tools" sections. Each section includes a shared snippet introduction and a table enumerating available tools. Several toolsets refine content—ArgoCD adds a new tool entry, Grafana Dashboards includes a configuration tip, and Docker relabels a heading—while most follow a consistent pattern of tool documentation additions.

Changes

Builtin Toolset Tools Documentation

Layer / File(s) Summary
Database toolsets standard tools section
docs/data-sources/builtin-toolsets/database-clickhouse.md, database-mariadb.md, database-mysql.md, database-postgresql.md, database-sqlite.md, database-sqlserver.md
Database documentation files all add identical "Tools" sections with a shared snippet and tables documenting standard database operations: *_query, *_list_tables, and *_describe_table.
Monitoring and observability toolsets
docs/data-sources/builtin-toolsets/prometheus.md, grafana-mcp.md, grafanaloki.md, grafanatempo.md, victoriametrics.md, victorialogs.md, datadog.md, newrelic.md, zabbix.md
Prometheus-compatible, Grafana, and observability toolsets add "Tools" sections documenting query execution (PromQL, LogQL, TraceQL), metric/log/trace retrieval, dashboard operations, and alert management capabilities.
Container and orchestration toolsets
docs/data-sources/builtin-toolsets/docker.md, cilium.md, openshift.md, crossplane.md, kubevela.md, kubernetes-remediation-mcp.md
Container and Kubernetes toolsets add "Tools" sections with kubectl operations, Cilium/Hubble network inspection, OpenShift cluster management, Crossplane resource configuration, and KubeVela application deployment capabilities.
MCP-based proxy toolsets
docs/data-sources/builtin-toolsets/github-mcp.md, grafana-mcp.md, jenkins-mcp.md, azure-mcp.md, sentry-mcp.md, splunk-mcp.md, prefect-mcp.md, confluence-mcp.md, mariadb-mcp.md
MCP server proxy toolsets add "Tools" sections explaining upstream relationships and enumerating HolmesGPT-exposed capabilities (repository/code access, build logs, error tracking, SQL analysis, workflow inspection).
Cloud and infrastructure toolsets
docs/data-sources/builtin-toolsets/aks.md, aks-node-health.md, azure-sql.md
Azure cloud toolsets add "Tools" sections documenting AKS cluster operations, node health checks, Azure resource inspection, and SQL Database query/performance diagnostics.
Application and data integrations
docs/data-sources/builtin-toolsets/kafka.md, helm.md, internet.md, servicenow.md, robusta.md, slab.md, notion.md, rabbitmq.md, coralogix-logs.md, connectivity-check.md, inspektor-gadget.md
Application-level toolsets add "Tools" sections documenting messaging (Kafka, RabbitMQ), Helm release management, web access, ServiceNow ticket queries, Kubernetes observability, and network diagnostics capabilities.
Content refinements and special cases
docs/data-sources/builtin-toolsets/argocd.md, grafanadashboards.md, docker.md
ArgoCD adds new tool entry and refines description phrasing; Grafana Dashboards adds a configuration tip and tools section; Docker changes heading from "Capabilities" to "Tools".

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • HolmesGPT/holmesgpt#1021: Modifies New Relic toolset documentation (newrelic.md) to add/adjust the "Tools" section and capabilities narrative.
  • HolmesGPT/holmesgpt#1268: Updates Prometheus toolset documentation (prometheus.md) with dedicated "Tools" section and refreshed tools table.
  • HolmesGPT/holmesgpt#778: Updates ArgoCD tools/capabilities tables with argocd_app_diff description revisions and related tool entries.

Suggested labels

codex

Suggested reviewers

  • arikalon1
  • moshemorad
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Add Tools section to all toolset documentation pages' accurately and directly describes the primary objective of this PR, which is to add a standardized Tools section across 40+ toolset documentation files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 5

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (6)
docs/data-sources/builtin-toolsets/grafanaloki.md (1)

129-142: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Clarify relationship between Tools and Capabilities sections.

The "Tools" section lists grafana_loki_query while the "Capabilities" section lists fetch_pod_logs - completely different tools. This creates confusion about which tools are actually available.

If both tools exist, consolidate them into a single section. If grafana_loki_query replaced fetch_pod_logs, remove the outdated "Capabilities" section.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/grafanaloki.md` around lines 129 - 142,
The Tools vs Capabilities sections are inconsistent: the Tools table lists
grafana_loki_query while the Capabilities table lists fetch_pod_logs; reconcile
them by either consolidating into one unified section or removing the outdated
entry. Update the markdown so only the actual available tool(s) are listed
(e.g., keep grafana_loki_query and remove fetch_pod_logs if it was replaced, or
list both with clear descriptions if both exist), and ensure the table headers
and introductory text reference the same set of identifiers (grafana_loki_query,
fetch_pod_logs) so readers are not confused.
docs/data-sources/builtin-toolsets/newrelic.md (1)

119-134: 🛠️ Refactor suggestion | 🟠 Major | ⚡ Quick win

Duplicate Tools and Capabilities sections create maintenance burden.

The "Tools" section (lines 119-127) and "Capabilities" section (lines 128-134) list the same 2 tools with slightly different wording. This duplication increases maintenance effort without adding clarity.

Consider removing the "Capabilities" section since "Tools" now serves as the canonical tool reference.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/newrelic.md` around lines 119 - 134,
Remove the duplicated "Capabilities" section and keep the canonical "Tools"
table as the single reference; delete the "Capabilities" header and its table
that repeats newrelic_execute_nrql_query and newrelic_list_organization_accounts
(and any identical rows) so maintenance is centralized under the "Tools"
section, preserving any unique notes such as the enable_multi_account remark
into the remaining Tools row if needed.
docs/data-sources/builtin-toolsets/grafanatempo.md (1)

330-383: 🛠️ Refactor suggestion | 🟠 Major | ⚡ Quick win

Duplicate Tools and Capabilities sections create maintenance burden.

The "Tools" section (lines 330-344) and "Capabilities" section (lines 372-383) list the same 8 tools with nearly identical descriptions. This duplication requires updating both sections when tools change, increasing maintenance effort and inconsistency risk.

Consider removing the "Capabilities" section since "Tools" now provides the canonical reference.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/grafanatempo.md` around lines 330 - 383,
The file currently duplicates the same list of tools in both the "Tools" section
and the "Capabilities" section (e.g., tempo_fetch_traces_comparative_sample,
tempo_search_traces_by_query, tempo_search_traces_by_tags,
tempo_query_trace_by_id, tempo_search_tag_names, tempo_search_tag_values,
tempo_query_metrics_instant, tempo_query_metrics_range); remove the redundant
"Capabilities" section (the block starting with the second table) and keep the
"Tools" section as the single canonical reference, ensuring the remaining
"Tools" table contains the authoritative descriptions and examples reference the
tool names accordingly.
docs/data-sources/builtin-toolsets/prometheus.md (1)

450-477: 🛠️ Refactor suggestion | 🟠 Major | ⚡ Quick win

Duplicate Tools and Capabilities sections create maintenance burden.

The newly added "Tools" section (lines 450-464) and the existing "Capabilities" section (lines 465-477) list the same 8 tools with overlapping descriptions. Maintaining two near-identical sections increases the risk of inconsistency when updates are made.

Consider removing the "Capabilities" section since the "Tools" section now provides the canonical tool reference, or clearly differentiate their purposes if both are needed.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/prometheus.md` around lines 450 - 477, The
file currently has two near-identical sections headed "Tools" and "Capabilities"
that duplicate the same 8 entries; remove the redundant "## Capabilities"
section (the header and its following table) and keep the canonical "## Tools"
table as the single source of truth, or if you prefer both, change the "##
Capabilities" heading to clarify a different purpose (e.g., usage guidance or
supported operations) and update the content so it differs from the "Tools"
table; locate these by the "Tools" and "Capabilities" headings in the Prometheus
doc and either delete the entire Capabilities table block or replace it with
distinct content.
docs/data-sources/builtin-toolsets/rabbitmq.md (1)

58-77: ⚠️ Potential issue | 🟠 Major | ⚡ Quick win

RabbitMQ tool coverage is contradictory between sections.

Line 58 (Capabilities) and Line 69 (Tools) currently advertise different tool inventories. This creates immediate user-facing ambiguity about available commands. Please keep a single authoritative tools table.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/rabbitmq.md` around lines 58 - 77, The
docs show two conflicting tool tables—keep a single authoritative table and
remove the duplicate/conflicting one; consolidate all unique tools (e.g.,
get_rabbitmq_cluster_status, get_rabbitmq_node_info, get_rabbitmq_queue_info,
get_rabbitmq_exchange_info, get_rabbitmq_memory_usage, get_rabbitmq_disk_usage,
list_configured_clusters) into that one table and ensure the Description text
for get_rabbitmq_cluster_status matches its detailed description (split-brain
detection, nodes/listeners/partitions) so no entries are duplicated or
contradictory between the Capabilities and Tools sections.
docs/data-sources/builtin-toolsets/kafka.md (1)

468-495: ⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Conflicting tool inventories are now published in two sections.

Line 468 (## Capabilities) and Line 480 (## Tools) list different Kafka tool names, which makes the docs ambiguous about the real API surface. Please keep a single canonical tools table and remove the older one.

Suggested cleanup
-## Capabilities
-
---8<-- "snippets/toolset_capabilities_intro.md"
-
-| Tool Name | Description |
-|-----------|-------------|
-| kafka_list_topics | List all Kafka topics |
-| kafka_describe_topic | Get detailed information about a specific topic |
-| kafka_list_consumers | List all consumer groups |
-| kafka_describe_consumer | Get detailed information about a consumer group |
-| kafka_consumer_lag | Check consumer lag for a consumer group |
-
 ## Tools
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/kafka.md` around lines 468 - 495, There
are two conflicting tool tables under the headers "## Capabilities" and "##
Tools"; remove the older/duplicate table so only a single canonical tools
inventory remains (keep the comprehensive table that lists functions like
list_kafka_clusters, list_kafka_consumers, describe_consumer_group, list_topics,
describe_topic, find_consumer_groups_by_topic, consume_messages, list_brokers,
describe_configs, cluster_overview) and delete the other section (the one that
lists kafka_list_topics, kafka_describe_topic, kafka_list_consumers,
kafka_describe_consumer, kafka_consumer_lag) so the docs present one clear API
surface under a single header (e.g., "## Tools").
🧹 Nitpick comments (10)
docs/data-sources/builtin-toolsets/sentry-mcp.md (1)

248-258: ⚡ Quick win

Consolidate the two tools sections.

Line 248 introduces a new tools section, but Line 209 already contains an “Available Tools” table. Keeping only one canonical section will prevent duplication and drift.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/sentry-mcp.md` around lines 248 - 258,
Remove the duplicated "## Tools" section introduced at the bottom and
consolidate its contents into the existing "Available Tools" table earlier in
the doc so there is a single canonical tools reference; specifically, merge the
bullet list items under the "## Tools" heading into the earlier "Available
Tools" table (or replace the table with the bullets if you prefer that format)
and delete the later "## Tools" heading and its bullets to avoid duplication and
drift.
docs/data-sources/builtin-toolsets/splunk-mcp.md (1)

165-175: ⚡ Quick win

Avoid duplicate tools documentation in the same page.

Line 165 adds a second tools section while Line 134 already introduces available tools. Consolidating these into a single section will keep docs consistent and easier to maintain.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/splunk-mcp.md` around lines 165 - 175,
There are duplicate "## Tools" sections; remove this second "## Tools" block and
merge its content (the "Common capabilities exposed to HolmesGPT include:"
bullet list) into the original "## Tools" section already introduced earlier so
the page has one consolidated Tools section; update the earlier "## Tools"
content to include any unique bullets from this block and delete the duplicate
header and paragraphs.
docs/data-sources/builtin-toolsets/github-mcp.md (1)

557-569: ⚡ Quick win

Consolidate duplicate tool sections to one source of truth.

Line 557 introduces a second tools section while Line 446 already documents available tools in detail. Please merge these into one section (or make one section a short pointer) to avoid future drift.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/github-mcp.md` around lines 557 - 569,
There are two "## Tools" sections describing the same GitHub MCP toolset; merge
them so there's a single source of truth: remove the duplicate detailed block
starting with the second "## Tools" and either consolidate its content into the
original "available tools" section or replace the duplicate with a short pointer
linking to that original section and the GitHub MCP server docs; ensure the
header "## Tools" appears only once and that the canonical list (the earlier
"available tools" documentation) remains the authoritative description.
docs/data-sources/builtin-toolsets/confluence-mcp.md (1)

249-259: ⚡ Quick win

Please merge duplicate tool documentation blocks.

Line 249 adds a new tools section, but Line 218 already lists available tools. A single consolidated section will reduce duplication and keep future updates accurate.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/confluence-mcp.md` around lines 249 - 259,
Merge the duplicate "Tools" documentation blocks by removing the redundant block
introduced under the "Tools" header and consolidating its content into the
existing tools/available-tools section (the earlier block that lists available
tools for HolmesGPT); keep a single canonical "Tools" section that includes the
CQL/confluence search, page retrieval (by ID/title), comment reading, and
optional create/update bullets, retain the MCP server link for authoritative
details, and ensure the consolidated section uses the exact descriptive text
from both blocks so no capabilities are lost.
docs/data-sources/builtin-toolsets/azure-sql.md (1)

70-102: ⚡ Quick win

Clarify relationship between "Capabilities" and "Tools" sections.

All three files in this layer now have both a "Capabilities" section and a "Tools" section, creating duplication that may confuse users:

  • aks.md: Lists different tool sets (6 in Capabilities vs. 15 in Tools)
  • aks-node-health.md: Lists different tool sets (6 in Capabilities vs. 9 in Tools)
  • azure-sql.md: Lists the same tools but with brief vs. detailed descriptions

Consider either removing the "Capabilities" section (if "Tools" is meant to replace it) or adding a note explaining how these sections differ and which readers should consult.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/azure-sql.md` around lines 70 - 102, The
docs have duplicated/overlapping "Capabilities" and "Tools" sections across
aks.md, aks-node-health.md, and azure-sql.md which confuses readers; pick one
approach and make the content consistent: either remove the "Capabilities"
section from each file if "Tools" replaces it, or keep both but add a short
explanatory note under the "Capabilities" heading (or at top of the file)
clarifying that "Capabilities" is a brief capability summary for quick scanning
while "Tools" gives detailed tool-by-tool usage and configuration; ensure
azure-sql.md reconciles the brief vs. detailed tool descriptions by moving brief
summaries into "Capabilities" and keeping full examples and parameters in
"Tools" so each section has distinct, non-duplicative content.
docs/data-sources/builtin-toolsets/notion.md (1)

80-94: ⚡ Quick win

Duplicate tables for the same tool should be consolidated.

Line 80 and Line 88 both describe fetch_notion_webpage. Please keep the standardized ## Tools table only, so future edits happen in one place.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/notion.md` around lines 80 - 94, Duplicate
entries for the same tool exist under the "## Capabilities" and "## Tools"
sections; remove the duplicate row under the "## Capabilities" table and keep
the canonical entry under "## Tools" describing fetch_notion_webpage. Locate the
"## Capabilities" heading and the table row that references fetch_notion_webpage
and delete that row (or the whole Capabilities table if it only contains that
duplicate), ensuring only the standardized "## Tools" table contains the
fetch_notion_webpage description; leave the snippet includes
(snippets/toolset_capabilities_intro.md) intact if used elsewhere.
docs/data-sources/builtin-toolsets/robusta.md (1)

57-89: ⚡ Quick win

Avoid maintaining both Capabilities and Tools for the same inventory.

This file now documents the Robusta tools twice (Line 57 and Line 80). Consolidating to the standardized ## Tools section will keep docs consistent with the PR objective and reduce drift.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/robusta.md` around lines 57 - 89, The file
contains duplicated documentation for the same Robusta tools under the "##
Capabilities" heading and again under "## Tools"; remove the redundant "##
Capabilities" block (including its table and the "Multi-Cluster Support" and
"External Changes" subsections) and consolidate any unique, necessary
explanatory text into the standardized "## Tools" section so only the canonical
"## Tools" heading remains with entries for fetch_finding_by_id,
fetch_configuration_changes_metadata, fetch_resource_issues_metadata, and
fetch_resource_recommendation.
docs/data-sources/builtin-toolsets/slab.md (1)

45-59: ⚡ Quick win

Please keep a single standardized tools section.

Line 45 (## Capabilities) and Line 53 (## Tools) duplicate the same fetch_slab_document entry. Keeping only ## Tools avoids redundant maintenance.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/slab.md` around lines 45 - 59, Remove the
duplicate "## Capabilities" section and consolidate the tool entry under a
single standardized "## Tools" section: delete the entire "## Capabilities"
block (including its table and the fetch_slab_document row) and ensure the
existing "## Tools" table contains the canonical fetch_slab_document
description; keep only one table with the tool name fetch_slab_document and its
preferred description to avoid redundant maintenance.
docs/data-sources/builtin-toolsets/internet.md (1)

37-49: ⚡ Quick win

Consolidate duplicate capability/tool tables.

Line 37 (## Capabilities) and Line 43 (## Tools) both document fetch_webpage. Please keep only the standardized ## Tools section to avoid parallel docs.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/internet.md` around lines 37 - 49, Remove
the duplicate "## Capabilities" section and its table that lists fetch_webpage,
keeping only the standardized "## Tools" section; specifically delete the "##
Capabilities" header and its table rows so the file contains a single
authoritative "## Tools" section documenting fetch_webpage (refer to the "##
Capabilities" header, the "## Tools" header, and the fetch_webpage tool name to
locate the duplicated content).
docs/data-sources/builtin-toolsets/helm.md (1)

48-76: ⚡ Quick win

Use one canonical section to avoid doc drift.

Line 48 and Line 63 now duplicate the same Helm tools in two sections (Capabilities and Tools). Keeping only ## Tools will prevent divergence in future edits.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/data-sources/builtin-toolsets/helm.md` around lines 48 - 76, The
document duplicates the same Helm tool list under two headings; remove the
entire "## Capabilities" section and retain only the canonical "## Tools"
section to avoid future drift—specifically delete the duplicated table under the
"## Capabilities" heading and keep the table under "## Tools" (ensure headings
"## Capabilities" and "## Tools" are the identifiers you target).
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/data-sources/builtin-toolsets/argocd.md`:
- Around line 161-177: Remove the duplicated tool table by deleting the older
"## Capabilities" section and keep a single canonical "## Tools" table; ensure
the new row for argocd_app_manifest_source_revision is merged into that retained
"## Tools" table so there is only one tools list (include argocd_app_list,
argocd_app_get, argocd_app_diff, argocd_app_manifests, argocd_app_resources,
argocd_app_manifest_source_revision, argocd_app_history, argocd_repo_list,
argocd_proj_list, argocd_proj_get, argocd_cluster_list) and remove the redundant
table block.

In `@docs/data-sources/builtin-toolsets/cilium.md`:
- Around line 174-215: The new "## Tools" table duplicates and conflicts with
the existing "## Capabilities" section (e.g., entries like hubble_observe have
differing details), so consolidate into a single canonical catalog: remove the
duplicate "## Tools" table or merge its rows into the existing "## Capabilities"
table, reconcile divergent entries (ensure hubble_observe flow count and other
descriptions match the canonical source), and if you must keep the old list mark
it explicitly as "legacy" and link to the canonical table; target the headings
and tool names such as "## Tools", "## Capabilities", and specific tool rows
like "hubble_observe" and "cilium_status" when making the changes.

In `@docs/data-sources/builtin-toolsets/kubernetes-remediation-mcp.md`:
- Around line 237-245: The new "## Tools" section duplicates the existing "##
Available Tools" content; remove one and consolidate into a single authoritative
section (preferably keep the existing "## Available Tools" heading), merge the
tool list/table and the upstream MCP server link into that single section, and
ensure any descriptive bullets (Validated kubectl, temporary pods, retrieving
MCP config) are present under the retained header so there's only one source of
truth (look for the "## Tools" and "## Available Tools" headings to locate the
duplicated content).

In `@docs/data-sources/builtin-toolsets/mongodb-atlas.md`:
- Around line 116-123: The docs list a non-existent tool name
"atlas_return_event_type_from_project"; update the documented tool name to the
implemented plural form "atlas_return_events_type_from_project" so it matches
the function/implementation (atlas_return_events_type_from_project) and prevents
callers from referencing a wrong identifier; search for any other occurrences of
the singular name in the same doc and replace them with the plural form.

In `@docs/data-sources/builtin-toolsets/servicenow.md`:
- Around line 204-205: The documentation hardcodes the ServiceNow endpoints as
/api/now/v2/... but the integration supports a configurable api_version
(including empty/unversioned); update the descriptions for
servicenow_get_records and servicenow_get_record to be version-agnostic by
replacing the fixed /api/now/v2/ path with a placeholder or wording that
reflects the configured api_version (e.g., /api/now/{api_version}/table/... or
“/api/now/<configured api_version or unversioned>/table/...”), so the docs match
the api_version configuration option.

---

Outside diff comments:
In `@docs/data-sources/builtin-toolsets/grafanaloki.md`:
- Around line 129-142: The Tools vs Capabilities sections are inconsistent: the
Tools table lists grafana_loki_query while the Capabilities table lists
fetch_pod_logs; reconcile them by either consolidating into one unified section
or removing the outdated entry. Update the markdown so only the actual available
tool(s) are listed (e.g., keep grafana_loki_query and remove fetch_pod_logs if
it was replaced, or list both with clear descriptions if both exist), and ensure
the table headers and introductory text reference the same set of identifiers
(grafana_loki_query, fetch_pod_logs) so readers are not confused.

In `@docs/data-sources/builtin-toolsets/grafanatempo.md`:
- Around line 330-383: The file currently duplicates the same list of tools in
both the "Tools" section and the "Capabilities" section (e.g.,
tempo_fetch_traces_comparative_sample, tempo_search_traces_by_query,
tempo_search_traces_by_tags, tempo_query_trace_by_id, tempo_search_tag_names,
tempo_search_tag_values, tempo_query_metrics_instant,
tempo_query_metrics_range); remove the redundant "Capabilities" section (the
block starting with the second table) and keep the "Tools" section as the single
canonical reference, ensuring the remaining "Tools" table contains the
authoritative descriptions and examples reference the tool names accordingly.

In `@docs/data-sources/builtin-toolsets/kafka.md`:
- Around line 468-495: There are two conflicting tool tables under the headers
"## Capabilities" and "## Tools"; remove the older/duplicate table so only a
single canonical tools inventory remains (keep the comprehensive table that
lists functions like list_kafka_clusters, list_kafka_consumers,
describe_consumer_group, list_topics, describe_topic,
find_consumer_groups_by_topic, consume_messages, list_brokers, describe_configs,
cluster_overview) and delete the other section (the one that lists
kafka_list_topics, kafka_describe_topic, kafka_list_consumers,
kafka_describe_consumer, kafka_consumer_lag) so the docs present one clear API
surface under a single header (e.g., "## Tools").

In `@docs/data-sources/builtin-toolsets/newrelic.md`:
- Around line 119-134: Remove the duplicated "Capabilities" section and keep the
canonical "Tools" table as the single reference; delete the "Capabilities"
header and its table that repeats newrelic_execute_nrql_query and
newrelic_list_organization_accounts (and any identical rows) so maintenance is
centralized under the "Tools" section, preserving any unique notes such as the
enable_multi_account remark into the remaining Tools row if needed.

In `@docs/data-sources/builtin-toolsets/prometheus.md`:
- Around line 450-477: The file currently has two near-identical sections headed
"Tools" and "Capabilities" that duplicate the same 8 entries; remove the
redundant "## Capabilities" section (the header and its following table) and
keep the canonical "## Tools" table as the single source of truth, or if you
prefer both, change the "## Capabilities" heading to clarify a different purpose
(e.g., usage guidance or supported operations) and update the content so it
differs from the "Tools" table; locate these by the "Tools" and "Capabilities"
headings in the Prometheus doc and either delete the entire Capabilities table
block or replace it with distinct content.

In `@docs/data-sources/builtin-toolsets/rabbitmq.md`:
- Around line 58-77: The docs show two conflicting tool tables—keep a single
authoritative table and remove the duplicate/conflicting one; consolidate all
unique tools (e.g., get_rabbitmq_cluster_status, get_rabbitmq_node_info,
get_rabbitmq_queue_info, get_rabbitmq_exchange_info, get_rabbitmq_memory_usage,
get_rabbitmq_disk_usage, list_configured_clusters) into that one table and
ensure the Description text for get_rabbitmq_cluster_status matches its detailed
description (split-brain detection, nodes/listeners/partitions) so no entries
are duplicated or contradictory between the Capabilities and Tools sections.

---

Nitpick comments:
In `@docs/data-sources/builtin-toolsets/azure-sql.md`:
- Around line 70-102: The docs have duplicated/overlapping "Capabilities" and
"Tools" sections across aks.md, aks-node-health.md, and azure-sql.md which
confuses readers; pick one approach and make the content consistent: either
remove the "Capabilities" section from each file if "Tools" replaces it, or keep
both but add a short explanatory note under the "Capabilities" heading (or at
top of the file) clarifying that "Capabilities" is a brief capability summary
for quick scanning while "Tools" gives detailed tool-by-tool usage and
configuration; ensure azure-sql.md reconciles the brief vs. detailed tool
descriptions by moving brief summaries into "Capabilities" and keeping full
examples and parameters in "Tools" so each section has distinct, non-duplicative
content.

In `@docs/data-sources/builtin-toolsets/confluence-mcp.md`:
- Around line 249-259: Merge the duplicate "Tools" documentation blocks by
removing the redundant block introduced under the "Tools" header and
consolidating its content into the existing tools/available-tools section (the
earlier block that lists available tools for HolmesGPT); keep a single canonical
"Tools" section that includes the CQL/confluence search, page retrieval (by
ID/title), comment reading, and optional create/update bullets, retain the MCP
server link for authoritative details, and ensure the consolidated section uses
the exact descriptive text from both blocks so no capabilities are lost.

In `@docs/data-sources/builtin-toolsets/github-mcp.md`:
- Around line 557-569: There are two "## Tools" sections describing the same
GitHub MCP toolset; merge them so there's a single source of truth: remove the
duplicate detailed block starting with the second "## Tools" and either
consolidate its content into the original "available tools" section or replace
the duplicate with a short pointer linking to that original section and the
GitHub MCP server docs; ensure the header "## Tools" appears only once and that
the canonical list (the earlier "available tools" documentation) remains the
authoritative description.

In `@docs/data-sources/builtin-toolsets/helm.md`:
- Around line 48-76: The document duplicates the same Helm tool list under two
headings; remove the entire "## Capabilities" section and retain only the
canonical "## Tools" section to avoid future drift—specifically delete the
duplicated table under the "## Capabilities" heading and keep the table under
"## Tools" (ensure headings "## Capabilities" and "## Tools" are the identifiers
you target).

In `@docs/data-sources/builtin-toolsets/internet.md`:
- Around line 37-49: Remove the duplicate "## Capabilities" section and its
table that lists fetch_webpage, keeping only the standardized "## Tools"
section; specifically delete the "## Capabilities" header and its table rows so
the file contains a single authoritative "## Tools" section documenting
fetch_webpage (refer to the "## Capabilities" header, the "## Tools" header, and
the fetch_webpage tool name to locate the duplicated content).

In `@docs/data-sources/builtin-toolsets/notion.md`:
- Around line 80-94: Duplicate entries for the same tool exist under the "##
Capabilities" and "## Tools" sections; remove the duplicate row under the "##
Capabilities" table and keep the canonical entry under "## Tools" describing
fetch_notion_webpage. Locate the "## Capabilities" heading and the table row
that references fetch_notion_webpage and delete that row (or the whole
Capabilities table if it only contains that duplicate), ensuring only the
standardized "## Tools" table contains the fetch_notion_webpage description;
leave the snippet includes (snippets/toolset_capabilities_intro.md) intact if
used elsewhere.

In `@docs/data-sources/builtin-toolsets/robusta.md`:
- Around line 57-89: The file contains duplicated documentation for the same
Robusta tools under the "## Capabilities" heading and again under "## Tools";
remove the redundant "## Capabilities" block (including its table and the
"Multi-Cluster Support" and "External Changes" subsections) and consolidate any
unique, necessary explanatory text into the standardized "## Tools" section so
only the canonical "## Tools" heading remains with entries for
fetch_finding_by_id, fetch_configuration_changes_metadata,
fetch_resource_issues_metadata, and fetch_resource_recommendation.

In `@docs/data-sources/builtin-toolsets/sentry-mcp.md`:
- Around line 248-258: Remove the duplicated "## Tools" section introduced at
the bottom and consolidate its contents into the existing "Available Tools"
table earlier in the doc so there is a single canonical tools reference;
specifically, merge the bullet list items under the "## Tools" heading into the
earlier "Available Tools" table (or replace the table with the bullets if you
prefer that format) and delete the later "## Tools" heading and its bullets to
avoid duplication and drift.

In `@docs/data-sources/builtin-toolsets/slab.md`:
- Around line 45-59: Remove the duplicate "## Capabilities" section and
consolidate the tool entry under a single standardized "## Tools" section:
delete the entire "## Capabilities" block (including its table and the
fetch_slab_document row) and ensure the existing "## Tools" table contains the
canonical fetch_slab_document description; keep only one table with the tool
name fetch_slab_document and its preferred description to avoid redundant
maintenance.

In `@docs/data-sources/builtin-toolsets/splunk-mcp.md`:
- Around line 165-175: There are duplicate "## Tools" sections; remove this
second "## Tools" block and merge its content (the "Common capabilities exposed
to HolmesGPT include:" bullet list) into the original "## Tools" section already
introduced earlier so the page has one consolidated Tools section; update the
earlier "## Tools" content to include any unique bullets from this block and
delete the duplicate header and paragraphs.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f01b70b2-2201-4bcb-95b4-d6a3cd29153c

📥 Commits

Reviewing files that changed from the base of the PR and between 31fa24c and 3bf9930.

📒 Files selected for processing (46)
  • docs/data-sources/builtin-toolsets/aks-node-health.md
  • docs/data-sources/builtin-toolsets/aks.md
  • docs/data-sources/builtin-toolsets/argocd.md
  • docs/data-sources/builtin-toolsets/azure-mcp.md
  • docs/data-sources/builtin-toolsets/azure-sql.md
  • docs/data-sources/builtin-toolsets/cilium.md
  • docs/data-sources/builtin-toolsets/confluence-mcp.md
  • docs/data-sources/builtin-toolsets/connectivity-check.md
  • docs/data-sources/builtin-toolsets/coralogix-logs.md
  • docs/data-sources/builtin-toolsets/crossplane.md
  • docs/data-sources/builtin-toolsets/database-clickhouse.md
  • docs/data-sources/builtin-toolsets/database-mariadb.md
  • docs/data-sources/builtin-toolsets/database-mysql.md
  • docs/data-sources/builtin-toolsets/database-postgresql.md
  • docs/data-sources/builtin-toolsets/database-sqlite.md
  • docs/data-sources/builtin-toolsets/database-sqlserver.md
  • docs/data-sources/builtin-toolsets/datadog.md
  • docs/data-sources/builtin-toolsets/docker.md
  • docs/data-sources/builtin-toolsets/github-mcp.md
  • docs/data-sources/builtin-toolsets/grafana-mcp.md
  • docs/data-sources/builtin-toolsets/grafanadashboards.md
  • docs/data-sources/builtin-toolsets/grafanaloki.md
  • docs/data-sources/builtin-toolsets/grafanatempo.md
  • docs/data-sources/builtin-toolsets/helm.md
  • docs/data-sources/builtin-toolsets/inspektor-gadget.md
  • docs/data-sources/builtin-toolsets/internet.md
  • docs/data-sources/builtin-toolsets/jenkins-mcp.md
  • docs/data-sources/builtin-toolsets/kafka.md
  • docs/data-sources/builtin-toolsets/kubernetes-remediation-mcp.md
  • docs/data-sources/builtin-toolsets/kubevela.md
  • docs/data-sources/builtin-toolsets/mariadb-mcp.md
  • docs/data-sources/builtin-toolsets/mongodb-atlas.md
  • docs/data-sources/builtin-toolsets/newrelic.md
  • docs/data-sources/builtin-toolsets/notion.md
  • docs/data-sources/builtin-toolsets/openshift.md
  • docs/data-sources/builtin-toolsets/prefect-mcp.md
  • docs/data-sources/builtin-toolsets/prometheus.md
  • docs/data-sources/builtin-toolsets/rabbitmq.md
  • docs/data-sources/builtin-toolsets/robusta.md
  • docs/data-sources/builtin-toolsets/sentry-mcp.md
  • docs/data-sources/builtin-toolsets/servicenow.md
  • docs/data-sources/builtin-toolsets/slab.md
  • docs/data-sources/builtin-toolsets/splunk-mcp.md
  • docs/data-sources/builtin-toolsets/victorialogs.md
  • docs/data-sources/builtin-toolsets/victoriametrics.md
  • docs/data-sources/builtin-toolsets/zabbix.md

Comment thread docs/data-sources/builtin-toolsets/argocd.md
Comment thread docs/data-sources/builtin-toolsets/cilium.md
Comment thread docs/data-sources/builtin-toolsets/kubernetes-remediation-mcp.md Outdated
Comment thread docs/data-sources/builtin-toolsets/mongodb-atlas.md
Comment thread docs/data-sources/builtin-toolsets/servicenow.md Outdated
- Remove stale "## Capabilities" tables that duplicated (and sometimes
  contradicted) the new "## Tools" tables added in the previous commit.
- Rename "## Available Tools" to "## Tools" in MCP toolset docs so all
  builtin toolset pages use a consistent heading.
- Fix mongodb-atlas tool name and make servicenow Tools descriptions
  api_version-agnostic.
- Preserve robusta's Multi-Cluster Support / External Changes notes as
  subsections under Tools.

Signed-off-by: Claude <noreply@anthropic.com>
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.

2 participants