openedx / openedx/frontend-app-authoring
Test 'oxlint' in the Authoring MFE
Open
@bradenmacdonald is already working on this.
Since Jan 29, 2026.
- Dominant language
- TypeScript
- Stars
- 17
- Forks
- 218
- Avg merge
- 9d 20h
- Merged PRs (30d)
- 20
Description
I'd like us to consider replacing eslint with oxlint across all Open edX frontend repos. This ticket is to track work to test Oxlint in this MFE's codebase. See the related PRs for findings.
PR to test oxlint in this repo on a trial basis: https://github.com/openedx/frontend-app-authoring/pull/2847
Issues found from testing
- 104 missing/spurious
awaitstatements in test code https://github.com/openedx/frontend-app-authoring/pull/2560 - 50+ additional missing/spurious
awaitstatements https://github.com/openedx/frontend-app-authoring/pull/2592 - 8 redundant declarations of
undefinedtype on optional parameters https://github.com/openedx/frontend-app-authoring/pull/2589 - 110 unused
errorvariables https://github.com/openedx/frontend-app-authoring/pull/2591 - 15 questionable
.toString()usages https://github.com/openedx/frontend-app-authoring/pull/2597 - 2 very bad usages of
eval()! https://github.com/openedx/frontend-app-authoring/pull/2848 - 2 nullish coalescing operators used pointlessly https://github.com/openedx/frontend-app-authoring/pull/2849
- 52 type issues of various severity https://github.com/openedx/frontend-app-authoring/pull/2850
Private ref MNG-4763
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.