Skip to content

fix: remove @edx/frontend-lib-learning-assistant dependency#1894

Open
arbrandes wants to merge 1 commit intoopenedx:masterfrom
arbrandes:remove-learning-assistant
Open

fix: remove @edx/frontend-lib-learning-assistant dependency#1894
arbrandes wants to merge 1 commit intoopenedx:masterfrom
arbrandes:remove-learning-assistant

Conversation

@arbrandes
Copy link
Copy Markdown
Contributor

@arbrandes arbrandes commented Apr 7, 2026

Description

The @edx/frontend-lib-learning-assistant library has been fully deprecated and is now being removed. This PR removes the dependency along with all its usage across the codebase: the Redux reducer registration, the learningAssistant state selectors, the learning_assistant_enabled field mapping, related config variables (CHAT_RESPONSE_URL, ENABLE_XPERT_AUDIT), and associated test and pact contract references.

The ContentTools component previously read sidebarIsOpen from the learningAssistant Redux state to conditionally hide itself when the chatbot sidebar was open. With the library removed, content tools now render unconditionally.

Fixes #1798

Testing

When testing this via npm run dev you may run into loud SASS build warnings. That's a separate issue, fixed by openedx/frontend-build#683.

LLM usage notice

Built with assistance from Claude models (mostly Opus 4.6).

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.26%. Comparing base (0664dc3) to head (72cc0a1).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1894      +/-   ##
==========================================
- Coverage   91.27%   91.26%   -0.01%     
==========================================
  Files         343      343              
  Lines        5766     5761       -5     
  Branches     1387     1385       -2     
==========================================
- Hits         5263     5258       -5     
  Misses        484      484              
  Partials       19       19              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@arbrandes
Copy link
Copy Markdown
Contributor Author

@jristau1984, hi! This is the PR that implements the DEPR, which by the timeline should be good to go (i.e, good to break). Objections?

@arbrandes arbrandes requested a review from jristau1984 April 7, 2026 17:58
It has been fully deprecated and is now being removed.

Fixes openedx#1798

Co-Authored-By: Claude <noreply@anthropic.com>
@arbrandes arbrandes force-pushed the remove-learning-assistant branch from f710d7d to 72cc0a1 Compare April 9, 2026 12:35
@jristau1984
Copy link
Copy Markdown
Contributor

@jristau1984, hi! This is the PR that implements the DEPR, which by the timeline should be good to go (i.e, good to break). Objections?

Feel free to proceed, from 2U perspective. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DEPR]: frontend-lib-learning-assistant

3 participants