@naylence/agent-sdk 0.3.5-test.101 → 0.3.5-test.103

package/dist/browser/index.js

@@ -15674,12 +15674,12 @@
     // --- END ENV SHIM ---
 
     // This file is auto-generated during build - do not edit manually
-    // Generated from package.json version: 0.3.6
+    // Generated from package.json version: 0.3.6-test.108
     /**
      * The package version, injected at build time.
      * @internal
      */
-    const VERSION$1 = '0.3.6';
+    const VERSION$1 = '0.3.6-test.108';
 
     /**
      * Fame protocol specific error classes with WebSocket close codes and proper inheritance.
@@ -17275,6 +17275,50 @@
         });
     }
 
+    function isModuleNotFoundError(error) {
+        if (!(error instanceof Error)) {
+            return false;
+        }
+        const message = error.message || '';
+        if (message.includes('Cannot find module') ||
+            message.includes('ERR_MODULE_NOT_FOUND') ||
+            message.includes('MODULE_NOT_FOUND')) {
+            return true;
+        }
+        const code = error.code;
+        if (typeof code === 'string') {
+            return code === 'MODULE_NOT_FOUND' || code === 'ERR_MODULE_NOT_FOUND';
+        }
+        return false;
+    }
+    /**
+     * Wraps a dynamic import loader and enriches "module not found" failures with an actionable error message.
+     */
+    async function safeImport(loader, dependencyNameOrOptions, maybeOptions) {
+        const options = typeof dependencyNameOrOptions === 'string'
+            ? { dependencyName: dependencyNameOrOptions, ...(maybeOptions ?? {}) }
+            : dependencyNameOrOptions;
+        const dependencyName = options.dependencyName;
+        try {
+            return await loader();
+        }
+        catch (error) {
+            if (isModuleNotFoundError(error)) {
+                const message = options.helpMessage ??
+                    `Missing optional dependency "${dependencyName}". Install it to enable this feature.`;
+                const enrichedError = new Error(message);
+                try {
+                    enrichedError.cause = error;
+                }
+                catch {
+                    // Ignore environments that do not support attaching a cause.
+                }
+                throw enrichedError;
+            }
+            throw error;
+        }
+    }
+
     /**
      * flow_controller.ts - credit window management with cooperative back-pressure.
      *
@@ -18209,50 +18253,6 @@
         return SecretSource.normalize(value);
     }
 
-    function isModuleNotFoundError(error) {
-        if (!(error instanceof Error)) {
-            return false;
-        }
-        const message = error.message || '';
-        if (message.includes('Cannot find module') ||
-            message.includes('ERR_MODULE_NOT_FOUND') ||
-            message.includes('MODULE_NOT_FOUND')) {
-            return true;
-        }
-        const code = error.code;
-        if (typeof code === 'string') {
-            return code === 'MODULE_NOT_FOUND' || code === 'ERR_MODULE_NOT_FOUND';
-        }
-        return false;
-    }
-    /**
-     * Wraps a dynamic import loader and enriches "module not found" failures with an actionable error message.
-     */
-    async function safeImport(loader, dependencyNameOrOptions, maybeOptions) {
-        const options = typeof dependencyNameOrOptions === 'string'
-            ? { dependencyName: dependencyNameOrOptions, ...(maybeOptions ?? {}) }
-            : dependencyNameOrOptions;
-        const dependencyName = options.dependencyName;
-        try {
-            return await loader();
-        }
-        catch (error) {
-            if (isModuleNotFoundError(error)) {
-                const message = options.helpMessage ??
-                    `Missing optional dependency "${dependencyName}". Install it to enable this feature.`;
-                const enrichedError = new Error(message);
-                try {
-                    enrichedError.cause = error;
-                }
-                catch {
-                    // Ignore environments that do not support attaching a cause.
-                }
-                throw enrichedError;
-            }
-            throw error;
-        }
-    }
-
     const indexedDBConfigSchema = object({
         type: literal('IndexedDBStorageProvider')
             .default('IndexedDBStorageProvider'),
@@ -21831,9 +21831,6 @@
             this.ackDoneSince = new Map();
             this.replyDoneSince = new Map();
             this.pendingAckDispatches = new Set();
-            this.recentlyHandled = new Map();
-            this.recentlyHandledOrder = [];
-            this.recentlyHandledTtlMs = 60000;
             this.isPreparingToStop = false;
             this.shutdownRequestedAtMs = null;
             this.shutdownRetryGraceMs = 1000;
@@ -22055,22 +22052,6 @@
                     }
                 }
                 else {
-                    const wasRecentlyHandled = await this.lock.runExclusive(async () => this.wasRecentlyHandled(envelope.id));
-                    if (wasRecentlyHandled) {
-                        logger$12.debug('tracker_duplicate_envelope_recently_handled', {
-                            envp_id: envelope.id,
-                        });
-                        return new TrackedEnvelope({
-                            timeoutAtMs: 0,
-                            overallTimeoutAtMs: 0,
-                            expectedResponseType: envelope.rtype ?? FameResponseType.NONE,
-                            createdAtMs: Date.now(),
-                            status: EnvelopeStatus.HANDLED,
-                            mailboxType: MailboxType.INBOX,
-                            originalEnvelope: envelope,
-                            serviceName: inboxName,
-                        });
-                    }
                     tracked = new TrackedEnvelope({
                         timeoutAtMs: 0,
                         overallTimeoutAtMs: 0,
@@ -22092,12 +22073,8 @@
         async onEnvelopeHandled(envelope) {
             const inbox = this.ensureInbox();
             envelope.status = EnvelopeStatus.HANDLED;
+            // Delete the envelope from inbox to prevent growth
             await inbox.delete(envelope.originalEnvelope.id);
-            await this.lock.runExclusive(async () => {
-                this.markRecentlyHandled(envelope.originalEnvelope.id);
-            });
-            // Preserve handled envelope to prevent duplicate redelivery during shutdown drains.
-            // await inbox.set(envelope.originalEnvelope.id, envelope);
         }
         async onEnvelopeHandleFailed(inboxName, envelope, context, error, isFinalFailure = false) {
             const inbox = this.ensureInbox();
@@ -22312,9 +22289,9 @@
             });
             await this.markDoneSince(this.replyFutures, trackedEnvelope.originalEnvelope.id, this.replyDoneSince);
             await this.markDoneSince(this.ackFutures, trackedEnvelope.originalEnvelope.id, this.ackDoneSince);
-            if (envelope.rtype && Boolean(envelope.rtype & FameResponseType.ACK)) {
-                await this.sendAck(envelope);
-            }
+            // Note: ACK is already sent in onCorrelatedMessage (lines 655-657)
+            // when the reply envelope is first delivered. No need to send it again here.
+            // Removing this duplicate sendAck call fixes the duplicate DeliveryAck bug.
             for (const handler of this.eventHandlers) {
                 await handler.onEnvelopeReplied?.(trackedEnvelope, envelope);
             }
@@ -22464,8 +22441,6 @@
                 }
                 this.streamDone.clear();
                 this.correlationToEnvelope.clear();
-                this.recentlyHandled.clear();
-                this.recentlyHandledOrder.length = 0;
                 return values;
             });
             for (const timer of timers) {
@@ -23047,39 +23022,6 @@
                 this.pendingAckDispatches.delete(ackDispatch);
             }
         }
