fix: address pre-merge review items for pnpm migration

Fix test:debug vitest flag typo, set HUSKY=0 in cache/proxy Docker
installs, and skip build-storybook in CI when a package has no script.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: mrTuomoK

.github/workflows/ci-packages.yml

@@ -87,7 +87,7 @@ jobs:
         run: pnpm --filter '[origin/main]' --filter '!{apps/**}' --filter '!{proxies/**}' --filter '!events-helsinki-monorepo' run build
 
       - name: Run storybook build for changed packages
-        run: pnpm --filter '[origin/main]' --filter '!{apps/**}' --filter '!{proxies/**}' --filter '!events-helsinki-monorepo' run build-storybook
+        run: pnpm --filter '[origin/main]' --filter '!{apps/**}' --filter '!{proxies/**}' --filter '!events-helsinki-monorepo' --if-present run build-storybook
 
       - name: Run tests
         working-directory: packages/components

DockerfileCache

@@ -129,6 +129,7 @@ COPY --from=deps --chown=default:root /workspace-install ./
 
 USER default
 
+ENV HUSKY=0
 RUN pnpm install --frozen-lockfile
 
 FROM ${BUILDER_FROM_IMAGE} AS cache

apps/events-helsinki/package.json

@@ -31,7 +31,7 @@
     "test": "cross-env TZ=Europe/Helsinki vitest run",
     "test:changed": "pnpm run test --changed",
     "test:coverage": "pnpm run test --coverage",
-    "test:debug": "pnpm run test --inspect-brk ----no-file-parallelism",
+    "test:debug": "pnpm run test --inspect-brk --no-file-parallelism",
     "test:staged": "cross-env TZ=Europe/Helsinki vitest related --run",
     "test:watch": "cross-env TZ=Europe/Helsinki vitest watch",
     "test:browser": "testcafe \"chrome --disable-features=LocalNetworkAccessChecks --window-size='1249,720'\" browser-tests/ --live --dev",

apps/hobbies-helsinki/package.json

@@ -31,7 +31,7 @@
     "test": "cross-env TZ=Europe/Helsinki vitest run",
     "test:changed": "pnpm run test --changed",
     "test:coverage": "pnpm run test --coverage",
-    "test:debug": "pnpm run test --inspect-brk ----no-file-parallelism",
+    "test:debug": "pnpm run test --inspect-brk --no-file-parallelism",
     "test:staged": "cross-env TZ=Europe/Helsinki vitest related --run",
     "test:watch": "cross-env TZ=Europe/Helsinki vitest watch",
     "test:browser": "testcafe \"chrome --disable-features=LocalNetworkAccessChecks --window-size='1249,720'\" browser-tests/ --live --dev",

apps/sports-helsinki/package.json

@@ -32,7 +32,7 @@
     "test": "cross-env TZ=Europe/Helsinki vitest run",
     "test:changed": "pnpm run test --changed",
     "test:coverage": "pnpm run test --coverage",
-    "test:debug": "pnpm run test --inspect-brk ----no-file-parallelism",
+    "test:debug": "pnpm run test --inspect-brk --no-file-parallelism",
     "test:staged": "cross-env TZ=Europe/Helsinki vitest related --run",
     "test:watch": "cross-env TZ=Europe/Helsinki vitest watch",
     "test:browser": "testcafe \"chrome --disable-features=LocalNetworkAccessChecks --window-size='1249,720'\" browser-tests/ --live --dev",

packages/components/package.json

@@ -33,7 +33,7 @@
     "test": "cross-env TZ=Europe/Helsinki vitest run",
     "test:changed": "pnpm run test --changed",
     "test:coverage": "pnpm run test --coverage",
-    "test:debug": "pnpm run test --inspect-brk ----no-file-parallelism",
+    "test:debug": "pnpm run test --inspect-brk --no-file-parallelism",
     "test:staged": "cross-env TZ=Europe/Helsinki vitest related --run",
     "test:watch": "cross-env TZ=Europe/Helsinki vitest watch",
     "test-watch": "vitest watch --ui",

packages/graphql-proxy-server/package.json

@@ -19,7 +19,7 @@
     "test": "cross-env TZ=Europe/Helsinki vitest run",
     "test:changed": "pnpm run test --changed",
     "test:coverage": "pnpm run test --coverage",
-    "test:debug": "pnpm run test --inspect-brk ----no-file-parallelism",
+    "test:debug": "pnpm run test --inspect-brk --no-file-parallelism",
     "test:staged": "cross-env TZ=Europe/Helsinki vitest related --run",
     "test:watch": "cross-env TZ=Europe/Helsinki vitest watch",
     "typecheck": "tsc --project ./tsconfig.json --noEmit"

proxies/events-graphql-proxy/package.json

@@ -22,7 +22,7 @@
     "test": "cross-env TZ=Europe/Helsinki vitest run",
     "test:changed": "pnpm run test --changed",
     "test:coverage": "pnpm run test --coverage",
-    "test:debug": "pnpm run test --inspect-brk ----no-file-parallelism",
+    "test:debug": "pnpm run test --inspect-brk --no-file-parallelism",
     "test:staged": "cross-env TZ=Europe/Helsinki vitest related --run",
     "test:watch": "cross-env TZ=Europe/Helsinki vitest watch",
     "typecheck": "tsc --project ./tsconfig.json --noEmit",

proxies/venue-graphql-proxy/package.json

@@ -22,7 +22,7 @@
     "test": "cross-env TZ=Europe/Helsinki vitest run",
     "test:changed": "pnpm run test --changed",
     "test:coverage": "pnpm run test --coverage",
-    "test:debug": "pnpm run test --inspect-brk ----no-file-parallelism",
+    "test:debug": "pnpm run test --inspect-brk --no-file-parallelism",
     "test:staged": "cross-env TZ=Europe/Helsinki vitest related --run",
     "test:watch": "cross-env TZ=Europe/Helsinki vitest watch",
     "typecheck": "tsc --project ./tsconfig.json --noEmit",

proxy.Dockerfile

@@ -117,6 +117,7 @@ COPY --from=deps --chown=default:root /workspace-install ./
 
 # Optional: if the app depends on global /static shared assets like images, locales...
 
+ENV HUSKY=0
 RUN pnpm install --frozen-lockfile
 
 # build