@a2a-js/sdk 0.3.10 → 0.3.12

package/dist/server/express/index.js

@@ -30,7 +30,7 @@ import {
   Task,
   TaskPushNotificationConfig,
   ToProto
-} from "../../chunk-U3QAVT4H.js";
+} from "../../chunk-2TNRJNPO.js";
 import {
   A2AError
 } from "../../chunk-UHZEIZLS.js";
@@ -245,23 +245,32 @@ var RestTransportHandler = class _RestTransportHandler {
   async getAuthenticatedExtendedAgentCard(context) {
     return this.requestHandler.getAuthenticatedExtendedAgentCard(context);
   }
+  /**
+   * Validate MessageSendParams.
+   */
+  validateMessageSendParams(params) {
+    if (!params.message) {
+      throw A2AError.invalidParams("message is required");
+    }
+    if (!params.message.messageId) {
+      throw A2AError.invalidParams("message.messageId is required");
+    }
+  }
   /**
    * Sends a message to the agent.
-   * Accepts both snake_case and camelCase input, returns camelCase.
    */
   async sendMessage(params, context) {
-    const normalized = this.normalizeMessageParams(params);
-    return this.requestHandler.sendMessage(normalized, context);
+    this.validateMessageSendParams(params);
+    return this.requestHandler.sendMessage(params, context);
   }
   /**
    * Sends a message with streaming response.
-   * Accepts both snake_case and camelCase input, returns camelCase stream.
    * @throws {A2AError} UnsupportedOperation if streaming not supported
    */
   async sendMessageStream(params, context) {
     await this.requireCapability("streaming");
-    const normalized = this.normalizeMessageParams(params);
-    return this.requestHandler.sendMessageStream(normalized, context);
+    this.validateMessageSendParams(params);
+    return this.requestHandler.sendMessageStream(params, context);
   }
   /**
    * Gets a task by ID.
@@ -293,13 +302,17 @@ var RestTransportHandler = class _RestTransportHandler {
   }
   /**
    * Sets a push notification configuration.
-   * Accepts both snake_case and camelCase input, returns camelCase.
    * @throws {A2AError} PushNotificationNotSupported if push notifications not supported
    */
   async setTaskPushNotificationConfig(config, context) {
     await this.requireCapability("pushNotifications");
-    const normalized = this.normalizeTaskPushNotificationConfig(config);
-    return this.requestHandler.setTaskPushNotificationConfig(normalized, context);
+    if (!config.taskId) {
+      throw A2AError.invalidParams("taskId is required");
+    }
+    if (!config.pushNotificationConfig) {
+      throw A2AError.invalidParams("pushNotificationConfig is required");
+    }
+    return this.requestHandler.setTaskPushNotificationConfig(config, context);
   }
   /**
    * Lists all push notification configurations for a task.
@@ -325,28 +338,6 @@ var RestTransportHandler = class _RestTransportHandler {
       context
     );
   }
-  // ==========================================================================
-  // Private Transformation Methods
-  // ==========================================================================
-  // All type conversion between REST (snake_case) and internal (camelCase) formats
-  /**
-   * Validates and normalizes message parameters.
-   * Accepts both snake_case and camelCase input.
-   * @throws {A2AError} InvalidParams if message is missing or conversion fails
-   */
-  normalizeMessageParams(input) {
-    if (!input.message) {
-      throw A2AError.invalidParams("message is required");
-    }
-    try {
-      return this.normalizeMessageSendParams(input);
-    } catch (error) {
-      if (error instanceof A2AError) throw error;
-      throw A2AError.invalidParams(
-        error instanceof Error ? error.message : "Invalid message parameters"
-      );
-    }
-  }
   /**
    * Static map of capability to error for missing capabilities.
    */
@@ -380,86 +371,6 @@ var RestTransportHandler = class _RestTransportHandler {
     }
     return parsed;
   }
