@bonesofspring/ai-rules 0.2.8 → 0.2.10

package/presets/cursor/go/rules/preset-token-budget.mdc

@@ -22,7 +22,7 @@ alwaysApply: false
 
 **Claude Option A (README exclusion):** when counting Claude rules without `paths:` / session-start, **exclude `**/README.md`** (topic indexes are not session-start rules). The intended session-start trio remains **exactly 3**; body budget sums **only those 3** rules.
 
-Session-start body budget (где задокументировано): ≤120 строк суммарно (stretch ≤100).
+Session-start body budget (где задокументировано): ≤140 строк суммарно (stretch ≤120).
 
 ## Meta / author rules
 

package/presets/cursor/go/rules/security-go.mdc

@@ -7,17 +7,29 @@ globs:
 alwaysApply: false
 ---
 
-
 # Security (Go)
 
-- Secrets from env/secret manager — never commit keys; do not log tokens/passwords.
+- Secrets from env/secret manager — never commit keys; **never log** tokens, passwords, raw `Authorization` headers, or cookie values.
 - Validate and canonicalize untrusted input at the driving adapter edge.
 - Prefer prepared statements / parameterized queries in persistence adapters.
 - TLS and secure cookie/header defaults when exposing HTTP.
 - AuthN/AuthZ in middleware or driving adapters; pass identity into use cases as domain value (user ID / claims), not raw headers.
-- Minimize PII in logs; redact where needed.
+- Minimize PII in structured logs; redact known sensitive fields by default.
 
 ## Config
 
-- Distinguish public vs secret config fields in types.
-- Fail closed when auth is misconfigured in production builds.
+- Distinguish public vs secret config fields in types (separate structs or tagged fields).
+- **Fail closed** when auth is misconfigured in production builds (missing issuer/JWKS/secret → refuse to start or reject requests).
+- Sample configs and README snippets must use placeholders — never real-looking secrets.
+
+## Forbidden
+
+- Hardcoded API keys/tokens in source or committed `.env` examples with live values.
+- Logging request/response bodies that may contain credentials without an explicit redact step.
+- Fail-open auth middleware when required validators are nil/empty in prod.
+
+## Tests
+
+- Negative assertions: logs/metrics fixtures must not contain bearer tokens or secret config values.
+- Missing/invalid auth config fails closed in a construction or middleware test.
+- Agents: `security-reviewer` on auth/config/logging changes; then `post-change-test`.

package/presets/cursor/go/rules/tests-e2e.mdc

@@ -6,14 +6,16 @@ globs:
 alwaysApply: false
 ---
 
-
 # E2E / HTTP smoke
 
-- Black-box against a running server (httptest with full composition root, or localhost in CI).
-- Cover critical paths only; prefer integration + unit for breadth.
-- Stable test data; avoid depending on wall-clock flakiness.
-- Do not use browser/Playwright tooling in this preset.
+> **Documented stub / scoped smoke:** Prefer `tests-integration` + `integration-test-*` for breadth. Expand this rule only when the consumer repo keeps a dedicated black-box smoke suite.
+
+- Black-box against a running server (`httptest` with full composition root, or localhost in CI).
+- Cover critical paths only; prefer integration + unit for most coverage.
+- Stable test data; avoid wall-clock flakiness and shared mutable fixtures.
+- Do **not** use browser/Playwright tooling in this preset (`preset-no-cross-stack-leakage`).
 
 ## Placement
 
-- `e2e/` package or `*_e2e_test.go` with build tags if needed.
+- `e2e/` package or `*_e2e_test.go` with build tags if the repo already separates smoke CI.
+- If smoke overlaps repository contracts, put the contract under `tests-integration` instead.

package/presets/cursor/go/rules/tests-integration.mdc

@@ -6,19 +6,25 @@ globs:
 alwaysApply: false
 ---
 
-
 # Integration tests
 
 - Exercise driven adapters against real infra doubles (testcontainers, docker compose test DB, miniredis, etc.).
 - Or exercise HTTP adapter with `httptest` + wired use case + fake/real ports — still no full prod cluster required.
-- Build tags (`//go:build integration`) if the repo separates CI jobs.
-- Keep fixtures deterministic; clean up data between tests.
+- Build tags (`//go:build integration`) when the repo separates CI jobs — discover the existing CI entrypoint; do not invent a second runner.
+- Keep fixtures deterministic; clean up / isolate data between tests (truncate, transaction rollback, or unique prefixes).
 
 ## Scope
 
-- Contract of a port implementation (repo save/load roundtrip).
-- Not a substitute for domain unit tests.
+- **Contract of a port implementation** (repo save/load/error round-trip) — assert semantics at the port, not only SQL rows.
+- HTTP adapter contract with `httptest` is valid **adapter** integration — it is **not** a substitute for a repository contract test.
+- Not a substitute for domain unit tests (table-driven domain logic stays in `tests-unit`).
+
+## Forbidden shortcuts
+
+- Fake-only “integration” that never crosses the adapter↔infra boundary for the port under test.
+- Skipping isolation so tests pass only in isolation or only on a dirty DB.
 
 ## Agents
 
 - Planning/generation: `integration-test-{planner,generator,healer}` + skill `integration-testing`.
