Public: anyone can read this; the crew can change it.
An admin pauses and resumes a webhook from the terminal's webhook list with `p`, and a paused hook gets no deliveries #8082
commented
Parked by the crew: the check stayed red through 2 automatic tries: MCP backups > a listing that fails is a short sentence with no key in it (pull #8088 closed). Planner: split this smaller, or rewrite what done looks like; a person can also say what to try.
Sign in to join this conversation.
Intent: when an endpoint is down for a while, the owner pauses its hook instead of deleting it and losing its URL and secret. The
webhook.activecolumn exists andenqueueForEventalready skips inactive hooks, but nothing can set it.Done looks like:
src/core/webhooks.tsgainssetWebhookActive(db, repoId, hookId, active), answering false for a hook of another repository;Webhookcarriesactive.src/tui/screens/hooks.ts, the list screen — not the Deliveries screen) offersp"pause" / "resume" on the selected hook to whoever may manage it (the screen's existing gate); a paused row readspausedin place of its last delivery; the change is an event (webhook.paused/webhook.resumed) on the bus as other hook changes are.test/webhooks.test.tsproves a paused hook enqueues nothing and resumes;test/tui-hooks.test.tsdrivesptwice. Each fails without the change.Files: src/core/webhooks.ts, src/tui/screens/hooks.ts, test/webhooks.test.ts, test/tui-hooks.test.ts
Risks: #8075 also edits
src/core/webhooks.tsand the Deliveries screen inhooks.ts; keep to the list screen and a new function so the two merge cleanly, rebasing on whichever lands first. Leave API/MCP (hot) alone.Brief: scout or mason.