fix(feishu): emit non-empty value schema for bitable write tools (#94547)

TwinsLee · TwinsLee · commit fa7bf3faac8c · 2026-06-19T22:37:27.000+08:00
Type.Record(Type.String(), Type.Any()) serializes via TypeBox to an empty nested value schema (patternProperties: { "^.*$": {} }). AWS Bedrock's strict tool-schema validation rejects the empty sub-schema, breaking all agent turns for Bedrock-backed Anthropic users with Feishu bitable tools enabled.

Replace the three affected free-form field maps (create_record, update_record, create_field) with an explicit non-empty FlexibleFieldValue value schema that still accepts any JSON value. Type.Unknown() is avoided because it also serializes to an empty schema. Adds regression coverage asserting the emitted value schema is non-empty.
diff --git a/extensions/feishu/src/bitable.test.ts b/extensions/feishu/src/bitable.test.ts
@@ -172,3 +172,49 @@ describe("feishu bitable create app cleanup", () => {
     expect(client.bitable.appTableRecord.list).toHaveBeenCalledTimes(1);
   });
 });
+
+describe("feishu bitable write tool schemas (#94547)", () => {
+  afterAll(() => {
+    vi.doUnmock("./client.js");
+    vi.resetModules();
+  });
+
+  beforeEach(() => {
+    createFeishuClientMock.mockReset();
+  });
+
+  function patternValueSchemaOf(
+    schema: Record<string, unknown> | undefined,
+  ): Record<string, unknown> | undefined {
+    const pattern = schema?.patternProperties as Record<string, unknown> | undefined;
+    if (!pattern) {
+      return undefined;
+    }
+    const keys = Object.keys(pattern);
+    return keys.length > 0 ? (pattern[keys[0]] as Record<string, unknown>) : undefined;
+  }
+
+  // Bedrock's strict tool-schema validation rejects empty nested schemas
+  // (`patternProperties: { "^(.*)$": {} }`). Each bitable write tool must emit a
+  // non-empty value schema for its free-form field map.
+  it.each([
+    ["feishu_bitable_create_record", "fields"],
+    ["feishu_bitable_update_record", "fields"],
+    ["feishu_bitable_create_field", "property"],
+  ])("%s emits a non-empty value schema for %s", (toolName, propName) => {
+    const { client } = createBitableClient([]);
+    createFeishuClientMock.mockReturnValue(client);
+    const { api, resolveTool } = createToolFactoryHarness(createConfig());
+    registerFeishuBitableTools(api);
+
+    const tool = resolveTool(toolName) as unknown as {
+      parameters?: { properties?: Record<string, Record<string, unknown>> };
+    };
+    const fieldSchema = tool.parameters?.properties?.[propName];
+    expect(fieldSchema).toBeDefined();
+
+    const valueSchema = patternValueSchemaOf(fieldSchema);
+    expect(valueSchema).toBeDefined();
+    expect(Object.keys(valueSchema as Record<string, unknown>).length).toBeGreaterThan(0);
+  });
+});
diff --git a/extensions/feishu/src/bitable.ts b/extensions/feishu/src/bitable.ts
@@ -525,12 +525,21 @@ const GetRecordSchema = Type.Object({
   record_id: Type.String({ description: "Record ID to retrieve" }),
 });
 
+// Flexible value schema for bitable field maps. TypeBox serializes both
+// Type.Any() and Type.Unknown() to an empty `{}` sub-schema, which produces
+// `patternProperties: { "^(.*)$": {} }`. AWS Bedrock's strict tool-schema
+// validation rejects that empty nested schema, so we emit an explicit
+// non-empty value schema that still accepts any JSON value. See issue #94547.
+const FlexibleFieldValue = Type.Unsafe<unknown>({
+  type: ["string", "number", "boolean", "object", "array", "null"],
+});
+
 const CreateRecordSchema = Type.Object({
   app_token: Type.String({
     description: "Bitable app token (use feishu_bitable_get_meta to get from URL)",
   }),
   table_id: Type.String({ description: "Table ID (from URL: ?table=YYY)" }),
-  fields: Type.Record(Type.String(), Type.Any(), {
+  fields: Type.Record(Type.String(), FlexibleFieldValue, {
     description:
       "Field values keyed by field name. Format by type: Text='string', Number=123, SingleSelect='Option', MultiSelect=['A','B'], DateTime=timestamp_ms, User=[{id:'ou_xxx'}], URL={text:'Display',link:'https://...'}",
   }),
@@ -560,7 +569,7 @@ const CreateFieldSchema = Type.Object({
     minimum: 1,
   }),
   property: Type.Optional(
-    Type.Record(Type.String(), Type.Any(), {
+    Type.Record(Type.String(), FlexibleFieldValue, {
       description: "Field-specific properties (e.g., options for SingleSelect, format for Number)",
     }),
   ),
@@ -572,7 +581,7 @@ const UpdateRecordSchema = Type.Object({
   }),
   table_id: Type.String({ description: "Table ID (from URL: ?table=YYY)" }),
   record_id: Type.String({ description: "Record ID to update" }),
-  fields: Type.Record(Type.String(), Type.Any(), {
+  fields: Type.Record(Type.String(), FlexibleFieldValue, {
     description: "Field values to update (same format as create_record)",
   }),
 });
