Merge pull request #1470 from glimmerjs/fix-npm-package-contents

Fix the package contents of each package, only publishing dist
glimmerjs · Oct 27, 2023 · f008604 · f008604
2 parents e0c0ae0 + 8302817
commit f008604
Show file tree

Hide file tree

Showing 23 changed files with 86 additions and 39 deletions.
diff --git a/packages/@glimmer-workspace/eslint-plugin/package.json b/packages/@glimmer-workspace/eslint-plugin/package.json
@@ -9,16 +9,16 @@
     "eslint-import-resolver-typescript": "^3.5.5",
     "eslint-plugin-import": "^2.27.5",
     "eslint-plugin-n": "^15.7.0",
-    "eslint-plugin-regexp": "^1.15.0",
     "eslint-plugin-qunit": "^7.3.4",
+    "eslint-plugin-regexp": "^1.15.0",
     "eslint-plugin-simple-import-sort": "^10.0.0",
     "eslint-plugin-unused-imports": "^2.0.0"
   },
   "peerDependencies": {
     "eslint": "^8.40.0"
   },
   "devDependencies": {
-    "eslint": "^8.40.0",
-    "@types/eslint": "^8.37.0"
+    "@types/eslint": "^8.37.0",
+    "eslint": "^8.40.0"
   }
 }
diff --git a/packages/@glimmer/.eslintrc.cjs b/packages/@glimmer/.eslintrc.cjs
@@ -15,6 +15,10 @@ module.exports = {
       },
       plugins: ['@glimmer-workspace'],
       extends: ['plugin:@glimmer-workspace/recommended'],
+      rules: {
+        // We don't publish source, we build to a dist directory
+        'n/no-unpublished-import': ['off'],
+      },
     },
     // QUnit is a weird package, and there are some issues open about fixing it
     // - https://github.com/qunitjs/qunit/issues/1729

diff --git a/packages/@glimmer/compiler/package.json b/packages/@glimmer/compiler/package.json
@@ -11,6 +11,9 @@
   "engines": {
     "node": ">= 16.0.0"
   },
+  "files": [
+    "dist"
+  ],
   "publishConfig": {
     "access": "public",
     "main": "dist/index.js",

diff --git a/packages/@glimmer/debug/package.json b/packages/@glimmer/debug/package.json
@@ -9,6 +9,9 @@
     "types": "./index.ts",
     "development": "./index.ts"
   },
+  "files": [
+    "dist"
+  ],
   "publishConfig": {
     "access": "public",
     "main": "dist/index.js",

diff --git a/packages/@glimmer/destroyable/package.json b/packages/@glimmer/destroyable/package.json
@@ -7,6 +7,9 @@
   "license": "MIT",
   "main": "index.ts",
   "types": "index.ts",
+  "files": [
+    "dist"
+  ],
   "publishConfig": {
     "access": "public",
     "main": "dist/index.js",

diff --git a/packages/@glimmer/dom-change-list/package.json b/packages/@glimmer/dom-change-list/package.json
@@ -15,6 +15,9 @@
       "default": "./dist/index.js"
     }
   },
+  "files": [
+    "dist"
+  ],
   "dependencies": {
     "@glimmer/util": "workspace:^",
     "@glimmer/interfaces": "workspace:^"

diff --git a/packages/@glimmer/encoder/package.json b/packages/@glimmer/encoder/package.json
@@ -15,6 +15,9 @@
       "default": "./dist/index.js"
     }
   },
