update to modern variant

Signed-off-by: Kirill Mokevnin <[email protected]>

Author: mokevnin

.gitignore

@@ -56,3 +56,4 @@ profile-*
 profile*
 *clinic*
 *flamegraph*
+.env

AGENTS.md

@@ -0,0 +1,40 @@
+# Repository Guidelines
+
+## Project Structure & Module Organization
+
+- `app.js`: Fastify entry; autoloads `plugins/` and `routes/`.
+- `routes/`: HTTP handlers.
+- `plugins/`: App plugins (JWT auth, DB, sensible errors, response validation, route glue).
+- `db/`: Drizzle ORM schema and seeds; `drizzle/` holds generated migrations.
+- `lib/`: Utilities and test data builders.
+- `test/`: Node tests; helpers in `test/helper.js`, route specs in `test/routes/*.test.js`.
+- `types/`: ts types
+- `main.tsp`, `tsp-output/`: TypeSpec and generated OpenAPI/handler typings.
+
+## Build, Test, and Development Commands
+- `npm run dev`: Start Fastify with watch on http://localhost:3000.
+- `npm start`: Start in production mode.
+- `npm test` or `make test`: Run Node tests (`node --test`).
+- `make lint` / `make lint-fix`: Lint and autofix via ESLint.
+- `make check-types`: Type-check with `tsc` (JS + d.ts).
+- `make generate-types`: Compile TypeSpec and generate Fastify handler types.
+- `make migration-generate`: Generate Drizzle migrations.
+- `make mock`: Serve mocked API from generated OpenAPI.
+
+## Coding Style & Naming Conventions
+- **Modules**: ESM only (`type: module`). Prefer named exports; keep JSDoc types consistent with `types/`.
+- **Formatting**: 2-space indent, no semicolons; follow ESLint + `@stylistic` rules. Run `make lint` before committing.
+- **Files**: Group endpoints by resource in `routes/api/` (e.g., `routes/api/books.js`). Co-locate validators and serializers by domain when present.
+
+## Testing Guidelines
+- **Framework**: Node built-in `node:test` with `app.inject()`; see `test/helper.js` for server bootstrap.
+- **Naming**: Place specs under `test/routes/` as `*.test.js` (e.g., `test/routes/users.test.js`).
+- **Scope**: Add success tests for each new/changed route. No coverage gate enforced.
+
+## Commit & Pull Request Guidelines
+- **Commits**: Use clear, imperative messages (optionally Conventional Commits). Reference issues when applicable.
+- **PRs**: Provide purpose, summary, linked issues, test plan, and example requests/responses (curl or HTTPie). Keep diffs focused.
+
+## Security & Configuration Tips
+- **Secrets**: Move JWT secret to env (e.g., `JWT_SECRET`) rather than hardcoding; use `.env` locally and never commit secrets.
+- **DB**: Current DB is in-memory SQLite (`plugins/drizzle.js`). Switch to file/real DB for persistence before production.

Makefile

@@ -5,10 +5,10 @@ dev:
 	npm run dev
 
 check-types:
-	npx tsc
+	npx tsc --noEmit
 
 routes:
-	npx fastify print-routes routes/users.js
+	npx fastify print-routes routes/api/users.js
 
 migration-generate:
 	npx drizzle-kit generate
@@ -19,13 +19,15 @@ lint:
 lint-fix:
 	npx eslint --fix .
 
-types-to-openapi:
+generate-openapi:
 	npx tsp compile .
 
-types-to-typebox:
-	npx openapi-box ./tsp-output/@typespec/openapi3/openapi.v1.json
+generate-openapi-ts-types:
+# 	# npx openapi-box ./tsp-output/@typespec/openapi3/openapi.v1.json
+	# npx openapi-typescript ./tsp-output/@typespec/openapi3/openapi.v1.json -o types/openapi.ts
+	npx openapi-ts
 
-types: types-to-openapi types-to-typebox
+generate-types: generate-openapi generate-openapi-ts-types
 
 mock:
 	npx prism mock ./tsp-output/@typespec/openapi3/openapi.v1.json

app.js

@@ -0,0 +1,63 @@
+import path from 'node:path';
+import type { AutoloadPluginOptions } from '@fastify/autoload';
+import AutoLoad from '@fastify/autoload';
+import { errors } from '@vinejs/vine';
+import type { FastifyPluginAsync, FastifyServerOptions } from 'fastify';
+import glue from 'fastify-openapi-glue';
+import serviceHandlers from './routes/index.ts';
+
+export interface AppOptions
+  extends FastifyServerOptions,
+    Partial<AutoloadPluginOptions> {}
+
+// Pass --options via CLI arguments in command to enable these options.
+const options: AppOptions = {};
+
+const app: FastifyPluginAsync<AppOptions> = async (
+  fastify,
+  opts,
+): Promise<void> => {
+  fastify.setErrorHandler((error, _request, reply) => {
+    if (error instanceof errors.E_VALIDATION_ERROR) {
+      const errorDetail = {
+        status: 422,
+        title: 'Validation Error',
+        detail: 'Errors related to business logic such as uniqueness',
+        errors: error.messages,
+      };
+      reply.type('application/problem+json').code(422).send(errorDetail);
+    } else {
+      reply.send(error);
+    }
+  });
+
+  fastify.addContentTypeParser(
+    'application/problem+json',
+    { parseAs: 'string' },
+    fastify.getDefaultJsonParser('ignore', 'ignore'),
+  );
+
+  // This loads all plugins defined in plugins
+  // those should be support plugins that are reused
+  // through your application
+  fastify.register(AutoLoad, {
+    dir: path.join(import.meta.dirname, 'plugins'),
+    options: opts,
+  });
+
+  fastify.register(glue, {
+    // prefix: 'v1',
+    serviceHandlers,
+    specification: './tsp-output/@typespec/openapi3/openapi.v1.json',
+  });
+
+  // This loads all plugins defined in routes
+  // define your routes in one of these
+  // fastify.register(AutoLoad, {
+  //   dir: path.join(import.meta.dirname, 'routes'),
+  //   options: opts,
+  // });
+};
+
+export default app;
+export { app, options };

