feat(feature-parity) LF <> LFX Services Layer Tiering & Separation

[dkaushik94]

TBD

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (1)

• release-.*

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.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f5a3066f-dc4b-439f-ad2a-927371bdb992

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

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch lfx-services-migration

---

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.}

[github-actions]

✅ Test Coverage Advisor

No source changes detected without accompanying tests. Thanks for keeping coverage up! 🎉

Advisory check only — never blocks merge.

[github-actions]

✅ Migration Validation Passed

All migrations follow the Expand-Contract pattern correctly.

[github-actions]

Frontend Unit Test Coverage Report

Coverage Summary

Lines	Statements	Branches	Functions
	44.71% (60951/136324)	69.59% (8343/11988)	43.19% (1381/3197)

Unit Test Results

Tests	Skipped	Failures	Errors	Time
5153	0 💤	0 ❌	0 🔥	16m 17s ⏱️

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.07%. Comparing base (8023291) to head (2237751).

Additional details and impacted files

@@                      Coverage Diff                       @@
##           refactor/orm-models-to-lfx   #14031      +/-   ##
==============================================================
- Coverage                       59.17%   59.07%   -0.11%     
==============================================================
  Files                            2372     2325      -47     
  Lines                          226897   225075    -1822     
  Branches                        32310    31746     -564     
==============================================================
- Hits                           134267   132954    -1313     
+ Misses                          91021    90513     -508     
+ Partials                         1609     1608       -1     

Flag	Coverage Δ
frontend	58.50% <ø> (-0.12%)	⬇️

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

Files with missing lines	Coverage Δ
...backend/base/langflow/services/database/service.py	50.00% <ø> (-20.36%)	⬇️
src/backend/base/langflow/services/factory.py	83.05% <ø> (-1.08%)	⬇️
src/backend/base/langflow/services/schema.py	100.00% <ø> (ø)
src/backend/base/langflow/services/utils.py	86.40% <ø> (ø)
src/lfx/src/lfx/__main__.py	77.77% <ø> (ø)
src/lfx/src/lfx/cli/commands.py	76.10% <ø> (ø)
src/lfx/src/lfx/schema/image.py	54.83% <ø> (ø)
src/lfx/src/lfx/services/base.py	100.00% <ø> (ø)
src/lfx/src/lfx/services/database/service.py	100.00% <ø> (ø)
src/lfx/src/lfx/services/deps.py	77.38% <ø> (+9.45%)	⬆️
... and 5 more

... and 113 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.

[jordanrfrazier]

Is there a specific reason we have tests in the src/ class?