improve production build (#158)

* improve verify * fix preact * fix collections * dont check debug & verify packages * wip * fix reactive * fix lint * improve build * fix type & lint * more lint fixes * fix lint * update @starbeam-dev/eslint-plugin
starbeamjs · May 8, 2024 · 7099168 · 7099168
1 parent e2a724b
commit 7099168
Show file tree

Hide file tree

Showing 94 changed files with 621 additions and 507 deletions.
diff --git a/@types/ansicolor/package.json b/@types/ansicolor/package.json
@@ -22,7 +22,7 @@
     "test:types": "tsc -b"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "@types/node": "18.16.18"
   }
 }
diff --git a/@types/fsify/index.d.ts b/@types/fsify/index.d.ts
@@ -16,8 +16,8 @@ export interface Options {
    */
   cwd?: string;
   /**
-   * A persistent fsify tree remains on the file system after the process exits.
-   * Defaults to true.
+   * A persistent fsify tree remains on the file system after the process
+   * exits. Defaults to true.
    */
   persistent?: boolean;
   /**

diff --git a/@types/fsify/package.json b/@types/fsify/package.json
@@ -21,7 +21,7 @@
     "test:types": "tsc -b"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "@types/node": "18.16.1"
   }
 }
diff --git a/@types/lines-and-columns/package.json b/@types/lines-and-columns/package.json
@@ -20,7 +20,7 @@
     "test:types": "tsc -b"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "@types/node": "18.16.1"
   }
 }
diff --git a/@types/plugin-commonjs/package.json b/@types/plugin-commonjs/package.json
@@ -24,6 +24,6 @@
     "@rollup/plugin-commonjs": "25.0.5"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.3"
+    "@starbeam-dev/eslint-plugin": "^1.0.6"
   }
 }
diff --git a/@types/plugin-node-resolve/package.json b/@types/plugin-node-resolve/package.json
@@ -24,6 +24,6 @@
     "@rollup/plugin-node-resolve": "^15.2.3"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.3"
+    "@starbeam-dev/eslint-plugin": "^1.0.6"
   }
 }
diff --git a/@types/rollup-plugin-polyfill-node/package.json b/@types/rollup-plugin-polyfill-node/package.json
@@ -24,6 +24,6 @@
     "rollup": "^4.0.2"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4"
+    "@starbeam-dev/eslint-plugin": "^1.0.6"
   }
 }
diff --git a/@types/shell-escape-tag/package.json b/@types/shell-escape-tag/package.json
@@ -20,7 +20,7 @@
     "test:types": "tsc -b"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "@types/node": "18.16.1"
   }
 }
diff --git a/@types/shell-split/package.json b/@types/shell-split/package.json
@@ -21,7 +21,7 @@
     "test:types": "tsc -b"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "@types/node": "18.16.1"
   }
 }
diff --git a/@types/vite-env/package.json b/@types/vite-env/package.json
@@ -24,6 +24,6 @@
     "vitest": "^1.0.0-beta.2"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4"
+    "@starbeam-dev/eslint-plugin": "^1.0.6"
   }
 }
diff --git a/package.json b/package.json
@@ -110,13 +110,13 @@
     "@nrr/pnpm-duplicate-cli": "^0.0.1",
     "@starbeam-dev/compile": "workspace:*",
     "@starbeam-dev/core": "^1.0.2",
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "@types/eslint": "^8.56.0",
     "@types/node": "^20.10.5",
     "@typescript-eslint/eslint-plugin": "^6.15.0",
     "@typescript-eslint/parser": "^6.15.0",
     "@vitest/ui": "^1.1.0",
-    "eslint": "^8.56.0",
+    "eslint": "^8.57.0",
     "eslint-config-airbnb": "^19.0.4",
     "eslint-config-prettier": "^9.1.0",
     "eslint-import-resolver-typescript": "^3.6.1",
@@ -162,7 +162,8 @@
       "preact": "10.17.1",
       "typescript": "$typescript",
       "vite": "$vite",
-      "vue": "3.3.4"
+      "vue": "3.3.4",
+      "terser": "^5.31.0"
     },
     "peerDependencyRules": {
       "allowAny": [

diff --git a/packages/preact/preact-testing-utils/package.json b/packages/preact/preact-testing-utils/package.json
@@ -29,7 +29,7 @@
   },
   "devDependencies": {
     "@starbeam-dev/compile": "workspace:*",
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "preact-render-to-string": "^6.2.1",
     "rollup": "^4.0.2"
   },

diff --git a/packages/preact/preact-testing-utils/tests/package.json b/packages/preact/preact-testing-utils/tests/package.json
@@ -25,7 +25,7 @@
     "preact": "*"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "preact-render-to-string": "^6.2.1"
   }
 }
diff --git a/packages/preact/preact-utils/package.json b/packages/preact/preact-utils/package.json
@@ -31,7 +31,7 @@
   },
   "devDependencies": {
     "@starbeam-dev/compile": "workspace:*",
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "rollup": "^4.0.2"
   },
   "peerDependencies": {

diff --git a/packages/preact/preact-utils/tests/package.json b/packages/preact/preact-utils/tests/package.json
@@ -24,7 +24,7 @@
     "preact": "*"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "preact-render-to-string": "^6.2.1"
   }
 }
diff --git a/packages/preact/preact/package.json b/packages/preact/preact/package.json
@@ -48,7 +48,7 @@
   },
   "devDependencies": {
     "@starbeam-dev/compile": "workspace:*",
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "rollup": "^4.0.2"
   },
   "peerDependencies": {

diff --git a/packages/preact/preact/src/frame.ts b/packages/preact/preact/src/frame.ts
@@ -52,10 +52,10 @@ export class ComponentFrame {
   ): FinalizedFormula {
     const frame = ComponentFrame.#frames.get(component);
 
-    verify(
+    verify?.(
       frame,
       isPresent,
-      expected.when("in Preact's _diff hook").as("a tracking frame"),
+      expected?.when("in Preact's _diff hook").as("a tracking frame"),
     );
 
     const end = frame.#end(subscription);
@@ -101,10 +101,10 @@ export class ComponentFrame {
   }
 
   #end(subscription: (() => void) | undefined) {
-    verify(
+    verify?.(
       this.#active,
       isPresent,
-      expected.when("in preact's _diff hook").as("an active tracking frame"),
+      expected?.when("in preact's _diff hook").as("an active tracking frame"),
     );
 
     const frame = (this.#frame = this.#active.done());

diff --git a/packages/preact/preact/tests/package.json b/packages/preact/preact/tests/package.json
@@ -26,7 +26,7 @@
     "preact": "*"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "preact-render-to-string": "^6.2.1"
   }
 }
diff --git a/packages/react/react/package.json b/packages/react/react/package.json
@@ -46,7 +46,7 @@
   },
   "devDependencies": {
     "@starbeam-dev/compile": "workspace:*",
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "@types/node": "18.16.1",
     "@types/react": "^18.2.22",
     "@types/scheduler": "^0.16.3",

diff --git a/packages/react/react/src/hooks/setup.ts b/packages/react/react/src/hooks/setup.ts
@@ -152,8 +152,8 @@ interface ScheduledHandler {
  * Creates a {@linkcode ScheduledHandler} that will keep track of the
  * synchronization functions to run.
  *
- * This function sets up a `useEffect` to run the handlers. This `useEffect` has
- * a dependency on a `useState` that represents the set of handlers. The
+ * This function sets up a `useEffect` to run the handlers. This `useEffect`
+ * has a dependency on a `useState` that represents the set of handlers. The
  * `useState` invalidates whenever a handler is added or whenever
  * {@linkcode scheduleDep} is explicitly run.
  *

diff --git a/packages/react/react/tests/package.json b/packages/react/react/tests/package.json
@@ -24,6 +24,6 @@
     "vitest": "^0.34.4"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4"
+    "@starbeam-dev/eslint-plugin": "^1.0.6"
   }
 }
diff --git a/packages/react/react/tests/test-use.ts b/packages/react/react/tests/test-use.ts
@@ -73,8 +73,8 @@ type DefineSetupReactiveComponent = (
 ) => Promise<void>;
 
 /**
- * Sets up a test suite for a reactive component using `setupReactive` to create
- * subscribe to a reactive value.
+ * Sets up a test suite for a reactive component using `setupReactive` to
+ * create subscribe to a reactive value.
  *
  * Each test case has three variations, representing the three overloads of
  * `IntoReactiveBlueprint`.
@@ -105,8 +105,8 @@ export function testSetupReactive(
 }
 
 /**
- * Sets up a test suite for a reactive component using `setupReactive` to create
- * subscribe to a reactive value.
+ * Sets up a test suite for a reactive component using `setupReactive` to
+ * create subscribe to a reactive value.
  *
  * Each test case has three variations, representing the three overloads of
  * `IntoReactiveBlueprint`.

diff --git a/packages/react/test-utils/package.json b/packages/react/test-utils/package.json
@@ -33,7 +33,7 @@
   },
   "devDependencies": {
     "@starbeam-dev/compile": "workspace:*",
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "@testing-library/react": "^14.0.0",
     "@types/react": "^18.2.22",
     "@types/scheduler": "^0.16.3",

diff --git a/packages/react/use-strict-lifecycle/package.json b/packages/react/use-strict-lifecycle/package.json
@@ -33,7 +33,7 @@
   },
   "devDependencies": {
     "@starbeam-dev/compile": "workspace:*",
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "@types/react": "^18.2.22",
     "@types/scheduler": "^0.16.3",
     "rollup": "^4.0.2"

diff --git a/packages/react/use-strict-lifecycle/src/lifecycle.ts b/packages/react/use-strict-lifecycle/src/lifecycle.ts
@@ -64,10 +64,10 @@
  *
  * ## The Solution
  *
- * The `useLifecycle` hook gives you a way to create a new instance of something
- * when the component is first instantiated, clean it up when the component is
- * unmounted, and create a brand **new** instance when the component is
- * reactivated.
+ * The `useLifecycle` hook gives you a way to create a new instance of
+ * something when the component is first instantiated, clean it up when the
+ * component is unmounted, and create a brand **new** instance when the
+ * component is reactivated.
  *
  * TL;DR It works almost the same way that per-component state in React works,
  * but gives you a fresh copy whenever React re-attaches the component.

diff --git a/packages/react/use-strict-lifecycle/tests/package.json b/packages/react/use-strict-lifecycle/tests/package.json
@@ -18,7 +18,7 @@
     "react": "18.2.0"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "@types/react": "^18.2.22",
     "jsdom": "^22.1.0"
   }

diff --git a/packages/universal/collections/package.json b/packages/universal/collections/package.json
@@ -43,7 +43,7 @@
   },
   "devDependencies": {
     "@starbeam-dev/compile": "workspace:*",
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "rollup": "^4.0.2"
   },
   "files": [

diff --git a/packages/universal/collections/src/decorator.ts b/packages/universal/collections/src/decorator.ts
@@ -10,10 +10,10 @@ export const cached = <T>(
 ): void => {
   const get = descriptor.get;
 
-  verify(
+  verify?.(
     get,
     isPresent,
-    expected(`the target of @cached`)
+    expected?.(`the target of @cached`)
       .toBe(`a getter`)
       .butGot(() =>
         typeof descriptor.value === "function" ? `a method` : `a field`,

diff --git a/packages/universal/collections/tests/package.json b/packages/universal/collections/tests/package.json
@@ -17,6 +17,6 @@
     "@starbeam-workspace/test-utils": "workspace:^"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4"
+    "@starbeam-dev/eslint-plugin": "^1.0.6"
   }
 }
diff --git a/packages/universal/core-utils/package.json b/packages/universal/core-utils/package.json
@@ -29,7 +29,7 @@
   "dependencies": {},
   "devDependencies": {
     "@starbeam-dev/compile": "workspace:*",
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "rollup": "^4.0.2"
   }
 }
diff --git a/packages/universal/core-utils/tests/package.json b/packages/universal/core-utils/tests/package.json
@@ -15,6 +15,6 @@
     "@starbeam/core-utils": "workspace:^"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4"
+    "@starbeam-dev/eslint-plugin": "^1.0.6"
   }
 }
diff --git a/packages/universal/core/package.json b/packages/universal/core/package.json
@@ -30,7 +30,7 @@
   },
   "devDependencies": {
     "@starbeam-dev/compile": "workspace:*",
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "rollup": "^4.0.2"
   },
   "files": [

diff --git a/packages/universal/debug/package.json b/packages/universal/debug/package.json
@@ -48,7 +48,7 @@
   },
   "devDependencies": {
     "@starbeam-dev/compile": "workspace:*",
-    "@starbeam-dev/eslint-plugin": "^1.0.4",
+    "@starbeam-dev/eslint-plugin": "^1.0.6",
     "@types/node": "18.16.1",
     "rollup": "^4.0.2"
   },

diff --git a/packages/universal/debug/src/call-stack/debug/module.ts b/packages/universal/debug/src/call-stack/debug/module.ts
@@ -37,8 +37,8 @@ function stripLeadingSlash(path: string) {
 
 /**
  * The whole `Stack` system is only intended to be used for logging, so the
- * edge-cases where this normalization wouldn't work (verbatim paths on Windows)
- * shouldn't matter.
+ * edge-cases where this normalization wouldn't work (verbatim paths on
+ * Windows) shouldn't matter.
  */
 function normalizePath(...pathParts: (string | null | undefined)[]): string {
   return pathParts
@@ -48,8 +48,8 @@ function normalizePath(...pathParts: (string | null | undefined)[]): string {
 }
 
 /**
- * This function takes two paths and returns the suffix of the target that comes
- * after any shared prefix with the source.
+ * This function takes two paths and returns the suffix of the target that
+ * comes after any shared prefix with the source.
  *
  * For example, if the source is `/src/app/foo/bar` and the target is
  * `/src/app/baz/qux`, this function will return `baz/qux`.

diff --git a/packages/universal/debug/src/call-stack/debug/stack.ts b/packages/universal/debug/src/call-stack/debug/stack.ts
@@ -33,7 +33,7 @@ export function callerStack(
       "An error created in the internals of Stack.create",
     ).stack;
 
-    verify(stack, hasType("string"));
+    verify?.(stack, hasType("string"));
     return callStack(stack)?.slice(internal + CALLER);
   }
 }

diff --git a/packages/universal/debug/tests/package.json b/packages/universal/debug/tests/package.json
@@ -16,6 +16,6 @@
     "@starbeam-workspace/test-utils": "workspace:^"
   },
   "devDependencies": {
-    "@starbeam-dev/eslint-plugin": "^1.0.4"
+    "@starbeam-dev/eslint-plugin": "^1.0.6"
   }
 }
diff --git a/packages/universal/interfaces/package.json b/packages/universal/interfaces/package.json
@@ -31,6 +31,6 @@
   },
   "devDependencies": {
     "@starbeam-dev/compile": "workspace:*",
-    "@starbeam-dev/eslint-plugin": "^1.0.4"
+    "@starbeam-dev/eslint-plugin": "^1.0.6"
   }
 }