@holon-run/agentinbox 1.0.5 → 1.1.0

package/dist/src/adapters.js

@@ -197,7 +197,7 @@ class AdapterRegistry {
     async listDeliveryOperations(source, handle) {
         const module = await this.resolveDeliveryModule(source, handle);
         if (!module?.listDeliveryOperations) {
-            return [];
+            return null;
         }
         return module.listDeliveryOperations({ handle, source });
     }

package/dist/src/http.js

@@ -1870,6 +1870,8 @@ function isBadRequestError(message) {
         message.startsWith("deliver send is not supported") ||
         message.startsWith("deliveryHandle requires") ||
         message.startsWith("delivery operations are not supported") ||
+        message.startsWith("unsupported delivery surface") ||
+        message.startsWith("unsupported delivery operation") ||
         message.startsWith("source operations are not supported") ||
         message.startsWith("unknown GitHub delivery operation") ||
         message.startsWith("unknown Feishu delivery operation") ||

package/dist/src/service.js

@@ -1620,6 +1620,12 @@ class AgentInboxService {
         const handle = resolveDeliveryHandle(request);
         const source = resolveDeliverySource(this.store, request.sourceId, handle);
         const operations = await this.adapters.listDeliveryOperations(source, handle);
+        if (operations === null) {
+            throw new Error(`delivery operations are not supported for provider ${handle.provider}`);
+        }
+        if (operations.length === 0) {
+            throw new Error(noDeliveryOperationsMessage(handle));
+        }
         return {
             sourceId: source?.sourceId ?? null,
             handle,
@@ -1629,6 +1635,15 @@ class AgentInboxService {
     async invokeDelivery(request) {
         const handle = resolveDeliveryHandle(request);
         const source = resolveDeliverySource(this.store, request.sourceId, handle);
+        const operations = await this.adapters.listDeliveryOperations(source, handle);
+        if (operations !== null) {
+            if (operations.length === 0) {
+                throw new Error(noDeliveryOperationsMessage(handle));
+            }
+            if (!operations.some((operation) => operation.name === request.operation)) {
+                throw new Error(unsupportedDeliveryOperationMessage(handle, request.operation, operations));
+            }
+        }
         const attempt = {
             deliveryId: (0, util_1.generateCanonicalId)("dlv"),
             provider: handle.provider,
@@ -1648,9 +1663,9 @@ class AgentInboxService {
     }
     async sendCanonicalDeliveryViaModule(source, handle, payload, attempt) {
         const operations = await this.adapters.listDeliveryOperations(source, handle);
-        const canonicalOperation = operations.find((operation) => operation.canonicalTextAlias);
+        const canonicalOperation = operations?.find((operation) => operation.canonicalTextAlias);
         if (!canonicalOperation) {
-            throw new Error(`deliver send is not supported for ${handle.provider}/${handle.surface}; use deliver invoke`);
+            throw new Error(deliverSendUnsupportedMessage(handle, operations));
         }
         const input = payload.text != null ? { ...payload, body: String(payload.text) } : payload;
         return this.adapters.invokeDeliveryOperation(source, handle, canonicalOperation.name, input, attempt);
@@ -3885,6 +3900,43 @@ function listHostStreamKinds(hostType) {
             return ["default"];
     }
 }
+const KNOWN_DELIVERY_SURFACES = {
+    feishu: ["message_reply", "chat_message"],
+    github: ["issue_comment", "pull_request_comment", "review_comment"],
+};
+function unsupportedDeliverySurfaceMessage(handle) {
+    const supported = KNOWN_DELIVERY_SURFACES[handle.provider];
+    const supportedHint = supported?.length
+        ? ` Supported surfaces for ${handle.provider}: ${supported.join(", ")}.`
+        : "";
+    return `unsupported delivery surface for ${handle.provider}: ${handle.surface}.${supportedHint} ${deliveryActionsHelp(handle)}`;
+}
+function noDeliveryOperationsMessage(handle) {
+    if (!isKnownDeliverySurface(handle)) {
+        return unsupportedDeliverySurfaceMessage(handle);
+    }
+    return `delivery operations are not supported for ${handle.provider}/${handle.surface}. ${deliveryActionsHelp(handle)}`;
+}
+function unsupportedDeliveryOperationMessage(handle, operation, operations) {
+    const names = operations.map((item) => item.name).join(", ");
+    return `unsupported delivery operation for ${handle.provider}/${handle.surface}: ${operation}. Available operations: ${names}. ${deliveryActionsHelp(handle)}`;
+}
+function deliverSendUnsupportedMessage(handle, operations) {
+    if (!operations || operations.length === 0) {
+        if (!isKnownDeliverySurface(handle)) {
+            return unsupportedDeliverySurfaceMessage(handle);
+        }
+        return `deliver send is not supported for ${handle.provider}/${handle.surface}. ${deliveryActionsHelp(handle)}`;
+    }
+    const names = operations.map((item) => item.name).join(", ");
+    return `deliver send is not supported for ${handle.provider}/${handle.surface}; use deliver invoke with one of: ${names}. ${deliveryActionsHelp(handle)}`;
+}
+function deliveryActionsHelp(handle) {
+    return `Run \`agentinbox deliver actions --provider ${handle.provider} --surface ${handle.surface} --target ${handle.targetRef}\` to inspect supported operations and input schemas.`;
+}
+function isKnownDeliverySurface(handle) {
+    return KNOWN_DELIVERY_SURFACES[handle.provider]?.includes(handle.surface) === true;
+}
 function getHostConfigFields(hostType) {
     switch (hostType) {
         case "github":

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@holon-run/agentinbox",
-  "version": "1.0.5",
+  "version": "1.1.0",
   "description": "Local event subscription and delivery service for agents.",
   "license": "Apache-2.0",
   "repository": {