@memberjunction/server 6.1.0-edge.0 → 6.1.0-edge.1

package/dist/services/TaskGraphProviderFactory.d.ts

@@ -0,0 +1,39 @@
+/**
+ * @fileoverview MJServer's implementation of the task-graph `ProviderFactory` seam.
+ *
+ * The durable dispatcher runs outside any HTTP request and executes tasks concurrently, so it must
+ * never share one provider — and therefore one transaction scope and one set of entity instances —
+ * across parallel work. It needs a way to mint providers on demand.
+ *
+ * Rather than have `@memberjunction/task-graph` import MJServer to get at that machinery (which
+ * would invert the dependency: MJServer depends on task-graph, not the reverse), the package
+ * declares a `ProviderFactory` interface and the host supplies an implementation. This is that
+ * implementation.
+ *
+ * It mints exactly what the per-request path already mints — a fresh provider over the **shared
+ * connection pool**, with metadata reuse — which is proven cheap at request scale and is why the
+ * plan chose this mechanism rather than inventing a dispatcher-specific one. The pool remains the
+ * real concurrency governor; the dispatcher's own `MaxConcurrentTasks` sits below it.
+ *
+ * @module @memberjunction/server
+ */
+import { DatabaseProviderBase, IMetadataProvider } from '@memberjunction/core';
+import type { ProviderFactory } from '@memberjunction/task-graph';
+import sql from 'mssql';
+/**
+ * Creates providers for the dispatcher, one per task execution.
+ *
+ * SQL Server path only for now, matching where the dispatcher is hosted. The PostgreSQL branch of
+ * `createPerRequestProviders` is deliberately not duplicated here — when PG hosting is needed, the
+ * right move is to lift the shared branch out of `context.ts` rather than fork it a second time.
+ */
+export declare class TaskGraphProviderFactory implements ProviderFactory {
+    private readonly pool;
+    constructor(pool: sql.ConnectionPool);
+    CreateProvider(): Promise<IMetadataProvider>;
+}
+/** Convenience for hosts that already hold a pool. */
+export declare function CreateTaskGraphProviderFactory(pool: sql.ConnectionPool): ProviderFactory;
+/** Re-exported for hosts that need the concrete type. */
+export type { DatabaseProviderBase };
+//# sourceMappingURL=TaskGraphProviderFactory.d.ts.map

package/dist/services/TaskGraphProviderFactory.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"TaskGraphProviderFactory.d.ts","sourceRoot":"","sources":["../../src/services/TaskGraphProviderFactory.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AACH,OAAO,EAAE,oBAAoB,EAAE,iBAAiB,EAAE,MAAM,sBAAsB,CAAC;AAC/E,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,4BAA4B,CAAC;AAElE,OAAO,GAAG,MAAM,OAAO,CAAC;AAGxB;;;;;;GAMG;AACH,qBAAa,wBAAyB,YAAW,eAAe;IAChD,OAAO,CAAC,QAAQ,CAAC,IAAI;gBAAJ,IAAI,EAAE,GAAG,CAAC,cAAc;IAExC,cAAc,IAAI,OAAO,CAAC,iBAAiB,CAAC;CAS5D;AAED,sDAAsD;AACtD,wBAAgB,8BAA8B,CAAC,IAAI,EAAE,GAAG,CAAC,cAAc,GAAG,eAAe,CAExF;AAED,yDAAyD;AACzD,YAAY,EAAE,oBAAoB,EAAE,CAAC"}

package/dist/services/TaskGraphProviderFactory.js

