chore: introduce oxlint for faster linting (#5416)

Author: KazariEX

.oxlintrc.json

@@ -0,0 +1,11 @@
+{
+	"$schema": "./node_modules/oxlint/configuration_schema.json",
+	"ignorePatterns": [
+		"**/*.vue"
+	],
+	"rules": {
+		"curly": "warn",
+		"no-unused-expressions": "warn",
+		"require-await": "warn"
+	}
+}

.vscode/settings.json

@@ -1,15 +1,4 @@
 {
-	"typescript.format.semicolons": "insert",
-	"editor.insertSpaces": false,
-	"editor.detectIndentation": false,
-	"editor.codeActionsOnSave": {
-		"source.organizeImports": "always"
-	},
-	"json.format.keepLines": true,
-	"typescript.tsdk": "node_modules/typescript/lib",
-	"[typescript]": {
-		"editor.defaultFormatter": "vscode.typescript-language-features"
-	},
 	"[javascript]": {
 		"editor.defaultFormatter": "vscode.typescript-language-features"
 	},
@@ -19,12 +8,23 @@
 	"[jsonc]": {
 		"editor.defaultFormatter": "vscode.json-language-features"
 	},
+	"[typescript]": {
+		"editor.defaultFormatter": "vscode.typescript-language-features"
+	},
+	"editor.codeActionsOnSave": {
+		"source.organizeImports": "always"
+	},
+	"editor.detectIndentation": false,
+	"editor.insertSpaces": false,
 	"files.exclude": {
 		"packages/*/*.d.ts": true,
 		"packages/*/*.js": true,
 		"packages/*/*.map": true,
 		"packages/*/lib/**/*.d.ts": true,
 		"packages/*/lib/**/*.js": true,
 		"packages/*/lib/**/*.map": true
-	}
+	},
+	"json.format.keepLines": true,
+	"typescript.format.semicolons": "insert",
+	"typescript.tsdk": "node_modules/typescript/lib"
 }

package.json

@@ -15,8 +15,12 @@
 		"release:next-vue": "cd ./extensions/vscode && pnpm run release:next",
 		"test": "vitest run",
 		"test:update": "vitest run --update",
-		"lint": "tsslint --project {tsconfig.json,packages/*/tsconfig.json,extensions/*/tsconfig.json}",
-		"lint:fix": "pnpm run lint -- --fix --vscode-settings .vscode/settings.json"
+		"lint": "pnpm run oxlint && pnpm run tsslint",
+		"lint:fix": "pnpm run oxlint:fix && pnpm run tsslint:fix",
+		"oxlint": "oxlint",
+		"oxlint:fix": "oxlint --fix",
+		"tsslint": "tsslint --project {tsconfig.json,packages/*/tsconfig.json,extensions/*/tsconfig.json}",
+		"tsslint:fix": "pnpm run lint -- --fix --vscode-settings .vscode/settings.json"
 	},
 	"devDependencies": {
 		"@lerna-lite/cli": "^4.1.2",
@@ -25,6 +29,7 @@
 		"@tsslint/config": "^1.5.0",
 		"@tsslint/eslint": "^1.5.0",
 		"@typescript-eslint/eslint-plugin": "^8.19.0",
+		"oxlint": "^1.0.0",
 		"typescript": "latest",
 		"vitest": "^3.1.3"
 	},

packages/language-core/lib/codegen/template/elementProps.ts

