Linters unification

[bydawen]

Description

There is an issue, that in the different MFE repositories and related packages, we have different configurations for stylelint, eslint, tsconfig. We have appropriate repositories to unify these configurations:

• eslint
• stylelint-config-edx (some repositories have styles, but no stylelint at all)
• typescript-config

The right solution should be to install the necessary configurations in each MFE repository and reuse them with minimal modification (overrides), if necessary. Currently, these modifications (overrides) of the base configurations are not minimal. After analysis of the MFE repos it can be sorted by minimal/minor/high overrides for the linters configurations or the absence of any of it.

Here's tables with corresponding linter config/overrides:

MFE name	stylelint-config-edx	eslint	typescript-config
frontend-app-authn	-	.eslintrc.js with additional modifiers	-
frontend-app-account	-	.eslintrc.js	-
frontend-component-footer	-	.eslintrc.js	-
frontend-component-header	-	.eslintrc.js	tsconfig.json with additional modifiers (MINOR)
frontend-app-authoring	.stylelintrc.json with additional overides	.eslintrc.js with additional modifiers	tsconfig.json with additional modifiers (MINOR)
frontend-app-discussions	-	.eslintrc.js with additional modifiers	tsconfig.json with additional modifiers (MINOR)
frontend-app-profile	-	.eslintrc.js	-
frontend-app-learner-dashboard	-	.eslintrc.js with additional modifiers	tsconfig.json with additional modifiers (MINOR)
frontend-app-communications	-	.eslintrc.js with additional modifiers (MINOR)	tsconfig.json with additional modifiers (MINOR)
frontend-app-learning	-	.eslintrc.js with additional modifiers	tsconfig.json with additional modifiers (MINOR)
frontend-app-gradebook	-	.eslintrc.js with additional modifiers	-
frontend-app-learner-record	-	.eslintrc.js with additional modifiers (MINOR)	-
frontend-app-ora-grading	-	.eslintrc.js with additional modifiers	-
frontend-app-ora	-	.eslintrc.js with additional modifiers	tsconfig.json with additional modifiers (MINOR)
frontend-app-catalog	.stylelintrc.json with additional overides	.eslintrc.js with additional modifiers	tsconfig.json with additional modifiers (MINOR)
frontend-template-application	-	.eslintrc.js	-
paragon	.stylelintrc.json with additional overides	.eslintrc.js with additional modifiers	tsconfig.json with additional modifiers (MINOR)
frontend-lib-special-exams	-	.eslintrc.js	-
frontend-app-publisher	.stylelintrc.json with additional overides	.eslintrc.js	-
frontend-app-admin-portal	-	.eslintrc.js with additional modifiers	tsconfig.json with additional modifiers (MINOR)
frontend-build	-	.eslintrc.js with additional modifiers (MINOR)	tsconfig.json with additional modifiers (MINOR)
frontend-platform	-	.eslintrc.js with additional modifiers	-
frontend-plugin-framework	-	.eslintrc.js with additional modifiers (MINOR)	-
edx-bootstrap	.stylelintrc.json with additional overides (MINOR)	-	-
openedx/cypress-e2e-tests	-	.eslintrc.js with additional modifiers (MINOR)	-

[bradenmacdonald]

I would like to propose that we start migrating all of our frontend repos away from eslint to use oxlint instead. It's way faster, has way better defaults (so we could use it with no config file, or a very minimal one), and it supports fast type-aware linting (like making sure that promises are awaited).

I am testing it now on the Authoring MFE (openedx/frontend-app-authoring#2559) and it's already found 100+ incorrect await statements, many of which meant that the test code was not even running. So lots of the tests were completely broken and we didn't know.