Docs/telemetry mcp tools - #48417
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
6 Skipped Deployments
|
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughTelemetry 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. ChangesMonitoring and Debugging documentation
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
- 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
…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
cca4316 to
64529af
Compare
|
Found 3 test failures on Blacksmith runners: Failures
|
|
Closing in favor of docs/telemetry-mcp-tools |
![Fix with [code]smith](https://pr-comments-assets.blacksmith.sh/codesmith/fix-with-codesmith-light.png)
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?
Additional context
Add any other context or screenshots.
Summary by CodeRabbit