Deprecate svelte field in package.json

Author: benmccann

packages/package/src/index.js

@@ -32,27 +32,10 @@ export async function build(config, cwd = process.cwd()) {
 
 	const pkg = generate_pkg(cwd, files);
 
-	if (!pkg.svelte && files.some((file) => file.is_svelte)) {
-		// Several heuristics in Kit/vite-plugin-svelte to tell Vite to mark Svelte packages
-		// rely on the "svelte" property. Vite/Rollup/Webpack plugin can all deal with it.
-		// See https://github.com/sveltejs/kit/issues/1959 for more info and related threads.
-		if (pkg.exports['.']) {
-			const svelte_export =
-				typeof pkg.exports['.'] === 'string'
-					? pkg.exports['.']
-					: pkg.exports['.'].import || pkg.exports['.'].default;
-			if (svelte_export) {
-				pkg.svelte = svelte_export;
-			} else {
-				console.warn(
-					'Cannot generate a "svelte" entry point because the "." entry in "exports" is not a string. If you set it by hand, please also set one of the options as a "svelte" entry point\n'
-				);
-			}
-		} else {
-			console.warn(
-				'Cannot generate a "svelte" entry point because the "." entry in "exports" is missing. Please specify one or set a "svelte" entry point yourself\n'
-			);
-		}
+	if (pkg.svelte) {
+		console.warn(
+			'The "svelte" field in package.json has been deprecated. Please ensure that svelte is a dependency, devDependency, or peerDependency instead.\n'
+		);
 	}
 
 	write(join(dir, 'package.json'), JSON.stringify(pkg, null, 2));

packages/package/test/fixtures/emitTypes-false/expected/package.json

@@ -9,6 +9,5 @@
 		"./Test.svelte": "./Test.svelte",
 		"./Test2.svelte": "./Test2.svelte",
 		".": "./index.js"
-	},
-	"svelte": "./index.js"
-}
+	}
+}

packages/package/test/fixtures/exports-merge/expected/package.json

@@ -17,6 +17,5 @@
 		"./Test": "./Test.svelte",
 		"./package.json": "./package.json"
 	},
-	"type": "module",
-	"svelte": "./index.js"
-}
+	"type": "module"
+}

packages/package/test/fixtures/exports-replace/expected/package.json

@@ -9,6 +9,5 @@
 			"import": "./index.js"
 		},
 		"./package.json": "./package.json"
-	},
-	"svelte": "./Test.svelte"
-}
+	}
+}

packages/package/test/fixtures/exports-replace/package.json

@@ -8,6 +8,5 @@
 		".": {
 			"import": "./index.js"
 		}
-	},
-	"svelte": "./Test.svelte"
+	}
 }

packages/package/test/fixtures/files-exclude/expected/package.json

@@ -9,6 +9,5 @@
 		"./internal": "./internal/index.js",
 		"./Test.svelte": "./Test.svelte",
 		".": "./index.js"
-	},
-	"svelte": "./index.js"
-}
+	}
+}

packages/package/test/fixtures/javascript/expected/package.json

@@ -12,6 +12,5 @@
 		"./Test2.svelte": "./Test2.svelte",
 		"./utils": "./utils.js",
 		".": "./index.js"
-	},
-	"svelte": "./index.js"
-}
+	}
+}

packages/package/test/fixtures/resolve-alias/expected/package.json

@@ -12,6 +12,5 @@
 		"./baz": "./baz.js",
 		"./sub/bar": "./sub/bar.js",
 		"./sub/foo": "./sub/foo.js"
-	},
-	"svelte": "./index.js"
+	}
 }

packages/package/test/fixtures/svelte-kit/expected/package.json

@@ -8,6 +8,5 @@
 		"./package.json": "./package.json",
 		"./Test.svelte": "./Test.svelte",
 		".": "./index.js"
-	},
-	"svelte": "./index.js"
-}
+	}
+}

packages/package/test/fixtures/typescript/expected/package.json

@@ -11,6 +11,5 @@
 		"./Test2.svelte": "./Test2.svelte",
 		"./utils": "./utils.js",
 		".": "./index.js"
-	},
-	"svelte": "./index.js"
-}
+	}
+}