style: remove semi

Author: lihbr

.prettierrc

@@ -1,33 +1,34 @@
 {
 	"plugins": ["@trivago/prettier-plugin-sort-imports", "prettier-plugin-jsdoc"],
+	"jsdocCapitalizeDescription": false,
+	"jsdocSeparateReturnsFromParam": true,
+	"jsdocSeparateTagGroups": true,
+	"jsdocCommentLineStrategy": "multiline",
+	"tsdoc": true,
 	"importOrder": [
 		"^vitest(/|$)",
-		"^node:(.*)$",
+		"^node:",
 		"<THIRD_PARTY_MODULES>",
-		"^(?!.*/src(?:/|$)).*/__testutils__(?:/|$)",
-		"^(?!.*/src(?:/|$)).*/__fixtures__(?:/|$)",
-		"^(?!.*/src(?:/|$)).*/lib(?:/|$)",
-		"^(?!.*/src(?:/|$)).*/types(?:/|$)",
-		"^(?!.*/src(?:/|$)).*/errors(?:/|$)",
-		"^(?!.*/src(?:/|$))../../../../../",
-		"^(?!.*/src(?:/|$))../../../../",
-		"^(?!.*/src(?:/|$))../../../",
-		"^(?!.*/src(?:/|$))../../",
-		"^(?!.*/src(?:/|$))../",
-		"^(?!.*/src(?:/|$))./(.*/)+",
-		"^(?!.*/src(?:/|$))./",
-		"/src(?:/|$)"
+		"^(?!.*/src(/|$)).*/__testutils__(/|$)",
+		"^(?!.*/src(/|$)).*/__fixtures__(/|$)",
+		"^(?!.*/src(/|$)).*/lib(/|$)",
+		"^(?!.*/src(/|$)).*/types(/|$)",
+		"^(?!.*/src(/|$)).*/errors(/|$)",
+		"^(?!.*/src(/|$))../../../../../",
+		"^(?!.*/src(/|$))../../../../",
+		"^(?!.*/src(/|$))../../../",
+		"^(?!.*/src(/|$))../../",
+		"^(?!.*/src(/|$))../",
+		"^(?!.*/src(/|$))./(.*/)+",
+		"^(?!.*/src(/|$))./",
+		"/src(/|$)"
 	],
 	"importOrderSeparation": true,
 	"importOrderSortSpecifiers": true,
 	"importOrderGroupNamespaceSpecifiers": true,
-	"jsdocSeparateReturnsFromParam": true,
-	"jsdocSeparateTagGroups": true,
-	"jsdocCommentLineStrategy": "multiline",
-	"tsdoc": true,
 	"printWidth": 80,
 	"useTabs": true,
-	"semi": true,
+	"semi": false,
 	"singleQuote": false,
 	"quoteProps": "as-needed",
 	"jsxSingleQuote": false,

.size-limit.cjs

@@ -1,28 +1,28 @@
-const pkg = require("./package.json");
+const pkg = require("./package.json")
 
 module.exports = [
 	...new Set([
 		pkg.main,
 		pkg.module,
 		...Object.values(pkg.exports).flatMap((exportValue) => {
 			if (typeof exportValue === "string") {
-				return exportValue;
+				return exportValue
 			} else {
-				return Object.values(exportValue);
+				return Object.values(exportValue)
 			}
 		}),
 	]),
 ]
 	.filter((path) => {
-		return path && path !== "./package.json";
+		return path && path !== "./package.json"
 	})
 	.map((path) => {
 		return {
 			path,
 			modifyEsbuildConfig(config) {
-				config.platform = "node";
+				config.platform = "node"
 
-				return config;
+				return config
 			},
-		};
-	});
+		}
+	})

eslint.config.mjs

@@ -1,7 +1,7 @@
-import eslint from "@eslint/js";
-import prettier from "eslint-plugin-prettier/recommended";
-import tsdoc from "eslint-plugin-tsdoc";
-import tseslint from "typescript-eslint";
+import eslint from "@eslint/js"
+import prettier from "eslint-plugin-prettier/recommended"
+import tsdoc from "eslint-plugin-tsdoc"
+import tseslint from "typescript-eslint"
 
 export default tseslint.config(
 	{
@@ -41,4 +41,4 @@ export default tseslint.config(
 			"tsdoc/syntax": "warn",
 		},
 	},
-);
+)

package-lock.json

@@ -56,6 +56,7 @@
 		"@types/fs-extra": "^11.0.4",
 		"@vitest/coverage-v8": "^2.1.8",
 		"eslint": "^9.16.0",
+		"eslint-config-prettier": "^9.1.0",
 		"eslint-plugin-prettier": "^5.2.1",
 		"eslint-plugin-tsdoc": "^0.4.0",
 		"prettier": "^3.4.1",

package.json

@@ -1,26 +1,26 @@
-import type { Plugin } from "vite";
+import type { Plugin } from "vite"
 
-import { readPackageJSON } from "./lib/readPackageJSON";
+import { readPackageJSON } from "./lib/readPackageJSON"
 
-import type { Options } from "./types";
+import type { Options } from "./types"
 
