fix: Remove outdated modules in favor of LFX versions

[erichare]

This pull request removes several forwarding modules and related code from the src/backend/base/langflow/components package. These modules previously re-exported components from the lfx package, but are no longer needed. The cleanup helps reduce redundancy and simplifies the codebase.

Removed forwarding modules and re-exports:

• Removed forwarding modules for agents, anthropic, data, helpers, and openai that previously re-exported all symbols from their respective lfx.components modules. [1] [2] [3] [4] [5]
• Removed the forwarding import and __all__ declaration for convert_to_dataframe in processing/converter.py.
• Removed the placeholder comment from processing/__init__.py.

Summary by CodeRabbit

• Refactor

  • Removed legacy re-export layers for multiple components; update imports to use lfx.components directly.
  • Removed convert_to_dataframe from the processing API.

• Chores

  • Cleaned up obsolete forwarding modules and comments to reduce maintenance overhead.

Note: If your project relied on langflow.components.* import paths or the processing converter export, update code to reference the corresponding lfx.components modules or utilities. No functional behavior changes beyond import surface adjustments.

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

Removed multiple langflow.components forwarding modules that re-exported lfx.components symbols and trimmed processing package exports by deleting a comment and dropping convert_to_dataframe from public API.

Changes

Cohort / File(s)	Summary
Remove re-export shims (components root) src/backend/base/langflow/components/agents.py, src/backend/base/langflow/components/anthropic.py, src/backend/base/langflow/components/data.py, src/backend/base/langflow/components/helpers.py, src/backend/base/langflow/components/openai.py	Deleted modules that re-exported lfx.components.* symbols; removed wildcard imports and __all__ forwarding, eliminating these public proxies.
Processing package cleanups src/backend/base/langflow/components/processing/__init__.py, src/backend/base/langflow/components/processing/converter.py	Removed a comment-only line in __init__.py. In converter.py, removed import forwarding of convert_to_dataframe and its __all__ export.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• feat: introduce lfx package #9133 — Introduces the lfx package and replaces langflow imports with lfx, aligning with removal of forwarding shims here.
• fix: disable mcp auto install for not installed applications, refactor mcp-projects logic #9599 — Adjusts package export surfaces and forwarding modules similar to the re-export removals in this PR.
• fix: Refactor a few more components to proper folders #8324 — Changes exports for helpers/processing components, related to the export surface adjustments made here.

Suggested labels

refactor, size:M, lgtm

Suggested reviewers

• jordanrfrazier
• edwinjosechittilappilly
• rodrigosnader

Pre-merge checks and finishing touches

❌ Failed checks (1 error, 1 inconclusive)

Check name	Status	Explanation	Resolution
Test Coverage For New Implementations	❌ Error	No new or updated test files were added in this PR, and the changes remove forwarding modules without introducing replacement coverage, so the custom requirement for accompanying tests is not satisfied.	Please add or update regression tests that cover the removal or altered behavior of these forwarding modules so the change is validated by the test suite.
Test File Naming And Structure	❓ Inconclusive	No test files were added or modified in this pull request, leaving the existing backend and frontend test suites untouched; without changes to inspect, there is no evidence that the current revision violates the required naming, structure, or coverage guidelines for tests, but the absence of updated tests for the removed modules means the custom check cannot be conclusively verified.	Please review the relevant test suites to confirm that they still follow the expected naming conventions, structural patterns, and scenario coverage, and update or extend the tests if any gaps are found.

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title clearly and concisely summarizes the primary change of removing legacy forwarding modules and adopting the LFX implementations, directly reflecting the pull request’s main objective without extraneous detail.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.
Test Quality And Coverage	✅ Passed	The pull request exclusively removes previously existing forwarding modules and public re-exports without introducing new runtime behavior, so there is no new functionality requiring test coverage or adjustments to existing suites. Given the absence of new async routines, API endpoints, or logic changes, the existing testing strategy remains sufficient and no additional tests are necessary for this change.
Excessive Mock Usage Warning	✅ Passed	No test files in the repository currently exercise this change, and a search for references to mock or MagicMock returned no results, so there is no evidence of excessive mock usage to flag.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 24.21%. Comparing base (485358c) to head (94479d3).
⚠️ Report is 3 commits behind head on main.

❌ Your project status has failed because the head coverage (47.19%) is below the target coverage (55.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10091      +/-   ##
==========================================
+ Coverage   24.19%   24.21%   +0.02%     
==========================================
  Files        1091     1086       -5     
  Lines       40042    40029      -13     
  Branches     5544     5539       -5     
==========================================
+ Hits         9688     9695       +7     
+ Misses      30183    30163      -20     
  Partials      171      171              

Flag	Coverage Δ
backend	47.19% <ø> (+0.02%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...d/base/langflow/components/processing/converter.py	0.00% <ø> (ø)

... and 10 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[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