MC-4503 — Event-driven Controller hook path for Control Room v2

Home Board MC-4503

Event-driven Controller hook path for Control Room v2

Implement deterministic event detection/trigger path so ticket/comment/status changes call Controller only when material work exists. Acceptance: no LLM-driven noisy polling; no...

State Done Next Action Waiting: done Owner Luci Runtime Closed Age 12d ago

Description

MC-4503

Implement deterministic event detection/trigger path so ticket/comment/status changes call Controller only when material work exists. Acceptance: no LLM-driven noisy polling; no auto-runtime spawn on comments; clear audit rows; tests for human comment, system comment, status change, idempotency, and quiet/no-op.

Activity

done

Luci is working...

Details — Done · High · Luci ▼

State

Done

Waiting: done

State

Priority

People

Owner (assigned to)

Controller

Timing / Details

Due Date

Snooze

Source telegram (human)

Project -

Created 12d ago

Updated 12d ago

Advanced / Operator evidence

Routing owner

Routes via

Operator console

Evidence

Ticket is done; runtime is closed. tmux_session_missing profile claude_opus_1m_high · cwd /home/lucienne/workspace/state/control-room-worktrees/mc-4503-event-hooks · uptime 12d 10h · last activity 12d 10h ago MC is visibility-only. Hermes Luci launches and gates work outside MC, then mirrors evidence/status here. Raw console: luci · claude/anthropic/opus[1m]

Workflow

Start Dev Review + QA ▾

Select phases to include:

Research (scott) Implement (larry) (required) Council Review (council) Code Review (luci) Validate (tessa) (required) Sign-off (atlas)

Agents

Review Gates

Decision

WAT routing: choose an agent, review gate, or decision. Buttons use the live runtime when one is attached.

controller 12d ago

Created as remaining Control Room v2 rollout gate. Must follow Planner/Coder/Reviewer/Tester/Validator gates, TDD for behavior changes, independent review, and live validation before Done.

controller 12d ago

Coder gate dispatched: Claude Code Opus/high in isolated worktree mc-4503-event-hooks. Acceptance: default-off Controller event hook path, no auto-dispatch, tests/docs required.

Controller decision

validator 12d ago

Validator gate complete — MC-4503 landed. Commit: 51d5574 Branch: cr/mc-4503-event-hooks Master: fast-forwarded and pushed to origin/master. Evidence: - Coder gate completed commit 077d4ca, rebased to 51d5574 after docs commit. - Focused validation: `python3 -m pytest tests/test_control_room_event_hooks.py tests/test_mc4499_controller_notify.py tests/test_mc4500_comment_controller_notify.py tests/test_mc4501_comment_author_type_defaults.py -q` → 83 passed + 11 subtests. - Compile: `python3 -m py_compile app.py models.py control_room_events.py` → passed. - Independent review verdict: APPROVED. - graphify update completed. Scope shipped: - New default-off `MC_CONTROL_ROOM_V2_CONTROLLER_HOOKS` flag, separate from notify flag. - Hooks-off = durable audit only. - Hooks-on = exactly one `persistent_luci.send(..., source="control_room_event_hook")` after durable audit success for human comment and WAT paths. - Atomic audit failure sends nothing; hook send failure does not fail original API request. - Hook prompt explicitly says Controller decides and must not auto-dispatch blindly. - No pickup/status/assignee mutation or inline worker dispatch from hook path.

Controller decision