Skip to content

Document Code Origin coverage via Source Code Integration#38434

Open
flushentitypacket wants to merge 19 commits into
masterfrom
flushentitypacket/code-origin-sci-coverage
Open

Document Code Origin coverage via Source Code Integration#38434
flushentitypacket wants to merge 19 commits into
masterfrom
flushentitypacket/code-origin-sci-coverage

Conversation

@flushentitypacket

@flushentitypacket flushentitypacket commented Jul 23, 2026

Copy link
Copy Markdown

What does this PR do? What is the motivation?

Documents Code Origin coverage via Source Code Integration.

  • Adds a Go tab to the compatibility requirements table
  • Adds a "Code Origin coverage via Source Code Integration" section

NOTE: Do not merge until completing GA for Code Intelligence for Traces.

Merge readiness

  • Ready for merge

For Datadog employees:

AI assistance

Drafted and iterated on with Claude Code.

Additional notes

flushentitypacket and others added 10 commits July 22, 2026 17:21
Adds coverage of the new static-analysis-based Code Origin attribution
path (via Source Code Integration), which requires no SDK flag and is
the primary source of Code Origin data for Go and for additional
Python/Node.js frameworks not yet covered by tracer instrumentation.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Use product-facing terminology (Source Code Integration, with GitHub
as the example provider) instead of the internal "Code Intelligence"
team name, matching existing docs conventions (error_tracking,
flaky_management, api_endpoints).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
… section

Replaces the scattered mentions across Overview, Prerequisites,
compatibility tabs, Enable Code Origin, and Troubleshooting with a
single Alternative Setup section, linked from Getting Started and
Troubleshooting for readers whose service isn't covered by the
default tracer-instrumented setup.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Source Code Integration is already a required prerequisite for Code
Origin (used for code previews), so it isn't a separate setup path.
Reframe the section to describe what SCI alone already covers
automatically, and what enabling the SDK flag adds on top.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Source Code Integration is a general prerequisite for Code Origin,
not just for code previews.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Per CONTRIBUTING.md: avoid temporal words like "already", and use
em dashes with no surrounding spaces (rewritten as separate sentences
instead).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Comment thread content/en/tracing/code_origin/_index.md Outdated
Co-authored-by: sstonehill12 <sarah.stonehill@datadoghq.com>
Comment thread content/en/tracing/code_origin/_index.md Outdated
@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

flushentitypacket and others added 2 commits July 23, 2026 12:55
Co-authored-by: sstonehill12 <sarah.stonehill@datadoghq.com>
Replaces "via" with "through" per Datadog.words_case_insensitive,
renames the Source Code Integration section heading to match, updates
all anchor links accordingly, and splits/tightens the Prerequisites
callout to stay under the 25-word sentence-length guideline.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- For transpiled Node.js applications (for example, TypeScript), make sure to generate and publish source maps with the deployed application, run Node.js with the [`--enable-source-maps`][10] flag, and use v5.59.0 or newer of the Node.js tracer. Otherwise, code previews will not work. See the Node.js [Source Code Integration][9] documentation for more details.
- Code Origin is designed to reference user code only, but in some cases, third-party code references may slip through. You can report these cases to [Datadog support][13] and help improve these references.

## Code Origin coverage through Source Code Integration

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Leaving this heading as-is intentionally. Vale flags it because only the first word is capitalized under strict sentence-case, but "Code Origin" and "Source Code Integration" are treated as proper feature names capitalized throughout this page (e.g. "Using Code Origin", "Enable Code Origin"). Lowercasing them here would be inconsistent with the rest of the doc, so keeping the capitalization for consistency.

flushentitypacket and others added 3 commits July 23, 2026 14:56
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The {{% tabs %}} shortcode renders each tab's content in its own
isolated pass, so it can't resolve reference-style links defined at
the bottom of the page. Use an inline link instead, matching the
pattern already used elsewhere in the same tab.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@flushentitypacket
flushentitypacket marked this pull request as ready for review July 23, 2026 20:11
@flushentitypacket
flushentitypacket requested a review from a team as a code owner July 23, 2026 20:11

@maycmlee maycmlee 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.

Just a couple of small suggestions, also waiting for the Preview to be available to check how the changes look in the doc.

Comment thread content/en/tracing/code_origin/_index.md Outdated
Comment thread content/en/tracing/code_origin/_index.md Outdated
flushentitypacket and others added 3 commits July 23, 2026 17:55
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: May Lee <may.lee@datadoghq.com>
Co-authored-by: May Lee <may.lee@datadoghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Do Not Merge Just do not merge this PR :)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants