build: Switch from npmignore to files field (#9991)

Luca Forstner · web-flow · commit ff95fd5367d0 · 2024-01-02T13:15:02.000+01:00
diff --git a/.npmignore b/.npmignore
diff --git a/packages/angular/.npmignore b/packages/angular/.npmignore
diff --git a/packages/astro/.npmignore b/packages/astro/.npmignore
diff --git a/packages/astro/package.json b/packages/astro/package.json
@@ -17,6 +17,12 @@
     "node": ">=18.14.1"
   },
   "type": "module",
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.client.js",
   "module": "build/esm/index.server.js",
   "browser": "build/esm/index.client.js",
diff --git a/packages/browser/package.json b/packages/browser/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/npm/cjs/index.js",
   "module": "build/npm/esm/index.js",
   "types": "build/npm/types/index.d.ts",
diff --git a/packages/bun/package.json b/packages/bun/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/esm/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/core/package.json b/packages/core/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/ember/.npmignore b/packages/ember/.npmignore
@@ -1,6 +1,3 @@
-# Disable rules of .npmignore in workspace root for this package
-!*
-
 # compiled output
 /dist/
 /tmp/
diff --git a/packages/eslint-config-sdk/.npmignore b/packages/eslint-config-sdk/.npmignore
diff --git a/packages/eslint-config-sdk/package.json b/packages/eslint-config-sdk/package.json
@@ -14,6 +14,9 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "src"
+  ],
   "main": "src/index.js",
   "publishConfig": {
     "access": "public"
diff --git a/packages/eslint-plugin-sdk/.npmignore b/packages/eslint-plugin-sdk/.npmignore
diff --git a/packages/eslint-plugin-sdk/package.json b/packages/eslint-plugin-sdk/package.json
@@ -14,6 +14,7 @@
   "engines": {
     "node": ">=8"
   },
+  "files": ["src"],
   "main": "src/index.js",
   "publishConfig": {
     "access": "public"
diff --git a/packages/feedback/package.json b/packages/feedback/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=12"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/npm/cjs/index.js",
   "module": "build/npm/esm/index.js",
   "types": "build/npm/types/index.d.ts",
diff --git a/packages/gatsby/.npmignore b/packages/gatsby/.npmignore
diff --git a/packages/gatsby/package.json b/packages/gatsby/package.json
@@ -13,6 +13,16 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8",
+    "gatsby-browser.js",
+    "gatsby-node.js",
+    "gatsby-browser.d.ts",
+    "gatsby-node.d.ts"
+  ],
   "main": "build/cjs/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/hub/package.json b/packages/hub/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/integrations/package.json b/packages/integrations/package.json
@@ -12,6 +12,12 @@
   "publishConfig": {
     "access": "public"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/npm/cjs/index.js",
   "module": "build/npm/esm/index.js",
   "types": "build/npm/types/index.d.ts",
diff --git a/packages/node-experimental/package.json b/packages/node-experimental/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=14"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/node/package.json b/packages/node/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/opentelemetry-node/package.json b/packages/opentelemetry-node/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/opentelemetry/package.json b/packages/opentelemetry/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=14"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/react/package.json b/packages/react/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/remix/.npmignore b/packages/remix/.npmignore
diff --git a/packages/remix/package.json b/packages/remix/package.json
@@ -12,6 +12,13 @@
   "engines": {
     "node": ">=14"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8",
+    "scripts"
+  ],
   "main": "build/cjs/index.server.js",
   "module": "build/esm/index.server.js",
   "browser": "build/esm/index.client.js",
diff --git a/packages/replay-worker/package.json b/packages/replay-worker/package.json
@@ -12,6 +12,12 @@
       ]
     }
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "sideEffects": false,
   "private": true,
   "scripts": {
diff --git a/packages/replay/package.json b/packages/replay/package.json
@@ -12,6 +12,12 @@
       ]
     }
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "sideEffects": false,
   "scripts": {
     "build": "run-p build:transpile build:types build:bundle",
diff --git a/packages/serverless/package.json b/packages/serverless/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=10"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/npm/cjs/index.js",
   "module": "build/npm/esm/index.js",
   "types": "build/npm/types/index.d.ts",
diff --git a/packages/svelte/package.json b/packages/svelte/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/sveltekit/package.json b/packages/sveltekit/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=16"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.server.js",
   "module": "build/esm/index.server.js",
   "browser": "build/esm/index.client.js",
diff --git a/packages/tracing-internal/package.json b/packages/tracing-internal/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/tracing/package.json b/packages/tracing/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/npm/cjs/index.js",
   "module": "build/npm/esm/index.js",
   "types": "build/npm/types/index.d.ts",
diff --git a/packages/types/package.json b/packages/types/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/typescript/.npmignore b/packages/typescript/.npmignore
diff --git a/packages/typescript/package.json b/packages/typescript/package.json
@@ -10,6 +10,9 @@
   "publishConfig": {
     "access": "public"
   },
+  "files": [
+    "tsconfig.json"
+  ],
   "peerDependencies": {
     "typescript": "4.9.5"
   },
diff --git a/packages/utils/package.json b/packages/utils/package.json
@@ -9,6 +9,12 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "cjs",
+    "esm",
+    "types",
+    "types-ts3.8"
+  ],
   "main": "build/cjs/index.js",
   "module": "build/esm/index.js",
   "types": "build/types/index.d.ts",
diff --git a/packages/vercel-edge/package.json b/packages/vercel-edge/package.json
diff --git a/packages/vue/package.json b/packages/vue/package.json
diff --git a/packages/wasm/package.json b/packages/wasm/package.json
diff --git a/scripts/prepack.ts b/scripts/prepack.ts
diff --git a/scripts/tarball-checksums.sh b/scripts/tarball-checksums.sh

-Original file line number
+Diff line change
@@ @@ -1,6 +1,3 @@ @@
 -# Disable rules of .npmignore in workspace root for this package
 -!*
+-
 # compiled output
 /dist/
 /tmp/
Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,12 @@`
`12`	`12`	`]`
`13`	`13`	`}`
`14`	`14`	`},`
	`15`	`+ "files": [`
	`16`	`+ "cjs",`
	`17`	`+ "esm",`
	`18`	`+ "types",`
	`19`	`+ "types-ts3.8"`
	`20`	`+ ],`
`15`	`21`	`"sideEffects": false,`
`16`	`22`	`"private": true,`
`17`	`23`	`"scripts": {`