chore: test discipline + DX upgrades

.github/workflows/test.yaml

@@ -9,7 +9,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node-version: [20.x, 22.x]
+        node-version: [22.x, 24.x]
     steps:
       - uses: actions/checkout@v4
       - uses: pnpm/action-setup@v4
@@ -22,5 +22,14 @@ jobs:
         run: pnpm install --frozen-lockfile
       - name: Lint
         run: pnpm lint
-      - name: Test
-        run: pnpm test
+      - name: Build CLI
+        run: pnpm build
+      - name: Tests with coverage (unit + integration + e2e)
+        run: pnpm test:coverage
+      - name: Upload coverage report
+        if: matrix.node-version == '24.x'
+        uses: actions/upload-artifact@v4
+        with:
+          name: coverage-report
+          path: coverage/
+          retention-days: 14

.gitignore

@@ -204,3 +204,7 @@ package-lock.json
 **/obj/*
 **/nppBackup/*
 .claude/
+
+# Stryker mutation reports
+reports/
+.stryker-tmp/

CHANGELOG.md

@@ -0,0 +1 @@
+# Changelog

README.md

@@ -130,6 +130,27 @@ console.log(`Elements: ${report.elementsCount}`);
 - [ADR](ADRs/) — Architecture Decision Records
 - [Roadmap](roadmap.md) — планы развития
 
+## Testing
+
+Тестовый стек разделён на четыре уровня:
+
+```bash
+pnpm test            # все unit + integration + e2e
+pnpm test:unit       # только unit
+pnpm test:integration # интеграционные на реальных фикстурах
+pnpm test:e2e        # subprocess-тесты CLI через execa
+pnpm test:coverage   # с v8 coverage + порогами
+pnpm test:mutation   # Stryker mutation testing
+```
+
+**Метрики качества тестов:**
+
+- **Coverage** (v8): порог в CI — statements ≥97%, branches ≥90%, functions ≥99%, lines ≥98%
+- **Mutation score** (Stryker) ≥95% — каждое смысловое изменение в исходнике должно ломать хотя бы один тест
+- **Property-based** (`@fast-check/vitest`) на option-bearing правилах — защита от «hardcoded literal where option should be read» бага
+- **Inline snapshots** на генераторах для regression-pin'а формата вывода
+- **E2E** на цепочке `init → check → fix → recheck` через `npx aact` в subprocess
+
 ## Публичные материалы
 
 ### Раз архитектура — «as Code», почему бы её не покрыть тестами?!

changelog.config.ts

@@ -0,0 +1,25 @@
+import type { ChangelogConfig } from "changelogen";
+
+// changelogen — UnJS-aligned CHANGELOG.md generator from conventional commits.
+// Workflow: write commits in conventional format (we have commitlint enforcing
+// it), then run `pnpm release` to bump version, generate CHANGELOG section,
+// commit, and tag.
+//
+// Reference: https://github.com/unjs/changelogen
+export default <Partial<ChangelogConfig>>{
+  // Github org/repo for issue + PR + compare links in CHANGELOG.
+  repo: "Byndyusoft/aact",
+  // Hide author email in the contributors footer — keep CHANGELOG public-safe.
+  hideAuthorEmail: true,
+  // Sections in display order. Anything not listed is hidden.
+  types: {
+    feat: { title: "Features" },
+    perf: { title: "Performance" },
+    fix: { title: "Fixes" },
+    refactor: { title: "Refactors" },
+    docs: { title: "Documentation" },
+    test: { title: "Tests" },
+    build: { title: "Build" },
+    chore: { title: "Chore" },
+  },
+};

eslint.config.ts

@@ -1,15 +1,28 @@
 import js from "@eslint/js";
+import eslintComments from "@eslint-community/eslint-plugin-eslint-comments/configs";
+import vitest from "@vitest/eslint-plugin";
+import citty from "eslint-plugin-citty";
+import importX from "eslint-plugin-import-x";
 import nodePlugin from "eslint-plugin-n";
 import simpleImportSort from "eslint-plugin-simple-import-sort";
 import sonarjs from "eslint-plugin-sonarjs";
 import unicorn from "eslint-plugin-unicorn";
 import globals from "globals";
-import tseslint, { type ConfigArray } from "typescript-eslint";
+import type { ConfigArray } from "typescript-eslint";
+import tseslint from "typescript-eslint";
 
 // eslint-disable-next-line sonarjs/deprecation -- tseslint.config() is the recommended API
 export default tseslint.config(
   {
-    ignores: ["dist/", "node_modules/", "resources/"],
+    ignores: [
+      "dist/",
+      "node_modules/",
+      "resources/",
+      "coverage/",
+      "reports/",
+      ".stryker-tmp/",
+      "stryker.config.mjs",
+    ],
   },
   js.configs.recommended,
   ...tseslint.configs.recommendedTypeChecked,
@@ -27,6 +40,37 @@ export default tseslint.config(
   unicorn.configs.recommended,
   sonarjs.configs!.recommended as ConfigArray[number],
   nodePlugin.configs["flat/recommended-module"],
+  eslintComments.recommended,
+  // citty plugin — narrowly scoped to src/cli/**. Cherry-picked rules
+  // that match our coding style without overlap with simple-import-sort/etc.
+  {
+    files: ["src/cli/**/*.ts"],
+    plugins: { citty },
+    rules: {
+      "citty/no-duplicated-version": "error",
+      "citty/valid-version": "error",
+      "citty/no-meaningless-value-hint": "warn",
+      "citty/no-hidden-root-command": "warn",
+      "citty/enum-must-have-options": "error",
+      "citty/must-have-run-or-sub-commands": "error",
+      "citty/no-empty-command-properties": "warn",
+      "citty/no-default-on-positional": "warn",
+      "citty/no-alias-on-positional": "error",
+    },
+  },
+  // eslint-plugin-import-x: cherry-pick value rules. Skip rules that
+  // require eslint-import-resolver-typescript (no-unresolved, default,
+  // namespace, no-duplicates — they need a resolver setup we don't have).
+  {
+    plugins: { "import-x": importX },
+    rules: {
+      "import-x/no-cycle": ["error", { maxDepth: 10 }],
+      "import-x/no-self-import": "error",
+      "import-x/consistent-type-specifier-style": ["error", "prefer-top-level"],
+      "import-x/first": "error",
+      "import-x/newline-after-import": "error",
+    },
+  },
   {
     plugins: {
       "simple-import-sort": simpleImportSort,
@@ -64,6 +108,9 @@ export default tseslint.config(
       // node plugin
       "n/no-unsupported-features/node-builtins": "off",
       "n/no-missing-import": "off",
+      // Tool config files (*.config.ts, changelog.config.ts, etc.) import
+      // from devDeps — that's expected, not "extraneous".
+      "n/no-extraneous-import": "off",
 
       // sonarjs relaxations
       "sonarjs/cognitive-complexity": "warn",
@@ -72,6 +119,24 @@ export default tseslint.config(
       "sonarjs/slow-regex": "warn",
       "sonarjs/prefer-regexp-exec": "warn",
       "sonarjs/deprecation": "warn",
+
+      // eslint-comments: require justification for disable comments.
+      "@eslint-community/eslint-comments/no-unused-disable": "error",
+    },
+  },
+  // @vitest/eslint-plugin — cherry-pick high-value rules. Skip
+  // no-conditional-expect/no-standalone-expect — our property-based tests
+  // legitimately use both patterns.
+  {
+    files: ["test/**/*.test.ts", "examples/**/*.test.ts"],
+    plugins: { vitest },
+    rules: {
+      "vitest/no-focused-tests": "error",
+      "vitest/no-identical-title": "error",
+      "vitest/expect-expect": "warn",
+      "vitest/no-disabled-tests": "warn",
+      "vitest/prefer-to-be": "warn",
+      "vitest/valid-title": "warn",
     },
   },
   {
@@ -85,8 +150,16 @@ export default tseslint.config(
       "@typescript-eslint/no-explicit-any": "off",
       "@typescript-eslint/restrict-template-expressions": "off",
       "no-console": "off",
+      // vitest inline snapshots escape `$` inside backticks — we accept it.
+      "no-useless-escape": "off",
       "sonarjs/no-duplicate-string": "off",
       "sonarjs/cognitive-complexity": "off",
+      // Test fixtures use http URLs and small inline sorts; the rules are
+      // for production code, not test data.
+      "sonarjs/no-clear-text-protocols": "off",
+      "sonarjs/no-alphabetical-sort": "off",
+      "sonarjs/no-identical-functions": "off",
+      "unicorn/import-style": "off",
     },
   },
 );

examples/banking-plantuml/rules.test.ts

@@ -36,13 +36,17 @@ describe("Rules demo on C4L2.puml", () => {
 
   it("API Gateway — external calls go through gateway", () => {
     const violations = checkApiGateway(containers);
+    // Informational: list violations for inspection. Banking fixture
+    // intentionally has gateway gaps to demo the rule's output.
+    expect(violations).toBeDefined();
     for (const v of violations) {
       console.log(`${v.container}: ${v.message}`);
     }
   });
 
   it("Stable Dependencies — dependencies point toward stability", () => {
     const violations = checkStableDependencies(containers);
+    expect(violations).toBeDefined();
     for (const v of violations) {
       console.log(`${v.container}: ${v.message}`);
     }

knip.config.ts

@@ -0,0 +1,22 @@
+import type { KnipConfig } from "knip";
+
+// Knip — finds unused exports/files/deps. Run via `pnpm knip`.
+// Most config files are auto-detected; only non-default entries listed here.
+export default <KnipConfig>{
+  entry: [
+    "test/**/*.test.ts",
+    "examples/**/*.test.ts",
+    "examples/**/aact.config.ts",
+    "vitest.mutation.config.ts",
+  ],
+  project: ["src/**/*.ts", "test/**/*.ts", "examples/**/*.ts"],
+  ignoreDependencies: [
+    // Prettier plugin auto-loaded by prettier from name pattern; not imported.
+    "prettier-plugin-packagejson",
+  ],
+  // stryker.config.mjs imports types from `@stryker-mutator/api/core` which
+  // is transitive via @stryker-mutator/core — knip flags as "unlisted".
+  // Ignoring the config from knip's analysis entirely is simpler than
+  // installing an extra devDep just for types.
+  ignore: ["stryker.config.mjs"],
+};

package.json

@@ -47,6 +47,15 @@
   "scripts": {
     "build": "unbuild",
     "test": "vitest run",
+    "test:unit": "vitest run --project unit",
+    "test:integration": "vitest run --project integration",
+    "test:e2e": "vitest run --project e2e",
+    "test:coverage": "vitest run --coverage",
+    "test:mutation": "stryker run",
+    "changelog": "changelogen",
+    "release": "changelogen --release --push",
+    "knip": "knip",
+    "publint": "publint",
     "lint": "npm run lint:eslint && npm run lint:prettier",
     "lint:eslint": "eslint .",
     "lint:prettier": "prettier --ignore-path ./.gitignore --check \"./**/*.{ts,js,json,yaml,yml,md}\"",
@@ -63,34 +72,46 @@
   "devDependencies": {
     "@commitlint/cli": "20.4.1",
     "@commitlint/config-conventional": "20.4.1",
+    "@eslint-community/eslint-plugin-eslint-comments": "^4.7.1",
     "@eslint/js": "^9",
-    "@types/node": "^20.19.32",
+    "@fast-check/vitest": "^0.4.1",
+    "@stryker-mutator/core": "^9.6.1",
+    "@stryker-mutator/vitest-runner": "^9.6.1",
+    "@types/node": "^22.10.0",
+    "@vitest/coverage-v8": "^4.1.6",
+    "@vitest/eslint-plugin": "^1.6.17",
+    "changelogen": "^0.6.2",
     "eslint": "^9",
+    "eslint-plugin-citty": "^1.0.2",
+    "eslint-plugin-import-x": "^4.16.2",
     "eslint-plugin-n": "^17",
     "eslint-plugin-simple-import-sort": "^12",
     "eslint-plugin-sonarjs": "^3",
     "eslint-plugin-unicorn": "^62.0.0",
-    "globals": "^17.3.0",
+    "execa": "^9.6.1",
+    "globals": "^17.6.0",
     "husky": "9.1.7",
+    "knip": "^6.12.2",
     "lint-staged": "16.2.7",
-    "prettier": "3.8.1",
-    "prettier-plugin-packagejson": "3.0.0",
+    "prettier": "3.8.3",
+    "prettier-plugin-packagejson": "3.0.2",
+    "publint": "^0.3.20",
     "typescript": "5.9.3",
-    "typescript-eslint": "^8",
+    "typescript-eslint": "^8.59.3",
     "unbuild": "^3.0.0",
-    "vitest": "^4.0.18"
+    "vitest": "^4.1.6"
   },
   "engines": {
-    "node": ">=20"
+    "node": ">=22"
   },
   "dependencies": {
     "c12": "4.0.0-beta.2",
-    "citty": "^0.2.0",
+    "citty": "^0.2.2",
     "consola": "^3.4.2",
-    "jiti": "^2.6.1",
-    "picocolors": "^1.1.1",
+    "jiti": "^2.7.0",
+    "pathe": "^2.0.3",
     "plantuml-parser": "0.4.0",
-    "valibot": "^1.2.0",
-    "yaml": "2.8.2"
+    "valibot": "^1.4.0",
+    "yaml": "2.9.0"
   }
 }