Skip to content

feat(deno): Add conversationIdIntegration to default integrations #24299

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.

Sign up for GitHub

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

Merged
mydea merged 2 commits into from
Sep 11, 2026
+5 −0
Merged

feat(deno): Add conversationIdIntegration to default integrations #24299

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
7a6643c
feat(deno): Add conversationIdIntegration to default integrations
mydea Sep 10, 2026
94ad3c6
fix tests
mydea Sep 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions packages/deno/src/sdk.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import type { Client, Integration, Options, StackParser } from '@sentry/core';
import type { ServerRuntimeClientOptions } from '@sentry/core/server';
import {
conversationIdIntegration,
createStackParser,
dedupeIntegration,
eventFiltersIntegration,
Expand Down Expand Up @@ -36,6 +37,7 @@ export function getDefaultIntegrations(options: Options): Integration[] {
functionToStringIntegration(),
linkedErrorsIntegration(),
dedupeIntegration(),
conversationIdIntegration(),

@cursor cursor Bot Sep 10, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Feat PR lacks integration test

Low Severity

This feat PR does not include an integration or E2E test for the new default. Flagged because the project's testing conventions require at least one integration or E2E test on feat PRs. Existing sdk.test.ts already asserts other default integrations, and the user-visible behavior is conversation IDs being stamped onto gen_ai spans.

Fix in Cursor Fix in Web

Triggered by project rule: PR Review Guidelines for Cursor Bot

Reviewed by Cursor Bugbot for commit 7a6643c. Configure here.

// Deno Specific
breadcrumbsIntegration(),
denoContextIntegration(),
Expand Down
3 changes: 3 additions & 0 deletions packages/deno/test/__snapshots__/mod.test.ts.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ snapshot[`captureMessage 1`] = `
"FunctionToString",
"LinkedErrors",
"Dedupe",
"ConversationId",
"Breadcrumbs",
"DenoContext",
"DenoServe",
Expand Down Expand Up @@ -137,6 +138,7 @@ snapshot[`captureMessage twice 1`] = `
"FunctionToString",
"LinkedErrors",
"Dedupe",
"ConversationId",
"Breadcrumbs",
"DenoContext",
"DenoServe",
Expand Down Expand Up @@ -229,6 +231,7 @@ snapshot[`captureMessage twice 2`] = `
"FunctionToString",
"LinkedErrors",
"Dedupe",
"ConversationId",
"Breadcrumbs",
"DenoContext",
"DenoServe",
Expand Down
Loading