Skip to content

ref(seer): Capitalize node type headings in markdown renderer#112398

Merged
Mihir-Mavalankar merged 2 commits intomasterfrom
mihir-mavalankar/ref/capitalize-llm-node-headings
Apr 7, 2026
Merged

ref(seer): Capitalize node type headings in markdown renderer#112398
Mihir-Mavalankar merged 2 commits intomasterfrom
mihir-mavalankar/ref/capitalize-llm-node-headings

Conversation

@Mihir-Mavalankar
Copy link
Copy Markdown
Contributor

  • Apply .capitalize() to nodeType in _render_node so markdown headings render as "# Dashboard", "## Widget" instead of lowercase.

Apply .capitalize() to nodeType in _render_node so markdown headings
render as "# Dashboard", "## Widget" instead of lowercase.

Co-Authored-By: Claude Opus 4.6 <noreply@example.com>
@Mihir-Mavalankar Mihir-Mavalankar self-assigned this Apr 7, 2026
@Mihir-Mavalankar Mihir-Mavalankar marked this pull request as ready for review April 7, 2026 19:37
@Mihir-Mavalankar Mihir-Mavalankar requested a review from a team as a code owner April 7, 2026 19:37
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 7, 2026
Comment thread src/sentry/seer/explorer/client_utils.py Outdated
Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 1c5c6f5. Configure here.

Comment thread src/sentry/seer/explorer/client_utils.py Outdated
Comment thread src/sentry/seer/explorer/client_utils.py Outdated
Use `or 'unknown'` instead of dict.get default to handle explicit
None values, not just missing keys.

Co-Authored-By: Claude Opus 4.6 <noreply@example.com>
@Mihir-Mavalankar Mihir-Mavalankar enabled auto-merge (squash) April 7, 2026 19:43
@Mihir-Mavalankar Mihir-Mavalankar merged commit 2caf618 into master Apr 7, 2026
58 checks passed
@Mihir-Mavalankar Mihir-Mavalankar deleted the mihir-mavalankar/ref/capitalize-llm-node-headings branch April 7, 2026 19:51
george-sentry pushed a commit that referenced this pull request Apr 9, 2026
+ Apply .capitalize() to nodeType in _render_node so markdown headings
render as "# Dashboard", "## Widget" instead of lowercase.

---------

Co-authored-by: Claude Opus 4.6 <noreply@example.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants