fix: close acknowledgement lifecycle race gaps

Author: zarnivoop

apps/web/src/app/(sandbox)/task/[taskId]/sidebar-actions/OverflowMenu.client.test.tsx

@@ -141,8 +141,12 @@ function OverflowMenuBase({
 
   const handleMarkDone = async () => {
     try {
-      await acknowledgeResolution.mutateAsync({ taskId });
-      toast.success('Task marked done.');
+      const { changed } = await acknowledgeResolution.mutateAsync({ taskId });
+      if (changed) {
+        toast.success('Task marked done.');
+      } else {
+        toast.error('Task status changed before it could be marked done.');
+      }
     } catch (error) {
       toast.error(
         error instanceof Error ? error.message : 'Failed to mark task done.',

apps/web/src/app/(sandbox)/task/[taskId]/sidebar-actions/OverflowMenu.tsx

@@ -67,8 +67,12 @@ function MarkDoneButton({ taskId }: { taskId: string }) {
 
   const handleMarkDone = async () => {
     try {
-      await acknowledgeResolution.mutateAsync({ taskId });
-      toast.success('Task marked done.');
+      const { changed } = await acknowledgeResolution.mutateAsync({ taskId });
+      if (changed) {
+        toast.success('Task marked done.');
+      } else {
+        toast.error('Task status changed before it could be marked done.');
+      }
     } catch (error) {
       toast.error(
         error instanceof Error ? error.message : 'Failed to mark task done.',

apps/web/src/components/tasks/TaskBoard.client.test.tsx

@@ -26,6 +26,7 @@ import {
 import {
   and,
   compareAndSetTrustedRunActingUser,
+  clearTaskResolution,
   db,
   environments,
   eq,
@@ -306,6 +307,8 @@ export async function sendSandboxPromptCommand(
   let discordReplyQuoteId: string | null = null;
 
   try {
+    await clearTaskResolution(parsed.taskId);
+
     // The actor switch must land before the prompt reaches the sandbox so the
     // worker's sender-vs-acting-user guard passes (critical for runs that
     // start without an acting user, e.g. automation-started tasks).