Build lib before linting

Author: Oblosys

.github/workflows/build-test.yml

@@ -18,8 +18,9 @@ jobs:
           cache: 'yarn'
       - run: yarn
       - run: yarn prettier-check
-      - run: yarn lint-check
+      # Need to build before linting because of linting rules that use type information.
       - run: yarn build-lib
+      - run: yarn lint-check
       - run: yarn type-check-app
       - run: yarn test-lib
       - run: yarn test-app

eslint.config.mjs

@@ -48,6 +48,8 @@ export default typescriptEslint.config([
   //
   // TypeScript
   //
+  // NOTE: 'recommendedTypeChecked' requires a build step before linting. Unfortunately there is no clear error if
+  // packages/react-hook-tracer/dist is missing, just lots of @typescript-eslint/no-unsafe-.. errors.
   typescriptEslint.configs.recommendedTypeChecked,
   {
     name: 'typescript-rules',

package.json

@@ -31,7 +31,7 @@
     "login": "yarn workspace react-hook-tracer npm login --publish",
     "pack-lib": "yarn pre-pack-publish && yarn workspace react-hook-tracer pack -o 'dist/%s@%v.tgz'",
     "pack-lib-no-test": "yarn clean && yarn build-lib && yarn workspace react-hook-tracer pack -o 'dist/%s@%v.tgz'",
-    "pre-pack-publish": "yarn prettier-check && yarn lint-check && yarn clean && yarn build",
+    "pre-pack-publish": "yarn prettier-check && yarn clean && yarn build && yarn lint-check",
     "publish-lib": "yarn pre-pack-publish && yarn workspace react-hook-tracer npm publish",
     "tag-release": "bash-scripts/tag-release.sh",
     "test-lib": "yarn workspace react-hook-tracer test",