@@ -168,7 +168,7 @@ export function* generateElementProps(
 			if (isComponent && prop.name === 'model' && prop.modifiers.length) {
 				const propertyName = prop.arg?.type === CompilerDOM.NodeTypes.SIMPLE_EXPRESSION
 					? !prop.arg.isStatic
-						? `[__VLS_tryAsConstant(\`$\{${prop.arg.content}\}Modifiers\`)]`
+						? `[__VLS_tryAsConstant(\`\${${prop.arg.content}}Modifiers\`)]`
 						: camelize(propName) + `Modifiers`
 					: `modelModifiers`;
 				const codes = [...generateModifiers(

packages/language-core/lib/plugins/file-html.ts

@@ -1,8 +1,8 @@
 import type { SFCParseResult } from '@vue/compiler-sfc';
 import type { VueLanguagePlugin } from '../types';
 
-const sfcBlockReg = /\<(script|style)\b([\s\S]*?)\>([\s\S]*?)\<\/\1\>/g;
-const langReg = /\blang\s*=\s*(['\"]?)(\S*)\b\1/;
+const sfcBlockReg = /<(script|style)\b([\s\S]*?)>([\s\S]*?)<\/\1>/g;
+const langReg = /\blang\s*=\s*(['"]?)(\S*)\b\1/;
 
 const plugin: VueLanguagePlugin = ({ vueCompilerOptions }) => {
 

packages/language-core/lib/plugins/file-md.ts

@@ -9,9 +9,9 @@ const frontmatterReg = /^---[\s\S]*?\n---(?:\r?\n|$)/;
 const codeblockReg = /(`{3,})[\s\S]+?\1/g;
 const inlineCodeblockReg = /`[^\n`]+?`/g;
 const latexBlockReg = /(\${2,})[\s\S]+?\1/g;
-const scriptSetupReg = /\\\<[\s\S]+?\>\n?/g;
-const sfcBlockReg = /\<(script|style)\b[\s\S]*?\>([\s\S]*?)\<\/\1\>/g;
-const angleBracketReg = /\<\S*\:\S*\>/g;
+const scriptSetupReg = /\\<[\s\S]+?>\n?/g;
+const sfcBlockReg = /<(script|style)\b[\s\S]*?>([\s\S]*?)<\/\1>/g;
+const angleBracketReg = /<\S*:\S*>/g;
 const linkReg = /\[[\s\S]*?\]\([\s\S]*?\)/g;
 const codeSnippetImportReg = /^\s*<<<\s*.+/gm;
 

packages/language-core/lib/utils/ts.ts

@@ -32,7 +32,7 @@ export function createParsedCommandLineByJson(
 			const obj = ts.convertToObject(configFile, []);
 			const rawOptions: RawVueCompilerOptions = obj?.vueCompilerOptions ?? {};
 			resolver.addConfig(rawOptions, path.dirname(configFile.fileName));
-		} catch (err) { }
+		} catch { }
 	}
 
 	const resolvedVueOptions = resolver.build();
@@ -88,7 +88,7 @@ export function createParsedCommandLine(
 				const obj = ts.convertToObject(configFile, []);
 				const rawOptions: RawVueCompilerOptions = obj?.vueCompilerOptions ?? {};
 				resolver.addConfig(rawOptions, path.dirname(configFile.fileName));
-			} catch (err) { }
+			} catch { }
 		}
 
 		const resolvedVueOptions = resolver.build();
@@ -124,7 +124,7 @@ export function createParsedCommandLine(
 			vueOptions: resolvedVueOptions,
 		};
 	}
-	catch (err) {
+	catch {
 		// console.warn('Failed to resolve tsconfig path:', tsConfigPath, err);
 		return {
 			fileNames: [],
@@ -254,7 +254,7 @@ function resolvePath(scriptPath: string, root: string) {
 			// console.warn('failed to resolve path:', scriptPath, 'require.resolve is not supported in web');
 		}
 	}
-	catch (error) {
+	catch {
 		// console.warn(error);
 	}
 }

packages/language-server/lib/reactionsAnalyze.ts

@@ -308,7 +308,7 @@ function collect(ts: typeof import('typescript'), sourceFile: ts.SourceFile) {
 			}
 		}
 		else if (ts.isParameter(node)) {
-			if (node.type && node.type && ts.isTypeReferenceNode(node.type)) {
+			if (node.type && ts.isTypeReferenceNode(node.type)) {
 				const typeName = node.type.typeName.getText(sourceFile);
 				if (typeName.endsWith('Ref')) {
 					signals.push({

packages/language-service/lib/plugins/data.ts

@@ -41,6 +41,7 @@ export function loadTemplateData(lang: string) {
 
 	resolveReferences(data);
 
+	// oxlint-disable-next-line no-useless-spread
 	for (const attr of [...data.globalAttributes ?? []]) {
 		if (!attr.name.startsWith('v-')) {
 			data.globalAttributes?.push(

packages/language-service/lib/plugins/vue-inlayhints.ts

@@ -221,7 +221,9 @@ export function findDestructuredProps(
 		});
 
 		function enter(node: ts.Node) {
-			parent && parentStack.push(parent);
+			if (parent) {
+				parentStack.push(parent);
+			}
 
 			if (
 				ts.isTypeLiteralNode(node) ||
@@ -273,7 +275,10 @@ export function findDestructuredProps(
 		}
 
 		function leave(node: ts.Node) {
-			parent && parentStack.pop();
+			if (parent) {
+				parentStack.pop();
+			}
+
 			if (
 				ts.isFunctionLike(node)
 				|| ts.isCatchClause(node)

packages/language-service/scripts/update-html-data.js

@@ -82,16 +82,16 @@ const langs = [
 	},
 ];
 
+localeWorker();
 for (const lang of langs) {
 	if (lang.supported) {
-		localeWorker(lang);
 		templateWorker(lang);
 		sfcWorker(lang);
 		modelWorker(lang);
 	}
 }
 
-function localeWorker(lang) {
+function localeWorker() {
 
 	const data = langs.map(({ name, url }) => ({ name, url }));
 
@@ -417,7 +417,7 @@ async function templateWorker(lang) {
 async function fetchText(url, baseUrl) {
 	let text = await (await fetch(url)).text();
 	text = text.replace(/```vue-html/g, '```html');
-	text = text.replace(/\{\#.*?\}/g, '')
+	text = text.replace(/\{#.*?\}/g, '')
 	text = resolveMarkdownLinks(text, baseUrl);
 	return text;
 }

packages/language-service/tests/utils/format.ts

@@ -22,7 +22,7 @@ export function defineFormatTest(options: {
 	languageId: string;
 	settings?: any;
 }) {
-	describe(`format: ${options.title}`, async () => {
+	describe(`format: ${options.title}`, () => {
 
 		it(`format`, async () => {
 

packages/tsc/tests/typecheck.spec.ts

@@ -44,7 +44,7 @@ function getTscOutput() {
 			{ paths: [path.resolve(__dirname, '../../../test-workspace')] }
 		);
 		run(tscPath);
-	} catch (err) { }
+	} catch { }
 	process.stdout.write = originalConsoleLog;
 	process.argv = originalArgv;
 	return consoleOutput;

packages/typescript-plugin/lib/common.ts

@@ -142,7 +142,7 @@ function getCompletionEntryDetails<T>(
 		// @ts-expect-error
 		if (args[6]?.__isComponentAutoImport) {
 			// @ts-expect-error
-			const { ext, suffix, originalName, newName } = args[6]?.__isComponentAutoImport;
+			const { originalName, newName } = args[6].__isComponentAutoImport;
 			for (const codeAction of details?.codeActions ?? []) {
 				for (const change of codeAction.changes) {
 					for (const textChange of change.textChanges) {
@@ -154,7 +154,7 @@ function getCompletionEntryDetails<T>(
 		// @ts-expect-error
 		if (args[6]?.__isAutoImport) {
 			// @ts-expect-error
-			const { fileName } = args[6]?.__isAutoImport;
+			const { fileName } = args[6].__isAutoImport;
 			const sourceScript = language.scripts.get(asScriptId(fileName));
 			if (sourceScript?.generated?.root instanceof VueVirtualCode) {
 				const sfc = sourceScript.generated.root.vueSfc;