@skyramp/mcp 0.3.2-rc.pom-3 → 0.3.2

package/build/tools/code-refactor/codeReuseTool.test.js

@@ -1,572 +0,0 @@
-// @ts-nocheck - Jest ESM type inference issues
-import { jest } from "@jest/globals";
-import * as fs from "fs";
-import * as os from "os";
-import * as path from "path";
-jest.unstable_mockModule("../../utils/logger.js", () => ({
-    logger: { info: jest.fn(), error: jest.fn(), warning: jest.fn(), debug: jest.fn() },
-}));
-jest.unstable_mockModule("../../services/AnalyticsService.js", () => ({
-    AnalyticsService: { pushMCPToolEvent: jest.fn().mockResolvedValue(undefined) },
-}));
-jest.unstable_mockModule("../../utils/gitStaging.js", () => ({
-    stageGeneratedPaths: jest.fn().mockResolvedValue(undefined),
-}));
-jest.unstable_mockModule("../../utils/pom-verify/index.js", () => ({
-    verifyReuse: jest.fn().mockResolvedValue({ ok: true, violations: [], unverifiable: [], checkedCalls: 3 }),
-}));
-const { registerCodeReuseTool } = await import("./codeReuseTool.js");
-const { formatVerifyReport, formatZeroReuseGateFailure } = await import("./verify-gates.js");
-const { stageGeneratedPaths } = await import("../../utils/gitStaging.js");
-const { verifyReuse } = await import("../../utils/pom-verify/index.js");
-// Minimal Skyramp runtime infra so zero-reuse-gate fixtures pass the (separate) infra gate and
-// reach the logic under test.
-const SKYRAMP_INFRA_PREFIX = `const { test, expect, newSkyrampPlaywrightPage } = require('@skyramp/skyramp');\nconst skyrampPage = newSkyrampPlaywrightPage(page, testInfo);\n`;
-function captureToolHandler() {
-    let handler;
-    registerCodeReuseTool({ registerTool: (_n, _o, fn) => { handler = fn; } });
-    return handler;
-}
-describe("skyramp_reuse_code — verify mode", () => {
-    it("returns the verification report and skips prompt generation/staging when verify:true", async () => {
-        const handler = captureToolHandler();
-        const result = await handler({
-            testFile: "t.spec.ts",
-            language: "typescript",
-            framework: "playwright",
-            modularizeCode: false,
-            prompt: "",
-            verify: true,
-        });
-        expect(result.content[0].text).toBe("VERIFICATION PASSED — 3 calls checked.");
-        expect(verifyReuse).toHaveBeenCalledWith("t.spec.ts", "typescript");
-        expect(stageGeneratedPaths).not.toHaveBeenCalled();
-    });
-});
-describe("skyramp_reuse_code — verify mode zero-reuse compliance gate", () => {
-    it("fails verification when a zero-binding spec's selectors match a fixture POM file", async () => {
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 0 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-gate-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const pages = path.join(dir, "pages");
-            fs.mkdirSync(pages);
-            fs.writeFileSync(path.join(pages, "loginPage.js"), `export class LoginPage { constructor(page){ this.signIn = page.locator('[data-test="login-sign-in"]'); } }\n`);
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `${SKYRAMP_INFRA_PREFIX}await page.locator('[data-test="login-sign-in"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toContain("VERIFICATION FAILED — this test contains no page-object reuse, but detection found reusable POM candidates for its selectors:");
-            expect(result.content[0].text).toContain("loginPage.js");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("returns the normal PASSED response when a zero-binding spec's selectors only match discovery-fallback app source (no conventional POM dirs)", async () => {
-        // A POM-less repo whose in-repo frontend legitimately contains the spec's test-ids (built the
-        // markup, not a page-object layer) must not be gated — the selector-grep discovery fallback
-        // finding overlap here is not evidence that reuse was skipped.
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 0 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-gate-nomatch-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            fs.mkdirSync(path.join(dir, "src", "components"), { recursive: true });
-            fs.writeFileSync(path.join(dir, "src", "components", "LoginForm.jsx"), `export function LoginForm() {\n  return (<form>\n    <input data-test="login-email" />\n    <input data-test="login-password" />\n    <button data-test="login-sign-in">Sign in</button>\n  </form>);\n}\n`);
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `${SKYRAMP_INFRA_PREFIX}await page.locator('[data-test="login-email"]').fill('a@b.c');\nawait page.locator('[data-test="login-password"]').fill('x');\nawait page.locator('[data-test="login-sign-in"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toBe("VERIFICATION PASSED — 0 calls checked.");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("does not fire the gate when a deep-chain-only reuse leaves unverifiable entries (0 checked calls is not 0 reuse)", async () => {
-        // Reuse expressed entirely as a 2-hop chain (assetPage.section.method()) is non-strict, so
-        // verifyReuse reports it under `unverifiable`, not `checkedCalls`. Without the unverifiable
-        // guard this would look identical to a zero-reuse spec and falsely fire the gate.
-        verifyReuse.mockResolvedValueOnce({
-            ok: true,
-            violations: [],
-            unverifiable: ["assetPage.section.method"],
-            checkedCalls: 0,
-        });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-gate-deepchain-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const pages = path.join(dir, "pages");
-            fs.mkdirSync(pages);
-            fs.writeFileSync(path.join(pages, "assetPage.js"), `export class AssetPage { constructor(page){ this.entryBtn = page.locator('[data-test="asset-entry"]'); } }\n`);
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "refactored.spec.js");
-            fs.writeFileSync(testFile, `${SKYRAMP_INFRA_PREFIX}await assetPage.section.method();\nawait page.locator('[data-test="asset-entry"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toBe("VERIFICATION PASSED — 0 calls checked.\nUnverifiable (informational): assetPage.section.method");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("falls through to PASSED when every tier1 candidate is documented with a `// kept inline:` comment", async () => {
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 0 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-gate-documented-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const pages = path.join(dir, "pages");
-            fs.mkdirSync(pages);
-            fs.writeFileSync(path.join(pages, "loginPage.js"), `export class LoginPage { constructor(page){ this.signIn = page.locator('[data-test="login-sign-in"]'); } }\n`);
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `${SKYRAMP_INFRA_PREFIX}// kept inline: loginPage.js — this flow doesn't own the sign-in frame\nawait page.locator('[data-test="login-sign-in"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toBe("VERIFICATION PASSED — 0 calls checked.");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("accepts the exact comment format the demote flow instructs (demote → re-verify must not gate-fail)", async () => {
-        // formatVerifyReport tells a demoting agent to write `// kept inline: <POM file basename> — no
-        // matching POM method`. When the last strict call is demoted, the re-verify pass reaches this
-        // gate with checkedCalls === 0 — the demote comment itself must satisfy it, or the agent is
-        // told to re-document locators it already correctly documented.
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 0 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-gate-demoted-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const pages = path.join(dir, "pages");
-            fs.mkdirSync(pages);
-            fs.writeFileSync(path.join(pages, "loginPage.js"), `export class LoginPage { constructor(page){ this.signIn = page.locator('[data-test="login-sign-in"]'); } }\n`);
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `${SKYRAMP_INFRA_PREFIX}await page.locator('[data-test="login-sign-in"]').click(); // kept inline: loginPage.js — no matching POM method\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toBe("VERIFICATION PASSED — 0 calls checked.");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("does NOT accept a bare-basename kept-inline comment (a reason is required)", async () => {
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 0 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-gate-bare-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const pages = path.join(dir, "pages");
-            fs.mkdirSync(pages);
-            fs.writeFileSync(path.join(pages, "loginPage.js"), `export class LoginPage { constructor(page){ this.signIn = page.locator('[data-test="login-sign-in"]'); } }\n`);
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, 
-            // bare basename, no dash-separated reason — the lazy bypass the format check rejects
-            `${SKYRAMP_INFRA_PREFIX}// kept inline: loginPage.js\nawait page.locator('[data-test="login-sign-in"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toContain("VERIFICATION FAILED — this test contains no page-object reuse");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("does NOT accept a kept-inline marker embedded in a string literal (only a real comment counts)", async () => {
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 0 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-gate-string-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const pages = path.join(dir, "pages");
-            fs.mkdirSync(pages);
-            fs.writeFileSync(path.join(pages, "loginPage.js"), `export class LoginPage { constructor(page){ this.signIn = page.locator('[data-test="login-sign-in"]'); } }\n`);
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, 
-            // full marker format, but inside quoted text — must not satisfy the gate
-            `${SKYRAMP_INFRA_PREFIX}console.log("// kept inline: loginPage.js — not a real comment");\nawait page.locator('[data-test="login-sign-in"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toContain("VERIFICATION FAILED — this test contains no page-object reuse");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("surfaces the parse-error remediation, not the gate text, when the spec has broken syntax", async () => {
-        verifyReuse.mockResolvedValueOnce({
-            ok: false,
-            violations: [],
-            unverifiable: [],
-            checkedCalls: 0,
-            parseError: "SyntaxError: Unexpected end of input",
-        });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-gate-parseerror-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const pages = path.join(dir, "pages");
-            fs.mkdirSync(pages);
-            fs.writeFileSync(path.join(pages, "loginPage.js"), `export class LoginPage { constructor(page){ this.signIn = page.locator('[data-test="login-sign-in"]'); } }\n`);
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `await page.locator('[data-test="login-sign-in"]').click(\n`); // unbalanced paren
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toContain("Parse error (fix the syntax and re-verify): SyntaxError: Unexpected end of input");
-            expect(result.content[0].text).not.toContain("VERIFICATION FAILED — this test contains no page-object reuse");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("never fires the gate for a non-Playwright framework, even with matching POM candidates", async () => {
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 0 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-gate-nonplaywright-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const pages = path.join(dir, "pages");
-            fs.mkdirSync(pages);
-            fs.writeFileSync(path.join(pages, "loginPage.js"), `export class LoginPage { constructor(page){ this.signIn = page.locator('[data-test="login-sign-in"]'); } }\n`);
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `await page.locator('[data-test="login-sign-in"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "cypress",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toBe("VERIFICATION PASSED — 0 calls checked.");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("falls through to PASSED when POM scoping fails (e.g. testFile in a nonexistent directory)", async () => {
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 0 });
-        const testFile = path.join(os.tmpdir(), "code-reuse-gate-does-not-exist", "generated.spec.js");
-        const handler = captureToolHandler();
-        const result = await handler({
-            testFile,
-            language: "javascript",
-            framework: "playwright",
-            modularizeCode: false,
-            prompt: "",
-            verify: true,
-        });
-        expect(result.content[0].text).toBe("VERIFICATION PASSED — 0 calls checked.");
-    });
-});
-describe("skyramp_reuse_code — verify mode infra gate", () => {
-    it("fails verification when the Skyramp runtime import is missing entirely (repo fixtures used instead)", async () => {
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 2 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-infra-noimport-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `const { test, expect } = require('../fixtures/enhanced-baseFixtures');\nawait page.locator('[data-test="login-sign-in"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toContain("VERIFICATION FAILED — the Skyramp runtime infrastructure is missing");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("fails verification when @skyramp/skyramp is imported but newSkyrampPlaywrightPage( is never called", async () => {
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 2 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-infra-nowrapper-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `const { test, expect } = require('@skyramp/skyramp');\nawait page.locator('[data-test="login-sign-in"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toContain("VERIFICATION FAILED — the Skyramp runtime infrastructure is missing");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("fails verification when the runtime import and wrapper call are only present commented out", async () => {
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 2 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-infra-commented-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `// const { test, expect, newSkyrampPlaywrightPage } = require('@skyramp/skyramp');\n// const skyrampPage = newSkyrampPlaywrightPage(page, testInfo);\nconst { test, expect } = require('../fixtures/enhanced-baseFixtures');\nawait page.locator('[data-test="login-sign-in"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toContain("VERIFICATION FAILED — the Skyramp runtime infrastructure is missing");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("fails verification when the wrapper call appears only inside a string literal", async () => {
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 2 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-infra-string-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, 
-            // real import, but the wrapper is never actually called — only mentioned in prose
-            `const { test, expect } = require('@skyramp/skyramp');\nconsole.log("newSkyrampPlaywrightPage( is not needed here");\nawait page.locator('[data-test="login-sign-in"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toContain("VERIFICATION FAILED — the Skyramp runtime infrastructure is missing");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("appends the mapping-violation report when the infra gate fires on a spec that also has violations", async () => {
-        verifyReuse.mockResolvedValueOnce({
-            ok: false,
-            violations: [{ binding: "pm", member: "fakeMethod", searched: ["pages/loginPage.js"], availableMembers: ["login"] }],
-            unverifiable: [],
-            checkedCalls: 1,
-        });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-infra-violations-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `const { test, expect } = require('../fixtures/enhanced-baseFixtures');\nawait pm.fakeMethod();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            const text = result.content[0].text;
-            expect(text).toContain("VERIFICATION FAILED — the Skyramp runtime infrastructure is missing");
-            expect(text).toContain("pm.fakeMethod — not found in pages/loginPage.js. Available members: login");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("does not fire the infra gate when both the import and the wrapper call are present", async () => {
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 2 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-infra-present-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `${SKYRAMP_INFRA_PREFIX}await page.locator('[data-test="login-sign-in"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).not.toContain("VERIFICATION FAILED — the Skyramp runtime infrastructure is missing");
-            expect(result.content[0].text).toBe("VERIFICATION PASSED — 2 calls checked.");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("never fires the infra gate for a non-Playwright framework", async () => {
-        verifyReuse.mockResolvedValueOnce({ ok: true, violations: [], unverifiable: [], checkedCalls: 2 });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-infra-nonplaywright-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `const { test, expect } = require('../fixtures/enhanced-baseFixtures');\nawait page.locator('[data-test="login-sign-in"]').click();\n`);
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "cypress",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toBe("VERIFICATION PASSED — 2 calls checked.");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-    it("surfaces the parse-error remediation, not the infra text, when the spec has broken syntax and is missing the imports", async () => {
-        verifyReuse.mockResolvedValueOnce({
-            ok: false,
-            violations: [],
-            unverifiable: [],
-            checkedCalls: 0,
-            parseError: "SyntaxError: Unexpected end of input",
-        });
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "code-reuse-infra-parseerror-"));
-        try {
-            fs.writeFileSync(path.join(dir, "package.json"), "{}");
-            const testsDir = path.join(dir, "tests");
-            fs.mkdirSync(testsDir);
-            const testFile = path.join(testsDir, "generated.spec.js");
-            fs.writeFileSync(testFile, `await page.locator('[data-test="login-sign-in"]').click(\n`); // unbalanced paren, no skyramp imports
-            const handler = captureToolHandler();
-            const result = await handler({
-                testFile,
-                language: "javascript",
-                framework: "playwright",
-                modularizeCode: false,
-                prompt: "",
-                verify: true,
-            });
-            expect(result.content[0].text).toContain("Parse error (fix the syntax and re-verify): SyntaxError: Unexpected end of input");
-            expect(result.content[0].text).not.toContain("VERIFICATION FAILED — the Skyramp runtime infrastructure is missing");
-        }
-        finally {
-            fs.rmSync(dir, { recursive: true, force: true });
-        }
-    });
-});
-describe("formatVerifyReport", () => {
-    it("reports a violation block with the fix-rule paragraph, not the passed message", () => {
-        const text = formatVerifyReport({
-            ok: false,
-            violations: [{ binding: "pm", member: "fakeMethod", searched: ["a.ts", "b.ts"], availableMembers: ["realMethod"] }],
-            unverifiable: [],
-            checkedCalls: 1,
-        });
-        expect(text).toContain("VERIFICATION FAILED");
-        expect(text).toContain("pm.fakeMethod — not found in a.ts (+1-hop). Available members: realMethod");
-        expect(text).toContain("Do not finish until it reports PASSED.");
-    });
-});
-describe("formatZeroReuseGateFailure", () => {
-    it("lists each tier1 candidate with its matched tokens", () => {
-        const text = formatZeroReuseGateFailure([
-            { file: "pages/loginPage.js", score: 1, rareHits: 1, matchedTokens: ["login-sign-in", "login-submit"] },
-        ]);
-        expect(text).toContain("VERIFICATION FAILED — this test contains no page-object reuse, but detection found reusable POM candidates for its selectors:");
-        expect(text).toContain("- pages/loginPage.js (matched: login-sign-in, login-submit)");
-        expect(text).toContain("A raw locator that matches a POM property verbatim is evidence TO substitute, never a reason to stay inline.");
-        expect(text).toContain("Only finish with raw locators if you add a comment for each file above — `// kept inline: <file basename> — <reason it cannot cover this test>` — then re-run this verification, which will accept the documented files.");
-    });
-    it("labels an import-expanded candidate (no own token hits) instead of printing an empty matched list", () => {
-        const text = formatZeroReuseGateFailure([
-            { file: "pages/basePage.js", score: 1, rareHits: 0, matchedTokens: [] },
-        ]);
-        expect(text).toContain("- pages/basePage.js (imported by a matched page-object file)");
-        expect(text).not.toContain("(matched: )");
-    });
-});

package/build/tools/generate-tests/generateBatchScenarioRestTool.test.d.ts

@@ -1 +0,0 @@
-export {};