Resolve 26 spec items from 2026-05-17 multi-agent review

.github/workflows/ci.yml

@@ -12,15 +12,11 @@ jobs:
 
       - uses: actions/setup-node@v4
         with:
-          node-version: '22'
+          node-version: '22.x'
+          cache: 'npm'
 
       - name: Install dependencies
-        run: |
-          if [ -f package-lock.json ]; then
-            npm ci
-          else
-            npm install
-          fi
+        run: npm ci
 
       - name: Typecheck
         run: npm run typecheck

docs/superpowers/specs/2026-04-25-file-organizer-design.md

@@ -134,7 +134,7 @@ The catalog is the single source of truth. One SQLite file, on a user-chosen dri
 | last_verified_at | DATETIME | |
 | scan_id | TEXT FK | |
 
-**Indexes:** `(sha256)`, unique `(drive_id, path)`, `(category, exif_date)`, `(state)`.
+**Indexes:** `(sha256)`, unique `(drive_id, path)`, `(category, exif_date)`, `(scan_id)`, `(state)` (partial: `WHERE state != 'indexed'` — only non-indexed states are stored, keeping write cost low).
 
 **`rules`** — user-defined organizing rules. Schema in §6.1.
 

eslint.config.js

@@ -1,26 +1,37 @@
-import tsParser from '@typescript-eslint/parser';
-import tsPlugin from '@typescript-eslint/eslint-plugin';
+import tseslint from 'typescript-eslint';
 
-export default [
+export default tseslint.config(
   {
     ignores: ['**/dist/**', '**/node_modules/**'],
   },
+  // Base recommended (non-type-checked) rules
+  ...tseslint.configs.recommended,
   {
     files: ['**/*.ts', '**/*.tsx'],
     languageOptions: {
-      parser: tsParser,
       parserOptions: {
-        sourceType: 'module',
-        ecmaVersion: 2022,
+        projectService: true,
+        tsconfigRootDir: import.meta.dirname,
       },
     },
-    plugins: {
-      '@typescript-eslint': tsPlugin,
-    },
     rules: {
       '@typescript-eslint/no-unused-vars': ['error', { argsIgnorePattern: '^_' }],
       '@typescript-eslint/no-explicit-any': 'warn',
       'no-console': 'off',
+      // Type-checked rules: high-value subset that catches real bugs.
+      // Full recommendedTypeChecked was not enabled because no-unsafe-* produces
+      // excessive noise on legitimate use of untyped external libraries (exifr,
+      // better-sqlite3 raw rows, MediaInfo subprocess output). These two rules
+      // catch the load-bearing correctness issues without the noise.
+      '@typescript-eslint/no-floating-promises': 'error',
+      '@typescript-eslint/no-misused-promises': [
+        'error',
+        {
+          // JSX event-handler attributes (onClick, onChange, etc.) accept async
+          // functions in Preact — false positives otherwise.
+          checksVoidReturn: { attributes: false },
+        },
+      ],
     },
   },
   // Engine HTTP path: ban synchronous directory reads. Recursing
@@ -54,4 +65,4 @@ export default [
       ],
     },
   },
-];
+);