-        markRecentlyHandled(envelopeId) {
-            const now = Date.now();
-            this.recentlyHandled.set(envelopeId, now);
-            this.recentlyHandledOrder.push(envelopeId);
-            this.trimRecentlyHandled(now);
-        }
-        wasRecentlyHandled(envelopeId) {
-            const now = Date.now();
-            const timestamp = this.recentlyHandled.get(envelopeId);
-            if (timestamp === undefined) {
-                return false;
-            }
-            if (now - timestamp > this.recentlyHandledTtlMs) {
-                this.recentlyHandled.delete(envelopeId);
-                return false;
-            }
-            return true;
-        }
-        trimRecentlyHandled(now) {
-            while (this.recentlyHandledOrder.length > 0) {
-                const candidate = this.recentlyHandledOrder[0];
-                const timestamp = this.recentlyHandled.get(candidate);
-                if (timestamp === undefined) {
-                    this.recentlyHandledOrder.shift();
-                    continue;
-                }
-                if (now - timestamp <= this.recentlyHandledTtlMs) {
-                    break;
-                }
-                this.recentlyHandled.delete(candidate);
-                this.recentlyHandledOrder.shift();
-            }
-        }
         getShutdownRetryDeferDelay(nowMs) {
             if (!this.isPreparingToStop || this.shutdownRequestedAtMs === null) {
                 return null;
@@ -24652,6 +24594,7 @@
                 local_node_id: this.localNodeId,
                 target_node_id: this.targetNodeId ?? null,
                 inbox_capacity: preferredCapacity,
+                passive: config.passive ?? false,
                 timestamp: new Date().toISOString(),
             });
             this.onMsg = (event) => {
@@ -25492,6 +25435,20 @@
             await connector.start(this.wrappedHandler);
             this.connector = connector;
             const callbackGrants = this.node.gatherSupportedCallbackGrants();
+            logger$Z.debug('callback_grants_before_augmentation', {
+                count: callbackGrants.length,
+                types: callbackGrants.map(g => g.type),
+            });
+            // Check if we should create a broadcast callback grant before processing connection grants
+            // This prevents adding duplicate broadcast grants
+            const shouldAddBroadcastGrant = this.shouldAdvertiseBroadcastGrant(grant, callbackGrants);
+            const broadcastCallbackGrant = shouldAddBroadcastGrant
+                ? this.createBroadcastCallbackGrant(grant)
+                : null;
+            logger$Z.debug('broadcast_callback_grant_check', {
+                should_add: shouldAddBroadcastGrant,
+                grant_created: !!broadcastCallbackGrant,
+            });
             // Include admission client's connection grants as callback grants
             // This ensures DirectAdmissionClient grants are available for grant selection
             if (welcome.frame.connectionGrants && Array.isArray(welcome.frame.connectionGrants)) {
@@ -25501,16 +25458,31 @@
                         const isDuplicate = callbackGrants.some(existing => JSON.stringify(existing) === JSON.stringify(grant));
                         if (!isDuplicate) {
                             callbackGrants.push(grant);
+                            logger$Z.debug('added_connection_grant_as_callback', {
+                                type: grant.type,
+                            });
+                        }
+                        else {
+                            logger$Z.debug('skipped_duplicate_connection_grant', {
+                                type: grant.type,
+                            });
                         }
                     }
                 }
             }
