code-ai-installer 3.0.0 → 4.0.0

package/domains/development/.agents/skills/testing-strategy-js-reference/agents/kimi.yaml

@@ -0,0 +1,14 @@
+name: "testing-strategy-js-reference"
+display_name: "Стратегия тестирования JS/TS"
+description: "Канонический справочник кода тестов JS/TS — детальные DO/DON'T примеры по 7 темам — Unit тесты (AAA, edge cases, hooks), React Testing Library, Integration (supertest), Mocks и фабрики, naming и структура файлов, Coverage конфиг (Vitest), anti-patterns. Активируй при поиске конкретного кодового паттерна теста."
+default_prompt: "Используй $testing-strategy-js-reference, когда задача относится к навыку \"Справочник кода тестов JS/TS (DO/DON'T)\"."
+triggers:
+  - "testing-strategy-js-reference"
+  - "testing strategy js reference"
+  - "Стратегия тестирования JS/TS"
+  - "Стратегия тестирования для JS/TS проектов — unit (Vitest), integration (API/DB), e2e (Playwright)"
+capabilities:
+  - "testing"
+  - "reference"
+tools: []
+implicit_invocation: true

package/domains/development/.agents/skills/testing-strategy-js-reference/agents/openai.yaml

@@ -0,0 +1,8 @@
+interface:
+  display_name: "Стратегия тестирования JS/TS"
+  short_description: "Канонический справочник кода тестов JS/TS — детальные DO/DON'T примеры по 7 темам — Unit тесты (AAA, edge cases, hooks), React Testing Library, Integration (supertest), Mocks и фабрики, naming и структура файлов, Coverage конфиг (Vitest), anti-patterns. Активируй при поиске конкретного кодового паттерна теста."
+  default_prompt: "Используй $testing-strategy-js-reference, когда задача относится к навыку \"Справочник кода тестов JS/TS (DO/DON'T)\"."
+dependencies:
+  tools: []
+policy:
+  allow_implicit_invocation: true

package/domains/development/.agents/skills/testing-strategy-js-reference/agents/qwen.json

@@ -0,0 +1,18 @@
+{
+  "name": "testing-strategy-js-reference",
+  "display_name": "Стратегия тестирования JS/TS",
+  "description": "Канонический справочник кода тестов JS/TS — детальные DO/DON'T примеры по 7 темам — Unit тесты (AAA, edge cases, hooks), React Testing Library, Integration (supertest), Mocks и фабрики, naming и структура файлов, Coverage конфиг (Vitest), anti-patterns. Активируй при поиске конкретного кодового паттерна теста.",
+  "default_prompt": "Используй $testing-strategy-js-reference, когда задача относится к навыку \"Справочник кода тестов JS/TS (DO/DON'T)\".",
+  "triggers": [
+    "testing-strategy-js-reference",
+    "testing strategy js reference",
+    "Стратегия тестирования JS/TS",
+    "Стратегия тестирования для JS/TS проектов — unit (Vitest), integration (API/DB), e2e (Playwright)"
+  ],
+  "capabilities": [
+    "testing",
+    "reference"
+  ],
+  "tools": [],
+  "implicit_invocation": true
+}

package/domains/development/.agents/skills/testing-strategy-js-reference/agents/skill.yaml

@@ -0,0 +1,22 @@
+version: 1
+name: "testing-strategy-js-reference"
+display_name: "Стратегия тестирования JS/TS"
+description: "Канонический справочник кода тестов JS/TS — детальные DO/DON'T примеры по 7 темам — Unit тесты (AAA, edge cases, hooks), React Testing Library, Integration (supertest), Mocks и фабрики, naming и структура файлов, Coverage конфиг (Vitest), anti-patterns. Активируй при поиске конкретного кодового паттерна теста."
+default_prompt: "Используй $testing-strategy-js-reference, когда задача относится к навыку \"Справочник кода тестов JS/TS (DO/DON'T)\"."
+triggers:
+  - "testing-strategy-js-reference"
+  - "testing strategy js reference"
+  - "Стратегия тестирования JS/TS"
+  - "Стратегия тестирования для JS/TS проектов — unit (Vitest), integration (API/DB), e2e (Playwright)"
+capabilities:
+  - "testing"
+  - "reference"
+tools: []
+invocation:
+  explicit: true
+  implicit: true
+localization:
+  default_locale: "ru"
+  available_locales:
+    - "ru"
+    - "en"

