Skip to content

fix(nextjs-insights): add extraction pattern type and adapt for new SDK versions#112456

Merged
shellmayr merged 4 commits intomasterfrom
shellmayr/ref/nextjs-insights-extraction-patterns
Apr 8, 2026
Merged

fix(nextjs-insights): add extraction pattern type and adapt for new SDK versions#112456
shellmayr merged 4 commits intomasterfrom
shellmayr/ref/nextjs-insights-extraction-patterns

Conversation

@shellmayr
Copy link
Copy Markdown
Member

  • The Next.js SDK >= 10.32.0 changed span descriptions for function.nextjs spans from Page Server Component (/route) to resolve page server component "/route". The tree endpoint regex only matched the old format, causing the SSR File Tree widget to be empty.

  • Add a second regex pattern to handle the new format and normalize both
    to the same component_type + path output.

sergical and others added 3 commits April 7, 2026 21:43
…File Tree

The Next.js SDK >= 10.32.0 changed span descriptions for function.nextjs
spans from `Page Server Component (/route)` to
`resolve page server component "/route"`. The tree endpoint regex only
matched the old format, causing the SSR File Tree widget to be empty.

Add a second regex pattern to handle the new format and normalize both
to the same component_type + path output.
…r typed extraction patterns

Extract `_ExtractionPattern` NamedTuple and replace nested if/else matching
with a loop over `_PATTERNS`. Add unit tests verifying old and new SDK formats
produce identical output.
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 8, 2026
@shellmayr shellmayr requested review from a team and sergical April 8, 2026 09:06
@shellmayr shellmayr marked this pull request as ready for review April 8, 2026 09:06
@shellmayr shellmayr requested review from a team as code owners April 8, 2026 09:06
Copy link
Copy Markdown
Member

@sergical sergical left a comment

Choose a reason for hiding this comment

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

my commits are there, feels wrong to give it a stamp 😅 but the refactor is a lot easier to read and the regex stayed the same, thank you!

@shellmayr shellmayr merged commit 16bf78b into master Apr 8, 2026
56 checks passed
@shellmayr shellmayr deleted the shellmayr/ref/nextjs-insights-extraction-patterns branch April 8, 2026 12:32
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.

3 participants