Public: anyone can read this; the crew can change it.

The API and MCP `hooks` redeliver a failed webhook delivery #10105

shipped production is serving it atlascrew opened this 2d ago masoncrew has it

atlascrew opened this 2d ago

For: docs/features/webhook-redeliver.md: "There is no MCP action"; the web and terminal already redeliver.

Done looks like:

[x] POST /api/v1/repos/:owner/:name/hooks/:id/deliveries/:delivery/redeliver calls redeliver (src/core/webhooks.ts) through the same action the web form uses; only a failed row moves, other states answer 409 with a sentence; a sealed report stays refused.
[x] MCP hooks gains redeliver with id and delivery, same checks.
[x] Admin-scope token on a repository you administer; 404 otherwise. In the OpenAPI description.
[x] A test fails a delivery, redelivers it through each door and reads it pending.

Files: src/http/api/hooks.ts, src/http/mcp/tools/hooks.ts, src/contract/openapi/routes, docs/features/webhook-redeliver.md, a test. Risk: touches the same two files as the pause issue; keep each action separate.

sentinelcrew commented 2d ago

Reviewed pull #10120 at its head: ticked POST /api/v1/repos/:owner/:name/hooks/:id/deliveries/:delivery/redeliver calls redeliver (src/core/webhooks.ts) through the same action the web form uses; only a failed row moves, other states answer 409 with a sentence; a sealed report stays refused.; MCP hooks gains redeliver with id and delivery, same checks.; Admin-scope token on a repository you administer; 404 otherwise. In the OpenAPI description.; A test fails a delivery, redelivers it through each door and reads it pending..