PR 9/N (Stage 2): move async queue to common + trivial cleanups

[elisiondan]

Summary

First Stage 2 PR. Tasks #25 and #30 from the backlog. Net code: -99 lines.

Move async queue (task #25)

The hook + common were importing useEnhancedAsyncQueue from extensions/module/src/composables/use-async-queue.ts — a Vue-reactive composable. Rollup inlined it at build time, so the hook bundle was shipping Vue's ref/computed/watchEffect runtime (~25 KB minified) for what was essentially Promise.all-with-delays.

Replaced with extensions/common/utilities/async-queue.ts — plain Promise orchestration, same { add, execute } surface. Renamed the factory to createAsyncQueue to signal it's not a Vue composable anymore.

Bundle size impact:

Bundle	Before	After	Delta
extensions/sync-hook/dist/index.js	610 KB	582 KB	−28 KB / −4.6%
extensions/module/dist/index.js	386 KB	385 KB	−1.2 KB

Removed @vueuse/core from extensions/module/package.json — the old composable was its only consumer in module source.

Trivial cleanups (task #30)

• Delete unused private method ExportToLocalazyService.loadProject (and the now-orphaned LocalazyApiThrottleService import). It was declared but never called.
• Delete unused appMode computed in module/components/ConfigNotice.vue — isDemo is the only thing consuming getConfig().APP_MODE.
• Rename catch (e) / catch (e: any) to catch (_e) / catch (_e: unknown) in 6 sites where the error was discarded. Matches the eslint ^_ ignore convention.
• Delete 5 unused exported types (knip-flagged): ContentTransferSetup, CollectionsTranslatableContent, TranslationStringsTranslatableContent, TranslationStringKeyEntry, CollectionsKeyEntry.

ESLint config tightening

Add caughtErrorsIgnorePattern: '^_' to the no-unused-vars rule so catch parameters follow the same ^_ convention as args/vars. Without it, typescript-eslint@8 defaults flag _e as unused even with argsIgnorePattern set.

Verified

• npm run check — 49/49 tests, lint clean of unused-vars warnings (was 6, now 0), 99 no-explicit-any baseline remains.
• npm run build — production minified, bundle sizes as above.
• npm run knip — zero findings (was 3 unused types before this PR; all cleaned).

Open issue surfaced

The husky pre-commit hook breaks on Node < 22.18 because ESLint 10's stylish formatter uses util.styleText.validateStream (Node 22.18+). If a contributor's globally-installed Node is older than 22 and they haven't run nvm use for the shell, the hook fails with TypeError: util.styleText is not a function. The repo's .nvmrc says 22 but git hooks don't honor it automatically.

Added Stage 2 task #36 to make the hook source nvm (or fall back gracefully). For now contributors should ensure nvm use is in effect when committing.

Stage 2 backlog status

#	Description	Status
25	Move async queue to common	✅ this PR
30	Trivial cleanups	✅ this PR
26	boolean === number fixes	pending
27	DirectusApiService missing methods	pending
28	fetchDirectusItems generic variance	pending
29	Pinia store typing	pending
31	@localazy/generic-connector-client bump	pending
32	@localazy/languages bump	pending
33	TypeScript 5.9 → 6	pending
34	Enable typecheck CI gate	pending
35	no-explicit-any cleanup (99 baseline)	pending
36	Husky hook Node-version-aware	pending

🤖 Generated with Claude Code