Skip to content

Docs/telemetry mcp tools - #48417

Closed
smeubank wants to merge 9 commits into
masterfrom
docs/telemetry-mcp-tools
Closed

Docs/telemetry mcp tools#48417
smeubank wants to merge 9 commits into
masterfrom
docs/telemetry-mcp-tools

Conversation

@smeubank

@smeubank smeubank commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

I have read the CONTRIBUTING.md file.

YES

What kind of change does this PR introduce?

docs update

What is the current behavior?

the PR will be merged to this one: #48243

once these 2 are merged:

What is the new behavior?

  • documents agent related features that will be useful for the skill we will be shipping

Additional context

Add any other context or screenshots.

Summary by CodeRabbit

  • Documentation
    • Renamed the Telemetry section to Monitoring and Debugging.
    • Reorganized documentation into dedicated Debugging and Monitoring sections.
    • Added troubleshooting guidance for isolating issues and finding product-specific resources.
    • Added curated listings for monitoring and debugging guides.
    • Improved log documentation with links to debugging guidance and advanced filtering tips.

@smeubank
smeubank requested a review from a team as a code owner July 28, 2026 20:17
@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Error Error Jul 31, 2026 10:41am
6 Skipped Deployments
Project Deployment Actions Updated (UTC)
studio Ignored Ignored Jul 31, 2026 10:41am
design-system Skipped Skipped Jul 31, 2026 10:41am
studio-self-hosted Skipped Skipped Jul 31, 2026 10:41am
studio-staging Skipped Skipped Jul 31, 2026 10:41am
ui-library Skipped Skipped Jul 31, 2026 10:41am
zone-www-dot-com Skipped Skipped Jul 31, 2026 10:41am

Request Review

@supabase

supabase Bot commented Jul 28, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project xguihxuzqibwxjnimxev because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Jul 28, 2026
@smeubank
smeubank marked this pull request as draft July 28, 2026 20:18
@smeubank smeubank added the do-not-merge Not ready to be merged yet; pending other dependencies label Jul 28, 2026
@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: b1ccedc7-9659-4b4a-81ac-ac1d524aa078

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

Telemetry documentation is renamed to “Monitoring and Debugging,” with new debugging and monitoring content listings, reorganized navigation, updated debugging and logs guides, and revised landing-page content.

Changes

Monitoring and Debugging documentation

Layer / File(s) Summary
Telemetry content listing definitions and registration
apps/docs/data/content-listings/telemetry.data.ts, apps/docs/data/content-listings/index.ts
Adds debugging and monitoring listing groups and registers them in CONTENT_LISTINGS.
Monitoring and debugging guide content
apps/docs/content/guides/telemetry.mdx, apps/docs/content/guides/telemetry/debugging.mdx, apps/docs/content/guides/telemetry/logs.mdx
Updates the landing page, debugging workflow, product-specific debugging links, log guidance, and section anchors.
Navigation structure update
apps/docs/components/Navigation/NavigationMenu/NavigationMenu.constants.ts
Renames the telemetry navigation section and reorganizes it into Debugging and Monitoring groups.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

Suggested reviewers: jeremenichelli

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is brief and matches the docs/telemetry and MCP tooling focus of the PR, though it is a bit broad.
Description check ✅ Passed The description matches the required template and includes the key sections, with only brief detail in the behavior fields.
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.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/telemetry-mcp-tools

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.

smeubank and others added 9 commits July 31, 2026 12:33
- telemetry.mdx: adds "Start here" decision tree, docs-first architecture note
  in the AI agents section, and expanded chaos-o11y mention
- debugging.mdx: adds MCP tip admonition at the top; adds "For AI agents"
  section at the bottom describing the get_logs-first workflow
- logs.mdx: adds debugging guide back-link at the top; adds advanced-log-
  filtering cross-link at the end of best practices

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
addressing some feedback
display as grid and ensure they display equally in each section with only 2 columns
…s split

this committ assume we will have merged or should be merged to gether with cahnges on skills and MCP

- supabase/agent-skills#112
- supabase/mcp#333

 query_logs (hosted only, ClickHouse SQL) is now the preferred tool for
  production projects; get_logs remains the only option for local and
  self-hosted. Updates the MCP tools table, log-querying section, and skill
  step 3 in ai-agents.mdx, and the tip admonition in debugging.mdx.
  logs.mdx already had this right at line 321.
Removes content that depends on supabase/agent-skills#112 and
supabase/mcp#341 before those PRs have merged:
- Delete ai-agents.mdx (new page documenting query_logs / get_logs / skill)
- Remove MCP tip admonition from debugging.mdx
- Remove telemetry-ai ContentListings panel from overview
- Remove telemetryAI export from telemetry.data.ts
- Remove telemetryAI from content-listings index
- Remove "AI & automation" nav group from NavigationMenu.constants.ts

These changes move to docs/telemetry-mcp-tools which targets this branch
as its base and will not merge until the two external PRs land.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
These changes have been removed from the docs PR, which are dependent on changes to the skills and MCP repo,

Free those changes to be merged, Do not merge these until the others are in
@jeremenichelli
jeremenichelli force-pushed the docs/telemetry-mcp-tools branch from cca4316 to 64529af Compare July 31, 2026 10:40
@blacksmith-sh

blacksmith-sh Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Found 3 test failures on Blacksmith runners:

Failures

Test View Logs
local-smoke/
no-credentials.spec.ts:15:3 › docs dev runs locally without credentials › a normal guid
e page renders and the feedback widget does not crash
View Logs
local-smoke/
no-credentials.spec.ts:28:3 › docs dev runs locally without credentials › database-advi
sors page renders (full content or graceful fallback, never a crash)
View Logs
local-smoke/
no-credentials.spec.ts:39:3 › docs dev runs locally without credentials › a troubleshoo
ting article page renders without Supabase credentials
View Logs

Fix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@jeremenichelli

Copy link
Copy Markdown
Contributor

Closing in favor of docs/telemetry-mcp-tools

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge Not ready to be merged yet; pending other dependencies documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants