@skyramp/mcp 0.0.59 → 0.0.60-rc.2

package/build/utils/branchDiff.js

@@ -0,0 +1,47 @@
+import { simpleGit } from "simple-git";
+import { logger } from "./logger.js";
+const MAX_DIFF_LENGTH = 50_000;
+export async function computeBranchDiff(repositoryPath, providedBaseBranch) {
+    const git = simpleGit(repositoryPath);
+    const isRepo = await git.checkIsRepo();
+    if (!isRepo) {
+        throw new Error(`"${repositoryPath}" is not a Git repository.`);
+    }
+    const branchInfo = await git.branch();
+    const currentBranch = branchInfo.current || "HEAD";
+    let baseBranch;
+    if (providedBaseBranch) {
+        baseBranch = `origin/${providedBaseBranch}`;
+    }
+    else {
+        baseBranch = "origin/main";
+        try {
+            const remoteBranches = await git.branch(["-r"]);
+            if (remoteBranches.all.some((b) => b.endsWith("/main"))) {
+                baseBranch = "origin/main";
+            }
+            else if (remoteBranches.all.some((b) => b.endsWith("/master"))) {
+                baseBranch = "origin/master";
+            }
+        }
+        catch {
+            logger.debug("Could not determine remote default branch, falling back to origin/main");
+        }
+    }
+    const changedFilesRaw = await git.diff([
+        `${baseBranch}...HEAD`,
+        "--name-only",
+    ]);
+    const changedFiles = changedFilesRaw
+        .split("\n")
+        .map((f) => f.trim())
+        .filter((f) => f.length > 0);
+    const diffStat = await git.diff([`${baseBranch}...HEAD`, "--stat"]);
+    let diffContent = await git.diff([`${baseBranch}...HEAD`]);
+    if (diffContent.length > MAX_DIFF_LENGTH) {
+        diffContent =
+            diffContent.substring(0, MAX_DIFF_LENGTH) +
+                `\n\n... [diff truncated at ${MAX_DIFF_LENGTH} chars, ${changedFiles.length} files total] ...`;
+    }
+    return { currentBranch, baseBranch, changedFiles, diffContent, diffStat };
+}

package/build/utils/pr-comment-parser.js