-  /**
-   * Normalizes Part input - accepts both snake_case and camelCase for file mimeType.
-   */
-  normalizePart(part) {
-    if (part.kind === "text") return { kind: "text", text: part.text };
-    if (part.kind === "file") {
-      const file = this.normalizeFile(part.file);
-      return { kind: "file", file, metadata: part.metadata };
-    }
-    return { kind: "data", data: part.data, metadata: part.metadata };
-  }
-  /**
-   * Normalizes File input - accepts both snake_case (mime_type) and camelCase (mimeType).
-   */
-  normalizeFile(f) {
-    const file = f;
-    const mimeType = file.mimeType ?? file.mime_type;
-    if ("bytes" in file) {
-      return { bytes: file.bytes, mimeType, name: file.name };
-    }
-    return { uri: file.uri, mimeType, name: file.name };
-  }
-  /**
-   * Normalizes Message input - accepts both snake_case and camelCase.
-   */
-  normalizeMessage(input) {
-    const m = input;
-    const messageId = m.messageId ?? m.message_id;
-    if (!messageId) {
-      throw A2AError.invalidParams("message.messageId is required");
-    }
-    if (!m.parts || !Array.isArray(m.parts)) {
-      throw A2AError.invalidParams("message.parts must be an array");
-    }
-    return {
-      contextId: m.contextId ?? m.context_id,
-      extensions: m.extensions,
-      kind: "message",
-      messageId,
-      metadata: m.metadata,
-      parts: m.parts.map((p) => this.normalizePart(p)),
-      referenceTaskIds: m.referenceTaskIds ?? m.reference_task_ids,
-      role: m.role,
-      taskId: m.taskId ?? m.task_id
-    };
-  }
-  /**
-   * Normalizes MessageSendParams - accepts both snake_case and camelCase.
-   */
-  normalizeMessageSendParams(input) {
-    const p = input;
-    const config = p.configuration;
-    return {
-      configuration: config ? {
-        acceptedOutputModes: config.acceptedOutputModes ?? config.accepted_output_modes,
-        blocking: config.blocking,
-        historyLength: config.historyLength ?? config.history_length
-      } : void 0,
-      message: this.normalizeMessage(p.message),
-      metadata: p.metadata
-    };
-  }
-  /**
-   * Normalizes TaskPushNotificationConfig - accepts both snake_case and camelCase.
-   */
-  normalizeTaskPushNotificationConfig(input) {
-    const c = input;
-    const taskId = c.taskId ?? c.task_id;
-    if (!taskId) {
-      throw A2AError.invalidParams("taskId is required");
-    }
-    const pnConfig = c.pushNotificationConfig ?? c.push_notification_config;
-    if (!pnConfig) {
-      throw A2AError.invalidParams("pushNotificationConfig is required");
-    }
-    return {
-      pushNotificationConfig: pnConfig,
-      taskId
-    };
-  }
 };
 
 // src/server/express/rest_handler.ts
