feat(gps-tracks): GPX download from public track popup (ET-011) #21

Merged

admin merged 9 commits from feature/ET-011-popup-enduro-trails into main 2026-06-04 02:08:45 +03:00

Conversation 0 Commits 9 Files Changed 30 +6143 -21

9 Commits

Author	SHA1	Message	Date
claude-bot	d2bc769160	tester(ET): auto-commit from tester run_id=70	2026-06-03 23:08:11 +00:00
claude-bot	ff18afed8c	reviewer(ET): auto-commit from reviewer run_id=69	2026-06-03 23:04:25 +00:00
claude-bot	721b33a2f6	fix(gps-tracks): address ET-011 review — JS UI tests + flat 403 contract ... Закрывает findings из docs/work-items/ET-011/12-review.md (REQUEST_CHANGES, попытка 3/3): P1-01 — добавлены поведенческие JS unit-тесты UI download-flow - tests/web/track_download.test.js — 28 кейсов (node --test): • _parseFilenameFromCD — RFC 5987 приоритет, plain fallback, битый percent-encoding, null/empty (REQ-F-05.2, AC-2 UI) • _handleDownloadError — 400/403/404/413/5xx тосты, defensive при отсутствии showToast, поддержка flat (ADR-015 §G) и legacy wrapped 403-форм (REQ-F-05.4, AC-7 UI) • _renderTrackPopupHtml — наличие кнопки, aria-label «Скачать GPX», data-track-id, отсутствие при невалидном id, регрессия прочих полей (REQ-F-01, AC-1) - tests/web/test_track_download.py — pytest-обёртка (статические проверки + запуск Node-раннера), исполняется в обычном pytest tests/ - 04b-ui-test-cases.md: AC-13 (mobile-bbox) явно маркирован как MANUAL release-smoke (Playwright-раннер в проекте не настроен; альтернатива согласована reviewer'ом в P1-01). P2-01 — устранено расхождение «doc vs runtime» по контракту 403 - endpoint.py: HTTPException(detail={...}) → JSONResponse(content={...}), чтобы FastAPI не оборачивал dict во второй слой «detail». Контракт теперь совпадает с ADR-015 §G и ADR-014 §6: {"detail":"source_forbidden","external_urls":[...]} - test_gps_tracks_download.py IT-05: упрощено — body уже плоский, без двухуровневого `body.get("detail", body)` workaround. - gps_tracks.js::_handleDownloadError: flat-форма стала приоритетной, wrapped-форма оставлена как defensive fallback (с комментарием). Регрессия: 89/89 API-тестов + 24/24 предыдущих JS-тестов + 28 новых JS-тестов download-flow проходят. ruff check — clean. Refs: ET-011 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-06-03 23:01:19 +00:00
claude-bot	716bff3126	reviewer(ET): auto-commit from reviewer run_id=67	2026-06-03 22:53:53 +00:00
claude-bot	7d8407a378	fix(ci): hoist imports to satisfy E402 + declare runtime/test deps in pyproject ... CI failures на feature/ET-011 были вызваны двумя проблемами: 1. ruff `E402 Module level import not at top of file` × 10 в src/api/main.py: - 9 ошибок от ET-008 (GPS_TRACKS_DB_PATH между импортами) + 1 новая от ET-011 (`from src.api.gps_tracks.endpoint import ...` после определения `app`). Перенёс все импорты наверх; константы GPS_TRACKS_DB_PATH и GPS_SOURCES_CONFIG_PATH теперь сразу после import-блока, а создание router-а остаётся в нижней части файла (зависит от `app`). 2. pyproject.toml не объявлял runtime-deps, которые реально импортируются в src/ (defusedxml, pyyaml) и в тестах (lxml). Dockerfile брал их из src/api/requirements.txt, но CI jobs `lint`/`test` ставят `.[dev]` — поэтому `pytest tests/` падал на ModuleNotFoundError при коллекции тестов из ET-008/ET-009/ET-011. Добавил недостающие пины в pyproject (defusedxml/pyyaml в основные deps, lxml — только в dev, нужен для XSD-валидации в test_gps_tracks_download/_gpx_builder). Проверено локально в чистом venv после `pip install .[dev]`: - `ruff check src/` → All checks passed - `pytest tests/` → 200 passed, 2 deselected Refs: ET-011 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-06-03 22:48:27 +00:00
claude-bot	eea6c846c2	feat(gps-tracks): GPX download from public track popup ... Реализация ET-011: кнопка «Скачать GPX» в popup публичного GPS-трека и новый эндпоинт GET /api/gps-tracks/{track_id}/download (GPX 1.1 + Content-Disposition с UTF-8 именем по RFC 5987). Реэкспорт защищён per-source флагом `download_allowed` в `config/gps_sources.yaml` (default-deny, MVP whitelist = `osm`). Backend: - `src/api/gps_tracks/export.py` — чистый stdlib-builder GPX 1.1 (`build_gpx`) + санитизация имени файла (`safe_filename`, RFC 5987). - `src/api/gps_tracks/endpoint.py` — новый route с проверками 400 / 403 / 404 / 413; cap 200 000 точек (REQ-NF-02). - `src/api/gps_tracks/config.py` — `load_download_allowed_sources()` читает YAML, default-deny при отсутствии поля; fallback на `{"osm"}` при отсутствии конфига. - `src/api/main.py` — пробрасывает `GPS_SOURCES_CONFIG_PATH` в router. Frontend: - `src/web/gps_tracks.js` — кнопка в `_renderTrackPopupHtml`, обработчик `_downloadPublicTrack` (fetch + Blob + a.download — тот же паттерн, что в `app.js::downloadGPX`, R-1 митигирован), парсер `_parseFilenameFromCD` для RFC 5987, маппинг ошибок `_handleDownloadError` (403/404/413/5xx → showToast). - `src/web/app.css` — стиль кнопки, 32×32 CSS px (REQ-NF-04). Тесты: - 13 unit для GPX-builder (UT-01/02/03/05; XSD-валидация против `tests/fixtures/gpx-1.1/gpx.xsd`). - 10 unit для `safe_filename` (UT-04). - 11 integration для download-эндпоинта (IT-01..08 + ANY-rule license check + default-deny без конфига). ADR-014 (gpx-download-endpoint), ADR-015 (source-redistribution-policy). Refs: ET-011 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-06-03 20:59:53 +00:00
claude-bot	6fe2ecf12b	architect(ET): auto-commit from architect run_id=64	2026-06-03 20:44:55 +00:00
claude-bot	2bf08a10e3	analyst(ET): auto-commit from analyst run_id=63	2026-06-03 20:05:12 +00:00
Slava	44b7af9ad0	docs: init ET-011 business request	2026-06-03 22:59:55 +03:00