@adminforth/mcp 1.4.2 → 1.4.3

package/dist/adminPanelConfirmation.js

@@ -1,5 +1,6 @@
 import { randomUUID } from 'node:crypto';
 import { MAX_ACTIONS_PER_CONFIRMATION } from './schemas/confirmation.js';
+import { isErrorOutput } from './toolOutput.js';
 import { adminApiPrefix, CONFIRMATION_PAGE_PATH } from './urls.js';
 export const WAIT_FOR_CONFIRMATION_TOOL_NAME = 'wait_for_confirmation';
 const CONFIRMATION_TTL_MS = 10 * 60 * 1000;
@@ -208,7 +209,7 @@ export class McpAdminPanelConfirmation {
             clearTimeout(timer);
             input.abortSignal?.removeEventListener('abort', abort);
         }
-        return { output, isError: responseStatus >= 400 || Boolean(output && typeof output === 'object' && ('error' in output || output.ok === false)) };
+        return { output, isError: isErrorOutput(responseStatus, output) };
     }
     /** All actions of a group belong to one user and run as one agent, so they can share one verification. */
     openGroup(ids, adminUser) {

package/dist/apiTools.js

@@ -1,4 +1,5 @@
 import { compactInputSchema } from './compactInputSchema.js';
+import { isErrorOutput } from './toolOutput.js';
 import { adminApiPrefix } from './urls.js';
 import { AdminForthDataTypes } from 'adminforth';
 const METHODS_WITHOUT_REQUEST_BODY = new Set(['GET', 'HEAD']);
@@ -308,7 +309,7 @@ export class AdminForthApiTools {
                 isError: true,
             };
         }
-        const isError = response.status >= 400 || Boolean(output && typeof output === 'object' && 'error' in output);
+        const isError = isErrorOutput(response.status, output);
         return {
             output: isError || !override?.project ? output : override.project(output, args, params.arguments ?? {}),
             isError,

package/dist/toolOutput.d.ts

@@ -0,0 +1 @@
+export declare function isErrorOutput(status: number, output: unknown): boolean;

package/dist/toolOutput.js

@@ -0,0 +1,13 @@
+// AdminForth core reports a beforeSave hook that returned { ok: false, error: null } with this error.
+// The hook saved the record itself and may point to it with redirectToRecordId; the admin panel treats it as a success.
+const HOOK_ABORT_ERROR = 'Operation aborted by hook';
+export function isErrorOutput(status, output) {
+    if (status >= 400)
+        return true;
+    if (!output || typeof output !== 'object')
+        return false;
+    const { error, ok } = output;
+    if (error === HOOK_ABORT_ERROR)
+        return false;
+    return 'error' in output || ok === false;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adminforth/mcp",
-  "version": "1.4.2",
+  "version": "1.4.3",
   "description": "Expose AdminForth APIs as an authenticated MCP server",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",