chore: use dprint to format code

johnsoncodehk · johnsoncodehk · commit f6386e7b99e1 · 2025-06-23T01:20:51.000+08:00
diff --git a/.github/workflows/auto-fix.yml b/.github/workflows/auto-fix.yml
@@ -28,7 +28,11 @@ jobs:
 
       # lint
       - name: Auto-fix
-        run: pnpm run lint:fix
+        run: npm run lint:fix
+
+      # format
+      - name: Format
+        run: npm run format
 
       # commit
       - name: Commit
diff --git a/.vscode/extensions.json b/.vscode/extensions.json
@@ -0,0 +1,5 @@
+{
+	"recommendations": [
+		"dprint.dprint"
+	]
+}
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -1,16 +1,18 @@
 {
+	"dprint.path": "node_modules/dprint/dprint",
 	"[javascript]": {
-		"editor.defaultFormatter": "vscode.typescript-language-features"
+		"editor.defaultFormatter": "dprint.dprint"
+	},
+	"[typescript]": {
+		"editor.defaultFormatter": "dprint.dprint"
 	},
 	"[json]": {
-		"editor.defaultFormatter": "vscode.json-language-features"
+		"editor.defaultFormatter": "dprint.dprint"
 	},
 	"[jsonc]": {
-		"editor.defaultFormatter": "vscode.json-language-features"
-	},
-	"[typescript]": {
-		"editor.defaultFormatter": "vscode.typescript-language-features"
+		"editor.defaultFormatter": "dprint.dprint"
 	},
+	"editor.formatOnSave": true,
 	"editor.codeActionsOnSave": {
 		"source.organizeImports": "always"
 	},
@@ -27,4 +29,4 @@
 	"json.format.keepLines": true,
 	"typescript.format.semicolons": "insert",
 	"typescript.tsdk": "node_modules/typescript/lib"
-}
+}
diff --git a/dprint.json b/dprint.json
@@ -0,0 +1,18 @@
+{
+	"useTabs": true,
+	"typescript": {
+		"quoteStyle": "preferSingle",
+		"arrowFunction.useParentheses": "preferNone"
+	},
+	"json": {
+	},
+	"excludes": [
+		"**/node_modules",
+		"**/*-lock.json",
+		"packages/language-service/data"
+	],
+	"plugins": [
+		"https://plugins.dprint.dev/typescript-0.95.7.wasm",
+		"https://plugins.dprint.dev/json-0.20.0.wasm"
+	]
+}
diff --git a/extensions/vscode/package.json b/extensions/vscode/package.json
@@ -460,7 +460,7 @@
 		"watch": "rolldown --watch --config",
 		"pack": "pnpm run build:prod && vsce package",
 		"release": "pnpm run build:prod && vsce publish",
