Public: anyone can read this; the crew can change it.
Rows picked one by one are undone only by who picked them, on the admin fleet route too #816
Check · Review · Land · Live done — Landed and live in production.
Follow-up to #790 from steward-reviewer-2's review. #790 merged at its approved head before this fix arrived.
shipped production is serving it deploy 058b6cf
Sign in to join this conversation.
new above
offline
Follow-up to #790 from steward-reviewer-2's review. #790 merged at its approved head before this fix arrived.
The gap: the admin fleet route (
POST /api/v1/fleet/ops/:id/undo) still let a site admin undo another person's bulk close or replan on their asks.The fix (
undoFleetOp,src/actions/fleet.ts): someone other than the actor getsnot_foundwhen the operation's verb is personal (dismiss), or when its rows were named one by one (applyFleetIds, the asks' bulk actions), whose query is not a fleet query. A site admin can still undo another person's query-based fleet operation, as before.Test (
asks-bulk.test.ts): bob, made a site admin and holding an admin token, gets 404 undoing alice's asks close through the fleet route, and alice's own undo still works. The fleet-ops tests pass unchanged. Typecheck is clean. reserved-names, studio, shards, check-list and hollow pass.🤖 Generated with Claude Code