feat(@langchain/langgraph): initial integration support#7412
Open
feat(@langchain/langgraph): initial integration support#7412
Conversation
crysmags
force-pushed
the
crysmags/agentic-llm-langgraph-test
branch
from
5e87531 to
4de611f
Compare
Contributor
Overall package sizeSelf size: 4.57 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 2.0.3 | 76.87 kB | 808.03 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #7412 +/- ##
==========================================
- Coverage 80.39% 80.26% -0.13%
==========================================
Files 732 736 +4
Lines 31027 31134 +107
==========================================
+ Hits 24945 24991 +46
- Misses 6082 6143 +61
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
BenchmarksBenchmark execution time: 2026-02-03 18:52:50 Comparing candidate commit fec5e8c in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 225 metrics, 25 unstable metrics. |
These changes were made by the llm obs workflow to improve test robustness but are not needed for the LangGraph integration itself. The changes have been saved to /tmp/util-js-llmobs-improvements.patch for future reference and can be applied later if needed. Changes reverted: - Mock value handling for timing fields (span_id, duration, start_ns) - Better error handling in test assertions - Input validation improvements - Edge case handling for outputValue === 0
- Add trailing commas to all arrays and objects per @stylistic/comma-dangle - Break long line in llmobs plugin to comply with max-len rule - Remove unused 'llmobs' variable in test spec All langgraph-related linting errors have been fixed.
crysmags
force-pushed
the
crysmags/agentic-llm-langgraph-test
branch
from
522cd8e to
6d3c48d
Compare
|
sabrenner
reviewed
Feb 3, 2026
Comment on lines
+51
to
+55
| const asyncIterable = ctx.result | ||
|
|
||
| const originalAsyncIterator = asyncIterable[Symbol.asyncIterator].bind(asyncIterable) | ||
|
|
||
| asyncIterable[Symbol.asyncIterator] = function () { |
Collaborator
There was a problem hiding this comment.
i think we should try and do this in the langgraph.js instrumentation file maybe, and add a custom subscriber there and use shimmer.wrap for robustness. then we can emit a new event with the context that this PregelStreamPlugin can listen for
Collaborator
Author
There was a problem hiding this comment.
We are trying to avoid this so that we can continue to use Orchestrion, right now since we only have tracePromise we need to handle this on the subscriber.
Collaborator
There was a problem hiding this comment.
yeah, what i mean is we can do this out directly in the langgraph.js instead
const orchestrionStreamCh = channel('tracing:orchestrion:@langchain/langgraph:Pregel_stream')
orchestrionStreamCh.subscribe({
asyncStart (ctx) {
// shimmer.wrap ctx.result
}
})just so any/all custom wrapping logic happens in instrumentation and not in the plugins directly. lmk if this doesn't make any sense tho as idk if we do something like this currently.
Collaborator
Author
There was a problem hiding this comment.
I don't think this is a common pattern to mix publisher and subscriber logic in the instrumentation. I would think it would be cleaner to keep it on the subscriber/plugin side, but I can defer to @rochdev on this.
Collaborator
There was a problem hiding this comment.
ah ok i don't have too strong a preference, just that we had done something like that previously for langchain before we had some more orchestrion capabilities (ref).
i think in either case we should use shimmer to patch instead of modifying properties directly (even if shimmer just does this directly under the hood)
sabrenner
reviewed
Feb 3, 2026
|
|
||
| getLLMObsSpanRegisterOptions (ctx) { | ||
| const span = ctx.currentStore?.span | ||
| const name = span?.context()._tags?.['resource.name'] || 'langgraph.workflow' |
Collaborator
There was a problem hiding this comment.
i gotta check it out but ideally if we can use ctx.self.name if it exists (like this.name, this referring to the graph instance), that would be better i think
sabrenner
reviewed
Feb 3, 2026
|
|
||
| formatIO (data) { | ||
| // Handle null/undefined explicitly | ||
| if (data === null || data === undefined) return '' |
Collaborator
There was a problem hiding this comment.
Suggested change
| if (data === null || data === undefined) return '' | |
| if (data == null) return '' |
tlhunter
added
the
AI Generated
tlhunter
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This will add support for LangGraph and was made using the APM Instrumentation Toolkit.