@@ -492,8 +403,8 @@ function restHandler(options) {
     if (statusCode === HTTP_STATUS.NO_CONTENT) {
       res.end();
     } else {
-      if (!responseType) {
-        throw new Error("Bug: toJson serializer must be provided for non-204 responses.");
+      if (!responseType || body === void 0) {
+        throw new Error("Bug: toJson serializer and body must be provided for non-204 responses.");
       }
       res.json(responseType.toJSON(body));
     }

package/dist/server/grpc/index.d.cts

@@ -1,8 +1,8 @@
 import { BinaryWriter, BinaryReader } from '@bufbuild/protobuf/wire';
 import * as grpc from '@grpc/grpc-js';
 import { UntypedServiceImplementation, handleUnaryCall, handleServerStreamingCall } from '@grpc/grpc-js';
-import { U as User, a as UnauthenticatedUser, A as A2ARequestHandler } from '../../a2a_request_handler-B3LxMq3P.cjs';
-import '../../extensions-DvruCIzw.cjs';
+import { a as User, U as UnauthenticatedUser, A as A2ARequestHandler } from '../../a2a_request_handler-BiphzMH4.cjs';
+import '../../extensions-APfrw8gz.cjs';
 
 /**
  * A generic empty message that you can re-use to avoid defining duplicated

package/dist/server/grpc/index.d.ts

@@ -1,8 +1,8 @@
 import { BinaryWriter, BinaryReader } from '@bufbuild/protobuf/wire';
 import * as grpc from '@grpc/grpc-js';
 import { UntypedServiceImplementation, handleUnaryCall, handleServerStreamingCall } from '@grpc/grpc-js';
-import { U as User, a as UnauthenticatedUser, A as A2ARequestHandler } from '../../a2a_request_handler-BuP9LgXH.js';
-import '../../extensions-DvruCIzw.js';
+import { a as User, U as UnauthenticatedUser, A as A2ARequestHandler } from '../../a2a_request_handler-C9bZITjN.js';
+import '../../extensions-APfrw8gz.js';
 
 /**
  * A generic empty message that you can re-use to avoid defining duplicated

package/dist/server/grpc/index.js

@@ -14,7 +14,7 @@ import {
 import {
   FromProto,
   ToProto
-} from "../../chunk-U3QAVT4H.js";
+} from "../../chunk-2TNRJNPO.js";
 import {
   A2AError
 } from "../../chunk-UHZEIZLS.js";

package/dist/server/index.cjs

@@ -287,7 +287,6 @@ var ExecutionEventQueue = class {
   eventQueue = [];
   resolvePromise;
   stopped = false;
-  boundHandleEvent;
   constructor(eventBus) {
     this.eventBus = eventBus;
     this.eventBus.on("event", this.handleEvent);

package/dist/server/index.d.cts

@@ -1,6 +1,6 @@
-import { F as Message, ay as Task, aQ as TaskStatusUpdateEvent, aS as TaskArtifactUpdateEvent, z as PushNotificationConfig, ae as AgentCard, x as MessageSendParams, X as TaskQueryParams, Z as TaskIdParams, $ as TaskPushNotificationConfig, a3 as GetTaskPushNotificationConfigParams, a7 as ListTaskPushNotificationConfigParams, a9 as DeleteTaskPushNotificationConfigParams, j as JSONRPCResponse, aw as JSONRPCError } from '../extensions-DvruCIzw.cjs';
-import { S as ServerCallContext, A as A2ARequestHandler } from '../a2a_request_handler-B3LxMq3P.cjs';
-export { a as UnauthenticatedUser, U as User } from '../a2a_request_handler-B3LxMq3P.cjs';
+import { ae as Message, aL as Task, b3 as TaskStatusUpdateEvent, aO as TaskArtifactUpdateEvent, az as PushNotificationConfig, i as AgentCard, aj as MessageSendParams, aY as TaskQueryParams, aP as TaskIdParams, aU as TaskPushNotificationConfig, U as GetTaskPushNotificationConfigParams, aa as ListTaskPushNotificationConfigParams, E as DeleteTaskPushNotificationConfigParams, a8 as JSONRPCResponse, a5 as JSONRPCError } from '../extensions-APfrw8gz.cjs';
+import { S as ServerCallContext, A as A2ARequestHandler } from '../a2a_request_handler-BiphzMH4.cjs';
+export { U as UnauthenticatedUser, a as User } from '../a2a_request_handler-BiphzMH4.cjs';
 
 type AgentExecutionEvent = Message | Task | TaskStatusUpdateEvent | TaskArtifactUpdateEvent;
 /**
@@ -141,7 +141,6 @@ declare class ExecutionEventQueue {
     private eventQueue;
     private resolvePromise?;
     private stopped;
-    private boundHandleEvent;
     constructor(eventBus: ExecutionEventBus);
     private handleEvent;
     private handleFinished;

package/dist/server/index.d.ts

@@ -1,6 +1,6 @@
-import { F as Message, ay as Task, aQ as TaskStatusUpdateEvent, aS as TaskArtifactUpdateEvent, z as PushNotificationConfig, ae as AgentCard, x as MessageSendParams, X as TaskQueryParams, Z as TaskIdParams, $ as TaskPushNotificationConfig, a3 as GetTaskPushNotificationConfigParams, a7 as ListTaskPushNotificationConfigParams, a9 as DeleteTaskPushNotificationConfigParams, j as JSONRPCResponse, aw as JSONRPCError } from '../extensions-DvruCIzw.js';
-import { S as ServerCallContext, A as A2ARequestHandler } from '../a2a_request_handler-BuP9LgXH.js';
-export { a as UnauthenticatedUser, U as User } from '../a2a_request_handler-BuP9LgXH.js';
+import { ae as Message, aL as Task, b3 as TaskStatusUpdateEvent, aO as TaskArtifactUpdateEvent, az as PushNotificationConfig, i as AgentCard, aj as MessageSendParams, aY as TaskQueryParams, aP as TaskIdParams, aU as TaskPushNotificationConfig, U as GetTaskPushNotificationConfigParams, aa as ListTaskPushNotificationConfigParams, E as DeleteTaskPushNotificationConfigParams, a8 as JSONRPCResponse, a5 as JSONRPCError } from '../extensions-APfrw8gz.js';
+import { S as ServerCallContext, A as A2ARequestHandler } from '../a2a_request_handler-C9bZITjN.js';
+export { U as UnauthenticatedUser, a as User } from '../a2a_request_handler-C9bZITjN.js';
 
 type AgentExecutionEvent = Message | Task | TaskStatusUpdateEvent | TaskArtifactUpdateEvent;
 /**
@@ -141,7 +141,6 @@ declare class ExecutionEventQueue {
     private eventQueue;
     private resolvePromise?;
     private stopped;
-    private boundHandleEvent;
     constructor(eventBus: ExecutionEventBus);
     private handleEvent;
     private handleFinished;

package/dist/server/index.js

@@ -262,7 +262,6 @@ var ExecutionEventQueue = class {
   eventQueue = [];
   resolvePromise;
   stopped = false;
-  boundHandleEvent;
   constructor(eventBus) {
     this.eventBus = eventBus;
     this.eventBus.on("event", this.handleEvent);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@a2a-js/sdk",
-  "version": "0.3.10",
+  "version": "0.3.12",
   "description": "Server & Client SDK for Agent2Agent protocol",
   "license": "Apache-2.0",
   "repository": {
@@ -70,6 +70,8 @@
     "README.md"
   ],
   "devDependencies": {
+    "@betterer/cli": "^6.0.0-alpha.1",
+    "@betterer/typescript": "^6.0.0-alpha.1",
     "@bufbuild/buf": "^1.61.0",
     "@bufbuild/protobuf": "^2.10.2",
     "@cloudflare/vitest-pool-workers": "^0.12.5",
@@ -97,9 +99,9 @@
     "build": "tsup",
     "test": "vitest run",
     "test:edge": "vitest run --config vitest.edge.config.ts",
-    "lint": "tsc --noEmit && npx eslint .",
+    "lint": "tsc --noEmit && npx eslint . --fix && betterer precommit",
+    "lint:ci": "tsc --noEmit && npx eslint . && betterer ci",
     "format:readme": "prettier --write ./README.md",
-    "lint:fix": "npx eslint . --fix",
     "coverage": "vitest run --coverage",
     "generate": "curl https://raw.githubusercontent.com/google-a2a/A2A/refs/heads/main/specification/json/a2a.json > spec.json && node scripts/generateTypes.js && rm spec.json",
     "test-build": "esbuild ./dist/client/index.js ./dist/server/index.js ./dist/index.js --bundle --platform=neutral --outdir=dist/tmp-checks --outbase=./dist"