[Debt] Split large files router into focused route modules #25

New Issue

zemion · 2026-07-11T11:39:43+02:00

zemion commented

2026-07-11 11:39:43 +02:00

govoplan-files has a very large backend router that mixes connector settings, browsing/import, upload/download, sharing, folders, deltas, transfers, and archive handling. This increases regression risk and makes focused review hard.

Hotspot: src/govoplan_files/backend/router.py is over 3,000 lines.

Acceptance: split routes by responsibility while preserving API paths, keep shared helpers in service/storage modules, and add smoke tests for route registration/key endpoints.

`govoplan-files` has a very large backend router that mixes connector settings, browsing/import, upload/download, sharing, folders, deltas, transfers, and archive handling. This increases regression risk and makes focused review hard. Hotspot: `src/govoplan_files/backend/router.py` is over 3,000 lines. Acceptance: split routes by responsibility while preserving API paths, keep shared helpers in service/storage modules, and add smoke tests for route registration/key endpoints.

zemion added the

labels 2026-07-11 11:39:43 +02:00

zemion commented

2026-07-12 01:12:18 +02:00

Codex State: progress

Summary

Added route-registration smoke coverage for connector and bulk organize endpoints so future router splits can preserve API paths.
Extracted connector settings delta response assembly helpers out of the route body, reducing the highest-risk connector route hotspot without moving paths yet.

Changed Files

src/govoplan_files/backend/router.py
tests/test_router_contract.py

Verification

same focused files test suite as above

Next / Blocked

The full router is still not physically split into focused route modules; keep this issue open for the actual route-module extraction.

## Codex State: progress ### Summary - Added route-registration smoke coverage for connector and bulk organize endpoints so future router splits can preserve API paths. - Extracted connector settings delta response assembly helpers out of the route body, reducing the highest-risk connector route hotspot without moving paths yet. ### Changed Files - `src/govoplan_files/backend/router.py` - `tests/test_router_contract.py` ### Verification - `same focused files test suite as above` ### Next / Blocked - The full router is still not physically split into focused route modules; keep this issue open for the actual route-module extraction.

Sign in to join this conversation.