Merge pull request #72 from github-samples/dependabot/npm_and_yarn/react-three/fiber-9.5.0

chore(deps): bump @react-three/fiber from 9.4.2 to 9.5.0

Author: chrisreddington

package-lock.json

@@ -18,8 +18,8 @@
     "test": "npm run test --workspace=shared && npm run test --workspace=web && npm run test --workspace=mcp-server",
     "test:watch": "npm run test:watch --workspace=shared",
     "test:coverage": "npm run test:coverage --workspace=shared && npm run test:coverage --workspace=web && npm run test:coverage --workspace=mcp-server",
-    "lint": "eslint .",
-    "lint:fix": "eslint . --fix",
+    "lint": "npm run lint --workspace=shared && npm run lint --workspace=web && npm run lint --workspace=mcp-server",
+    "lint:fix": "npm run lint:fix --workspace=shared && npm run lint:fix --workspace=web && npm run lint:fix --workspace=mcp-server",
     "lint:all": "npm run lint --workspace=shared && npm run lint --workspace=web && npm run lint --workspace=mcp-server",
     "ci:tests": "npm run test --workspace=@turn-based-mcp/shared && npm run test --workspace=@turn-based-mcp/web && npm run test --workspace=@turn-based-mcp/mcp-server",
     "ci": "npm run build --workspace=@turn-based-mcp/shared && npm run type-check && npm run build --workspace=@turn-based-mcp/web && npm run build --workspace=@turn-based-mcp/mcp-server && npm run ci:tests && npm run lint --workspace=@turn-based-mcp/web"

package.json

@@ -34,6 +34,7 @@
   "devDependencies": {
     "@eslint/js": "^9.39.2",
     "@jest/types": "30.2.0",
+    "@types/better-sqlite3": "^7.6.13",
     "@types/jest": "^30.0.0",
     "@vitest/coverage-v8": "^4.0.18",
     "@vitest/ui": "^4.0.18",
@@ -44,6 +45,6 @@
     "vitest": "^4.0.18"
   },
   "dependencies": {
-    "sqlite3": "^5.1.7"
+    "better-sqlite3": "^12.6.2"
   }
 }

shared/package.json

@@ -1,34 +1,24 @@
 import { vi } from 'vitest'
 import * as sqliteStorage from './sqlite-storage';
 
-// Mock sqlite3 to prevent actual database operations
+// Mock better-sqlite3 to prevent actual database operations
 // In vitest v4, mocks used as constructors must use 'function' or 'class' syntax
-vi.mock('sqlite3', () => ({
-  default: {
-    Database: vi.fn(function() {
+vi.mock('better-sqlite3', () => {
+  const mockStatement = {
+    run: vi.fn(() => ({ lastInsertRowid: 1, changes: 1 })),
+    get: vi.fn(() => undefined),
+    all: vi.fn(() => []),
+  }
+  return {
+    default: vi.fn(function() {
       return {
-        run: vi.fn(function(sql: string, params: unknown, callback?: (this: { lastID: number; changes: number }) => void) {
-          if (callback) {
-            // Mock the 'this' context with lastID and changes
-            callback.call({ lastID: 1, changes: 1 });
-          }
-        }),
-        get: vi.fn((sql: string, params: unknown, callback?: (err: Error | null, row: unknown) => void) => {
-          if (callback) callback(null, null);
-        }),
-        all: vi.fn((sql: string, params: unknown, callback?: (err: Error | null, rows: unknown[]) => void) => {
-          if (callback) callback(null, []);
-        }),
-        close: vi.fn((callback?: () => void) => {
-          if (callback) callback();
-        }),
-        serialize: vi.fn((fn?: () => void) => {
-          if (fn) fn();
-        })
-      };
+        prepare: vi.fn(() => mockStatement),
+        exec: vi.fn(),
+        close: vi.fn(),
+      }
     })
   }
-}));
+});
 
 // Integration-style tests for SQLite storage
 // These tests verify the public API without heavy mocking