feat(dashboards): Add dashboard catalog tools#1079
Merged
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit b53b02d. Configure here.
Add catalog tools to find dashboards and inspect saved dashboard details. Validate dashboard API responses against schemas confirmed from Sentry, resolve active project constraints to numeric project IDs, and filter dashboard visibility locally because the dashboard list API does not support project scoping. Generated dashboard links now use the canonical dashboard route and include project and statsPeriod query parameters when available. Fixes #1073 Co-Authored-By: GPT-5 Codex <codex@openai.com>
dcramer
force-pushed
the
feat/dashboard-catalog-tools
branch
from
b53b02d to
6215193
Compare
Project-constrained dashboard searches filtered each Sentry API page locally and could stop before later visible dashboards. Preserve Sentry's upstream cursor page size and use an MCP-owned cursor to resume within a filtered page when needed. Reject MCP-owned project dashboard cursors in org-wide searches before they reach Sentry, and document that dashboard cursors must stay within the same search scope and project constraint. Add regression coverage for empty filtered pages, overflow within a visible page, first-page cursor resume, and cross-scope cursor rejection. Co-Authored-By: GPT-5 Codex <noreply@openai.com>
dcramer
force-pushed
the
feat/dashboard-catalog-tools
branch
from
6215193 to
e023ca0
Compare
![sentry-warden[bot]](https://avatars.githubusercontent.com/in/2755838?s=60&v=4){kind=small}
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds catalog tools for finding dashboards and inspecting saved dashboard details. The implementation validates dashboard list/detail payloads against schemas confirmed from Sentry, uses MSW dashboard fixtures, and keeps project-constrained sessions scoped by resolving the constrained project slug to a numeric ID before filtering dashboard project filters locally.
Dashboard links now use the canonical
/dashboard/:id/route and carryprojectandstatsPeriodquery parameters when available, matching the user-facing Sentry route while leaving API requests aligned with the dashboard endpoints.Fixes #1073