?
TRCKR-4412
refactor server tech-debt feature
Created: 2026-01-02 Updated: 2026-01-03
Relationships Loading...
Attachments
Loading...
Comments (3)
QA Agent · 2026-01-03
[QA] Verification complete. All structural checks passed: - ✓ server/routes/web/milestones.py created (414 lines) - ✓ All 8 routes properly implemented and registered - ✓ routes.py refactored (-370 lines) - ✓ Module imports without errors - ✓ Code follows established patterns Limitations: - E2E tests not run (require Docker) - Browser testing not performed See /tmp/qa-TRCKR-4412.md for full report. Recommendation: Deploy to staging and run E2E suite before production.
agent · 2026-01-03
## Implementation Complete **Changes Made:** 1. Created `server/routes/web/milestones.py` with 8 milestone-related routes: - GET /milestones/new - New milestone form modal - POST /milestones - Create milestone - POST /milestones/bulk - Bulk operations on milestones - GET /milestones - List milestones page with filtering/sorting - GET /milestones/{milestone_id} - Milestone detail view - PATCH /milestones/{milestone_id} - Update milestone - DELETE /milestones/{milestone_id} - Delete milestone - POST /milestones/{milestone_id}/comments - Add comment to milestone 2. Updated `server/routes/web/routes.py`: - Added import and include for milestones router - Removed milestone route definitions (~370 lines) - Cleaned up unused imports (resolve_milestone_id, resolve_milestone_ids) 3. Updated `server/routes/web/__init__.py`: - Updated docstring to document new module structure **Verification:** - All imports compile successfully - 1590 unit tests pass (12 failures are pre-existing, unrelated to these changes) - Code follows the same pattern established by issues.py and projects.py **Files Modified:** - `server/routes/web/milestones.py` (new file, 413 lines) - `server/routes/web/routes.py` (2741 -> 2371 lines, -370 lines) - `server/routes/web/__init__.py` (updated docstring)
triage-agent · 2026-01-03
## Feature Triage Complete **Status:** NOT IMPLEMENTED - Ready for implementation **Summary:** This is a straightforward refactoring task to extract milestone web routes from `routes.py` (~2741 lines) to a dedicated `milestones.py` module. The pattern is already established with `issues.py` and `projects.py`. **Routes to Extract:** 8 routes (~250-300 lines) - GET/POST /milestones (list + create) - GET/PATCH/DELETE /milestones/{id} (detail + update + delete) - POST /milestones/bulk (bulk operations) - GET /milestones/new (form modal) - POST /milestones/{id}/comments (add comment) **Implementation Location:** - Create: `server/routes/web/milestones.py` - Modify: `server/routes/web/routes.py` (remove routes, add include) **Complexity:** Low - Copy/paste following existing patterns **Test Coverage:** 22 E2E tests covering all milestone operations See attached triage report for full details.