@myagentroam/node 0.10.61 → 0.10.62

package/dist/service/scheduled-session-coordinator.js

@@ -3,13 +3,84 @@ export class ScheduledSessionCoordinator {
     options;
     #now;
     #timer;
-    #running = false;
+    #running;
     #closed = true;
     #recovered = [];
     constructor(options) {
         this.options = options;
         this.#now = options.now ?? Date.now;
     }
+    executionTerminated(run, status) {
+        if (run.source !== 'SCHEDULED')
+            return;
+        const store = this.options.store();
+        store.finish(run.clientMessageId, status === 'SUCCEEDED' ? 'SUCCEEDED' : 'FAILED', status === 'SUCCEEDED' ? undefined : `SCHEDULE_RUN_${status}`, run.id);
+        const task = store.getById(run.clientMessageId.split(':', 1)[0]);
+        if (task !== undefined) {
+            if (task.notifyOnTerminal)
+                this.publishTaskNotification(task, run.clientMessageId, {
+                    kind: 'SCHEDULE_TERMINAL',
+                    outcome: status,
+                    runId: run.id
+                });
+            else if (status !== 'SUCCEEDED')
+                this.notify(task, run.clientMessageId, `SCHEDULE_RUN_${status}`, 'FAILED');
+        }
+    }
+    notify = (task, occurrenceId, code, status) => this.publishTaskNotification(task, occurrenceId, {
+        kind: status === 'FAILED' ? 'SCHEDULE_FAILED' : 'SCHEDULE_SKIPPED',
+        code
+    });
+    publishTaskNotification = (task, occurrenceId, event) => {
+        const occurredAt = Date.now();
+        const recipient = this.options.store().getById(task.id)?.ownerUserId ?? task.ownerUserId;
+        this.options.emitNotification({
+            event: {
+                runId: event.kind === 'SCHEDULE_TERMINAL' ? event.runId : occurrenceId,
+                sequence: 1,
+                eventType: 'notification',
+                payload: {
+                    targetType: 'WORKSPACE_SESSION',
+                    userId: recipient,
+                    workspaceId: task.workspaceId,
+                    sessionId: task.sessionId,
+                    sessionTitle: this.options.sessionTitle(task.sessionId),
+                    runId: event.kind === 'SCHEDULE_TERMINAL' ? event.runId : occurrenceId,
+                    notificationId: event.kind === 'SCHEDULE_TERMINAL'
+                        ? `schedule-terminal:${task.id}:${occurrenceId}`
+                        : `schedule:${task.id}:${occurrenceId}`,
+                    ...event,
+                    occurredAt
+                },
+                createdAt: occurredAt
+            }
+        });
+    };
+    proposalCreated = (sessionId, proposalId) => {
+        const task = this.options.store().get(sessionId);
+        if (task === undefined)
+            return;
+        const occurredAt = Date.now();
+        this.options.emitNotification({
+            event: {
+                runId: proposalId,
+                sequence: 1,
+                eventType: 'notification',
+                payload: {
+                    targetType: 'WORKSPACE_SESSION',
+                    userId: task.ownerUserId,
+                    workspaceId: task.workspaceId,
+                    sessionId,
+                    sessionTitle: this.options.sessionTitle(sessionId),
+                    runId: proposalId,
+                    notificationId: `schedule-proposal:${task.id}:${proposalId}`,
+                    kind: 'SCHEDULE_PROPOSAL',
+                    occurredAt
+                },
+                createdAt: occurredAt
+            }
+        });
+    };
     /** Called only at Node startup, before admitting any new scheduled occurrence. */
     recoverInterrupted() {
         for (const recovered of this.options.store().recoverInterrupted())
@@ -18,7 +89,7 @@ export class ScheduledSessionCoordinator {
     connected() {
         // Startup has no control connection yet. Send these best-effort alerts once it is ready.
         for (const { task, occurrenceId } of this.#recovered.splice(0)) {
-            this.options.notify(task, occurrenceId, SCHEDULE_RESTART_OUTCOME_UNKNOWN, 'FAILED');
+            this.notify(task, occurrenceId, SCHEDULE_RESTART_OUTCOME_UNKNOWN, 'FAILED');
         }
     }
     start() {
@@ -36,6 +107,7 @@ export class ScheduledSessionCoordinator {
         if (this.#timer !== undefined)
             clearTimeout(this.#timer);
         this.#timer = undefined;
+        return this.#running;
     }
     #arm() {
         if (this.#timer !== undefined)
@@ -50,59 +122,58 @@ export class ScheduledSessionCoordinator {
         this.#timer = setTimeout(() => void this.#tick(false), delay);
         this.#timer.unref();
     }
-    async #tick(initial) {
+    #tick(initial) {
         if (this.#closed || this.#running)
             return;
-        this.#running = true;
-        try {
-            const now = this.#now();
-            for (const task of this.options.store().listDue(now)) {
-                if (this.#closed || task.nextRunAt === null)
-                    break;
-                const occurrenceId = this.options
-                    .store()
-                    .claim(task.sessionId, task.nextRunAt, task.revision);
-                if (occurrenceId === undefined)
-                    continue;
-                let failureCode;
-                let submitting = false;
-                try {
-                    if (initial && task.nextRunAt < now)
-                        failureCode = 'SCHEDULE_NODE_RESTARTED';
+        this.#running = this.#runDue(initial).finally(() => {
+            this.#running = undefined;
+            this.#arm();
+        });
+    }
+    async #runDue(initial) {
+        const now = this.#now();
+        for (const task of this.options.store().listDue(now)) {
+            if (this.#closed || task.nextRunAt === null)
+                break;
+            const occurrenceId = this.options
+                .store()
+                .claim(task.sessionId, task.nextRunAt, task.revision);
+            if (occurrenceId === undefined)
+                continue;
+            let failureCode;
+            let submitting = false;
+            try {
+                if (initial && task.nextRunAt < now)
+                    failureCode = 'SCHEDULE_NODE_RESTARTED';
+                else if (this.options.busy(task))
+                    failureCode = 'SCHEDULE_SESSION_BUSY';
+                if (failureCode === undefined) {
+                    const context = await this.options.authorize(task, occurrenceId);
+                    const currentTask = this.options.store().get(task.sessionId);
+                    if (this.#closed)
+                        failureCode = 'SCHEDULE_NODE_STOPPED';
+                    else if (currentTask?.revision !== task.revision || currentTask.enabled !== true)
+                        failureCode = 'SCHEDULE_TASK_CHANGED';
                     else if (this.options.busy(task))
                         failureCode = 'SCHEDULE_SESSION_BUSY';
-                    if (failureCode === undefined) {
-                        const context = await this.options.authorize(task, occurrenceId);
-                        const currentTask = this.options.store().get(task.sessionId);
-                        if (this.#closed)
-                            failureCode = 'SCHEDULE_NODE_STOPPED';
-                        else if (currentTask?.revision !== task.revision || currentTask.enabled !== true)
-                            failureCode = 'SCHEDULE_TASK_CHANGED';
-                        else if (this.options.busy(task))
-                            failureCode = 'SCHEDULE_SESSION_BUSY';
-                        else {
-                            submitting = true;
-                            const submitted = await this.options.submit(task, occurrenceId, context);
-                            this.options.store().finish(occurrenceId, 'SUBMITTED', undefined, submitted.runId);
-                        }
+                    else {
+                        submitting = true;
+                        const submitted = await this.options.submit(task, occurrenceId, context);
+                        this.options.store().finish(occurrenceId, 'SUBMITTED', undefined, submitted.runId);
                     }
                 }
-                catch (error) {
-                    failureCode =
-                        error instanceof Error && /^[A-Z][A-Z0-9_]{0,127}$/.test(error.message)
-                            ? error.message
-                            : 'SCHEDULE_EXECUTION_FAILED';
-                }
-                if (failureCode !== undefined) {
-                    const status = submitting && failureCode !== 'SCHEDULE_SESSION_BUSY' ? 'FAILED' : 'SKIPPED';
-                    this.options.store().finish(occurrenceId, status, failureCode);
-                    this.options.notify(task, occurrenceId, failureCode, status);
-                }
             }
-        }
-        finally {
-            this.#running = false;
-            this.#arm();
+            catch (error) {
+                failureCode =
+                    error instanceof Error && /^[A-Z][A-Z0-9_]{0,127}$/.test(error.message)
+                        ? error.message
+                        : 'SCHEDULE_EXECUTION_FAILED';
+            }
+            if (failureCode !== undefined) {
+                const status = submitting && failureCode !== 'SCHEDULE_SESSION_BUSY' ? 'FAILED' : 'SKIPPED';
+                this.options.store().finish(occurrenceId, status, failureCode);
+                this.notify(task, occurrenceId, failureCode, status);
+            }
         }
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@myagentroam/node",
-  "version": "0.10.61",
+  "version": "0.10.62",
   "description": "MyAgentRoam Node runtime CLI.",
   "license": "Apache-2.0 AND GPL-2.0-or-later",
   "type": "module",
@@ -37,10 +37,10 @@
     "playwright": "1.62.1",
     "ws": "^8.21.3",
     "zod": "4.4.3",
+    "@myagentroam/agent": "0.10.62",
     "@myagentroam/code-navigation": "0.10.1",
     "@myagentroam/marctl": "0.10.12",
-    "@myagentroam/agent": "0.10.61",
-    "@myagentroam/protocol": "0.10.61"
+    "@myagentroam/protocol": "0.10.62"
   },
   "devDependencies": {
     "@types/ws": "^8.18.1"