@minesa-org/mini-interaction 0.2.22 → 0.2.24

package/dist/clients/MiniInteraction.d.ts

@@ -37,6 +37,25 @@ export type InteractionHandlerResult = {
     body: APIInteractionResponse | {
         error: string;
     };
+    /**
+     * Promise that resolves when all background work (like editReply) completes.
+     * Pass this to Vercel's waitUntil() to prevent premature termination.
+     *
+     * @example
+     * ```typescript
+     * // In Next.js App Router
+     * import { waitUntil } from '@vercel/functions';
+     *
+     * export async function POST(request: Request) {
+     *   const result = await client.handleRequest({ ... });
+     *   if (result.backgroundWork) {
+     *     waitUntil(result.backgroundWork);
+     *   }
+     *   return Response.json(result.body, { status: result.status });
+     * }
+     * ```
+     */
+    backgroundWork?: Promise<void>;
 };
 /** Configuration for interaction timeout handling. */
 export type InteractionTimeoutConfig = {

package/dist/clients/MiniInteraction.js

@@ -1142,10 +1142,11 @@ export class MiniInteraction {
                 }
                 return resolvedResponse;
             }, this.timeoutConfig.initialResponseTimeout, `Component "${customId}"`, this.timeoutConfig.enableTimeoutWarnings, ackPromise);
-            const resolvedResponse = await timeoutWrapper();
+            const { response: resolvedResponse, backgroundWork } = await timeoutWrapper();
             return {
                 status: 200,
                 body: resolvedResponse,
+                backgroundWork,
             };
         }
         catch (error) {
@@ -1208,10 +1209,11 @@ export class MiniInteraction {
                 }
                 return resolvedResponse;
             }, this.timeoutConfig.initialResponseTimeout, `Modal "${customId}"`, this.timeoutConfig.enableTimeoutWarnings, ackPromise);
-            const resolvedResponse = await timeoutWrapper();
+            const { response: resolvedResponse, backgroundWork } = await timeoutWrapper();
             return {
                 status: 200,
                 body: resolvedResponse,
+                backgroundWork,
             };
         }
         catch (error) {
@@ -1324,7 +1326,7 @@ export class MiniInteraction {
                 }
                 return resolvedResponse;
             }, this.timeoutConfig.initialResponseTimeout, `Command "${commandName}"`, this.timeoutConfig.enableTimeoutWarnings, ackPromise);
-            const finalResponse = await timeoutWrapper();
+            const { response: finalResponse, backgroundWork } = await timeoutWrapper();
             if (this.timeoutConfig.enableResponseDebugLogging) {
                 console.log(`[MiniInteraction] handleApplicationCommand: initial response determined (type=${finalResponse?.type})`);
             }
@@ -1344,6 +1346,7 @@ export class MiniInteraction {
             return {
                 status: 200,
                 body: finalResponse,
+                backgroundWork,
             };
         }
         catch (error) {
@@ -1588,13 +1591,16 @@ function resolveOAuthConfig(provided) {
 }
 /**
  * Wraps a handler function with timeout detection and error handling.
+ *
+ * CRITICAL FOR HTTP INTERACTIONS:
+ * When deferReply() is called, we MUST return the ACK to Discord immediately.
+ * The handler continues executing and sends follow-up via webhook.
+ * The ACK must reach Discord before any webhook PATCH requests can succeed.
  */
 function createTimeoutWrapper(handler, timeoutMs, handlerName, enableWarnings = true, ackPromise) {
     return async (...args) => {
         const startTime = Date.now();
         let timeoutId;
-        let ackResult = null;
-        let ackReceived = false;
         const timeoutPromise = new Promise((_, reject) => {
             timeoutId = setTimeout(() => {
                 const elapsed = Date.now() - startTime;
@@ -1602,55 +1608,54 @@ function createTimeoutWrapper(handler, timeoutMs, handlerName, enableWarnings =
                 reject(new Error(`Handler timeout: ${handlerName} exceeded ${timeoutMs}ms limit`));
             }, timeoutMs);
         });
-        // If we have an ackPromise, listen for it but DON'T return early
-        // Instead, capture the ACK result and continue waiting for handler
+        // Start handler execution immediately (don't await yet)
+        const handlerPromise = Promise.resolve(handler(...args));
+        // Attach a default error handler to prevent unhandled rejections
+        const backgroundWork = handlerPromise.catch((error) => {
+            console.error(`[MiniInteraction] ${handlerName} background execution failed:`, error instanceof Error ? error.message : String(error));
+        }).then(() => {
+            // Ensure it always resolves to void
+        });
+        // If we have an ackPromise, race between ACK and timeout
         if (ackPromise) {
-            ackPromise.then((result) => {
-                ackResult = result;
-                ackReceived = true;
-                // Clear the timeout once we have an ACK - handler can now take longer
+            try {
+                const response = await Promise.race([
+                    ackPromise,
+                    timeoutPromise,
+                ]);
+                // ACK received! Clear timeout and return immediately
                 if (timeoutId) {
                     clearTimeout(timeoutId);
-                    timeoutId = undefined;
                 }
-            }).catch(() => {
-                // ACK promise rejection is fine, we'll use handler result
-            });
+                return { response, backgroundWork };
+            }
+            catch (error) {
+                // Timeout occurred before ACK - fall through to check handler
+                if (timeoutId) {
+                    clearTimeout(timeoutId);
+                }
+                throw error;
+            }
         }
+        // No ACK promise - wait for handler with timeout
         try {
-            // ALWAYS wait for handler to complete
-            const handlerResult = await Promise.race([
-                handler(...args),
+            const response = await Promise.race([
+                handlerPromise,
                 timeoutPromise,
             ]);
             if (timeoutId) {
                 clearTimeout(timeoutId);
             }
             const elapsed = Date.now() - startTime;
-            if (enableWarnings && elapsed > timeoutMs * 0.8 && !ackReceived) {
+            if (enableWarnings && elapsed > timeoutMs * 0.8) {
                 console.warn(`[MiniInteraction] ${handlerName} completed in ${elapsed}ms (${Math.round((elapsed / timeoutMs) * 100)}% of timeout limit)`);
             }
-            // If we got an ACK, return that for the HTTP response
-            // The handler has completed at this point so all background work is done
-            if (ackReceived && ackResult !== null) {
-                return ackResult;
-            }
-            return handlerResult;
+            return { response, backgroundWork };
         }
         catch (error) {
             if (timeoutId) {
                 clearTimeout(timeoutId);
             }
-            // If handler timed out but we have an ACK, return the ACK
-            // This allows deferReply to work even if later operations are slow
-            if (error instanceof Error && error.message.includes("Handler timeout")) {
-                if (ackReceived && ackResult !== null) {
-                    console.warn(`[MiniInteraction] ${handlerName} timed out but ACK was already captured. ` +
-                        `Background work may not complete in serverless environments.`);
-                    return ackResult;
-                }
-                throw error;
-            }
             console.error(`[MiniInteraction] ${handlerName} failed:`, error);
             throw error;
         }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@minesa-org/mini-interaction",
-	"version": "0.2.22",
+	"version": "0.2.24",
 	"description": "Mini interaction, connecting your app with Discord via HTTP-interaction (Vercel support).",
 	"type": "module",
 	"main": "dist/index.js",