viberadar 0.3.205 → 0.3.207

package/dist/server/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/server/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAO7B,OAAO,EAAE,UAAU,EAA4H,MAAM,YAAY,CAAC;AAOlK,UAAU,aAAa;IACrB,IAAI,EAAE,UAAU,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;CACrB;AAED,MAAM,WAAW,YAAY;IAC3B,MAAM,EAAE,IAAI,CAAC,MAAM,CAAC;CACrB;AAiuED,wBAAgB,WAAW,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,WAAW,EAAE,EAAE,aAAa,GAAG,OAAO,CAAC,YAAY,CAAC,CAggG1G"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/server/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAO7B,OAAO,EAAE,UAAU,EAA4H,MAAM,YAAY,CAAC;AAOlK,UAAU,aAAa;IACrB,IAAI,EAAE,UAAU,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;CACrB;AAED,MAAM,WAAW,YAAY;IAC3B,MAAM,EAAE,IAAI,CAAC,MAAM,CAAC;CACrB;AA0vED,wBAAgB,WAAW,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,WAAW,EAAE,EAAE,aAAa,GAAG,OAAO,CAAC,YAAY,CAAC,CAggG1G"}

package/dist/server/index.js

@@ -1253,8 +1253,11 @@ function buildObsFeatureReviewPrompt(featureKey, data) {
             .slice(0, 5)
             .map(fp => `    - строка ~${fp.lineApprox}: ${FP_TYPE_LABELS[fp.type] || fp.type} — \`${fp.snippet}\``)
             .join('\n');
+        const coverageState = item.hasAnyWarnError
+            ? 'часть warn/error уже есть, но не все точки покрыты'
+            : 'warn/error покрытия не найдено';
         return [
-            `- ${item.modulePath} (${item.roleHint}, ${item.riskTier}, score ${item.riskScore})`,
+            `- ${item.modulePath} (${item.failurePoints.length} точек отказа; ${coverageState})`,
             fpLines || `    - Нет warn/error покрытия: найди точки отказа вручную`,
         ].join('\n');
     });
