Conversation
|
This PR was not deployed automatically as @LeonardoVieira1630 does not have access to the Railway project. In order to get automatic PR deploys, please add @LeonardoVieira1630 to your workspace on Railway. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
LeonardoVieira1630
requested review from
PedroBinotto,
alextnetto,
brunod-e and
pikonha
as code owners
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 553bda1738
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "@apollo/client": | ||
| specifier: ^3.13.8 | ||
| version: 3.14.0(@types/react@19.2.8)(graphql-ws@6.0.7(crossws@0.3.5)(graphql@16.13.1)(ws@8.18.0(bufferutil@4.0.9)(utf-8-validate@5.0.10)))(graphql@16.13.1)(react-dom@19.2.3(react@19.2.3))(react@19.2.3) | ||
| version: 3.14.0(@types/react@19.2.8)(graphql-ws@6.0.7(crossws@0.3.5)(graphql@16.13.1)(ws@7.5.10(bufferutil@4.0.9)(utf-8-validate@5.0.10)))(graphql@16.13.1)(react-dom@19.2.3(react@19.2.3))(react@19.2.3) |
There was a problem hiding this comment.
This lockfile change now resolves the dashboard’s Apollo/Wagmi websocket chain against ws@7.5.10, but graphql-ws@6.0.7 in the same lockfile declares a peer requirement of ws: ^8. Because this commit switched the resolved peer instance from ws@8.18.0 to v7, websocket-dependent paths (for example Node-side GraphQL subscriptions or wallet connector flows) can execute against an unsupported major and fail at runtime. Please preserve a ws@^8 resolution for this dependency graph.
Useful? React with 👍 / 👎.
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 870565990e
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "@solana/subscribable": 3.0.3(typescript@5.9.3) | ||
| typescript: 5.9.3 | ||
| ws: 8.18.0(bufferutil@4.0.9)(utf-8-validate@5.0.10) | ||
| ws: 7.5.10(bufferutil@4.0.9)(utf-8-validate@5.0.10) |
There was a problem hiding this comment.
This lockfile update downgrades the Solana websocket transport path to ws@7.5.10, but @solana/rpc-subscriptions-channel-websocket@3.0.3 declares a ws: ^8.18.0 peer in the same pnpm-lock.yaml. That creates an unsupported major-peer resolution for the Coinbase/Solana connector chain used via wagmi/rainbowkit, so websocket-based subscription flows can fail at runtime even though install succeeds; keep this dependency chain on a ws@^8 resolution.
Useful? React with 👍 / 👎.
2 participants
Summary
Prepares
@anticapture/observabilityfor publishing to npm and restructures its OpenTelemetry dependency so consumers share a single@opentelemetry/apiinstance. Bumps the package to1.0.1and wires each consuming app to depend on@opentelemetry/apidirectly to satisfy the new peer dependency.Changes
packages/observability/package.json: bump version to1.0.1; addfiles: ["dist"],publishConfig.access: public, andrepositorymetadata for npm publishing.packages/observability/package.json: move@opentelemetry/apifromdependenciestodevDependencies+peerDependenciesto avoid duplicate tracer/meter instances in consumers.apps/address-enrichment,apps/api-gateway,apps/gateful,apps/offchain-indexer: add@opentelemetry/api@^1.9.0as a direct dependency to satisfy the new peer requirement.pnpm-lock.yaml: regenerated to reflect the dependency reshuffling.