RHIDP-13441: remove unused langchain dependencies

[Jdubrick]

Hey, I just made a Pull Request!

• Removes unused langchain dependencies and updates the knip report outlining their redundancy

✔️ Checklist

https://redhat.atlassian.net/browse/RHIDP-13441

• 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-gh-app]

Changed Packages

Package Name	Package Path	Changeset Bump	Current Version
@red-hat-developer-hub/backstage-plugin-intelligent-assistant-backend	workspaces/intelligent-assistant/plugins/intelligent-assistant-backend	patch	v4.0.0

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[rhdh-qodo-merge]

PR Summary by Qodo

Remove unused LangChain deps from intelligent-assistant backend

⚙️ Configuration changes 📝 Documentation 🕐 10-20 Minutes

AI Description

• Remove unused LangChain-related dependencies from intelligent-assistant backend packages.
• Add a changeset documenting the cleanup after switching to the Lightspeed Core HTTP proxy.
• Update knip report and yarn.lock to reflect the reduced dependency graph.

Diagram

graph TD
  A["Dependency cleanup PR"] --> B["backend/package.json"] --> C["yarn.lock"]
  A --> D["workspace/package.json"] --> C
  B --> E["knip-report.md"]
  A --> F["changeset (.md)"]

Loading

High-Level Assessment

The approach is appropriate: removing unused direct dependencies (and any resolutions that only supported them) reduces install surface area and avoids carrying stale packages. Alternatives like keeping them as optional/dev dependencies would still increase maintenance and supply-chain footprint without providing runtime value.

Files changed (5) +15 / -150

Documentation (2) +5 / -7

remove-unused-langchain.mdAdd patch changeset for LangChain dependency removal +5/-0

Add patch changeset for LangChain dependency removal

• Introduces a changeset to publish a patch for the intelligent-assistant backend package. Documents that @langchain/core and @langchain/openai were leftover after migrating to the Lightspeed Core HTTP proxy.

workspaces/intelligent-assistant/.changeset/remove-unused-langchain.md

knip-report.mdRefresh knip report after dependency cleanup +0/-7

Refresh knip report after dependency cleanup

• Updates the knip report by removing the section that previously flagged @langchain/core and @langchain/openai as unused dependencies. Leaves the report header as the remaining content.

workspaces/intelligent-assistant/plugins/intelligent-assistant-backend/knip-report.md

Other (3) +10 / -143

package.jsonRemove unused langsmith dependency from workspace manifest +0/-1

Remove unused langsmith dependency from workspace manifest

• Deletes the langsmith entry from the intelligent-assistant workspace package.json. This aligns the workspace manifest with the backend no longer relying on LangChain-related tooling.

workspaces/intelligent-assistant/package.json

package.jsonDrop @langchain/core/@langchain/openai and langsmith resolution +0/-5

Drop @langchain/core/@langchain/openai and langsmith resolution

• Removes unused @langchain/core and @langchain/openai from backend dependencies. Also removes the langsmith resolution block that is no longer needed after dropping LangChain packages.

workspaces/intelligent-assistant/plugins/intelligent-assistant-backend/package.json

yarn.lockPrune lockfile entries for removed LangChain-related packages +10/-137

Prune lockfile entries for removed LangChain-related packages

• Updates yarn.lock to remove entries for @langchain/core, @langchain/openai, langsmith, and related transitive packages. Includes minor key normalization where remaining packages are referenced under fewer specifier aliases.

workspaces/intelligent-assistant/yarn.lock

[rhdh-qodo-merge]

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

Tip of the day

💡 Did you know, you can describe a rule in plain language on the Rules page and Qodo drafts it for you

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.15%. Comparing base (238427a) to head (88aa183).
⚠️ Report is 4 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4325   +/-   ##
=======================================
  Coverage   60.15%   60.15%           
=======================================
  Files        2514     2514           
  Lines      100666   100666           
  Branches    28191    28185    -6     
=======================================
  Hits        60557    60557           
  Misses      39555    39555           
  Partials      554      554           

Flag	Coverage Δ		*Carryforward flag
adoption-insights	84.55% <ø> (ø)		Carriedforward from 238427a
ai-integrations	71.95% <ø> (ø)		Carriedforward from 238427a
app-defaults	69.79% <ø> (ø)		Carriedforward from 238427a
augment	46.67% <ø> (ø)		Carriedforward from 238427a
boost	79.00% <ø> (ø)		Carriedforward from 238427a
bulk-import	72.79% <ø> (ø)		Carriedforward from 238427a
cost-management	13.55% <ø> (ø)		Carriedforward from 238427a
dcm	67.21% <ø> (ø)		Carriedforward from 238427a
e2e-adoption-insights	60.00% <ø> (ø)		Carriedforward from 238427a
e2e-extensions	62.13% <ø> (ø)		Carriedforward from 238427a
e2e-global-header	49.45% <ø> (ø)		Carriedforward from 238427a
e2e-homepage	43.49% <ø> (ø)		Carriedforward from 238427a
e2e-intelligent-assistant	46.68% <ø> (ø)		Carriedforward from 238427a
e2e-orchestrator	50.42% <ø> (ø)		Carriedforward from 238427a
e2e-quickstart	55.21% <ø> (ø)		Carriedforward from 238427a
e2e-scorecard	50.21% <ø> (ø)		Carriedforward from 238427a
e2e-theme	16.36% <ø> (ø)		Carriedforward from 238427a
extensions	56.59% <ø> (ø)		Carriedforward from 238427a
global-floating-action-button	71.18% <ø> (ø)		Carriedforward from 238427a
global-header	66.50% <ø> (ø)		Carriedforward from 238427a
homepage	47.59% <ø> (ø)		Carriedforward from 238427a
install-dynamic-plugins	59.95% <ø> (ø)		Carriedforward from 238427a
intelligent-assistant	75.42% <ø> (ø)
konflux	91.98% <ø> (ø)		Carriedforward from 238427a
lightspeed	69.02% <ø> (ø)		Carriedforward from 238427a
mcp-integrations	83.40% <ø> (ø)		Carriedforward from 238427a
orchestrator	70.87% <ø> (ø)		Carriedforward from 238427a
quickstart	63.74% <ø> (ø)		Carriedforward from 238427a
sandbox	79.56% <ø> (ø)		Carriedforward from 238427a
scorecard	87.09% <ø> (ø)		Carriedforward from 238427a
theme	88.91% <ø> (ø)		Carriedforward from 238427a
translations	5.12% <ø> (ø)		Carriedforward from 238427a
x2a	55.02% <ø> (ø)		Carriedforward from 238427a

*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 238427a...88aa183. 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.