@dudousxd/nestjs-agent 1.16.1 → 1.17.0

package/dist/index.d.cts

@@ -1,4 +1,4 @@
-import { DynamicModule, Type, OnApplicationBootstrap, OnModuleInit } from '@nestjs/common';
+import { DynamicModule, Type, OnApplicationBootstrap, OnModuleInit, ConflictException } from '@nestjs/common';
 import { b as AgentModuleOptions, c as AgentModuleAsyncOptions, A as AgentDepsFactory, C as ChatQueueService, d as AgentSkillsOptions } from './chat-queue.service-Bqq00NN1.cjs';
 export { e as AGENT_TOOL_BRAND, f as AgentAttachmentsOptions, g as AgentDeps, h as AgentMemoryOptions, a as AgentSurface, F as FunctionalTool, L as LedgerQuotaProvider, Q as QueuePlan, i as QueueSettleOutcome, j as QueuedTurn, k as QueuedTurnStarter, l as QuotaLimits, m as QuotaWindowLimits, n as delegateToolName, p as provideAgentTool, o as provideAgentTools, u as utcDay } from './chat-queue.service-Bqq00NN1.cjs';
 import { AgentRunner, AgentStore, AttachmentStagingStore, ModelCatalog, QuotaProvider, Actor, QuotaReport, ModelCatalogView, AttachmentRef, PageContext, ChatQueueState, HumanReply, ThreadSummary, ThreadDetail, UpdateThreadInput, MessageFeedbackValue, MessageFeedback, StagedAttachment, AgentApprovalPort, ActorResolver, MessageAttachment, AgentAttachmentConfig, AgentClientConfig, ToolPresentation, AgentHistoryWindow, AgentIntake, SkillContext, ToolRegistry, AgentRegistry, PromptContributor, SkillsConfig, SkillCatalogEntry, ToolCatalogEntry, MemoryConfig, MemoryDigestEntry, SkillProvider, AgentRunInput, AgentRunStartOptions, TokenStreamSink, SinkWriter } from '@dudousxd/nestjs-agent-core';
