@ziggs-ai/agent-sdk 0.7.2 → 0.8.1

package/dist/agent/ClaudeSdkAgent.d.ts

@@ -1,7 +1,22 @@
 import { query } from '@anthropic-ai/claude-agent-sdk';
 import type { Workflow } from '../types.js';
 import type { ToolManager } from '../tools/ToolManager.js';
+import { type RawEvent } from '../runtime/buildOutcome.js';
 import type { TickInput, TickOutput } from './Agent.js';
+/**
+ * ZIG-1035 / ZIG-1052: status notices must not spend a Claude turn answering
+ * nobody. Real work wakes arrive as chat `message`, synthetic `task_result`
+ * from `wakeTaskAssignment`, or `agreement_lifecycle` (hire / proposal
+ * resolution — the FSM kickoffs on the same event; Claude must act too).
+ *
+ * Intentionally not mirrored as an FSM "notice skip": the FSM already no-ops
+ * enter-only outcomes in AgentMachine; `agreement_lifecycle` never produces
+ * `enter`, so hire wakes always step. Claude's filter is narrower — pure
+ * `notification` only.
+ *
+ * Empty / missing batch → notice-only (do not burn a turn on a malformed wake).
+ */
+export declare function isNoticeOnlyWake(event: RawEvent | undefined): boolean;
 export type CognitionKind = 'fsm' | 'claude-sdk';
 export interface ClaudeSdkAgentOptions {
     workflow: Workflow;

package/dist/agent/ClaudeSdkAgent.js

@@ -9,19 +9,23 @@ import { claudeQueryOptions, buildClaudeEventPrompt, summarizeContext, extractAs
 import { assertHostedAgentZiggsTools } from './hostedAgentConnection.js';
 import { runtimeLog } from '../shared/runtimeLog.js';
 /**
- * ZIG-1035: lifecycle / status notices must not spend a Claude turn answering
- * nobody. Real work wakes arrive as chat `message` or as synthetic `task_result`
- * from `wakeTaskAssignment` (task channel). Terminal task_result notices still
- * run so orchestrators can consume completions.
+ * ZIG-1035 / ZIG-1052: status notices must not spend a Claude turn answering
+ * nobody. Real work wakes arrive as chat `message`, synthetic `task_result`
+ * from `wakeTaskAssignment`, or `agreement_lifecycle` (hire / proposal
+ * resolution — the FSM kickoffs on the same event; Claude must act too).
+ *
+ * Intentionally not mirrored as an FSM "notice skip": the FSM already no-ops
+ * enter-only outcomes in AgentMachine; `agreement_lifecycle` never produces
+ * `enter`, so hire wakes always step. Claude's filter is narrower — pure
+ * `notification` only.
+ *
+ * Empty / missing batch → notice-only (do not burn a turn on a malformed wake).
  */
-function isNoticeOnlyWake(event) {
+export function isNoticeOnlyWake(event) {
     const events = getBatchEvents(event);
     if (events.length === 0)
-        return false;
-    return events.every((ev) => {
-        const t = ev?.type;
-        return t === 'notification' || t === 'agreement_lifecycle';
-    });
+        return true;
+    return events.every((ev) => ev?.type === 'notification');
 }
 const DEFAULT_STATE = 'ready';
 /**

package/dist/agent/claudeCognition.js

@@ -72,17 +72,27 @@ export function summarizeContext(snap) {
 function renderEventLines(event, lines) {
     if (event?.type)
         lines.push(`Event type: ${event.type}`);
-    // ZIG-1035: lifecycle notices are events, not counterparty speech.
+    // ZIG-1035 / ZIG-1052: notifications are not counterparty speech.
+    // agreement_lifecycle is a work signal (hire approval) — label it as such.
     if (event?.text) {
-        const notice = event.type === 'notification' ||
-            event.type === 'agreement_lifecycle' ||
-            event.entryType === 'notification';
-        lines.push(`${notice ? 'Notification' : 'Message'}: ${event.text}`);
+        const label = event.type === 'agreement_lifecycle'
+            ? 'Agreement lifecycle'
+            : event.type === 'notification' || event.entryType === 'notification'
+                ? 'Notification'
+                : 'Message';
+        lines.push(`${label}: ${event.text}`);
     }
     if (event?.senderId)
         lines.push(`Sender: ${event.senderId}`);
     if (event?.operation)
         lines.push(`Operation: ${event.operation}`);
+    if (event?.type === 'agreement_lifecycle' &&
+        (event.operation === 'proposal_approved' || event.operation === 'proposal_approved_execute')) {
+        lines.push('Work signal: proposal approved. If you proposed this agreement, spawn the first task (task_create) under it now. If you are the hired provider, begin fulfilling per the agreement — do not ignore this wake.');
+    }
+    if (typeof event?.agreementId === 'string' && event.agreementId) {
+        lines.push(`Agreement: ${event.agreementId}`);
+    }
     // ZIG-832: surface the task/agreement this event carries. Without it the brain
     // can't tell WHICH task to update/complete (workers otherwise stall asking for
     // their own taskId) and can't see whether the referenced step already ran or

package/dist/ingress/normalizeIncoming.d.ts

@@ -64,7 +64,14 @@ interface WireMetadata {
     service?: {
         task?: WireTask;
         operation?: string;
-        agreement?: unknown;
+        agreement?: {
+            agreementId?: string;
+            status?: string;
+            proposal?: {
+                status?: string;
+            };
+            parties?: WireParties;
+        };
         agreementId?: string;
     };
     [key: string]: unknown;

package/dist/ingress/normalizeIncoming.js

@@ -19,12 +19,41 @@ function isTaskRelated(metadata, taskData) {
         || contentType === 'task' || contentType === 'task_update'
         || Boolean(taskData.state || taskData.proposal || taskData.parentTaskId || taskData.executorId || taskData.agentId || taskData.payerId || taskData.createdBy));
 }
+function wireOperation(metadata) {
+    const op = metadata.service?.operation ?? metadata.operation;
+    return typeof op === 'string' ? op : undefined;
+}
+function wireAgreementId(metadata) {
+    const raw = metadata.service?.agreementId ??
+        metadata.service?.agreement?.agreementId ??
+        metadata.agreementId;
+    return typeof raw === 'string' ? raw : '';
+}
+/**
+ * Self-hire / fully-consented create activates before the chat fan-out lands.
+ * The wire still says `proposal_created`, but the embedded agreement is already
+ * `active` / proposal `approved`. That is the hire work signal (same as
+ * `proposal_approved_execute`) — Claude must not park it as a notice (ZIG-1052).
+ */
+function isAlreadyActivatedProposalCreated(metadata) {
+    if (wireOperation(metadata) !== 'proposal_created')
+        return false;
+    const agr = metadata.service?.agreement;
+    if (!agr || typeof agr !== 'object')
+        return false;
+    const status = agr.status;
+    const proposalStatus = agr.proposal?.status;
+    return status === 'active' || proposalStatus === 'approved';
+}
 function isAgreementLifecycleNotification(metadata) {
     const entryType = metadata.entryType || 'message';
-    const operation = metadata.service?.operation ?? metadata.operation;
-    return (entryType === 'notification' &&
-        typeof operation === 'string' &&
-        PROPOSAL_RESOLVED_OPERATIONS.has(operation));
+    if (entryType !== 'notification')
+        return false;
+    const operation = wireOperation(metadata);
+    if (typeof operation === 'string' && PROPOSAL_RESOLVED_OPERATIONS.has(operation)) {
+        return true;
+    }
+    return isAlreadyActivatedProposalCreated(metadata);
 }
 function isTerminalTaskState(state) {
     return state === 'completed' || state === 'failed' || state === 'cancelled';
@@ -57,9 +86,10 @@ export function normalizeIncomingEvent({ text, metadata = {} }) {
         ...(entryType ? { entryType } : {}),
     };
     if (isAgreementLifecycleNotification(metadata)) {
-        const operation = metadata.service?.operation ?? metadata.operation;
-        const rawAgreementId = metadata.service?.agreementId ?? metadata.agreementId;
-        const agreementId = typeof rawAgreementId === 'string' ? rawAgreementId : '';
+        const operation = isAlreadyActivatedProposalCreated(metadata)
+            ? 'proposal_approved_execute'
+            : wireOperation(metadata);
+        const agreementId = wireAgreementId(metadata);
         return {
             chatId,
             event: {

package/dist/server/agreements/AgreementService.d.ts

@@ -1,4 +1,4 @@
-import { type Agreement, type ProposeAgreementData, type ProposeDirectInput, type ProposeBroadcastInput, type PublishOfferPayload, type DelegateAgreementData, type CounterAgreementData, type CreateAgreementBody, type ListAgreementsFilters, type GetMyAgreementsFilters, type ChatLinkType, type ArtifactLinkType, type UserRole } from '@ziggs-ai/api-client';
+import { type Agreement, type ClaimedKind, type ProposeAgreementData, type ProposeDirectInput, type ProposeBroadcastInput, type PublishOfferPayload, type DelegateAgreementData, type CounterAgreementData, type CreateAgreementBody, type ListAgreementsFilters, type GetMyAgreementsFilters, type ChatLinkType, type ArtifactLinkType, type UserRole } from '@ziggs-ai/api-client';
 export declare class AgreementService {
     private creds;
     constructor(operatorKey: string, agentId: string);
@@ -20,12 +20,14 @@ export declare class AgreementService {
     delegate(payload: DelegateAgreementData): Promise<Agreement>;
     respond(agreementId: string, action: 'approve' | 'reject'): Promise<Agreement>;
     /**
-     * ZIG-1021 — claim any open broadcast (quest, standing offer, link invite).
-     * Routing (offer vs agreement claim endpoint) is decided in the shared flow.
+     * ZIG-1021 — claim any open broadcast (quest, standing offer, hand-off,
+     * link invite). Routing (offer vs agreement claim endpoint) is decided in
+     * the shared flow; `kind` is the api-client ClaimedKind so this signature
+     * can't drift from the flow again (a 'hand-off' addition broke the build).
      */
     claim(agreementId: string): Promise<{
         agreement: Agreement;
-        kind: 'link' | 'offer' | 'quest';
+        kind: ClaimedKind;
     }>;
     /**
      * ZIG-1022 — a link proposal is an agreement (engagementKind 'link') to one

package/dist/server/agreements/AgreementService.js

@@ -38,8 +38,10 @@ export class AgreementService {
         return respondToAgreement(agreementId, action, this.creds);
     }
     /**
-     * ZIG-1021 — claim any open broadcast (quest, standing offer, link invite).
-     * Routing (offer vs agreement claim endpoint) is decided in the shared flow.
+     * ZIG-1021 — claim any open broadcast (quest, standing offer, hand-off,
+     * link invite). Routing (offer vs agreement claim endpoint) is decided in
+     * the shared flow; `kind` is the api-client ClaimedKind so this signature
+     * can't drift from the flow again (a 'hand-off' addition broke the build).
      */
     claim(agreementId) {
         return claimOpenAgreement(agreementId, this.creds);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ziggs-ai/agent-sdk",
-  "version": "0.7.2",
+  "version": "0.8.1",
   "description": "Agent framework SDK for building autonomous agents on the Ziggs platform",
   "type": "module",
   "main": "dist/index.js",
@@ -26,7 +26,7 @@
     "@anthropic-ai/claude-agent-sdk": "^0.3.204",
     "@anthropic-ai/sdk": "^0.110.0",
     "@modelcontextprotocol/sdk": "^1.29.0",
-    "@ziggs-ai/api-client": "^0.6.3",
+    "@ziggs-ai/api-client": "^0.7.1",
     "ajv": "^8.17.1",
     "dotenv": "^17.2.3",
     "openai": "^4.0.0",