@abot-ai/runtime 1.6.0 → 1.7.0

package/README.md

@@ -8,6 +8,8 @@ plugins, model routing, persisted events, and a local graphical interface. It
 does not choose a model for you or hide model-provider behavior behind
 hardcoded defaults.
 
+[Website](https://abot-ai.com/) · [Documentation](#documentation) · [GitHub](https://github.com/Dorshitrit/abot-runtime)
+
 ## Features
 
 - Configurable local and hosted model providers

package/dist/src/bridge/request-steering.js

@@ -1,4 +1,4 @@
-import { traceDebug } from "../runtime/observability/debug-logger.js";
+import { createRequestControlDiagnosticId, diagnosticErrorType, traceRequestControl, } from "../runtime/observability/request-control-diagnostics.js";
 import { createRequestSteeringInbox, } from "../runtime/request/request-steering.js";
 const LOG_SCOPE = "agent-bridge.request_steering";
 export function createBridgeRequestSteeringRegistry(steer) {
@@ -8,10 +8,16 @@ export function createBridgeRequestSteeringRegistry(steer) {
             const current = active.get(requestId);
             if (current?.inbox === inbox) {
                 active.delete(requestId);
+                traceRequestControl(LOG_SCOPE, "request.removed", {
+                    requestId,
+                    inboxId: current.inboxId,
+                    source: "bridge_handler_settled",
+                });
             }
             await inbox.close();
-            traceDebug(LOG_SCOPE, "request.closed", {
+            traceRequestControl(LOG_SCOPE, "request.closed", {
                 requestId,
+                inboxId: current?.inboxId,
                 activeRequestCount: active.size,
             });
         },
@@ -23,13 +29,33 @@ export function createBridgeRequestSteeringRegistry(steer) {
             const steerId = readString(message.steerId).trim();
             const text = readString(message.text).trim();
             const current = active.get(requestId);
-            const result = (steer && current
-                ? await steer(requestId, { steerId, text })
-                : current?.inbox.append({ steerId, text })) ??
-                Object.freeze({
-                    ok: false,
-                    reason: "request_not_active",
+            const diagnosticScope = {
+                requestId,
+                steerId,
+                inboxId: current?.inboxId,
+                registryPresent: Boolean(current),
+                ownerSteerAvailable: Boolean(steer),
+                textLength: text.length,
+            };
+            traceRequestControl(LOG_SCOPE, "message.received", diagnosticScope);
+            let result;
+            try {
+                result =
+                    (steer && current
+                        ? await steer(requestId, { steerId, text })
+                        : current?.inbox.append({ steerId, text })) ??
+                        Object.freeze({
+                            ok: false,
+                            reason: "request_not_active",
+                        });
+            }
+            catch (error) {
+                traceRequestControl(LOG_SCOPE, "message.failed", {
+                    ...diagnosticScope,
+                    errorType: diagnosticErrorType(error),
                 });
+                throw error;
+            }
             const response = result.ok
                 ? {
                     type: "steer_ack",
@@ -45,10 +71,9 @@ export function createBridgeRequestSteeringRegistry(steer) {
                     accepted: false,
                     reason: result.reason,
                 };
-            await sendJson(ws, response);
-            traceDebug(LOG_SCOPE, result.ok ? "message.accepted" : "message.rejected", {
-                requestId,
-                steerId,
+            traceRequestControl(LOG_SCOPE, result.ok ? "message.accepted" : "message.rejected", {
+                ...diagnosticScope,
+                registryUnchanged: active.get(requestId) === current,
                 accepted: result.ok,
                 ...(result.ok
                     ? {
@@ -58,6 +83,21 @@ export function createBridgeRequestSteeringRegistry(steer) {
                     }
                     : { reason: result.reason }),
             });
+            traceRequestControl(LOG_SCOPE, "ack.dispatch_attempted", {
+                ...diagnosticScope,
+                accepted: result.ok,
+                readyState: ws.readyState,
+            });
+            try {
+                await sendJson(ws, response);
+            }
+            catch (error) {
+                traceRequestControl(LOG_SCOPE, "ack.dispatch_failed", {
+                    ...diagnosticScope,
+                    errorType: diagnosticErrorType(error),
+                });
+                throw error;
+            }
             return true;
         },
         open(requestId) {
@@ -67,10 +107,15 @@ export function createBridgeRequestSteeringRegistry(steer) {
             if (active.has(requestId)) {
                 throw new Error("request_steering_request_already_active");
             }
-            const inbox = createRequestSteeringInbox({ requestId });
-            active.set(requestId, Object.freeze({ inbox, requestId }));
-            traceDebug(LOG_SCOPE, "request.opened", {
+            const inboxId = createRequestControlDiagnosticId("inbox");
+            const inbox = createRequestSteeringInbox({
+                requestId,
+                diagnostics: { inboxId, origin: "bridge_projection" },
+            });
+            active.set(requestId, Object.freeze({ inbox, requestId, inboxId }));
+            traceRequestControl(LOG_SCOPE, "request.opened", {
                 requestId,
+                inboxId,
                 activeRequestCount: active.size,
             });
             return inbox;

package/dist/src/capabilities/normal-invocation/contracts.d.ts

@@ -1,3 +1,5 @@
+import type { ToolNormalInvocationOutputChannels } from "./output-channels.js";
+export type { ToolNormalInvocationOutputChannels, ToolNormalInvocationOutputChannelRole } from "./output-channels.js";
 export declare const TOOL_NORMAL_INVOCATION_VERSION: 1;
 export declare const TOOL_NORMAL_INVOCATION_IDENTIFIER_MAX_LENGTH = 64;
 export declare const TOOL_NORMAL_INVOCATION_STRING_MAX_LENGTH = 8192;
@@ -64,6 +66,7 @@ export type ToolNormalInvocationAttachmentInput = Readonly<{
 export type ToolNormalInvocationOperation = Readonly<{
     operationId: string;
     summary: string;
+    outputChannels?: ToolNormalInvocationOutputChannels;
     input: ToolNormalInvocationInput;
     selectionControlIds?: readonly string[];
     fixedParams?: Readonly<Record<string, ToolNormalInvocationFixedValue>>;

package/dist/src/capabilities/normal-invocation/output-channels.d.ts

@@ -0,0 +1,10 @@
+/** Passive descriptions of saved Studio output; not execution guarantees or binding defaults. */
+export type ToolNormalInvocationOutputChannelRole = "content" | "metadata" | "conditional";
+export type ToolNormalInvocationOutputChannels = Readonly<{
+    text: ToolNormalInvocationOutputChannelRole;
+    data: ToolNormalInvocationOutputChannelRole;
+}>;
+/** Shared by manifest and normal-invocation validation; omission stays undeclared. */
+export declare function parseOptionalToolOutputChannels(value: unknown, path: string): Readonly<{
+    outputChannels?: ToolNormalInvocationOutputChannels;
+}>;

package/dist/src/capabilities/normal-invocation/output-channels.js

@@ -0,0 +1,29 @@
+function isOutputChannelsRecord(value) {
+    if (value === null || typeof value !== "object" || Array.isArray(value))
+        return false;
+    const prototype = Object.getPrototypeOf(value);
+    return prototype === Object.prototype || prototype === null;
+}
+function hasExactOutputChannelKeys(value) {
+    if (Object.keys(value).length !== 2)
+        return false;
+    return Object.hasOwn(value, "text") && Object.hasOwn(value, "data");
+}
+function requireOutputChannelRole(value, path) {
+    if (value === "content" || value === "metadata" || value === "conditional")
+        return value;
+    throw new Error(`${path} must be content, metadata, or conditional`);
+}
+/** Shared by manifest and normal-invocation validation; omission stays undeclared. */
+export function parseOptionalToolOutputChannels(value, path) {
+    if (value === undefined)
+        return {};
+    if (!isOutputChannelsRecord(value))
+        throw new Error(`${path} must be an object`);
+    if (!hasExactOutputChannelKeys(value))
+        throw new Error(`${path} must contain only text and data`);
+    return { outputChannels: Object.freeze({
+            text: requireOutputChannelRole(value.text, `${path}.text`),
+            data: requireOutputChannelRole(value.data, `${path}.data`),
+        }) };
+}

package/dist/src/capabilities/normal-invocation/validator.js

@@ -1,3 +1,4 @@
+import { parseOptionalToolOutputChannels } from "./output-channels.js";
 import { TOOL_NORMAL_INVOCATION_IDENTIFIER_MAX_LENGTH, TOOL_NORMAL_INVOCATION_STRING_MAX_LENGTH as MAX_STRING_LENGTH_BOUND, TOOL_NORMAL_INVOCATION_PAYLOAD_MAX_BYTES, TOOL_NORMAL_INVOCATION_VERSION, } from "./contracts.js";
 const MAX_OPERATIONS = 32;
 const MAX_SUMMARY_LENGTH = 512;
@@ -286,8 +287,8 @@ function parseAttachmentInput(toolName, path, input, value) {
 }
 function parseOperation(toolName, path, raw) {
     if (!isPlainObject(raw) ||
-        !hasExactKeys(raw, ["operationId", "summary", "input", "effect", "approval"], ["selectionControlIds", "fixedParams", "payload", "attachmentInput"])) {
-        fail(toolName, path, "operation must contain operationId/summary/input/effect/approval and optional selectionControlIds/fixedParams/payload/attachmentInput");
+        !hasExactKeys(raw, ["operationId", "summary", "input", "effect", "approval"], ["selectionControlIds", "fixedParams", "payload", "attachmentInput", "outputChannels"])) {
+        fail(toolName, path, "operation must contain operationId/summary/input/effect/approval and optional selectionControlIds/fixedParams/payload/attachmentInput/outputChannels");
     }
     if (raw.effect !== "read_only" &&
         raw.effect !== "mutating" &&
@@ -315,6 +316,7 @@ function parseOperation(toolName, path, raw) {
     return Object.freeze({
         operationId: identifier(toolName, `${path}.operationId`, raw.operationId),
         summary: boundedText(toolName, `${path}.summary`, raw.summary, MAX_SUMMARY_LENGTH),
+        ...parseOptionalToolOutputChannels(raw.outputChannels, `${toolName}.${path}.outputChannels`),
         input,
         ...(selectionControlIds ? { selectionControlIds } : {}),
         ...(fixedParams ? { fixedParams } : {}),

package/dist/src/capabilities/tool-types.d.ts

@@ -7,7 +7,7 @@ import type { RuntimeAttachmentKind } from "../shared/attachments.js";
 import type { ToolImageEvidence, ToolMediaWriter, ToolRequestDisposer } from "./tool-media.js";
 import type { ToolNormalInvocationContract } from "./normal-invocation/contracts.js";
 import type { ToolFileOutputReporter } from "./file-output-presentation.js";
-export type { ToolNormalInvocationApproval, ToolNormalInvocationAttachmentInput, ToolNormalInvocationArrayInput, ToolNormalInvocationBooleanInput, ToolNormalInvocationBoundedStringInput, ToolNormalInvocationContract, ToolNormalInvocationEnumInput, ToolNormalInvocationEffect, ToolNormalInvocationFixedValue, ToolNormalInvocationInput, ToolNormalInvocationInputValidation, ToolNormalInvocationNumberInput, ToolNormalInvocationOperation, ToolNormalInvocationPayload, ToolNormalInvocationPropertyInput, ToolNormalInvocationScalarInput, ToolNormalInvocationUnboundedStringInput, } from "./normal-invocation/contracts.js";
+export type { ToolNormalInvocationApproval, ToolNormalInvocationAttachmentInput, ToolNormalInvocationArrayInput, ToolNormalInvocationBooleanInput, ToolNormalInvocationBoundedStringInput, ToolNormalInvocationContract, ToolNormalInvocationEnumInput, ToolNormalInvocationEffect, ToolNormalInvocationFixedValue, ToolNormalInvocationInput, ToolNormalInvocationInputValidation, ToolNormalInvocationNumberInput, ToolNormalInvocationOperation, ToolNormalInvocationOutputChannels, ToolNormalInvocationOutputChannelRole, ToolNormalInvocationPayload, ToolNormalInvocationPropertyInput, ToolNormalInvocationScalarInput, ToolNormalInvocationUnboundedStringInput, } from "./normal-invocation/contracts.js";
 export type ToolRoutingCapability = "filesystem_inspection" | "filesystem_mutation" | "semantic_lookup" | "semantic_mutation" | "web_lookup";
 export type ToolDevelopmentRole = "inspect" | "establish" | "mutate" | "verify" | "auxiliary";
 export declare const TOOL_CATALOG_GROUP_ID_MAX_LENGTH = 128;