Skip to content

test(node): Port LangChain tests to span streaming #25084

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
nicohrubec merged 10 commits into from
Oct 7, 2026
Merged

test(node): Port LangChain tests to span streaming #25084

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
3ba7799
test(node): Port LangChain tests to span streaming
nicohrubec Oct 6, 2026
fb768a7
test(node): Remove redundant langchain assertions
nicohrubec Oct 6, 2026
3e3412c
test(node): Collect langchain spans across envelopes
nicohrubec Oct 6, 2026
c5fe381
test(node): Use the SENTRY_ORIGIN convention in langchain tests
nicohrubec Oct 6, 2026
f3c1054
test(node): Use convention constants for langchain span attributes
nicohrubec Oct 6, 2026
2a7d74d
Merge branch 'develop' into feat/langchain-tests-span-streaming
nicohrubec Oct 6, 2026
6c4e3b8
test(node): Assert LangChain spans after envelope selection
nicohrubec Oct 7, 2026
2f5ea7d
Merge develop and simplify LangChain span assertions
nicohrubec Oct 7, 2026
021d90f
test(node): Collect LangChain timing scenario spans until main ends
nicohrubec Oct 7, 2026
1021de1
test(node): Collect LangChain v1 spans across Deno envelopes
nicohrubec Oct 7, 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
11 changes: 0 additions & 11 deletions dev-packages/node-integration-tests/suites/tracing/langchain/instrument-span-streaming.mjs
View file
Open in desktop

This file was deleted.

8 changes: 0 additions & 8 deletions dev-packages/node-integration-tests/suites/tracing/langchain/instrument-with-pii.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,9 @@ import * as Sentry from '@sentry/node';
import { loggingTransport } from '@sentry-internal/node-integration-tests';

Sentry.init({
traceLifecycle: 'static',
dsn: 'https://public@dsn.ingest.sentry.io/1337',
release: '1.0',
tracesSampleRate: 1.0,
dataCollection: { genAI: { inputs: true, outputs: true } },
transport: loggingTransport,
beforeSendTransaction: event => {
// Filter out mock express server transactions
if (event.transaction.includes('/v1/messages') || event.transaction.includes('/v1/embeddings')) {
return null;
}
return event;
},
});
8 changes: 0 additions & 8 deletions dev-packages/node-integration-tests/suites/tracing/langchain/instrument.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,9 @@ import * as Sentry from '@sentry/node';
import { loggingTransport } from '@sentry-internal/node-integration-tests';

Sentry.init({
traceLifecycle: 'static',
dsn: 'https://public@dsn.ingest.sentry.io/1337',
release: '1.0',
tracesSampleRate: 1.0,
dataCollection: { genAI: { inputs: false, outputs: false } },
transport: loggingTransport,
beforeSendTransaction: event => {
// Filter out mock express server transactions
if (event.transaction.includes('/v1/messages') || event.transaction.includes('/v1/embeddings')) {
return null;
}
return event;
},
});
298 changes: 100 additions & 198 deletions dev-packages/node-integration-tests/suites/tracing/langchain/test.ts
View file
Open in desktop

Large diffs are not rendered by default.

8 changes: 0 additions & 8 deletions dev-packages/node-integration-tests/suites/tracing/langchain/v1/instrument-with-pii.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,9 @@ import * as Sentry from '@sentry/node';
import { loggingTransport } from '@sentry-internal/node-integration-tests';

Sentry.init({
traceLifecycle: 'static',
dsn: 'https://public@dsn.ingest.sentry.io/1337',
release: '1.0',
tracesSampleRate: 1.0,
dataCollection: { genAI: { inputs: true, outputs: true } },
transport: loggingTransport,
beforeSendTransaction: event => {
// Filter out mock express server transactions
if (event.transaction.includes('/v1/messages') || event.transaction.includes('/v1/chat/completions')) {
return null;
}
return event;
},
});
8 changes: 0 additions & 8 deletions dev-packages/node-integration-tests/suites/tracing/langchain/v1/instrument.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,9 @@ import * as Sentry from '@sentry/node';
import { loggingTransport } from '@sentry-internal/node-integration-tests';

Sentry.init({
traceLifecycle: 'static',
dsn: 'https://public@dsn.ingest.sentry.io/1337',
release: '1.0',
tracesSampleRate: 1.0,
dataCollection: { genAI: { inputs: false, outputs: false } },
transport: loggingTransport,
beforeSendTransaction: event => {
// Filter out mock express server transactions
if (event.transaction.includes('/v1/messages') || event.transaction.includes('/v1/chat/completions')) {
return null;
}
return event;
},
});
404 changes: 190 additions & 214 deletions dev-packages/node-integration-tests/suites/tracing/langchain/v1/test.ts
View file
Open in desktop

Large diffs are not rendered by default.

Loading