package/domains/development/.agents/skills/tests-integrity-rules/SKILL.md

@@ -0,0 +1,256 @@
+---
+name: tests-integrity-rules
+description: "SFS-side декларативные правила для написания тестов без gaming patterns — 9 forbidden patterns с BAD/DO примерами и conditions exception, commit annotations (TEST-CHANGED-WHY / DELETED-WHY / MOCK-INCREASE-WHY) для test changes, 3 минимум-viable property-based invariants, minimal pre-push lint snippet. Парный к $qa-test-integrity-audit который детектирует нарушения на TEST gate. Активируй перед написанием новых тестов, перед GREEN commit с test diff, перед push."
+type: discretionary
+domain: development
+owners:
+  - senior_full_stack
+gates:
+  - DEV
+tech:
+  - eslint
+  - ruff
+  - fast-check
+  - hypothesis
+topic:
+  - general
+  - testing
+triggers:
+  - tests integrity rules
+  - forbidden test patterns
+  - TEST-CHANGED-WHY
+  - DELETED-WHY
+  - MOCK-INCREASE-WHY
+  - property-based invariants
+  - правила написания тестов
+  - запрещённые паттерны тестов
+related:
+  - testing-strategy-js
+  - testing-strategy-js-reference
+  - tdd-workflow
+  - tdd-workflow-reference
+  - qa-test-integrity-audit
+  - qa-property-based-testing
+  - qa-mutation-testing
+  - tests-quality-review
+  - test-integrity-hooks-setup
+budget_lines: 320
+schema_version: 1
+---
+
+
+# Skill: Tests Integrity Rules (SFS)
+
+SFS-side канонические правила для написания тестов без gaming patterns. Парный к `$qa-test-integrity-audit` который детектирует нарушения на TEST gate. Этот скилл — что писать; audit — что ловится.
+
+**Разделы:**
+1. [Когда применять](#1-когда)
+2. [9 forbidden patterns как правила](#2-forbidden)
+3. [Commit annotations для test changes](#3-commit)
+4. [Property-based invariants (минимум viable)](#4-property)
+5. [Minimal pre-push lint](#5-prepush)
+6. [Anti-patterns SFS-стороны](#6-anti)
+7. [DoD](#7-dod)
+
+---
+
+## 1. Когда применять
+
+- **Перед написанием новых тестов** — особенно для tier 1-2 модулей (auth/billing/payments/security/crypto). Прочитай §2 forbidden patterns как чек-лист "чего НЕ делать".
+- **Во время GREEN commit** — §3 commit annotations: если меняешь тест, нужен `TEST-CHANGED-WHY`. Если удаляешь — `DELETED-WHY`. Если добавил >2 моков в PR — `MOCK-INCREASE-WHY`.
+- **При написании property-based тестов** — §4 даёт 3 минимум-viable examples; полный методологический набор → `$qa-property-based-testing`.
+- **Перед push** — §5 для быстрого локального прогона lint hook.
+
+---
+
+## 2. 9 forbidden patterns как правила
+
+Каждое правило формулируется как **запрет** + DO-замена + когда **exception допустим**. Detection logic (AST queries, ESLint configs) — в `$qa-test-integrity-audit`.
+
+### Rule 1: Нет одиночных `expect.anything()` / `expect.any(X)`
+- ❌ `expect(result).toEqual(expect.anything())`
+- ✅ `expect(result).toEqual({ status: "ok", id: expect.any(String) })`
+- **Exception:** `expect.any(Date)` для timestamps когда время не критично — добавь `// audit:any-date OK <reason>`.
+- Детект: `$qa-test-integrity-audit` pattern #1.
+
+### Rule 2: Нет `toBeDefined()` / `toBeTruthy()` без content
+- ❌ `assert user is not None` (и только)
+- ✅ `assert user is not None` + `assert user.email == "x@y"` + `assert user.role == "admin"`
+- **Exception:** existence test перед последующим fetch — `// audit:existence-only`.
+- Детект: pattern #2.
+
+### Rule 3: Snapshot тест должен иметь semantic assertion
+- ❌ `it("renders", () => expect(render(Foo)).toMatchSnapshot())`
+- ✅ Snapshot + минимум `toContain("Submit")` / `toHaveTextContent(...)` / `expect.objectContaining({...})`
+- **Exception:** pure visual regression test — суффикс `_snapshot.test.ts` отдельная категория.
+- Детект: pattern #3.
+
+### Rule 4: Нет `.skip` / `.only` в commit
+- ❌ `it.skip(...)`, `describe.only(...)`
+- ✅ Удали блок или заведи TODO + linked issue
+- **Exception:** временный quarantine flake — `// quarantine: <issue-link> deadline 2026-XX-XX`.
+- Детект: pattern #4. ESLint `vitest/no-disabled-tests` + `vitest/no-focused-tests` блокирует pre-commit.
+
+### Rule 5: `try/catch` не должен глотать assertion
+- ❌ `try: assert user.balance > 0 except: pass`
+- ✅ `with pytest.raises(SomeError): ...` или `expect(() => ...).toThrow(...)`
+- Детект: pattern #5 (custom AST rule).
+
+### Rule 6: Удалил тест — нужен `DELETED-WHY` в commit
+```
+test: remove obsolete X test
+DELETED-WHY: replaced by Y integration test; behavior fully covered there.
+```
+- **Exception:** нет — без `DELETED-WHY` PR блокируется gh check'ом.
+- Детект: pattern #6 + diff parser.
+
+### Rule 7: Mock count +>2 — нужен `MOCK-INCREASE-WHY`
+```
+MOCK-INCREASE-WHY: external API X недоступен в CI без prod credentials;
+mocked at boundary per Boundary Mocking Rule ($testing-strategy-js §5).
+```
+- Детект: pattern #7 (diff analysis).
+
+### Rule 8: Каждый тест должен иметь `expect()` / `assert`
+- ❌ Test body без assertion (вечно зелёный)
+- ✅ Каждый `it` / `def test_` минимум один `expect()` / `assert`
+- Детект: pattern #8. ESLint `vitest/expect-expect`.
+
+### Rule 9: Property test не должен быть tautology
+- ❌ `fc.property(fc.integer(), x => x === x)` — всегда true, ничего не тестирует
+- ✅ `fc.property(fc.integer(), x => Math.abs(x) >= 0)` — реальный invariant
+- Детект: pattern #9. См. `$qa-property-based-testing §4` за библиотекой valid invariants.
+
+---
+
+## 3. Commit annotations для test changes
+
+Формализованные templates для commit messages — feeds downstream checks ($qa-test-integrity-audit + DEMO envelope в $tester).
+
+### TEST-CHANGED-WHY (в GREEN commit)
+
+Если изменяешь существующий тест в GREEN commit:
+```
+feat: add validation for X
+TEST-CHANGED-WHY: existing test expected old error code 422, new code returns
+400 per RFC X — adjusted assertion. Behavior is more correct, not loosened.
+TEST-BEHAVIOR-PRESERVED: yes — new assertion is stricter (400 covers also Y case).
+```
+**Без `TEST-CHANGED-WHY`** в GREEN commit с test diff — self-confirming паттерн (AI-gaming). `$qa-test-integrity-audit` это ловит.
+
+### DELETED-WHY (в commit удаляющем test)
+```
+test: remove redundant Y test
+DELETED-WHY: behavior fully covered by integration test in suite Z. Removing
+reduces maintenance and runtime by 1.2s.
+```
+
+### MOCK-INCREASE-WHY (при PR adding >2 моков)
+```
+feat: add subscription tier handling
+MOCK-INCREASE-WHY: 3 new external SDKs (Stripe Customer, Stripe Subscription,
+SendGrid mail) mocked at boundary — production endpoints untouchable in CI
+without prod credentials. Each follows Boundary Mocking Rule ($testing-strategy-js §5).
+```
+
+### RED_COMMIT_HASH / GREEN_COMMIT_HASH (DEMO envelope для tier 1-2)
+
+См. `$tdd-workflow §1 Commit discipline` — TDD фазы это отдельные коммиты для tier 1-2, hashes уходят в DEMO envelope для TEST gate verification.
+
+---
+
+## 4. Property-based invariants (минимум viable)
+
+Полный методологический набор (6 patterns + shrinking + counter-example) → `$qa-property-based-testing §4`. Здесь — 3 минимум-viable шаблона как точка входа.
+
+### Pattern A: Roundtrip (encode → decode = identity)
+```ts
+import fc from "fast-check";
+fc.assert(fc.property(fc.string(), (s) => {
+  expect(decode(encode(s))).toBe(s);
+}));
+```
+```python
+from hypothesis import given, strategies as st
+@given(st.text())
+def test_encode_decode_roundtrip(s):
+    assert decode(encode(s)) == s
+```
+
+### Pattern B: Idempotence (f(f(x)) = f(x))
+```ts
+fc.assert(fc.property(fc.string(), (s) => {
+  expect(normalize(normalize(s))).toBe(normalize(s));
+}));
+```
+
+### Pattern C: Postcondition invariant (длина / сортировка)
+```ts
+fc.assert(fc.property(fc.array(fc.integer()), (arr) => {
+  const sorted = sort(arr);
+  expect(sorted.length).toBe(arr.length);
+  for (let i = 1; i < sorted.length; i++) {
+    expect(sorted[i] >= sorted[i-1]).toBe(true);
+  }
+}));
+```
+
+**Когда писать property test:** validators, parsers, serializers, normalizers, dedupers, math/business rules. UI rendering — НЕ подходит (см. `$qa-property-based-testing §1` decision tree).
+
+---
+
+## 5. Minimal pre-push lint
+
+Быстрый локальный прогон ДО push (≤5 сек) для catching очевидных нарушений на dev workstation. Не production CI — full setup в `$test-integrity-hooks-setup`.
+
+```bash
+# package.json scripts (JS/TS)
+"lint:test-integrity:fast": "eslint 'src/**/*.{test,spec}.{ts,tsx}' --rule 'vitest/no-disabled-tests:error' --rule 'vitest/no-focused-tests:error' --rule 'vitest/expect-expect:error'"
+```
+
+```bash
+# Python
+ruff check tests/ --select PT,S101 --extend-select PT001,PT003,PT006
+```
+
+Запуск перед push:
+```bash
+npm run lint:test-integrity:fast && git push
+```
+
+Production-grade hooks (pre-commit + gh check + report aggregation + exemption registry) — `$test-integrity-hooks-setup` (DEV-094 roadmap).
+
+---
+
+## 6. Anti-patterns SFS-стороны
+
+- ❌ **`eslint-disable` без justification** — `// eslint-disable-next-line vitest/no-disabled-tests` без объяснения почему. Always: `// eslint-disable-next-line ... -- quarantine for issue #N, deadline 2026-XX-XX`.
+- ❌ **Tautological property** — `fc.property(_, x => x === x)`. Property должна тестировать **реальный** invariant. См. `$qa-property-based-testing §4`.
+- ❌ **`TEST-CHANGED-WHY: refactor only`** без диффа — тест поменялся существенно, "refactor only" неверно. Опиши реальную причину поведенческого изменения.
+- ❌ **Mock-count inflation через split commits** — добавил 2 моков в commit A, ещё 2 в commit B чтобы каждый раз не триггерить `MOCK-INCREASE-WHY`. PR aggregate audit это ловит.
+
+---
+
+## 7. DoD
+
+- [ ] Все тесты в PR проходят §2 9 forbidden patterns (или имеют justified exception comment)
+- [ ] Если test files изменены в GREEN commit — `TEST-CHANGED-WHY` + `TEST-BEHAVIOR-PRESERVED` в commit message
+- [ ] Если тест удалён — `DELETED-WHY` в commit message
+- [ ] Если mock count увеличен >2 — `MOCK-INCREASE-WHY` в commit message
+- [ ] Property-based тесты добавлены для validators/parsers/serializers/normalizers (§4 применим)
+- [ ] `npm run lint:test-integrity:fast` зелёный локально перед push
+- [ ] Для tier 1-2 модулей — `RED_COMMIT_HASH` + `GREEN_COMMIT_HASH` в DEMO envelope (см. `$tdd-workflow §1`)
+
+---
+
+## См. также
+
+- `$testing-strategy-js` — общая стратегия тестов (Boundary Mocking Rule §5, mock-vs-real decision tree)
+- `$testing-strategy-js-reference` — справочник кода тестов
+- `$tdd-workflow` — TDD discipline + Commit discipline (RED/GREEN/REFACTOR commits для tier 1-2)
+- `$tdd-workflow-reference` — справочник кода TDD
+- `$qa-test-integrity-audit` — Tester-side detector (детектирует нарушения §2 правил)
+- `$qa-property-based-testing` — полный набор 6 invariant patterns + shrinking + counter-example
+- `$qa-mutation-testing` — tier model (определяет когда правила strict)
+- `$tests-quality-review` — Reviewer-side проверка тестов
+- `$test-integrity-hooks-setup` — production pre-commit + gh check + report aggregator (DEV-094 roadmap)

package/domains/development/.agents/skills/tests-integrity-rules/agents/claude.json

@@ -0,0 +1,19 @@
+{
+  "name": "tests-integrity-rules",
+  "display_name": "Tests Integrity Rules (SFS)",
+  "description": "SFS-side декларативные правила для написания тестов без gaming patterns — 9 forbidden patterns с BAD/DO примерами и conditions exception, commit annotations (TEST-CHANGED-WHY / DELETED-WHY / MOCK-INCREASE-WHY) для test changes, 3 минимум-viable property-based invariants, minimal pre-push lint snippet. Парный к $qa-test-integrity-audit который детектирует нарушения на TEST gate. Активируй перед написанием новых тестов, перед GREEN commit с test diff, перед push.",
+  "default_prompt": "Используй $tests-integrity-rules, когда задача относится к навыку \"Tests Integrity Rules (SFS)\".",
+  "triggers": [
+    "tests-integrity-rules",
+    "tests integrity rules",
+    "Tests Integrity Rules (SFS)"
+  ],
+  "capabilities": [
+    "testing",
+    "integrity",
+    "rules",
+    "sfs"
+  ],
+  "tools": [],
+  "implicit_invocation": true
+}

package/domains/development/.agents/skills/tests-integrity-rules/agents/copilot.json

@@ -0,0 +1,19 @@
+{
+  "name": "tests-integrity-rules",
+  "display_name": "Tests Integrity Rules (SFS)",
+  "description": "SFS-side декларативные правила для написания тестов без gaming patterns — 9 forbidden patterns с BAD/DO примерами и conditions exception, commit annotations (TEST-CHANGED-WHY / DELETED-WHY / MOCK-INCREASE-WHY) для test changes, 3 минимум-viable property-based invariants, minimal pre-push lint snippet. Парный к $qa-test-integrity-audit который детектирует нарушения на TEST gate. Активируй перед написанием новых тестов, перед GREEN commit с test diff, перед push.",
+  "default_prompt": "Используй $tests-integrity-rules, когда задача относится к навыку \"Tests Integrity Rules (SFS)\".",
+  "triggers": [
+    "tests-integrity-rules",
+    "tests integrity rules",
+    "Tests Integrity Rules (SFS)"
+  ],
+  "capabilities": [
+    "testing",
+    "integrity",
+    "rules",
+    "sfs"
+  ],
+  "tools": [],
+  "implicit_invocation": true
+}

package/domains/development/.agents/skills/tests-integrity-rules/agents/gemini.json

@@ -0,0 +1,19 @@
+{
+  "name": "tests-integrity-rules",
+  "display_name": "Tests Integrity Rules (SFS)",
+  "description": "SFS-side декларативные правила для написания тестов без gaming patterns — 9 forbidden patterns с BAD/DO примерами и conditions exception, commit annotations (TEST-CHANGED-WHY / DELETED-WHY / MOCK-INCREASE-WHY) для test changes, 3 минимум-viable property-based invariants, minimal pre-push lint snippet. Парный к $qa-test-integrity-audit который детектирует нарушения на TEST gate. Активируй перед написанием новых тестов, перед GREEN commit с test diff, перед push.",
+  "default_prompt": "Используй $tests-integrity-rules, когда задача относится к навыку \"Tests Integrity Rules (SFS)\".",
+  "triggers": [
+    "tests-integrity-rules",
+    "tests integrity rules",
+    "Tests Integrity Rules (SFS)"
+  ],
+  "capabilities": [
+    "testing",
+    "integrity",
+    "rules",
+    "sfs"
+  ],
+  "tools": [],
+  "implicit_invocation": true
+}

package/domains/development/.agents/skills/tests-integrity-rules/agents/kimi.yaml

@@ -0,0 +1,15 @@
+name: "tests-integrity-rules"
+display_name: "Tests Integrity Rules (SFS)"
+description: "SFS-side декларативные правила для написания тестов без gaming patterns — 9 forbidden patterns с BAD/DO примерами и conditions exception, commit annotations (TEST-CHANGED-WHY / DELETED-WHY / MOCK-INCREASE-WHY) для test changes, 3 минимум-viable property-based invariants, minimal pre-push lint snippet. Парный к $qa-test-integrity-audit который детектирует нарушения на TEST gate. Активируй перед написанием новых тестов, перед GREEN commit с test diff, перед push."
+default_prompt: "Используй $tests-integrity-rules, когда задача относится к навыку \"Tests Integrity Rules (SFS)\"."
+triggers:
+  - "tests-integrity-rules"
+  - "tests integrity rules"
+  - "Tests Integrity Rules (SFS)"
+capabilities:
+  - "testing"
+  - "integrity"
+  - "rules"
+  - "sfs"
+tools: []
+implicit_invocation: true

package/domains/development/.agents/skills/tests-integrity-rules/agents/openai.yaml

@@ -0,0 +1,8 @@
+interface:
+  display_name: "Tests Integrity Rules (SFS)"
+  short_description: "SFS-side декларативные правила для написания тестов без gaming patterns — 9 forbidden patterns с BAD/DO примерами и..."
+  default_prompt: "Используй $tests-integrity-rules, когда задача относится к навыку \"Tests Integrity Rules (SFS)\"."
+dependencies:
+  tools: []
+policy:
+  allow_implicit_invocation: true

package/domains/development/.agents/skills/tests-integrity-rules/agents/qwen.json

@@ -0,0 +1,19 @@
+{
+  "name": "tests-integrity-rules",
+  "display_name": "Tests Integrity Rules (SFS)",
+  "description": "SFS-side декларативные правила для написания тестов без gaming patterns — 9 forbidden patterns с BAD/DO примерами и conditions exception, commit annotations (TEST-CHANGED-WHY / DELETED-WHY / MOCK-INCREASE-WHY) для test changes, 3 минимум-viable property-based invariants, minimal pre-push lint snippet. Парный к $qa-test-integrity-audit который детектирует нарушения на TEST gate. Активируй перед написанием новых тестов, перед GREEN commit с test diff, перед push.",
+  "default_prompt": "Используй $tests-integrity-rules, когда задача относится к навыку \"Tests Integrity Rules (SFS)\".",
+  "triggers": [
+    "tests-integrity-rules",
+    "tests integrity rules",
+    "Tests Integrity Rules (SFS)"
+  ],
+  "capabilities": [
+    "testing",
+    "integrity",
+    "rules",
+    "sfs"
+  ],
+  "tools": [],
+  "implicit_invocation": true
+}

package/domains/development/.agents/skills/tests-integrity-rules/agents/skill.yaml

@@ -0,0 +1,23 @@
+version: 1
+name: "tests-integrity-rules"
+display_name: "Tests Integrity Rules (SFS)"
+description: "SFS-side декларативные правила для написания тестов без gaming patterns — 9 forbidden patterns с BAD/DO примерами и conditions exception, commit annotations (TEST-CHANGED-WHY / DELETED-WHY / MOCK-INCREASE-WHY) для test changes, 3 минимум-viable property-based invariants, minimal pre-push lint snippet. Парный к $qa-test-integrity-audit который детектирует нарушения на TEST gate. Активируй перед написанием новых тестов, перед GREEN commit с test diff, перед push."
+default_prompt: "Используй $tests-integrity-rules, когда задача относится к навыку \"Tests Integrity Rules (SFS)\"."
+triggers:
+  - "tests-integrity-rules"
+  - "tests integrity rules"
+  - "Tests Integrity Rules (SFS)"
+capabilities:
+  - "testing"
+  - "integrity"
+  - "rules"
+  - "sfs"
+tools: []
+invocation:
+  explicit: true
+  implicit: true
+localization:
+  default_locale: "ru"
+  available_locales:
+    - "ru"
+    - "en"

package/domains/development/.agents/skills/tests-quality-review/SKILL.md

@@ -1,8 +1,42 @@
 ---
 name: tests-quality-review
 description: Ревью качества unit/integration тестов — покрытие сценариев, границы, устойчивость, отсутствие флейков и тестовой "магии". Чеклист с приоритетами, примеры хорошего/плохого теста, шаблон выходного отчёта. Активируй при ревью существующих тестов, при стабилизации flaky тестов, или при аудите test suite quality.
+type: triggered
+domain: development
+owners:
+  - reviewer
+  - senior_full_stack
+gates:
+  - REV
+  - DEV
+tech: []
+topic:
+  - general
+  - review
+  - security
+triggers:
+  - ревью тестов
+  - качество теста
+  - флейки тесты
+  - test suite аудит
+  - тестовая магия
+  - стабилизировать тест
+related:
+  - testing-strategy-js
+  - testing-strategy-js-reference
+  - tdd-workflow
+  - tests-integrity-rules
+  - code-review-checklist
+  - qa-e2e-playwright
+  - qa-mutation-testing
+  - qa-property-based-testing
+  - qa-test-integrity-audit
+  - dev-reference-snippets
+budget_lines: 400
+schema_version: 1
 ---
 
+
 # Skill: Tests Quality Review
 
 Структурированное ревью качества тестов: чеклист, приоритеты, примеры плохого → хорошего, шаблон отчёта.
@@ -16,16 +50,17 @@ description: Ревью качества unit/integration тестов — по
 
 ---
 
-## 1. Когда активировать
+## 1. Когда активировать <a id="1-когда-активировать"></a>
 
 - REV gate — ревью тестов как часть code review
 - После TDD-итерации — проверить качество написанных тестов
 - При стабилизации flaky-CI — найти и починить хрупкие тесты
 - При аудите coverage — не гнаться за %, а проверить *что* покрыто
+- Когда PR содержит test diff с новыми моками или test modifications — обязательно прогнать §2 F + G
 
 ---
 
-## 2. Чеклист ревью
+## 2. Чеклист ревью <a id="2-чеклист-ревью"></a>
 
 ### A. Покрытие сценариев (P0)
 
@@ -74,9 +109,34 @@ description: Ревью качества unit/integration тестов — по
 | E2 | Snapshot **компактный** (не 500 строк) — inline preferred | ✅/❌ |
 | E3 | Обновление snapshot не «авто-accept» (diff ревьюится) | ✅/❌ |
 
+### F. AI-gaming detection (P1)
+
+REV-side **контекстное суждение** для того что static/dynamic scanner'ы не ловят. Cross-check: то что `$qa-test-integrity-audit` уже поймал автоматом (9 patterns) — skip ручной проверки.
+
+| # | Проверка | Вердикт |
+|---|---------|---------|
+| F1 | Mock не воспроизводит логику которую тестирует (mock-as-production-double) | ✅/❌ |
+| F2 | Mock-to-real ratio в норме (compare с repo baseline; флаг при >3× от среднего) | ✅/❌ |
+| F3 | Snapshot тесты имеют semantic assertion (не только `toMatchSnapshot()`) | ✅/❌ |
+| F4 | Property tests тестируют реальные invariants (не tautology типа `x === x`) | ✅/❌ |
+| F5 | `eslint-disable` имеет justification comment + deadline (если quarantine) | ✅/❌ |
+
+### G. Test-modification audit (P0)
+
+PR diff verification для test files. SFS-side правила в `$tests-integrity-rules` (что должен был соблюдать SFS).
+
+| # | Проверка | Вердикт |
+|---|---------|---------|
+| G1 | Каждый изменённый test file имеет `TEST-CHANGED-WHY` + `TEST-BEHAVIOR-PRESERVED` в commit message | ✅/❌ |
+| G2 | Rationale в `TEST-CHANGED-WHY` соответствует фактическому diff'у (не "refactor only" когда поменялось assertion) | ✅/❌ |
+| G3 | Изменения сделали тест **строже** или **слабее** (если слабее — есть обоснование в rationale) | ✅/❌ |
+| G4 | Удалён тест — `DELETED-WHY` присутствует + reason verifiable (проверь упомянутое покрытие) | ✅/❌ |
+| G5 | +>2 моков в PR — `MOCK-INCREASE-WHY` есть + boundary mock rule соблюдено (`$testing-strategy-js §5`) | ✅/❌ |
+| G6 | Для tier 1-2 модулей — `RED_COMMIT_HASH` + `GREEN_COMMIT_HASH` в DEMO envelope (см. `$tdd-workflow §1`) | ✅/❌ |
+
 ---
 
-## 3. Примеры: плохо → хорошо
+## 3. Примеры: плохо → хорошо <a id="3-примеры-плохо--хорошо"></a>
 
 ### A1. Поведение vs реализация
 
@@ -195,7 +255,7 @@ it('should process order and save to DB', async () => {
 
 ---
 
-## 4. Flaky-тесты: диагностика
+## 4. Flaky-тесты: диагностика <a id="4-flaky-тесты-диагностика"></a>
 
 ### Частые причины
 
@@ -221,7 +281,7 @@ npx vitest run --repeat=10
 
 ---
 
-## 5. Шаблон выходного отчёта
+## 5. Шаблон выходного отчёта <a id="5-шаблон-выходного-отчёта"></a>
 
 ```markdown
 # Tests Quality Review Report
@@ -249,6 +309,21 @@ npx vitest run --repeat=10
 |---|----------------|---------|-------------|
 | 1 | E2 (snapshot size) | Snapshot на 200 строк в ProductForm.test | Перейти на inline snapshot или RTL assertion |
 
+### F — AI-gaming detections
+| # | Пункт чеклиста | Проблема | Рекомендация |
+|---|----------------|---------|-------------|
+| 1 | F1 (mock-as-production-double) | PaymentMock воспроизводит логику validator'а | Заменить на реальный validator или поднять тест на integration уровень |
+
+### G — Test-modification audit findings
+| # | Пункт чеклиста | Проблема | Рекомендация |
+|---|----------------|---------|-------------|
+| 1 | G2 (rationale mismatch) | `TEST-CHANGED-WHY: refactor only` для diff'a с поменявшимся assertion 422 → 400 | Запросить точный rationale; если behavior changed — отдельный TDD цикл |
+
+### Mock-to-real ratio
+- **Repo baseline:** X mocks per test file (avg over last 50 PRs)
+- **This PR:** Y mocks per test file
+- **Verdict:** within / above (>3× от baseline → flag F2)
+
 ## Missing Test Cases
 - [ ] `fetchItems` — error path (network failure)
 - [ ] `updateRecord` — edge case (empty name)
@@ -259,21 +334,22 @@ npx vitest run --repeat=10
 |------|-------------------|-------------|
 | `pricing.test.js:42` | Зависимость от Date.now | vi.useFakeTimers |
 ```
-
----
-
 ## Критерии вердикта
 
 | Вердикт | Условия |
 |---------|---------|
-| **PASS ✅** | Все P0 ✅, P1 ≤ 2 findings, нет flaky |
-| **PASS with findings ⚠️** | Все P0 ✅, но есть P1 findings или missing cases | 
-| **FAIL ❌** | Любой P0 ❌ (нет error paths, тесты зависимы, flaky) |
+| **PASS ✅** | Все P0 ✅, P1 ≤ 2 findings, нет flaky, G findings = 0 |
+| **PASS with findings ⚠️** | Все P0 ✅, но есть P1 findings или missing cases или F findings |
+| **FAIL ❌** | Любой P0 ❌ (нет error paths, тесты зависимы, flaky) ИЛИ любой G ❌ (отсутствие commit annotations) |
+
+**Severity mapping:** G findings count as P0 (mandatory commit annotations feed downstream checks). F findings count as P1 (contextual gaming detections — review-required но не block).
 
 ---
 
 ## См. также
 - `$testing-strategy-js` — стратегия и паттерны написания тестов
+- `$tdd-workflow` — TDD discipline + Commit discipline (RED/GREEN/REFACTOR commits для tier 1-2)
+- `$tests-integrity-rules` — SFS-side rules companion (что должен был соблюдать SFS перед PR)
 - `$code-review-checklist` — общий чеклист code review
 - `$qa-e2e-playwright` — E2E через Playwright
 - `$dev-reference-snippets` — примеры кода