diff options
Diffstat (limited to 'SESSION_STATE.md')
| -rw-r--r-- | SESSION_STATE.md | 60 |
1 files changed, 60 insertions, 0 deletions
diff --git a/SESSION_STATE.md b/SESSION_STATE.md new file mode 100644 index 0000000..6d599f1 --- /dev/null +++ b/SESSION_STATE.md @@ -0,0 +1,60 @@ +# SESSION_STATE.md + +## Current Task Goal +Add `project` field to tasks — allow tasks to carry a project label displayed in the UI. + +## Status: COMPLETED + +--- + +## Completed Items + +| Step | Description | Test / Verification | +|------|-------------|---------------------| +| 1 | Add `Project` field to `Task` struct and YAML parsing (`internal/task/task.go:77`) | `TestTask_ParseFile` (existing, passing) | +| 2 | Add `project` column to SQLite schema and all CRUD queries (`internal/storage/db.go`) | `TestDB_*` suite in `internal/storage` — all passing | +| 3 | Expose `project` in REST API create/list/get endpoints (`internal/api/`) | `internal/api` suite — all passing | +| 4 | Expose `project` in CLI (`internal/cli/`) | `internal/cli` suite — all passing | +| 5 | Display `project` badge on task cards in web UI (`web/app.js:159-163`) | Visual; `grep -n project web/app.js` confirms presence | + +### Commits +``` +054ec8b feat: add Project field to Task struct and YAML parsing +072652f feat: add project column to storage +26dc313 feat: expose project field in API and CLI +b838150 feat: display project field in web UI +``` + +--- + +## Test Suite Results (2026-03-16) + +``` +ok internal/api 5.664s +ok internal/cli 6.136s +ok internal/config 1.023s +ok internal/notify 1.024s +ok internal/reporter 1.017s +ok internal/storage 1.421s +ok internal/task 1.022s +FAIL internal/executor 1.391s ← PRE-EXISTING failures (unrelated) +``` + +### Pre-existing failures in `internal/executor` (not caused by this feature) + +- `TestTeardownSandbox_AutocommitsChanges` — `git log in bare repo: exit status 128` +- `TestTeardownSandbox_BuildSuccess_ProceedsToAutocommit` — `git log in bare repo: exit status 128` +- `TestPool_RecoverStaleRunning` — `execution status: want FAILED, got [...]` + +These fail on commits predating the project field feature (verified by running tests against `local/master`). + +--- + +## Next Steps +None — feature is complete. Pre-existing executor failures are tracked separately. + +--- + +## Process Notes +- Schema migration uses `ALTER TABLE tasks ADD COLUMN project TEXT` (idempotent via existing migration guard). +- `scanTask` uses `sql.NullString` for backward compatibility with existing rows that have no project value. |