@@ -0,0 +1,124 @@
+/**
+ * PR Comment Parser — extracts TestBot history from GitHub PR comments.
+ *
+ * Parses previous Skyramp TestBot comments on a PR to build context
+ * about what has already been recommended, implemented, and tested.
+ * This prevents re-recommendations and enables contextual awareness
+ * across the PR lifecycle.
+ */
+import { execSync } from "child_process";
+import { logger } from "./logger.js";
+const TESTBOT_MARKERS = [
+    "Generated by Skyramp",
+    "Skyramp Testbot",
+    "skyramp-testbot",
+];
+const TEST_TYPE_PATTERN = /\b(Smoke|Contract|Integration|Fuzz|Load|E2E|UI)\b/gi;
+const ENDPOINT_PATTERN = /\b(GET|POST|PUT|PATCH|DELETE)\s+(\/\S+)/gi;
+const TEST_FILE_PATTERN = /[\w/.-]+(?:_test|_smoke|_contract|_fuzz|_integration|_load|_e2e|_ui)\.\w+/gi;
+const STATUS_PATTERN = /\b(Pass|Fail|Skipped)\b/gi;
+function isTestBotComment(body) {
+    return TESTBOT_MARKERS.some((marker) => body.includes(marker));
+}
+function extractRecommendations(comment) {
+    const results = [];
+    const body = comment.body;
+    const endpointMatches = [...body.matchAll(ENDPOINT_PATTERN)];
+    const typeMatches = [...body.matchAll(TEST_TYPE_PATTERN)];
+    const implementedFiles = [...body.matchAll(TEST_FILE_PATTERN)];
+    const hasImplementedSection = body.includes("## New Tests Created") ||
+        body.includes("newTestsCreated");
+    for (const ep of endpointMatches) {
+        const endpoint = `${ep[1].toUpperCase()} ${ep[2]}`;
+        const nearbyType = typeMatches.find((t) => Math.abs(t.index - ep.index) < 200);
+        const testType = nearbyType
+            ? nearbyType[1].toLowerCase()
+            : "unknown";
+        const isImplemented = hasImplementedSection && implementedFiles.length > 0;
+        results.push({
+            testType,
+            endpoint,
+            status: isImplemented ? "implemented" : "recommended",
+            commentId: String(comment.id),
+        });
+    }
+    return results;
+}
+function extractExecutionResults(body) {
+    const results = [];
+    const fileMatches = [...body.matchAll(TEST_FILE_PATTERN)];
+    const statusMatches = [...body.matchAll(STATUS_PATTERN)];
+    for (let i = 0; i < fileMatches.length; i++) {
+        const file = fileMatches[i][0];
+        const nearbyStatus = statusMatches.find((s) => Math.abs(s.index - fileMatches[i].index) < 150);
+        results.push({
+            testFile: file,
+            status: nearbyStatus?.[1].toLowerCase() === "pass"
+                ? "pass"
+                : "fail",
+            timestamp: new Date().toISOString(),
+        });
+    }
+    return results;
+}
+function extractImplementedFiles(body) {
+    const matches = [...body.matchAll(TEST_FILE_PATTERN)];
+    return [...new Set(matches.map((m) => m[0]))];
+}
+/**
+ * Fetch and parse PR comments to build TestBot history context.
+ *
+ * Uses the `gh` CLI (available in CI and local dev environments with
+ * GitHub CLI installed). Falls back gracefully if `gh` is unavailable.
+ */
+export async function parsePRComments(repoOwner, repoName, prNumber, _token) {
+    const empty = {
+        prNumber,
+        previousRecommendations: [],
+        implementedTestFiles: [],
+        executionResults: [],
+    };
+    let rawComments;
+    try {
+        rawComments = execSync(`gh api repos/${repoOwner}/${repoName}/issues/${prNumber}/comments --paginate`, {
+            encoding: "utf-8",
+            timeout: 15_000,
+            stdio: ["pipe", "pipe", "pipe"],
+        });
+    }
+    catch (err) {
+        logger.warning("Failed to fetch PR comments via gh CLI — skipping PR context", {
+            error: err instanceof Error ? err.message : String(err),
+        });
+        return empty;
+    }
+    let comments;
+    try {
+        comments = JSON.parse(rawComments);
+        if (!Array.isArray(comments)) {
+            return empty;
+        }
+    }
+    catch {
+        logger.warning("Failed to parse PR comments JSON");
+        return empty;
+    }
+    const testBotComments = comments.filter((c) => isTestBotComment(c.body));
+    if (testBotComments.length === 0) {
+        return empty;
+    }
+    const allRecommendations = [];
+    const allFiles = [];
+    const allExecutionResults = [];
+    for (const comment of testBotComments) {
+        allRecommendations.push(...extractRecommendations(comment));
+        allFiles.push(...extractImplementedFiles(comment.body));
+        allExecutionResults.push(...extractExecutionResults(comment.body));
+    }
+    return {
+        prNumber,
+        previousRecommendations: allRecommendations,
+        implementedTestFiles: [...new Set(allFiles)],
+        executionResults: allExecutionResults,
+    };
+}

package/build/utils/projectMetadata.js

