From 527b82767afc40ac2af16fbd5f721d665cbe92cb Mon Sep 17 00:00:00 2001 From: Sadmin Date: Thu, 23 Jul 2026 21:01:44 +0000 Subject: [PATCH] Add vector tech-debt task queue --- vector_tasks.md | 638 ++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 638 insertions(+) create mode 100644 vector_tasks.md diff --git a/vector_tasks.md b/vector_tasks.md new file mode 100644 index 0000000..7eb4315 --- /dev/null +++ b/vector_tasks.md @@ -0,0 +1,638 @@ +# ВЕКТОР — очередь технического долга для делегирования + +**Проект:** ВЕКТОР (Система определения приоритетных направлений поиска, SAR/МЧС) +**Стек:** FastAPI + PostgreSQL 16 + React (CRA), docker-compose +**Расположение:** LXC 108 (`Lab`, 192.168.0.108), путь `/root/vector` +**Дата анализа:** 2026-07-21 +**Состояние на момент анализа:** git чистый, ветка `master`, `126 passed, 26 skipped` (async-тесты пропущены — нет `pytest-asyncio`). + +## Как пользоваться этим файлом + +- **Раздел A — ATOMIC.** Задачи для локальной LLM (Qwen2.5-coder, 12GB VRAM, ~16–32k контекста). Каждый промт самодостаточен: весь нужный код встроен в промт, модель НЕ читает остальной проект. Бери задачу, копируй блок `PROMPT` целиком в модель, применяй ответ, проверяй по `Критерий приёмки`. +- **Раздел B — NEEDS-CONTEXT.** НЕ давать локальной модели. Требуют понимания нескольких файлов, контрактов данных, роутинга или бизнес-логики SAR. Оставлены для более сильной модели / человека. + +## Жёсткие исключения (соблюдены во всём файле) + +Ни одна ATOMIC-задача не трогает: +- ❌ алгоритм зонального скоринга (7 факторов) — весь `backend/services/scoring_service.py` вынесен в раздел B; +- ❌ схему БД / миграции (`backend/models.py`, `backend/migrations/*`); +- ❌ auth/security (`backend/routers/auth.py`, JWT, bcrypt, роли); +- ❌ бизнес-логику МЧС/SAR (коэффициенты дистанции, поведенческие профили, психотипы как методику). + +--- + +# РАЗДЕЛ A — ATOMIC (для локальной LLM) + +Отсортировано по модулю. Внутри модуля — по порядку файла. + +--- + +## A1 — `backend/services/distance_service.py` · защита от None + +- **Файл / строки:** `backend/services/distance_service.py:111-121`, `148-158`, `168-178` +- **Тип:** missing validation +- **Сложность:** atomic + +**PROMPT (скопировать в модель целиком):** + +```` +Ты правишь один Python-файл. Ниже три функции. У каждой первый аргумент — строка, и сразу вызывается `.lower()`. Если аргумент окажется None, будет AttributeError. Добавь защиту: если аргумент None или пустой, используй значение по умолчанию (для terrain → вернуть 0.5; для time_of_day → вести себя как 'день' → вернуть 1.0; для weather → вести себя как отсутствие осадков → вернуть 1.0). Не меняй числовые коэффициенты и словари. Верни ТОЛЬКО три изменённые функции целиком, в блоке ```python. + +def get_terrain_coefficient(terrain: str) -> float: + terrain_lower = terrain.lower() + terrain_map = { + 'лесная дорога': 0.8, 'сложный лес': 0.25, 'густой лес': 0.25, + 'простой лес': 0.5, 'лес': 0.5, 'дорога': 0.8, 'тропа': 0.8, + 'болото': 0.2, 'поле': 0.9, 'луг': 0.9, 'город': 1.0, + 'населённый пункт': 1.0, 'горы': 0.3, 'овраг': 0.3 + } + for key, value in terrain_map.items(): + if key in terrain_lower: + return value + return 0.5 + +def get_time_of_day_coefficient(time_of_day: str) -> float: + time_lower = time_of_day.lower() + if 'ночь' in time_lower: + return 0.5 + elif 'сумерки' in time_lower or 'вечер' in time_lower: + return 0.5 + else: + return 1.0 + +def get_weather_coefficient(weather: str) -> float: + weather_lower = weather.lower() + if 'ливень' in weather_lower or 'сильный дождь' in weather_lower: + return 0.6 + elif 'дождь' in weather_lower: + return 0.8 + elif 'туман' in weather_lower: + return 0.7 + elif 'снег' in weather_lower or 'метель' in weather_lower: + return 0.6 + elif 'жара' in weather_lower: + return 0.8 + else: + return 1.0 +```` + +**Критерий приёмки:** +- `get_terrain_coefficient(None) == 0.5`, `get_time_of_day_coefficient(None) == 1.0`, `get_weather_coefficient(None) == 1.0`. +- Все существующие значения не изменились: `get_terrain_coefficient('густой лес') == 0.25`, `get_weather_coefficient('туман') == 0.7`. +- `PYTHONPATH=/root/vector python -m pytest backend/tests/test_distance_service.py -q` — зелёный. + +--- + +## A2 — `backend/services/claude_service.py` · заменить `print()` на логирование + +- **Файл / строки:** `backend/services/claude_service.py:66`, `111`, `247` (и добавить импорт/логгер вверху файла, строки 5-9) +- **Тип:** missing error handling (диагностика в stdout вместо логгера) +- **Сложность:** atomic + +**PROMPT:** + +```` +В Python-файле три места пишут диагностику через print(). Замени их на стандартный модуль logging. Вверху файла (рядом с существующими импортами `import os`, `import json`) добавь `import logging` и создай модульный логгер `logger = logging.getLogger(__name__)`. Затем замени каждый print на соответствующий вызов logger. Для строк с ошибками используй logger.warning(...). Не меняй никакую другую логику. Верни ТОЛЬКО unified diff (формат `diff`), затрагивающий импорты и три строки. + +Текущие строки: + (строка 66) — внутри except при недоступности Claude API: + print(f"Claude API unavailable: {e}. Using fallback scoring service.") + (строка 111) — внутри except geo/scoring в analyze_with_claude: + print(f"Geo/scoring service error: {e}") + (строка 247) — внутри except geo/scoring в analyze_with_fallback: + print(f"Geo/scoring service error in fallback: {e}") + +Существующие импорты вверху файла: +import os +import json +from typing import Dict, List, Optional +from pydantic import BaseModel +import httpx +```` + +**Критерий приёмки:** +- `grep -n "print(" backend/services/claude_service.py` → пусто. +- В начале файла есть `import logging` и `logger = logging.getLogger(__name__)`. +- `PYTHONPATH=/root/vector python -c "import backend.services.claude_service"` — без ошибок. + +--- + +## A3 — `backend/services/claude_service.py` · устаревший ID модели Claude + +- **Файл / строки:** `backend/services/claude_service.py:157` +- **Тип:** bug (несуществующая/устаревшая модель) +- **Сложность:** atomic + +**PROMPT:** + +```` +В Python-файле в теле POST-запроса к Anthropic API указан устаревший идентификатор модели. Замени строковое значение модели с "claude-sonnet-4-20250514" на "claude-sonnet-5". Больше НИЧЕГО не меняй (max_tokens, заголовки, структуру messages оставь как есть). Верни ТОЛЬКО одну изменённую строку. + +Текущая строка (157): + "model": "claude-sonnet-4-20250514", +```` + +**Критерий приёмки:** +- Строка 157 содержит `"model": "claude-sonnet-5",`. +- `grep -n "claude-sonnet-4" backend/services/claude_service.py` → пусто. + +> Примечание для проверяющего: актуальные ID на дату анализа — `claude-sonnet-5` (баланс скорость/качество), `claude-opus-4-8` (максимум качества). Для JSON-анализа кейса `claude-sonnet-5` — разумный дефолт. + +--- + +## A4 — `backend/routers/analyze.py` · устаревший `datetime.utcnow()` + +- **Файл / строки:** `backend/routers/analyze.py:3` (импорт) и `:106` +- **Тип:** bug (deprecated API, начиная с Python 3.12 `datetime.utcnow()` deprecated) +- **Сложность:** atomic + +**PROMPT:** + +```` +В Python-файле используется устаревший `datetime.utcnow()`. Замени его на timezone-aware вариант `datetime.now(timezone.utc)`. Для этого: +1. В строке импорта `from datetime import datetime` добавь `timezone`: `from datetime import datetime, timezone`. +2. Замени `datetime.utcnow().isoformat()` на `datetime.now(timezone.utc).isoformat()`. +Больше ничего не меняй. Верни ТОЛЬКО две изменённые строки с указанием, какую на какую. + +Текущий импорт (строка 3): +from datetime import datetime + +Текущая строка (106), внутри словаря result: + 'analyzed_at': datetime.utcnow().isoformat(), +```` + +**Критерий приёмки:** +- `grep -n "utcnow" backend/routers/analyze.py` → пусто. +- Импорт содержит `timezone`. +- `PYTHONPATH=/root/vector python -c "import backend.routers.analyze"` — без ошибок. + +--- + +## A5 — `backend/main.py` · миграция `@app.on_event('startup')` → lifespan + +- **Файл / строки:** `backend/main.py:38-40` +- **Тип:** bug (deprecated в текущих версиях FastAPI/Starlette) +- **Сложность:** atomic + +**PROMPT:** + +```` +В FastAPI-приложении используется устаревший декоратор `@app.on_event('startup')`. Перепиши его на современный lifespan-контекст. Требования: +1. Добавь `from contextlib import asynccontextmanager` в импорты. +2. Создай асинхронную функцию lifespan, которая на старте вызывает init_db(), затем yield. +3. Передай её в конструктор: `app = FastAPI(title='Vector API', version='0.1.0', lifespan=lifespan)`. +4. Удали старый блок `@app.on_event('startup')` / `def startup()`. +Функция lifespan должна быть объявлена ДО создания app. init_db уже импортирован как `from backend.database import init_db`. Верни ТОЛЬКО изменённые фрагменты в блоке ```python. + +Текущий релевантный код: +app = FastAPI(title='Vector API', version='0.1.0') + +app.add_middleware( + CORSMiddleware, + allow_origins=cors_origins, + allow_credentials=allow_credentials, + allow_methods=['*'], + allow_headers=['*'], +) + + +@app.on_event('startup') +def startup() -> None: + init_db() +```` + +**Критерий приёмки:** +- `grep -n "on_event" backend/main.py` → пусто. +- Есть `@asynccontextmanager` и `lifespan=lifespan` в вызове `FastAPI(...)`. +- `PYTHONPATH=/root/vector python -c "import backend.main"` — без ошибок; приложение стартует, `GET /api/v1/health` → `{"status":"ok"}`. + +--- + +## A6 — `backend/routers/admin.py` · обработка ошибок парсинга .docx + +- **Файл / строки:** `backend/routers/admin.py:28-38` (функция `_extract_docx_text`) и `:134-139` (эндпоинт `admin_parse_doc`) +- **Тип:** missing error handling +- **Сложность:** atomic + +> Не трогает auth: декоратор роутера и `require_roles` остаются без изменений. + +**PROMPT:** + +```` +В этом FastAPI-роутере функция `_extract_docx_text` разбирает .docx (это zip с XML). Если загружен НЕ .docx (битый файл, не-zip, нет word/document.xml), `zipfile.ZipFile` или `archive.read` бросят исключение, и клиент получит непонятную 500-ю. Оберни разбор так, чтобы при ошибке чтения архива/XML эндпоинт возвращал HTTP 400 с понятным сообщением. + +Сделай ДВА изменения, не трогая ничего другого: +1. В функции `_extract_docx_text` оберни блок чтения zip и парсинга XML в try/except (перехватывай `zipfile.BadZipFile`, `KeyError`, `ET.ParseError`). При любой из этих ошибок брось `ValueError("Не удалось прочитать .docx: файл повреждён или имеет неверный формат")`. +2. В эндпоинте `admin_parse_doc` оберни вызов `_extract_docx_text(content)` в try/except ValueError и при ошибке брось `raise HTTPException(status_code=400, detail=str(e))`. + +`HTTPException` уже импортирован. `zipfile`, `ET` (xml.etree.ElementTree) уже импортированы. Верни ТОЛЬКО две изменённые функции целиком в блоке ```python. + +Текущий код: + +def _extract_docx_text(content: bytes) -> str: + with zipfile.ZipFile(BytesIO(content)) as archive: + xml = archive.read('word/document.xml') + root = ET.fromstring(xml) + ns = {'w': 'http://schemas.openxmlformats.org/wordprocessingml/2006/main'} + paragraphs: list[str] = [] + for paragraph in root.findall('.//w:body/w:p', ns): + texts = [node.text for node in paragraph.findall('.//w:t', ns) if node.text] + if texts: + paragraphs.append(''.join(texts).strip()) + return '\n'.join(paragraphs).strip() + +@router.post('/parse-doc', response_model=ParseDocResponse) +async def admin_parse_doc(file: UploadFile = File(...)) -> dict: + content = await file.read() + raw_text = _extract_docx_text(content) + preview = _coerce_preview(raw_text) + return {'filename': file.filename, 'parsed': True, 'preview': preview, 'raw_text': raw_text} +```` + +**Критерий приёмки:** +- Загрузка валидного .docx по-прежнему возвращает 200 и корректный `preview` (тест `backend/tests/test_parse_doc_api.py` зелёный). +- Загрузка мусорного файла (не-zip) → HTTP 400 с сообщением, а не 500. +- `PYTHONPATH=/root/vector python -m pytest backend/tests/test_parse_doc_api.py -q` — зелёный. + +--- + +## A7 — тесты: включить `pytest-asyncio` (26 пропущенных тестов) + +- **Файл / строки:** `backend/requirements.txt:15` (добавить строку) + новый файл `pytest.ini` в корне репозитория +- **Тип:** missing test (тесты есть, но молча пропускаются) +- **Сложность:** atomic + +**PROMPT:** + +```` +В проекте есть async-тесты, помеченные @pytest.mark.asyncio, но плагин pytest-asyncio не установлен, поэтому 26 тестов молча пропускаются. Нужно: + +1. В конец файла requirements (см. содержимое ниже) добавить строку: pytest-asyncio==0.24.0 +2. Создать НОВЫЙ файл pytest.ini в корне репозитория со следующим содержимым (включает автоматический режim asyncio и регистрирует маркер): + +[pytest] +asyncio_mode = auto +testpaths = backend/tests +markers = + asyncio: async test powered by pytest-asyncio + +Верни: (а) финальную строку, которую добавить в requirements.txt; (б) полное содержимое нового файла pytest.ini. + +Текущий requirements.txt (последние строки): +pydantic==2.10.3 +pydantic-settings==2.6.1 +email-validator==2.1.0 +psycopg2-binary==2.9.12 +pytest==8.3.4 +```` + +**Критерий приёмки:** +- После `pip install pytest-asyncio==0.24.0` (внутри `.venv`) и добавления `pytest.ini`: + `PYTHONPATH=/root/vector python -m pytest backend/tests -q` показывает **0 skipped** по причине "async def not natively supported" (число passed вырастает с 126 в сторону ~150). +- Никаких новых падений; предупреждение `PytestUnknownMarkWarning` для `asyncio` исчезает. + +--- + +## A8 — удаление мёртвого кода backend (механическое) + +- **Файлы (удалить целиком):** + - `backend/api/v1/analyze.py`, `backend/api/v1/auth.py`, `backend/api/v1/cases.py`, `backend/api/v1/stats.py`, `backend/api/v1/__init__.py`, `backend/api/v1/analyze.py.backup`, и пустой каталог `backend/api/` (весь `api/` — старая копия, `main.py` подключает только `backend/routers/*`) + - `backend/models_new.py`, `backend/models_updated.py`, `backend/models_backup.py` (используется только `backend/models.py`) + - `backend/services/geo_service.py.bak`, `backend/services/scoring_service.py.bak` + - `backend/check_model.py` (отладочный скрипт с битым `import models`) +- **Тип:** dead code +- **Сложность:** atomic (чисто механическое удаление — не требует чтения кода) + +> Подтверждено анализом: `main.py` импортирует `backend.routers.*`; ничего вне каталога `backend/api/` его не импортирует; `models_*.py` не импортируются нигде; `.bak/.backup` — резервные копии. + +**PROMPT (это shell-задача, не правка кода):** + +```` +Выполни удаление перечисленных файлов и проверь, что тесты не сломались. Не удаляй ничего сверх списка. + +git rm backend/api/v1/analyze.py backend/api/v1/auth.py backend/api/v1/cases.py backend/api/v1/stats.py backend/api/v1/__init__.py backend/api/v1/analyze.py.backup +git rm backend/api/__init__.py +git rm backend/models_new.py backend/models_updated.py backend/models_backup.py +git rm backend/services/geo_service.py.bak backend/services/scoring_service.py.bak +git rm backend/check_model.py + +Затем: +PYTHONPATH=/root/vector python -m pytest backend/tests -q +```` + +**Критерий приёмки:** +- Перечисленные файлы отсутствуют; каталог `backend/api/` удалён. +- `PYTHONPATH=/root/vector python -m pytest backend/tests -q` — то же число passed, что и до удаления (никаких новых ошибок импорта). +- `PYTHONPATH=/root/vector python -c "import backend.main"` — без ошибок. + +--- + +## A9 — документация: единый адрес и имя БД + +- **Файлы / строки:** `PSYCHOTYPE_SUMMARY.md` (везде `192.168.0.99`), `SERVICES_README.md`, `PSYCHOTYPE_API.md`, `MOBILE_FORM_README.md` (адреса `.99`); `DEPLOY.md` (имя БД `sar_mchs` и путь `/root/sar-mchs`) +- **Тип:** naming / консистентность документации +- **Сложность:** atomic + +> Канон (сверено с `docker-compose.yml` и реальным LXC): адрес **192.168.0.108**, БД **vector_mchs**, путь проекта **/root/vector**. + +**PROMPT:** + +```` +Задача — привести документацию к единым значениям. В markdown-файлах замени по всему тексту: +- 192.168.0.99 → 192.168.0.108 +- имя базы данных sar_mchs → vector_mchs +- путь /root/sar-mchs → /root/vector +- в командах `cd /root/sar-mchs` → `cd /root/vector` + +Это простая текстовая замена, кода не касается. Выполни sed-заменой по указанным .md-файлам и верни список выполненных команд: + +sed -i 's/192\.168\.0\.99/192.168.0.108/g' PSYCHOTYPE_SUMMARY.md SERVICES_README.md PSYCHOTYPE_API.md MOBILE_FORM_README.md +sed -i 's/sar_mchs/vector_mchs/g; s#/root/sar-mchs#/root/vector#g' DEPLOY.md +```` + +**Критерий приёмки:** +- `grep -rn "192.168.0.99\|sar_mchs\|sar-mchs" *.md` → пусто. +- Смысл документов не изменился, только адреса/имена. + +--- + +## A10 — `frontend/src/pages/analysis/AnalysisResult.jsx` · неверный парсинг времени суток + +- **Файл / строки:** `frontend/src/pages/analysis/AnalysisResult.jsx:42-48` +- **Тип:** bug +- **Сложность:** atomic + +**PROMPT:** + +```` +В React-компоненте функция getTimeOfDay ожидает строку "HH:MM" и берёт часы через lostTime.split(':')[0]. Но на вход приходит значение из в формате "YYYY-MM-DDTHH:MM" (например "2026-07-21T14:30"). Тогда split(':')[0] === "2026-07-21T14", а parseInt даёт 2026 → время суток определяется неверно почти всегда. + +Почини getTimeOfDay так, чтобы она корректно извлекала час из обоих форматов: +- если строка содержит 'T' — берём часть после 'T', затем часы до ':'; +- иначе — берём часы до ':' как раньше. +Логика диапазонов (день/сумерки/ночь) и возвращаемые строки не меняются. Верни ТОЛЬКО исправленную функцию в блоке ```jsx. + +Текущий код: + const getTimeOfDay = (lostTime) => { + if (!lostTime) return 'день'; + const hour = parseInt(lostTime.split(':')[0]); + if (hour >= 6 && hour < 18) return 'день'; + if (hour >= 18 && hour < 22) return 'сумерки'; + return 'ночь'; + }; +```` + +**Критерий приёмки:** +- `getTimeOfDay("2026-07-21T14:30")` → `'день'`; `getTimeOfDay("2026-07-21T23:00")` → `'ночь'`; `getTimeOfDay("2026-07-21T20:00")` → `'сумерки'`. +- `getTimeOfDay("08:15")` (старый формат) → `'день'`; `getTimeOfDay("")` → `'день'`. + +--- + +## A11 — `frontend/src/components/CaseForm/Step4Environment.jsx` · неверный номер шага в заголовке + +- **Файл / строки:** `frontend/src/components/CaseForm/Step4Environment.jsx:58` +- **Тип:** bug (текст UI противоречит динамическому индикатору «Шаг X из Y») +- **Сложность:** atomic + +**PROMPT:** + +```` +В React-компоненте захардкожен заголовок "Шаг 5: Среда и местность", но этот компонент рендерится как шаг 4 (в родителе CaseForm он идёт под индексом, отображаемым как «Шаг 4»). Заголовок вводит в заблуждение и конфликтует с динамическим индикатором. Убери жёсткий номер шага из заголовка, оставив только название раздела. + +Замени строку: +

