Skip to content

chore(lightspeed): remove unused LangChain dependencies - #5387

Merged
Jdubrick merged 1 commit into
redhat-developer:release-1.10/lightspeedfrom
Jdubrick:codex/lightspeed-1.10-remove-langchain
Oct 27, 2026
Merged

Jdubrick merged 1 commit into
redhat-developer:release-1.10/lightspeedfrom
Jdubrick:codex/lightspeed-1.10-remove-langchain

Conversation

@Jdubrick

Copy link
Copy Markdown
Member

Remove the unused LangChain dependencies and LangSmith overrides from the Lightspeed 1.10 release line, and prune their transitive packages from the lockfile.

Include a backend patch changeset, matching the cleanup previously made on main in #4325.

Hey, I just made a Pull Request!

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

@rhdh-qodo-merge

Copy link
Copy Markdown

PR Summary by Qodo

Remove unused LangChain dependencies from Lightspeed

⚙️ Configuration changes 🕐 10-20 Minutes


AI Description

• Remove unused LangChain backend dependencies and LangSmith overrides after the move to the HTTP
 proxy.
• Prune their lockfile entries and add a backend patch changeset.
Diagram

graph TD
  Backend["Backend dependencies"] --> Lock["Yarn lockfile"] --> Install["Workspace install"]
  Overrides["Workspace overrides"] --> Lock
  Changeset["Patch changeset"] --> Release["Backend release"]
  Backend --> Release
Loading
High-Level Assessment

Removing unused direct dependencies and overrides, then regenerating the lockfile, is the appropriate cleanup. Retaining the packages offers no benefit now that the backend uses the Lightspeed Core HTTP proxy.

Files changed (4) +16 / -144

Documentation (1) +5 / -0
remove-unused-langchain.mdRecord a backend patch release +5/-0

Record a backend patch release

• Adds a patch changeset explaining that the LangChain dependencies are no longer needed after the move to the Lightspeed Core HTTP proxy.

workspaces/lightspeed/.changeset/remove-unused-langchain.md

Other (3) +11 / -144
package.jsonRemove the workspace LangSmith override +1/-2

Remove the workspace LangSmith override

• Drops the LangSmith resolution from the Lightspeed workspace manifest.

workspaces/lightspeed/package.json

package.jsonRemove unused backend LangChain dependencies +0/-5

Remove unused backend LangChain dependencies

• Removes @langchain/core and @langchain/openai from backend dependencies and deletes the package-level LangSmith resolution.

workspaces/lightspeed/plugins/lightspeed-backend/package.json

yarn.lockPrune unused LangChain dependency resolutions +10/-137

Prune unused LangChain dependency resolutions

• Removes the LangChain and LangSmith entries and transitive packages no longer required by the workspace. Retained shared packages have their dependency selectors adjusted.

workspaces/lightspeed/yarn.lock

@rhdh-qodo-merge

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Powered by Qodo

@rhdh-qodo-merge

Copy link
Copy Markdown

Important

The /generate_labels command by Qodo was deprecated on the 1st of October 2026 and is no longer available. We recommend switching to the latest Qodo review capabilities. Learn more

@codecov

codecov Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (release-1.10/lightspeed@bcd91ab). Learn more about missing BASE report.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@                    Coverage Diff                     @@
##             release-1.10/lightspeed    #5387   +/-   ##
==========================================================
  Coverage                           ?   60.96%           
==========================================================
  Files                              ?     2098           
  Lines                              ?    65167           
  Branches                           ?    16940           
==========================================================
  Hits                               ?    39726           
  Misses                             ?    25219           
  Partials                           ?      222           
Flag Coverage Δ *Carryforward flag
adoption-insights 83.58% <ø> (?) Carriedforward from 230b90c
ai-integrations 70.03% <ø> (?) Carriedforward from 230b90c
app-defaults 69.60% <ø> (?) Carriedforward from 230b90c
augment 69.36% <ø> (?) Carriedforward from 230b90c
bulk-import 72.86% <ø> (?) Carriedforward from 230b90c
cost-management 16.49% <ø> (?) Carriedforward from 230b90c
dcm 32.85% <ø> (?) Carriedforward from 230b90c
extensions 61.79% <ø> (?) Carriedforward from 230b90c
global-floating-action-button 74.30% <ø> (?) Carriedforward from 230b90c
global-header 61.68% <ø> (?) Carriedforward from 230b90c
homepage 50.95% <ø> (?) Carriedforward from 230b90c
konflux 91.01% <ø> (?) Carriedforward from 230b90c
lightspeed 68.13% <ø> (?)
mcp-integrations 81.59% <ø> (?) Carriedforward from 230b90c
orchestrator 36.36% <ø> (?) Carriedforward from 230b90c
quickstart 62.88% <ø> (?) Carriedforward from 230b90c
sandbox 79.56% <ø> (?) Carriedforward from 230b90c
scorecard 83.58% <ø> (?) Carriedforward from 230b90c
theme 64.54% <ø> (?) Carriedforward from 230b90c
translations 8.49% <ø> (?) Carriedforward from 230b90c
x2a 78.28% <ø> (?) Carriedforward from 230b90c

*This pull request uses carry forward flags. Click here to find out more.


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bcd91ab...66f32f2. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Jdubrick

Copy link
Copy Markdown
Member Author

/hold trying to figure out if we should bump playwright here or pin the gh runner

Remove the unused LangChain dependencies and LangSmith overrides from the Lightspeed 1.10 release line, and prune their transitive packages from the lockfile.

Include a backend patch changeset, matching the cleanup previously made on main in redhat-developer#4325.

Signed-off-by: Jordan Dubrick <jdubrick@redhat.com>
@Jdubrick
Jdubrick force-pushed the codex/lightspeed-1.10-remove-langchain branch from 088ff5d to 66f32f2 Compare October 27, 2026 19:42

@Omar-AlJaljuli Omar-AlJaljuli left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@sonarqubecloud

Copy link
Copy Markdown

@Jdubrick
Jdubrick merged commit 809b498 into redhat-developer:release-1.10/lightspeed Oct 27, 2026
88 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants