TRCKR-4408 - trckr

← TRCKR-4408

refactor

Created: 2026-01-02 • Updated: 2026-01-03

Relationships Loading...

## Overview
Move `statuses.py` from project root to `shared/statuses.py` as foundation for other refactoring work.

## Current State
- `statuses.py` at project root
- Imported by 9 files across CLI, server, TUI, client, and shared

## Scope
1. Move `statuses.py` → `shared/statuses.py`
2. Update all imports:
   - `tui/constants.py`
   - `tui/screens/edit.py`
   - `tui/dialogs.py`
   - `server/models.py`
   - `server/database.py`
   - `shared/models.py`
   - `trckr.py`
   - `client/migration.py`
   - `client/schema.py`
3. Run full test suite

## QA Instructions
```bash
# Verify file moved
test -f shared/statuses.py && echo "PASS: File exists" || echo "FAIL: File missing"
test ! -f statuses.py && echo "PASS: Old file removed" || echo "FAIL: Old file still exists"

# Verify imports updated (should return nothing)
grep -r "from statuses import" --include="*.py" | grep -v __pycache__ && echo "FAIL: Old imports found" || echo "PASS: No old imports"

# Run tests
make test && echo "PASS: Tests pass" || echo "FAIL: Tests failed"

# Verify CLI works
trckr --help && echo "PASS: CLI works" || echo "FAIL: CLI broken"
```

## Acceptance Criteria
- [ ] `shared/statuses.py` exists with same content
- [ ] No `statuses.py` at project root
- [ ] All imports updated to `from shared.statuses import`
- [ ] `make test` passes
- [ ] `trckr --help` works
- [ ] `trckr-server` starts without errors

Pull Requests (comma-separated URLs)

Attachments

Comments (2)

agent · 2026-01-03

Implementation complete: 1. Moved `statuses.py` from project root to `shared/statuses.py` 2. Updated all 9 import statements: - `tui/constants.py` - `tui/dialogs.py` - `tui/screens/edit.py` - `trckr.py` - `shared/models.py` - `server/models.py` - `server/database.py` - `client/migration.py` - `client/schema.py` 3. Verified: - No old imports remain - Test suite passes (1595 passed, 7 pre-existing failures unrelated) - CLI works Ready for merge.

triage-agent · 2026-01-03

## Feature Triage Complete **Status:** NOT IMPLEMENTED **Summary:** This is a straightforward file reorganization task. The `statuses.py` file (330 lines of status/priority definitions) needs to be moved from project root to `shared/statuses.py`, and 9 import statements across CLI, TUI, server, and client modules need updating. **Implementation Location:** - Move `statuses.py` -> `shared/statuses.py` - Update imports in: tui/constants.py, tui/screens/edit.py, tui/dialogs.py, server/models.py, server/database.py, shared/models.py, trckr.py, client/migration.py, client/schema.py **Complexity:** LOW - No logic changes, just file organization **Risk Assessment:** - No circular import issues (statuses.py has no trckr module dependencies) - No database/API changes needed - Similar pattern already used for identifiers.py move (TRCKR-2337) See attached document for full triage report.