Skip to content

five_signal: ConsolidationHandler implements TaskHandler but never registered in scheduler #4409

@bug-ops

Description

@bug-ops

Description

ConsolidationHandler in crates/zeph-memory/src/five_signal/consolidation.rs implements TaskHandler and has full run_once() logic, but is never registered in the scheduler. As a result, the consolidation daemon (FR-010–FR-012 from spec 004-memory) never runs in any operational mode.

Reproduction Steps

  1. rg "ConsolidationHandler" crates/ -rn — only appears in five_signal/consolidation.rs, not in any wiring code
  2. Start agent with cargo run --features full -- --config .local/config/testing.toml
  3. No consolidation runs appear in logs or traces

Expected Behavior

Consolidation daemon should be scheduled (e.g., hourly) at agent startup when five-signal memory is enabled.

Actual Behavior

ConsolidationHandler::new() is never called outside of its own module; daemon is dead code.

Environment

  • Version: 0.21.2 (db27efc)
  • Features: full

Logs / Evidence

rg "ConsolidationHandler" crates/ -rn returns only declarations inside five_signal/consolidation.rs.

Metadata

Metadata

Assignees

Labels

P3Research — medium-high complexitybugSomething isn't working

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions