@ondc/automation-mock-runner 1.3.19 → 1.3.21

package/dist/lib/MockRunner.d.ts

@@ -6,6 +6,7 @@ export declare class MockRunner {
     private config;
     private static sharedRunner;
     logger: Logger;
+    private static getSharedRunner;
     constructor(config: MockPlaygroundConfigType, skipValidation?: boolean);
     getRunnerInstance(): BaseCodeRunner;
     getConfig(): {

package/dist/lib/MockRunner.js

@@ -12,6 +12,12 @@ const jsonpath_1 = __importDefault(require("jsonpath"));
 const function_registry_1 = require("./constants/function-registry");
 const uuid_1 = require("uuid");
 class MockRunner {
+    static getSharedRunner(logger) {
+        if (!MockRunner.sharedRunner) {
+            MockRunner.sharedRunner = runner_factory_1.RunnerFactory.createRunner({}, logger);
+        }
+        return MockRunner.sharedRunner;
+    }
     constructor(config, skipValidation = false) {
         this.logger = logger_1.Logger.getInstance();
         if (!skipValidation) {
@@ -38,12 +44,9 @@ class MockRunner {
     }
     getRunnerInstance() {
         this.logger.debug("Getting code runner instance");
-        if (!MockRunner.sharedRunner) {
-            MockRunner.sharedRunner = runner_factory_1.RunnerFactory.createRunner({}, this.logger);
-        }
-        this.logger.debug("Code runner instance obtained successfully: " +
-            MockRunner.sharedRunner.toString());
-        return MockRunner.sharedRunner;
+        const runner = MockRunner.getSharedRunner(this.logger);
+        this.logger.debug("Code runner instance obtained successfully: " + runner.toString());
+        return runner;
     }
     getConfig() {
         return this.config;
@@ -287,10 +290,7 @@ class MockRunner {
         }
     }
     getDefaultStep(api, actionId, formType) {
-        if (formType === "html_form") {
-            throw new Error("HTML form generation is not implemented yet");
-        }
-        if (formType === "dynamic_form") {
+        if (formType === "dynamic_form" || formType === "html_form") {
             return {
                 api: api,
                 action_id: actionId,
@@ -567,7 +567,7 @@ class MockRunner {
     }
     static async runGetSave(payload, expression) {
         const evalExpression = MockRunner.decodeBase64(expression);
-        const runner = runner_factory_1.RunnerFactory.createRunner();
+        const runner = MockRunner.getSharedRunner();
         const schema = (0, function_registry_1.getFunctionSchema)("getSave");
         return await runner.execute(evalExpression, schema, [payload]);
     }

package/dist/lib/configHelper.js

@@ -197,7 +197,6 @@ async function getMinifiedCode(base64Code) {
  */
 async function generatePlaygroundConfigFromFlowConfig(payloads, flowConfig) {
     flowConfig = JSON.parse(JSON.stringify(flowConfig));
-    flowConfig.sequence = flowConfig.sequence.filter((step) => step.type !== "HTML_FORM" && step.type !== "DYNAMIC_FORM");
     payloads = payloads.sort((a, b) => new Date(a.context.timestamp).getTime() -
         new Date(b.context.timestamp).getTime());
     const domain = payloads[0].context.domain;
@@ -206,33 +205,37 @@ async function generatePlaygroundConfigFromFlowConfig(payloads, flowConfig) {
     const mockRunner = new MockRunner_1.MockRunner(config);
     let index = 0;
     for (const step of flowConfig.sequence) {
-        if (step.type === "HTML_FORM" ||
+        const isFormStep = step.type === "HTML_FORM" ||
             step.type === "DYNAMIC_FORM" ||
-            step.type === "FORM") {
-            continue;
+            step.type === "FORM";
+        let stepConfig;
+        if (isFormStep) {
+            // HTML_FORM is not yet fully implemented — fall back to dynamic_form default
+            stepConfig = mockRunner.getDefaultStep(step.type, step.key, "dynamic_form");
         }
-        let stepPayload = payloads.findIndex((p) => p.context.action === step.type);
-        const payload = stepPayload === -1 ? {} : payloads[stepPayload];
-        if (stepPayload !== -1) {
-            payloads.splice(stepPayload, 1); // remove used payload
-        }
-        const stepConfig = mockRunner.getDefaultStep(step.type, step.key);
-        if (index === 0) {
-            stepConfig.mock.generate = MockRunner_1.MockRunner.encodeBase64(`async function generate(defaultPayload, sessionData) {
+        else {
+            stepConfig = mockRunner.getDefaultStep(step.type, step.key);
+            if (index === 0) {
+                stepConfig.mock.generate = MockRunner_1.MockRunner.encodeBase64(`async function generate(defaultPayload, sessionData) {
   	setCityFromInputs(defaultPayload, sessionData.user_inputs);
   return defaultPayload;
   }`);
-            stepConfig.mock.inputs = cityInputs;
+                stepConfig.mock.inputs = cityInputs;
+            }
+            else {
+                stepConfig.mock.inputs = {};
+            }
+            const stepPayloadIndex = payloads.findIndex((p) => p.context.action === step.type);
+            if (stepPayloadIndex !== -1) {
+                stepConfig.mock.defaultPayload = payloads[stepPayloadIndex];
+                payloads.splice(stepPayloadIndex, 1); // remove used payload
+            }
+            index++;
         }
-        else {
-            stepConfig.mock.inputs = {};
-        }
-        stepConfig.mock.defaultPayload = payload;
         const findResponseFor = flowConfig.sequence.find((s) => s.pair === step.key);
         stepConfig.responseFor = findResponseFor ? findResponseFor.key : null;
         stepConfig.unsolicited = step.unsolicited;
         config.steps.push(stepConfig);
-        index++;
     }
     return config;
 }

package/dist/lib/runners/node-runner.js

@@ -37,7 +37,7 @@ exports.NodeRunner = void 0;
 const worker_threads_1 = require("worker_threads");
 const code_validator_1 = require("../validators/code-validator");
 const path = __importStar(require("path"));
-const DEFAULT_POOL_SIZE = 10;
+const DEFAULT_POOL_SIZE = 2;
 const MAX_EXECUTIONS_PER_WORKER = 100;
 const MAX_WORKER_AGE_MS = 10 * 60 * 1000; // 10 minutes
 class NodeRunner {

package/dist/test/MockRunner.test.js

@@ -5,6 +5,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const configHelper_1 = require("../lib/configHelper");
 const MockRunner_1 = require("../lib/MockRunner");
+const runner_factory_1 = require("../lib/runners/runner-factory");
 describe("MockRunner", () => {
     let mockRunner;
     let mockConfig;
@@ -35,7 +36,12 @@ describe("MockRunner", () => {
             throw error;
         }
     });
-    afterEach(() => {
+    afterEach(async () => {
+        const sharedRunner = MockRunner_1.MockRunner.sharedRunner;
+        if (sharedRunner?.terminate) {
+            await sharedRunner.terminate();
+        }
+        MockRunner_1.MockRunner.sharedRunner = undefined;
         jest.clearAllMocks();
     });
     describe("Constructor", () => {
@@ -68,6 +74,16 @@ describe("MockRunner", () => {
             const mockRunnerB = new MockRunner_1.MockRunner(optimizedConfig, true);
             expect(mockRunnerA.getRunnerInstance()).toBe(mockRunnerB.getRunnerInstance());
         });
+        it("should reuse singleton runner for runGetSave", async () => {
+            MockRunner_1.MockRunner.sharedRunner = undefined;
+            const createRunnerSpy = jest.spyOn(runner_factory_1.RunnerFactory, "createRunner");
+            mockRunner.getRunnerInstance();
+            const getSaveFunction = MockRunner_1.MockRunner.encodeBase64(`async function getSave(payload){ return payload.value; }`);
+            const result = await MockRunner_1.MockRunner.runGetSave({ value: "singleton-check" }, getSaveFunction);
+            expect(result.success).toBe(true);
+            expect(result.result).toBe("singleton-check");
+            expect(createRunnerSpy).toHaveBeenCalledTimes(1);
+        });
     });
     describe("Config Validation", () => {
         it("should validate correct config successfully", () => {

package/dist/test/helper-config.test.js

@@ -686,71 +686,6 @@ describe("configHelper", () => {
             expect(onSearchStepConfig.unsolicited).toBe(false);
             expect(onSearchStepConfig.mock.defaultPayload).toBe(onSearchPayload);
         });
-        it("should ignore HTML_FORM and DYNAMIC_FORM steps and preserve unsolicited flag", async () => {
-            const payloads = [
-                {
-                    context: {
-                        action: "search",
-                        timestamp: "2025-01-01T09:00:00.000Z",
-                        domain: "ONDC:RET10",
-                        version: "2.0.0",
-                    },
-                },
-                {
-                    context: {
-                        action: "on_status",
-                        timestamp: "2025-01-01T10:00:00.000Z",
-                        domain: "ONDC:RET10",
-                        version: "2.0.0",
-                    },
-                },
-            ];
-            const flowConfig = {
-                id: "flow_with_forms",
-                sequence: [
-                    {
-                        key: "search_step",
-                        type: "search",
-                        unsolicited: false,
-                        description: "Search step",
-                        pair: null,
-                        owner: "BAP",
-                    },
-                    {
-                        key: "html_form_step",
-                        type: "HTML_FORM",
-                        unsolicited: false,
-                        description: "HTML form step",
-                        pair: null,
-                        owner: "BAP",
-                    },
-                    {
-                        key: "dynamic_form_step",
-                        type: "DYNAMIC_FORM",
-                        unsolicited: false,
-                        description: "Dynamic form step",
-                        pair: null,
-                        owner: "BAP",
-                    },
-                    {
-                        key: "on_status_step",
-                        type: "on_status",
-                        unsolicited: true,
-                        description: "Unsolicited status",
-                        pair: null,
-                        owner: "BPP",
-                    },
-                ],
-            };
-            const config = await (0, configHelper_1.generatePlaygroundConfigFromFlowConfig)(payloads, flowConfig);
-            // Only non-form steps should be present
-            expect(config.steps.map((s) => s.action_id)).toEqual([
-                "search_step",
-                "on_status_step",
-            ]);
-            const onStatusStep = config.steps.find((s) => s.action_id === "on_status_step");
-            expect(onStatusStep?.unsolicited).toBe(true);
-        });
         it("should derive version from core_version when version is missing", async () => {
             const payloads = [
                 {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@ondc/automation-mock-runner",
-	"version": "1.3.19",
+	"version": "1.3.21",
 	"description": "A TypeScript library for ONDC automation mock runner",
 	"main": "dist/index.js",
 	"types": "dist/index.d.ts",