code-ai-installer 1.2.0 → 1.3.1

package/locales/en/.agents/dev_reference_snippets/SKILL.md

@@ -1,397 +1,585 @@
----
-name: dev_reference_snippets
-description: A single file with snippets and anti-examples (do/don’t) for Senior Full Stack: TDD, API, validation, errors, logging, React, state (Zustand/RTK), security, legacy React 15.3 (Wix), DoD scripts.
----
-
-# Skill: Dev Reference Snippets (Do/Don't)
-
-## Goal
-Provide copy-paste standards and anti-patterns so that the code is consistent, testable (TDD), safe and easy to maintain.
-
----
-
-## 1) TDD: RED → GREEN → REFACTOR (example)
-
-### ✅ DO: test first (Vitest)
-```ts
-// src/lib/slugify.test.ts
-import { describe, it, expect } from "vitest";
-import { slugify } from "./slugify";
-
-describe("slugify", () => {
-  it("makes lowercase and replaces spaces with hyphens", () => {
-    expect(slugify("Hello World")).toBe("hello-world");
-  });
-
-  it("removes non-alphanumeric characters", () => {
-    expect(slugify("Hi, John!")).toBe("hi-john");
-  });
-
-  it("collapses multiple spaces", () => {
-    expect(slugify("  Hello   World  ")).toBe("hello-world");
-  });
-});
-```
-
-### ✅ GREEN: minimal implementation
-```ts
-// src/lib/slugify.ts
-export function slugify(input: string): string {
-  return input
-    .trim()
-    .toLowerCase()
-    .replace(/[^a-z0-9\s-]/g, "")
-    .replace(/\s+/g, "-");
-}
-```
-
-### ❌ DON'T: “code first, tests later” + testing details
-```ts
-// Плохо: тест проверяет внутренние переменные/шаги реализации, а не поведение.
-// Итог: тесты ломаются при рефакторинге без изменения поведения.
-```
-
----
-
-## 2) API: layers Route → Controller → Service → Repo (Express)
-
-### ✅ DO: project structure (example)
-```txt
-src/
-  http/
-    routes/
-    controllers/
-    middleware/
-  domain/
-    services/
-    models/
-  data/
-    repos/
-  lib/
-```
-
-### ✅ DO: Routes are thin, logic is in controller/service
-```ts
-// src/http/routes/users.routes.ts
-import { Router } from "express";
-import { createUser } from "../controllers/users.create";
-import { requireAuth } from "../middleware/requireAuth";
-
-export const usersRouter = Router();
-
-usersRouter.post("/", requireAuth, createUser);
-```
-
-### ❌ DON'T: everything in one route file
-```ts
-// Плохо: в routes файл запихана валидация, бизнес-логика и доступ к БД.
-// Тестирование и переиспользование становится болью.
-```
-
----
-
-## 3) Validation at the boundary + safe errors (Zod)
-
-### ✅ DO: parse on input, safe error handling
-```ts
-// src/http/controllers/users.create.ts
-import { z } from "zod";
-import type { Request, Response, NextFunction } from "express";
-import { usersService } from "../../domain/services/users.service";
-import { AppError } from "../middleware/errors";
-
-const CreateUserSchema = z.object({
-  email: z.string().email(),
-  name: z.string().min(1).max(100),
-});
-
-export async function createUser(req: Request, res: Response, next: NextFunction) {
-  try {
-    const input = CreateUserSchema.parse(req.body); // ✅ boundary validation
-    const user = await usersService.create(input);
-    res.status(201).json({ id: user.id });
-  } catch (err) {
-    // zod errors -> 422
-    next(err instanceof z.ZodError ? AppError.validation(err.flatten()) : err);
-  }
-}
-```
-
-### ❌ DON'T: trust req.body
-```ts
-// Плохо: нет валидации, можно словить мусор/инъекции/500
-export async function createUser(req: any, res: any) {
-  const user = await usersService.create(req.body);
-  res.json(user);
-}
-```
-
----
-
-## 4) Centralized error handler (single error format)
-
-### ✅ DO: AppError + single format { error_code, message, details? }
-```ts
-// src/http/middleware/errors.ts
-import type { ErrorRequestHandler } from "express";
-
-export class AppError extends Error {
-  constructor(
-    public readonly status: number,
-    public readonly code: string,
-    message: string,
-    public readonly details?: unknown
-  ) {
-    super(message);
-  }
-
-  static validation(details: unknown) {
-    return new AppError(422, "VALIDATION_ERROR", "Invalid input", details);
-  }
-
-  static forbidden() {
-    return new AppError(403, "FORBIDDEN", "Not enough permissions");
-  }
-}
-
-export const errorHandler: ErrorRequestHandler = (err, _req, res, _next) => {
-  const isApp = err instanceof AppError;
-  const status = isApp ? err.status : 500;
-  const code = isApp ? err.code : "INTERNAL_ERROR";
-
-  // Без утечек: для 500 — нейтральное сообщение
-  const message = status === 500 ? "Unexpected error" : err.message;
-
-  res.status(status).json({
-    error_code: code,
-    message,
-    ...(isApp && err.details ? { details: err.details } : {}),
-  });
-};
-```
-
-### ❌ DON'T: give the entire “err” (stack/SQL/secrets leak)
-```ts
-// Плохо: утечка внутренностей и нестабильный контракт
-res.status(500).json({ err });
-```
-
----
-
-## 5) Logging: request_id, structured logs, prohibition of PII/secrets
-
-### ✅ DO: request_id middleware + structured logs
-```ts
-// src/http/middleware/requestContext.ts
-import { randomUUID } from "node:crypto";
-import type { RequestHandler } from "express";
-
-export const requestContext: RequestHandler = (req, res, next) => {
-  const id = req.header("x-request-id") ?? randomUUID();
-  res.setHeader("x-request-id", id);
-  (req as any).requestId = id;
-  next();
-};
-```
-
-```ts
-// usage example
-logger.info(
-  { request_id: (req as any).requestId, user_id: ctx.user?.id },
-  "user_created"
-);
-```
-
-### ❌ DON'T: log the entire body (passwords/tokens/PII there)
-```ts
-// Плохо: утечка секретов в логи
-logger.info({ body: req.body }, "incoming_request");
-```
-
----
-
-## 6) React: loading/empty/error/success (TanStack Query)
-
-### ✅ DO: explicit states
-```tsx
-import { useQuery } from "@tanstack/react-query";
-
-type User = { id: string; name: string };
-
-async function fetchUsers(): Promise<User[]> {
-  const r = await fetch("/api/users");
-  if (!r.ok) throw new Error("Failed to load users");
-  return r.json();
-}
-
-export function UsersList() {
-  const q = useQuery({ queryKey: ["users"], queryFn: fetchUsers });
-
-  if (q.isLoading) return <div aria-busy="true">Loading…</div>;
-  if (q.isError) return <div role="alert">Error: {String(q.error)}</div>;
-  if (!q.data || q.data.length === 0) return <div>No users yet</div>;
-
-  return (
-    <ul>
-      {q.data.map((u) => (
-        <li key={u.id}>{u.name}</li>
-      ))}
-    </ul>
-  );
-}
-```
-
-### ❌ DON'T: fetch inside render / no error/empty
-```tsx
-// Плохо: нет обработки ошибок/пустоты, легко словить гонки и “мерцания”.
-```
-
----
-
-## 7) Zustand: small domain stores + selectors
-
-### ✅ DO: domain store + selector
-```ts
-import { create } from "zustand";
-
-type AuthState = {
-  token: string | null;
-  setToken: (t: string | null) => void;
-};
-
-export const useAuthStore = create<AuthState>((set) => ({
-  token: null,
-  setToken: (t) => set({ token: t }),
-}));
-
-// usage: подписываемся только на нужное
-const token = useAuthStore((s) => s.token);
-```
-
-### ❌ DON'T: “god-store” for the entire application
-```ts
-// Плохо: один огромный store с десятками полей и действий → лишние ререндеры и каша доменов.
-```
-
----
-
-## 8) RTK: slices by domain + normalization (EntityAdapter)
-
-### ✅ DO: entityAdapter for user list
-```ts
-// src/state/usersSlice.ts
-import { createSlice, createAsyncThunk, createEntityAdapter } from "@reduxjs/toolkit";
-import type { RootState } from "./store";
-
-type User = { id: string; name: string };
-
-export const usersAdapter = createEntityAdapter<User>();
-
-export const fetchUsers = createAsyncThunk("users/fetch", async () => {
-  const r = await fetch("/api/users");
-  if (!r.ok) throw new Error("Failed to load users");
-  return (await r.json()) as User[];
-});
-
-const slice = createSlice({
-  name: "users",
-  initialState: usersAdapter.getInitialState({ status: "idle" as "idle" | "loading" | "failed" }),
-  reducers: {},
-  extraReducers: (b) => {
-    b.addCase(fetchUsers.pending, (s) => {
-      s.status = "loading";
-    });
-    b.addCase(fetchUsers.fulfilled, (s, a) => {
-      s.status = "idle";
-      usersAdapter.setAll(s, a.payload);
-    });
-    b.addCase(fetchUsers.rejected, (s) => {
-      s.status = "failed";
-    });
-  },
-});
-
-export const usersReducer = slice.reducer;
-
-export const usersSelectors = usersAdapter.getSelectors<RootState>((st) => st.users);
-```
-
-### ❌ DON'T: side effects in components + manual “sheets” of reducers
-```ts
-// Плохо: компонент сам дергает fetch, сам хранит кэш, сам нормализует.
-// Итог: непредсказуемость и дублирование.
-```
-
----
-
-## 9) Security: authz on the server (don't trust the client)
-
-### ✅ DO: checking rights on the server
-```ts
-import { AppError } from "../http/middleware/errors";
-
-export function requireRole(user: { role: string } | null, role: string) {
-  if (!user) throw new AppError(401, "UNAUTHORIZED", "Authentication required");
-  if (user.role !== role) throw AppError.forbidden();
-}
-```
-
-### ❌ DON'T: “if the button is hidden, it means it’s safe”
-```ts
-// Плохо: безопасность только на UI.
-// Сервер обязан проверять авторизацию/права.
-```
-
----
-
-## 10) Legacy: React 15.3 (Wix iFrame) + DoD scripts
-
-### ✅ DO: React 15.3 class component + lifecycle
-```jsx
-class Widget extends React.Component {
-  constructor(props) {
-    super(props);
-    this.state = { loading: true, data: null, error: null };
-  }
-
-  componentDidMount() {
-    this.load();
-  }
-
-  load() {
-    apiFetch()
-      .then((data) => this.setState({ loading: false, data: data }))
-      .catch(() => this.setState({ loading: false, error: "Failed" }));
-  }
-
-  render() {
-    if (this.state.loading) return React.createElement("div", null, "Loading…");
-    if (this.state.error) return React.createElement("div", { role: "alert" }, this.state.error);
-    return React.createElement("div", null, "OK");
-  }
-}
-```
-
-### ❌ DON'T: hooks/modern APIs (not allowed in React 15.3)
-```tsx
-// Нельзя в React 15.3
-// function Widget(){ const [x,setX]=useState(...) }
-```
-
-### ✅ DO: DoD scripts (package.json)
-```json
-{
-  "scripts": {
-    "dev": "node ./src/index.js",
-    "test": "vitest",
-    "test:coverage": "vitest --coverage",
-    "lint": "biome lint .",
-    "format": "biome format . --write"
-  }
-}
-```
-
-### ❌ DON'T: mix formatters/linters without reason
-```txt
-// Плохо: eslint + prettier + biome одновременно без согласованных правил → вечные конфликты и шум в PR.
-```
+---
+name: dev_reference_snippets
+description: Reference snippets and DO/DON'T patterns for Senior Full Stack: TDD (Vitest), Express API (Route?Controller?Service?Repo), validation (Zod), centralized errors, structured logging, React + TanStack Query, Zustand, RTK (EntityAdapter), server-side authorization, legacy React 15.3 (Wix). Use this skill when writing any production code in Node.js/TypeScript/React ? as a source of copy-paste reference patterns and a checklist for code. Also use it for questions like "how to do X correctly" for any of the listed patterns.
+---
+
+# Skill: Dev Reference Snippets (Do/Don't)
+
+Copy-paste references and anti-patterns for consistent, tested, secure code.
+
+**Sections:**
+1. [TDD: RED → GREEN → REFACTOR](#1-tdd)
+2. [API: Route → Controller → Service → Repo](#2-api-layers)
+3. [Validation on boundary (Zod)](#3-validation)
+4. [Centralized error handler](#4-error-handler)
+5. [Logging: request_id + structure](#5-logging)
+6. [React: loading/empty/error/success](#6-react-states)
+7. [Zustand: domain stores](#7-zustand)
+8. [RTK: slices + EntityAdapter](#8-rtk)
+9. [Authorization on server](#9-authorization)
+10. [Legacy React 15.3 (Wix)](#10-legacy-react-153)
+11. [DoD Scripts](#11-dod-scripts)
+
+---
+
+## 1. TDD
+
+### ✅ DO: test first (Vitest)
+```ts
+// src/lib/slugify.test.ts
+import { describe, it, expect } from "vitest";
+import { slugify } from "./slugify";
+
+describe("slugify", () => {
+  it("lowercases and replaces spaces with hyphens", () => {
+    expect(slugify("Hello World")).toBe("hello-world");
+  });
+  it("removes non-alphanumeric characters", () => {
+    expect(slugify("Hi, John!")).toBe("hi-john");
+  });
+  it("collapses multiple spaces", () => {
+    expect(slugify("  Hello   World  ")).toBe("hello-world");
+  });
+});
+```
+
+### ✅ GREEN: minimal implementation
+```ts
+// src/lib/slugify.ts
+export function slugify(input: string): string {
+  return input
+    .trim()
+    .toLowerCase()
+    .replace(/[^a-z0-9\s-]/g, "")
+    .replace(/\s+/g, "-");
+}
+```
+
+### ❌ DON'T: test details implementation
+```ts
+// Bad: test checks internal variables, and not behavior.
+// Will break with refactoring without changes behavior — fragile tests.
+```
+
+**Rule**: test *what it does*, not *how it is implemented inside*.
+
+---
+
+## 2. API-layers
+
+### ✅ DO: structure project
+```
+src/
+  http/
+    routes/       ? routing only
+    controllers/  ← I/O: parse input, call service, format response
+    middleware/   ← auth, errors, request-context
+  domain/
+    services/     ← business logic
+    models/       ← types domain
+  data/
+    repos/        ← SQL/ORM, no business logic
+  lib/            ← utilities without dependencies
+```
+
+### ? DO: thin route
+```ts
+// src/http/routes/users.routes.ts
+import { Router } from "express";
+import { createUser, getUser } from "../controllers/users.controller";
+import { requireAuth } from "../middleware/requireAuth";
+
+export const usersRouter = Router();
+usersRouter.post("/", requireAuth, createUser);
+usersRouter.get("/:id", requireAuth, getUser);
+```
+
+### ✅ DO: controller — only I/O
+```ts
+// src/http/controllers/users.controller.ts
+import { z } from "zod";
+import type { Request, Response, NextFunction } from "express";
+import { usersService } from "../../domain/services/users.service";
+import { AppError } from "../middleware/errors";
+
+const CreateUserSchema = z.object({
+  email: z.string().email(),
+  name: z.string().min(1).max(100),
+});
+
+export async function createUser(req: Request, res: Response, next: NextFunction) {
+  try {
+    const input = CreateUserSchema.parse(req.body);
+    const user = await usersService.create(input);
+    res.status(201).json({ id: user.id });
+  } catch (err) {
+    next(err instanceof z.ZodError ? AppError.validation(err.flatten()) : err);
+  }
+}
+```
+
+### ✅ DO: service — only business logic, without HTTP
+```ts
+// src/domain/services/users.service.ts
+import { usersRepo } from "../../data/repos/users.repo";
+import { AppError } from "../../http/middleware/errors";
+
+export const usersService = {
+  async create(input: { email: string; name: string }) {
+    const existing = await usersRepo.findByEmail(input.email);
+    if (existing) throw new AppError(409, "CONFLICT", "Email already in use");
+    return usersRepo.insert(input);
+  },
+};
+```
+
+### ❌ DON'T: everything in one file
+```ts
+// Bad: routes, validation, business logic, and SQL in one file.
+// Cannot test service without HTTP, cannot reuse logic.
+```
+
+---
+
+## 3. Validation
+
+### ? DO: Zod at the boundary + parse (throws on invalid data)
+```ts
+const CreateUserSchema = z.object({
+  email: z.string().email(),
+  name: z.string().min(1).max(100),
+  role: z.enum(["admin", "user"]).default("user"),
+});
+
+// In controller:
+const input = CreateUserSchema.parse(req.body); // ? valid type guaranteed further
+```
+
+### ? DO: safeParse for non-throwing handling
+```ts
+const result = CreateUserSchema.safeParse(req.body);
+if (!result.success) {
+  return next(AppError.validation(result.error.flatten()));
+}
+const input = result.data; // type is inferred automatically
+```
+
+### ❌ DON'T: trust req.body without validation
+```ts
+// Bad: any garbage, extra-field injection, 500 at runtime ? all of this is waiting ahead.
+export async function createUser(req: any, res: any) {
+  const user = await usersService.create(req.body);
+  res.json(user);
+}
+```
+
+---
+
+## 4. Error Handler
+
+### ✅ DO: AppError + unified format answer
+```ts
+// src/http/middleware/errors.ts
+import type { ErrorRequestHandler } from "express";
+
+export class AppError extends Error {
+  constructor(
+    public readonly status: number,
+    public readonly code: string,
+    message: string,
+    public readonly details?: unknown
+  ) {
+    super(message);
+    this.name = "AppError";
+  }
+
+  static validation(details: unknown) {
+    return new AppError(422, "VALIDATION_ERROR", "Invalid input", details);
+  }
+
+  static notFound(resource = "Resource") {
+    return new AppError(404, "NOT_FOUND", `${resource} not found`);
+  }
+
+  static forbidden() {
+    return new AppError(403, "FORBIDDEN", "Not enough permissions");
+  }
+}
+
+export const errorHandler: ErrorRequestHandler = (err, _req, res, _next) => {
+  const isApp = err instanceof AppError;
+  const status = isApp ? err.status : 500;
+  const code   = isApp ? err.code  : "INTERNAL_ERROR";
+  // ? For 500 ? neutral message, stack does not leak to the client
+  const message = status === 500 ? "Unexpected error" : err.message;
+
+  res.status(status).json({
+    error_code: code,
+    message,
+    ...(isApp && err.details ? { details: err.details } : {}),
+  });
+};
+```
+
+### ✅ DO: connect last middleware
+```ts
+// src/app.ts
+app.use(requestContext);
+app.use("/api/users", usersRouter);
+app.use(errorHandler); // ← last
+```
+
+### ❌ DON'T: return err entirely
+```ts
+// Bad: leak stack traces, SQL errors, environment variables ? directly in the client response.
+res.status(500).json({ err });
+```
+
+---
+
+## 5. Logging
+
+### ✅ DO: request_id + structured logs
+```ts
+// src/http/middleware/requestContext.ts
+import { randomUUID } from "node:crypto";
+import type { RequestHandler } from "express";
+
+export const requestContext: RequestHandler = (req, res, next) => {
+  const id = req.header("x-request-id") ?? randomUUID();
+  res.setHeader("x-request-id", id);
+  (req as any).requestId = id;
+  next();
+};
+```
+
+```ts
+// Usage in service/controller
+logger.info(
+  { request_id: (req as any).requestId, user_id: ctx.user?.id, action: "user_created" },
+  "User created successfully"
+);
+
+// For errors — warn/error with context
+logger.error(
+  { request_id: id, error_code: err.code },
+  "Request failed"
+);
+```
+
+### ❌ DON'T: log body entirely
+```ts
+// Bad: passwords, tokens, PII ? everything directly in the logs.
+logger.info({ body: req.body }, "incoming_request");
+
+// ❌ Also bad:
+logger.info({ user }); // can contain password_hash, secrets and etc.etc.
+```
+
+**Rule**: log only IDs, codes, statuses. Never raw request bodies.
+
+---
+
+## 6. React: States request
+
+### ✅ DO: TanStack Query — explicit loading/empty/error/success
+```tsx
+import { useQuery } from "@tanstack/react-query";
+
+type User = { id: string; name: string };
+
+async function fetchUsers(): Promise<User[]> {
+  const r = await fetch("/api/users");
+  if (!r.ok) throw new Error(`HTTP ${r.status}`);
+  return r.json();
+}
+
+export function UsersList() {
+  const q = useQuery({ queryKey: ["users"], queryFn: fetchUsers });
+
+  if (q.isLoading) return <div aria-busy="true" role="status">Loading…</div>;
+  if (q.isError)   return <div role="alert">Error: {String(q.error)}</div>;
+  if (!q.data?.length) return <div>Users for now no</div>;
+
+  return (
+    <ul>
+      {q.data.map(u => <li key={u.id}>{u.name}</li>)}
+    </ul>
+  );
+}
+```
+
+### ? DO: mutation with optimistic update
+```tsx
+import { useMutation, useQueryClient } from "@tanstack/react-query";
+
+export function useDeleteUser() {
+  const qc = useQueryClient();
+  return useMutation({
+    mutationFn: (id: string) =>
+      fetch(`/api/users/${id}`, { method: "DELETE" }).then(r => {
+        if (!r.ok) throw new Error("Delete failed");
+      }),
+    onSuccess: () => qc.invalidateQueries({ queryKey: ["users"] }),
+  });
+}
+```
+
+### ❌ DON'T: fetch inside component without handling states
+```tsx
+// Bad: race conditions, flicker, no loading/error, request leak on unmount.
+function UsersList() {
+  const [users, setUsers] = useState([]);
+  useEffect(() => {
+    fetch("/api/users").then(r => r.json()).then(setUsers);
+  }, []);
+  return <ul>{users.map(u => <li key={u.id}>{u.name}</li>)}</ul>;
+}
+```
+
+---
+
+## 7. Zustand
+
+### ✅ DO: domain store + selectors
+```ts
+import { create } from "zustand";
+import { devtools } from "zustand/middleware";
+
+type AuthState = {
+  token: string | null;
+  userId: string | null;
+  setAuth: (token: string, userId: string) => void;
+  clearAuth: () => void;
+};
+
+export const useAuthStore = create<AuthState>()(
+  devtools(
+    (set) => ({
+      token: null,
+      userId: null,
+      setAuth: (token, userId) => set({ token, userId }, false, "setAuth"),
+      clearAuth: () => set({ token: null, userId: null }, false, "clearAuth"),
+    }),
+    { name: "AuthStore" }
+  )
+);
+
+// ✅ Selector — subscription only on needed field (no extra re-renders)
+const token = useAuthStore(s => s.token);
+const isAuthed = useAuthStore(s => s.token !== null);
+```
+
+### ❌ DON'T: one "god store" on everything
+```ts
+// Bad: auth + cart + ui + settings in one store.
+// Result: any change = rerender of all subscribers.
+// Rule: one store = one domain.
+```
+
+---
+
+## 8. RTK
+
+### ✅ DO: slice + createAsyncThunk + EntityAdapter
+```ts
+// src/state/usersSlice.ts
+import { createSlice, createAsyncThunk, createEntityAdapter } from "@reduxjs/toolkit";
+import type { RootState } from "./store";
+
+type User = { id: string; name: string };
+
+const usersAdapter = createEntityAdapter<User>();
+
+export const fetchUsers = createAsyncThunk("users/fetch", async () => {
+  const r = await fetch("/api/users");
+  if (!r.ok) throw new Error("Failed to load users");
+  return (await r.json()) as User[];
+});
+
+const slice = createSlice({
+  name: "users",
+  initialState: usersAdapter.getInitialState({
+    status: "idle" as "idle" | "loading" | "failed",
+  }),
+  reducers: {},
+  extraReducers: b => {
+    b.addCase(fetchUsers.pending,   s => { s.status = "loading"; });
+    b.addCase(fetchUsers.fulfilled, (s, a) => {
+      s.status = "idle";
+      usersAdapter.setAll(s, a.payload);
+    });
+    b.addCase(fetchUsers.rejected,  s => { s.status = "failed"; });
+  },
+});
+
+export const usersReducer = slice.reducer;
+export const usersSelectors = usersAdapter.getSelectors<RootState>(st => st.users);
+```
+
+### ✅ DO: connect reducer in store
+```ts
+// src/state/store.ts
+import { configureStore } from "@reduxjs/toolkit";
+import { usersReducer } from "./usersSlice";
+
+export const store = configureStore({
+  reducer: { users: usersReducer },
+});
+
+export type RootState = ReturnType<typeof store.getState>;
+export type AppDispatch = typeof store.dispatch;
+```
+
+### ? DON'T: side effects in the component + manual reducers
+```ts
+// Bad: fetch in component, manual switch/case, normalization by hand.
+// Result: duplication, unpredictability, tests require real HTTP.
+```
+
+---
+
+## 9. Authorization
+
+### ✅ DO: checking rights on the server
+```ts
+// src/lib/authz.ts
+import { AppError } from "../http/middleware/errors";
+
+type Principal = { id: string; role: "admin" | "user" } | null;
+
+export function requireRole(principal: Principal, role: "admin" | "user") {
+  if (!principal) throw new AppError(401, "UNAUTHORIZED", "Authentication required");
+  if (principal.role !== role) throw AppError.forbidden();
+}
+
+// Usage in service:
+export const postsService = {
+  async delete(id: string, actor: Principal) {
+    requireRole(actor, "admin"); // ✅ server always checks
+    return postsRepo.delete(id);
+  },
+};
+```
+
+### ✅ DO: requireAuth middleware
+```ts
+// src/http/middleware/requireAuth.ts
+import type { RequestHandler } from "express";
+import { verifyToken } from "../../lib/jwt";
+import { AppError } from "./errors";
+
+export const requireAuth: RequestHandler = (req, _res, next) => {
+  const header = req.header("Authorization");
+  if (!header?.startsWith("Bearer ")) return next(new AppError(401, "UNAUTHORIZED", "Missing token"));
+  try {
+    (req as any).principal = verifyToken(header.slice(7));
+    next();
+  } catch {
+    next(new AppError(401, "UNAUTHORIZED", "Invalid token"));
+  }
+};
+```
+
+### ❌ DON'T: security only on UI
+```ts
+// Bad: hide the "Delete" button for non-admins and treat this as protection.
+// Any fetch("/api/posts/1", { method: "DELETE" }) bypasses this.
+// Server Must check permissions independently from UI.
+```
+
+---
+
+## 10. Legacy React 15.3 (Wix)
+
+### ✅ DO: class component + lifecycle
+```jsx
+// ❗ Only ES5/ES6 classes, no hooks
+class Widget extends React.Component {
+  constructor(props) {
+    super(props);
+    this.state = { loading: true, data: null, error: null };
+    this.handleRetry = this.handleRetry.bind(this);
+  }
+
+  componentDidMount() {
+    this.load();
+  }
+
+  componentWillUnmount() {
+    this._unmounted = true; // prevent setState after unmount
+  }
+
+  load() {
+    apiFetch()
+      .then(data => {
+        if (!this._unmounted) this.setState({ loading: false, data });
+      })
+      .catch(() => {
+        if (!this._unmounted) this.setState({ loading: false, error: "Failed to load" });
+      });
+  }
+
+  handleRetry() {
+    this.setState({ loading: true, error: null }, () => this.load());
+  }
+
+  render() {
+    const { loading, data, error } = this.state;
+    if (loading) return React.createElement("div", null, "Loading…");
+    if (error)   return React.createElement("div", { role: "alert" },
+      error,
+      React.createElement("button", { onClick: this.handleRetry }, "Retry")
+    );
+    return React.createElement("div", null, JSON.stringify(data));
+  }
+}
+```
+
+### ? DON'T: hooks and modern APIs in React 15.3
+```tsx
+// ❌ Cannot — hooks appeared in React 16.8
+function Widget() { const [x, setX] = useState(0); }
+// ❌ Cannot — React.memo appeared in React 16.6
+export default React.memo(Widget);
+// ❌ Cannot — fragments <>...</> appeared in React 16.2
+```
+
+---
+
+## 11. DoD Scripts
+
+### ✅ DO: standard package.json
+```json
+{
+  "scripts": {
+    "dev":           "tsx watch ./src/index.ts",
+    "build":         "tsc -p tsconfig.build.json",
+    "test":          "vitest run",
+    "test:watch":    "vitest",
+    "test:coverage": "vitest run --coverage",
+    "lint":          "biome lint .",
+    "lint:fix":      "biome lint . --apply",
+    "format":        "biome format . --write",
+    "typecheck":     "tsc --noEmit",
+    "check":         "npm run typecheck && npm run lint && npm run test"
+  }
+}
+```
+
+### ✅ DO: pre-commit hook (husky + lint-staged)
+```json
+// package.json
+{
+  "lint-staged": {
+    "*.{ts,tsx}": ["biome lint --apply", "biome format --write"]
+  }
+}
+```
+
+### ❌ DON'T: mix eslint + prettier + biome
+```
+// Bad: three tools with conflicting rules ? constant noise in PR, 
+// different results on different developers' machines.
+// Choose one: Biome (quick, all-in-one) or ESLint + Prettier.
+```