Skip to content

fix: Upgrade to latest version of graphql-codegen cli#479

Merged
JustinRatner merged 3 commits intomainfrom
jratner/upgrade-graphql-codegen-cli
Apr 20, 2026
Merged

fix: Upgrade to latest version of graphql-codegen cli#479
JustinRatner merged 3 commits intomainfrom
jratner/upgrade-graphql-codegen-cli

Conversation

@JustinRatner
Copy link
Copy Markdown
Contributor

@JustinRatner JustinRatner commented Apr 20, 2026

Summary

From Brad Bunce

We're seeing 13 high-severity npm audit findings when using @LaunchDarkly/observability-node@1.1.0. They all trace back to lodash@4.17.21 pulled in through the @graphql-codegen/cli dependency chain:

@LaunchDarkly/observability-node
  → @graphql-codegen/cli
    → @graphql-codegen/plugin-helpers
      → lodash <=4.17.23
The specific lodash advisories are:

GHSA-r5fr-rjxr-66jc — Code injection via _.template
GHSA-f23m-r3pf-42rh — Prototype pollution via _.unset / _.omit
GHSA-xxjr-mmjv-4gpg — Prototype pollution in _.unset / _.omit
npm audit fix can't resolve these without downgrading @LaunchDarkly/observability-node to 0.1.0, which is a breaking change. The only consumer-side workaround is an overrides entry to force a patched lodash version, which carries compatibility risk.

Could the @graphql-codegen dependencies in the observability-node package be updated to versions that no longer depend on vulnerable lodash? Appreciate any timeline or workaround guidance you can share.

The latest version of graphql-codegen cli removed lodash as an unnecessary dependency.

How did you test this change?

Ran codegen of observability-node successfully


Note

Low Risk
Low risk because changes are confined to dev/build tooling (graphql-codegen) and lockfile updates, with no production runtime code modifications.

Overview
Upgrades @graphql-codegen/cli from ^5.0.7 to ^6.3.1 for both @launchdarkly/observability-node and @launchdarkly/observability-shared, and moves it out of observability-node runtime dependencies into devDependencies.

Regenerates yarn.lock to reflect the new codegen dependency tree (notably updating related @graphql-codegen/*, @graphql-tools/*, and @babel/* packages).

Reviewed by Cursor Bugbot for commit 425f432. Bugbot is set up for automated code reviews on this repo. Configure here.

@JustinRatner JustinRatner changed the title Upgrade to latest version of graphql-codegen cli fix: Upgrade to latest version of graphql-codegen cli Apr 20, 2026
@Vadman97
Copy link
Copy Markdown
Contributor

@JustinRatner might need to run a yarn dedupe

@JustinRatner JustinRatner merged commit 97061e6 into main Apr 20, 2026
23 checks passed
@JustinRatner JustinRatner deleted the jratner/upgrade-graphql-codegen-cli branch April 20, 2026 22:47
Vadman97 pushed a commit that referenced this pull request Apr 20, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>observability-node: 1.1.1</summary>

##
[1.1.1](observability-node-1.1.0...observability-node-1.1.1)
(2026-04-20)


### Bug Fixes

* Upgrade to latest version of graphql-codegen cli
([#479](#479))
([97061e6](97061e6))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Low risk release metadata update: only version/changelog/manifest
bumps with no runtime code changes shown in this diff.
> 
> **Overview**
> Publishes `@launchdarkly/observability-node` `v1.1.1` by bumping the
version in `package.json` and updating `.release-please-manifest.json`.
> 
> Adds a `1.1.1` changelog entry noting a bugfix: upgrading the
`graphql-codegen` CLI.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
e51ea90. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants