[Core] Clean Up Factory Pause Checkpoints

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Author: MRayermannMSFT

nodejs/src/factory.ts

@@ -14,7 +14,7 @@ import type {
 } from "./generated/rpc.js";
 import type { ContextTier } from "./generated/session-events.js";
 import type { CopilotSession } from "./session.js";
-import type { FactoryLimits, FactoryMeta } from "./types.js";
+import type { FactoryMeta } from "./types.js";
 
 export type { FactoryRunResult };
 export type {

nodejs/src/session.ts

@@ -128,10 +128,7 @@ function runInFactoryHelperScope<TResult>(
     callback: () => Promise<TResult> | TResult
 ): Promise<TResult> | TResult {
     const current = factoryExecutionStore.getStore();
-    return factoryExecutionStore.run(
-        { active: current?.active ?? false, helperScope },
-        callback
-    );
+    return factoryExecutionStore.run({ active: current?.active ?? false, helperScope }, callback);
 }
 
 /**

nodejs/test/factory.test.ts

@@ -1656,8 +1656,8 @@ describe("factories", () => {
         session.registerFactories([factory]);
 
         let settled = false;
-        const execution = session.clientSessionApis.factory!
-            .execute({
+        const execution = session.clientSessionApis
+            .factory!.execute({
                 sessionId: session.sessionId,
                 name: "checkpoint-pause",
                 runId: "run-checkpoint-pause",
@@ -2223,16 +2223,12 @@ describe("factories", () => {
     it("ignores a late abort for an older execution token with the same run id", async () => {
         const oldAgent = Promise.withResolvers<{ result: string }>();
         const currentAgent = Promise.withResolvers<{ result: string }>();
-        const sendRequest = vi.fn(
-            async (method: string, params: { executionToken?: string }) => {
-                if (method !== "session.factory.agent") {
-                    return {};
-                }
-                return params.executionToken === "old-token"
-                    ? oldAgent.promise
-                    : currentAgent.promise;
+        const sendRequest = vi.fn(async (method: string, params: { executionToken?: string }) => {
+            if (method !== "session.factory.agent") {
+                return {};
             }
-        );
+            return params.executionToken === "old-token" ? oldAgent.promise : currentAgent.promise;
+        });
         const session = new CopilotSession("session-token-scoped-abort", {
             sendRequest,
         } as never);
@@ -2264,9 +2260,7 @@ describe("factories", () => {
             executionToken: "current-token",
             args: {},
         });
-        await vi.waitFor(() =>
-            expect(sendRequest).toHaveBeenCalledTimes(2)
-        );
+        await vi.waitFor(() => expect(sendRequest).toHaveBeenCalledTimes(2));
 
         await session.clientSessionApis.factory!.abort({
             sessionId: session.sessionId,