+  "files": [
+    "dist"
+  ],
   "dependencies": {
     "@glimmer/interfaces": "workspace:^",
     "@glimmer/vm": "workspace:^"

diff --git a/packages/@glimmer/global-context/package.json b/packages/@glimmer/global-context/package.json
@@ -5,6 +5,9 @@
   "repository": "https://github.com/glimmerjs/glimmer-vm/tree/master/packages/@glimmer/global-context",
   "main": "index.ts",
   "types": "index.ts",
+  "files": [
+    "dist"
+  ],
   "publishConfig": {
     "access": "public",
     "main": "dist/index.js",

diff --git a/packages/@glimmer/interfaces/package.json b/packages/@glimmer/interfaces/package.json
@@ -7,6 +7,9 @@
   "exports": {
     "types": "./index.d.ts"
   },
+  "files": [
+    "dist"
+  ],
   "scripts": {
     "test:lint": "eslint .",
     "test:types": "tsc --noEmit -p ../tsconfig.json"

diff --git a/packages/@glimmer/manager/package.json b/packages/@glimmer/manager/package.json
@@ -14,6 +14,9 @@
     "@glimmer/util": "workspace:^",
     "@glimmer/validator": "workspace:^"
   },
+  "files": [
+    "dist"
+  ],
   "devDependencies": {
     "@glimmer/local-debug-flags": "workspace:^",
     "@glimmer-workspace/build-support": "workspace:^"

diff --git a/packages/@glimmer/node/package.json b/packages/@glimmer/node/package.json
@@ -9,6 +9,9 @@
     "@glimmer/util": "workspace:^",
     "@simple-dom/document": "^1.4.0"
   },
+  "files": [
+    "dist"
+  ],
   "devDependencies": {
     "@types/qunit": "^2.19.7",
     "@glimmer/compiler": "workspace:^",

diff --git a/packages/@glimmer/opcode-compiler/package.json b/packages/@glimmer/opcode-compiler/package.json
@@ -15,6 +15,9 @@
     "@glimmer/vm": "workspace:^",
     "@glimmer/wire-format": "workspace:^"
   },
+  "files": [
+    "dist"
+  ],
   "devDependencies": {
     "@glimmer/local-debug-flags": "workspace:^",
     "@glimmer-workspace/build-support": "workspace:^"

diff --git a/packages/@glimmer/owner/package.json b/packages/@glimmer/owner/package.json
@@ -8,6 +8,9 @@
   "dependencies": {
     "@glimmer/util": "workspace:^"
   },
+  "files": [
+    "dist"
+  ],
   "devDependencies": {
     "@glimmer/local-debug-flags": "workspace:^",
     "@glimmer-workspace/build-support": "workspace:^"

diff --git a/packages/@glimmer/program/package.json b/packages/@glimmer/program/package.json
@@ -3,6 +3,9 @@
   "type": "module",
   "version": "0.85.3",
   "repository": "https://github.com/glimmerjs/glimmer-vm/tree/master/packages/@glimmer/program",
+  "files": [
+    "dist"
+  ],
   "dependencies": {
     "@glimmer/env": "0.1.7",
     "@glimmer/util": "workspace:^",

diff --git a/packages/@glimmer/reference/package.json b/packages/@glimmer/reference/package.json
@@ -9,6 +9,9 @@
     "url": "https://github.com/glimmerjs/glimmer-vm.git",
     "directory": "packages/@glimmer/reference"
   },
+  "files": [
+    "dist"
+  ],
   "dependencies": {
     "@glimmer/env": "^0.1.7",
     "@glimmer/global-context": "workspace:^",

diff --git a/packages/@glimmer/runtime/package.json b/packages/@glimmer/runtime/package.json
@@ -6,6 +6,9 @@
   "repository": "https://github.com/glimmerjs/glimmer-vm/tree/master/packages/@glimmer/runtime",
   "license": "MIT",
   "main": "index.ts",
+  "files": [
+    "dist"
+  ],
   "publishConfig": {
     "access": "public",
     "main": "dist/index.js",

diff --git a/packages/@glimmer/syntax/package.json b/packages/@glimmer/syntax/package.json
@@ -15,6 +15,9 @@
     "@types/qunit": "^2.19.7",
     "@glimmer-workspace/build-support": "workspace:^"
   },
+  "files": [
+    "dist"
+  ],
   "scripts": {
     "test:lint": "eslint .",
     "test:types": "tsc --noEmit -p ../tsconfig.json",

diff --git a/packages/@glimmer/util/package.json b/packages/@glimmer/util/package.json
@@ -9,6 +9,9 @@
     "@glimmer/env": "0.1.7",
     "@glimmer/interfaces": "workspace:^"
   },
+  "files": [
+    "dist"
+  ],
   "devDependencies": {
     "@glimmer/local-debug-flags": "workspace:^",
     "@types/qunit": "^2.19.7",

diff --git a/packages/@glimmer/validator/package.json b/packages/@glimmer/validator/package.json
@@ -9,6 +9,9 @@
     "url": "https://github.com/glimmerjs/glimmer-vm",
     "directory": "packages/@glimmer/validator"
   },
+  "files": [
+    "dist"
+  ],
   "dependencies": {
     "@glimmer/env": "^0.1.7",
     "@glimmer/global-context": "workspace:^",

diff --git a/packages/@glimmer/vm-babel-plugins/package.json b/packages/@glimmer/vm-babel-plugins/package.json
@@ -10,6 +10,9 @@
   "engines": {
     "node": ">=16"
   },
+  "files": [
+    "dist"
+  ],
   "scripts": {
     "test": "mocha",
     "test:lint": "eslint .",

diff --git a/packages/@glimmer/vm/package.json b/packages/@glimmer/vm/package.json
@@ -8,6 +8,9 @@
     "@glimmer/interfaces": "workspace:^"
   },
   "types": "index.ts",
+  "files": [
+    "dist"
+  ],
   "scripts": {
     "test:lint": "eslint .",
     "test:types": "tsc --noEmit -p ../tsconfig.json",

diff --git a/packages/@glimmer/wire-format/package.json b/packages/@glimmer/wire-format/package.json
@@ -9,6 +9,9 @@
     "@glimmer/util": "workspace:^",
     "@glimmer/interfaces": "workspace:^"
   },
+  "files": [
+    "dist"
+  ],
   "scripts": {
     "test:lint": "eslint .",
     "test:types": "tsc --noEmit -p ../tsconfig.json",