@@ -0,0 +1,28 @@
+import { SQLServerDataProvider, SQLServerProviderConfigData } from '@memberjunction/sqlserver-dataprovider';
+import { mj_core_schema } from '../config.js';
+/**
+ * Creates providers for the dispatcher, one per task execution.
+ *
+ * SQL Server path only for now, matching where the dispatcher is hosted. The PostgreSQL branch of
+ * `createPerRequestProviders` is deliberately not duplicated here — when PG hosting is needed, the
+ * right move is to lift the shared branch out of `context.ts` rather than fork it a second time.
+ */
+export class TaskGraphProviderFactory {
+    constructor(pool) {
+        this.pool = pool;
+    }
+    async CreateProvider() {
+        // `loadIfNeeded = false` (the trailing false) reuses already-loaded metadata rather than
+        // re-reading it per provider — the difference between "cheap enough to do per task" and
+        // "prohibitively expensive".
+        const config = new SQLServerProviderConfigData(this.pool, mj_core_schema, 0, undefined, undefined, false);
+        const provider = new SQLServerDataProvider();
+        await provider.Config(config);
+        return provider;
+    }
+}
+/** Convenience for hosts that already hold a pool. */
+export function CreateTaskGraphProviderFactory(pool) {
+    return new TaskGraphProviderFactory(pool);
+}
+//# sourceMappingURL=TaskGraphProviderFactory.js.map

package/dist/services/TaskGraphProviderFactory.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"TaskGraphProviderFactory.js","sourceRoot":"","sources":["../../src/services/TaskGraphProviderFactory.ts"],"names":[],"mappings":"AAqBA,OAAO,EAAE,qBAAqB,EAAE,2BAA2B,EAAE,MAAM,wCAAwC,CAAC;AAE5G,OAAO,EAAE,cAAc,EAAE,MAAM,cAAc,CAAC;AAE9C;;;;;;GAMG;AACH,MAAM,OAAO,wBAAwB;IACjC,YAA6B,IAAwB;QAAxB,SAAI,GAAJ,IAAI,CAAoB;IAAG,CAAC;IAElD,KAAK,CAAC,cAAc;QACvB,yFAAyF;QACzF,wFAAwF;QACxF,6BAA6B;QAC7B,MAAM,MAAM,GAAG,IAAI,2BAA2B,CAAC,IAAI,CAAC,IAAI,EAAE,cAAc,EAAE,CAAC,EAAE,SAAS,EAAE,SAAS,EAAE,KAAK,CAAC,CAAC;QAC1G,MAAM,QAAQ,GAAG,IAAI,qBAAqB,EAAE,CAAC;QAC7C,MAAM,QAAQ,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC;QAC9B,OAAO,QAAwC,CAAC;IACpD,CAAC;CACJ;AAED,sDAAsD;AACtD,MAAM,UAAU,8BAA8B,CAAC,IAAwB;IACnE,OAAO,IAAI,wBAAwB,CAAC,IAAI,CAAC,CAAC;AAC9C,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@memberjunction/server",
-  "version": "6.1.0-edge.0",
+  "version": "6.1.0-edge.1",
   "description": "MemberJunction: This project provides API access via GraphQL to the common data store.",
   "main": "./dist/index.js",
   "types": "./src/index.ts",
@@ -19,94 +19,95 @@
     "@as-integrations/express5": "^1.0.0",
     "@graphql-tools/schema": "^10.0.38",
     "@graphql-tools/utils": "^11.0.0",
