Skip to content

chore: revert le-499#12055

Closed
viktoravelino wants to merge 4 commits intorelease-1.8.0from
fix/LE-499-revert
Closed

chore: revert le-499#12055
viktoravelino wants to merge 4 commits intorelease-1.8.0from
fix/LE-499-revert

Conversation

@viktoravelino
Copy link
Collaborator

Revert: #12054

…edge Base

Move ingestion component to deactivated folder so it's excluded from
dynamic discovery. Rename KnowledgeRetrievalComponent to
KnowledgeBaseComponent with display_name "Knowledge Base". Update all
exports, component index, starter project, frontend sidebar filter,
and tests.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 4, 2026

Important

Review skipped

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

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: 405d3cee-4ea1-474b-ba3b-d2e41f495727

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
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/LE-499-revert

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

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

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

Frontend Unit Test Coverage Report

Coverage Summary

Lines Statements Branches Functions
Coverage: 23%
23.29% (8155/35005) 16.03% (4404/27461) 16.13% (1188/7365)

Unit Test Results

Tests Skipped Failures Errors Time
2653 0 💤 0 ❌ 0 🔥 47.552s ⏱️

@codecov
Copy link

codecov bot commented Mar 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (release-1.8.0@d6b7d88). Learn more about missing BASE report.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff                @@
##             release-1.8.0   #12055   +/-   ##
================================================
  Coverage                 ?   37.18%           
================================================
  Files                    ?     1591           
  Lines                    ?    78096           
  Branches                 ?    11837           
================================================
  Hits                     ?    29042           
  Misses                   ?    47395           
  Partials                 ?     1659           
Flag Coverage Δ
frontend 20.83% <ø> (?)
lfx 41.49% <ø> (?)

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

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

@Adam-Aghili
Copy link
Collaborator

Was this intended to be labeled as ignore-for-release or is this the bot going rougue?

@Adam-Aghili Adam-Aghili requested a review from deon-sanchez March 5, 2026 14:47
@Adam-Aghili
Copy link
Collaborator

Adam-Aghili commented Mar 5, 2026

This PR also feel like it is trying to solve the smae issue as #12054

@Adam-Aghili Adam-Aghili closed this Mar 5, 2026
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.

2 participants