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

package/build/utils/pythonMountPrefixes.test.js

@@ -1,113 +0,0 @@
-import * as fs from "fs";
-import * as os from "os";
-import * as path from "path";
-import { joinRoutePrefix, resolvePythonMountPrefixes } from "./pythonMountPrefixes.js";
-describe("joinRoutePrefix", () => {
-    it("composes with exactly one slash regardless of surrounding slashes", () => {
-        expect(joinRoutePrefix("/api", "/recipes")).toBe("/api/recipes");
-        expect(joinRoutePrefix("/api/", "recipes")).toBe("/api/recipes");
-        expect(joinRoutePrefix("/api", "recipes/")).toBe("/api/recipes/");
-    });
-    it("returns the prefix unchanged for an empty child", () => {
-        expect(joinRoutePrefix("/api", "")).toBe("/api");
-    });
-    it("makes the child absolute for an empty prefix", () => {
-        expect(joinRoutePrefix("", "/recipes")).toBe("/recipes");
-        expect(joinRoutePrefix("", "recipes")).toBe("/recipes");
-    });
-});
-describe("resolvePythonMountPrefixes", () => {
-    const tempRepos = [];
-    afterEach(() => {
-        for (const repo of tempRepos.splice(0)) {
-            fs.rmSync(repo, { recursive: true, force: true });
-        }
-    });
-    function makeRepo(files) {
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "py-mount-"));
-        tempRepos.push(repo);
-        for (const [rel, content] of Object.entries(files)) {
-            const full = path.join(repo, rel);
-            fs.mkdirSync(path.dirname(full), { recursive: true });
-            fs.writeFileSync(full, content);
-        }
-        return repo;
-    }
-    it("resolves a relative-import mount chain to the ancestor prefix", () => {
-        const repo = makeRepo({
-            "app/__init__.py": `
-from fastapi import APIRouter
-from . import users
-
-router = APIRouter(prefix="/api")
-router.include_router(users.router)
-`,
-            "app/users.py": `
-from fastapi import APIRouter
-router = APIRouter(prefix="/users")
-@router.get("")
-def list_users(): pass
-`,
-        });
-        const result = resolvePythonMountPrefixes(repo);
-        // users.router inherits "/api" (aggregator own prefix); its own "/users" is
-        // applied later by parseFileEndpoints, so it is NOT part of the ancestor.
-        expect(result.get("app/users.py")?.get("router")).toBe("/api");
-        // The root aggregator has no ancestor prefix, so it's omitted entirely.
-        expect(result.get("app/__init__.py")).toBeUndefined();
-    });
-    it("resolves a mount prefix given as a module-level string constant", () => {
-        const repo = makeRepo({
-            "svc/__init__.py": `
-from fastapi import APIRouter
-from . import billing
-
-BILLING = "/billing"
-router = APIRouter()
-router.include_router(billing.router, prefix=BILLING)
-`,
-            "svc/billing.py": `
-from fastapi import APIRouter
-router = APIRouter()
-@router.get("/invoices")
-def invoices(): pass
-`,
-        });
-        const result = resolvePythonMountPrefixes(repo);
-        expect(result.get("svc/billing.py")?.get("router")).toBe("/billing");
-    });
-    it("fails safe (no ancestor prefix) when the mounted module can't be resolved", () => {
-        const repo = makeRepo({
-            "app/main.py": `
-from fastapi import FastAPI
-from some_external_pkg import router as ext_router
-
-app = FastAPI()
-app.include_router(ext_router, prefix="/api")
-`,
-        });
-        // The external module isn't in the repo, so the edge is dropped and nothing
-        // is returned — the scanner then behaves exactly as it does today.
-        const result = resolvePythonMountPrefixes(repo);
-        expect(result.size).toBe(0);
-    });
-    it("does not loop forever on a mutually-recursive mount cycle", () => {
-        const repo = makeRepo({
-            "a.py": `
-from fastapi import APIRouter
-from b import router as b_router
-router = APIRouter(prefix="/a")
-router.include_router(b_router)
-`,
-            "b.py": `
-from fastapi import APIRouter
-from a import router as a_router
-router = APIRouter(prefix="/b")
-router.include_router(a_router)
-`,
-        });
-        // Should terminate; the exact prefixes are unimportant, only that it returns.
-        const result = resolvePythonMountPrefixes(repo);
-        expect(result).toBeInstanceOf(Map);
-    });
-});

package/build/utils/repoScanner.test.d.ts

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

package/build/utils/repoScanner.test.js

