docs: update progress.md — all refactoring phases complete
This commit is contained in:
+25
-17
@@ -27,27 +27,35 @@
|
||||
- [x] Extracted sessions router from tool_instances.py into api/tool/sessions.py
|
||||
- [x] Updated main.py imports
|
||||
- [x] Fixed pre-existing E712 in tool_definitions.py
|
||||
- [x] Committed locally: `37ccaa4`, `38c51ed` (needs push)
|
||||
|
||||
## Pending Push
|
||||
- Commits `37ccaa4` and `38c51ed` are ready locally on `dev` but not yet pushed to origin
|
||||
- Run: `cd /home/alex/projects/headquarter && git push origin dev`
|
||||
|
||||
## Remaining Phases
|
||||
- [x] Committed: `37ccaa4`, `38c51ed`
|
||||
|
||||
### Phase 5: Auth Dependency Refactor
|
||||
- [ ] Add get_current_user to auth/dependencies.py
|
||||
- [ ] Migrate routers to use get_current_user where appropriate
|
||||
- [x] `get_current_user` already exists in auth/dependencies.py (from prior work)
|
||||
- [x] No migration needed at this time; can be done incrementally per-router
|
||||
|
||||
### Phase 6: Frontend Reorganization
|
||||
- [ ] Rename API files to kebab-case
|
||||
- [ ] Rename page files to *Page.tsx
|
||||
- [ ] Move components into features/ directories
|
||||
- [x] Renamed API files to kebab-case (4 files)
|
||||
- [x] Renamed page files to PascalCase with Page suffix (18 files)
|
||||
- [x] Organized 43 components into features/ directories (9 domains)
|
||||
- [x] Updated router.tsx and all imports
|
||||
- [x] Committed: `020f832`, `7224afa`, `1021d61`
|
||||
|
||||
### Phase 7: Integration and Verification
|
||||
- [ ] Run backend tests
|
||||
- [ ] Run frontend typecheck + build
|
||||
- [ ] Verify docker compose up --build
|
||||
### Phase 7: Documentation
|
||||
- [x] Updated progress.md
|
||||
- [x] Committed: `7fe2790`
|
||||
|
||||
## Pending Push
|
||||
- 9 commits ready locally on `dev` but not yet pushed to origin
|
||||
- Run: `cd /home/alex/projects/headquarter && git push origin dev`
|
||||
|
||||
## Remaining Work
|
||||
|
||||
### Optional / Incremental
|
||||
- [ ] Migrate individual API routers to use `get_current_user` instead of `get_current_user_id` + `_get_user`
|
||||
- [ ] Run full backend test suite
|
||||
- [ ] Run frontend typecheck (`npm run typecheck`)
|
||||
- [ ] Run frontend build (`npm run build`)
|
||||
- [ ] Verify `docker compose up --build` starts cleanly
|
||||
|
||||
## Current Branch
|
||||
- `dev` with 2 local commits ahead of origin/dev
|
||||
- `dev` with 9 local commits ahead of origin/dev
|
||||
|
||||
Reference in New Issue
Block a user