@pushwoosh/frontend-builder-engine 3.1.0 → 3.1.2

package/README.md

@@ -12,8 +12,33 @@ The CLI is invoked via `pushwoosh-engine` (installed as a dependency in consumin
 pushwoosh-engine app:command:dev      # Start webpack dev server (HTTPS)
 pushwoosh-engine app:command:build    # Production build
 pushwoosh-engine app:command:analyze  # Bundle analysis
+pushwoosh-engine app:command:check-dead-code  # Unused files, exports and dependencies (knip)
 ```
 
+`app:command:check-dead-code` runs knip with the shipped `lib/knip.application.json`: entries are
+`src/index.{ts,tsx}` plus every `src/parcels/**/index.{ts,tsx}`, so a file the entry graph never
+reaches is reported as unused. Exports used only inside their own file and everything under a
+`generated/` directory are not reported; the binaries and unlisted rules are off, since consuming
+projects get eslint/tsc/webpack from this package rather than declaring them; config hints are
+suppressed, since the config lives here and a consuming project cannot act on them. A project opts
+in by adding the script and folding it into `check`:
+
+```json
+"check": "npm run check:lint && npm run check:types && npm run check:dead-code",
+"check:dead-code": "echo \"Run check:dead-code\" && `pushwoosh-engine app:command:check-dead-code`"
+```
+
+`examples/application` ships with exactly that wiring, so a project copied from the template gets the
+gate on its `pre-commit` hook from the first commit.
+
+CI is a separate step: the shared `frontend/microfrontend-release.yml` pipeline runs `check:lint` and
+`check:types` as their own jobs and never calls `npm run check`, so the script alone gates the
+pre-commit hook only. Blocking a merge needs a `check:dead-code` job next to those two.
+
+A project whose parcel entry sits outside the globs above (a file no other entry re-exports) needs
+its own `knip.json` in the project root and a plain `knip` call — the shared config is passed with
+`--config` and cannot be extended per project.
+
 ### Library commands (reusable packages)
 
 ```bash

package/lib/browser-targets.js

@@ -27,6 +27,15 @@
  *  - Below Chrome/Edge 91 it enables
  *    plugin-bugfix-v8-spread-parameters-in-optional-chaining, which downlevels
  *    every `?.` in the bundle.
+ *  - Below Safari/iOS 16 it enables
+ *    plugin-bugfix-safari-class-field-initializer-scope, wrapping initializers
+ *    that resolve to a bare identifier reference (through calls, member access,
+ *    arrays, objects, or templates) in an IIFE; literals, `this`, and
+ *    function/arrow/class expressions are left alone. It guards WebKit 236843,
+ *    which fires only on redundant parentheses inside the initializer; Babel
+ *    wraps blindly because its default AST drops them. To check an artifact for
+ *    real exposure, parse it with `createParenthesizedExpressions: true` and
+ *    look for ParenthesizedExpression under ClassProperty/ClassPrivateProperty.
  */
 const BROWSER_TARGETS = [
   'Chrome >= 94',

package/lib/knip.application.json

@@ -0,0 +1,19 @@
+{
+  "$schema": "https://unpkg.com/knip@6/schema.json",
+  "entry": [
+    "src/index.{ts,tsx}",
+    "src/parcels/**/index.{ts,tsx}"
+  ],
+  "project": [
+    "src/**/*.{ts,tsx}"
+  ],
+  "ignore": [
+    "**/generated/**"
+  ],
+  "ignoreExportsUsedInFile": true,
+  "includeEntryExports": false,
+  "rules": {
+    "binaries": "off",
+    "unlisted": "off"
+  }
+}

package/lib/pushwoosh-engine.js

@@ -112,6 +112,21 @@ program
       }),
   );
 
+program
+  .addCommand(
+    new Command('app:command:check-dead-code')
+      .description('App dead code check command')
+      .action(async () => {
+        const cmd = [
+          './node_modules/.bin/knip',
+          '--config',
+          './node_modules/@pushwoosh/frontend-builder-engine/lib/knip.application.json',
+          '--no-config-hints',
+        ];
+        console.log(cmd.join(' '));
+      }),
+  );
+
 program
   .addCommand(
     new Command('app:command:analyze')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushwoosh/frontend-builder-engine",
-  "version": "3.1.0",
+  "version": "3.1.2",
   "description": "Pushwoosh frontend builder engine",
   "main": "./lib/pushwoosh-frontend-builder-engine.js",
   "publishConfig": {
@@ -44,6 +44,7 @@
     "eslint-plugin-react-hooks": "^7.0.1",
     "fork-ts-checker-webpack-plugin": "^9.0.2",
     "html-webpack-plugin": "^5.6.0",
+    "knip": "^6.35.1",
     "style-loader": "^4.0.0",
     "systemjs-webpack-interop": "^2.3.7",
     "ts-loader": "^9.5.1",