Skip to content

Conversation

chargome
Copy link
Member

@chargome chargome commented Aug 8, 2025

No description provided.

betegon and others added 10 commits August 6, 2025 11:19
…calls (#17284)

closes #17283

includes these attributes for `mcp.server` spans:
- `mcp.prompt.result.description`
- `mcp.prompt.result.message_content`
- `mcp.prompt.result.message_role`
- `mcp.prompt.result.message_count`

Example: 
<img width="835" height="300" alt="Screenshot 2025-08-01 at 12 40 46"
src="https://github.com/user-attachments/assets/592d876b-807a-4f3e-a9b2-406e10f5a83d"
/>

Needed to make `attributeExtraction.ts` <300 lines of code (requirement)
so it's now split between `sessionExtraction.ts`, `sessionExtraction.ts`
and `resultExtraction.ts`.

So changes explained so it's easier to review:
- The only function this PR adds is `extractPromptResultAttributes`
inside `resultExtraction.ts`.
- It adds the prompt results as PII in `piiFiltering.ts`. Just add them
to the `set`.
- adds a `else if (method === 'prompts/get')` to execute the
`extractPromptResultAttributes` function.
- adds a test that checks we're capturing the results and updates the
PII test to check PII result attributes are being removed if sending PII
is not enabled.
[Gitflow] Merge master into develop
Commits like `meta: Re-organize changelog to add v8 page` tripped this
up. Our changelog entries always follow `meta(changelog)` so we should
just check for that.
This streamlines the build logs for sveltekit. For this, I aligned how
we handle source maps setting with other SDKs, which is a bit different
then how sveltekit did it.

1. Only show log messages when `debug: true`
2. Show short warning when `sourceMaps: false` and `debug: false`, more
details when `debug: true` is set
3. Re-organize how `filesToDeleteAfterUpload` is set, based on other
SDKs (?? I hope that makes sense....)

---------

Co-authored-by: Abhijeet Prasad <[email protected]>
…CLI fails (#17081)

Sentry CLI's `cli.releases.uploadSourceMaps` method previously never
rejected when the actual CLI binary execution exited with an error code.
In CLI 2.49.0 and 2.50.0 I added a new execution mode variant
(`rejectOnError`) which continues to pipe stdio to the process (the RR
SDKs' upload script) but now also rejects on error.

This patch bumps Sentry CLI and configures it to actually reject now. We
already catch the rejection and fail silently. Nothing changed here.
…egration` (#17216)

[#17218](#17218)
adjusted data URI stack line parsing for most errors that go through the
stack parsers (fully for node, line truncation in general). This patch now
applies a similar logic to `globalHandlersIntegration` which in some
conditions applies a stack frame that doesn't go through the same stack
parser.
As raised in #16969, the Bun SDK doesn't currently export
`skipOpenTelemetrySetup` in the SDK init options type, altghough the
option can be correctly passed to the underlying `NodeClient`. This patch
exports the option now.
- Increase the threshold of inactivity between potential navigations
from 300ms to 1.5s
- Change how we treat interactions:
- Previously, we'd take interactions happening _before_ the last root
span was started into account
- Now, we reset the interaction timestamp when we start a new navigation
_root_ span. So from that moment on, we'll consider every new navigation
span a redirect iff:
    - no interaction happened within 1.5s (from root span start)
    - the new span was started within 1.5s
    - there is still an active root span
@chargome chargome self-assigned this Aug 8, 2025
Copy link
Contributor

github-actions bot commented Aug 8, 2025

size-limit report 📦

Path Size % Change Change
@sentry/browser 24.11 kB added added
@sentry/browser - with treeshaking flags 22.68 kB added added
@sentry/browser (incl. Tracing) 39.83 kB added added
@sentry/browser (incl. Tracing, Replay) 77.92 kB added added
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 67.77 kB added added
@sentry/browser (incl. Tracing, Replay with Canvas) 82.63 kB added added
@sentry/browser (incl. Tracing, Replay, Feedback) 94.73 kB added added
@sentry/browser (incl. Feedback) 40.77 kB added added
@sentry/browser (incl. sendFeedback) 28.77 kB added added
@sentry/browser (incl. FeedbackAsync) 33.66 kB added added
@sentry/react 25.83 kB added added
@sentry/react (incl. Tracing) 41.8 kB added added
@sentry/vue 28.59 kB added added
@sentry/vue (incl. Tracing) 41.64 kB added added
@sentry/svelte 24.14 kB added added
CDN Bundle 25.62 kB added added
CDN Bundle (incl. Tracing) 39.72 kB added added
CDN Bundle (incl. Tracing, Replay) 75.77 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback) 81.19 kB added added
CDN Bundle - uncompressed 74.89 kB added added
CDN Bundle (incl. Tracing) - uncompressed 117.52 kB added added
CDN Bundle (incl. Tracing, Replay) - uncompressed 231.78 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 244.38 kB added added
@sentry/nextjs (client) 43.83 kB added added
@sentry/sveltekit (client) 40.27 kB added added
@sentry/node-core 47.73 kB added added
@sentry/node 146.62 kB added added
@sentry/node - without tracing 91.83 kB added added
@sentry/aws-serverless 103.27 kB added added

@chargome chargome force-pushed the prepare-release/10.3.0 branch from 6ffdb77 to f0661fc Compare August 8, 2025 11:05
@chargome chargome merged commit dd83160 into master Aug 8, 2025
183 checks passed
@chargome chargome deleted the prepare-release/10.3.0 branch August 8, 2025 11:20
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.

6 participants