fix(langgraph): support the locked OpenTelemetry 2 SDK - #565
Merged
Pal Lakatos-Toth (pallakatos) merged 2 commits intoSep 15, 2026
Merged
Conversation
Use resourceFromAttributes and constructor span processors without changing attributes, endpoints or optional initialization behavior. Add real trace/metric collector regressions, make Docker stages honor the existing lockfile, and run runtime qualification under the existing required TypeScript runtime check. No dependency version changes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: eb3654cd-f1e0-445a-8734-430800af1903
Record the real image-build failure, narrow SDK API repair, unchanged dependency graph and explicit pending hosted execution under the existing maintainer delegation. No deployment approval or check waiver. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: eb3654cd-f1e0-445a-8734-430800af1903
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Pal Lakatos-Toth (pallakatos)
marked this pull request as ready for review
September 15, 2026 11:22
Pal Lakatos-Toth (pallakatos)
requested review from
Johnson Shi (johnsonshi) and
Lachlan Evenson (lachie83)
as code owners
September 15, 2026 11:22
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
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.
Fix the actual LangGraph TypeScript beta image build
Merged into
kars-bridgeas3bc7ff58d8d2994e474be39fd2af201d750fc5c2from qualified head
0f70d285ba9bd3a2ff072c3aca874123d9dc6802.All 31 required checks and review protections remain intact. No main/release promotion.
Building the public beta candidate exposed two TypeScript errors:
@opentelemetry/resourcesno longer exports theResourceconstructor andNodeTracerProviderno longer hasaddSpanProcessor. The manifest and lockfilealready require OpenTelemetry SDK 2.8.0.
This repair uses
resourceFromAttributesand constructorspanProcessors,preserving attributes, endpoints, provider lifecycle and idempotence. No
dependency versions are changed and telemetry is not disabled.
Both Docker dependency stages now use the existing lockfile with
npm ci.The obsolete Vitest
basicreporter is replaced by the default reporter.Four real-SDK/HTTP-collector cases check trace and metric exports, attributes,
endpoint precedence and preservation of the initial provider configuration.
The existing required
Runtime OpenClaw Build & Testcheck retains its nameand prior work, and now also installs, typechecks, builds and tests this adapter.
Qualification
The source image failure and invalid reporter were reproduced. Source syntax,
unchanged lockfile, workflow preservation, copyright/size/crypto/stub gates and
the scoped delegated audit pass locally. The exact SDK 2.8.0 graph is unavailable
locally; cached 1.30.1 was not substituted.
Current-head public job
104341944758has now passed the locked install,typecheck, build and all four real trace/metric export regressions with Vitest
4.1.8. Existing OpenClaw tests also passed (274 cases). Bridge CI
34957185912passed all eleven jobs. The actuallinux/amd64container alsobuilt successfully from exact public head
0f70d285: both Dockernpm cistages and
tsc -p .passed. It is retained separately from the original betaimages; no old image provenance was changed. All 53 reported PR checks
subsequently passed, including the full core and native requirements.
The guarded integration merge restored its temporary review allowance.
All 31 integration checks and source-review protections remain required.
This is a necessary existing-runtime packaging correction, not a new feature.
H100 cleanup/reinstall remains a separate acceptance step using the merged source;
no old/private image is substituted or relabeled as this revision.