@ziggs-ai/agent-sdk 0.42.2 → 0.43.1

package/dist/platform/tasks/TaskService.js

@@ -1,4 +1,4 @@
-import { createTask, getTask, getActiveTasksForAgent, getActiveTasksForChat, getSubtasks, updateTaskState, updateTaskPlanStep, updateTaskPlanSteps, cancelTask, reportTask, setTaskSatisfaction, countTasks, acknowledgeTaskControl, presentTaskOutcome, } from '@ziggs-ai/api-client';
+import { createTask, getTask, getActiveTasksForAgent, getSubtasks, updateTaskState, updateTaskPlanStep, updateTaskPlanSteps, cancelTask, reportTask, setTaskSatisfaction, countTasks, acknowledgeTaskControl, presentTaskOutcome, } from '@ziggs-ai/api-client';
 import { runtimeLog } from '../../shared/runtimeLog.js';
 function safeJsonStringify(value) {
     if (value === null || value === undefined)
@@ -40,9 +40,6 @@ export class TaskService {
     async getActiveTasksForAgent(agentId) {
         return getActiveTasksForAgent(agentId, this.creds);
     }
-    async getActiveTasksForChat(chatId) {
-        return getActiveTasksForChat(chatId, this.creds);
-    }
     async getSubtasks(parentTaskId) {
         return getSubtasks(parentTaskId, this.creds);
     }

package/dist/platform/telemetryIngest.js

@@ -18,11 +18,11 @@ export function usageFromLlmResponse(response) {
 }
 export function buildLlmIngestPayload(response, ctx) {
     const { tokensTotal, tokensInput, tokensOutput } = usageFromLlmResponse(response);
-    const chatId = ctx.sessionId ?? null;
+    const addressId = ctx.sessionId ?? null;
     return {
         source: 'agent',
-        chatId,
-        sessionId: chatId,
+        addressId,
+        sessionId: addressId,
         execution: {
             llmCallsTotal: 1,
             tokensTotal,
@@ -41,7 +41,7 @@ export function buildLlmIngestPayload(response, ctx) {
 export function buildToolIngestPayload(ctx) {
     return {
         source: 'agent',
-        chatId: ctx.sessionId,
+        addressId: ctx.sessionId,
         sessionId: ctx.sessionId,
         execution: { executionLoops: 1 },
         resource: { durationMs: Math.max(0, Math.round(ctx.durationMs)) },

package/dist/platform/tools/index.d.ts

@@ -32,4 +32,3 @@ export { PARTY_TOOLS, partyNoteTool } from './tier2/partyTools.js';
 export { ARTIFACT_TOOLS, recordArtifactTool } from './tier2/artifactTools.js';
 export { LISTING_TOOLS } from './tier2/listingTools.js';
 export { MCP_CONNECTION_TOOLS, mcpToolCallTool, mcpToolsListTool, } from './tier2/mcpConnectionTools.js';
-export { CHAT_TOOLS, openConversationTool } from './tier2/chatTools.js';

package/dist/platform/tools/index.js

@@ -42,6 +42,3 @@ export { PARTY_TOOLS, partyNoteTool } from './tier2/partyTools.js';
 export { ARTIFACT_TOOLS, recordArtifactTool } from './tier2/artifactTools.js';
 export { LISTING_TOOLS } from './tier2/listingTools.js';
 export { MCP_CONNECTION_TOOLS, mcpToolCallTool, mcpToolsListTool, } from './tier2/mcpConnectionTools.js';
-// Decision: chat_open only — chat listing stays a context-rail read
-// (grant_list scopeKind=chat), so there is no duplicate lister.
-export { CHAT_TOOLS, openConversationTool } from './tier2/chatTools.js';

package/dist/platform/tools/tier1/protocolRunner.d.ts

@@ -36,7 +36,7 @@ interface ProtocolPayload {
     })[];
     [key: string]: unknown;
 }
-export declare function dispatchProtocolOp(services: Services, payload: ProtocolPayload, chatId: string | null, _agents?: unknown[], actingAgentId?: string | null, writeKeys?: DeliveryWriteKeys | null, 
+export declare function dispatchProtocolOp(services: Services, payload: ProtocolPayload, addressId: string | null, _agents?: unknown[], actingAgentId?: string | null, writeKeys?: DeliveryWriteKeys | null, 
 /** #8 — prefer the cached session snapshot over a fresh HTTP read. */
 cachedAgreements?: unknown[]): Promise<unknown>;
 export {};

package/dist/platform/tools/tier1/protocolRunner.js

@@ -1,5 +1,5 @@
 import { AGREEMENT_TERMINAL_STATES } from '@ziggs-ai/contracts';
-import { isBroadcastTarget, partyActorIds, presentClaimResult, } from '@ziggs-ai/api-client';
+import { isAgreementLaneId, currentWorkExecution, isBroadcastTarget, partyActorIds, presentClaimResult, } from '@ziggs-ai/api-client';
 // the link-summary rule lives in AgreementClient, at the point the
 // document is parsed. Every verb here inherits it through the client instead of
 // each one restating `engagementKind === 'link' ? linkSummary(a) : a` — which is
@@ -56,7 +56,7 @@ function sourcesOf(payload) {
         ? { sources: payload.sources }
         : {};
 }
-export async function dispatchProtocolOp(services, payload, chatId, _agents = [], actingAgentId = null, writeKeys = null, 
+export async function dispatchProtocolOp(services, payload, addressId, _agents = [], actingAgentId = null, writeKeys = null, 
 /** #8 — prefer the cached session snapshot over a fresh HTTP read. */
 cachedAgreements) {
     if (!payload?.operation)
@@ -66,9 +66,9 @@ cachedAgreements) {
     const actorId = actingAgentId?.trim() || null;
     switch (operation) {
         case 'agreement-propose':
-            return proposeAgreement(services, payload, chatId, writeKeys);
+            return proposeAgreement(services, payload, roomOf(addressId), writeKeys);
         case 'agreement-subcontract':
-            return subcontractAgreement(services, payload, namedSubcontractChat(payload, chatId), writeKeys);
+            return subcontractAgreement(services, payload, roomOf(namedSubcontractChat(payload, addressId)), writeKeys);
         case 'agreement-respond':
             if (!payload.agreementId || !payload.action)
                 return null;
@@ -129,7 +129,7 @@ cachedAgreements) {
             if (!payload.agreementId || !payload.description)
                 return null;
             let spawnAgreementId = payload.agreementId;
-            const ags = await agreementsInChat(services, chatId, cachedAgreements);
+            const ags = await agreementsForWork(services, payload.agreementId, cachedAgreements);
             // b2: if the assignee is not a party on the cited agreement,
             // redirect the spawn to the ACTIVE sub-agreement (same root) where the
             // assignee IS the provider — the orchestrator often cites the top-level
@@ -255,7 +255,7 @@ cachedAgreements) {
             throw new Error(`Unknown protocol operation: ${operation}`);
     }
 }
-async function proposeAgreement(services, payload, chatId, writeKeys = null) {
+async function proposeAgreement(services, payload, addressId, writeKeys = null) {
     if (!payload.description)
         throw new Error('agreement-propose requires `description`');
     if (!payload.proposedTo)
@@ -297,7 +297,7 @@ async function proposeAgreement(services, payload, chatId, writeKeys = null) {
     const interactionMode = oneOf(payload['interactionMode'], ['messages', 'tasks', 'both']);
     const { agreement, shape } = await services.agreementService.proposeUnified({
         proposedTo: payload.proposedTo,
-        chatId: chatId ?? undefined,
+        addressId: addressId ?? undefined,
         description: String(payload.description),
         providerId: payload.providerId,
         price: typeof payload.price === 'number' ? payload.price : undefined,
@@ -323,21 +323,21 @@ async function proposeAgreement(services, payload, chatId, writeKeys = null) {
 // task_spawns a worker under the TOP-LEVEL hire — where the worker is not a
 // party — instead of under that worker's sub-agreement, which the backend
 // rejects with 400 "assignee is not a party". Both are made deterministic here
-// using the chat's existing agreements.
+// using the selected work's authorized agreements.
 const TERMINAL_STATUS = new Set(AGREEMENT_TERMINAL_STATES);
-async function agreementsInChat(services, chatId, cachedAgreements) {
-    if (!chatId)
-        return [];
+async function agreementsForWork(services, agreementId, cachedAgreements) {
     // #8 — createZiggsEffectHandler already cached the session snapshot;
     // prefer that over another GET /context/snapshot just for the agreements array.
-    // Empty array is a real answer ("none in this chat") — do not re-fetch.
+    // Empty array is a real answer ("none in this work") — do not re-fetch.
     if (cachedAgreements !== undefined) {
         return Array.isArray(cachedAgreements) ? cachedAgreements : [];
     }
     try {
-        // The room is the lane, as for every read made on a wake.
+        const execution = currentWorkExecution();
+        const via = execution?.taskId ? `task:${execution.taskId}` : execution?.agreementId ? `agreement:${execution.agreementId}`
+            : execution?.action?.kind === 'message' ? `message:${execution.action.id}` : `agreement:${agreementId}`;
         const reader = services.contextReadClient;
-        const snap = await reader.snapshot(chatId, { maxMessages: 1 });
+        const snap = await reader.snapshot({ maxMessages: 1, via });
         const ags = snap.agreements;
         return Array.isArray(ags) ? ags : [];
     }
@@ -358,9 +358,16 @@ function agIsParty(a, agentId) {
 function agIsActiveStatus(a) {
     return String(a.status ?? '') === 'active';
 }
+/**
+ * The room a new agreement is discussed in. An agreement lane (`agrn-…`) is
+ * a session key, not a room, so it names none.
+ */
+function roomOf(addressId) {
+    return addressId && !isAgreementLaneId(addressId) ? addressId : null;
+}
 /** A named room on the call wins; otherwise the wake's chat. Empty is omit. */
 function namedSubcontractChat(payload, wakeChatId) {
-    const named = payload.chatId;
+    const named = payload.addressId;
     if (typeof named === 'string' && named.trim())
         return named.trim();
     return wakeChatId;
@@ -379,7 +386,7 @@ function namedSubcontractChat(payload, wakeChatId) {
  * request over MCP stays open. A crash replay is still covered by the
  * wake-derived idempotency key.
  */
-async function subcontractAgreement(services, payload, chatId, writeKeys = null) {
+async function subcontractAgreement(services, payload, addressId, writeKeys = null) {
     if (!payload.description || !payload.executorId)
         return null;
     // The price the model stated, or none; the server falls back to the
@@ -390,5 +397,5 @@ async function subcontractAgreement(services, payload, chatId, writeKeys = null)
         description: payload.description,
         parentAgreementId: payload.parentAgreementId ?? '',
     });
-    return services.agreementService.delegate({ ...fields, chatId, idempotencyKey });
+    return services.agreementService.delegate({ ...fields, addressId, idempotencyKey });
 }

package/dist/platform/tools/tier1/protocolTools.js

@@ -19,7 +19,7 @@ const planReviewTimingParam = {
  */
 const dueAtParam = {
     type: 'string',
-    description: 'Deadline, ISO 8601 (e.g. 2026-09-12T17:00:00Z). When you hold a task with this set: at the deadline, post what you have and where you are stuck in the room where the question was asked, then stop. Do not keep chasing past it. A deadline already past is refused.',
+    description: 'Deadline, ISO 8601 (e.g. 2026-09-12T17:00:00Z). When you hold a task with this set: at the deadline, post what you have and where you are stuck to the task’s authorized recipients, then stop. Do not keep chasing past it. A deadline already past is refused.',
 };
 const checkBackParam = {
     type: 'object',
@@ -47,7 +47,7 @@ function handlerFor(toolName) {
             taskService: c.taskService,
             agreementService: c.agreementService,
             contextReadClient: c.contextReadClient,
-        }, { operation, ...args }, c.chatId ?? null, c.agents ?? [], typeof c.agentId === 'string' ? c.agentId : null, c.writeKeys ?? null, Array.isArray(c.agreements) ? c.agreements : undefined);
+        }, { operation, ...args }, c.addressId ?? null, c.agents ?? [], typeof c.agentId === 'string' ? c.agentId : null, c.writeKeys ?? null, Array.isArray(c.agreements) ? c.agreements : undefined);
         if (result == null)
             throw new Error(`Operation ${operation} returned no result`);
         return result;

package/dist/platform/tools/tier1/wakeTools.d.ts

@@ -22,7 +22,7 @@ export interface WakeSchedulerPort {
     };
 }
 export interface WakeToolContext {
-    chatId?: string;
+    addressId?: string;
     sessionId?: string;
     wakeScheduler?: WakeSchedulerPort;
 }

package/dist/platform/tools/tier1/wakeTools.js

@@ -15,7 +15,8 @@ export const wakeAtTool = defineTool('wake_at', {
     if (!scheduler) {
         throw new Error('wake_at is unavailable here (no wake scheduler in tool context — expected when running under Agent)');
     }
-    const sessionId = c.chatId ?? c.sessionId;
+    // The timer belongs to the processing lane, not to a conversation.
+    const sessionId = c.sessionId;
     if (!sessionId)
         throw new Error('wake_at: no session in tool context');
     const raw = typeof args.at === 'string' ? args.at.trim() : '';

package/dist/platform/tools/tier2/mcpConnectionTools.js

@@ -66,7 +66,7 @@ async function withClient(ctx, toolName, connectionId, grantId, run) {
      * The lane comes from `credsFromToolContext`, the same reader every other
      * tool uses. Reading it off a field of its own (`ctx.laneId`) is how this
      * door came to send no lane at all: the executor states the lane as
-     * `chatId` / `sessionId`, so a key by any other name is always undefined,
+     * `addressId` / `sessionId`, so a key by any other name is always undefined,
      * and the gateway then resolves the wake to the agent's own operator and
      * refuses every grant its customer gave it.
      */

package/dist/runtime/AgentMachine.d.ts

@@ -149,8 +149,8 @@ export declare class AgentMachine {
      * it is a lane stuck in a room whose transitions were written for one
      * occasion, so every later event is evaluated against the wrong state and
      * anything that state does not route is swallowed. Measured: three lab
-     * workers tripped the depth guard inside the injected `claimKickingOff`
-     * greeting; `claimKickingOff` has no `task-assigned` route, so the task wake
+     * workers tripped the depth guard inside an injected claim greeting (since
+     * removed) that had no `task-assigned` route, so the task wake
      * that arrived next was dropped and three graph nodes were never worked at
      * all. The greeting is best-effort; the work is not.
      *

package/dist/runtime/AgentMachine.js

@@ -1,3 +1,4 @@
+import { responseCommand } from './messageCommand.js';
 import { runtimeLog } from '../shared/runtimeLog.js';
 import { applyOutcomeToCtx, outcomeFromEvent } from './buildOutcome.js';
 import { runMechanical } from './runMechanical.js';
@@ -155,8 +156,8 @@ export class AgentMachine {
      * it is a lane stuck in a room whose transitions were written for one
      * occasion, so every later event is evaluated against the wrong state and
      * anything that state does not route is swallowed. Measured: three lab
-     * workers tripped the depth guard inside the injected `claimKickingOff`
-     * greeting; `claimKickingOff` has no `task-assigned` route, so the task wake
+     * workers tripped the depth guard inside an injected claim greeting (since
+     * removed) that had no `task-assigned` route, so the task wake
      * that arrived next was dropped and three graph nodes were never worked at
      * all. The greeting is best-effort; the work is not.
      *
@@ -400,9 +401,7 @@ export class AgentMachine {
             await this.eff({
                 kind: 'send-message',
                 sessionId,
-                receiverId: userId,
-                text,
-                messageId: this.writeKeys?.messageKey({ text, sessionId, receiverId: userId }),
+                command: await responseCommand(this.eff, sessionId, text, { recipientId: userId, operationId: this.writeKeys?.messageKey({ text, sessionId, receiverId: userId }) }),
             });
         }
         catch { /* best-effort */ }

package/dist/runtime/PromptBuilder.d.ts

@@ -25,7 +25,7 @@ interface BuildArgs {
     definition: {
         description?: string;
     };
-    chatId: string | null;
+    addressId: string | null;
     stateId: string;
     ctx: Ctx;
     /** When true, respond-action schemas omit the "message" field — used for 2-call streaming. */

package/dist/runtime/PromptBuilder.js

@@ -115,8 +115,9 @@ ${this._renderDecisionSchemaFromActions(actions, senderMeta, ctxObj, omitMessage
         if (!ctx)
             return stateId ? `<machine_context>\n${JSON.stringify({ stateId }, null, 2)}\n</machine_context>` : '';
         const out = { stateId };
+        // The processing lane, shown as what it is: a lane name is not an address.
         if (ctx.identity?.sessionId)
-            out.chatId = ctx.identity.sessionId;
+            out.lane = ctx.identity.sessionId;
         if (ctx.identity?.laneKey)
             out.laneKey = ctx.identity.laneKey;
         if (ctx.identity?.agentId)
@@ -326,7 +327,7 @@ ${this._renderDecisionSchemaFromActions(actions, senderMeta, ctxObj, omitMessage
         // ref as it came; the backend resolves it in-room. Transfer (and any
         // account-id action) must never list opaque refs — lookup / wake / pay
         // against them 400s.
-        const chatIds = new Set([...userIds, ...agentIds]);
+        const addressIds = new Set([...userIds, ...agentIds]);
         const transferIds = new Set([...userIds, ...agentIds].filter((id) => !isOpaquePresentationRef(id)));
         const lines = ['<recipients>'];
         lines.push('<users>');
@@ -344,10 +345,10 @@ ${this._renderDecisionSchemaFromActions(actions, senderMeta, ctxObj, omitMessage
                 lines.push(`  - ${agentLabels.get(id)}`);
         lines.push('</agents>');
         lines.push('<chat>');
-        if (chatIds.size === 0)
+        if (addressIds.size === 0)
             lines.push('  (none)');
         else
-            for (const id of chatIds)
+            for (const id of addressIds)
                 lines.push(`  - ${tagged(id)}`);
         lines.push('</chat>');
         lines.push('<agreement>');

package/dist/runtime/buildOutcome.d.ts

@@ -29,7 +29,7 @@ export interface EmittedEvent {
     result?: WireResult;
     error?: string;
     receiverId?: string;
-    chatId?: string;
+    addressId?: string;
     message?: string;
     text?: string;
     operation?: string;

package/dist/runtime/deliveryWriteKeys.d.ts

@@ -10,7 +10,7 @@ import type { DeliveryWriteKeys } from '../types.js';
  *   the transition itself, so it dedups regardless of the order in which the
  *   replayed wake happens to re-issue its tool calls.
  * - chat messages:    `sdk_idem_wk_<wake-hash>_m_<hash(chat, receiver, text)>`
- *   — used directly as the messageId (the backend dedups on (chatId, messageId),
+ *   — used directly as the messageId (the backend dedups on (addressId, messageId),
  *). Content-derived, not ordinal: a wake may emit several
  *   distinct messages without colliding, and a crash-replay that re-sends the
  *   same text reuses the key; a re-think with different text gets a new key

package/dist/runtime/deliveryWriteKeys.js

@@ -79,7 +79,7 @@ function sha(input) {
  *   the transition itself, so it dedups regardless of the order in which the
  *   replayed wake happens to re-issue its tool calls.
  * - chat messages:    `sdk_idem_wk_<wake-hash>_m_<hash(chat, receiver, text)>`
- *   — used directly as the messageId (the backend dedups on (chatId, messageId),
+ *   — used directly as the messageId (the backend dedups on (addressId, messageId),
  *). Content-derived, not ordinal: a wake may emit several
  *   distinct messages without colliding, and a crash-replay that re-sends the
  *   same text reuses the key; a re-think with different text gets a new key

package/dist/runtime/kickoff.d.ts

@@ -1,60 +1,31 @@
 import type { ThinkingState, Transition, Workflow } from '../types.js';
 /** Reserved state name for the injected agreement-kickoff default (proposer). */
 export declare const KICKOFF_STATE = "kickingOff";
-/**
- * reserved state for the injected claim/provider kickoff default.
- * Store listing claimed → open buyer chat + greet (not task_create).
- */
-export declare const CLAIM_KICKOFF_STATE = "claimKickingOff";
 export declare function buildKickoffState(initial: string): ThinkingState;
-/** provider side after a store/listing claim. */
 /**
- * Declare that THIS state answers a store-claim wake itself, so the injected
- * `claimKickingOff` greeting is not added to it.
+ * Route a store-claim wake from this state to `to`. The SDK no longer greets
+ * on a claim (a claim opens no room; the buyer writes first in the agreement's
+ * conversation), so this is only for an author who wants to act on the claim
+ * itself.
  *
- * The opt-out has to fire on a claim wake and NOT on a bare counterparty
- * approve — `stateHandlesClaimKickoff` uses that difference to tell a
- * deliberate claim handler from a generic `proposal-resolved` route that
- * happens to match. Writing that predicate by hand is easy to get subtly wrong,
- * and getting it wrong is silent: the agent greets on every claim while its
- * author believes it opted out. That cost one lab cast 100% of its token spend
- * and three wedged workers before anyone noticed.
+ * It fires on a claim wake and NOT on a bare counterparty approve, which also
+ * keeps it out of the way of the proposer-side kickoff.
  *
  *   transitions: [
  *     handlesClaimWake('idle'),   // a claim does not make this agent talk
  *     { to: 'idle', when: (o) => o.kind === 'proposal-resolved' && o.action === 'approve' },
  *   ]
- *
- * Note the second route is still worth keeping when you also want to sit out
- * the PROPOSER-side kickoff: that one probes with a bare approve, which this
- * helper deliberately does not match.
  */
 export declare function handlesClaimWake(to: string): Transition;
-export declare function buildClaimKickoffState(initial: string): ThinkingState;
 /**
- * Agreement-kickoff defaults. For every state whose transitions do not already
- * handle the corresponding probe, add a transition to the injected kickoff
- * state. Returns a NEW workflow — input states are not mutated.
- *
- * Two kickoffs:
- *  - `kickingOff` — proposer-side approval → task_create
- *  - `claimKickingOff` — store-claim provider wake → chat_open + greet
- * Direct-proposal counterparties (lifecycle approve, !proposedByMe, no
- * fromClaimWake) still never auto-kickoff.
- *
- * **They are ordered differently on purpose** (first match wins in
- * AgentMachine._evaluateTransitions):
+ * Agreement-kickoff default. For every state whose transitions do not already
+ * handle a proposer-side approval, append a transition to the injected
+ * `kickingOff` state, which spawns the first task. Appended LAST: the approval
+ * is of a proposal THIS agent made, so a route its author wrote keeps winning.
+ * Returns a NEW workflow; input states are not mutated.
  *
- *  - `kickingOff` is appended LAST. It fires on an approval of a proposal THIS
- *    agent made, which the author was already expecting — whatever route they
- *    wrote for it should keep winning.
- *  - `claimKickingOff` is inserted FIRST. It fires on a wake the author never
- *    wrote a route for, and it has to beat the unconditional fallthrough an
- *    `idle` state uses to reach its listening loop — otherwise the claim is
- *    swallowed into the idle loop and the buyer is never greeted. Going first
- *    is safe because its `when` is narrow: only a `fromClaimWake` approve
- *    matches, so nothing else is shadowed. An author who wants the claim for
- *    themselves keys on `fromClaimWake`, which suppresses the injection
- *    outright (stateHandlesClaimKickoff).
+ * Counterparties never auto-kickoff, and neither does a store claim: a claim
+ * opens no room, and the buyer's first message or task is what wakes the
+ * provider.
  */
 export declare function injectKickoffDefault(workflow: Workflow): Workflow;