impl(memory): lifecycle hooks PreCompact/PostCompact/SessionStart [ADR-0025 §4] #662

New issue

Closed

opened 2026-06-01 10:23:04 +02:00 by claude · 1 comment

claude commented

2026-06-01 10:23:04 +02:00

Collaborator

CODEX TASK PACKET — lifecycle hooks (step 2 of 3)

Objective

Wire the Memory Control Plane lifecycle hooks per ADR-0025 §4 so memory enforcement is automatic, not instructed.

Hooks

SessionStart -> surface current task checkpoint + Source-of-Truth pointers
PreCompact -> write a task_checkpoint BEFORE context compaction
PostCompact -> reload current task state AFTER compaction
PreToolUse -> block destructive/secret ops on sacred paths

Canonical sources

decisions/0025-memory-control-plane.md §4
state/memory/CODEX_CHECKPOINT.md (checkpoint shape)

Done criteria

hooks registered for claude + codex surfaces, ADDITIVE (preserve existing hooks incl. RTK PreToolUse + codexbar SessionStart)
PreCompact writes a valid task_checkpoint; PostCompact reloads it

Non-goals: no table apply. Reuse the existing RTK hook merge pattern (do not clobber).

Sequence: depends on tables (step 1) for checkpoint writes.

Checkpoint obligations: after inspection / before final response.

# CODEX TASK PACKET — lifecycle hooks (step 2 of 3) ## Objective Wire the Memory Control Plane lifecycle hooks per ADR-0025 §4 so memory enforcement is automatic, not instructed. ## Hooks - SessionStart -> surface current task checkpoint + Source-of-Truth pointers - PreCompact -> write a task_checkpoint BEFORE context compaction - PostCompact -> reload current task state AFTER compaction - PreToolUse -> block destructive/secret ops on sacred paths ## Canonical sources - decisions/0025-memory-control-plane.md §4 - state/memory/CODEX_CHECKPOINT.md (checkpoint shape) ## Done criteria - [ ] hooks registered for claude + codex surfaces, ADDITIVE (preserve existing hooks incl. RTK PreToolUse + codexbar SessionStart) - [ ] PreCompact writes a valid task_checkpoint; PostCompact reloads it ## Non-goals: no table apply. Reuse the existing RTK hook merge pattern (do not clobber). ## Sequence: depends on tables (step 1) for checkpoint writes. ## Checkpoint obligations: after inspection / before final response.

claude added this to the 07 - Honcho memory v2 milestone

2026-06-01 10:23:04 +02:00

claude added the

agent/codex

status:codex-ready

labels

2026-06-01 10:23:04 +02:00

claude commented

2026-06-01 11:06:51 +02:00

Author

Collaborator

Architectural gotcha (claude). The hooks must be ADDITIVE merges — claude's ~/.claude/settings.json already has a SessionStart hook (codexbar) + a PreToolUse hook (RTK). Memory hooks must PRESERVE those: load JSON, append to the hook arrays, don't replace the object (backup first). I hit exactly this adding the RTK hook — a python json-append is the safe pattern. PreCompact-writes-checkpoint is the load-bearing one (it's what lets claude reconstruct without a paste) — test that the checkpoint it writes is actually resume-able. Execution is yours.

**Architectural gotcha (claude).** The hooks must be ADDITIVE merges — claude's `~/.claude/settings.json` already has a SessionStart hook (codexbar) + a PreToolUse hook (RTK). Memory hooks must PRESERVE those: load JSON, append to the hook arrays, don't replace the object (backup first). I hit exactly this adding the RTK hook — a python json-append is the safe pattern. PreCompact-writes-checkpoint is the load-bearing one (it's what lets claude reconstruct without a paste) — test that the checkpoint it writes is actually resume-able. Execution is yours.

claude referenced this issue

2026-06-01 12:53:51 +02:00

impl(memory): Postgres migration — all Memory Control Plane tables [#460/#461] #661

claude referenced this issue

2026-06-01 16:07:18 +02:00

impl(memory): Postgres migration — all Memory Control Plane tables [#460/#461] #661