@@ -249,6 +249,14 @@ declare class AgentService {
      * not the thread's own actor).
      */
     signalToolCall(toolCallId: string, reply: HumanReply): Promise<void>;
+    /**
+     * Refuse a decision addressed at a run that is over (`409 run_not_active`). The runtime would take
+     * the signal and buffer it for a run that never comes back — the person's "yes" is accepted, the
+     * card says so, and nothing runs. Settles the calls the dead run left awaiting a decision on the
+     * way out, so the card stops asking. A runner that cannot say whether a run is alive is taken at
+     * its word that it is, and so is one that fails to answer.
+     */
+    private assertRunWaiting;
     cancel(actor: Actor, runId: string): Promise<void>;
     listThreads(actorRef: string): Promise<ThreadSummary[]>;
     getThread(actor: Actor, threadId: string): Promise<ThreadDetail | null>;
@@ -1185,4 +1193,17 @@ declare function defaultRequestUserMapper(user: Record<string, unknown>): Actor;
  */
 declare function requestUserActorResolver<TUser = Record<string, unknown>>(map?: RequestUserMapper<TUser>): ActorResolver;
 
-export { AGENT_METADATA, AI_TOOL_METADATA, ATTACHMENT_PAGE_SIZE, Agent, AgentApprovalPortAdapter, AgentDepsFactory, AgentDiscoveryService, AgentModule, AgentModuleAsyncOptions, AgentModuleOptions, type AgentOptions, AgentService, AgentSkillsOptions, AiTool, AiToolDiscoveryService, type AiToolOptions, type AnonymousActorOptions, AnonymousActorResolver, type AttachmentLimits, AttachmentsController, type ChatParams, ChatQueueService, type ChatSendMode, type ChatSendResult, ConfigController, DEFAULT_ALLOWED_ATTACHMENT_CONTENT_TYPES, DEFAULT_MAX_ATTACHMENT_BYTES, type DeclaredSkill, HeaderActorResolver, InProcessTokenStreamSink, InlineAgentRunner, MAX_ATTACHMENTS_PER_MESSAGE, MemoriesController, MessagesController, ModelsController, QueueController, type QueuedSend, type RequestUserMapper, type ResolvedAiToolOptions, SKILL_METADATA, SYSTEM_PROMPT_CONTRIBUTOR_METADATA, SYSTEM_PROMPT_METADATA, Skill, type SkillBody, SkillDiscoveryService, type SkillOptions, SkillsController, SystemPrompt, SystemPromptContributor, type ThreadDefaultAgentReader, type ThreadModelReader, ToolsController, attachmentLimits, declaredSkillProvider, defaultRequestUserMapper, readAgentMetadata, readAiToolMetadata, readSkillMetadata, requestUserActorResolver, resolveSkillsConfig, skillScope, toolNameFromClass };
+/**
+ * `approve` / `reject` / `answer` / `skip` addressed at a run that is no longer running — answered
+ * `409 { code: 'run_not_active', message }`.
+ *
+ * The runtime would take the signal and buffer it for a run that never comes back: the person's
+ * "yes" is accepted, the card says so, and nothing runs. Refusing it is what lets a client say so.
+ */
+declare class RunNotActiveException extends ConflictException {
+    readonly runId: string;
+    readonly code = "run_not_active";
+    constructor(runId: string);
+}
+
+export { AGENT_METADATA, AI_TOOL_METADATA, ATTACHMENT_PAGE_SIZE, Agent, AgentApprovalPortAdapter, AgentDepsFactory, AgentDiscoveryService, AgentModule, AgentModuleAsyncOptions, AgentModuleOptions, type AgentOptions, AgentService, AgentSkillsOptions, AiTool, AiToolDiscoveryService, type AiToolOptions, type AnonymousActorOptions, AnonymousActorResolver, type AttachmentLimits, AttachmentsController, type ChatParams, ChatQueueService, type ChatSendMode, type ChatSendResult, ConfigController, DEFAULT_ALLOWED_ATTACHMENT_CONTENT_TYPES, DEFAULT_MAX_ATTACHMENT_BYTES, type DeclaredSkill, HeaderActorResolver, InProcessTokenStreamSink, InlineAgentRunner, MAX_ATTACHMENTS_PER_MESSAGE, MemoriesController, MessagesController, ModelsController, QueueController, type QueuedSend, type RequestUserMapper, type ResolvedAiToolOptions, RunNotActiveException, SKILL_METADATA, SYSTEM_PROMPT_CONTRIBUTOR_METADATA, SYSTEM_PROMPT_METADATA, Skill, type SkillBody, SkillDiscoveryService, type SkillOptions, SkillsController, SystemPrompt, SystemPromptContributor, type ThreadDefaultAgentReader, type ThreadModelReader, ToolsController, attachmentLimits, declaredSkillProvider, defaultRequestUserMapper, readAgentMetadata, readAiToolMetadata, readSkillMetadata, requestUserActorResolver, resolveSkillsConfig, skillScope, toolNameFromClass };

package/dist/index.d.ts

@@ -1,4 +1,4 @@
-import { DynamicModule, Type, OnApplicationBootstrap, OnModuleInit } from '@nestjs/common';
+import { DynamicModule, Type, OnApplicationBootstrap, OnModuleInit, ConflictException } from '@nestjs/common';
 import { b as AgentModuleOptions, c as AgentModuleAsyncOptions, A as AgentDepsFactory, C as ChatQueueService, d as AgentSkillsOptions } from './chat-queue.service-Bqq00NN1.js';
 export { e as AGENT_TOOL_BRAND, f as AgentAttachmentsOptions, g as AgentDeps, h as AgentMemoryOptions, a as AgentSurface, F as FunctionalTool, L as LedgerQuotaProvider, Q as QueuePlan, i as QueueSettleOutcome, j as QueuedTurn, k as QueuedTurnStarter, l as QuotaLimits, m as QuotaWindowLimits, n as delegateToolName, p as provideAgentTool, o as provideAgentTools, u as utcDay } from './chat-queue.service-Bqq00NN1.js';
 import { AgentRunner, AgentStore, AttachmentStagingStore, ModelCatalog, QuotaProvider, Actor, QuotaReport, ModelCatalogView, AttachmentRef, PageContext, ChatQueueState, HumanReply, ThreadSummary, ThreadDetail, UpdateThreadInput, MessageFeedbackValue, MessageFeedback, StagedAttachment, AgentApprovalPort, ActorResolver, MessageAttachment, AgentAttachmentConfig, AgentClientConfig, ToolPresentation, AgentHistoryWindow, AgentIntake, SkillContext, ToolRegistry, AgentRegistry, PromptContributor, SkillsConfig, SkillCatalogEntry, ToolCatalogEntry, MemoryConfig, MemoryDigestEntry, SkillProvider, AgentRunInput, AgentRunStartOptions, TokenStreamSink, SinkWriter } from '@dudousxd/nestjs-agent-core';
@@ -249,6 +249,14 @@ declare class AgentService {
      * not the thread's own actor).
      */
     signalToolCall(toolCallId: string, reply: HumanReply): Promise<void>;
+    /**
+     * Refuse a decision addressed at a run that is over (`409 run_not_active`). The runtime would take
+     * the signal and buffer it for a run that never comes back — the person's "yes" is accepted, the
+     * card says so, and nothing runs. Settles the calls the dead run left awaiting a decision on the
+     * way out, so the card stops asking. A runner that cannot say whether a run is alive is taken at
+     * its word that it is, and so is one that fails to answer.
+     */
+    private assertRunWaiting;
     cancel(actor: Actor, runId: string): Promise<void>;
     listThreads(actorRef: string): Promise<ThreadSummary[]>;
     getThread(actor: Actor, threadId: string): Promise<ThreadDetail | null>;
@@ -1185,4 +1193,17 @@ declare function defaultRequestUserMapper(user: Record<string, unknown>): Actor;
  */
 declare function requestUserActorResolver<TUser = Record<string, unknown>>(map?: RequestUserMapper<TUser>): ActorResolver;
 
-export { AGENT_METADATA, AI_TOOL_METADATA, ATTACHMENT_PAGE_SIZE, Agent, AgentApprovalPortAdapter, AgentDepsFactory, AgentDiscoveryService, AgentModule, AgentModuleAsyncOptions, AgentModuleOptions, type AgentOptions, AgentService, AgentSkillsOptions, AiTool, AiToolDiscoveryService, type AiToolOptions, type AnonymousActorOptions, AnonymousActorResolver, type AttachmentLimits, AttachmentsController, type ChatParams, ChatQueueService, type ChatSendMode, type ChatSendResult, ConfigController, DEFAULT_ALLOWED_ATTACHMENT_CONTENT_TYPES, DEFAULT_MAX_ATTACHMENT_BYTES, type DeclaredSkill, HeaderActorResolver, InProcessTokenStreamSink, InlineAgentRunner, MAX_ATTACHMENTS_PER_MESSAGE, MemoriesController, MessagesController, ModelsController, QueueController, type QueuedSend, type RequestUserMapper, type ResolvedAiToolOptions, SKILL_METADATA, SYSTEM_PROMPT_CONTRIBUTOR_METADATA, SYSTEM_PROMPT_METADATA, Skill, type SkillBody, SkillDiscoveryService, type SkillOptions, SkillsController, SystemPrompt, SystemPromptContributor, type ThreadDefaultAgentReader, type ThreadModelReader, ToolsController, attachmentLimits, declaredSkillProvider, defaultRequestUserMapper, readAgentMetadata, readAiToolMetadata, readSkillMetadata, requestUserActorResolver, resolveSkillsConfig, skillScope, toolNameFromClass };
+/**
+ * `approve` / `reject` / `answer` / `skip` addressed at a run that is no longer running — answered
+ * `409 { code: 'run_not_active', message }`.
+ *
+ * The runtime would take the signal and buffer it for a run that never comes back: the person's
+ * "yes" is accepted, the card says so, and nothing runs. Refusing it is what lets a client say so.
+ */
+declare class RunNotActiveException extends ConflictException {
+    readonly runId: string;
+    readonly code = "run_not_active";
+    constructor(runId: string);
+}
+
+export { AGENT_METADATA, AI_TOOL_METADATA, ATTACHMENT_PAGE_SIZE, Agent, AgentApprovalPortAdapter, AgentDepsFactory, AgentDiscoveryService, AgentModule, AgentModuleAsyncOptions, AgentModuleOptions, type AgentOptions, AgentService, AgentSkillsOptions, AiTool, AiToolDiscoveryService, type AiToolOptions, type AnonymousActorOptions, AnonymousActorResolver, type AttachmentLimits, AttachmentsController, type ChatParams, ChatQueueService, type ChatSendMode, type ChatSendResult, ConfigController, DEFAULT_ALLOWED_ATTACHMENT_CONTENT_TYPES, DEFAULT_MAX_ATTACHMENT_BYTES, type DeclaredSkill, HeaderActorResolver, InProcessTokenStreamSink, InlineAgentRunner, MAX_ATTACHMENTS_PER_MESSAGE, MemoriesController, MessagesController, ModelsController, QueueController, type QueuedSend, type RequestUserMapper, type ResolvedAiToolOptions, RunNotActiveException, SKILL_METADATA, SYSTEM_PROMPT_CONTRIBUTOR_METADATA, SYSTEM_PROMPT_METADATA, Skill, type SkillBody, SkillDiscoveryService, type SkillOptions, SkillsController, SystemPrompt, SystemPromptContributor, type ThreadDefaultAgentReader, type ThreadModelReader, ToolsController, attachmentLimits, declaredSkillProvider, defaultRequestUserMapper, readAgentMetadata, readAiToolMetadata, readSkillMetadata, requestUserActorResolver, resolveSkillsConfig, skillScope, toolNameFromClass };

package/dist/index.js

@@ -265,8 +265,8 @@ AgentDepsFactory2 = _ts_decorate([
 ], AgentDepsFactory2);
 
 // src/agent.service.ts
-import { AGENT_ATTACHMENT_STAGING as AGENT_ATTACHMENT_STAGING2, AGENT_DEPS_FACTORY, AGENT_MODEL_CATALOG, AGENT_OPTIONS as AGENT_OPTIONS3, AGENT_QUOTA_PROVIDER as AGENT_QUOTA_PROVIDER2, AGENT_RUNNER, AGENT_STORE as AGENT_STORE3, REQUESTER_APPROVER, findCatalogModel, mayDecideApproval, readElicitationQuestions, validateElicitationAnswer } from "@dudousxd/nestjs-agent-core";
-import { BadRequestException, ConflictException, ForbiddenException, GoneException, HttpException, HttpStatus, Inject as Inject3, Injectable as Injectable3, NotFoundException, NotImplementedException as NotImplementedException2, Optional as Optional3 } from "@nestjs/common";
+import { AGENT_ATTACHMENT_STAGING as AGENT_ATTACHMENT_STAGING2, AGENT_DEPS_FACTORY, AGENT_MODEL_CATALOG, AGENT_OPTIONS as AGENT_OPTIONS3, AGENT_QUOTA_PROVIDER as AGENT_QUOTA_PROVIDER2, AGENT_RUNNER, AGENT_STORE as AGENT_STORE3, REQUESTER_APPROVER, findCatalogModel, mayDecideApproval, readElicitationQuestions, settleDeadRun, validateElicitationAnswer } from "@dudousxd/nestjs-agent-core";
+import { BadRequestException, ConflictException as ConflictException2, ForbiddenException, GoneException, HttpException, HttpStatus, Inject as Inject3, Injectable as Injectable3, NotFoundException, NotImplementedException as NotImplementedException2, Optional as Optional3 } from "@nestjs/common";
 
 // src/agent-deps.ts
 function utcDay(date = /* @__PURE__ */ new Date()) {
@@ -711,6 +711,24 @@ function queueFrame(items, paused, started) {
 }
 __name(queueFrame, "queueFrame");
 
+// src/run-not-active.exception.ts
+import { RUN_NOT_ACTIVE_CODE, RUN_NOT_ACTIVE_MESSAGE } from "@dudousxd/nestjs-agent-core";
+import { ConflictException } from "@nestjs/common";
+var RunNotActiveException = class extends ConflictException {
+  static {
+    __name(this, "RunNotActiveException");
+  }
+  runId;
+  code = RUN_NOT_ACTIVE_CODE;
+  constructor(runId) {
+    super({
+      statusCode: 409,
+      code: RUN_NOT_ACTIVE_CODE,
+      message: RUN_NOT_ACTIVE_MESSAGE
+    }), this.runId = runId;
+  }
+};
+
 // src/agent.service.ts
 function _ts_decorate3(decorators, target, key, desc) {
   var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
@@ -908,7 +926,7 @@ var AgentService = class {
       };
     }
     await this.requireQueue().publish(result.threadId);
-    throw new ConflictException({
+    throw new ConflictException2({
       statusCode: 409,
       code: "run_active",
       message: `thread ${result.threadId} already has a turn running`
@@ -968,6 +986,11 @@ var AgentService = class {
     }
     const mode = params.mode ?? "auto";
     const { live, stale } = await queue.holder(threadId, this.runner);
+    if (stale !== null) {
+      await settleDeadRun(this.store, {
+        runId: stale
+      });
+    }
     if (live === null && mode !== "queue") {
       const runId = crypto.randomUUID();
       if (await queue.queueStore().claimActiveStream(threadId, runId, stale !== null ? {
@@ -980,7 +1003,7 @@ var AgentService = class {
       }
     }
     if (params.regenerate === true) {
-      throw new ConflictException({
+      throw new ConflictException2({
         statusCode: 409,
         code: "run_active",
         message: "cannot regenerate while a turn is running on this thread"
@@ -1328,8 +1351,29 @@ var AgentService = class {
   async signalToolCall(toolCallId2, reply) {
     await this.assertNotExpired(toolCallId2);
     const runId = await this.resolveRunForToolCall(toolCallId2);
+    await this.assertRunWaiting(runId);
     return this.runner.signal(runId, toolCallId2, reply);
   }
+  /**
+  * Refuse a decision addressed at a run that is over (`409 run_not_active`). The runtime would take
+  * the signal and buffer it for a run that never comes back — the person's "yes" is accepted, the
+  * card says so, and nothing runs. Settles the calls the dead run left awaiting a decision on the
+  * way out, so the card stops asking. A runner that cannot say whether a run is alive is taken at
+  * its word that it is, and so is one that fails to answer.
+  */
+  async assertRunWaiting(runId) {
+    if (typeof this.runner.isRunActive !== "function") {
+      return;
+    }
+    const active = await this.runner.isRunActive(runId).catch(() => true);
+    if (active) {
+      return;
+    }
+    await settleDeadRun(this.store, {
+      runId
+    });
+    throw new RunNotActiveException(runId);
+  }
   async cancel(actor, runId) {
     await this.assertOwnsActiveStream(actor, runId);
     return this.runner.cancel(runId);
@@ -2013,7 +2057,7 @@ AttachmentsController = _ts_decorate6([
 ], AttachmentsController);
 
 // src/controller/chat.controller.ts
-import { AGENT_ACTOR_RESOLVER as AGENT_ACTOR_RESOLVER2, AgentStreamError } from "@dudousxd/nestjs-agent-core";
+import { AGENT_ACTOR_RESOLVER as AGENT_ACTOR_RESOLVER2, AgentStreamError, streamFailure } from "@dudousxd/nestjs-agent-core";
 import { BadRequestException as BadRequestException3, Body, Controller as Controller3, Get as Get3, Inject as Inject6, Param, Post as Post2, Query, Req as Req2, Res } from "@nestjs/common";
 function _ts_decorate7(decorators, target, key, desc) {
   var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
@@ -2209,10 +2253,7 @@ data: ${line}
       const payload = error instanceof AgentStreamError ? {
         code: error.code,
         message: error.message
-      } : {
-        code: "run_failed",
-        message: error instanceof Error ? error.message : "stream error"
-      };
+      } : streamFailure(error);
       res.write(`event: error
 data: ${JSON.stringify(payload)}
 
@@ -4228,7 +4269,7 @@ var LedgerQuotaProvider = class {
 };
 
 // src/runner/inline-agent-runner.ts
-import { AGENT_DEPS_FACTORY as AGENT_DEPS_FACTORY4, AGENT_STORE as AGENT_STORE4, RunCancelledError, agentFailureCode, encodeStreamEvent as encodeStreamEvent2, publishAgentRunFailed, releaseThreadRun, runAgentLoop, settleAll, settleUnsettledDelegation } from "@dudousxd/nestjs-agent-core";
+import { AGENT_DEPS_FACTORY as AGENT_DEPS_FACTORY4, AGENT_STORE as AGENT_STORE4, RUN_ENDED_BEFORE_TOOL_CALL, RunCancelledError, agentFailureCode, encodeStreamEvent as encodeStreamEvent2, publishAgentRunFailed, releaseThreadRun, runAgentLoop, settleAll, settleUnsettledDelegation, streamFailure as streamFailure2 } from "@dudousxd/nestjs-agent-core";
 import { Inject as Inject20, Injectable as Injectable8, Logger as Logger5, Optional as Optional8 } from "@nestjs/common";
 function _ts_decorate21(decorators, target, key, desc) {
   var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
@@ -4283,7 +4324,15 @@ var InlineAgentRunner = class _InlineAgentRunner {
   * that was running it is gone (a restart), and a thread it still holds is a stale claim.
   */
   async isRunActive(runId) {
-    return this.live.has(runId);
+    if (this.live.has(runId)) {
+      return true;
+    }
+    for (const key of this.pending.keys()) {
+      if (key.startsWith(`${runId}:`)) {
+        return true;
+      }
+    }
+    return false;
   }
   async start(input, options = {}) {
     const runId = options.runId ?? crypto.randomUUID();
@@ -4326,13 +4375,11 @@ var InlineAgentRunner = class _InlineAgentRunner {
         errorCode: code,
         errorMessage: message
       });
+      await Promise.resolve(this.store.failUnsettledToolCalls?.(runId, RUN_ENDED_BEFORE_TOOL_CALL)).catch(() => 0);
       const writer = await deps.sink.open(runId);
       await this.settleQueue(writer, input, runId, "failed", message);
       await releaseThreadRun(this.store, input.threadId, runId);
-      await writer.fail({
-        code,
-        message
-      });
+      await writer.fail(streamFailure2(error));
     }).finally(() => {
       this.cancelled.delete(runId);
       this.live.delete(runId);
@@ -4606,6 +4653,7 @@ var InlineAgentRunner = class _InlineAgentRunner {
           errorMessage: message
         }
       });
+      await Promise.resolve(this.store.failUnsettledToolCalls?.(runId, RUN_ENDED_BEFORE_TOOL_CALL)).catch(() => 0);
       await settleUnsettledDelegation({
         store: this.store,
         delivery: deliverTo,
@@ -5204,6 +5252,7 @@ export {
   MessagesController,
   ModelsController,
   QueueController,
+  RunNotActiveException,
   SKILL_METADATA,
   SYSTEM_PROMPT_CONTRIBUTOR_METADATA,
   SYSTEM_PROMPT_METADATA,