feat: add code quality checks with clang-tidy and memory tests in CI workflow

Author: mceachen

.github/workflows/code-quality.yml

@@ -0,0 +1,59 @@
+name: Code Quality
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+jobs:
+  clang-tidy:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 20
+          cache: "npm"
+      
+      - name: Install dependencies
+        run: |
+          sudo apt-get update
+          sudo apt-get install -y clang-tidy
+      
+      - name: Install Node dependencies
+        run: npm ci
+      
+      - name: Build project
+        run: npm run build
+      
+      - name: Run clang-tidy
+        run: npm run clang-tidy
+        
+  memory-tests-basic:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 20
+          cache: "npm"
+      
+      - name: Install dependencies
+        run: |
+          sudo apt-get update
+          sudo apt-get install -y valgrind
+      
+      - name: Install Node dependencies
+        run: npm ci
+      
+      - name: Build project
+        run: npm run build
+      
+      - name: Run JavaScript memory tests
+        run: npm run test:memory
+      
+      - name: Run valgrind tests
+        run: bash scripts/valgrind.sh

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "@photostructure/sqlite",
   "version": "0.1.0",
-  "description": "Drop-in replacement for node:sqlite - Node.js SQLite implementation extracted from Node.js core",
+  "description": "Drop-in replacement for node:sqlite",
   "homepage": "https://github.com/photostructure/node-sqlite",
   "types": "./dist/index.d.ts",
   "main": "./dist/index.js",
@@ -17,21 +17,27 @@
     "clean": "run-p clean:*",
     "clean:dist": "del-cli dist \"*.tsbuildinfo\"",
     "clean:gyp": "node-gyp clean",
-    "sync": "node scripts/sync-from-node.js",
+    "sync": "node scripts/sync-from-node.mjs",
     "node-gyp-rebuild": "node-gyp rebuild",
     "prebuild": "prebuildify --napi --tag-libc --strip",
     "compile": "tsc --noEmit",
     "bundle": "tsup src/index.ts --format cjs,esm --dts",
     "test": "jest",
     "test:unit": "jest --testPathPattern=src/",
     "test:integration": "jest --testPathPattern=test/",
+    "test:memory": "cross-env TEST_MEMORY=1 node --expose-gc node_modules/jest/bin/jest.js --no-coverage test/memory.test.ts",
+    "tests:memory": "node scripts/check-memory.mjs",
+    "asan": "cross-env ENABLE_ASAN=1 node scripts/check-memory.mjs",
+    "clang-tidy": "node scripts/clang-tidy.mjs",
+    "precommit:quality": "node -e \"if(process.platform==='linux'||process.platform==='darwin')process.exit(1)\" || npm run clang-tidy",
+    "precommit:memory": "node -e \"if(process.platform!=='linux')process.exit(1)\" && npm run test:memory || echo 'Skipping memory tests (Linux only)'",
     "lint": "run-s lint:*",
     "lint:eslint": "eslint src/ test/",
     "lint:api": "tsc --noEmit src/api-compatibility.test.ts",
     "lint:fix": "eslint --fix src/ test/",
     "fmt": "prettier --write \"**/*.{ts,js,json,md}\"",
     "build": "run-s compile bundle",
-    "precommit": "run-s fmt lint build test",
+    "precommit": "node scripts/precommit.mjs",
     "prepare-release": "npm run build",
     "release": "release-it"
   },
@@ -71,7 +77,7 @@
   "devDependencies": {
     "@eslint/js": "^9.27.0",
     "@types/jest": "^29.5.14",
-    "@types/node": "^22.15.24",
+    "@types/node": "^22.15.26",
     "@typescript-eslint/eslint-plugin": "^8.33.0",
     "@typescript-eslint/parser": "^8.33.0",
     "cross-env": "^7.0.3",