fix: resolve verification issues from Round 1 remediation

[atlanticplatformgroup]

Summary

Fixes three issues discovered in the Round 1 verification report:

P1-ERR-08: Astro adapter err → error rename (blocker)

• File: packages/adapters/astro/src/index.ts:105
• Problem: catch (error) block referenced ${err} instead of ${error}, causing a TypeScript compilation error
• Fix: Changed ${err} → ${error}

P2-LINT-01: CLI test imports need .js extensions

• Files: 6 test files in packages/cli/src/commands/__tests__/
• Problem: Relative imports missing .js extension for ESM (TS2835)
• Fix: Added .js to all relative imports (e.g. ../auth → ../auth.js)

P2-LINT-02: Web lint — conditional hook call

• File: packages/web/src/contexts/DarkModeContext.tsx:24
• Problem: useUserPreferences was called inside a try block at the top level of the component body, which ESLint flagged as a conditional hook call
• Fix: Extracted the try/catch into a dedicated useUserPreferencesSafe custom hook so the hook call is unconditional at the component level

Verification

Check	Result
npm run build -w @ori/shared	✅ PASS
npm run type-check -w @ori/api	✅ PASS
npm run build:check -w @ori/web	✅ PASS
npm run lint --workspaces	✅ PASS (0 errors, only pre-existing warnings)
npm run test -w @ori/api	🟡 52 pre-existing failures (rate-limit/auth middleware tests need DB)
npm run test -w @ori/web	🟡 378 pre-existing failures (React act() in production build — known issue)