audit the whole app for stupid cases of missing functionality or basic things like this (like the stats panel at the top that i requested).
What is wrong: the three delete confirmations on the webhook page (templates/source_detail.html) are generic: "Delete this webhook and all its data?", "Delete this entrypoint?" and "Delete this target?". None of them names the thing being deleted.
The webhook prompt says all its data goes, but its archive files stay on disk (owner-confirmed, #376). It also does not say how many stored events are about to be destroyed.
The entrypoint prompt does not say that its URL stops working for good. A new entrypoint gets a new URL, so a deleted one cannot be restored.
Definition of done:
Each confirmation names the item, for example: Delete webhook "Orders"?
Each one says what is lost:
webhook: its stored events and their deliveries, with the number of events, while its archive files are kept;
entrypoint: senders using this URL get an error from now on, and the URL cannot be restored;
target: nothing more is delivered to it, and its past deliveries stay in the event log.
Each confirmation works without the page's script library, as the current browser prompt does.
Tests: each prompt's text contains the item's name and the event count where it applies.
PRIORITY: from the owner's audit request of 1 October (#377), in the tier of #367 to #376.
Model: opus-5-5
Owner's request, https://git.eeqj.de/sneak/webhooker/issues/377 (chat, 2026-10-01):
> audit the whole app for stupid cases of missing functionality or basic things like this (like the stats panel at the top that i requested).
What is wrong: the three delete confirmations on the webhook page (`templates/source_detail.html`) are generic: "Delete this webhook and all its data?", "Delete this entrypoint?" and "Delete this target?". None of them names the thing being deleted.
- The webhook prompt says all its data goes, but its archive files stay on disk (owner-confirmed, https://git.eeqj.de/sneak/webhooker/issues/376). It also does not say how many stored events are about to be destroyed.
- The entrypoint prompt does not say that its URL stops working for good. A new entrypoint gets a new URL, so a deleted one cannot be restored.
Definition of done:
- Each confirmation names the item, for example: Delete webhook "Orders"?
- Each one says what is lost:
- webhook: its stored events and their deliveries, with the number of events, while its archive files are kept;
- entrypoint: senders using this URL get an error from now on, and the URL cannot be restored;
- target: nothing more is delivered to it, and its past deliveries stay in the event log.
- Each confirmation works without the page's script library, as the current browser prompt does.
- Tests: each prompt's text contains the item's name and the event count where it applies.
PRIORITY: from the owner's audit request of 1 October (https://git.eeqj.de/sneak/webhooker/issues/377), in the tier of https://git.eeqj.de/sneak/webhooker/issues/367 to https://git.eeqj.de/sneak/webhooker/issues/376.
Model: opus-5-5
clawbot
self-assigned this 2026-10-01 22:06:26 +02:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Owner's request, #377 (chat, 2026-10-01):
What is wrong: the three delete confirmations on the webhook page (
templates/source_detail.html) are generic: "Delete this webhook and all its data?", "Delete this entrypoint?" and "Delete this target?". None of them names the thing being deleted.Definition of done:
PRIORITY: from the owner's audit request of 1 October (#377), in the tier of #367 to #376.
Model: opus-5-5