app.ts

@@ -0,0 +1,51 @@
+{
+  "$schema": "./node_modules/@biomejs/biome/configuration_schema.json",
+  "vcs": {
+    "enabled": true,
+    "clientKind": "git",
+    "useIgnoreFile": true
+  },
+  "files": {
+    "ignoreUnknown": false,
+    "includes": [
+      "**",
+      "!test/**",
+      "!public/**",
+      "!types/handlers/**"
+    ]
+  },
+  "formatter": {
+    "enabled": true,
+    "indentStyle": "space"
+  },
+  "linter": {
+    "enabled": true,
+    "rules": {
+      "recommended": true,
+      "complexity": {
+        "noBannedTypes": "off"
+      },
+      "correctness": {
+        "noUnusedImports": "off",
+        "noUnusedFunctionParameters": "off",
+        "noUnusedVariables": "off"
+      },
+      "style": {
+        "noNonNullAssertion": "off"
+      }
+    }
+  },
+  "javascript": {
+    "formatter": {
+      "quoteStyle": "single"
+    }
+  },
+  "assist": {
+    "enabled": true,
+    "actions": {
+      "source": {
+        "organizeImports": "on"
+      }
+    }
+  }
+}

biome.json

@@ -1,36 +1,30 @@
-import { sql } from 'drizzle-orm'
-import {
-  text,
-  integer,
-  sqliteTable,
-} from 'drizzle-orm/sqlite-core'
+import { sql } from 'drizzle-orm';
+import { integer, sqliteTable, text } from 'drizzle-orm/sqlite-core';
 
 export const users = sqliteTable('users', {
   id: integer('id').primaryKey(),
   fullName: text('full_name'),
   email: text('email').notNull().unique(),
   updatedAt: text('updated_at'),
-  createdAt: text('created_at')
-    .notNull()
-    .default(sql`(unixepoch())`),
-})
+  createdAt: text('created_at').notNull().default(sql`(unixepoch())`),
+});
 
 export const courses = sqliteTable('courses', {
   id: integer('id').primaryKey(),
   name: text('name').notNull(),
-  creatorId: integer('creator_id').references(() => users.id).notNull(),
+  creatorId: integer('creator_id')
+    .references(() => users.id)
+    .notNull(),
   description: text('description').notNull(),
-  createdAt: text('created_at')
-    .notNull()
-    .default(sql`(unixepoch())`),
-})
+  createdAt: text('created_at').notNull().default(sql`(unixepoch())`),
+});
 
 export const courseLessons = sqliteTable('course_lessons', {
   id: integer('id').primaryKey(),
   name: text('name').notNull(),
-  courseId: integer('courseId').references(() => courses.id).notNull(),
+  courseId: integer('courseId')
+    .references(() => courses.id)
+    .notNull(),
   body: text('body').notNull(),
-  createdAt: text('created_at')
-    .notNull()
-    .default(sql`(unixepoch())`),
-})
+  createdAt: text('created_at').notNull().default(sql`(unixepoch())`),
+});

db/schema.js renamed to db/schema.ts

@@ -1,9 +1,10 @@
-import * as schemas from './schema.js'
-import { buildCourse, buildCourseLesson, buildUser } from '../lib/data.js'
+import { buildCourse, buildCourseLesson, buildUser } from '../lib/data.ts'
+import type { DrizzleDB } from '../types/index.ts'
+import * as schemas from './schema.ts'
 /**
  * @param {import("drizzle-orm/better-sqlite3").BetterSQLite3Database<typeof schemas>} db
  */
-export default async (db) => {
+export default async (db: DrizzleDB) => {
   const [_user1] = await db.insert(schemas.users).values(buildUser()).returning()
   const [user2] = await db.insert(schemas.users).values(buildUser()).returning()
   const [_user3] = await db.insert(schemas.users).values(buildUser({

db/seeds.js renamed to db/seeds.ts

@@ -0,0 +1,7 @@
+import { defineConfig } from "drizzle-kit";
+
+export default defineConfig({
+  dialect: "sqlite",
+  schema: "./db/schema.ts",
+  // out: "./drizzle",
+});