Skip to content

docs(godot): Add Source Context page for script frames#18393

Merged
limbonaut merged 8 commits into
masterfrom
limbonaut/docs/godot-source-context
Jun 15, 2026
Merged

docs(godot): Add Source Context page for script frames#18393
limbonaut merged 8 commits into
masterfrom
limbonaut/docs/godot-source-context

Conversation

@limbonaut

@limbonaut limbonaut commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

DESCRIBE YOUR PR

Sentry can now fetch source on demand from a connected SCM integration and show it around stack trace frames — for Godot users, this is the first way to get GDScript source context without exporting scripts as text. This PR adds a Source Context page to the Godot SDK docs covering both options for GDScript and C# (including the res:// → project-directory code mapping), and cross-links it from Readable Stack Traces, which keeps covering native (C/C++) frames.

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.

  • Urgent deadline (GA date, etc.):
  • Other deadline:
  • None: Not urgent, can wait up to 1 week+

PRE-MERGE CHECKLIST

Make sure you've checked the following before merging your changes:

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

🤖 Generated with Claude Code

@vercel

vercel Bot commented Jun 12, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
sentry-docs Ready Ready Preview, Comment Jun 15, 2026 10:34am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored Preview Jun 15, 2026 10:34am

Request Review

Sentry can now fetch source on demand from a connected SCM integration
and display it around stack trace frames. For GDScript this previously
required exporting projects with scripts as text. Document both options
for GDScript and C#, including res:// code mapping guidance, and
cross-link from the Readable Stack Traces page, which keeps covering
native (C/C++) frames.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
limbonaut and others added 3 commits June 12, 2026 20:08
Move the language-scope claim into the SCM fetching section where it
applies, clarify that native frames require symbolication to carry
file paths and line numbers, tell the story from the C# perspective,
and document source bundle uploads for compiled code (C# and native).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The guide covers compiled code — native engine, GDExtension (C/C++),
and C# — so the cross-link Alert shouldn't omit managed code.
@limbonaut limbonaut force-pushed the limbonaut/docs/godot-source-context branch from e383f0b to b7f024e Compare June 12, 2026 18:30
@limbonaut limbonaut marked this pull request as ready for review June 12, 2026 18:48

@JoshuaMoelans JoshuaMoelans left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice! Just one suggestion + one question, but nothing blocking

Comment thread docs/platforms/godot/configuration/source-context.mdx Outdated
Comment thread docs/platforms/godot/configuration/source-context.mdx Outdated
Co-authored-by: JoshuaMoelans <60878493+JoshuaMoelans@users.noreply.github.com>
@codeowner-assignment codeowner-assignment Bot requested a review from a team June 15, 2026 09:53

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

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 90de47a. Configure here.

Comment thread docs/platforms/godot/configuration/source-context.mdx Outdated
Comment thread docs/platforms/godot/configuration/source-context.mdx Outdated
Comment thread docs/platforms/godot/configuration/source-context.mdx Outdated

@JoshuaMoelans JoshuaMoelans left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🚀

@limbonaut limbonaut merged commit 46ae6ff into master Jun 15, 2026
21 checks passed
@limbonaut limbonaut deleted the limbonaut/docs/godot-source-context branch June 15, 2026 15:23
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