fix(mcp): route dependency-blocked fields to failed[] in deleteFields; add test

Author: ARHAEEM

packages/mcp-server/src/client.js

@@ -708,7 +708,11 @@ export class AirtableClient {
       const { fieldId, expectedName } = fields[i];
       try {
         const result = await this.deleteField(appId, fieldId, expectedName, { force });
-        succeeded.push({ fieldId, name: expectedName, deleted: result.deleted, forced: result.forced ?? false });
+        if (!result.deleted) {
+          failed.push({ fieldId, name: expectedName, error: result.message ?? 'Field was not deleted' });
+        } else {
+          succeeded.push({ fieldId, name: expectedName, deleted: true, forced: result.forced ?? false });
+        }
       } catch (error) {
         failed.push({ fieldId, name: expectedName, error: error.message });
       }

packages/mcp-server/test/test-bulk-delete.test.js

@@ -150,4 +150,38 @@ describe('AirtableClient.deleteFields', () => {
     assert.equal(result.succeeded.length, 1);
     assert.equal(result.failed.length, 1);
   });
+
+  it('routes deleted:false (dependency-blocked) to failed[] when force=false', async () => {
+    const DEPENDENCY_RESPONSE = {
+      ok: false,
+      status: 400,
+      json: async () => ({
+        error: {
+          type: 'SCHEMA_DEPENDENCIES_VALIDATION_FAILED',
+          details: { dependentColumns: [{ id: 'fldDEP', type: 'formula' }] },
+        },
+      }),
+      text: async () => JSON.stringify({ error: { type: 'SCHEMA_DEPENDENCIES_VALIDATION_FAILED' } }),
+    };
+    const auth = createMockAuth({
+      get() {
+        return {
+          ok: true, status: 200,
+          json: async () => ({
+            data: { tableSchemas: [{ id: 'tbl1', columns: [{ id: 'fld001', name: 'HasDeps', type: 'formula', typeOptions: {} }], views: [] }] },
+          }),
+          text: async () => '{}',
+        };
+      },
+      postForm() {
+        return DEPENDENCY_RESPONSE;
+      },
+    });
+    const client = new AirtableClient(auth);
+    const result = await client.deleteFields('appTEST', [{ fieldId: 'fld001', expectedName: 'HasDeps' }], { force: false });
+    assert.equal(result.succeeded.length, 0, 'no fields should be in succeeded');
+    assert.equal(result.failed.length, 1, 'dependency-blocked field should be in failed');
+    assert.ok(result.failed[0].error.includes('dependencies') || result.failed[0].error.includes('not deleted'),
+      `error message should mention dependencies or not deleted, got: "${result.failed[0].error}"`);
+  });
 });