-            if (this.shouldAdvertiseBroadcastGrant(grant, callbackGrants)) {
-                const augmented = this.createBroadcastCallbackGrant(grant);
-                if (augmented) {
-                    callbackGrants.push(augmented);
-                }
+            // Add broadcast grant after connection grants to ensure we don't duplicate
+            // any broadcast grants that may have been in connectionGrants
+            if (broadcastCallbackGrant && this.shouldAdvertiseBroadcastGrant(grant, callbackGrants)) {
+                callbackGrants.push(broadcastCallbackGrant);
+                logger$Z.debug('added_broadcast_callback_grant');
+            }
+            else if (broadcastCallbackGrant) {
+                logger$Z.debug('skipped_duplicate_broadcast_callback_grant');
             }
+            logger$Z.debug('callback_grants_after_augmentation', {
+                count: callbackGrants.length,
+                types: callbackGrants.map(g => g.type),
+            });
             const attachInfo = await this.attachClient.attach(this.node, this.outboundOriginType, connector, welcome.frame, this.wrappedHandler, this.getKeys() ?? undefined, callbackGrants);
             this.targetSystemId = attachInfo.targetSystemId ?? null;
             if (this.targetSystemId) {
@@ -27306,15 +27278,29 @@
         constructor(options = {}) {
             this.buffer = [];
             this.inHandshake = false;
+            this.expectedSystemId = null;
             this.timeoutMs = options.timeoutMs ?? 10000;
             this.attachmentKeyValidator = options.attachmentKeyValidator;
             this.replicaStickinessManager = options.replicaStickinessManager ?? null;
         }
         async attach(node, originType, connector, welcomeFrame, finalHandler, keys, callbackGrants) {
             this.inHandshake = true;
+            this.expectedSystemId = welcomeFrame.systemId;
             const interimHandler = async (envelope, context) => {
                 if (this.inHandshake) {
-                    this.buffer.push(envelope);
+                    // Filter: only buffer frames related to our systemId or frames without systemId info
+                    const frameSystemId = envelope.frame?.systemId;
+                    if (!frameSystemId || frameSystemId === this.expectedSystemId) {
+                        this.buffer.push(envelope);
+                    }
+                    else {
+                        // Silently ignore frames from other agents during concurrent handshakes
+                        logger$W.debug('handshake_ignoring_frame_from_different_system', {
+                            frame_type: envelope.frame.type,
+                            frame_system_id: frameSystemId,
+                            expected_system_id: this.expectedSystemId,
+                        });
+                    }
                     return null;
                 }
                 return finalHandler(envelope, context);
@@ -27441,6 +27427,7 @@
                 parent_id: ackFrame.targetSystemId,
             });
             this.inHandshake = false;
+            this.expectedSystemId = null;
             await connector.replaceHandler(finalHandler);
             while (this.buffer.length > 0) {
                 const bufferedEnvelope = this.buffer.shift();
@@ -27517,9 +27504,20 @@
                     if (envelope.frame.type === 'NodeAttachAck') {
                         return envelope;
                     }
-                    logger$W.error('unexpected_frame_during_handshake', {
-                        frame_type: envelope.frame.type,
-                    });
+                    // NodeAttach frames during handshake are expected in multi-agent scenarios
+                    // where multiple agents attach concurrently to the same channel
+                    if (envelope.frame.type === 'NodeAttach') {
+                        logger$W.debug('handshake_ignoring_concurrent_attach', {
+                            frame_type: envelope.frame.type,
+                            frame_system_id: envelope.frame?.systemId ?? 'unknown',
+                        });
+                    }
+                    else {
+                        // Other unexpected frames are still logged as errors
+                        logger$W.error('unexpected_frame_during_handshake', {
+                            frame_type: envelope.frame.type,
+                        });
+                    }
                 }
                 await delay$2(HANDSHAKE_POLL_INTERVAL_MS);
             }
@@ -43127,6 +43125,7 @@
                 inboxCapacity,
                 localNodeId,
                 initialTargetNodeId: resolvedTarget,
+                passive: normalized.passive,
             };
             const connector = new BroadcastChannelConnector(connectorConfig, baseConfig);
             if (options.authorization) {
@@ -43165,6 +43164,9 @@
             if (normalizedLocalNodeId) {
                 normalized.localNodeId = normalizedLocalNodeId;
             }
+            if (typeof candidate.passive === 'boolean') {
+                normalized.passive = candidate.passive;
+            }
             if (typeof candidate.flowControl === 'boolean') {
                 normalized.flowControl = candidate.flowControl;
             }
@@ -44508,11 +44510,19 @@
                 ? Math.floor(initialWindowCandidate)
                 : undefined;
             const initialTargetNodeId = this._normalizeNodeId(targetCandidate) ?? targetSystemId;
+            const passive = typeof passiveCandidate === 'boolean' ? passiveCandidate : true;
+            logger$o.debug('broadcast_channel_listener_building_connector_config', {
+                system_id: systemId,
+                channel_name: channelName,
+                passive,
+                has_base_config: !!baseConfig,
+                passive_candidate: passiveCandidate,
+            });
             return {
                 type: BROADCAST_CHANNEL_CONNECTOR_TYPE,
                 channelName,
                 inboxCapacity,
-                passive: typeof passiveCandidate === 'boolean' ? passiveCandidate : true,
+                passive,
                 initialWindow,
                 localNodeId,
                 initialTargetNodeId,