fix: allow non-email action types (generate_packet, manual_packet, etc.) without external adapter

apps/worker/src/lib/action-executor.ts

@@ -179,29 +179,39 @@ export async function executeAction(
     return { attemptId: attempt.id, attemptNo: (latestAttempt?.attemptNo ?? 0) + 1 };
   });
 
-  const submissionResult = await emailAdapter.submit({
-    missionId: action.missionId,
-    actionId,
-    destination: inputPayload.destination,
-    subject: inputPayload.subject,
-    body: inputPayload.body,
-    attachments: await (async () => {
-      if (inputPayload.attachments.length === 0) return [];
-      // Look up actual storage keys from evidence files table
-      const evidenceRows = await txDb
-        .select({ id: schema.evidenceItems.id, storageKey: schema.files.storageKey })
-        .from(schema.evidenceItems)
-        .innerJoin(schema.files, eq(schema.evidenceItems.fileId, schema.files.id))
-        .where(eq(schema.evidenceItems.missionId, action.missionId));
-      const storageKeyMap = new Map(evidenceRows.map((r) => [r.id, r.storageKey]));
-      return inputPayload.attachments.map((a) => ({
-        id: a.id,
-        storageKey: storageKeyMap.get(a.id) ?? a.sha256,
-        filename: a.id || "attachment",
-      }));
-    })(),
-    idempotencyKey: action.idempotencyKey ?? `action-${actionId}`,
-  });
+  // Determine submission strategy based on action type.
+  // Email-type actions use the email adapter; local/user-driven actions mark as
+  // ready_for_user_submission (no external call).
+  const isEmailAction = action.actionType === "send_email" || action.actionType === "takedown";
+  const submissionResult = isEmailAction
+    ? await emailAdapter.submit({
+        missionId: action.missionId,
+        actionId,
+        destination: inputPayload.destination,
+        subject: inputPayload.subject,
+        body: inputPayload.body,
+        attachments: await (async () => {
+          if (inputPayload.attachments.length === 0) return [];
+          const evidenceRows = await txDb
+            .select({ id: schema.evidenceItems.id, storageKey: schema.files.storageKey })
+            .from(schema.evidenceItems)
+            .innerJoin(schema.files, eq(schema.evidenceItems.fileId, schema.files.id))
+            .where(eq(schema.evidenceItems.missionId, action.missionId));
+          const storageKeyMap = new Map(evidenceRows.map((r) => [r.id, r.storageKey]));
+          return inputPayload.attachments.map((a) => ({
+            id: a.id,
+            storageKey: storageKeyMap.get(a.id) ?? a.sha256,
+            filename: a.id || "attachment",
+          }));
+        })(),
+        idempotencyKey: action.idempotencyKey ?? `action-${actionId}`,
+      })
+    : {
+        proofState: "ready_for_user_submission" as const,
+        proofRef: { note: "Local action — user must submit manually." },
+        providerMessageId: null,
+        externalReference: null,
+      };
 
   const succeeded = submissionResult.proofState !== "failed";
 

packages/domain/src/policies/action-execution-policy.ts

@@ -33,16 +33,12 @@ export type CanExecuteResult = ExecutionRefusal | ExecutionAllowance;
 // ---------------------------------------------------------------------------
 
 function adapterForAction(action: Action): string | null {
-  switch (action.actionType) {
-    case "send_email":
-      return "resend_email";
-    case "post_social":
-      return "social_publisher";
-    case "generate_packet":
-      return "packet_generator";
-    default:
-      return null;
-  }
+  const t = action.actionType;
+  if (t === "send_email" || t === "takedown") return "resend_email";
+  if (t === "post_social") return "social_publisher";
+  // All other types (generate_packet, manual_packet, prepare_portal,
+  // submit_portal, inform_user, etc.) are local/user-driven — no adapter needed.
+  return null;
 }
 
 function capabilityIsAvailable(cap: AdapterCapability): boolean {