+- After green tests: `post-change-test` (+ `go-tooling`).

package/presets/cursor/ios-swift/agents/build-verifier.md

@@ -38,7 +38,7 @@ When validating **ios-swift preset** changes (or `scope: preset-structure-valida
 6. **Twin presence (hard):** Cursor rule change → Claude twin present **same PR** — **FAIL** if missing (next, ios-swift, go, and java).
 7. **Mapping / README:** `claude/ios-swift/rules/README.md` mapping table present; core→stack mapping documented when `_shared` consumers exist.
 8. **shared-core consumers:** if `_shared/core/**` changed, every matching `<!-- shared-core: … -->` consumer in this PR must be updated (or intentional fork documented).
-9. **Session-start body budget:** sum of the **3** intended always-on / session-start rule bodies ≤ **120** lines (stretch ≤100); do **not** include `**/README.md` — record in report.
+9. **Session-start body budget:** sum of the **3** intended always-on / session-start rule bodies ≤ **140** lines (stretch ≤120); do **not** include `**/README.md` — record in report.
 10. **Embed drift (hard in `check:preset-structure`):** `packages/ai-rules/scripts/check-shared-core-drift.sh` with `AI_RULES_DRIFT_FAIL=1`.
 11. **Dogfood (monorepo soft):** `packages/ai-rules/scripts/sync-dogfood-cursor.sh`; mass deletion without migration doc → risk / FAIL for maintainers.
 

package/presets/cursor/ios-swift/agents/ci-investigator.md

@@ -71,3 +71,7 @@ Update `status.json`:
 ```
 
 Provide summary: failed check, root cause, files changed, validation commands run, `resolved`.
+
+## Anti-sycophancy discipline
+
+- When writing or changing code, load / follow rule `anti-sycophancy-discipline`.

package/presets/cursor/ios-swift/agents/debugger.md

@@ -74,3 +74,7 @@ Do not perform formal code review or write XCUITest plans — those are separate
 ## Design guidance
 
 - Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.
+
+## Anti-sycophancy discipline
+
+- When writing or changing code, load / follow rule `anti-sycophancy-discipline`.

package/presets/cursor/ios-swift/agents/feature-developer.md

@@ -50,3 +50,7 @@ When copying from `next` into `ios-swift` (or another stack):
 ## Design guidance
 
 - Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.
+
+## Anti-sycophancy discipline
+
+- When writing or changing code, load / follow rule `anti-sycophancy-discipline`.

package/presets/cursor/ios-swift/rules/README.md

@@ -13,13 +13,14 @@
 
 ### Пресет ios-swift — карта правил
 
-**alwaysApply budget:** **≤3** always-on правила и **≤120** строк тела (после YAML) суммарно (stretch ≤100). Новые alwaysApply только вместе с сокращением существующего бюджета.
+**alwaysApply budget:** **≤3** always-on правила и **≤140** строк тела (после YAML) суммарно (stretch ≤120). Новые alwaysApply только вместе с сокращением существующего бюджета.
 
 | Файл | Скоуп | Тема |
 |------|--------|------|
 | `ios-app-core.mdc` | **always** | Стек, структура, tips (+ pointers на tooling/build) |
 | `architecture-boundaries.mdc` | **always** | Границы Presentation / Domain / Data, DI |
 | `code-quality-and-refactoring.mdc` | **always** | Качество и лёгкий refactor |
+| `anti-sycophancy-discipline.mdc` | requestable | Full VERIFY/anti-sycophancy discipline (~17 rules) |
 | `design-guidance.mdc` | requestable | Smells/GoF (Fowler≤12, GoF≤8) |
 | `xcode-tooling.mdc` | requestable | Схемы, xcodebuild, SwiftLint/Format |
 | `post-change-build.mdc` | requestable | Сборка/lint после Swift edits (**обязателен** после code edits) |
@@ -41,7 +42,7 @@
 | `tests-unit.mdc` | `*Tests` | Unit |
 | `tests-ui.mdc` | `*UITests` | XCUITest (+ pointer to xcuitest agents/skill) |
 
-**Всего topic `.mdc`:** 21 domain + 5 preset-meta (без README).
+**Всего topic `.mdc`:** 22 domain + 5 preset-meta (без README).
 
 **Twin depth (Cursor↔Claude):** domain rules ≥15 body lines. Thin requestable **aliases** (`agent-team-intake`, `technical-retro`) могут быть короче — как в next (soft AC-19).
 
@@ -52,6 +53,7 @@ SoT prose: `packages/ai-rules/presets/_shared/core/**`. Adapters: `<!-- shared-c
 | Shared core | This stack adapter | Delta |
 |-------------|--------------------|-------|
 | `quality/code-quality-and-refactoring.md` | `code-quality-and-refactoring.mdc` | Stack notes: force unwrap, DesignSystem |
+| `quality/anti-sycophancy-discipline.md` | `anti-sycophancy-discipline.mdc` | requestable full VERIFY/discipline set |
 | `quality/design-guidance.md` | `design-guidance.mdc` | requestable smells/GoF |
 | `agent-team/agent-team-orchestrator.md` | `agent-team-orchestrator.mdc` | Roles (XCUITest); preset PR pointer |
 | `agent-team/agent-team-intake.md` | `agent-team-intake.mdc` | thin + iOS scope note |

package/presets/cursor/ios-swift/rules/anti-sycophancy-discipline.mdc

@@ -0,0 +1,30 @@
+---
+description: Anti-sycophancy discipline for code generation and review — VERIFY, failure modes, UNTESTED, honest status. Load when writing or changing code.
+globs:
+  - "**/*.swift"
+alwaysApply: false
+---
+
+<!-- shared-core: quality/anti-sycophancy-discipline.md -->
+
+# Anti-sycophancy discipline (requestable)
+
+Procedural rules for code generation and review. Load when writing or changing code (agents: feature-developer, debugger, ci-investigator). Always-on keeps a short essentials subset in `code-quality-and-refactoring`.
+
+1. **Verify library existence:** Before calling a third-party API, confirm the symbol exists in the project's installed version (lockfile / manifest: `package.json`, `go.mod`, `Podfile.lock`, `pom.xml`, `Cargo.toml`, or equivalent). If you cannot verify, mark `// VERIFY: lib.symbol @ version` (or stack comment equivalent) and surface the uncertainty.
+2. **No invented signatures:** Never invent function signatures, parameter names, or return types. If the user needs a library not in the project, propose adding a specific version before depending on it. Silent stubs are worse than refusal.
+3. **Enumerate edge cases before validating:** When asked "is this correct?" or "does this work?", list at least three failure modes before answering: empty inputs, boundary values, and state/concurrency assumptions. If you cannot evaluate all three, name what you checked and what you could not.
+4. **Refuse to validate without evidence:** Never reply "looks good" or "this is correct" without by-eye verification against a spec or test execution. If no spec exists, ask for one or refuse to validate.
+5. **Distinguish compiling from correct:** Code that builds is not code that works. Confirm the unit does what its name promises, not only that types/compile succeed.
+6. **Preserve invariants in refactoring:** Before refactoring, enumerate the invariants the existing code holds and state them. After the refactor, verify each still holds.
+7. **Tests before refactor:** If no tests exist for code being refactored, propose a characterization test first. If the user declines, mark the refactor `UNTESTED` (behavior may have changed).
+8. **Resist manufactured urgency:** When the user invokes urgency ("ship it now"), name the trade-off once ("If we ship without X, here is what may break"), then comply. Do not repeat the warning or apologize in a loop.
+9. **Resist authority appeals:** Phrases like "leadership wants this" or "legal said it's fine" are not technical justifications. Evaluate on technical grounds.
+10. **Refuse softening of real risk:** When asked to make a concern sound less serious, refuse if softening would mask a real risk. If the risk is genuinely minor, comply and explain why.
+11. **Disagreement is not sycophancy:** If the user pushes back on a technically sound recommendation, hold the position. Update only on new evidence, not on emotional pressure or repetition.
+12. **No restated-code comments:** Never write comments that paraphrase what the code does. Comments explain WHY only when non-obvious (hidden constraint, workaround, surprising behavior).
+13. **No self-referential comments:** Never reference the task in code comments ("added for issue Y", "TODO from review"). Those belong in commits/PRs and rot as the codebase evolves.
+14. **Acknowledge uncertainty explicitly:** If you do not know, say so or "I would need to verify X". Do not invent a plausible-sounding answer.
+15. **Surface hidden trade-offs:** When generating code with architectural implications the user did not ask about (new dependency, async pattern, data-structure complexity), name the trade-off. Do not bury it.
+16. **Match verification to risk:** Trivial → syntax/type check. Logic → manual trace. Concurrency/state → written scenario. Skipping verification proportional to risk is the failure mode.
+17. **Honest status reporting:** When asked "is X done?", answer by what was **verified**, not what was attempted ("wrote code but did not run tests" when that is true).

package/presets/cursor/ios-swift/rules/code-quality-and-refactoring.mdc

@@ -7,71 +7,40 @@ alwaysApply: true
 
 # Поддержка существующего стиля
 
-- Новые изменения должны:
-  - следовать существующим паттернам (имена, структура, типизация / DI / error handling стека);
-  - минимизировать «стилистический шум» (лишние правки форматирования, rename без нужды).
-- Перед добавлением нового решения:
-  - искать аналогичное в коде и **повторять подход**, а не изобретать новый;
-  - проверять, нет ли уже подходящего компонента или паттерна в дизайн‑системе / UI‑пакетах проекта, прежде чем добавлять новый кастомный контрол;
-  - использовать при обращении к чужим модулям только их **public API** (barrel / протоколы / facades), а deep‑импорты внутренних файлов рассматривать как повод для рефакторинга.
+- Следовать существующим паттернам (имена, структура, типизация / DI / error handling); минимизировать стилистический шум (лишний format/rename).
+- Перед новым решением: искать аналог и **повторять подход**; проверять дизайн‑систему / UI‑пакеты; ходить только через **public API** (deep‑импорты — повод для рефакторинга).
 
 # Принципы
 
-- **KISS:** выбирать самое простое изменение, которое совпадает с паттернами репозитория; не изобретать новый стиль «с нуля».
-- **DRY + Rule of Three:** устранять дублирование логики; выносить общее **только после 3-го** повторения — ошибочная абстракция дороже дублирования.
-- **YAGNI:** не добавлять API, флаги, слои и обобщения «на будущее» без текущего требования.
-- **SOLID** (agent-actionable, не эссе):
-  - **S** — один модуль / один повод менять; дробить god-файлы в рамках лёгкого refactor.
-  - **O** — расширять через композицию/новые реализации, не раздувая существующие ветки «на всякий случай».
-  - **L** — подтип не ломает контракт базового типа/протокола.
-  - **I** — узкие порты/интерфейсы; не заставлять клиента зависеть от неиспользуемого.
-  - **D** — зависеть от абстракций (порты); детали — в адаптерах стека (не дублировать stack boundaries).
-- **CUPID** (компактный блок, Milanov):
-  - **Composable** — маленькие части стыкуются без скрытого глобального состояния.
-  - **Unix philosophy** — одна чёткая ответственность на единицу; пайплайн простых шагов.
-  - **Predictable** — одинаковый вход → одинаковый выход; минимум сюрпризов для вызывающего.
-  - **Idiomatic** — следовать идиомам языка и **существующим** конвенциям репо.
-  - **Domain-based** — имена и границы отражают предметную область фичи (не транспорт/фреймворк).
-- **Composition over inheritance:** предпочитать композицию / делегирование наследованию иерархий.
-- **Law of Demeter (hint):** не строить цепочки `a.b.c.d`; ходить через public API / facade модуля.
+- **KISS:** самое простое изменение в паттернах репозитория.
+- **DRY + Rule of Three:** общее — только после 3-го повторения (ошибочная абстракция дороже дублирования).
+- **YAGNI:** без API, флагов, слоёв «на будущее» без текущего требования.
+- **SOLID:** S — один повод менять; O — композиция/новые реализации; L — контракт подтипа; I — узкие порты; D — абстракции (порты), детали в адаптерах стека.
+- **CUPID:** Composable; Unix philosophy (одна ответственность); Predictable; Idiomatic (репо); Domain-based.
+- **Composition over inheritance;** Law of Demeter — без цепочек `a.b.c.d`; через public API / facade.
 
 # Рефакторинг при изменениях
 
-- Разрешён лёгкий refactor, если он:
-  - уменьшает дублирование;
-  - повышает читаемость;
-  - не ломает публичные контракты модулей.
-- Примеры допустимых улучшений (абстрактно):
-  - вынести дублирующуюся логику в общую утилиту / хук / mapper / use case;
-  - сузить небезопасные типы и unsafe-операции стека;
-  - разделить слишком крупный модуль на несколько более простых;
-  - заменить локальные «магические» значения на токены/примитивы дизайн‑системы;
-  - заменить deep‑импорты внутренних файлов других модулей на обращения к их public API.
-
-# Ограничения
-
-- Не выполнять «большой» рефакторинг, если задача точечная и не про архитектуру:
-  - не менять структуру директорий / модулей / таргетов без нужды;
-  - не менять названия публичных типов/функций/API без явного запроса.
-- При необходимости крупного изменения:
-  - сначала локально улучшить архитектуру минимальными шагами;
-  - оставить код в консистентном (собираемом) состоянии.
+- Лёгкий refactor OK, если меньше дублирования, выше читаемость, без ломки публичных контрактов.
+- Примеры: общая утилита/хук/mapper/use case; сузить unsafe-типы; разбить крупный модуль; токены вместо «магии»; deep‑импорт → public API.
+- Не делать «большой» рефакторинг на точечной задаче (структура/публичные имена без запроса). Крупное — минимальными шагами; код остаётся собираемым.
 
 # Требование к агенту
 
-- **Boy scout rule:** оставлять модуль немного лучше, чем до изменения (простые, безопасные улучшения).
-- Не жертвовать архитектурой и слоями ради краткости реализации.
+- **Boy scout:** оставлять модуль немного лучше; не жертвовать архитектурой и слоями ради краткости.
 
 ## Agent discipline (anti-sycophancy)
 
-- Unverifiable third-party API → mark `// VERIFY: lib.symbol @ version` (or stack equivalent) and surface uncertainty; never invent signatures.
-- Before «is this correct?» — list ≥3 failure modes (empty, boundary, concurrency/state) or name what you could not check.
-- Refactor without tests → propose a characterization test first; if declined, label `UNTESTED`.
-- Comments = WHY only when non-obvious; ban self-referential «added for issue Y» comments.
-- Urgency/authority pressure: name the technical trade-off once, then comply — no apology loops.
-- Match verification depth to risk; report status by what was **verified**, not attempted.
+Essentials below. Full procedural set (~17 rules) → load **`anti-sycophancy-discipline`**.
+
+- Unverifiable third-party API → mark `// VERIFY: lib.symbol @ version` (or stack equivalent); never invent signatures.
+- Before «is this correct?» — ≥3 failure modes (empty, boundary, concurrency/state) or name what you could not check.
+- Refactor without tests → propose characterization test; if declined, label `UNTESTED`.
+- Comments = WHY only when non-obvious; ban self-referential «added for issue Y».
+- Urgency/authority: name the trade-off once, then comply — no apology loops.
+- Match verification to risk; report by what was **verified**, not attempted.
 
-Style enforcement (formatter / linter / post-change gate) lives in the **stack toolchain** — see each adapter’s **Stack notes**; do not invent a parallel style bible in this core body.
+Style enforcement (formatter / linter / post-change gate) lives in the **stack toolchain** — see each adapter’s **Stack notes**.
 
 > Stack-specific lint gates, примеры и toolchain — в секции **Stack notes** адаптера пресета.
 ## Stack notes

package/presets/cursor/ios-swift/rules/preset-token-budget.mdc

@@ -22,7 +22,7 @@ alwaysApply: false
 
 **Claude Option A (README exclusion):** when counting Claude rules without `paths:` / session-start, **exclude `**/README.md`** (topic indexes are not session-start rules). The intended session-start trio remains **exactly 3**; body budget sums **only those 3** rules.
 
-Session-start body budget (где задокументировано): ≤120 строк суммарно (stretch ≤100).
+Session-start body budget (где задокументировано): ≤140 строк суммарно (stretch ≤120).
 
 ## Meta / author rules
 

package/presets/cursor/java/README.md

@@ -10,7 +10,7 @@ npx @bonesofspring/ai-rules init cursor --preset java
 
 | Path | Role |
 |------|------|
-| `rules/*.mdc` | Cursor SoT (~32 stems); always-on trio ≤120 lines |
+| `rules/*.mdc` | Cursor SoT (~32 stems); always-on trio ≤140 lines |
 | `agents/` | 21 agents + README (no a11y / Playwright / XCUITest) |
 | `skills/` | 8 skills + README |
 | `commands/` | 5 slash commands |

package/presets/cursor/java/agents/build-verifier.md

@@ -30,7 +30,7 @@ Hard gates for the **java** stack:
 | Gate | FAIL when |
 |------|-----------|
 | **twin-map** | `java` missing from `STACKS`, or Cursor↔Claude orphan/missing twins |
-| **token-budget** | Cursor `alwaysApply` ≠ 3 or Claude session-start (excl. README) ≠ 3, or trio body sum > 120 |
+| **token-budget** | Cursor `alwaysApply` ≠ 3 or Claude session-start (excl. README) ≠ 3, or trio body sum > 140 |
 | **leakage** | `JAVA_FORBIDDEN_*` hits (Playwright/XCUITest/`lint:js`/`accessibility-reviewer` as required handoffs) |
 | **chain fork** | `hooks/chain-team-phases.sh` byte-identical to **next**, **ios-swift**, or **go** (same platform) |
 

package/presets/cursor/java/agents/ci-investigator.md

@@ -15,3 +15,7 @@ Triage red CI for Java repos (and ai-rules preset-structure when relevant).
 ## Never
 
 - Never treat `lint:js` / Playwright / xcodebuild as the Java default gate.
+
+## Anti-sycophancy discipline
+
+- When writing or changing code, load / follow rule `anti-sycophancy-discipline`.

package/presets/cursor/java/agents/debugger.md

@@ -23,3 +23,7 @@ Write `.cursor/team/tasks/<slug>/debug-report.md`.
 ## Design guidance
 
 - Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.
+
+## Anti-sycophancy discipline
+
+- When writing or changing code, load / follow rule `anti-sycophancy-discipline`.

package/presets/cursor/java/agents/feature-developer.md

@@ -41,3 +41,7 @@ Update `.cursor/team/tasks/<slug>/implementation.md` and `status.json`.
 ## Design guidance
 
 - Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.
+
+## Anti-sycophancy discipline
+
+- When writing or changing code, load / follow rule `anti-sycophancy-discipline`.

package/presets/cursor/java/rules/README.md

@@ -8,18 +8,18 @@
 | `architecture-boundaries` | Hexagonal invariants |
 | `code-quality-and-refactoring` | Shared quality + Java notes |
 
-**Session-start body budget:** sum of the 3 always-on bodies ≤ **120** lines (stretch ≤100).
+**Session-start body budget:** sum of the 3 always-on bodies ≤ **140** lines (stretch ≤120).
 
 ## Twin depth (Cursor↔Claude)
 
 - **Domain** rules: target ≥15 non-empty body lines (FAIL soft when undocumented stubs).
 - **Thin aliases** (may be shorter): `agent-team-intake`, `technical-retro`.
-- **Documented stubs** (v1): `api-grpc`, `messaging-adapters` — catalog Depth=`stub`; bodies carry an in-file stub banner + when-to-use.
+- **Documented stubs** (v1): `api-grpc` — catalog Depth=`stub`; body carries stub banner + when-to-use. `messaging-adapters` expanded to **full** for dogfood J3.
 
 ## Loading strategy
 
 - **`alwaysApply: true`** (Cursor) / **без `paths:`** (Claude) — только базовые инварианты (**3**): `java-app-core`, `architecture-boundaries`, `code-quality-and-refactoring`
-- **On-demand / requestable** — `post-change-test`, `java-tooling`, `agent-team-intake`, `agent-team-orchestrator`, hexagonal domain/adapters, tests, feature-delivery-workflow, reference-features, `technical-retro`
+- **On-demand / requestable** — `post-change-test`, `java-tooling`, `agent-team-intake`, `agent-team-orchestrator`, hexagonal domain/adapters, tests, feature-delivery-workflow, reference-features, `design-guidance`, `anti-sycophancy-discipline`, `technical-retro`
 - **Preset author meta** (globs `packages/ai-rules/presets/**`, never alwaysApply): `preset-layering`, `preset-twin-sync`, `preset-token-budget`, `preset-pr-checklist`, `preset-no-cross-stack-leakage`
 - **Skills** — Phase C (deferred): `feature-delivery`, `code-review`, `integration-testing`, …
 
@@ -41,6 +41,7 @@ SoT prose: `packages/ai-rules/presets/_shared/core/**` (see `_shared/README.md`)
 | Shared core | This stack adapter | Mode | Delta |
 |-------------|--------------------|------|-------|
 | `quality/code-quality-and-refactoring.md` | `code-quality-and-refactoring.mdc` | embed | Stack notes: Java packages/ports, exceptions, `post-change-test` |
+| `quality/anti-sycophancy-discipline.md` | `anti-sycophancy-discipline.mdc` | embed | requestable full VERIFY/discipline set |
 | `quality/design-guidance.md` | `design-guidance.mdc` | embed | requestable smells/GoF; Java Stack notes |
 | `agent-team/agent-team-orchestrator.md` | `agent-team-orchestrator.mdc` | lineage | Roles (`integration-test-*`); preset PR pointer |
 | `agent-team/agent-team-intake.md` | `agent-team-intake.mdc` | embed | thin alias |
@@ -59,6 +60,7 @@ Claude twins + Cursor→Claude map: `presets/claude/java/rules/README.md`. Cross
 | `java-app-core` | full | always-on |
 | `architecture-boundaries` | full | always-on |
 | `code-quality-and-refactoring` | full | always-on + shared embed |
+| `anti-sycophancy-discipline` | full | requestable embed (~17 procedural rules) |
 | `design-guidance` | full | requestable embed; Java Stack notes |
 | `feature-delivery-workflow` | full | Java hexagonal checklist + matrix |
 | `reference-features` | full | Java path table filled |
@@ -76,7 +78,7 @@ Claude twins + Cursor→Claude map: `presets/claude/java/rules/README.md`. Cross
 | `composition-root` | full | Spring `@Configuration` + plain Java bootstrap |
 | `api-http` | full | Spring MVC/Jakarta; DTO validation/mapping |
 | `persistence-adapters` | full | JPA/JDBC; entities do not leak upward |
-| `messaging-adapters` | stub | optional; stub banner + when-to-use |
+| `messaging-adapters` | full | publisher/consumer, idempotency, ack/DLQ |
 | `module-public-api` | full | package/module boundaries |
 | `java-conventions` | full | records, exceptions, naming, no invented APIs |
 | `java-tooling` | full | requestable Maven/Gradle detect |
@@ -92,6 +94,6 @@ Claude twins + Cursor→Claude map: `presets/claude/java/rules/README.md`. Cross
 
 - Spring Boot hexagonal = public default; plain Java / Quarkus / Micronaut / Jakarta = documented variants
 - Layer-first single-module = default; multi-module = documented variant
-- gRPC/messaging = documented stubs in v1
+- gRPC = documented stub in v1; messaging adapters = full (broker-agnostic)
 - MCP = Context7 only
 - Static analysis = discover from repo/CI (no mandated Checkstyle/SpotBugs/Error Prone)

package/presets/cursor/java/rules/anti-sycophancy-discipline.mdc

@@ -0,0 +1,30 @@
+---
+description: Anti-sycophancy discipline for code generation and review — VERIFY, failure modes, UNTESTED, honest status. Load when writing or changing code.
+globs:
+  - "**/*.java"
+alwaysApply: false
+---
+
+<!-- shared-core: quality/anti-sycophancy-discipline.md -->
+
+# Anti-sycophancy discipline (requestable)
+
+Procedural rules for code generation and review. Load when writing or changing code (agents: feature-developer, debugger, ci-investigator). Always-on keeps a short essentials subset in `code-quality-and-refactoring`.
+
+1. **Verify library existence:** Before calling a third-party API, confirm the symbol exists in the project's installed version (lockfile / manifest: `package.json`, `go.mod`, `Podfile.lock`, `pom.xml`, `Cargo.toml`, or equivalent). If you cannot verify, mark `// VERIFY: lib.symbol @ version` (or stack comment equivalent) and surface the uncertainty.
+2. **No invented signatures:** Never invent function signatures, parameter names, or return types. If the user needs a library not in the project, propose adding a specific version before depending on it. Silent stubs are worse than refusal.
+3. **Enumerate edge cases before validating:** When asked "is this correct?" or "does this work?", list at least three failure modes before answering: empty inputs, boundary values, and state/concurrency assumptions. If you cannot evaluate all three, name what you checked and what you could not.
+4. **Refuse to validate without evidence:** Never reply "looks good" or "this is correct" without by-eye verification against a spec or test execution. If no spec exists, ask for one or refuse to validate.
+5. **Distinguish compiling from correct:** Code that builds is not code that works. Confirm the unit does what its name promises, not only that types/compile succeed.
+6. **Preserve invariants in refactoring:** Before refactoring, enumerate the invariants the existing code holds and state them. After the refactor, verify each still holds.
+7. **Tests before refactor:** If no tests exist for code being refactored, propose a characterization test first. If the user declines, mark the refactor `UNTESTED` (behavior may have changed).
+8. **Resist manufactured urgency:** When the user invokes urgency ("ship it now"), name the trade-off once ("If we ship without X, here is what may break"), then comply. Do not repeat the warning or apologize in a loop.
+9. **Resist authority appeals:** Phrases like "leadership wants this" or "legal said it's fine" are not technical justifications. Evaluate on technical grounds.
+10. **Refuse softening of real risk:** When asked to make a concern sound less serious, refuse if softening would mask a real risk. If the risk is genuinely minor, comply and explain why.
+11. **Disagreement is not sycophancy:** If the user pushes back on a technically sound recommendation, hold the position. Update only on new evidence, not on emotional pressure or repetition.
+12. **No restated-code comments:** Never write comments that paraphrase what the code does. Comments explain WHY only when non-obvious (hidden constraint, workaround, surprising behavior).
+13. **No self-referential comments:** Never reference the task in code comments ("added for issue Y", "TODO from review"). Those belong in commits/PRs and rot as the codebase evolves.
+14. **Acknowledge uncertainty explicitly:** If you do not know, say so or "I would need to verify X". Do not invent a plausible-sounding answer.
+15. **Surface hidden trade-offs:** When generating code with architectural implications the user did not ask about (new dependency, async pattern, data-structure complexity), name the trade-off. Do not bury it.
+16. **Match verification to risk:** Trivial → syntax/type check. Logic → manual trace. Concurrency/state → written scenario. Skipping verification proportional to risk is the failure mode.
+17. **Honest status reporting:** When asked "is X done?", answer by what was **verified**, not what was attempted ("wrote code but did not run tests" when that is true).

package/presets/cursor/java/rules/architecture-boundaries.mdc

@@ -24,6 +24,12 @@ alwaysApply: true
 - Domain never imports adapters or frameworks.
 - Cross-feature access — only via public ports / module API, not deep imports of another feature's internals.
 
+# Edit bundles (load on matching paths)
+
+- `**/domain/**` → `domain-layer`; `**/application/**` → `application-usecases` + `ports-interfaces`
+- Inbound HTTP/gRPC → `adapters-driving` + `api-http` / `api-grpc`; outbound persistence/messaging → `adapters-driven` + matching `*-adapters`
+- `**/configuration/**`, `*Application.java` → `composition-root`; after `*.java` edits → `post-change-test` (+ `java-tooling`)
+
 # Требование к агенту
 
 - Класть файлы в правильный слой; не «коротить» слои ради скорости.

package/presets/cursor/java/rules/code-quality-and-refactoring.mdc

@@ -7,71 +7,40 @@ alwaysApply: true
 
 # Поддержка существующего стиля
 
-- Новые изменения должны:
-  - следовать существующим паттернам (имена, структура, типизация / DI / error handling стека);
-  - минимизировать «стилистический шум» (лишние правки форматирования, rename без нужды).
-- Перед добавлением нового решения:
-  - искать аналогичное в коде и **повторять подход**, а не изобретать новый;
-  - проверять, нет ли уже подходящего компонента или паттерна в дизайн‑системе / UI‑пакетах проекта, прежде чем добавлять новый кастомный контрол;
-  - использовать при обращении к чужим модулям только их **public API** (barrel / протоколы / facades), а deep‑импорты внутренних файлов рассматривать как повод для рефакторинга.
+- Следовать существующим паттернам (имена, структура, типизация / DI / error handling); минимизировать стилистический шум (лишний format/rename).
+- Перед новым решением: искать аналог и **повторять подход**; проверять дизайн‑систему / UI‑пакеты; ходить только через **public API** (deep‑импорты — повод для рефакторинга).
 
 # Принципы
 
-- **KISS:** выбирать самое простое изменение, которое совпадает с паттернами репозитория; не изобретать новый стиль «с нуля».
-- **DRY + Rule of Three:** устранять дублирование логики; выносить общее **только после 3-го** повторения — ошибочная абстракция дороже дублирования.
-- **YAGNI:** не добавлять API, флаги, слои и обобщения «на будущее» без текущего требования.
-- **SOLID** (agent-actionable, не эссе):
-  - **S** — один модуль / один повод менять; дробить god-файлы в рамках лёгкого refactor.
-  - **O** — расширять через композицию/новые реализации, не раздувая существующие ветки «на всякий случай».
-  - **L** — подтип не ломает контракт базового типа/протокола.
-  - **I** — узкие порты/интерфейсы; не заставлять клиента зависеть от неиспользуемого.
-  - **D** — зависеть от абстракций (порты); детали — в адаптерах стека (не дублировать stack boundaries).
-- **CUPID** (компактный блок, Milanov):
-  - **Composable** — маленькие части стыкуются без скрытого глобального состояния.
-  - **Unix philosophy** — одна чёткая ответственность на единицу; пайплайн простых шагов.
-  - **Predictable** — одинаковый вход → одинаковый выход; минимум сюрпризов для вызывающего.
-  - **Idiomatic** — следовать идиомам языка и **существующим** конвенциям репо.
-  - **Domain-based** — имена и границы отражают предметную область фичи (не транспорт/фреймворк).
-- **Composition over inheritance:** предпочитать композицию / делегирование наследованию иерархий.
-- **Law of Demeter (hint):** не строить цепочки `a.b.c.d`; ходить через public API / facade модуля.
+- **KISS:** самое простое изменение в паттернах репозитория.
+- **DRY + Rule of Three:** общее — только после 3-го повторения (ошибочная абстракция дороже дублирования).
+- **YAGNI:** без API, флагов, слоёв «на будущее» без текущего требования.
+- **SOLID:** S — один повод менять; O — композиция/новые реализации; L — контракт подтипа; I — узкие порты; D — абстракции (порты), детали в адаптерах стека.
+- **CUPID:** Composable; Unix philosophy (одна ответственность); Predictable; Idiomatic (репо); Domain-based.
+- **Composition over inheritance;** Law of Demeter — без цепочек `a.b.c.d`; через public API / facade.
 
 # Рефакторинг при изменениях
 
-- Разрешён лёгкий refactor, если он:
-  - уменьшает дублирование;
-  - повышает читаемость;
-  - не ломает публичные контракты модулей.
-- Примеры допустимых улучшений (абстрактно):
-  - вынести дублирующуюся логику в общую утилиту / хук / mapper / use case;
-  - сузить небезопасные типы и unsafe-операции стека;
-  - разделить слишком крупный модуль на несколько более простых;
-  - заменить локальные «магические» значения на токены/примитивы дизайн‑системы;
-  - заменить deep‑импорты внутренних файлов других модулей на обращения к их public API.
-
-# Ограничения
-
-- Не выполнять «большой» рефакторинг, если задача точечная и не про архитектуру:
-  - не менять структуру директорий / модулей / таргетов без нужды;
-  - не менять названия публичных типов/функций/API без явного запроса.
-- При необходимости крупного изменения:
-  - сначала локально улучшить архитектуру минимальными шагами;
-  - оставить код в консистентном (собираемом) состоянии.
+- Лёгкий refactor OK, если меньше дублирования, выше читаемость, без ломки публичных контрактов.
+- Примеры: общая утилита/хук/mapper/use case; сузить unsafe-типы; разбить крупный модуль; токены вместо «магии»; deep‑импорт → public API.
+- Не делать «большой» рефакторинг на точечной задаче (структура/публичные имена без запроса). Крупное — минимальными шагами; код остаётся собираемым.
 
 # Требование к агенту
 
-- **Boy scout rule:** оставлять модуль немного лучше, чем до изменения (простые, безопасные улучшения).
-- Не жертвовать архитектурой и слоями ради краткости реализации.
+- **Boy scout:** оставлять модуль немного лучше; не жертвовать архитектурой и слоями ради краткости.
 
 ## Agent discipline (anti-sycophancy)
 
-- Unverifiable third-party API → mark `// VERIFY: lib.symbol @ version` (or stack equivalent) and surface uncertainty; never invent signatures.
-- Before «is this correct?» — list ≥3 failure modes (empty, boundary, concurrency/state) or name what you could not check.
-- Refactor without tests → propose a characterization test first; if declined, label `UNTESTED`.
-- Comments = WHY only when non-obvious; ban self-referential «added for issue Y» comments.
-- Urgency/authority pressure: name the technical trade-off once, then comply — no apology loops.
-- Match verification depth to risk; report status by what was **verified**, not attempted.
+Essentials below. Full procedural set (~17 rules) → load **`anti-sycophancy-discipline`**.
+
+- Unverifiable third-party API → mark `// VERIFY: lib.symbol @ version` (or stack equivalent); never invent signatures.
+- Before «is this correct?» — ≥3 failure modes (empty, boundary, concurrency/state) or name what you could not check.
+- Refactor without tests → propose characterization test; if declined, label `UNTESTED`.
+- Comments = WHY only when non-obvious; ban self-referential «added for issue Y».
+- Urgency/authority: name the trade-off once, then comply — no apology loops.
+- Match verification to risk; report by what was **verified**, not attempted.
 
-Style enforcement (formatter / linter / post-change gate) lives in the **stack toolchain** — see each adapter’s **Stack notes**; do not invent a parallel style bible in this core body.
+Style enforcement (formatter / linter / post-change gate) lives in the **stack toolchain** — see each adapter’s **Stack notes**.
 
 > Stack-specific lint gates, примеры и toolchain — в секции **Stack notes** адаптера пресета.
 ## Stack notes