@memberjunction/server 5.51.0 → 6.1.0-edge.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@memberjunction/server",
-  "version": "5.51.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",
@@ -14,107 +14,100 @@
   "exports": {
     ".": "./dist/index.js"
   },
-  "scripts": {
-    "build": "tsc && tsc-alias -f",
-    "bundle": "tsc -noEmit && pkgroll --sourcemap --minify",
-    "watch": "tsc -w",
-    "test": "vitest run",
-    "test:watch": "vitest",
-    "test:coverage": "vitest run --coverage"
-  },
   "dependencies": {
     "@apollo/server": "^5.0.0",
     "@as-integrations/express5": "^1.0.0",
-    "@graphql-tools/schema": "latest",
+    "@graphql-tools/schema": "^10.0.38",
     "@graphql-tools/utils": "^11.0.0",
-    "@memberjunction/actions": "5.51.0",
-    "@memberjunction/actions-apollo": "5.51.0",
-    "@memberjunction/actions-base": "5.51.0",
-    "@memberjunction/actions-bizapps-accounting": "5.51.0",
-    "@memberjunction/actions-bizapps-crm": "5.51.0",
-    "@memberjunction/actions-bizapps-formbuilders": "5.51.0",
-    "@memberjunction/actions-bizapps-lms": "5.51.0",
-    "@memberjunction/actions-bizapps-social": "5.51.0",
-    "@memberjunction/ai": "5.51.0",
-    "@memberjunction/ai-agent-manager": "5.51.0",
-    "@memberjunction/ai-agent-manager-actions": "5.51.0",
-    "@memberjunction/ai-agents": "5.51.0",
-    "@memberjunction/ai-bridge-base": "5.51.0",
-    "@memberjunction/ai-bridge-ringcentral": "5.51.0",
-    "@memberjunction/ai-bridge-server": "5.51.0",
-    "@memberjunction/ai-bridge-teams": "5.51.0",
-    "@memberjunction/ai-bridge-twilio": "5.51.0",
-    "@memberjunction/ai-bridge-vonage": "5.51.0",
-    "@memberjunction/ai-core-plus": "5.51.0",
-    "@memberjunction/ai-engine-base": "5.51.0",
-    "@memberjunction/ai-mcp-client": "5.51.0",
-    "@memberjunction/ai-prompts": "5.51.0",
-    "@memberjunction/ai-provider-bundle": "5.51.0",
-    "@memberjunction/ai-vector-sync": "5.51.0",
-    "@memberjunction/ai-vectordb": "5.51.0",
-    "@memberjunction/ai-vectors-pinecone": "5.51.0",
-    "@memberjunction/aiengine": "5.51.0",
-    "@memberjunction/api-keys": "5.51.0",
-    "@memberjunction/auth-providers": "5.51.0",
-    "@memberjunction/clustering-engine": "5.51.0",
-    "@memberjunction/codegen-lib": "5.51.0",
-    "@memberjunction/communication-engine": "5.51.0",
-    "@memberjunction/communication-ms-graph": "5.51.0",
-    "@memberjunction/communication-sendgrid": "5.51.0",
-    "@memberjunction/communication-types": "5.51.0",
-    "@memberjunction/component-registry-client-sdk": "5.51.0",
-    "@memberjunction/computer-use": "5.51.0",
-    "@memberjunction/computer-use-engine": "5.51.0",
-    "@memberjunction/config": "5.51.0",
-    "@memberjunction/core": "5.51.0",
-    "@memberjunction/core-actions": "5.51.0",
-    "@memberjunction/core-entities": "5.51.0",
-    "@memberjunction/core-entities-server": "5.51.0",
-    "@memberjunction/credentials": "5.51.0",
-    "@memberjunction/data-context": "5.51.0",
-    "@memberjunction/data-context-server": "5.51.0",
-    "@memberjunction/doc-utils": "5.51.0",
-    "@memberjunction/encryption": "5.51.0",
-    "@memberjunction/entity-communications-base": "5.51.0",
-    "@memberjunction/entity-communications-server": "5.51.0",
-    "@memberjunction/esignature": "5.51.0",
-    "@memberjunction/external-change-detection": "5.51.0",
-    "@memberjunction/generic-database-provider": "5.51.0",
-    "@memberjunction/global": "5.51.0",
-    "@memberjunction/graphql-dataprovider": "5.51.0",
-    "@memberjunction/integration-engine": "5.51.0",
-    "@memberjunction/integration-engine-base": "5.51.0",
-    "@memberjunction/integration-progress-artifacts": "5.51.0",
-    "@memberjunction/integration-schema-builder": "5.51.0",
-    "@memberjunction/interactive-component-types": "5.51.0",
-    "@memberjunction/lists": "5.51.0",
-    "@memberjunction/lists-base": "5.51.0",
-    "@memberjunction/livekit-room-server": "5.51.0",
-    "@memberjunction/notifications": "5.51.0",
-    "@memberjunction/postgresql-dataprovider": "5.51.0",
-    "@memberjunction/queue": "5.51.0",
-    "@memberjunction/record-comparison": "5.51.0",
-    "@memberjunction/redis-provider": "5.51.0",
-    "@memberjunction/remote-browser-base": "5.51.0",
-    "@memberjunction/remote-browser-cdp": "5.51.0",
-    "@memberjunction/remote-browser-selfhost": "5.51.0",
-    "@memberjunction/remote-browser-server": "5.51.0",
-    "@memberjunction/scheduling-actions": "5.51.0",
-    "@memberjunction/scheduling-base-types": "5.51.0",
-    "@memberjunction/scheduling-engine": "5.51.0",
-    "@memberjunction/scheduling-engine-base": "5.51.0",
-    "@memberjunction/schema-engine": "5.51.0",
-    "@memberjunction/search-engine": "5.51.0",
-    "@memberjunction/server-extensions-core": "5.51.0",
-    "@memberjunction/sql-dialect": "5.51.0",
-    "@memberjunction/sqlserver-dataprovider": "5.51.0",
-    "@memberjunction/storage": "5.51.0",
-    "@memberjunction/tag-engine": "5.51.0",
-    "@memberjunction/tag-engine-base": "5.51.0",
-    "@memberjunction/templates": "5.51.0",
-    "@memberjunction/testing-engine": "5.51.0",
-    "@memberjunction/testing-engine-base": "5.51.0",
-    "@memberjunction/version-history": "5.51.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",
@@ -154,13 +147,23 @@
     "zod": "^3.25.0"
   },
   "devDependencies": {
-    "@types/express": "^4.17.25",
+    "@types/express": "^5.0.6",
     "pkgroll": "2.23.0",
+    "rimraf": "6.1.2",
     "typescript": "^5.9.3",
     "vitest": "^3.1.1"
   },
   "repository": {
     "type": "git",
     "url": "https://github.com/MemberJunction/MJ"
+  },
+  "scripts": {
+    "prebuild": "rimraf dist",
+    "build": "tsc && tsc-alias -f",
+    "bundle": "tsc -noEmit && pkgroll --sourcemap --minify",
+    "watch": "tsc -w",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "test:coverage": "vitest run --coverage"
   }