@@ -1301,7 +1304,7 @@ function buildObsFeatureReviewPrompt(featureKey, data) {
         ...catalogLines,
         ``,
         `Правила принятия решений:`,
-        `- Для каждого изменения сначала определи: add_error_log | enrich_fields | remove_noise | downgrade_to_debug | keep | needs_human_review.`,
+        `- Для каждой точки/лога сначала определи: add_error_log | add_warn_log | enrich_fields | remove_noise | downgrade_to_debug | keep_without_log | keep | needs_human_review.`,
         `- Никогда не трогай logger.warn/logger.error/logger.fatal как "шум".`,
         `- Не понижай ERROR/WARN уровни.`,
         `- Не добавляй ERROR-лог для штатных 400/401/403/404/409/422, validation/not-found/conflict/domain ошибок, если это ожидаемый ответ API.`,
@@ -1311,7 +1314,7 @@ function buildObsFeatureReviewPrompt(featureKey, data) {
         `- В payload логгера используй только переменные из текущего scope; не угадывай поля из типов.`,
         `- Не добавляй process.env в клиентские Vite/React файлы; используй import.meta.env.MODE, если env реально нужен.`,
         ``,
-        `Перед правками выведи краткий план: файл, действие, причина. Если уверенность низкая — не меняй код, пометь как needs_human_review.`,
+        `Перед правками выведи краткий план: файл, действие, причина. Меняй код только для add_error_log/add_warn_log/enrich_fields/remove_noise/downgrade_to_debug. Если уверенность низкая — не меняй код, пометь как needs_human_review.`,
         ``,
         `Проверка после изменений:`,
         `- npm run check -- --pretty false или npm run typecheck, если check отсутствует.`,
@@ -1324,17 +1327,24 @@ function buildObsAddCriticalLogsPrompt(modulePath, catalog) {
     const moduleItem = catalog.find(c => c.modulePath === modulePath);
     const missingFields = moduleItem?.missingFields || [];
     return [
-        `Добавь критичные логи (warn/error) в модуль \`${modulePath}\`.`,
+        `Проверь точки отказа и добавь нужные warn/error логи в модуль \`${modulePath}\`.`,
         ``,
-        `Сейчас в модуле нет warn/error событий. При сбоях мы не увидим ошибку в логах.`,
+        `VibeRadar не нашёл достаточного warn/error покрытия. Это статический сигнал, а не приказ логировать всё подряд.`,
         ``,
         moduleItem
             ? `Текущее состояние модуля:\n- Формат: ${moduleItem.format}\n- Уровень: ${moduleItem.level}\n- Пропущенные поля: ${missingFields.length > 0 ? missingFields.join(', ') : 'нет'}`
             : '',
         ``,
+        `Важно: VibeRadar дал кандидатов, а не список обязательных правок.`,
+        `Сначала классифицируй каждую точку:`,
+        `- add_error_log — реальный неожиданный сбой операции, должен быть виден в production`,
+        `- add_warn_log — деградация, retry, graceful fallback или частичный результат`,
+        `- keep_without_log — ожидаемый domain/validation/not-found/conflict/permission ответ, UI validation или intentional fallback`,
+        `- needs_human_review — контекст спорный`,
+        ``,
         `Что сделать:`,
         `- Найди в модуле точки, где может произойти ошибка (catch-блоки, проверки null/undefined, HTTP-ответы с ошибкой, DB-ошибки)`,
-        `- Добавь logger.warn или logger.error с обязательными полями по стандарту`,
+        `- Меняй код только для add_error_log/add_warn_log`,
         `- НЕ добавляй ERROR-лог для штатных validation/not-found/conflict/domain throws, если это ожидаемый ответ API`,
         `- НЕ логируй намеренные .catch(() => null/undefined) cleanup/json-parse fallbacks без реального operational риска`,
         `- Каждый лог должен включать: event_name, outcome, error_code (для error)`,
@@ -1357,9 +1367,8 @@ function buildObsAddCriticalLogsPromptV2(item, catalog) {
     const catalogEntry = catalog.find(c => c.modulePath === item.modulePath);
     const fpDescriptions = item.failurePoints.map(fp => `- Строка ~${fp.lineApprox}: ${FP_TYPE_LABELS[fp.type] || fp.type}\n  \`${fp.snippet}\``).join('\n');
     return [
-        `Добавь критичные логи (warn/error) в модуль \`${item.modulePath}\`.`,
+        `Проверь точки отказа и добавь нужные warn/error логи в модуль \`${item.modulePath}\`.`,
         ``,
-        `Роль модуля: ${item.roleHint} (приоритет: ${item.riskTier})`,
         item.hasAnyWarnError
             ? `В модуле есть некоторые warn/error, но обнаружены незакрытые точки отказа.`
             : `В модуле НЕТ ни одного warn/error. При сбоях мы не увидим ошибку в логах.`,
@@ -1371,13 +1380,17 @@ function buildObsAddCriticalLogsPromptV2(item, catalog) {
         `Обнаруженные точки отказа без логирования (${item.failurePoints.length}):`,
         fpDescriptions,
         ``,
+        `Важно: VibeRadar дал кандидатов, а не список обязательных правок.`,
         `Что сделать с каждой точкой:`,
-        `- Пустые catch-блоки: добавь logger.error с контекстом, event_name, error_code, outcome:failure`,
-        `- catch без лога: добавь logger.error/warn рядом с обработкой ошибки`,
-        `- .catch() без лога: добавь logger.error в обработчик промиса`,
-        `- HTTP/DB без обработки: оберни в try/catch с logger.error`,
-        `- throw без лога: добавь logger.error ДО throw`,
-        `- if(err) без лога: добавь logger.warn/error в ветку ошибки`,
+        `- Сначала прочитай контекст и реши: add_error_log | add_warn_log | keep_without_log | needs_human_review.`,
+        `- Меняй код только для add_error_log/add_warn_log.`,
+        `- Пустые catch-блоки: добавь logger.error только если ошибка реально теряется и влияет на операцию.`,
+        `- catch без лога: добавь logger.error/warn рядом с обработкой ошибки, если это operational-сбой.`,
+        `- .catch() без лога: добавь лог только если это не намеренный безопасный fallback/cleanup.`,
+        `- HTTP/DB без обработки: оберни в try/catch с logger.error только при реальном риске потери сбоя.`,
+        `- throw без лога: НЕ добавляй лог для штатных validation/not-found/conflict/domain ошибок.`,
+        `- if(err) без лога: добавь logger.warn/error в ветку ошибки, если ошибка должна быть видна в production.`,
+        `- Для keep_without_log не добавляй лог даже если VibeRadar подсветил точку.`,
         ``,
         `Каждый лог: event_name, outcome, error_code (для error).`,
         `event_name: <domain>.<entity>.<action> (lower_snake_case через точку)`,
@@ -1395,7 +1408,8 @@ function guessTestFile(modulePath) {
 function buildObsBatchAddCriticalLogsPrompt(items, catalog) {
     const moduleBlocks = items.map(item => {
         const fpSummary = item.failurePoints.map(fp => `  - строка ~${fp.lineApprox}: ${FP_TYPE_LABELS[fp.type] || fp.type} — \`${fp.snippet}\``).join('\n');
-        return `### \`${item.modulePath}\` (${item.roleHint}, ${item.riskTier})\n${fpSummary || '  - Нет warn/error, проверь весь модуль на точки отказа'}`;
+        const coverageState = item.hasAnyWarnError ? 'частично покрыто warn/error' : 'warn/error не найдено';
+        return `### \`${item.modulePath}\` (${item.failurePoints.length} точек отказа; ${coverageState})\n${fpSummary || '  - Нет warn/error, проверь весь модуль на точки отказа'}`;
     }).join('\n\n');
     const testCommands = items
         .map(item => guessTestFile(item.modulePath))
@@ -1404,9 +1418,20 @@ function buildObsBatchAddCriticalLogsPrompt(items, catalog) {
         .map(f => `  npm run test -- ${f}`)
         .join('\n');
     return [
-        `Добавь критичные логи в ${items.length} модулей.`,
+        `Проверь точки отказа и добавь нужные warn/error логи в ${items.length} модулей.`,
+        ``,
+        `Для каждого модуля: прочитай контекст точек отказа (указаны ниже) и добавь logger.warn/error только там, где это реальный operational-сбой.`,
+        ``,
+        `Важно: VibeRadar дал список кандидатов, а не список обязательных правок.`,
+        `Перед любым изменением классифицируй каждую точку:`,
+        `- add_error_log — реальный неожиданный сбой операции, должен быть виден в production`,
+        `- add_warn_log — деградация, retry, graceful fallback или частичный результат`,
+        `- keep_without_log — ожидаемый domain/validation/not-found/conflict/permission ответ, UI validation, intentional cleanup или best-effort fallback`,
+        `- needs_human_review — контекст спорный или уверенность низкая`,
         ``,
-        `Для каждого модуля: найди точки отказа (указаны ниже) и добавь logger.warn/error с обязательными полями.`,
+        `Меняй код только для add_error_log/add_warn_log.`,
+        `Для keep_without_log не добавляй лог даже если VibeRadar подсветил точку.`,
+        `В финальном ответе отдельно перечисли: что изменил, что оставил без изменений и почему.`,
         ``,
         moduleBlocks,
         ``,
@@ -1415,8 +1440,8 @@ function buildObsBatchAddCriticalLogsPrompt(items, catalog) {
         `- outcome: failure|partial`,
         `- error_code из словаря (VALIDATION_ERROR, DEPENDENCY_TIMEOUT, INTERNAL_ERROR и т.д.)`,
         `- Не стремись обнулить все сигналы VibeRadar: validation/not-found/conflict/domain throws и intentional silent fallbacks можно оставить без ERROR-лога`,
-        `- Пустые catch: добавь logger.error, не оставляй пустыми`,
-        `- HTTP/DB без обработки: оберни в try/catch с logger.error`,
+        `- Пустые catch: добавь logger.error только если ошибка теряется и влияет на операцию; intentional cleanup/best-effort fallback можно оставить без error-лога или оформить как debug/warn по контексту`,
+        `- HTTP/DB без обработки: оберни в try/catch с logger.error только если ошибка не обрабатывается выше и должна быть видна в production`,
         `- В payload-объекте логгера используй ТОЛЬКО переменные из текущего скоупа (не угадывай имена полей из типов)`,
         ``,
         `⚠️ ВАЖНО — React / Vite файлы (.tsx):`,
@@ -3327,7 +3352,7 @@ function startServer({ data: initialData, port, projectRoot }) {
             }
             else if (task === 'obs-add-critical-logs') {
                 const modulePath = meta?.modulePath || 'unknown';
-                title = `${agentLabel} — добавить критичные логи в "${modulePath}"`;
+                title = `${agentLabel} — добавить логи точек отказа в "${modulePath}"`;
             }
             else if (task === 'obs-enrich-field') {
                 const fieldName = meta?.fieldName || 'unknown';