Chore/cleanup api

.github/workflows/npm-publish.yml → .github/workflows/ci.yml

@@ -6,6 +6,11 @@ on:
   push:
     branches:
       - master
+      - develop
+  pull_request:
+    branches:
+      - master
+      - develop
 
 permissions:
   contents: read
@@ -30,6 +35,9 @@ jobs:
       - name: Install dependencies
         run: pnpm install --frozen-lockfile
 
+      - name: Lint
+        run: pnpm run lint:check
+
       # Generate coverage (ensure coverage/lcov.info exists after this)
       - name: Test (with coverage)
         run: pnpm run test:cov

.qlty/qlty.toml

@@ -2,7 +2,5 @@
 exclude_patterns = [
     "**/database.types.ts",
     "**/swagger-ui-bundle.js",
-    "**/swagger-ui-standalone-preset.js",
-    "**/swagger-ui-standalone-preset.js",
-    "**/database.types.ts"
-]
+    "**/swagger-ui-standalone-preset.js"
+]

eslint.config.mjs

@@ -26,10 +26,21 @@ export default tseslint.config(
   },
   {
     rules: {
-      '@typescript-eslint/no-explicit-any': 'off',
-      '@typescript-eslint/no-floating-promises': 'warn',
+      '@typescript-eslint/no-explicit-any': 'error',
+      '@typescript-eslint/no-floating-promises': 'error',
       '@typescript-eslint/no-unsafe-argument': 'warn',
-      "prettier/prettier": ["error", { endOfLine: "auto" }],
+      '@typescript-eslint/no-unused-vars': [
+        'error',
+        { argsIgnorePattern: '^_', varsIgnorePattern: '^_' },
+      ],
+      'prettier/prettier': ['error', { endOfLine: 'auto' }],
+    },
+  },
+  {
+    // Test files may use `any` for mocks and private-member access.
+    files: ['**/*.spec.ts', 'src/v1/common/test-helpers.ts'],
+    rules: {
+      '@typescript-eslint/no-explicit-any': 'off',
     },
   },
 );

package.json

@@ -13,6 +13,7 @@
     "start:debug": "nest start --debug --watch",
     "start:prod": "node dist/main",
     "lint": "eslint \"{src,apps,libs,test}/**/*.ts\" --fix",
+    "lint:check": "eslint \"{src,apps,libs,test}/**/*.ts\"",
     "test": "jest",
     "test:watch": "jest --watch",
     "test:cov": "jest --coverage",
@@ -28,22 +29,18 @@
     "@nestjs/mapped-types": "^2.1.1",
     "@nestjs/passport": "^11.0.5",
     "@nestjs/platform-express": "^11.1.19",
-    "@nestjs/platform-socket.io": "^11.1.19",
     "@nestjs/serve-static": "^5.0.5",
     "@nestjs/swagger": "^11.3.2",
     "@nestjs/throttler": "^6.5.0",
-    "@nestjs/websockets": "^11.1.19",
     "@polar-sh/sdk": "^0.48.1",
     "@rekog/mcp-nest": "^1.9.10",
     "@supabase/supabase-js": "^2.104.0",
     "class-transformer": "^0.5.1",
     "class-validator": "^0.14.4",
-    "csrf-csrf": "^4.0.3",
     "helmet": "^8.1.0",
     "passport-jwt": "^4.0.1",
     "reflect-metadata": "^0.2.2",
     "rxjs": "^7.8.2",
-    "swagger-ui-dist": "^5.32.4",
     "zod": "^4.4.3"
   },
   "devDependencies": {
@@ -55,14 +52,14 @@
     "@types/express": "^5.0.6",
     "@types/jest": "^30.0.0",
     "@types/node": "^22.19.17",
+    "@types/passport-jwt": "^4.0.1",
     "@types/supertest": "^6.0.3",
     "eslint": "^9.39.4",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-prettier": "^5.5.5",
     "globals": "^16.5.0",
     "jest": "^30.3.0",
     "prettier": "^3.8.3",
-    "source-map-support": "^0.5.21",
     "supabase": "^2.93.0",
     "supertest": "^7.2.2",
     "ts-jest": "^29.4.9",

src/app.controller.ts

@@ -5,7 +5,6 @@ import type { Response } from 'express';
 
 @Controller()
 export class AppController {
-
   @Get()
   @ApiExcludeEndpoint()
   @ApiOkResponse({