-}
+}

package/src/__tests__/RealtimeBridgeResolver.test.ts

@@ -31,6 +31,12 @@ vi.mock('@memberjunction/ai-agents', () => ({
   FinalizeBridgeCoAgentRuns: vi.fn(),
   GetRealtimeModelVoices: vi.fn(),
   CreateBridgeRoomTranscriptSink: vi.fn(),
+  // SessionManager's constructor defaults to `new RealtimeClientSessionService()` when this resolver
+  // doesn't inject one (it never finalizes client-direct co-agent runs itself) — the mock must still
+  // export the class so that default construction doesn't throw.
+  RealtimeClientSessionService: class {
+    FinalizeCoAgentRun = vi.fn(async () => undefined);
+  },
 }));
 
 // Mock the meeting-recording registration so the thin resolver is tested in isolation (no MJStorage /

package/src/__tests__/SessionManager.test.ts

@@ -321,6 +321,50 @@ describe('SessionManager.CloseSession', () => {
         expect(finalizeCoAgentRunMock).not.toHaveBeenCalled();
     });
 
+    it('finalizes through an INJECTED RealtimeClientSessionService instance, not the default one', async () => {
+        // Regression test for the wrong-instance leak: a caller (e.g. RealtimeClientSessionResolver)
+        // that created the session's co-agent AIPromptRun through its OWN RealtimeClientSessionService
+        // must have CloseSession finalize through that SAME instance — otherwise the service's internal
+        // per-run write-chain cleanup (finalizePromptRun -> promptRunWriteChains.Delete) silently no-ops
+        // on a throwaway instance while the real accumulated state on the caller's instance never clears.
+        const injectedFinalizeMock = vi.fn(async () => undefined);
+        const injectedService = { FinalizeCoAgentRun: injectedFinalizeMock } as unknown as ConstructorParameters<
+            typeof SessionManager
+        >[0];
+        const session = makeSessionEntity({
+            ID: 'session-injected',
+            Status: 'Active',
+            Config_: JSON.stringify({ targetAgentID: 't1', coAgentRunID: 'co-run-injected', promptRunID: 'prompt-run-injected' }),
+        });
+        const { provider } = makeProvider(() => session);
+        const mgr = new SessionManager(injectedService);
+        const user = makeUser();
+
+        const ok = await mgr.CloseSession('session-injected', user, provider);
+
+        expect(ok).toBe(true);
+        // The finalize call landed on the INJECTED instance, never the module-default one.
+        expect(injectedFinalizeMock).toHaveBeenCalledTimes(1);
+        expect(injectedFinalizeMock).toHaveBeenCalledWith('co-run-injected', 'prompt-run-injected', user, provider, true, null);
+        expect(finalizeCoAgentRunMock).not.toHaveBeenCalled();
+    });
+
+    it('defaults to its own RealtimeClientSessionService instance when none is injected (back-compat)', async () => {
+        const session = makeSessionEntity({
+            ID: 'session-default-svc',
+            Status: 'Active',
+            Config_: JSON.stringify({ targetAgentID: 't1', coAgentRunID: 'co-run-default', promptRunID: 'prompt-run-default' }),
+        });
+        const { provider } = makeProvider(() => session);
+        const mgr = new SessionManager(); // no injection — e.g. SessionJanitor, telephony services
+        const user = makeUser();
+
+        const ok = await mgr.CloseSession('session-default-svc', user, provider);
+
+        expect(ok).toBe(true);
+        expect(finalizeCoAgentRunMock).toHaveBeenCalledWith('co-run-default', 'prompt-run-default', user, provider, true, null);
+    });
+
     it('stamps CloseReason = Explicit by default (untouched call sites get the right reason)', async () => {
         const session = makeSessionEntity({ ID: 'session-default', Status: 'Active' });
         const { provider } = makeProvider(() => session);

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();
+    });
+});