-
-
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 <[email protected]>
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 <[email protected]>
|
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 <[email protected]>
#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
Lms24
left a comment
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.