Generate type declarations with dts-buddy (#8702)

* use dts-buddy

* remove debug output

* remove existing type generation script

* fix package.json

* update gitignore

* bump dts-buddy

* remove unused action entry point

* add svelte/compiler and svelte/types/compiler/preprocess modules

* bump dts-buddy

* annoying

* changeset

* bump dts-buddy

* get rid of .d.ts files

* another one

* Update packages/svelte/package.json

Co-authored-by: gtmnayan <[email protected]>

---------

Co-authored-by: Rich Harris <[email protected]>
Co-authored-by: gtmnayan <[email protected]>

Author: 3 people

.changeset/green-sheep-learn.md

@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+Generate type declarations with dts-buddy

packages/svelte/.gitignore

@@ -1,15 +1,6 @@
 *.map
 /src/compiler/compile/internal_exports.js
-/compiler.d.ts
 /compiler.cjs
-/index.d.ts
-/action.d.ts
-/internal.d.ts
-/store.d.ts
-/easing.d.ts
-/motion.d.ts
-/transition.d.ts
-/animate.d.ts
 /scratch/
 /test/*/samples/_
 /test/runtime/shards

packages/svelte/generate-types.js

@@ -24,42 +24,42 @@
   "exports": {
     "./package.json": "./package.json",
     ".": {
-      "types": "./types/runtime/index.d.ts",
+      "types": "./types/index.d.ts",
       "browser": {
         "import": "./src/runtime/index.js"
       },
       "import": "./src/runtime/ssr.js"
     },
     "./compiler": {
-      "types": "./types/compiler/index.d.ts",
+      "types": "./types/index.d.ts",
       "import": "./src/compiler/index.js",
       "require": "./compiler.cjs"
     },
     "./action": {
-      "types": "./types/runtime/action/index.d.ts"
+      "types": "./types/index.d.ts"
     },
     "./animate": {
-      "types": "./types/runtime/animate/index.d.ts",
+      "types": "./types/index.d.ts",
       "import": "./src/runtime/animate/index.js"
     },
     "./easing": {
-      "types": "./types/runtime/easing/index.d.ts",
+      "types": "./types/index.d.ts",
       "import": "./src/runtime/easing/index.js"
     },
     "./internal": {
-      "types": "./types/runtime/internal/index.d.ts",
+      "types": "./types/index.d.ts",
       "import": "./src/runtime/internal/index.js"
     },
     "./motion": {
-      "types": "./types/runtime/motion/index.d.ts",
+      "types": "./types/index.d.ts",
       "import": "./src/runtime/motion/index.js"
     },
     "./store": {
-      "types": "./types/runtime/store/index.d.ts",
+      "types": "./types/index.d.ts",
       "import": "./src/runtime/store/index.js"
     },
     "./transition": {
-      "types": "./types/runtime/transition/index.d.ts",
+      "types": "./types/index.d.ts",
       "import": "./src/runtime/transition/index.js"
     },
     "./elements": {
@@ -69,18 +69,18 @@
   "engines": {
     "node": ">=16"
   },
-  "types": "types/runtime/index.d.ts",
+  "types": "types/index.d.ts",
   "scripts": {
     "format": "prettier . --cache --plugin-search-dir=. --write",
     "check": "prettier . --cache --plugin-search-dir=. --check",
     "test": "vitest run && echo \"manually check that there are no type errors in test/types by opening the files in there\"",
-    "build": "rollup -c && pnpm tsd",
+    "build": "rollup -c && pnpm types",
     "prepare": "pnpm build",
     "generate:version": "node ./scripts/generate-version.js",
     "dev": "rollup -cw",
     "posttest": "agadoo src/internal/index.js",
     "prepublishOnly": "pnpm lint && pnpm build && pnpm test",
-    "tsd": "node ./generate-types.js",
+    "types": "node ./scripts/generate-dts.js",
     "lint": "eslint \"{src,test}/**/*.{ts,js}\" --cache"
   },
   "repository": {
@@ -126,6 +126,7 @@
     "@types/node": "^14.14.31",
     "@typescript-eslint/eslint-plugin": "^5.58.0",
     "agadoo": "^3.0.0",
+    "dts-buddy": "^0.1.2",
     "esbuild": "^0.17.19",
     "happy-dom": "^9.18.3",
     "jsdom": "^21.1.1",

packages/svelte/package.json

@@ -17,23 +17,6 @@ fs.writeFileSync(
 		`export default new Set(${JSON.stringify(Object.keys(internal))});`
 );
 
-// Generate d.ts files for runtime entrypoints so that TS can find it also without `"moduleResolution": "bundler"`
-fs.readdirSync('src/runtime', { withFileTypes: true })
-	.filter((dirent) => dirent.isDirectory())
-	.forEach((dirent) =>
-		fs.writeFileSync(
-			`${dirent.name}.d.ts`,
-			`export * from './types/runtime/${dirent.name}/index.js';`
-		)
-	);
-
-fs.writeFileSync('./index.d.ts', `export * from './types/runtime/index.js';`);
-
-fs.writeFileSync(
-	'./compiler.d.ts',
-	`export { compile, parse, preprocess, walk, VERSION } from './types/compiler/index.js';`
-);
-
 /**
  * @type {import("rollup").RollupOptions[]}
  */

packages/svelte/rollup.config.js

@@ -0,0 +1,16 @@
+import { createBundle } from 'dts-buddy';
+
+await createBundle({
+	output: 'types/index.d.ts',
+	modules: {
+		svelte: 'src/runtime/public.d.ts',
+		'svelte/compiler': 'src/compiler/public.d.ts',
+		'svelte/types/compiler/preprocess': 'src/compiler/preprocess/public.d.ts',
+		'svelte/action': 'src/runtime/action/public.d.ts',
+		'svelte/animate': 'src/runtime/animate/public.d.ts',
+		'svelte/easing': 'src/runtime/easing/index.js',
+		'svelte/motion': 'src/runtime/motion/public.d.ts',
+		'svelte/store': 'src/runtime/store/public.d.ts',
+		'svelte/transition': 'src/runtime/transition/public.d.ts'
+	}
+});

packages/svelte/scripts/generate-dts.js

@@ -1,2 +1,3 @@
 export { CompileOptions, EnableSourcemap, CssHashGetter } from './interfaces';
 export * from './preprocess/public.js';
+export * from './index.js';

packages/svelte/src/compiler/public.d.ts

@@ -12,3 +12,5 @@ export interface FlipParams {
 	duration?: number | ((len: number) => number);
 	easing?: (t: number) => number;
 }
+
+export * from './index.js';

packages/svelte/src/runtime/action/index.js

@@ -91,3 +91,5 @@ export interface EventDispatcher<EventMap extends Record<string, any>> {
 			: [type: Type, parameter: EventMap[Type], options?: DispatchOptions]
 	): boolean;
 }
+
+export * from './index.js';