-    "@memberjunction/actions": "6.1.0-edge.0",
-    "@memberjunction/actions-apollo": "6.1.0-edge.0",
-    "@memberjunction/actions-base": "6.1.0-edge.0",
-    "@memberjunction/actions-bizapps-accounting": "6.1.0-edge.0",
-    "@memberjunction/actions-bizapps-crm": "6.1.0-edge.0",
-    "@memberjunction/actions-bizapps-formbuilders": "6.1.0-edge.0",
-    "@memberjunction/actions-bizapps-lms": "6.1.0-edge.0",
-    "@memberjunction/actions-bizapps-social": "6.1.0-edge.0",
-    "@memberjunction/ai": "6.1.0-edge.0",
-    "@memberjunction/ai-agent-manager": "6.1.0-edge.0",
-    "@memberjunction/ai-agent-manager-actions": "6.1.0-edge.0",
-    "@memberjunction/ai-agents": "6.1.0-edge.0",
-    "@memberjunction/ai-bridge-base": "6.1.0-edge.0",
-    "@memberjunction/ai-bridge-ringcentral": "6.1.0-edge.0",
-    "@memberjunction/ai-bridge-server": "6.1.0-edge.0",
-    "@memberjunction/ai-bridge-teams": "6.1.0-edge.0",
-    "@memberjunction/ai-bridge-twilio": "6.1.0-edge.0",
-    "@memberjunction/ai-bridge-vonage": "6.1.0-edge.0",
-    "@memberjunction/ai-core-plus": "6.1.0-edge.0",
-    "@memberjunction/ai-engine-base": "6.1.0-edge.0",
-    "@memberjunction/ai-mcp-client": "6.1.0-edge.0",
-    "@memberjunction/ai-prompts": "6.1.0-edge.0",
-    "@memberjunction/ai-provider-bundle": "6.1.0-edge.0",
-    "@memberjunction/ai-vector-sync": "6.1.0-edge.0",
-    "@memberjunction/ai-vectordb": "6.1.0-edge.0",
-    "@memberjunction/ai-vectors-pinecone": "6.1.0-edge.0",
-    "@memberjunction/aiengine": "6.1.0-edge.0",
-    "@memberjunction/api-keys": "6.1.0-edge.0",
-    "@memberjunction/auth-providers": "6.1.0-edge.0",
-    "@memberjunction/clustering-engine": "6.1.0-edge.0",
-    "@memberjunction/codegen-lib": "6.1.0-edge.0",
-    "@memberjunction/communication-engine": "6.1.0-edge.0",
-    "@memberjunction/communication-ms-graph": "6.1.0-edge.0",
-    "@memberjunction/communication-sendgrid": "6.1.0-edge.0",
-    "@memberjunction/communication-types": "6.1.0-edge.0",
-    "@memberjunction/component-registry-client-sdk": "6.1.0-edge.0",
-    "@memberjunction/computer-use": "6.1.0-edge.0",
-    "@memberjunction/computer-use-engine": "6.1.0-edge.0",
-    "@memberjunction/config": "6.1.0-edge.0",
-    "@memberjunction/core": "6.1.0-edge.0",
-    "@memberjunction/core-actions": "6.1.0-edge.0",
-    "@memberjunction/core-entities": "6.1.0-edge.0",
-    "@memberjunction/core-entities-server": "6.1.0-edge.0",
-    "@memberjunction/credentials": "6.1.0-edge.0",
-    "@memberjunction/data-context": "6.1.0-edge.0",
-    "@memberjunction/data-context-server": "6.1.0-edge.0",
-    "@memberjunction/doc-utils": "6.1.0-edge.0",
-    "@memberjunction/encryption": "6.1.0-edge.0",
-    "@memberjunction/entity-communications-base": "6.1.0-edge.0",
-    "@memberjunction/entity-communications-server": "6.1.0-edge.0",
-    "@memberjunction/esignature": "6.1.0-edge.0",
-    "@memberjunction/external-change-detection": "6.1.0-edge.0",
-    "@memberjunction/generic-database-provider": "6.1.0-edge.0",
-    "@memberjunction/global": "6.1.0-edge.0",
-    "@memberjunction/graphql-dataprovider": "6.1.0-edge.0",
-    "@memberjunction/integration-engine": "6.1.0-edge.0",
-    "@memberjunction/integration-engine-base": "6.1.0-edge.0",
-    "@memberjunction/integration-progress-artifacts": "6.1.0-edge.0",
-    "@memberjunction/integration-schema-builder": "6.1.0-edge.0",
-    "@memberjunction/interactive-component-types": "6.1.0-edge.0",
-    "@memberjunction/lists": "6.1.0-edge.0",
-    "@memberjunction/lists-base": "6.1.0-edge.0",
-    "@memberjunction/livekit-room-server": "6.1.0-edge.0",
-    "@memberjunction/notifications": "6.1.0-edge.0",
-    "@memberjunction/postgresql-dataprovider": "6.1.0-edge.0",
-    "@memberjunction/queue": "6.1.0-edge.0",
-    "@memberjunction/record-comparison": "6.1.0-edge.0",
-    "@memberjunction/redis-provider": "6.1.0-edge.0",
-    "@memberjunction/remote-browser-base": "6.1.0-edge.0",
-    "@memberjunction/remote-browser-cdp": "6.1.0-edge.0",
-    "@memberjunction/remote-browser-selfhost": "6.1.0-edge.0",
-    "@memberjunction/remote-browser-server": "6.1.0-edge.0",
-    "@memberjunction/scheduling-actions": "6.1.0-edge.0",
-    "@memberjunction/scheduling-base-types": "6.1.0-edge.0",
-    "@memberjunction/scheduling-engine": "6.1.0-edge.0",
-    "@memberjunction/scheduling-engine-base": "6.1.0-edge.0",
-    "@memberjunction/schema-engine": "6.1.0-edge.0",
-    "@memberjunction/search-engine": "6.1.0-edge.0",
-    "@memberjunction/server-extensions-core": "6.1.0-edge.0",
-    "@memberjunction/sql-dialect": "6.1.0-edge.0",
-    "@memberjunction/sqlserver-dataprovider": "6.1.0-edge.0",
-    "@memberjunction/storage": "6.1.0-edge.0",
-    "@memberjunction/tag-engine": "6.1.0-edge.0",
-    "@memberjunction/tag-engine-base": "6.1.0-edge.0",
-    "@memberjunction/templates": "6.1.0-edge.0",
-    "@memberjunction/testing-engine": "6.1.0-edge.0",
-    "@memberjunction/testing-engine-base": "6.1.0-edge.0",
-    "@memberjunction/version-history": "6.1.0-edge.0",
+    "@memberjunction/actions": "6.1.0-edge.1",
+    "@memberjunction/actions-apollo": "6.1.0-edge.1",
+    "@memberjunction/actions-base": "6.1.0-edge.1",
+    "@memberjunction/actions-bizapps-accounting": "6.1.0-edge.1",
+    "@memberjunction/actions-bizapps-crm": "6.1.0-edge.1",
+    "@memberjunction/actions-bizapps-formbuilders": "6.1.0-edge.1",
+    "@memberjunction/actions-bizapps-lms": "6.1.0-edge.1",
+    "@memberjunction/actions-bizapps-social": "6.1.0-edge.1",
+    "@memberjunction/ai": "6.1.0-edge.1",
+    "@memberjunction/ai-agent-manager": "6.1.0-edge.1",
+    "@memberjunction/ai-agent-manager-actions": "6.1.0-edge.1",
+    "@memberjunction/ai-agents": "6.1.0-edge.1",
+    "@memberjunction/ai-bridge-base": "6.1.0-edge.1",
+    "@memberjunction/ai-bridge-ringcentral": "6.1.0-edge.1",
+    "@memberjunction/ai-bridge-server": "6.1.0-edge.1",
+    "@memberjunction/ai-bridge-teams": "6.1.0-edge.1",
+    "@memberjunction/ai-bridge-twilio": "6.1.0-edge.1",
+    "@memberjunction/ai-bridge-vonage": "6.1.0-edge.1",
+    "@memberjunction/ai-core-plus": "6.1.0-edge.1",
+    "@memberjunction/ai-engine-base": "6.1.0-edge.1",
+    "@memberjunction/ai-mcp-client": "6.1.0-edge.1",
+    "@memberjunction/ai-prompts": "6.1.0-edge.1",
+    "@memberjunction/ai-provider-bundle": "6.1.0-edge.1",
+    "@memberjunction/ai-vector-sync": "6.1.0-edge.1",
+    "@memberjunction/ai-vectordb": "6.1.0-edge.1",
+    "@memberjunction/ai-vectors-pinecone": "6.1.0-edge.1",
+    "@memberjunction/aiengine": "6.1.0-edge.1",
+    "@memberjunction/api-keys": "6.1.0-edge.1",
+    "@memberjunction/auth-providers": "6.1.0-edge.1",
+    "@memberjunction/clustering-engine": "6.1.0-edge.1",
+    "@memberjunction/codegen-lib": "6.1.0-edge.1",
+    "@memberjunction/communication-engine": "6.1.0-edge.1",
+    "@memberjunction/communication-ms-graph": "6.1.0-edge.1",
+    "@memberjunction/communication-sendgrid": "6.1.0-edge.1",
+    "@memberjunction/communication-types": "6.1.0-edge.1",
+    "@memberjunction/component-registry-client-sdk": "6.1.0-edge.1",
+    "@memberjunction/computer-use": "6.1.0-edge.1",
+    "@memberjunction/computer-use-engine": "6.1.0-edge.1",
+    "@memberjunction/config": "6.1.0-edge.1",
+    "@memberjunction/core": "6.1.0-edge.1",
+    "@memberjunction/core-actions": "6.1.0-edge.1",
+    "@memberjunction/core-entities": "6.1.0-edge.1",
+    "@memberjunction/core-entities-server": "6.1.0-edge.1",
+    "@memberjunction/credentials": "6.1.0-edge.1",
+    "@memberjunction/data-context": "6.1.0-edge.1",
+    "@memberjunction/data-context-server": "6.1.0-edge.1",
+    "@memberjunction/doc-utils": "6.1.0-edge.1",
+    "@memberjunction/encryption": "6.1.0-edge.1",
+    "@memberjunction/entity-communications-base": "6.1.0-edge.1",
+    "@memberjunction/entity-communications-server": "6.1.0-edge.1",
+    "@memberjunction/esignature": "6.1.0-edge.1",
+    "@memberjunction/external-change-detection": "6.1.0-edge.1",
+    "@memberjunction/generic-database-provider": "6.1.0-edge.1",
+    "@memberjunction/global": "6.1.0-edge.1",
+    "@memberjunction/graphql-dataprovider": "6.1.0-edge.1",
+    "@memberjunction/integration-engine": "6.1.0-edge.1",
+    "@memberjunction/integration-engine-base": "6.1.0-edge.1",
+    "@memberjunction/integration-progress-artifacts": "6.1.0-edge.1",
+    "@memberjunction/integration-schema-builder": "6.1.0-edge.1",
+    "@memberjunction/interactive-component-types": "6.1.0-edge.1",
+    "@memberjunction/lists": "6.1.0-edge.1",
+    "@memberjunction/lists-base": "6.1.0-edge.1",
+    "@memberjunction/livekit-room-server": "6.1.0-edge.1",
+    "@memberjunction/notifications": "6.1.0-edge.1",
+    "@memberjunction/postgresql-dataprovider": "6.1.0-edge.1",
+    "@memberjunction/queue": "6.1.0-edge.1",
+    "@memberjunction/record-comparison": "6.1.0-edge.1",
+    "@memberjunction/redis-provider": "6.1.0-edge.1",
+    "@memberjunction/remote-browser-base": "6.1.0-edge.1",
+    "@memberjunction/remote-browser-cdp": "6.1.0-edge.1",
+    "@memberjunction/remote-browser-selfhost": "6.1.0-edge.1",
+    "@memberjunction/remote-browser-server": "6.1.0-edge.1",
+    "@memberjunction/scheduling-actions": "6.1.0-edge.1",
+    "@memberjunction/scheduling-base-types": "6.1.0-edge.1",
+    "@memberjunction/scheduling-engine": "6.1.0-edge.1",
+    "@memberjunction/scheduling-engine-base": "6.1.0-edge.1",
+    "@memberjunction/schema-engine": "6.1.0-edge.1",
+    "@memberjunction/search-engine": "6.1.0-edge.1",
+    "@memberjunction/server-extensions-core": "6.1.0-edge.1",
+    "@memberjunction/sql-dialect": "6.1.0-edge.1",
+    "@memberjunction/sqlserver-dataprovider": "6.1.0-edge.1",
+    "@memberjunction/storage": "6.1.0-edge.1",
+    "@memberjunction/tag-engine": "6.1.0-edge.1",
+    "@memberjunction/tag-engine-base": "6.1.0-edge.1",
+    "@memberjunction/task-graph": "6.1.0-edge.1",
+    "@memberjunction/templates": "6.1.0-edge.1",
+    "@memberjunction/testing-engine": "6.1.0-edge.1",
+    "@memberjunction/testing-engine-base": "6.1.0-edge.1",
+    "@memberjunction/version-history": "6.1.0-edge.1",
     "@octokit/auth-app": "^7.1.5",
     "@octokit/rest": "^21.1.1",
     "@types/compression": "^1.8.1",
