refactor: migrate to tsdown

Author: lishaduck

.cspell.json

@@ -52,7 +52,7 @@
     "tanstack",
     "tsbuildinfo",
     "tsdoc",
-    "tsup",
+    "tsdown",
     "typecheck",
     "vercel"
   ],

.vscode/settings.json

@@ -20,11 +20,5 @@
   "editor.codeActionsOnSave": {
     "source.fixAll.eslint": "explicit",
     "source.organizeImports": "never"
-  },
-  "json.schemas": [
-    {
-      "url": "https://cdn.jsdelivr.net/npm/tsup/schema.json",
-      "fileMatch": ["package.json", "tsup.config.json"]
-    }
-  ]
+  }
 }

knip.jsonc

@@ -6,10 +6,14 @@
     "apps/cli-playground",
     "apps/docs-website",
   ],
-  "ignoreDependencies": [
-    "@turbo/gen",
-    "eslint-import-resolver-typescript",
-    "eslint-config-sheriff",
-    "@typescript-eslint/parser",
-  ],
+  "ignore": ["**/tsdown.config.ts"],
+  "ignoreDependencies": ["@turbo/gen"],
+  "workspaces": {
+    "./packages/eslint-config-sheriff": {
+      "ignoreDependencies": [
+        "eslint-import-resolver-typescript",
+        "@typescript-eslint/parser",
+      ],
+    },
+  },
 }

package.json

@@ -12,14 +12,13 @@
     "format": "prettier . --write --cache --cache-location=node_modules/.cache/.prettiercache --log-level=warn",
     "format:check": "prettier . --check --cache --cache-location=node_modules/.cache/.prettiercache",
     "lint": "turbo run lint",
-    "publint": "turbo run publint --concurrency=1",
     "are-the-types-wrong": "turbo run are-the-types-wrong",
     "manypkg": "manypkg check",
     "typesync": "typesync --dry=fail",
     "typecheck": "turbo run typecheck",
     "knip": "knip",
     "validate-config": "turbo run validate-config",
-    "merge-checks": "pnpm build && turbo run publint manypkg typesync knip typecheck lint validate-config are-the-types-wrong && pnpm check-deduped-deps",
+    "merge-checks": "turbo run manypkg typesync knip typecheck lint validate-config are-the-types-wrong && pnpm check-deduped-deps",
     "check-deduped-deps": "pnpm dedupe --check",
     "clean": "turbo run clean && rm -rf .turbo",
     "delete-node-modules": "pnpm exec rm -rf node_modules && pnpm -r exec rm -rf node_modules",
@@ -56,10 +55,13 @@
     },
     "onlyBuiltDependencies": [
       "@swc/core",
-      "core-js",
       "esbuild",
+      "rolldown",
       "sharp",
       "unrs-resolver"
+    ],
+    "ignoredBuiltDependencies": [
+      "core-js"
     ]
   }
 }

packages/eslint-config-sheriff/dts-config.json

@@ -24,6 +24,6 @@ export default tseslint.config(
     },
   },
   {
-    ignores: ['tsup.config.js'],
+    ignores: ['tsdown.config.ts'],
   },
 );

packages/eslint-config-sheriff/eslint.config.ts

@@ -45,13 +45,11 @@
   "bug": "https://github.com/AndreaPontrandolfo/sheriff/labels/bug",
   "scripts": {
     "clean": "rm -rf .turbo dist node_modules/.cache",
-    "re-bundle-types": "dts-bundle-generator --config dts-config.json",
     "are-the-types-wrong": "attw --pack --profile=esm-only",
-    "build": "tsup && pnpm re-bundle-types",
-    "build:watch": "tsup --watch",
+    "build": "tsdown",
+    "build:watch": "tsdown --watch",
     "typecheck": "tsc",
-    "lint": "eslint src --max-warnings=0",
-    "publint": "publint"
+    "lint": "eslint src --max-warnings=0"
   },
   "dependencies": {
     "@eslint-react/eslint-plugin": "^1.10.1",
@@ -106,12 +104,11 @@
     "@types/node": "^22.10.3",
     "@typescript-eslint/parser": "^8.31.0",
     "@typescript-eslint/utils": "^8.31.0",
-    "dts-bundle-generator": "^9.5.1",
     "eslint": "^9.25.1",
     "jiti": "^2.4.2",
-    "publint": "^0.3.11",
+    "publint": "^0.3.12",
     "tsconfig": "workspace:*",
-    "tsup": "^8.3.0",
+    "tsdown": "^0.9.7",
     "typescript": "^5.8.3"
   },
   "publishConfig": {

packages/eslint-config-sheriff/package.json

@@ -99,7 +99,6 @@ export const getBaseConfig = (
     },
     {
       files: [supportedFileTypes],
-      // @ts-expect-error
       rules: {
         ...typescriptHandPickedRules,
         ...getTsNamingConventionRule({ isTsx: false }),

packages/eslint-config-sheriff/src/getBaseConfig.ts

@@ -1,9 +1,7 @@
-/* eslint-disable @typescript-eslint/explicit-module-boundary-types */
-
 import confusingBrowserGlobals from 'confusing-browser-globals';
 import type { TSESLint } from '@typescript-eslint/utils';
 
-export const getBaseEslintHandPickedRules = () => {
+export const getBaseEslintHandPickedRules = (): TSESLint.FlatConfig.Rules => {
   return {
     'func-style': 2,
     'no-promise-executor-return': 2,
@@ -99,5 +97,5 @@ export const getBaseEslintHandPickedRules = () => {
     'no-shadow': 0, // we are using the @typescript/eslint version
     'default-param-last': 0, // we are using the @typescript/eslint version
     'arrow-body-style': 0, // we are using the eslint-plugin-arrow-return-style version
-  } as const satisfies TSESLint.FlatConfig.Rules;
+  };
 };

packages/eslint-config-sheriff/src/handpickedRules/getBaseEslintHandPickedRules.ts

@@ -1,6 +1,6 @@
 import type { TSESLint } from '@typescript-eslint/utils';
 
-export const importHandPickedRules = {
+export const importHandPickedRules: TSESLint.FlatConfig.Rules = {
   // https://github.com/import-js/eslint-plugin-import/issues/2913
   // https://github.com/import-js/eslint-plugin-import/issues/2673
   'import/newline-after-import': 2,
@@ -11,4 +11,4 @@ export const importHandPickedRules = {
   'import/no-duplicates': [2, { 'prefer-inline': true }],
   'import/no-useless-path-segments': [2, { noUselessIndex: true }],
   // 'import/consistent-type-specifier-style': [2, 'prefer-inline'], // this is not actually needed when "@typescript-eslint/no-import-type-side-effects is set. Explanation here: https://github.com/import-js/eslint-plugin-import/issues/2676#issuecomment-1407107260
-} as const satisfies TSESLint.FlatConfig.Rules;
+};