@adminforth/mcp 1.4.1 → 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/custom/McpConfirm.vue

@@ -1,28 +1,25 @@
 <template>
   <div class="af-mcp-confirm flex min-h-screen items-center justify-center bg-gray-100 p-4 dark:bg-gray-800">
-    <div class="w-full max-w-lg rounded-lg bg-white p-6 shadow dark:bg-gray-700">
+    <div v-if="resultMessage || error" class="w-full max-w-lg rounded-lg bg-white p-6 shadow dark:bg-gray-700">
       <template v-if="pending">
         <h1 class="text-xl font-medium text-gray-900 dark:text-white">{{ title }}</h1>
         <p class="mt-3 text-sm text-gray-600 dark:text-gray-300">
           <b>{{ pending.connectionName }}</b>
           {{ $t('asks to run this action on your behalf. | asks to run these actions on your behalf.', pending.actions.length) }}
         </p>
-        <template v-if="resultMessage">
-          <p class="mt-4 text-sm font-medium" :class="failed ? 'text-red-700 dark:text-red-300' : 'text-green-700 dark:text-green-300'">
-            {{ resultMessage }}
-          </p>
-          <ul v-if="failedErrors.length" class="mt-2 list-disc space-y-1 pl-5 text-sm text-red-700 dark:text-red-300">
-            <li v-for="(failedError, i) in failedErrors" :key="i">{{ failedError }}</li>
-          </ul>
-        </template>
-        <Spinner v-else class="mx-auto mt-4 h-8 w-8" />
+        <p class="mt-4 text-sm font-medium" :class="failed ? 'text-red-700 dark:text-red-300' : 'text-green-700 dark:text-green-300'">
+          {{ resultMessage }}
+        </p>
+        <ul v-if="failedErrors.length" class="mt-2 list-disc space-y-1 pl-5 text-sm text-red-700 dark:text-red-300">
+          <li v-for="(failedError, i) in failedErrors" :key="i">{{ failedError }}</li>
+        </ul>
       </template>
-      <template v-else-if="error">
+      <template v-else>
         <h1 class="text-xl font-medium text-gray-900 dark:text-white">{{ $t('Cannot run the action') }}</h1>
         <p class="mt-3 text-sm text-gray-600 dark:text-gray-300">{{ error }}</p>
       </template>
-      <Spinner v-else class="mx-auto h-8 w-8" />
     </div>
+    <Spinner v-else-if="running" class="h-8 w-8" />
   </div>
 </template>
 
@@ -49,6 +46,7 @@ const route = useRoute();
 const pending = ref<PendingConfirmation | null>(null);
 const error = ref('');
 const failed = ref(false);
+const running = ref(false);
 const resultMessage = ref('');
 const failedErrors = ref<string[]>([]);
 const ids = computed(() => pending.value!.actions.map((action) => action.id));
@@ -90,6 +88,7 @@ async function confirm() {
       }
       return;
     }
+    running.value = true;
     const response = await callAdminForthApi({ method: 'POST', path: '/mcp/confirmation/run', body: { ids: ids.value, verificationResult } });
     if (!response || response.error) {
       failed.value = true;

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.1",
+  "version": "1.4.3",
   "description": "Expose AdminForth APIs as an authenticated MCP server",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",