@@ -1,190 +0,0 @@
-import * as fs from "fs";
-import * as os from "os";
-import * as path from "path";
-import { scanAllRepoEndpoints, scanRelatedEndpoints } from "./repoScanner.js";
-import { classifyEndpointsByChangedFiles } from "./routeParsers.js";
-describe("scanAllRepoEndpoints", () => {
-    const tempRepos = [];
-    afterEach(() => {
-        for (const repo of tempRepos.splice(0)) {
-            fs.rmSync(repo, { recursive: true, force: true });
-        }
-    });
-    function makeTempRepo() {
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "repo-scanner-"));
-        tempRepos.push(repo);
-        return repo;
-    }
-    it("preserves same router-relative paths from different source files", () => {
-        const repo = makeTempRepo();
-        const apiDir = path.join(repo, "src/prefect/server/api");
-        fs.mkdirSync(apiDir, { recursive: true });
-        fs.writeFileSync(path.join(apiDir, "flows.py"), `
-@router.delete(
-    "/{id:uuid}",
-    status_code=204,
-)
-async def delete_flow(): pass
-
-@router.post("/bulk_delete")
-async def bulk_delete_flows(): pass
-`);
-        fs.writeFileSync(path.join(apiDir, "deployments.py"), `
-@router.delete(
-    "/{id:uuid}",
-    status_code=204,
-)
-async def delete_deployment(): pass
-
-@router.post("/bulk_delete")
-async def bulk_delete_deployments(): pass
-`);
-        const endpoints = scanAllRepoEndpoints(repo);
-        const relatedEndpoints = scanRelatedEndpoints(repo, ["src/prefect/server/api/flows.py"]);
-        expect(endpoints).toEqual(expect.arrayContaining([
-            expect.objectContaining({ path: "/{id:uuid}", methods: ["DELETE"], sourceFile: "src/prefect/server/api/flows.py" }),
-            expect.objectContaining({ path: "/bulk_delete", methods: ["POST"], sourceFile: "src/prefect/server/api/flows.py" }),
-            expect.objectContaining({ path: "/{id:uuid}", methods: ["DELETE"], sourceFile: "src/prefect/server/api/deployments.py" }),
-            expect.objectContaining({ path: "/bulk_delete", methods: ["POST"], sourceFile: "src/prefect/server/api/deployments.py" }),
-        ]));
-        expect(relatedEndpoints).toEqual(expect.arrayContaining([
-            expect.objectContaining({ path: "/{id:uuid}", methods: ["DELETE"], sourceFile: "src/prefect/server/api/flows.py" }),
-            expect.objectContaining({ path: "/bulk_delete", methods: ["POST"], sourceFile: "src/prefect/server/api/flows.py" }),
-        ]));
-        const classified = classifyEndpointsByChangedFiles({
-            currentBranch: "feature/admin-key",
-            baseBranch: "origin/main",
-            changedFiles: ["src/prefect/server/api/flows.py"],
-            diffContent: "",
-            diffStat: "",
-            newFiles: [],
-            deletedFiles: [],
-        }, endpoints);
-        expect(classified.changedEndpoints).toEqual(expect.arrayContaining([
-            expect.objectContaining({ path: "/{id:uuid}", methods: ["DELETE"], sourceFile: "src/prefect/server/api/flows.py" }),
-            expect.objectContaining({ path: "/bulk_delete", methods: ["POST"], sourceFile: "src/prefect/server/api/flows.py" }),
-        ]));
-        expect(classified.changedEndpoints).not.toEqual(expect.arrayContaining([
-            expect.objectContaining({ path: "/{id:uuid}", methods: ["DELETE"], sourceFile: "src/prefect/server/api/deployments.py" }),
-        ]));
-    });
-    it("does not suppress removed router-relative endpoints with same-path siblings", () => {
-        const classified = classifyEndpointsByChangedFiles({
-            currentBranch: "feature/remove-flows",
-            baseBranch: "origin/main",
-            changedFiles: ["src/prefect/server/api/flows.py"],
-            diffContent: "",
-            diffStat: "",
-            newFiles: [],
-            deletedFiles: ["src/prefect/server/api/flows.py"],
-        }, [
-            { path: "/{id:uuid}", methods: ["DELETE"], sourceFile: "src/prefect/server/api/deployments.py" },
-            { path: "/bulk_delete", methods: ["POST"], sourceFile: "src/prefect/server/api/deployments.py" },
-        ], [
-            { path: "/{id:uuid}", methods: ["DELETE"], sourceFile: "src/prefect/server/api/flows.py" },
-            { path: "/bulk_delete", methods: ["POST"], sourceFile: "src/prefect/server/api/flows.py" },
-        ]);
-        expect(classified.removedEndpoints).toEqual(expect.arrayContaining([
-            { path: "/{id:uuid}", methods: ["DELETE"], sourceFile: "src/prefect/server/api/flows.py" },
-            { path: "/bulk_delete", methods: ["POST"], sourceFile: "src/prefect/server/api/flows.py" },
-        ]));
-    });
-    // FastAPI composes a route's full URL from prefixes applied at include_router()
-    // mount points that live in *other* files — the single-file parser can't see
-    // them. Mealie mounts its whole API under a top-level APIRouter(prefix="/api"),
-    // so a leaf router's own prefix ("/recipes") must be composed with the ancestor
-    // "/api" or generated tests hit the un-prefixed URL and 405.
-    it("composes cross-file include_router mount prefixes into FastAPI route paths", () => {
-        const repo = makeTempRepo();
-        const routesDir = path.join(repo, "mealie/routes");
-        const recipeDir = path.join(routesDir, "recipe");
-        fs.mkdirSync(recipeDir, { recursive: true });
-        // Top-level app: mounts the API router with no extra prefix.
-        fs.writeFileSync(path.join(repo, "mealie/app.py"), `
-from fastapi import FastAPI
-from mealie.routes import router
-
-app = FastAPI()
-app.include_router(router)
-`);
-        // API aggregator: APIRouter(prefix="/api") includes each resource router.
-        fs.writeFileSync(path.join(routesDir, "__init__.py"), `
-from fastapi import APIRouter
-from . import recipe
-
-router = APIRouter(prefix="/api")
-router.include_router(recipe.router)
-`);
-        // Recipe aggregator: no own prefix; mounts the CRUD router directly and the
-        // comments router under a "/recipes" mount prefix drawn from a local const.
-        fs.writeFileSync(path.join(recipeDir, "__init__.py"), `
-from fastapi import APIRouter
-from . import recipe_crud_routes, comments
-
-prefix = "/recipes"
-
-router = APIRouter()
-router.include_router(recipe_crud_routes.router, tags=["Recipe: CRUD"])
-router.include_router(comments.router, prefix=prefix, tags=["Recipe: Comments"])
-`);
-        // Leaf CRUD router declared via a custom APIRouter *subclass* with its own
-        // "/recipes" prefix.
-        fs.writeFileSync(path.join(recipeDir, "recipe_crud_routes.py"), `
-from mealie.routes._base.routers import UserAPIRouter
-
-router = UserAPIRouter(prefix="/recipes")
-
-@router.get("")
-async def get_all(): pass
-
-@router.get("/{slug}")
-async def get_one(): pass
-`);
-        // Comments router: no own prefix — its "/recipes" comes only from the mount.
-        fs.writeFileSync(path.join(recipeDir, "comments.py"), `
-from fastapi import APIRouter
-
-router = APIRouter()
-
-@router.get("/{slug}/comments")
-async def get_comments(): pass
-`);
-        const endpoints = scanAllRepoEndpoints(repo);
-        // CRUD: /api (aggregator) + /recipes (leaf subclass own prefix)
-        expect(endpoints).toEqual(expect.arrayContaining([
-            expect.objectContaining({ path: "/api/recipes", methods: ["GET"], sourceFile: "mealie/routes/recipe/recipe_crud_routes.py" }),
-            expect.objectContaining({ path: "/api/recipes/{slug}", methods: ["GET"], sourceFile: "mealie/routes/recipe/recipe_crud_routes.py" }),
-        ]));
-        // Comments: /api (aggregator) + /recipes (mount prefix from a local const) + own path
-        expect(endpoints).toEqual(expect.arrayContaining([
-            expect.objectContaining({ path: "/api/recipes/{slug}/comments", methods: ["GET"], sourceFile: "mealie/routes/recipe/comments.py" }),
-        ]));
-        // The un-prefixed path must NOT appear (the 405 bug).
-        expect(endpoints).not.toContainEqual(expect.objectContaining({ path: "/recipes" }));
-    });
-    it("composes an app-level include_router(prefix=...) mount", () => {
-        const repo = makeTempRepo();
-        const apiDir = path.join(repo, "src/api");
-        fs.mkdirSync(apiDir, { recursive: true });
-        // Prefix applied at the app mount point rather than on the APIRouter itself.
-        fs.writeFileSync(path.join(repo, "src/main.py"), `
-from fastapi import FastAPI
-from src.api.users import router as users_router
-
-app = FastAPI()
-app.include_router(users_router, prefix="/api/v1")
-`);
-        fs.writeFileSync(path.join(apiDir, "users.py"), `
-from fastapi import APIRouter
-
-router = APIRouter(prefix="/users")
-
-@router.get("/{id}")
-async def get_user(): pass
-`);
-        const endpoints = scanAllRepoEndpoints(repo);
-        expect(endpoints).toEqual(expect.arrayContaining([
-            expect.objectContaining({ path: "/api/v1/users/{id}", methods: ["GET"], sourceFile: "src/api/users.py" }),
-        ]));
-    });
-});