Шаг 5: Среда и местность

+на: +

Среда и местность

+ +Верни ТОЛЬКО изменённую строку. +```` + +**Критерий приёмки:** +- Строка 58 больше не содержит «Шаг 5». +- Никакой другой разметки не тронуто. + +--- + +## A12 — `frontend/src/components/CaseForm/Step4Environment.jsx` · обработка ошибок геолокации + +- **Файл / строки:** `frontend/src/components/CaseForm/Step4Environment.jsx:28-54` +- **Тип:** missing error handling (используется блокирующий `alert`, нет структурного состояния ошибки) +- **Сложность:** atomic + +**PROMPT:** + +```` +В React-компоненте функция handleGetLocation сообщает об ошибках геолокации через блокирующий alert(). Замени alert на запись сообщения об ошибке в состояние формы через updateData (ключ gps_error), чтобы UI мог показать ошибку не блокируя поток. Требования: +- при отсутствии navigator.geolocation: updateData({ gps_error: 'Геолокация не поддерживается вашим браузером' }); return; +- перед запросом координат сбрасывай ошибку: updateData({ gps_loading: true, gps_error: null }); +- в success-колбэке помимо координат ставь gps_error: null; +- в error-колбэке: updateData({ gps_error: 'Ошибка получения координат: ' + error.message, gps_loading: false }); +Не удаляй gps_loading, не меняй параметры getCurrentPosition (enableHighAccuracy/timeout/maximumAge). Верни ТОЛЬКО изменённую функцию handleGetLocation в блоке ```jsx. + +Текущий код: + const handleGetLocation = () => { + if (!navigator.geolocation) { + alert("Геолокация не поддерживается вашим браузером"); + return; + } + updateData({ gps_loading: true }); + navigator.geolocation.getCurrentPosition( + (position) => { + updateData({ + tnp_lat: position.coords.latitude.toFixed(6), + tnp_lon: position.coords.longitude.toFixed(6), + gps_loading: false + }); + }, + (error) => { + alert("Ошибка получения координат: " + error.message); + updateData({ gps_loading: false }); + }, + { + enableHighAccuracy: true, + timeout: 10000, + maximumAge: 0 + } + ); + }; +```` + +**Критерий приёмки:** +- В функции больше нет `alert(`. +- При ошибке/отказе в правах в состоянии формы появляется `gps_error` с текстом; при успехе `gps_error === null`. +- Координаты по-прежнему пишутся в `tnp_lat`/`tnp_lon` с 6 знаками. + +--- + +## A13 — `frontend/src/components/SearchMap.jsx` · CDN-иконки и рассинхрон версии Leaflet + +- **Файл / строки:** `frontend/src/components/SearchMap.jsx:186-187` +- **Тип:** bug (внешние `raw.githubusercontent`/`cdnjs`, версия 1.7.1 против 1.9.4 в остальном коде) +- **Сложность:** atomic + +**PROMPT:** + +```` +В React-компоненте иконки маркеров Leaflet грузятся с ненадёжных внешних CDN (raw.githubusercontent.com и cdnjs) и с версией 1.7.1, тогда как остальной код использует unpkg leaflet 1.9.4. Замени оба URL на unpkg-адреса версии 1.9.4 (стандартный маркер Leaflet), чтобы источник был единым и стабильным. + +Замени: + iconUrl: 'https://raw.githubusercontent.com/pointhi/leaflet-color-markers/master/img/marker-icon-2x-red.png', + shadowUrl: 'https://cdnjs.cloudflare.com/ajax/libs/leaflet/1.7.1/images/marker-shadow.png', +на: + iconUrl: 'https://unpkg.com/leaflet@1.9.4/dist/images/marker-icon-2x.png', + shadowUrl: 'https://unpkg.com/leaflet@1.9.4/dist/images/marker-shadow.png', + +Верни ТОЛЬКО две изменённые строки. +```` + +**Критерий приёмки:** +- В строках 186-187 нет `raw.githubusercontent.com` и `cdnjs.cloudflare.com`. +- Оба URL указывают на `unpkg.com/leaflet@1.9.4`. +- Карта на странице анализа отображает маркеры (визуальная проверка). + +--- + +## A14 — `frontend/src/components/CaseForm/CaseForm.jsx` · убрать отладочный `console.log` + +- **Файл / строки:** `frontend/src/components/CaseForm/CaseForm.jsx:119` +- **Тип:** dead code +- **Сложность:** atomic + +**PROMPT:** + +```` +В React-компоненте в боевом пути отправки формы остался отладочный вывод. Удали строку: + console.log("Case created:", result); +Ничего кроме этой строки не меняй. Верни подтверждение и номер удалённой строки. +```` + +**Критерий приёмки:** +- `grep -n "console.log" frontend/src/components/CaseForm/CaseForm.jsx` → пусто. +- Логика отправки/навигации не изменена. + +--- + +## A15 — `frontend/src/pages/AdminDashboard.jsx` · молча проглоченная ошибка загрузки дашборда + +- **Файл / строки:** `frontend/src/pages/AdminDashboard.jsx:98-121` +- **Тип:** missing error handling +- **Сложность:** atomic + +**PROMPT:** + +```` +В React-компоненте эффект загрузки дашборда при ошибке просто ставит dashboard=null и не показывает пользователю ничего (в отличие от загрузчика кейсов, который вызывает setError). Добавь пользовательское сообщение об ошибке через уже существующий сеттер setError. + +В catch-блоке эффекта загрузки дашборда, внутри `if (err.name !== 'AbortError') { ... }`, помимо `setDashboard(null);` добавь `setError(err.message || 'Ошибка загрузки дашборда');`. Ничего другого не меняй. Верни ТОЛЬКО изменённый catch/finally блок в ```jsx. + +Текущий код: + } catch (err) { + if (err.name !== 'AbortError') { + setDashboard(null); + } + } finally { + setDashboardLoading(false); + } +```` + +**Критерий приёмки:** +- При падении запроса дашборда в состоянии выставляется `error` с сообщением. +- `setDashboard(null)` сохранён; `AbortError` по-прежнему игнорируется. + +> Проверяющему: убедиться, что `setError` в этом компоненте — общий стейт с загрузчиком кейсов и его отображение не конфликтует. Если нужно раздельное сообщение — см. B-раздел (рефакторинг стейта AdminDashboard). + +--- + +## A16 — `frontend/src/App.js` · `isMobile` не реагирует на ресайз (опционально) + +- **Файл / строки:** `frontend/src/App.js:30` +- **Тип:** simple refactor +- **Сложность:** atomic + +**PROMPT:** + +```` +В React-приложении признак мобильного вычисляется один раз при монтировании и не обновляется при ресайзе/повороте экрана: + const isMobile = /iPhone|iPad|iPod|Android/i.test(navigator.userAgent) || window.innerWidth < 768; + +Замени на React-состояние с подпиской на событие resize. Используй хуки useState и useEffect (проверь, что они импортированы из 'react'; если нет — добавь в существующий импорт). Реализация: + const [isMobile, setIsMobile] = useState( + () => /iPhone|iPad|iPod|Android/i.test(navigator.userAgent) || window.innerWidth < 768 + ); + useEffect(() => { + const onResize = () => setIsMobile( + /iPhone|iPad|iPod|Android/i.test(navigator.userAgent) || window.innerWidth < 768 + ); + window.addEventListener('resize', onResize); + return () => window.removeEventListener('resize', onResize); + }, []); + +Не меняй, как isMobile передаётся дальше. Верни изменённый фрагмент и, при необходимости, обновлённую строку импорта из 'react', в блоке ```jsx. +```` + +**Критерий приёмки:** +- `isMobile` пересчитывается при изменении ширины окна (проверка в DevTools: сузить окно < 768 → рендерится мобильный путь без перезагрузки). +- Слушатель события снимается в cleanup (нет утечки). +- Импорт из `react` содержит `useState` и `useEffect`. + +--- + +# РАЗДЕЛ B — NEEDS-CONTEXT (не давать локальной модели) + +Требуют понимания нескольких файлов, контрактов данных, роутинга или бизнес-логики. Оставлено для сильной модели / человека. + +--- + +## B1 — Весь `backend/services/scoring_service.py` (ядро: 7 факторов) +**Тип:** исключено по правилам. Любые правки (в т.ч. неиспользуемый импорт `Optional` на строке 6) делать вручную: файл — ядро зонального скоринга, риск задеть коэффициенты. Отдельно: неиспользуемый `Optional` — тривиально, но НЕ делегировать. + +## B2 — `backend/services/claude_service.py:174-181` · хрупкий парсинг JSON из ответа Claude +**Тип:** missing error handling. `content.split("```json")[1]...` упадёт при ином форматировании ответа модели; `json.loads` без try/except уронит весь анализ. Нужно устойчивое извлечение JSON + graceful fallback на scoring — требует понимания контракта `AnalysisResult` и того, как `analyze_case` переключается на fallback. + +## B3 — `backend/services/stats_service.py:19-39` vs `backend/routers/stats.py:19-35` · дублированная логика скоринга рекомендаций +**Тип:** simple refactor, но needs-context. Две почти одинаковые реализации `score`-логики (одна в сервисе, одна прямо в роутере) с разными правилами. Свести к одной — требует решения, какая версия «правильная», и затрагивает контракт `RecommendationRequest`. + +## B4 — Удаление/подключение мёртвых frontend-компонентов +**Тип:** dead code, needs-context (продуктовое решение «удалить или доподключить»). +Не подключены ни к одному маршруту (цепочка `index.js → App.js`): +- `components/MobileForm.jsx` **и** `components/MobileForm.tsx` (дубликаты друг друга; ни один не импортируется — живая мобильная форма это `CaseForm`), `components/MobileForm.css` +- `components/BehavioralProfile.tsx` + `.css` +- `components/AnalysisLayout/*`, `components/MapView/*`, `components/ResultPanel/*` (MapView/ResultPanel импортируются только из неиспользуемого AnalysisLayout) +- `components/admin/CasesList*`, `components/admin/CaseDetail*`, `components/admin/Statistics*`, `components/admin/HeatmapView*` — `AdminDashboard` не содержит вложенных ``, поэтому весь набор недостижим +- `components/CaseForm/Step5Resources.jsx` — не импортируется в `CaseForm.jsx` (рендерятся только Step1-4 + Step2b) + +Решение (удалить vs довести маршрутизацию `/admin/*` и шаг ресурсов) — за человеком. Внутри этих файлов есть свои баги (напр. опечатка `isPsycotypeSelected` в MobileForm.tsx:269, захардкоженные URL `http://192.168.0.99:8000` в MobileForm.jsx:49 и HeatmapView.jsx:73, NaN-гварды в ResultPanel), но чинить их бессмысленно, пока не решён вопрос удаления. + +## B5 — Рассинхрон контрактов данных между тремя поддоменами (desktop/mobile/admin) +**Тип:** bug, needs-context. Одни и те же поля кодируются по-разному: +- **gender:** `'мужской'/'женский'` (MobileForm.jsx) vs `'male'/'female'` (MobileForm.tsx, admin) vs `'М'/'Ж'` (Step1Child.jsx) vs `'m'/'f'` (backend `_normalize_case_data`) +- **ключи психотипа:** `unfamiliar_behavior`/`leadership` (Step2bPsychotype.jsx, и так же ждёт backend `detect_psychotype`) vs `unfamiliar_env`/`group_role` (MobileForm, admin CaseDetail) — из-за этого ответы психотипа десктоп-формы не отображаются в админке +- **транспорт:** `'велосипед'/'автомобиль'` (BehavioralProfile.tsx) vs `'bike'/'scooter'/'other'/'none'` (Step2Health.jsx) vs профили `'велосипед'/'самокат'` (backend scoring) +Нужна единая согласованная схема через границу frontend↔backend — сознательное проектное решение. + +## B6 — `frontend/src/components/CaseForm/CaseForm.jsx:90-129` · навигация через `window.location.href` +**Тип:** bug, needs-context. После создания кейса используется `window.location.href = '/analysis/${id}'` — полная перезагрузка и потеря SPA-состояния, хотя приложение уже в ``. Замена на `useNavigate` затрагивает роутинг и структуру компонента. + +## B7 — `frontend/src/pages/AdminDashboard.jsx:58-96` · повторная загрузка из-за `selectedCaseId` в deps +**Тип:** bug, needs-context. Эффект загрузки списка зависит от `selectedCaseId` и сам же его выставляет внутри → двойной fetch при первом рендере и после сброса фильтров. Исправление требует понимания взаимодействия эффектов и стейта дашборда. + +## B8 — `note` vs `notes` — рассогласование имени поля +**Тип:** naming/bug, needs-context. Frontend (`AdminDashboard.jsx:215-216`, `handleSave`) и backend (`schemas.py` имеет и `note`, и `notes`; роутеры вручную маппят `note→notes`) по-разному называют одно поле. Нужно унифицировать по всей цепочке (schema + роутеры + фронт) — контрактное изменение. + +## B9 — Отсутствие тестов и валидации пропсов на фронтенде +**Тип:** missing test / missing PropTypes, needs-context. Под `frontend/src` нет ни одного теста; ни один `.jsx`-компонент не использует PropTypes. Расчёт зон/времени в `AnalysisResult.jsx` и многошаговая валидация `CaseForm` не покрыты. Объём — целая инициатива, не атомарная задача. + +## B10 — Секреты и деплой (`.env`) +**Тип:** security/deploy, исключено из делегирования. `.env` содержит плейсхолдеры `ANTHROPIC_API_KEY=your_...` и слабый `JWT_SECRET=your_jwt_secret_here_change_in_production`. Установка реального ключа Claude (для выхода из fallback) и генерация стойкого `JWT_SECRET` — ручная операция с секретами. + +--- + +# Сводка + +| Категория | Кол-во | Куда | +|---|---|---| +| Atomic (backend) | A1–A9 | локальной LLM | +| Atomic (frontend) | A10–A16 | локальной LLM | +| Needs-context | B1–B10 | сильной модели / человеку | + +**Рекомендуемый порядок для локальной модели:** A8 и A7 первыми (чистка мёртвого кода и зелёный CI дают чистую базу для проверки остальных), затем backend A1–A6, затем docs A9, затем frontend A10–A16. После каждой backend-задачи прогонять `PYTHONPATH=/root/vector python -m pytest backend/tests -q`.