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

package/build/utils/uiPageEnumerator.test.js

@@ -1,824 +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";
-// Under native ESM, imported module namespaces are read-only, so jest.spyOn on
-// workspaceAuth fails. Partial-mock the module: keep every real export the
-// subject's import graph needs and override only readWorkspaceConfigRaw.
-// jest.requireActual can't load an ESM module synchronously, so import the real
-// module first (unstable_mockModule is not hoisted) and spread its namespace.
-const realWorkspaceAuth = await import("./workspaceAuth.js");
-const readWorkspaceConfigRaw = jest.fn();
-jest.unstable_mockModule("./workspaceAuth.js", () => ({
-    ...realWorkspaceAuth,
-    readWorkspaceConfigRaw,
-}));
-const { frameworkFileToUrlPath, findCandidatePagesByFrameworkRoute, findCandidatePagesBySourceRoute, findRootFallbackPage, enumerateCandidateUiPages, pickFrontendBaseUrl, detectsFilesystemRouting, } = await import("./uiPageEnumerator.js");
-// Helper to create a tmp repo dir with optional config files; returns the path.
-function makeTmpRepo(configFiles = []) {
-    const dir = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-test-repo-"));
-    for (const cfg of configFiles) {
-        const fullPath = path.join(dir, cfg);
-        fs.mkdirSync(path.dirname(fullPath), { recursive: true });
-        fs.writeFileSync(fullPath, "");
-    }
-    return dir;
-}
-// ---------------------------------------------------------------------------
-// frameworkFileToUrlPath
-// ---------------------------------------------------------------------------
-describe("frameworkFileToUrlPath", () => {
-    it("maps Next.js App Router page.tsx to /path", () => {
-        expect(frameworkFileToUrlPath("app/orders/page.tsx")).toBe("/orders");
-    });
-    it("maps Next.js App Router root page.tsx to /", () => {
-        expect(frameworkFileToUrlPath("app/page.tsx")).toBe("/");
-    });
-    it("maps Next.js App Router dynamic [id] segment to :id", () => {
-        expect(frameworkFileToUrlPath("app/orders/[id]/page.tsx")).toBe("/orders/:id");
-    });
-    it("strips Next.js App Router route groups (parentheses)", () => {
-        expect(frameworkFileToUrlPath("app/(marketing)/about/page.tsx")).toBe("/about");
-    });
-    it("maps Next.js Pages Router /pages/foo.tsx to /foo", () => {
-        expect(frameworkFileToUrlPath("pages/orders.tsx")).toBe("/orders");
-    });
-    it("maps Next.js Pages Router /pages/index.tsx to /", () => {
-        expect(frameworkFileToUrlPath("pages/index.tsx")).toBe("/");
-    });
-    it("maps Next.js Pages Router /pages/foo/index.tsx to /foo", () => {
-        expect(frameworkFileToUrlPath("pages/orders/index.tsx")).toBe("/orders");
-    });
-    it("excludes Next.js Pages Router API handlers (/pages/api/*)", () => {
-        expect(frameworkFileToUrlPath("pages/api/orders.ts")).toBeNull();
-    });
-    it("maps Vue / Nuxt /pages/foo.vue to /foo", () => {
-        expect(frameworkFileToUrlPath("pages/orders.vue")).toBe("/orders");
-    });
-    it("maps Vue / Nuxt 2 /pages/_id.vue to :id", () => {
-        expect(frameworkFileToUrlPath("pages/_id.vue")).toBe("/:id");
-    });
-    it("maps SvelteKit routes/+page.svelte to /", () => {
-        expect(frameworkFileToUrlPath("src/routes/+page.svelte")).toBe("/");
-    });
-    it("maps SvelteKit routes/foo/+page.svelte to /foo", () => {
-        expect(frameworkFileToUrlPath("src/routes/orders/+page.svelte")).toBe("/orders");
-    });
-    it("returns null for a generic component file (not a route)", () => {
-        expect(frameworkFileToUrlPath("src/components/Button.tsx")).toBeNull();
-    });
-    it("returns null for a layout file in app/", () => {
-        expect(frameworkFileToUrlPath("app/orders/layout.tsx")).toBeNull();
-    });
-    it("returns null for a styles file under frontend dir", () => {
-        expect(frameworkFileToUrlPath("src/styles/global.css")).toBeNull();
-    });
-});
-// ---------------------------------------------------------------------------
-// findCandidatePagesByFrameworkRoute
-// ---------------------------------------------------------------------------
-describe("findCandidatePagesByFrameworkRoute", () => {
-    it("returns empty for empty input", () => {
-        expect(findCandidatePagesByFrameworkRoute([], "http://localhost:3000")).toEqual([]);
-    });
-    it("returns empty when no files map to routes", () => {
-        expect(findCandidatePagesByFrameworkRoute(["src/components/Button.tsx", "src/utils/format.ts"], "http://localhost:3000")).toEqual([]);
-    });
-    it("emits a single page with full URL for a Next.js App Router route", () => {
-        const pages = findCandidatePagesByFrameworkRoute(["app/orders/page.tsx"], "http://localhost:3000");
-        expect(pages).toEqual([
-            {
-                url: "http://localhost:3000/orders",
-                sourcedFrom: ["app/orders/page.tsx"],
-                strategy: "framework-route-grep",
-            },
-        ]);
-    });
-    it("collapses two files emitting the same URL into one entry with both sources", () => {
-        const pages = findCandidatePagesByFrameworkRoute(["app/orders/page.tsx", "app/orders/page.tsx"], "http://localhost:3000");
-        expect(pages).toHaveLength(1);
-        expect(pages[0].sourcedFrom).toEqual(["app/orders/page.tsx"]);
-    });
-    it("strips trailing slash from baseUrl before joining", () => {
-        const pages = findCandidatePagesByFrameworkRoute(["app/orders/page.tsx"], "http://localhost:3000/");
-        expect(pages[0].url).toBe("http://localhost:3000/orders");
-    });
-    it("ignores files that don't map to routes (mixed input)", () => {
-        const pages = findCandidatePagesByFrameworkRoute([
-            "src/components/Button.tsx", // not a route
-            "app/orders/page.tsx", // route
-            "src/styles/main.css", // not a route
-        ], "http://localhost:3000");
-        expect(pages).toHaveLength(1);
-        expect(pages[0].url).toBe("http://localhost:3000/orders");
-    });
-});
-// ---------------------------------------------------------------------------
-// findCandidatePagesBySourceRoute (strategy 2)
-// ---------------------------------------------------------------------------
-describe("findCandidatePagesBySourceRoute", () => {
-    function makeRouterRepo() {
-        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-strat2-"));
-        fs.mkdirSync(path.join(dir, "src", "pages"), { recursive: true });
-        fs.writeFileSync(path.join(dir, "src", "App.tsx"), `
-import { Routes, Route } from 'react-router-dom';
-import Cart from './pages/Cart';
-import Orders from './pages/Orders';
-
-export default function App() {
-  return (
-    <Routes>
-      <Route path="/cart" element={<Cart />} />
-      <Route path="/orders" element={<Orders />} />
-    </Routes>
-  );
-}
-`);
-        fs.writeFileSync(path.join(dir, "src", "pages", "Cart.tsx"), "export default function Cart() { return <div/>; }");
-        fs.writeFileSync(path.join(dir, "src", "pages", "Orders.tsx"), "export default function Orders() { return <div/>; }");
-        return dir;
-    }
-    it("emits candidate pages for changed files that match a route's component", () => {
-        const repo = makeRouterRepo();
-        const pages = findCandidatePagesBySourceRoute(["src/pages/Cart.tsx"], "http://localhost:3000", repo);
-        expect(pages).toHaveLength(1);
-        expect(pages[0]).toMatchObject({
-            url: "http://localhost:3000/cart",
-            strategy: "source-grounded-routes",
-        });
-        expect(pages[0].sourcedFrom).toEqual(["src/pages/Cart.tsx"]);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("emits multiple pages when multiple changed files match routes", () => {
-        const repo = makeRouterRepo();
-        const pages = findCandidatePagesBySourceRoute(["src/pages/Cart.tsx", "src/pages/Orders.tsx"], "http://localhost:3000", repo);
-        expect(pages.map((p) => p.url).sort()).toEqual([
-            "http://localhost:3000/cart",
-            "http://localhost:3000/orders",
-        ]);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("returns empty when no changed files match a route's component", () => {
-        const repo = makeRouterRepo();
-        const pages = findCandidatePagesBySourceRoute(["src/components/Button.tsx"], "http://localhost:3000", repo);
-        expect(pages).toEqual([]);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("returns empty when the repo has no router files", () => {
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-empty-"));
-        const pages = findCandidatePagesBySourceRoute(["src/pages/Cart.tsx"], "http://localhost:3000", repo);
-        expect(pages).toEqual([]);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("strips trailing slash from baseUrl", () => {
-        const repo = makeRouterRepo();
-        const pages = findCandidatePagesBySourceRoute(["src/pages/Cart.tsx"], "http://localhost:3000/", repo);
-        expect(pages[0].url).toBe("http://localhost:3000/cart");
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("emits candidate pages when the changed file is the route DECLARATION (router.tsx, not the component)", () => {
-        // Regression: previously we only matched route.componentFile. A PR that
-        // renames a route in App.tsx but doesn't touch the component itself would
-        // surface zero source-grounded candidates. Real example: directus P06
-        // renamed /settings/data-model in modules/settings/index.ts; the component
-        // was unchanged, so the agent fell through to root-fallback and missed
-        // the changed routes entirely.
-        const repo = makeRouterRepo();
-        const pages = findCandidatePagesBySourceRoute(["src/App.tsx"], "http://localhost:3000", repo);
-        expect(pages.map((p) => p.url).sort()).toEqual([
-            "http://localhost:3000/cart",
-            "http://localhost:3000/orders",
-        ]);
-        expect(pages[0].sourcedFrom).toEqual(["src/App.tsx"]);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-});
-// ---------------------------------------------------------------------------
-// pickFrontendBaseUrl
-// ---------------------------------------------------------------------------
-describe("pickFrontendBaseUrl", () => {
-    afterEach(() => {
-        readWorkspaceConfigRaw.mockReset();
-    });
-    it("returns null when workspace config is missing", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue(null);
-        expect(await pickFrontendBaseUrl("/repo")).toBeNull();
-    });
-    it("returns null when no services have baseUrl", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [{ serviceName: "backend", language: "python" }],
-        });
-        expect(await pickFrontendBaseUrl("/repo")).toBeNull();
-    });
-    it("picks the frontend service by name suffix", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [
-                { serviceName: "demoshop-backend", api: { baseUrl: "http://localhost:8000" } },
-                { serviceName: "demoshop-frontend", api: { baseUrl: "http://localhost:5173" } },
-            ],
-        });
-        expect(await pickFrontendBaseUrl("/repo")).toBe("http://localhost:5173");
-    });
-    it("picks the frontend service by framework when name doesn't signal it", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [
-                { serviceName: "api", language: "python", api: { baseUrl: "http://localhost:8000" } },
-                {
-                    serviceName: "client",
-                    language: "typescript",
-                    framework: "next",
-                    api: { baseUrl: "http://localhost:3000" },
-                },
-            ],
-        });
-        expect(await pickFrontendBaseUrl("/repo")).toBe("http://localhost:3000");
-    });
-    it("falls back to first service with baseUrl when no frontend signal", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [
-                { serviceName: "svc-a", language: "go", api: { baseUrl: "http://localhost:8000" } },
-                { serviceName: "svc-b", language: "go", api: { baseUrl: "http://localhost:8001" } },
-            ],
-        });
-        expect(await pickFrontendBaseUrl("/repo")).toBe("http://localhost:8000");
-    });
-    // Flutter support — the load-bearing assumption from the Confluence plan.
-    // init_workspace writes a single service with framework: playwright for a
-    // Flutter web app. Heuristic 1 must match by serviceName suffix
-    // (e.g. "birdle-frontend") because Heuristic 2 only knows JS/TS frontend
-    // frameworks. Without this, the agent never reaches the browser regardless
-    // of the .dart classifier fix.
-    it("picks the Flutter playwright-framework service by name suffix", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [
-                {
-                    serviceName: "birdle-frontend",
-                    language: "typescript",
-                    framework: "playwright",
-                    api: { baseUrl: "http://localhost:8080" },
-                },
-            ],
-        });
-        expect(await pickFrontendBaseUrl("/repo")).toBe("http://localhost:8080");
-    });
-    // Edge case: Flutter service has no "frontend"/"ui"/"app"/"web" suffix
-    // (so H1 misses) and `framework: playwright` isn't in H2's TS/JS framework
-    // allow-list. The chosen URL must come from H3 (first service with baseUrl).
-    // We add a second service to prove H3 picked the first, not H1 by accident.
-    it("falls back to first service when name+framework don't match H1 or H2", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [
-                {
-                    serviceName: "birdle",
-                    language: "typescript",
-                    framework: "playwright",
-                    api: { baseUrl: "http://localhost:8080" },
-                },
-                {
-                    serviceName: "other-service",
-                    language: "go",
-                    framework: "gin",
-                    api: { baseUrl: "http://localhost:9000" },
-                },
-            ],
-        });
-        expect(await pickFrontendBaseUrl("/repo")).toBe("http://localhost:8080");
-    });
-    // SKYR-3812: Environment variable override tests
-    describe("env var overrides (SKYR-3812)", () => {
-        let originalGlobal;
-        let originalPerSvc = {};
-        beforeEach(() => {
-            // Snapshot env vars this suite may touch so we can restore them exactly.
-            originalGlobal = process.env.SKYRAMP_TEST_BASE_URL;
-            originalPerSvc = {};
-            for (const key of Object.keys(process.env)) {
-                if (key.startsWith("SKYRAMP_TEST_SERVICE_URL_")) {
-                    originalPerSvc[key] = process.env[key];
-                }
-            }
-        });
-        afterEach(() => {
-            // Restore the global override.
-            if (originalGlobal === undefined) {
-                delete process.env.SKYRAMP_TEST_BASE_URL;
-            }
-            else {
-                process.env.SKYRAMP_TEST_BASE_URL = originalGlobal;
-            }
-            // Restore per-service vars: reset the union of keys that existed before
-            // the test and keys the test set, so neither pre-existing values nor
-            // test-set values leak across tests or out of the suite.
-            const keys = new Set([
-                ...Object.keys(originalPerSvc),
-                ...Object.keys(process.env).filter((k) => k.startsWith("SKYRAMP_TEST_SERVICE_URL_")),
-            ]);
-            for (const key of keys) {
-                if (originalPerSvc[key] === undefined) {
-                    delete process.env[key];
-                }
-                else {
-                    process.env[key] = originalPerSvc[key];
-                }
-            }
-            readWorkspaceConfigRaw.mockReset();
-        });
-        it("SKYRAMP_TEST_BASE_URL (global) overrides workspace.yml baseUrl", async () => {
-            readWorkspaceConfigRaw.mockResolvedValue({
-                services: [
-                    {
-                        serviceName: "frontend",
-                        api: { baseUrl: "http://localhost:3000" },
-                    },
-                ],
-            });
-            process.env.SKYRAMP_TEST_BASE_URL = "https://preview.example.com";
-            expect(await pickFrontendBaseUrl("/repo")).toBe("https://preview.example.com");
-        });
-        it("per-service SKYRAMP_TEST_SERVICE_URL_<NAME> overrides for the picked frontend service", async () => {
-            readWorkspaceConfigRaw.mockResolvedValue({
-                services: [
-                    { serviceName: "backend", api: { baseUrl: "http://localhost:8000" } },
-                    {
-                        serviceName: "demoshop-frontend",
-                        api: { baseUrl: "http://localhost:5173" },
-                    },
-                ],
-            });
-            process.env.SKYRAMP_TEST_SERVICE_URL_DEMOSHOP_FRONTEND =
-                "https://frontend-preview.example.com";
-            expect(await pickFrontendBaseUrl("/repo")).toBe("https://frontend-preview.example.com");
-        });
-        it("sanitizes service name: hyphens, dots, colons, slashes to underscores", async () => {
-            readWorkspaceConfigRaw.mockResolvedValue({
-                services: [
-                    {
-                        serviceName: "supabase-nextjs",
-                        api: { baseUrl: "http://localhost:3000" },
-                    },
-                ],
-            });
-            process.env.SKYRAMP_TEST_SERVICE_URL_SUPABASE_NEXTJS =
-                "https://supabase-preview.example.com";
-            expect(await pickFrontendBaseUrl("/repo")).toBe("https://supabase-preview.example.com");
-        });
-        it("sanitizes service name with dot: my.app -> MY_APP", async () => {
-            readWorkspaceConfigRaw.mockResolvedValue({
-                services: [
-                    { serviceName: "my.app", api: { baseUrl: "http://localhost:3000" } },
-                ],
-            });
-            process.env.SKYRAMP_TEST_SERVICE_URL_MY_APP =
-                "https://myapp-preview.example.com";
-            expect(await pickFrontendBaseUrl("/repo")).toBe("https://myapp-preview.example.com");
-        });
-        it("sanitizes service name with colon: proto://app -> PROTO___APP", async () => {
-            readWorkspaceConfigRaw.mockResolvedValue({
-                services: [
-                    {
-                        serviceName: "proto://app",
-                        api: { baseUrl: "http://localhost:3000" },
-                    },
-                ],
-            });
-            process.env.SKYRAMP_TEST_SERVICE_URL_PROTO___APP =
-                "https://proto-preview.example.com";
-            expect(await pickFrontendBaseUrl("/repo")).toBe("https://proto-preview.example.com");
-        });
-        it("per-service takes precedence over global when both are set", async () => {
-            readWorkspaceConfigRaw.mockResolvedValue({
-                services: [
-                    {
-                        serviceName: "frontend",
-                        api: { baseUrl: "http://localhost:3000" },
-                    },
-                ],
-            });
-            process.env.SKYRAMP_TEST_BASE_URL = "https://global-preview.example.com";
-            process.env.SKYRAMP_TEST_SERVICE_URL_FRONTEND =
-                "https://per-service-preview.example.com";
-            expect(await pickFrontendBaseUrl("/repo")).toBe("https://per-service-preview.example.com");
-        });
-        it("empty/whitespace global env var is ignored, falls back to workspace.yml", async () => {
-            readWorkspaceConfigRaw.mockResolvedValue({
-                services: [
-                    {
-                        serviceName: "frontend",
-                        api: { baseUrl: "http://localhost:3000" },
-                    },
-                ],
-            });
-            process.env.SKYRAMP_TEST_BASE_URL = "   ";
-            expect(await pickFrontendBaseUrl("/repo")).toBe("http://localhost:3000");
-        });
-        it("empty/whitespace per-service env var is ignored, falls back to global", async () => {
-            readWorkspaceConfigRaw.mockResolvedValue({
-                services: [
-                    {
-                        serviceName: "frontend",
-                        api: { baseUrl: "http://localhost:3000" },
-                    },
-                ],
-            });
-            process.env.SKYRAMP_TEST_BASE_URL = "https://global-preview.example.com";
-            process.env.SKYRAMP_TEST_SERVICE_URL_FRONTEND = "   ";
-            expect(await pickFrontendBaseUrl("/repo")).toBe("https://global-preview.example.com");
-        });
-        it("global override returned even with no matching service", async () => {
-            readWorkspaceConfigRaw.mockResolvedValue({
-                services: [
-                    {
-                        serviceName: "backend",
-                        language: "python",
-                        api: { baseUrl: "http://localhost:8000" },
-                    },
-                ],
-            });
-            process.env.SKYRAMP_TEST_BASE_URL = "https://global-preview.example.com";
-            // pickFrontendBaseUrl will select the first service with baseUrl (H3), but the override takes precedence
-            expect(await pickFrontendBaseUrl("/repo")).toBe("https://global-preview.example.com");
-        });
-        it("existing no-override behavior still returns workspace.yml value", async () => {
-            readWorkspaceConfigRaw.mockResolvedValue({
-                services: [
-                    {
-                        serviceName: "frontend",
-                        api: { baseUrl: "http://localhost:3000" },
-                    },
-                ],
-            });
-            // No env vars set
-            expect(await pickFrontendBaseUrl("/repo")).toBe("http://localhost:3000");
-        });
-        it("does not throw when picked service has a non-string serviceName (unvalidated YAML)", async () => {
-            // readWorkspaceConfigRaw can fall back to an unvalidated YAML parse where
-            // serviceName is e.g. a number. The per-service guard must skip it rather
-            // than calling sanitizeServiceName on a non-string and throwing.
-            readWorkspaceConfigRaw.mockResolvedValue({
-                services: [
-                    {
-                        serviceName: 123,
-                        language: "typescript",
-                        framework: "next",
-                        api: { baseUrl: "http://localhost:3000" },
-                    },
-                ],
-            });
-            process.env.SKYRAMP_TEST_BASE_URL = "https://global-preview.example.com";
-            // Falls through the per-service branch to the global override.
-            expect(await pickFrontendBaseUrl("/repo")).toBe("https://global-preview.example.com");
-        });
-    });
-});
-// ---------------------------------------------------------------------------
-// findRootFallbackPage
-// ---------------------------------------------------------------------------
-describe("findRootFallbackPage", () => {
-    afterEach(() => {
-        readWorkspaceConfigRaw.mockReset();
-    });
-    it("returns null when no frontend baseUrl resolvable", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue(null);
-        expect(await findRootFallbackPage("/repo")).toBeNull();
-    });
-    it("returns root-fallback page when frontend baseUrl resolves", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [{ serviceName: "frontend", api: { baseUrl: "http://localhost:3000" } }],
-        });
-        const page = await findRootFallbackPage("/repo");
-        expect(page).toEqual({
-            url: "http://localhost:3000",
-            sourcedFrom: [],
-            strategy: "root-fallback",
-        });
-    });
-});
-// ---------------------------------------------------------------------------
-// enumerateCandidateUiPages
-// ---------------------------------------------------------------------------
-describe("enumerateCandidateUiPages", () => {
-    afterEach(() => {
-        readWorkspaceConfigRaw.mockReset();
-    });
-    it("returns empty for empty file list", async () => {
-        expect(await enumerateCandidateUiPages("/repo", [])).toEqual([]);
-    });
-    it("returns empty when workspace has no resolvable baseUrl", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue(null);
-        expect(await enumerateCandidateUiPages("/repo", ["app/orders/page.tsx"])).toEqual([]);
-    });
-    it("uses strategy 1 (framework-route-grep) when route files match", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [{ serviceName: "frontend", api: { baseUrl: "http://localhost:3000" } }],
-        });
-        const repo = makeTmpRepo(["next.config.js"]);
-        const pages = await enumerateCandidateUiPages(repo, [
-            "app/orders/page.tsx",
-            "src/components/Button.tsx", // ignored — not a route
-        ]);
-        expect(pages).toHaveLength(1);
-        expect(pages[0]).toMatchObject({
-            url: "http://localhost:3000/orders",
-            strategy: "framework-route-grep",
-        });
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("falls back to strategy 3 (root-fallback) when no route files match", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [{ serviceName: "frontend", api: { baseUrl: "http://localhost:3000" } }],
-        });
-        const pages = await enumerateCandidateUiPages("/repo", [
-            "src/components/Button.tsx", // not a route
-            "src/state/store.ts", // not a route
-        ]);
-        expect(pages).toHaveLength(1);
-        expect(pages[0]).toMatchObject({
-            url: "http://localhost:3000",
-            strategy: "root-fallback",
-        });
-    });
-    it("emits multiple route pages when several match strategy 1", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [{ serviceName: "frontend", api: { baseUrl: "http://localhost:3000" } }],
-        });
-        // Need a real repoPath with framework config so detectsFilesystemRouting passes.
-        const repo = makeTmpRepo(["next.config.js"]);
-        const pages = await enumerateCandidateUiPages(repo, [
-            "app/orders/page.tsx",
-            "app/products/[id]/page.tsx",
-        ]);
-        expect(pages).toHaveLength(2);
-        const urls = pages.map((p) => p.url).sort();
-        expect(urls).toEqual([
-            "http://localhost:3000/orders",
-            "http://localhost:3000/products/:id",
-        ]);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("uses strategy 2 (source-grounded-routes) when strategy 1 doesn't match but routes are declared in source", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [{ serviceName: "frontend", api: { baseUrl: "http://localhost:5173" } }],
-        });
-        // Repo with no framework config (so strategy 1 won't fire) but with a
-        // React Router App.tsx declaring routes.
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-ladder-"));
-        fs.mkdirSync(path.join(repo, "src", "pages"), { recursive: true });
-        fs.writeFileSync(path.join(repo, "src", "App.tsx"), `
-import { Routes, Route } from 'react-router-dom';
-import Cart from './pages/Cart';
-export default function App() { return <Routes><Route path="/cart" element={<Cart />} /></Routes>; }
-`);
-        fs.writeFileSync(path.join(repo, "src", "pages", "Cart.tsx"), "export default function Cart() { return <div/>; }");
-        const pages = await enumerateCandidateUiPages(repo, [
-            "src/pages/Cart.tsx",
-        ]);
-        expect(pages).toHaveLength(1);
-        expect(pages[0]).toMatchObject({
-            url: "http://localhost:5173/cart",
-            strategy: "source-grounded-routes",
-        });
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("falls back to root when repo lacks a framework config (no false positives)", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [{ serviceName: "frontend", api: { baseUrl: "http://localhost:3000" } }],
-        });
-        // Repo has files matching the Pages Router pattern but no next.config —
-        // this is the demoshop case (React with conventional layout, no Next.js).
-        const repo = makeTmpRepo([]);
-        const pages = await enumerateCandidateUiPages(repo, [
-            "frontend/src/pages/Cart.tsx",
-        ]);
-        expect(pages).toHaveLength(1);
-        expect(pages[0]).toMatchObject({
-            url: "http://localhost:3000",
-            strategy: "root-fallback",
-        });
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    // Flutter end-to-end: pubspec.yaml declares the Flutter SDK, .dart files
-    // appear in the diff (passed in by callers after isFrontendFile filtering),
-    // workspace.yml has a baseUrl. Strategies 1, 2, and 2.5 don't match
-    // (no framework config; TS extractor doesn't see .dart; no GoRoute calls).
-    // Result: root-fallback with the workspace baseUrl.
-    it("returns root-fallback for a Flutter repo with no GoRoute declarations", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [
-                {
-                    serviceName: "birdle-frontend",
-                    framework: "playwright",
-                    api: { baseUrl: "http://localhost:8080" },
-                },
-            ],
-        });
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-flutter-enum-"));
-        fs.writeFileSync(path.join(repo, "pubspec.yaml"), "name: birdle\ndependencies:\n  flutter:\n    sdk: flutter\n");
-        fs.mkdirSync(path.join(repo, "lib"), { recursive: true });
-        fs.writeFileSync(path.join(repo, "lib", "main.dart"), "void main() => runApp(MyApp());");
-        const pages = await enumerateCandidateUiPages(repo, ["lib/main.dart"]);
-        expect(pages).toHaveLength(1);
-        expect(pages[0]).toMatchObject({
-            url: "http://localhost:8080",
-            strategy: "root-fallback",
-        });
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    // Strategy 2.5: Flutter repo with GoRouter declarations and no diff-match
-    // (frontendFiles names a screen file that's not imported by the router)
-    // surfaces every absolute route as a candidate URL via the all-routes
-    // fallback. Diff-match narrowing is exercised by the next test.
-    it("returns dart-go-router URLs for a Flutter repo with GoRoute declarations", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [
-                {
-                    serviceName: "books-frontend",
-                    framework: "playwright",
-                    api: { baseUrl: "http://localhost:8080" },
-                },
-            ],
-        });
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-go-router-"));
-        fs.writeFileSync(path.join(repo, "pubspec.yaml"), "name: books\ndependencies:\n  flutter:\n    sdk: flutter\n  go_router: ^14.0.0\n");
-        fs.mkdirSync(path.join(repo, "lib"), { recursive: true });
-        fs.writeFileSync(path.join(repo, "lib", "main.dart"), `
-final router = GoRouter(routes: [
-  GoRoute(path: '/signin', builder: (c, s) => SignIn()),
-  GoRoute(path: '/authors', builder: (c, s) => Authors()),
-  GoRoute(path: '/settings', builder: (c, s) => Settings()),
-]);
-`);
-        const pages = await enumerateCandidateUiPages(repo, ["lib/screens/authors.dart"]);
-        const urls = pages.map((p) => p.url).sort();
-        expect(urls).toEqual([
-            "http://localhost:8080/authors",
-            "http://localhost:8080/settings",
-            "http://localhost:8080/signin",
-        ]);
-        expect(pages.every((p) => p.strategy === "dart-go-router")).toBe(true);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    // Diff-matching: Strategy 2.5 should narrow candidate URLs to only those
-    // whose screen file is in frontendFiles. The fixture has 3 routes; only
-    // /authors's screen file (lib/src/screens/authors.dart) is in the diff.
-    it("narrows GoRouter candidates to routes whose screen file is in the diff", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [
-                {
-                    serviceName: "books-frontend",
-                    framework: "playwright",
-                    api: { baseUrl: "http://localhost:8080" },
-                },
-            ],
-        });
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-go-router-diff-"));
-        fs.writeFileSync(path.join(repo, "pubspec.yaml"), "name: books\ndependencies:\n  flutter:\n    sdk: flutter\n");
-        fs.mkdirSync(path.join(repo, "lib", "src", "screens"), { recursive: true });
-        fs.writeFileSync(path.join(repo, "lib", "main.dart"), `
-import 'src/screens/authors.dart';
-import 'src/screens/settings.dart';
-import 'src/screens/sign_in.dart';
-
-final router = GoRouter(routes: [
-  GoRoute(path: '/sign_in', builder: (c, s) => SignInScreen()),
-  GoRoute(path: '/authors', builder: (c, s) => AuthorsScreen()),
-  GoRoute(path: '/settings', builder: (c, s) => SettingsScreen()),
-]);
-`);
-        fs.writeFileSync(path.join(repo, "lib", "src", "screens", "authors.dart"), "// authors");
-        fs.writeFileSync(path.join(repo, "lib", "src", "screens", "settings.dart"), "// settings");
-        fs.writeFileSync(path.join(repo, "lib", "src", "screens", "sign_in.dart"), "// signin");
-        const pages = await enumerateCandidateUiPages(repo, ["lib/src/screens/authors.dart"]);
-        const urls = pages.map((p) => p.url).sort();
-        // Only /authors should surface — its screen file is the only one in the diff.
-        expect(urls).toEqual(["http://localhost:8080/authors"]);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    // Redirect-only routes (`redirect:` with no builder) must not surface — they
-    // have no UI to navigate to. Same fixture shape but the only diff-matching
-    // route is redirect-only, so we should fall back to the non-redirect-only
-    // routes.
-    it("filters out redirect-only GoRoutes", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [
-                {
-                    serviceName: "frontend",
-                    framework: "playwright",
-                    api: { baseUrl: "http://localhost:8080" },
-                },
-            ],
-        });
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-go-router-redirect-"));
-        fs.writeFileSync(path.join(repo, "pubspec.yaml"), "name: app\ndependencies:\n  flutter:\n    sdk: flutter\n");
-        fs.mkdirSync(path.join(repo, "lib"), { recursive: true });
-        fs.writeFileSync(path.join(repo, "lib", "main.dart"), `
-final router = GoRouter(routes: [
-  GoRoute(path: '/', redirect: (_, _) => '/home'),
-  GoRoute(path: '/home', builder: (c, s) => Home()),
-]);
-`);
-        const pages = await enumerateCandidateUiPages(repo, ["lib/main.dart"]);
-        const urls = pages.map((p) => p.url).sort();
-        // / is redirect-only — must not appear. Only /home is renderable.
-        expect(urls).toEqual(["http://localhost:8080/home"]);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    // Fallback path: when no route's screen file matches the diff (e.g. router
-    // file edited but our import-resolution heuristic missed the screen), surface
-    // ALL renderable routes rather than collapsing to empty.
-    it("falls back to all renderable routes when no screen file matches the diff", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [
-                {
-                    serviceName: "frontend",
-                    framework: "playwright",
-                    api: { baseUrl: "http://localhost:8080" },
-                },
-            ],
-        });
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-go-router-fallback-"));
-        fs.writeFileSync(path.join(repo, "pubspec.yaml"), "name: app\ndependencies:\n  flutter:\n    sdk: flutter\n");
-        fs.mkdirSync(path.join(repo, "lib"), { recursive: true });
-        fs.writeFileSync(path.join(repo, "lib", "main.dart"), `
-final router = GoRouter(routes: [
-  GoRoute(path: '/foo', builder: (c, s) => FooScreen()),
-  GoRoute(path: '/bar', builder: (c, s) => BarScreen()),
-]);
-`);
-        // Diff touches only main.dart (the router itself, not any screen). The
-        // import-resolution heuristic finds zero matches against frontendFiles, so
-        // we surface all renderable routes.
-        const pages = await enumerateCandidateUiPages(repo, ["lib/main.dart"]);
-        const urls = pages.map((p) => p.url).sort();
-        expect(urls).toEqual([
-            "http://localhost:8080/bar",
-            "http://localhost:8080/foo",
-        ]);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    // Negative gate: pure-Dart server with a `GoRoute(`-shaped string in source
-    // (e.g. shelf_router uses similar names in tests) must NOT trigger Strategy
-    // 2.5 — pubspec.yaml lacks `sdk: flutter`, hasFlutterSdkDep returns false.
-    it("does not invoke Dart extractor when pubspec.yaml lacks sdk: flutter", async () => {
-        readWorkspaceConfigRaw.mockResolvedValue({
-            services: [
-                {
-                    serviceName: "server-frontend",
-                    framework: "playwright",
-                    api: { baseUrl: "http://localhost:8080" },
-                },
-            ],
-        });
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-dart-server-"));
-        fs.writeFileSync(path.join(repo, "pubspec.yaml"), "name: server\ndependencies:\n  shelf: ^1.0.0\n");
-        fs.mkdirSync(path.join(repo, "lib"), { recursive: true });
-        fs.writeFileSync(path.join(repo, "lib", "routes.dart"), `GoRoute(path: '/api/v1', handler: (req) => Response.ok(''))`);
-        const pages = await enumerateCandidateUiPages(repo, ["lib/routes.dart"]);
-        expect(pages.map((p) => p.strategy)).toEqual(["root-fallback"]);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-});
-// ---------------------------------------------------------------------------
-// detectsFilesystemRouting
-// ---------------------------------------------------------------------------
-describe("detectsFilesystemRouting", () => {
-    let repo;
-    afterEach(() => {
-        if (repo) {
-            fs.rmSync(repo, { recursive: true, force: true });
-            repo = "";
-        }
-    });
-    it("returns false when no framework config exists", () => {
-        repo = makeTmpRepo([]);
-        expect(detectsFilesystemRouting(repo)).toBe(false);
-    });
-    it("detects Next.js via next.config.js at repo root", () => {
-        repo = makeTmpRepo(["next.config.js"]);
-        expect(detectsFilesystemRouting(repo)).toBe(true);
-    });
-    it("detects Next.js via next.config.ts", () => {
-        repo = makeTmpRepo(["next.config.ts"]);
-        expect(detectsFilesystemRouting(repo)).toBe(true);
-    });
-    it("detects Nuxt via nuxt.config.ts", () => {
-        repo = makeTmpRepo(["nuxt.config.ts"]);
-        expect(detectsFilesystemRouting(repo)).toBe(true);
-    });
-    it("detects SvelteKit via svelte.config.js", () => {
-        repo = makeTmpRepo(["svelte.config.js"]);
-        expect(detectsFilesystemRouting(repo)).toBe(true);
-    });
-    it("detects framework config one level deep (frontend/)", () => {
-        repo = makeTmpRepo(["frontend/next.config.js"]);
-        expect(detectsFilesystemRouting(repo)).toBe(true);
-    });
-    it("detects framework config in apps/web/", () => {
-        repo = makeTmpRepo(["apps/web/next.config.mjs"]);
-        expect(detectsFilesystemRouting(repo)).toBe(true);
-    });
-    it("returns false when only a pages/ directory exists (no config)", () => {
-        repo = makeTmpRepo(["pages/Cart.tsx"]);
-        expect(detectsFilesystemRouting(repo)).toBe(false);
-    });
-});