package/build/utils/reportVerification.test.d.ts

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

package/build/utils/reportVerification.test.js

@@ -1,185 +0,0 @@
-import { describe, it, expect } from "@jest/globals";
-import { execFileSync } from "child_process";
-import * as fs from "fs";
-import * as os from "os";
-import * as path from "path";
-import { DriftAction } from "../types/TestAnalysis.js";
-import { findUnbackedClaims, listChangedFiles } from "./reportVerification.js";
-const REPO = "/repo";
-describe("findUnbackedClaims", () => {
-    it("returns [] when every claim is backed by a working-tree change", () => {
-        const unbacked = findUnbackedClaims({
-            repoRoot: REPO,
-            changedFiles: [
-                "tests/skyramp/products_contract_test.py",
-                "dcim/tests/test_filtersets.py",
-            ],
-            newTests: [
-                {
-                    testId: "contract-get-products",
-                    fileName: "products_contract_test.py",
-                },
-            ],
-            verdicts: [
-                {
-                    action: DriftAction.Update,
-                    testFilePath: "/repo/dcim/tests/test_filtersets.py",
-                },
-            ],
-        });
-        expect(unbacked).toEqual([]);
-    });
-    it("flags a newTestsCreated claim whose file was never created (hallucination)", () => {
-        const unbacked = findUnbackedClaims({
-            repoRoot: REPO,
-            changedFiles: ["tests/skyramp/products_contract_test.py"],
-            newTests: [
-                {
-                    testId: "integration-phantom",
-                    fileName: "phantom_integration_test.py",
-                },
-            ],
-            verdicts: [],
-        });
-        expect(unbacked).toHaveLength(1);
-        expect(unbacked[0]).toContain("phantom_integration_test.py");
-    });
-    it("flags an UPDATE verdict whose file has no working-tree change (the SKYR-3883 case)", () => {
-        const unbacked = findUnbackedClaims({
-            repoRoot: REPO,
-            changedFiles: ["tests/skyramp/site_filter_integration_test.py"],
-            newTests: [],
-            verdicts: [
-                {
-                    action: DriftAction.Update,
-                    testFilePath: "/repo/dcim/tests/test_filtersets.py",
-                },
-            ],
-        });
-        expect(unbacked).toHaveLength(1);
-        expect(unbacked[0]).toContain("test_filtersets.py");
-    });
-    it("never flags REGENERATE/DELETE/IGNORE/VERIFY verdicts even when absent from the tree", () => {
-        // REGENERATE is exempt: external (user-authored) regenerations are report-only
-        // with no working-tree change, and internal ones may regenerate identically.
-        const unbacked = findUnbackedClaims({
-            repoRoot: REPO,
-            changedFiles: [],
-            newTests: [],
-            verdicts: [
-                {
-                    action: DriftAction.Regenerate,
-                    testFilePath: "/repo/frontend/tests/cart_pom.spec.ts",
-                },
-                { action: DriftAction.Delete, testFilePath: "/repo/tests/gone.py" },
-                { action: DriftAction.Ignore, testFilePath: "/repo/tests/kept.py" },
-                { action: DriftAction.Verify, testFilePath: "/repo/tests/checked.py" },
-            ],
-        });
-        expect(unbacked).toEqual([]);
-    });
-    it("exempts a cross-repo newTest claim (repository differs from the primary)", () => {
-        const unbacked = findUnbackedClaims({
-            repoRoot: REPO,
-            changedFiles: [],
-            newTests: [
-                {
-                    testId: "contract-related",
-                    fileName: "related_contract_test.py",
-                    repository: "acme/related-repo",
-                },
-            ],
-            verdicts: [],
-            primaryRepository: "acme/primary-repo",
-        });
-        expect(unbacked).toEqual([]);
-    });
-    it("exempts a cross-repo newTest claim even when the primary repo is unknown", () => {
-        // primaryRepository omitted: we can't confirm the file belongs to this
-        // checkout, so an explicitly-attributed claim must not be validated here.
-        const unbacked = findUnbackedClaims({
-            repoRoot: REPO,
-            changedFiles: [],
-            newTests: [
-                {
-                    testId: "contract-related",
-                    fileName: "related_contract_test.py",
-                    repository: "acme/related-repo",
-                },
-            ],
-            verdicts: [],
-        });
-        expect(unbacked).toEqual([]);
-    });
-    it("exempts a verdict whose absolute path resolves outside the checkout", () => {
-        const unbacked = findUnbackedClaims({
-            repoRoot: REPO,
-            changedFiles: [],
-            newTests: [],
-            verdicts: [
-                {
-                    action: DriftAction.Update,
-                    testFilePath: "/somewhere/else/outside_test.py",
-                },
-            ],
-        });
-        expect(unbacked).toEqual([]);
-    });
-    it("matches a nested newTest fileName by basename against the changed path", () => {
-        const backed = findUnbackedClaims({
-            repoRoot: REPO,
-            changedFiles: ["tests/skyramp/deep/nested_contract_test.py"],
-            newTests: [{ testId: "c", fileName: "nested_contract_test.py" }],
-            verdicts: [],
-        });
-        expect(backed).toEqual([]);
-    });
-});
-describe("listChangedFiles", () => {
-    it("lists staged, unstaged, and untracked paths as repo-relative strings", async () => {
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "rv-repo-"));
-        const git = (args) => execFileSync("git", args, { cwd: repo, stdio: "pipe" });
-        git(["init", "-q"]);
-        git(["config", "user.email", "t@t.dev"]);
-        git(["config", "user.name", "t"]);
-        // committed baseline
-        fs.mkdirSync(path.join(repo, "dcim/tests"), { recursive: true });
-        fs.writeFileSync(path.join(repo, "dcim/tests/test_filtersets.py"), "orig\n");
-        git(["add", "-A"]);
-        git(["commit", "-q", "-m", "base"]);
-        // modify a tracked file (unstaged) + add an untracked file
-        fs.writeFileSync(path.join(repo, "dcim/tests/test_filtersets.py"), "edited\n");
-        fs.mkdirSync(path.join(repo, "tests/skyramp"), { recursive: true });
-        fs.writeFileSync(path.join(repo, "tests/skyramp/new_contract_test.py"), "new\n");
-        const changed = await listChangedFiles(repo);
-        expect(changed).toContain("dcim/tests/test_filtersets.py");
-        expect(changed).toContain("tests/skyramp/new_contract_test.py");
-    });
-    it("attributes a renamed file to its destination path and skips the original", async () => {
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "rv-repo-"));
-        const git = (args) => execFileSync("git", args, { cwd: repo, stdio: "pipe" });
-        git(["init", "-q"]);
-        git(["config", "user.email", "t@t.dev"]);
-        git(["config", "user.name", "t"]);
-        fs.mkdirSync(path.join(repo, "dcim/tests"), { recursive: true });
-        fs.writeFileSync(path.join(repo, "dcim/tests/old_test.py"), "def test_x(): pass\n");
-        git(["add", "-A"]);
-        git(["commit", "-q", "-m", "base"]);
-        // Staged rename (identical content → git detects it as R100).
-        git(["mv", "dcim/tests/old_test.py", "dcim/tests/new_test.py"]);
-        const changed = await listChangedFiles(repo);
-        expect(changed).toContain("dcim/tests/new_test.py");
-        expect(changed).not.toContain("dcim/tests/old_test.py");
-    });
-    it("preserves a path containing spaces (NUL-delimited, no quoting)", async () => {
-        const repo = fs.mkdtempSync(path.join(os.tmpdir(), "rv-repo-"));
-        const git = (args) => execFileSync("git", args, { cwd: repo, stdio: "pipe" });
-        git(["init", "-q"]);
-        git(["config", "user.email", "t@t.dev"]);
-        git(["config", "user.name", "t"]);
-        fs.mkdirSync(path.join(repo, "tests/skyramp"), { recursive: true });
-        fs.writeFileSync(path.join(repo, "tests/skyramp/my new test.py"), "new\n");
-        const changed = await listChangedFiles(repo);
-        expect(changed).toContain("tests/skyramp/my new test.py");
-    });
-});

package/build/utils/routeParsers.test.d.ts

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