@@ -0,0 +1,188 @@
+import * as fs from "fs";
+import * as path from "path";
+export function detectProjectMetadata(repositoryPath) {
+    const langSet = new Set();
+    const fwSet = new Set();
+    const meta = {
+        projectType: "other", primaryLanguage: "unknown", primaryFramework: "unknown",
+        deploymentPattern: "unknown", languages: [], frameworks: [], runtime: "unknown",
+        isContainerized: false, hasDockerCompose: false, hasCiCd: false,
+    };
+    const exists = (rel) => fs.existsSync(path.join(repositoryPath, rel));
+    const readJson = (rel) => {
+        try {
+            return JSON.parse(fs.readFileSync(path.join(repositoryPath, rel), "utf-8"));
+        }
+        catch {
+            return null;
+        }
+    };
+    const readText = (rel) => {
+        try {
+            return fs.readFileSync(path.join(repositoryPath, rel), "utf-8");
+        }
+        catch {
+            return "";
+        }
+    };
+    meta.isContainerized = exists("Dockerfile") || exists("backend/Dockerfile") || exists("frontend/Dockerfile");
+    meta.hasDockerCompose = exists("docker-compose.yml") || exists("docker-compose.yaml");
+    meta.hasCiCd = exists(".github/workflows") || exists(".gitlab-ci.yml") || exists("Jenkinsfile");
+    if (meta.isContainerized)
+        meta.deploymentPattern = "containerized-monolith";
+    const pkg = readJson("package.json");
+    if (pkg) {
+        langSet.add("TypeScript/JavaScript");
+        meta.runtime = "Node.js";
+        const allDeps = { ...(pkg.dependencies || {}), ...(pkg.devDependencies || {}) };
+        if (allDeps["next"]) {
+            fwSet.add("Next.js");
+            meta.primaryFramework = "Next.js";
+        }
+        if (allDeps["express"]) {
+            fwSet.add("Express");
+            if (meta.primaryFramework === "unknown")
+                meta.primaryFramework = "Express";
+        }
+        if (allDeps["react"])
+            fwSet.add("React");
+        if (allDeps["vue"])
+            fwSet.add("Vue");
+        if (allDeps["@nestjs/core"]) {
+            fwSet.add("NestJS");
+            meta.primaryFramework = "NestJS";
+        }
+        meta.primaryLanguage = allDeps["typescript"] ? "TypeScript" : "JavaScript";
+    }
+    const reqs = readText("requirements.txt") + readText("backend/requirements.txt");
+    const pyproject = readText("pyproject.toml");
+    if (reqs || exists("setup.py") || pyproject) {
+        langSet.add("Python");
+        if (!meta.runtime || meta.runtime === "unknown")
+            meta.runtime = "Python";
+        if (/fastapi/i.test(reqs + pyproject)) {
+            fwSet.add("FastAPI");
+            meta.primaryFramework = "FastAPI";
+            meta.primaryLanguage = "Python";
+        }
+        if (/django/i.test(reqs + pyproject)) {
+            fwSet.add("Django");
+            if (meta.primaryFramework === "unknown") {
+                meta.primaryFramework = "Django";
+                meta.primaryLanguage = "Python";
+            }
+        }
+        if (/flask/i.test(reqs + pyproject)) {
+            fwSet.add("Flask");
+            if (meta.primaryFramework === "unknown") {
+                meta.primaryFramework = "Flask";
+                meta.primaryLanguage = "Python";
+            }
+        }
+    }
+    if (exists("pom.xml") || exists("build.gradle")) {
+        langSet.add("Java");
+        if (meta.primaryLanguage === "unknown")
+            meta.primaryLanguage = "Java";
+        fwSet.add("Spring");
+        if (meta.primaryFramework === "unknown")
+            meta.primaryFramework = "Spring";
+        meta.runtime = "JVM";
+    }
+    // Go
+    if (exists("go.mod")) {
+        langSet.add("Go");
+        if (meta.primaryLanguage === "unknown")
+            meta.primaryLanguage = "Go";
+        meta.runtime = "Go";
+        const gomod = readText("go.mod");
+        if (/gin-gonic/i.test(gomod)) {
+            fwSet.add("Gin");
+            if (meta.primaryFramework === "unknown")
+                meta.primaryFramework = "Gin";
+        }
+        else if (/labstack\/echo/i.test(gomod)) {
+            fwSet.add("Echo");
+            if (meta.primaryFramework === "unknown")
+                meta.primaryFramework = "Echo";
+        }
+        else if (/go-chi/i.test(gomod)) {
+            fwSet.add("Chi");
+            if (meta.primaryFramework === "unknown")
+                meta.primaryFramework = "Chi";
+        }
+    }
+    // Ruby
+    if (exists("Gemfile")) {
+        langSet.add("Ruby");
+        if (meta.primaryLanguage === "unknown")
+            meta.primaryLanguage = "Ruby";
+        meta.runtime = "Ruby";
+        const gemfile = readText("Gemfile");
+        if (/rails/i.test(gemfile)) {
+            fwSet.add("Rails");
+            meta.primaryFramework = "Rails";
+        }
+        else if (/sinatra/i.test(gemfile)) {
+            fwSet.add("Sinatra");
+            if (meta.primaryFramework === "unknown")
+                meta.primaryFramework = "Sinatra";
+        }
+    }
+    // PHP
+    const composerJson = readJson("composer.json");
+    if (composerJson) {
+        langSet.add("PHP");
+        if (meta.primaryLanguage === "unknown")
+            meta.primaryLanguage = "PHP";
+        meta.runtime = "PHP";
+        const allPhpDeps = { ...(composerJson.require || {}), ...(composerJson["require-dev"] || {}) };
+        if (allPhpDeps["laravel/framework"]) {
+            fwSet.add("Laravel");
+            meta.primaryFramework = "Laravel";
+        }
+        else if (allPhpDeps["symfony/framework-bundle"]) {
+            fwSet.add("Symfony");
+            if (meta.primaryFramework === "unknown")
+                meta.primaryFramework = "Symfony";
+        }
+    }
+    // Rust
+    if (exists("Cargo.toml")) {
+        langSet.add("Rust");
+        if (meta.primaryLanguage === "unknown")
+            meta.primaryLanguage = "Rust";
+        meta.runtime = "Rust";
+        const cargo = readText("Cargo.toml");
+        if (/actix-web/i.test(cargo)) {
+            fwSet.add("Actix");
+            if (meta.primaryFramework === "unknown")
+                meta.primaryFramework = "Actix";
+        }
+        else if (/axum/i.test(cargo)) {
+            fwSet.add("Axum");
+            if (meta.primaryFramework === "unknown")
+                meta.primaryFramework = "Axum";
+        }
+        else if (/rocket/i.test(cargo)) {
+            fwSet.add("Rocket");
+            if (meta.primaryFramework === "unknown")
+                meta.primaryFramework = "Rocket";
+        }
+    }
+    meta.languages = Array.from(langSet);
+    meta.frameworks = Array.from(fwSet);
+    const hasBackend = meta.frameworks.some(f => /express|fastapi|django|flask|nestjs|spring|gin|echo|chi|rails|sinatra|laravel|symfony|actix|axum|rocket/i.test(f));
+    const hasFrontend = meta.frameworks.some(f => /react|vue|next/i.test(f));
+    if (hasBackend && hasFrontend)
+        meta.projectType = "full-stack";
+    else if (hasBackend)
+        meta.projectType = "rest-api";
+    else if (hasFrontend)
+        meta.projectType = "frontend";
+    if (exists("backend") && exists("frontend") && meta.projectType !== "full-stack") {
+        meta.projectType = "full-stack";
+        meta.deploymentPattern = "full-stack";
+    }
+    return meta;
+}