@@ -148,6 +149,7 @@
   "devDependencies": {
     "@types/express": "^5.0.6",
     "pkgroll": "2.23.0",
+    "rimraf": "6.1.2",
     "typescript": "^5.9.3",
     "vitest": "^3.1.1"
   },
@@ -156,6 +158,7 @@
     "url": "https://github.com/MemberJunction/MJ"
   },
   "scripts": {
+    "prebuild": "rimraf dist",
     "build": "tsc && tsc-alias -f",
     "bundle": "tsc -noEmit && pkgroll --sourcemap --minify",
     "watch": "tsc -w",

package/src/__tests__/TaskGraphContinuationDeliverer.test.ts

@@ -0,0 +1,249 @@
+/**
+ * Tests for `TaskGraphContinuationDeliverer`.
+ *
+ * The gap this closes: the dispatcher was constructed with no deliverer at all, so a graph that
+ * finished logged its outcome, marked itself delivered, and said nothing to the conversation that
+ * asked for it — durable execution nobody hears about.
+ *
+ * The contract that matters most is that it **never throws**. The dispatcher calls this inside the
+ * compare-and-swap that marks a completion delivered; an escaping error would either abort that
+ * guard or leave the graph looking undelivered and re-notifying on every later sweep.
+ */
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+
+vi.mock('@memberjunction/core', () => ({
+    LogError: vi.fn(),
+    LogStatus: vi.fn(),
+}));
+
+vi.mock('@memberjunction/core-entities', () => ({
+    MJConversationDetailEntity: class {},
+    MJAIAgentRunEntity: class {},
+}));
+
+vi.mock('@memberjunction/ai-core-plus', () => ({ MJAIAgentEntityExtended: class {} }));
+vi.mock('@memberjunction/ai', () => ({ ChatMessageRole: { user: 'user' } }));
+
+const runAgent = vi.fn().mockResolvedValue({ success: true });
+vi.mock('@memberjunction/ai-agents', () => ({
+    AgentRunner: class { RunAgent = runAgent; },
+}));
+
+import { TaskGraphContinuationDeliverer } from '../services/TaskGraphContinuationDeliverer';
+import type { TaskContinuationParams } from '@memberjunction/task-graph';
+import type { UserInfo } from '@memberjunction/core';
+
+/** A conversation-detail row that can be loaded (the source) and saved (the reply). */
+function detailRow(over: Partial<{ loads: boolean; saves: boolean; conversationID: string }> = {}) {
+    const { loads = true, saves = true, conversationID = 'conv-1' } = over;
+    return {
+        ConversationID: conversationID,
+        Role: '', Status: '', HiddenToUser: true, Message: '',
+        NewRecord: vi.fn(),
+        Load: vi.fn().mockResolvedValue(loads),
+        Save: vi.fn().mockResolvedValue(saves),
+        LatestResult: { CompleteMessage: 'FK violation' },
+    };
+}
+
+function harness(over: Parameters<typeof detailRow>[0] = {}) {
+    const rows: ReturnType<typeof detailRow>[] = [];
+    const provider = {
+        GetEntityObject: vi.fn().mockImplementation(async () => {
+            const row = detailRow(over);
+            rows.push(row);
+            return row;
+        }),
+    };
+    const providerFactory = { CreateProvider: vi.fn().mockResolvedValue(provider) };
+    const deliverer = new TaskGraphContinuationDeliverer(
+        providerFactory as never,
+        { ID: 'user-1' } as UserInfo,
+    );
+    // First row is the source detail (Load), second is the reply (Save).
+    return { deliverer, providerFactory, provider, rows, reply: () => rows[1] };
+}
+
+const params = (over: Partial<TaskContinuationParams> = {}): TaskContinuationParams => ({
+    ParentTaskID: 'parent-1',
+    WorkflowName: 'Weekly digest',
+    ConversationDetailID: 'detail-1',
+    SubmittedByAgentRunID: 'run-1',
+    ReinvokeDepth: 0,
+    Tasks: [
+        { TaskID: 't1', Name: 'Gather', Status: 'Complete', Summary: '412 rows' },
+        { TaskID: 't2', Name: 'Summarize', Status: 'Complete' },
+    ],
+    Summary: '2 of 2 steps completed.',
+    ...over,
+});
+
+beforeEach(() => vi.clearAllMocks());
+
+describe('posting the outcome', () => {
+    it('writes an AI-role message into the graph\'s conversation', async () => {
+        const h = harness();
+        await h.deliverer.PostMessage(params());
+
+        const reply = h.reply();
+        expect(reply.Save).toHaveBeenCalled();
+        expect(reply.ConversationID).toBe('conv-1');
+        expect(reply.Role).toBe('AI');
+        expect(reply.Status).toBe('Complete');
+        expect(reply.HiddenToUser).toBe(false);
+    });
+
+    it('names the workflow and carries the roll-up', async () => {
+        const h = harness();
+        await h.deliverer.PostMessage(params());
+        expect(h.reply().Message).toContain('Weekly digest');
+        expect(h.reply().Message).toContain('2 of 2 steps completed.');
+    });
+
+    it('lists each step, not just an aggregate status', async () => {
+        // A graph where nine of ten steps succeeded is a materially different message from one that
+        // failed outright, and the roll-up alone cannot say which step went wrong.
+        const h = harness();
+        await h.deliverer.PostMessage(params({
+            Tasks: [
+                { TaskID: 't1', Name: 'Gather', Status: 'Complete', Summary: '412 rows' },
+                { TaskID: 't2', Name: 'Summarize', Status: 'Failed', ErrorMessage: 'model timed out' },
+            ],
+        }));
+        const message = h.reply().Message;
+        expect(message).toContain('Gather');
+        expect(message).toContain('412 rows');
+        expect(message).toContain('Summarize');
+        expect(message).toContain('model timed out');
+    });
+
+    it('truncates a long graph rather than burying the conversation', async () => {
+        const h = harness();
+        const many = Array.from({ length: 50 }, (_, i) => ({ TaskID: `t${i}`, Name: `Step ${i}`, Status: 'Complete' }));
+        await h.deliverer.PostMessage(params({ Tasks: many }));
+
+        const message = h.reply().Message;
+        expect(message).toContain('Step 0');
+        expect(message).not.toContain('Step 49');
+        // The count is stated rather than silently dropped — the task rows remain the full record.
+        expect(message).toContain('30 more');
+    });
+
+    it('mints a fresh provider per delivery', async () => {
+        // Deliveries run outside any request and concurrently with task execution; a shared provider
+        // would share one transaction scope across unrelated work.
+        const h = harness();
+        await h.deliverer.PostMessage(params());
+        expect(h.providerFactory.CreateProvider).toHaveBeenCalledTimes(1);
+    });
+});
+
+describe('it never throws — the dispatcher marks delivery inside a CAS guard', () => {
+    it('does nothing for a headless graph, without treating it as an error', async () => {
+        // A graph submitted by a schedule, an entity-change trigger or an API call has no
+        // conversation to answer. Most workflows are in this shape.
+        const h = harness();
+        await expect(h.deliverer.PostMessage(params({ ConversationDetailID: null }))).resolves.toBeUndefined();
+        expect(h.providerFactory.CreateProvider).not.toHaveBeenCalled();
+    });
+
+    it('survives a conversation detail that will not load', async () => {
+        const h = harness({ loads: false });
+        await expect(h.deliverer.PostMessage(params())).resolves.toBeUndefined();
+        expect(h.rows.length).toBe(1); // never got as far as building a reply
+    });
+
+    it('survives a failed save', async () => {
+        const h = harness({ saves: false });
+        await expect(h.deliverer.PostMessage(params())).resolves.toBeUndefined();
+    });
+
+    it('survives a provider that throws outright', async () => {
+        const deliverer = new TaskGraphContinuationDeliverer(
+            { CreateProvider: vi.fn().mockRejectedValue(new Error('pool exhausted')) } as never,
+            { ID: 'user-1' } as UserInfo,
+        );
+        await expect(deliverer.PostMessage(params())).resolves.toBeUndefined();
+    });
+});
+
+describe('Reinvoke — restarting the submitting agent', () => {
+    /** A provider that hands out loadable agent-run and agent rows, then the reply detail. */
+    function reinvokeHarness(over: { runLoads?: boolean; agentLoads?: boolean } = {}) {
+        const { runLoads = true, agentLoads = true } = over;
+        const rows: Array<Record<string, unknown>> = [];
+        let call = 0;
+        const provider = {
+            GetEntityObject: vi.fn().mockImplementation(async () => {
+                call++;
+                if (call === 1) {
+                    const run = { AgentID: 'agent-1', Load: vi.fn().mockResolvedValue(runLoads) };
+                    rows.push(run);
+                    return run;
+                }
+                if (call === 2) {
+                    const agent = { ID: 'agent-1', Load: vi.fn().mockResolvedValue(agentLoads) };
+                    rows.push(agent);
+                    return agent;
+                }
+                const detail = detailRow();
+                rows.push(detail as unknown as Record<string, unknown>);
+                return detail;
+            }),
+        };
+        const deliverer = new TaskGraphContinuationDeliverer(
+            { CreateProvider: vi.fn().mockResolvedValue(provider) } as never,
+            { ID: 'user-1' } as UserInfo,
+        );
+        return { deliverer, provider, rows };
+    }
+
+    it('starts the submitting agent a fresh turn carrying the outcome', async () => {
+        const h = reinvokeHarness();
+        await h.deliverer.Reinvoke(params());
+
+        expect(runAgent).toHaveBeenCalledTimes(1);
+        const call = runAgent.mock.calls[0][0];
+        expect(call.agent.ID).toBe('agent-1');
+        expect(call.conversationMessages[0].content).toContain('Weekly digest');
+    });
+
+    it('stamps depth + 1 — the value that makes MAX_REINVOKE_DEPTH real', async () => {
+        // Without this the next graph the restarted run submits begins the chain at zero again, and
+        // the cap can never fire. It is the whole reason ContinuationDepth exists as a column.
+        const h = reinvokeHarness();
+        await h.deliverer.Reinvoke(params({ ReinvokeDepth: 3 }));
+        expect(runAgent.mock.calls[0][0].continuationDepth).toBe(4);
+    });
+
+    it('starts a chain at 1, not 0 — depth 0 is "not a continuation"', async () => {
+        const h = reinvokeHarness();
+        await h.deliverer.Reinvoke(params({ ReinvokeDepth: 0 }));
+        expect(runAgent.mock.calls[0][0].continuationDepth).toBe(1);
+    });
+
+    it('falls back to posting when the graph records no submitting run', async () => {
+        // A schedule- or trigger-started graph has no turn to continue.
+        const h = reinvokeHarness();
+        await h.deliverer.Reinvoke(params({ SubmittedByAgentRunID: null }));
+        expect(runAgent).not.toHaveBeenCalled();
+    });
+
+    it('falls back to posting when the submitting run cannot be loaded', async () => {
+        const h = reinvokeHarness({ runLoads: false });
+        await h.deliverer.Reinvoke(params());
+        expect(runAgent).not.toHaveBeenCalled();
+    });
+
+    it('falls back to posting when the agent cannot be loaded', async () => {
+        const h = reinvokeHarness({ agentLoads: false });
+        await h.deliverer.Reinvoke(params());
+        expect(runAgent).not.toHaveBeenCalled();
+    });
+
+    it('never throws — the dispatcher marks delivery inside a CAS guard', async () => {
+        runAgent.mockRejectedValueOnce(new Error('agent exploded'));
+        const h = reinvokeHarness();
+        await expect(h.deliverer.Reinvoke(params())).resolves.toBeUndefined();
+    });
+});