code-ai-installer 1.2.0 → 1.3.0

package/.agents/observability_review/SKILL.md

@@ -1,20 +1,254 @@
----
-name: observability_review
-description: Ревью наблюдаемости: структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan.
----
-
-# Skill: Observability Review
-
-## Проверить
-- Структурированные логи (ключи, а не голые строки)
-- Корреляция request_id/trace_id (если предусмотрено)
-- Уровни логирования корректны (error только для реальных ошибок)
-- Не логируется PII/секреты
-- Ключевые события (audit) логируются при необходимости
-
-## Выход
-- Несоответствия плану
-- Конкретные рекомендации
-
-## См. также
-- Примеры и анти-примеры: $review_reference_snippets
+---
+name: observability_review
+description: Ревью наблюдаемости — структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan.
+---
+
+# Skill: Observability Review
+
+Ревью реализации наблюдаемости: логи, метрики, корреляция, соответствие плану.
+
+**Разделы:**
+1. [Checklist](#1-checklist)
+2. [DO / DON'T Examples](#2-examples)
+3. [Automated Detection](#3-detection)
+4. [Pino Config Review](#4-pino)
+5. [Output Template](#5-output)
+
+---
+
+## 1. Checklist
+
+### 1.1 Structured Logging
+
+| # | Check | Severity | Status |
+|---|-------|----------|--------|
+| OBS-01 | All logs are structured JSON (pino), no `console.log` | 🟠 P1 | ☐ |
+| OBS-02 | Log messages are constants (event names), not concatenated strings | 🟠 P1 | ☐ |
+| OBS-03 | Context fields as object properties, not in message string | 🟠 P1 | ☐ |
+| OBS-04 | Child loggers used for request scope (`req.log`) | 🟠 P1 | ☐ |
+
+### 1.2 Correlation
+
+| # | Check | Severity | Status |
+|---|-------|----------|--------|
+| OBS-05 | `requestId` present on all request-scoped logs | 🟠 P1 | ☐ |
+| OBS-06 | `requestId` generated from `X-Request-ID` header or UUID | 🟡 P2 | ☐ |
+| OBS-07 | `requestId` propagated to external API calls | 🟡 P2 | ☐ |
+
+### 1.3 Log Levels
+
+| # | Check | Severity | Status |
+|---|-------|----------|--------|
+| OBS-08 | `error` only for real errors (not validation failures, not 404) | 🟠 P1 | ☐ |
+| OBS-09 | `warn` for expected but noteworthy (rate limit, deprecated usage) | 🟡 P2 | ☐ |
+| OBS-10 | `info` for business events (create, update, delete, auth) | 🟠 P1 | ☐ |
+| OBS-11 | `debug` only in development (not in production log level) | 🟠 P1 | ☐ |
+
+### 1.4 PII & Secrets
+
+| # | Check | Severity | Status |
+|---|-------|----------|--------|
+| OBS-12 | No passwords, tokens, API keys in logs | 🔴 P0 | ☐ |
+| OBS-13 | No email/phone without masking | 🔴 P0 | ☐ |
+| OBS-14 | No `req.headers` or `req.body` logged without filtering | 🔴 P0 | ☐ |
+| OBS-15 | pino `redact` configured for sensitive paths | 🟠 P1 | ☐ |
+
+### 1.5 Business Events
+
+| # | Check | Severity | Status |
+|---|-------|----------|--------|
+| OBS-16 | Critical operations have audit logs (install, settings change, coupon CRUD) | 🟠 P1 | ☐ |
+| OBS-17 | Health endpoints exist (`/health/live`, `/health/ready`) | 🟠 P1 | ☐ |
+| OBS-18 | Error responses logged with context (but not PII) | 🟠 P1 | ☐ |
+
+---
+
+## 2. DO / DON'T Examples
+
+### Structured vs Unstructured
+
+```javascript
+// ❌ DON'T: Unstructured string logging
+console.log('User ' + userId + ' created coupon ' + couponId + ' at ' + new Date());
+console.log(`Error: ${err.message}`);
+
+// ✅ DO: Structured JSON logging
+logger.info({ userId, couponId }, 'coupon_created');
+logger.error({ err, requestId: req.id }, 'coupon_create_failed');
+```
+
+### Context in Fields vs Message
+
+```javascript
+// ❌ DON'T: Context baked into message string
+logger.info(`Processing webhook for appInstanceId=${appId} from ${siteUrl}`);
+
+// ✅ DO: Context as object properties (searchable, filterable)
+logger.info({ appInstanceId: appId, siteUrl }, 'webhook_processing');
+```
+
+### Correct Log Levels
+
+```javascript
+// ❌ DON'T: Error level for non-errors
+logger.error('Coupon not found');        // 404 is not an error
+logger.error('Validation failed');       // 400 is not an error
+
+// ✅ DO: Correct levels
+logger.warn({ code: couponCode }, 'coupon_not_found');     // Expected case
+logger.warn({ fields: errors }, 'validation_failed');      // Expected case
+logger.error({ err, requestId }, 'unhandled_error');       // Real error
+```
+
+### PII in Logs
+
+```javascript
+// ❌ DON'T: Log full request
+logger.info({ headers: req.headers, body: req.body }, 'request_received');
+//           ^^^ authorization header!   ^^^ passwords/tokens!
+
+// ✅ DO: Allowlist safe fields
+logger.info({
+  requestId: req.id,
+  method: req.method,
+  url: req.url,
+  status: res.statusCode,
+  durationMs: Date.now() - start
+}, 'http_request');
+```
+
+### Child Logger
+
+```javascript
+// ❌ DON'T: Pass requestId manually everywhere
+logger.info({ requestId: req.id, userId }, 'settings_updated');
+logger.info({ requestId: req.id, couponId }, 'coupon_created');
+
+// ✅ DO: Create child logger once, use everywhere in request
+// In middleware:
+req.log = logger.child({ requestId: req.id });
+// In handler:
+req.log.info({ userId }, 'settings_updated');
+req.log.info({ couponId }, 'coupon_created');
+// requestId is automatically included!
+```
+
+---
+
+## 3. Automated Detection
+
+### Grep patterns
+
+```bash
+# 🟠 P1: Unstructured logging
+grep_search: Query="console.log"     → Replace with pino logger
+grep_search: Query="console.error"   → Replace with logger.error
+grep_search: Query="console.warn"    → Replace with logger.warn
+grep_search: Query="console.info"    → Replace with logger.info
+
+# 🔴 P0: PII/secrets in logs
+grep_search: Query="req.headers" Includes=["*.js"]    → Check: not logging auth header?
+grep_search: Query="req.body" Includes=["*.js"]       → Check: not logging passwords?
+grep_search: Query="password" Includes=["*.js"]       → Check: not in log context
+grep_search: Query="token" Includes=["*.js"]          → Check: not in log context
+
+# 🟠 P1: Wrong log level
+grep_search: Query="logger.error.*not.found"         → Should be warn
+grep_search: Query="logger.error.*validat"           → Should be warn
+grep_search: Query="logger.error.*404"               → Should be warn
+
+# 🟠 P1: Missing correlation
+grep_search: Query="logger.info" Includes=["*/routes/*","*/controllers/*"]  → Check: has requestId?
+```
+
+---
+
+## 4. Pino Config Review
+
+### Expected pino configuration
+
+```javascript
+// ✅ Correct pino setup
+const logger = pino({
+  level: process.env.LOG_LEVEL || 'info',
+  redact: {
+    paths: [
+      'password',
+      'token',
+      'authorization',
+      'cookie',
+      'req.headers.authorization',
+      'req.headers.cookie',
+      '*.password',
+      '*.token',
+      '*.secret',
+      '*.apiKey',
+      '*.refreshToken'
+    ],
+    censor: '[REDACTED]'
+  },
+  serializers: {
+    err: pino.stdSerializers.err,
+    req: pino.stdSerializers.req,
+    res: pino.stdSerializers.res
+  }
+});
+```
+
+### Config checks
+
+| Check | Expected | grep pattern |
+|-------|----------|-------------|
+| Level from env | `process.env.LOG_LEVEL` | `grep "LOG_LEVEL"` |
+| Redact configured | `redact: { paths: [...] }` | `grep "redact"` |
+| password redacted | `'password'` in paths | `grep "password.*redact"` |
+| authorization redacted | `'authorization'` in paths | `grep "authorization.*redact"` |
+| Serializers set | `pino.stdSerializers` | `grep "stdSerializers"` |
+
+---
+
+## 5. Output Template
+
+```markdown
+# Observability Review Report
+
+**Scope:** <PR/module>
+**Reviewer:** Reviewer Agent
+**Date:** YYYY-MM-DD
+
+## Findings
+
+| # | Severity | Check | File:Line | Finding | Fix |
+|---|----------|-------|-----------|---------|-----|
+| 1 | 🔴 P0 | OBS-12 | `routes/webhook.js:15` | `req.body` logged with tokens | Use allowlist fields |
+| 2 | 🟠 P1 | OBS-01 | `services/coupon.js:42` | `console.log()` used | Replace with `req.log.info()` |
+| 3 | 🟠 P1 | OBS-08 | `controllers/settings.js:60` | `logger.error('Not found')` | Change to `logger.warn()` |
+
+## Pino Config
+| Check | Status |
+|-------|--------|
+| Level from env | ✅ |
+| Redact: password | ✅ |
+| Redact: authorization | ⚠️ Missing |
+| Serializers | ✅ |
+
+## Compliance with Observability Plan
+| Event | Expected | Implemented | Status |
+|-------|----------|-------------|--------|
+| server_started | info | ✅ | Pass |
+| http_request | info + requestId | ⚠️ no requestId | Fix |
+| coupon_created | info + couponId | ✅ | Pass |
+
+## Verdict
+- ✅ COMPLIANT — observability plan fully implemented
+- ⚠️ GAPS — partial implementation, P1 items to fix
+- ❌ NON-COMPLIANT — critical gaps (PII exposure, no correlation)
+```
+
+---
+
+## См. также
+- `$observability_plan` — what to log/measure/alert
+- `$observability_logging` — implementation patterns (pino setup, middleware)
+- `$review_reference_snippets` — DO/DON'T snippets (section N, O)
+- `$code_review_checklist` — general review (observability section)

package/.agents/observability_review/agents/claude.json

@@ -1,13 +1,13 @@
 {
   "name": "observability_review",
   "display_name": "Проверка наблюдаемости",
-  "description": "Ревью наблюдаемости: структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan.",
+  "description": "Ревью наблюдаемости — структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan.",
   "default_prompt": "Используй $observability_review, когда задача относится к навыку \"Проверка наблюдаемости\".",
   "triggers": [
     "observability_review",
     "observability review",
     "Проверка наблюдаемости",
-    "Ревью наблюдаемости"
+    "Ревью наблюдаемости — структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan"
   ],
   "capabilities": [
     "observability",

package/.agents/observability_review/agents/copilot.json

@@ -1,13 +1,13 @@
 {
   "name": "observability_review",
   "display_name": "Проверка наблюдаемости",
-  "description": "Ревью наблюдаемости: структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan.",
+  "description": "Ревью наблюдаемости — структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan.",
   "default_prompt": "Используй $observability_review, когда задача относится к навыку \"Проверка наблюдаемости\".",
   "triggers": [
     "observability_review",
     "observability review",
     "Проверка наблюдаемости",
-    "Ревью наблюдаемости"
+    "Ревью наблюдаемости — структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan"
   ],
   "capabilities": [
     "observability",

package/.agents/observability_review/agents/gemini.json

@@ -1,13 +1,13 @@
 {
   "name": "observability_review",
   "display_name": "Проверка наблюдаемости",
-  "description": "Ревью наблюдаемости: структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan.",
+  "description": "Ревью наблюдаемости — структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan.",
   "default_prompt": "Используй $observability_review, когда задача относится к навыку \"Проверка наблюдаемости\".",
   "triggers": [
     "observability_review",
     "observability review",
     "Проверка наблюдаемости",
-    "Ревью наблюдаемости"
+    "Ревью наблюдаемости — структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan"
   ],
   "capabilities": [
     "observability",

package/.agents/observability_review/agents/openai.yaml

@@ -1,6 +1,6 @@
 interface:
   display_name: "Проверка наблюдаемости"
-  short_description: "Ревью наблюдаемости: структурированные логи, request_id/trace_id, уровни логов, запрет утечек P…"
+  short_description: "Ревью наблюдаемости — структурированные логи, request_id/trace_id, уровни логов, запрет утечек…"
   default_prompt: "Используй $observability_review, когда задача относится к навыку \"Проверка наблюдаемости\"."
 dependencies:
   tools: []

package/.agents/observability_review/agents/qwen.json

@@ -1,13 +1,13 @@
 {
   "name": "observability_review",
   "display_name": "Проверка наблюдаемости",
-  "description": "Ревью наблюдаемости: структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan.",
+  "description": "Ревью наблюдаемости — структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan.",
   "default_prompt": "Используй $observability_review, когда задача относится к навыку \"Проверка наблюдаемости\".",
   "triggers": [
     "observability_review",
     "observability review",
     "Проверка наблюдаемости",
-    "Ревью наблюдаемости"
+    "Ревью наблюдаемости — структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan"
   ],
   "capabilities": [
     "observability",

package/.agents/observability_review/agents/skill.yaml

@@ -1,13 +1,13 @@
 version: 1
 name: "observability_review"
 display_name: "Проверка наблюдаемости"
-description: "Ревью наблюдаемости: структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan."
+description: "Ревью наблюдаемости — структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan."
 default_prompt: "Используй $observability_review, когда задача относится к навыку \"Проверка наблюдаемости\"."
 triggers:
   - "observability_review"
   - "observability review"
   - "Проверка наблюдаемости"
-  - "Ревью наблюдаемости"
+  - "Ревью наблюдаемости — структурированные логи, request_id/trace_id, уровни логов, запрет утечек PII/секретов, соответствие Observability Plan"
 capabilities:
   - "observability"
   - "review"

package/.agents/performance_review_baseline/SKILL.md

@@ -1,17 +1,239 @@
----
-name: performance_review_baseline
-description: Базовый perf-ревью: N+1, пагинация, батчи, лишние запросы, кеш по архитектуре, тяжелые операции в request-path.
----
-
-# Skill: Performance Review Baseline
-
-## Проверить
-- N+1 запросы / неоптимальные выборки
-- Нет пагинации там, где список может быть большим
-- Лишние round-trips (можно batch)
-- Тяжёлые операции в синхронном request-path (нужна async/queue?) — только если архитектура допускает
-- Кэширование применено там, где это было предусмотрено (и безопасно)
-
-## Выход
-- Bottlenecks (P1/P2 обычно, P0 если ломает SLA/UX)
-- Рекомендации оптимизации без premature optimization
+---
+name: performance_review_baseline
+description: Базовый perf-ревью — N+1, пагинация, батчи, лишние запросы, кеш по архитектуре, тяжелые операции в request-path.
+---
+
+# Skill: Performance Review Baseline
+
+Ревью производительности без premature optimization — фокус на реальных bottlenecks.
+
+**Разделы:**
+1. [Checklist](#1-checklist)
+2. [DO / DON'T Examples](#2-examples)
+3. [Latency Targets](#3-latency)
+4. [Automated Detection](#4-detection)
+5. [Output Template](#5-output)
+
+---
+
+## 1. Checklist
+
+### 1.1 Database Queries
+
+| # | Check | Severity | Status |
+|---|-------|----------|--------|
+| PERF-01 | No N+1 queries (loop of individual DB calls) | 🔴 P0 | ☐ |
+| PERF-02 | Indexes exist for all query patterns | 🟠 P1 | ☐ |
+| PERF-03 | `.select()` used — don't fetch unnecessary fields | 🟠 P1 | ☐ |
+| PERF-04 | `.lean()` used for read-only queries (Mongoose) | 🟡 P2 | ☐ |
+| PERF-05 | No unbounded queries (always limit/paginate) | 🔴 P0 | ☐ |
+| PERF-06 | Aggregation pipelines optimized (`$match` before `$lookup`) | 🟠 P1 | ☐ |
+
+### 1.2 API & Network
+
+| # | Check | Severity | Status |
+|---|-------|----------|--------|
+| PERF-07 | No redundant round-trips (batch where possible) | 🟠 P1 | ☐ |
+| PERF-08 | Pagination on list endpoints | 🔴 P0 | ☐ |
+| PERF-09 | Response payload minimal (no over-fetching) | 🟠 P1 | ☐ |
+| PERF-10 | Heavy operations offloaded to async (queue/worker) | 🟠 P1 | ☐ |
+| PERF-11 | Compression enabled (gzip/brotli) for API responses | 🟡 P2 | ☐ |
+
+### 1.3 Frontend
+
+| # | Check | Severity | Status |
+|---|-------|----------|--------|
+| PERF-12 | No unnecessary re-renders (React.memo, useMemo, useCallback where appropriate) | 🟠 P1 | ☐ |
+| PERF-13 | Images optimized (WebP, lazy loading, proper dimensions) | 🟠 P1 | ☐ |
+| PERF-14 | Bundle size reasonable (no giant imports like full lodash) | 🟠 P1 | ☐ |
+| PERF-15 | Code splitting / lazy routes for large apps | 🟡 P2 | ☐ |
+
+### 1.4 Caching
+
+| # | Check | Severity | Status |
+|---|-------|----------|--------|
+| PERF-16 | Cache applied where architecture allows (read-heavy, rarely changing) | 🟡 P2 | ☐ |
+| PERF-17 | Cache invalidation strategy defined | 🟠 P1 | ☐ |
+| PERF-18 | HTTP cache headers for static assets (Cache-Control, ETag) | 🟡 P2 | ☐ |
+
+---
+
+## 2. DO / DON'T Examples
+
+### N+1 Query Problem
+
+```javascript
+// ❌ DON'T: N+1 — one query per item in loop
+const coupons = await Coupon.find({ appInstanceId });
+for (const coupon of coupons) {
+  coupon.settings = await Settings.findOne({ couponId: coupon._id }); // N queries!
+}
+
+// ✅ DO: Single query with $in or $lookup
+const coupons = await Coupon.find({ appInstanceId });
+const couponIds = coupons.map(c => c._id);
+const settings = await Settings.find({ couponId: { $in: couponIds } }); // 1 query
+// Map settings to coupons in memory
+```
+
+### Unbounded Query
+
+```javascript
+// ❌ DON'T: Fetch ALL records
+const allCoupons = await Coupon.find({}); // Could be 100K records!
+
+// ✅ DO: Paginate
+const coupons = await Coupon.find({})
+  .sort({ createdAt: -1 })
+  .skip(page * limit)
+  .limit(limit)
+  .lean();
+```
+
+### Over-fetching
+
+```javascript
+// ❌ DON'T: Return entire document
+const settings = await Settings.findOne({ appInstanceId });
+res.json(settings); // Includes all internal fields
+
+// ✅ DO: Select only needed fields
+const settings = await Settings.findOne({ appInstanceId })
+  .select('templateId colors fonts timer coupon')
+  .lean();
+res.json({ data: settings });
+```
+
+### Sync Heavy Operation
+
+```javascript
+// ❌ DON'T: Heavy operation in request path
+router.post('/export', async (req, res) => {
+  const report = await generatePDFReport(allData); // 30 seconds!
+  res.json({ report });
+});
+
+// ✅ DO: Offload to async
+router.post('/export', async (req, res) => {
+  const jobId = await queue.add('generate-report', { userId: req.userId });
+  res.status(202).json({ jobId, status: 'processing' }); // Return immediately
+});
+```
+
+### React Re-renders
+
+```jsx
+// ❌ DON'T: New object/function on every render
+function ParentComponent() {
+  return <Child style={{ color: 'red' }} onClick={() => doSomething()} />;
+  //            ^^ new object every render  ^^ new function every render
+}
+
+// ✅ DO: Stable references
+const style = useMemo(() => ({ color: 'red' }), []);
+const handleClick = useCallback(() => doSomething(), []);
+return <Child style={style} onClick={handleClick} />;
+```
+
+---
+
+## 3. Latency Targets
+
+### API endpoints
+
+| Endpoint Type | p50 | p95 | p99 | Alert threshold (p95) |
+|--------------|-----|-----|-----|-----------------------|
+| Widget (public, read) | 10ms | 50ms | 100ms | > 200ms |
+| Dashboard (CRUD) | 20ms | 100ms | 500ms | > 1000ms |
+| List with pagination | 30ms | 150ms | 500ms | > 1000ms |
+| Webhooks (async OK) | 50ms | 500ms | 2000ms | > 5000ms |
+| File/image upload | 100ms | 1000ms | 5000ms | > 10000ms |
+
+### Frontend
+
+| Metric | Target | P0 if exceeds |
+|--------|--------|:---:|
+| FCP (First Contentful Paint) | < 1.5s | > 3s |
+| LCP (Largest Contentful Paint) | < 2.5s | > 4s |
+| CLS (Cumulative Layout Shift) | < 0.1 | > 0.25 |
+| TTI (Time to Interactive) | < 3.5s | > 7s |
+| JS Bundle (gzipped) | < 200KB | > 500KB |
+
+---
+
+## 4. Automated Detection
+
+### Grep patterns
+
+```bash
+# N+1 indicators
+grep_search: Query="await.*find"  Includes=["*.js"]  → check if inside loop (for/forEach/map)
+grep_search: Query="for.*await"   Includes=["*.js"]  → sequential async in loop
+
+# Unbounded queries
+grep_search: Query=".find({" Includes=["*.js"]  → check: has .limit() ?
+grep_search: Query=".find()" Includes=["*.js"]  → no filter at all
+
+# Over-fetching
+grep_search: Query=".findOne(" Includes=["*.js"]  → check: has .select() ?
+
+# Heavy sync operations
+grep_search: Query="readFileSync"  → P1: blocking I/O
+grep_search: Query="JSON.parse"    → check: large payloads?
+
+# React performance
+grep_search: Query="style={{" Includes=["*.jsx","*.tsx"]  → inline object → re-render
+grep_search: Query="() =>" Includes=["*.jsx","*.tsx"]     → inline function in JSX props
+```
+
+### Mongoose explain
+
+```javascript
+// Check query plan for slow queries
+const explain = await Model.find(query).explain('executionStats');
+// Look for: COLLSCAN (no index), nReturned vs totalDocsExamined
+```
+
+---
+
+## 5. Output Template
+
+```markdown
+# Performance Review Report
+
+**Scope:** <PR/module>
+**Reviewer:** Reviewer Agent
+**Date:** YYYY-MM-DD
+
+## Findings
+
+| # | Severity | Category | File:Line | Finding | Impact | Fix |
+|---|----------|----------|-----------|---------|--------|-----|
+| 1 | 🔴 P0 | N+1 | `services/coupon.js:45` | DB query inside for loop | 100 coupons = 100 queries | Use `$in` batch query |
+| 2 | 🔴 P0 | Unbounded | `routes/coupons.js:12` | `Coupon.find({})` without limit | Memory spike on large datasets | Add pagination |
+| 3 | 🟠 P1 | Over-fetch | `controllers/settings.js:8` | No `.select()` | 40 fields returned, 5 needed | Add `.select().lean()` |
+
+## Latency Check
+| Endpoint | Current p95 | Target p95 | Status |
+|----------|:-----------:|:----------:|:------:|
+| GET /widget/:id | ~30ms | 50ms | ✅ |
+| GET /coupons | ~800ms | 100ms | ❌ PERF-01 |
+
+## Checklist Summary
+- P0: X findings (blocking SLA/UX)
+- P1: Y findings (should fix)
+- P2: Z findings (optimization opportunities)
+
+## Verdict
+- ✅ PASS — no bottlenecks
+- ⚠️ OPTIMIZATION NEEDED — P0/P1 findings
+- ❌ BLOCKED — critical performance issue breaking SLA
+```
+
+---
+
+## См. также
+- `$observability_plan` — latency targets, metrics
+- `$observability_review` — logging/metrics compliance
+- `$data_model` — index strategy
+- `$review_reference_snippets` — general DO/DON'T examples

package/.agents/performance_review_baseline/agents/claude.json

@@ -1,13 +1,13 @@
 {
   "name": "performance_review_baseline",
   "display_name": "Базовая проверка производительности",
-  "description": "Базовый perf-ревью: N+1, пагинация, батчи, лишние запросы, кеш по архитектуре, тяжелые операции в request-path.",
+  "description": "Базовый perf-ревью — N+1, пагинация, батчи, лишние запросы, кеш по архитектуре, тяжелые операции в request-path.",
   "default_prompt": "Используй $performance_review_baseline, когда задача относится к навыку \"Базовая проверка производительности\".",
   "triggers": [
     "performance_review_baseline",
     "performance review baseline",
     "Базовая проверка производительности",
-    "Базовый perf-ревью"
+    "Базовый perf-ревью — N+1, пагинация, батчи, лишние запросы, кеш по архитектуре, тяжелые операции в request-path"
   ],
   "capabilities": [
     "performance",

package/.agents/performance_review_baseline/agents/copilot.json

@@ -1,13 +1,13 @@
 {
   "name": "performance_review_baseline",
   "display_name": "Базовая проверка производительности",
-  "description": "Базовый perf-ревью: N+1, пагинация, батчи, лишние запросы, кеш по архитектуре, тяжелые операции в request-path.",
+  "description": "Базовый perf-ревью — N+1, пагинация, батчи, лишние запросы, кеш по архитектуре, тяжелые операции в request-path.",
   "default_prompt": "Используй $performance_review_baseline, когда задача относится к навыку \"Базовая проверка производительности\".",
   "triggers": [
     "performance_review_baseline",
     "performance review baseline",
     "Базовая проверка производительности",
-    "Базовый perf-ревью"
+    "Базовый perf-ревью — N+1, пагинация, батчи, лишние запросы, кеш по архитектуре, тяжелые операции в request-path"
   ],
   "capabilities": [
     "performance",