package/build/utils/projectMetadata.test.js

@@ -0,0 +1,81 @@
+// @ts-ignore
+import { detectProjectMetadata } from "./projectMetadata.js";
+import * as fs from "fs";
+import * as path from "path";
+import * as os from "os";
+function makeTempDir() {
+    return fs.mkdtempSync(path.join(os.tmpdir(), "pm-test-"));
+}
+describe("detectProjectMetadata", () => {
+    it("detects Node.js + Express project", () => {
+        const dir = makeTempDir();
+        fs.writeFileSync(path.join(dir, "package.json"), JSON.stringify({ dependencies: { express: "^4.18.0" } }));
+        const meta = detectProjectMetadata(dir);
+        expect(meta.primaryLanguage).toBe("JavaScript");
+        expect(meta.frameworks).toContain("Express");
+        expect(meta.projectType).toBe("rest-api");
+    });
+    it("detects Node.js + Next.js as fullstack", () => {
+        const dir = makeTempDir();
+        fs.writeFileSync(path.join(dir, "package.json"), JSON.stringify({ dependencies: { next: "^14.0.0", react: "^18.0.0" } }));
+        const meta = detectProjectMetadata(dir);
+        expect(meta.frameworks).toContain("Next.js");
+        expect(meta.frameworks).toContain("React");
+        expect(["frontend", "full-stack"]).toContain(meta.projectType);
+    });
+    it("detects Python + FastAPI project", () => {
+        const dir = makeTempDir();
+        fs.writeFileSync(path.join(dir, "requirements.txt"), "fastapi==0.100.0\nuvicorn\n");
+        const meta = detectProjectMetadata(dir);
+        expect(meta.primaryLanguage).toBe("Python");
+        expect(meta.frameworks).toContain("FastAPI");
+    });
+    it("detects Python + Django project", () => {
+        const dir = makeTempDir();
+        fs.writeFileSync(path.join(dir, "requirements.txt"), "django==4.2\n");
+        const meta = detectProjectMetadata(dir);
+        expect(meta.primaryLanguage).toBe("Python");
+        expect(meta.frameworks).toContain("Django");
+    });
+    it("detects Java + Spring project", () => {
+        const dir = makeTempDir();
+        fs.writeFileSync(path.join(dir, "pom.xml"), "<project></project>");
+        const meta = detectProjectMetadata(dir);
+        expect(meta.primaryLanguage).toBe("Java");
+        expect(meta.frameworks).toContain("Spring");
+    });
+    it("detects Go + Gin project", () => {
+        const dir = makeTempDir();
+        fs.writeFileSync(path.join(dir, "go.mod"), "module example.com/app\nrequire github.com/gin-gonic/gin v1.9.0\n");
+        const meta = detectProjectMetadata(dir);
+        expect(meta.primaryLanguage).toBe("Go");
+        expect(meta.frameworks).toContain("Gin");
+    });
+    it("detects Ruby + Rails project", () => {
+        const dir = makeTempDir();
+        fs.writeFileSync(path.join(dir, "Gemfile"), 'source "https://rubygems.org"\ngem "rails", "~> 7.0"\n');
+        const meta = detectProjectMetadata(dir);
+        expect(meta.primaryLanguage).toBe("Ruby");
+        expect(meta.frameworks).toContain("Rails");
+    });
+    it("detects PHP + Laravel project", () => {
+        const dir = makeTempDir();
+        fs.writeFileSync(path.join(dir, "composer.json"), JSON.stringify({ require: { "laravel/framework": "^10.0" } }));
+        const meta = detectProjectMetadata(dir);
+        expect(meta.primaryLanguage).toBe("PHP");
+        expect(meta.frameworks).toContain("Laravel");
+    });
+    it("detects Rust + Actix project", () => {
+        const dir = makeTempDir();
+        fs.writeFileSync(path.join(dir, "Cargo.toml"), '[dependencies]\nactix-web = "4"\n');
+        const meta = detectProjectMetadata(dir);
+        expect(meta.primaryLanguage).toBe("Rust");
+        expect(meta.frameworks).toContain("Actix");
+    });
+    it("returns unknown for empty directory", () => {
+        const dir = makeTempDir();
+        const meta = detectProjectMetadata(dir);
+        expect(meta.primaryLanguage).toBe("unknown");
+        expect(meta.primaryFramework).toBe("unknown");
+    });
+});