-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
meta(changelog): Update changelog for 9.40.0 #17039
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
In https://github.com/vercel/ai/releases/tag/ai%404.3.17 of the `ai` SDK, provider metadata got exposed (vercel/ai@a288694) This is included in the `ai.response.providerMetadata` property. To get more detailed, info, we parse `ai.response.providerMetadata` and attach attributes as appropriate. Still need some help with the naming. resolves https://linear.app/getsentry/issue/JS-661/add-support-for-cached-and-reasoning-tokens
Instead, we can just use the parameter-less version of try-catch. In future versions of eslint, this will be disallowed, so extracting this out.
A future version of eslint flagged & auto-fixed these, extracting this out for easier review.
resolves #16943 Just give a quick double check to `packages/opentelemetry/test/helpers/initOtel.ts` to make sure it makes sense.
) Fix #16786 + e2e tests for cloudflare workers - [ ] If you've added code that should be tested, please add tests. - [x] Ensure your code lints and the test suite passes (`yarn lint`) & (`yarn test`). --------- Co-authored-by: cod1k <cod1k@centro.team>
resolves #16949
resolves #16955
resolves #16954
resolves #16950
trying to un-stupidify cursor in some cases, esp. background tasks.
140701a
to
6572f52
Compare
size-limit report 📦
|
resolves #16905 also renames the file to be `debug-logger.ts` instead of `logger.ts` to make usage inside `@sentry/core` a little easier to understand. Added a note to `MIGRATION.md`, but let me know if I should remove that. --------- Co-authored-by: Francesco Gringl-Novy <francesco.novy@sentry.io>
merged in #17040 |
…ad (#16981) Add two Browser SDK APIs to let the main thread know about debugIds of worker files: - `webWorkerIntegration({worker})` to be used in the main thread - `registerWebWorker(self)` to be used in the web worker - unit tests for the two APIs - integration test testing the `webWorkerIntegration` - e2e test demonstrating correct usage of both APIs together
…#17044) Make sure we use of Cloudflare's `waitUntil` function which fortunately is available within the error handler hook. This fixes error reporting for streamed server responses on Clourdlfare, where the returned promise rejects after the initial response was sent.
oven-sh/bun#5091 has been completed for some time
This PR adds the external contributor to the CHANGELOG.md file, so that they are credited for their contribution. See #17049 Co-authored-by: Lms24 <8420481+Lms24@users.noreply.github.com>
#16790) The error code 137 is a very common error when running E2E tests. Starting colima only with `colima start` often results in too little memory and it makes sense to add a more descriptive message here. To make it work with Colima, it's better to start it with `colima start --cpu 10 --memory 12 --network-address`
This PR adds official support for instrumenting OpenAI SDK calls in Node with Sentry tracing, following OpenTelemetry semantic conventions for Generative AI. We currently instrument the following OpenAI SDK methods: - client.chat.completions.create() - For chat-based completions - client.responses.create() - For the responses API Currently supported: - Node.js - Mentioned methods are supported in this PR - ESM and CJS - Both module systems are supported The openAIIntegration() accepts the following options: ``` // The integration respects your sendDefaultPii client option interface OpenAiOptions { recordInputs?: boolean; // Whether to record prompt messages recordOutputs?: boolean; // Whether to record response text } ``` Example: ``` Sentry.init({ dsn: '__DSN__', sendDefaultPii: false, // Even with PII disabled globally integrations: [ Sentry.openAIIntegration({ recordInputs: true, // Force recording prompts recordOutputs: true, // Force recording responses }), ], }); ```
6572f52
to
53a6f1e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding the note about the web worker change!
CHANGELOG.md
Outdated
This release adds two Browser SDK APIs to let the main thread know about debugIds of worker files: | ||
|
||
- `webWorkerIntegration({worker})` to be used in the main thread | ||
- `registerWebWorker(self)` to be used in the web worker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
l: small API correction
- `registerWebWorker(self)` to be used in the web worker | |
- `registerWebWorker({self})` to be used in the web worker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 fixed
53a6f1e
to
24cdfd3
Compare
CHANGELOG.md
Outdated
```js | ||
Sentry.init({ | ||
dsn: '__DSN__', | ||
sendDefaultPii: false, // Even with PII disabled globally |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would delete line 60
and add a comment // The integration respects your sendDefaultPii client option
under line 57
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated 👍
24cdfd3
to
d4ab7c0
Compare
No description provided.