NF-8: Legacy dispatch removal #329

New issue

Closed

opened 2026-04-24 11:29:51 +00:00 by code-lead · 0 comments

code-lead commented

2026-04-24 11:29:51 +00:00

Collaborator

As the platform, I want to delete the hard-coded handlers in webhook-handlers.ts / webhook-ci.ts / slash-commands.ts / deps.ts so that the default graph is the single source of truth for dispatch.

Acceptance criteria

Code removal

webhook-handlers.ts, webhook-ci.ts, slash-commands.ts, deps.ts strip out every dispatch path. What remains: event normalisation + SSE broadcasting only.
Default-graph JSON becomes the only thing that turns events into side effects.

Tests

Handler tests migrate 1:1 to graph-node assertions. Most stay shape-identical; rewire the fixtures through the executor instead of the handler.

Docs

CLAUDE.md modules table updated to reflect the reorg.
docs/*.md references to the legacy handlers redirected to the flow.

Out of scope

New functionality.
UI.

References

Spec: specs/node-flows.md § Migration path, phase 3.
Depends on NF-7 (#328) + two-release soak after NF-6.

As the platform, I want to delete the hard-coded handlers in `webhook-handlers.ts` / `webhook-ci.ts` / `slash-commands.ts` / `deps.ts` so that the default graph is the single source of truth for dispatch. ## Acceptance criteria ### Code removal - [ ] `webhook-handlers.ts`, `webhook-ci.ts`, `slash-commands.ts`, `deps.ts` strip out every dispatch path. What remains: event normalisation + SSE broadcasting only. - [ ] Default-graph JSON becomes the only thing that turns events into side effects. ### Tests - [ ] Handler tests migrate 1:1 to graph-node assertions. Most stay shape-identical; rewire the fixtures through the executor instead of the handler. ### Docs - [ ] `CLAUDE.md` modules table updated to reflect the reorg. - [ ] `docs/*.md` references to the legacy handlers redirected to the flow. ## Out of scope - New functionality. - UI. ## References - Spec: [`specs/node-flows.md`](../src/branch/main/specs/node-flows.md) § Migration path, phase 3. - Depends on NF-7 (#328) + two-release soak after NF-6.

code-lead added this to the M21 — Node Flows milestone

2026-04-24 11:29:51 +00:00

code-lead added the

area:webhook

type:user-story

labels

2026-04-24 11:29:51 +00:00

code-lead referenced this issue

2026-04-24 11:29:57 +00:00

M21 Tracking — Node Flows #340

code-lead was assigned by claude-desktop

2026-04-27 08:48:03 +00:00