-		"gen-ext-meta": "vscode-ext-gen --scope vue --output lib/generated-meta.ts && tsslint --project tsconfig.json --fix --vscode-settings ../../.vscode/settings.json"
+		"gen-ext-meta": "vscode-ext-gen --scope vue --output lib/generated-meta.ts && cd ../.. && pnpm run format"
 	},
 	"devDependencies": {
 		"@types/node": "^22.10.4",
diff --git a/package.json b/package.json
@@ -11,12 +11,13 @@
 		"release:next": "pnpm run release --dist-tag next",
 		"test": "vitest run",
 		"test:update": "vitest run --update",
+		"format": "dprint fmt",
 		"lint": "pnpm run oxlint && pnpm run tsslint",
-		"lint:fix": "pnpm run oxlint:fix && pnpm run tsslint:fix",
+		"lint:fix": "pnpm run oxlint:fix && pnpm run tsslint:fix && npm run format",
 		"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"
+		"tsslint:fix": "pnpm run lint -- --fix"
 	},
 	"devDependencies": {
 		"@lerna-lite/cli": "^4.1.2",
@@ -25,6 +26,7 @@
 		"@tsslint/config": "^1.5.0",
 		"@tsslint/eslint": "^1.5.0",
 		"@typescript-eslint/eslint-plugin": "^8.19.0",
+		"dprint": "^0.50.0",
 		"oxlint": "^1.0.0",
 		"typescript": "latest",
 		"vitest": "^3.1.3"
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/tsslint.config.ts b/tsslint.config.ts
@@ -1,7 +1,6 @@
 import { defineConfig } from '@tsslint/config';
 import { convertRules } from '@tsslint/eslint';
 import * as path from 'node:path';
-import type * as ts from 'typescript';
 
 export default defineConfig({
 	rules: {
@@ -28,7 +27,11 @@ export default defineConfig({
 				if (packageJson.private) {
 					return;
 				}
-				const parentPackageJsonPath = ts.findConfigFile(path.dirname(path.dirname(packageJsonPath)), ts.sys.fileExists, 'package.json');
+				const parentPackageJsonPath = ts.findConfigFile(
+					path.dirname(path.dirname(packageJsonPath)),
+					ts.sys.fileExists,
+					'package.json',
+				);
 				const parentPackageJson = !!parentPackageJsonPath && parentPackageJsonPath !== packageJsonPath
 					? JSON.parse(ts.sys.readFile(parentPackageJsonPath) ?? '')
 					: {};
@@ -66,138 +69,4 @@ export default defineConfig({
 			},
 		},
 	},
-	formatting: [
-		function trailingComma({ typescript: ts, sourceFile, insert, remove }) {
-			const { text } = sourceFile;
-			ts.forEachChild(sourceFile, function visit(node) {
-				let lastNode: ts.Node | undefined;
-				let end: number | undefined;
-				let allow = true;
-
-				if (ts.isObjectLiteralExpression(node)) {
-					lastNode = node.properties[node.properties.length - 1];
-					end = node.end;
-				}
-				else if (
-					ts.isArrayLiteralExpression(node)
-					|| ts.isObjectBindingPattern(node)
-					|| ts.isArrayBindingPattern(node)
-					|| ts.isNamedImports(node)
-					|| ts.isNamedExports(node)
-					|| ts.isImportAttributes(node)
-					|| ts.isTupleTypeNode(node)
-				) {
-					lastNode = node.elements[node.elements.length - 1];
-					end = node.end;
-				}
-				else if (ts.isEnumDeclaration(node)) {
-					lastNode = node.members[node.members.length - 1];
-					end = node.end;
-				}
-				else if (ts.isCallExpression(node) || ts.isNewExpression(node)) {
-					lastNode = node.arguments?.[node.arguments.length - 1];
-					end = node.end;
-				}
-				else if (ts.isFunctionLike(node)) {
-					const last = node.parameters[node.parameters.length - 1];
-					if (last && !last.dotDotDotToken) {
-						lastNode = last;
-						const right = 'body' in node && node.body?.getStart(sourceFile) || Infinity;
-						const parenIndex = text.indexOf(')', lastNode.end);
-						if (parenIndex !== -1 && parenIndex < right) {
-							end = parenIndex + 1;
-						}
-					}
-				}
-
-				if (lastNode && end) {
-					const trailings = text.slice(lastNode.end, end - 1);
-					const commaIndex = trailings.indexOf(',');
-					if (allow && trailings.includes('\n')) {
-						if (commaIndex === -1) {
-							insert(lastNode.end, ',');
-						}
-					}
-					else if (commaIndex !== -1) {
-						const start = lastNode.end + commaIndex;
-						remove(start, start + 1);
-					}
-				}
-				ts.forEachChild(node, visit);
-			});
-		},
-		/**
-		 * @example
-		 * ```diff
-		 * interface MyInterface {
-		 * -   prop: string,
-		 * +   prop: string;
-		 * }
-		 * ```
-		 */
-		function interfacePropertySemicolon({ typescript: ts, sourceFile, replace, insert }) {
-			const { text } = sourceFile;
-			ts.forEachChild(sourceFile, function visit(node) {
-				if (ts.isInterfaceDeclaration(node)) {
-					for (const member of node.members) {
-						if (text[member.end - 1] === ',') {
-							replace(member.end - 1, member.end, ';');
-						}
-						else if (text[member.end - 1] !== ';') {
-							insert(member.end, ';');
-						}
-					}
-				}
-				ts.forEachChild(node, visit);
-			});
-		},
-		/**
-		 * @example
-		 * ```diff
-		 * - const foo = (bar) => {};
-		 * + const foo = bar => {};
-		 * ```
-		 */
-		function arrowParens({ typescript: ts, sourceFile, remove }) {
-			ts.forEachChild(sourceFile, function visit(node) {
-				if (
-					ts.isArrowFunction(node)
-					&& node.parameters.length === 1
-					&& !node.type
-				) {
-					const parameter = node.parameters[0];
-					if (
-						ts.isIdentifier(parameter.name)
-						&& !parameter.type
-						&& !parameter.dotDotDotToken
-						&& !parameter.initializer
-						&& sourceFile.text[parameter.getStart(sourceFile) - 1] === '('
-						&& sourceFile.text[parameter.getEnd()] === ')'
-					) {
-						remove(parameter.getStart(sourceFile) - 1, parameter.getStart(sourceFile));
-						remove(parameter.getEnd(), parameter.getEnd() + 1);
-					}
-				}
-				ts.forEachChild(node, visit);
-			});
-		},
-		function noUnnecessaryParentheses({ typescript: ts, sourceFile, remove }) {
-			ts.forEachChild(sourceFile, function visit(node) {
-				if (ts.isParenthesizedExpression(node)) {
-					if (
-						ts.isIdentifier(node.expression)
-						|| ts.isPropertyAccessExpression(node.expression)
-						|| ts.isElementAccessExpression(node.expression)
-						|| ts.isCallExpression(node.expression)
-					) {
-						const start = node.getStart(sourceFile);
-						const end = node.getEnd();
-						remove(start, start + 1);
-						remove(end - 1, end);
-					}
-				}
-				ts.forEachChild(node, visit);
-			});
-		},
-	],
 });

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +{
 +	"recommendations": [
 +		"dprint.dprint"
 +	]
 +}