-import { extendConfigPlugin, moveTypeDeclarationsPlugin } from "./plugins";
+import { extendConfigPlugin, moveTypeDeclarationsPlugin } from "./plugins"
 
 const DEFAULT_OPTIONS = {
 	dts: true,
 	internalDependencies: [],
 	srcDir: "src",
-};
+}
 
 const sdkPlugin = (rawOptions?: Partial<Options>): (Plugin | null)[] => {
 	const options: Options = {
 		...DEFAULT_OPTIONS,
 		packageJSON: rawOptions?.packageJSON || readPackageJSON(),
 		...rawOptions,
-	};
+	}
 
-	return [extendConfigPlugin(options), moveTypeDeclarationsPlugin(options)];
-};
+	return [extendConfigPlugin(options), moveTypeDeclarationsPlugin(options)]
+}
 
-export default sdkPlugin;
-export { Options };
+export default sdkPlugin
+export { Options }

src/index.ts

@@ -2,13 +2,13 @@
  ** Core logic from https://github.com/sindresorhus/builtin-modules
  ** Many thanks to @sindresorhus
  */
-import Module from "node:module";
+import Module from "node:module"
 
-const ignoreList = ["sys"];
+const ignoreList = ["sys"]
 
 export const builtins = Module.builtinModules
 	.filter(
 		(x) =>
 			!/^_|^(internal|v8|node-inspect)\/|\//.test(x) && !ignoreList.includes(x),
 	)
-	.sort();
+	.sort()

src/lib/builtins.ts

@@ -1,20 +1,20 @@
-import * as fs from "node:fs";
-import * as path from "node:path";
+import * as fs from "node:fs"
+import * as path from "node:path"
 
-import type { PackageJSON } from "../types";
+import type { PackageJSON } from "../types"
 
 export const readPackageJSON = (rootDir = process.cwd()): PackageJSON => {
 	try {
-		const blob = fs.readFileSync(path.resolve(rootDir, "package.json"), "utf8");
+		const blob = fs.readFileSync(path.resolve(rootDir, "package.json"), "utf8")
 
-		return JSON.parse(blob);
+		return JSON.parse(blob)
 	} catch {
 		if (rootDir === "/") {
 			throw new Error(
 				`Could not locate \`package.json\` in \`${process.cwd()}\` or its parent directories.`,
-			);
+			)
 		}
 
-		return readPackageJSON(path.resolve(rootDir, ".."));
+		return readPackageJSON(path.resolve(rootDir, ".."))
 	}
-};
+}

src/lib/readPackageJSON.ts

@@ -1,14 +1,14 @@
-import * as path from "node:path";
+import * as path from "node:path"
 
-import typescript from "@rollup/plugin-typescript";
-import { defuFn } from "defu";
-import renameNodeModules from "rollup-plugin-rename-node-modules";
-import type { Plugin, UserConfig } from "vite";
-import { defineConfig } from "vite";
+import typescript from "@rollup/plugin-typescript"
+import { defuFn } from "defu"
+import renameNodeModules from "rollup-plugin-rename-node-modules"
+import type { Plugin, UserConfig } from "vite"
+import { defineConfig } from "vite"
 
-import { builtins } from "../lib/builtins";
+import { builtins } from "../lib/builtins"
 
-import type { Options } from "../types";
+import type { Options } from "../types"
 
 export const extendConfigPlugin = (options: Options): Plugin => {
 	const getDefaultConfig = (userConfig: UserConfig) =>
@@ -20,15 +20,15 @@ export const extendConfigPlugin = (options: Options): Plugin => {
 					fileName: (format) => {
 						switch (format) {
 							case "es": {
-								return "[name].js";
+								return "[name].js"
 							}
 
 							case "cjs": {
-								return "[name].cjs";
+								return "[name].cjs"
 							}
 						}
 
-						throw new Error(`Unsupported format: ${format}`);
+						throw new Error(`Unsupported format: ${format}`)
 					},
 				},
 				rollupOptions: {
@@ -69,18 +69,18 @@ export const extendConfigPlugin = (options: Options): Plugin => {
 				minify: false,
 				sourcemap: true,
 			},
-		});
+		})
 
 	return {
 		name: "sdk:extend-config",
 		config: (userConfig, env) => {
 			// Disable config extension in test otherwise coverage runs twice because of Vitest's watcher, see:
 			// https://github.com/vitest-dev/vitest/blob/main/packages/vitest/src/node/core.ts#L382-L426
 			if (env.mode === "test") {
-				return;
+				return
 			}
 
-			return defuFn(userConfig, getDefaultConfig(userConfig));
+			return defuFn(userConfig, getDefaultConfig(userConfig))
 		},
-	};
-};
+	}
+}

src/plugins/extendConfig.ts

@@ -1,2 +1,2 @@
-export { extendConfigPlugin } from "./extendConfig";
-export { moveTypeDeclarationsPlugin } from "./moveTypeDeclarations";
+export { extendConfigPlugin } from "./extendConfig"
+export { moveTypeDeclarationsPlugin } from "./moveTypeDeclarations"