1.5 KiB
1.5 KiB
Task domain consolidation into the Pod built-in Task feature is complete and merged.
Implementation commits:
5469335 refactor: move task feature into podc2ed71a docs: update tui task snapshot comments- merge commit on
develop: seemerge: move task domain into pod feature
Summary:
- Moved Task domain types and store into
pod::feature::builtin::task. - Moved Task tool implementations into the Pod Task feature module.
- Removed production Task APIs from
tools, includingtools::TaskStore, Task type re-exports,tools::task_tools, and Task-bearingbuiltin_tools(...). - Kept non-Task
tools::core_builtin_tools(...)and tracker/non-Task tool behavior intact. - Preserved Task tool schema/output behavior, Task snapshot/replay semantics, reminder hooks/state, and snapshot/restore/rewind/compaction façade.
- Updated TUI compatibility tests to use local fixtures without adding a production dependency on
podortools.
Review:
- External sibling reviewer approved with no blockers.
- Reviewer non-blocker about stale TUI comments was fixed before merge.
- Remaining accepted residual risk: local TUI fixtures can drift if Pod-owned Task snapshot format changes.
Post-merge validation passed on main workspace:
cargo test -p pod task --libcargo test -p tools --libcargo test -p tools --testscargo test -p tui task --libcargo fmt --checkgit diff --check./tickets.sh doctorcargo check --workspace --all-targets- search for remaining production
toolsTask APIs returned no matches