@skyramp/mcp 0.2.6 → 0.2.8-rc.1

package/build/tools/test-management/uiAnalyzeChangesTool.js

@@ -1,155 +0,0 @@
-import * as fs from "fs";
-import * as path from "path";
-import { z } from "zod";
-import { logger } from "../../utils/logger.js";
-import { enumerateCandidateUiPages } from "../../utils/uiPageEnumerator.js";
-import { hasFlutterSdkDep, isFrontendFile, isTestFile } from "../../prompts/test-recommendation/scopeAssessment.js";
-import { parseChangedFilesFromDiff } from "../../utils/branchDiff.js";
-import { toolText } from "../../utils/utils.js";
-import { isTestbotEnabled } from "../../utils/featureFlags.js";
-/**
- * skyramp_ui_analyze_changes — runs before skyramp_analyze_changes when the
- * caller (typically the testbot agent) wants blueprint-grounded recommendations.
- *
- * Computes candidateUiPages[] using the same source-grounded logic that
- * analyze_changes uses internally (shared enumerator), then emits explicit
- * instructions for the agent: navigate to each candidate URL, capture a
- * browser_blueprint, then call skyramp_analyze_changes with the captures
- * available as element vocabulary when the agent writes UI rec reasoning.
- *
- * Graceful degradation: backend-only PRs produce a "no UI work" instruction;
- * unresolvable frontend baseUrl produces "skip captures, call analyze_changes
- * with empty array." analyze_changes still works in either case — same
- * source-grounded output as before slice 4.
- */
-export async function runUiAnalyzeChanges(params) {
-    const repoPath = params.repositoryPath;
-    // Read changedFiles from the canonical PR diff file. The testbot runtime
-    // writes this before the prompt runs.
-    const diffPath = path.join(repoPath, ".skyramp_git_diff");
-    let changedFiles = [];
-    let diffFileMissing = false;
-    try {
-        const raw = fs.readFileSync(diffPath, "utf-8");
-        changedFiles = parseChangedFilesFromDiff(raw);
-    }
-    catch {
-        // No diff file — for testbot this would be a runtime bug (it always
-        // writes one); for non-testbot callers it means "diff unavailable",
-        // which is NOT the same as "no UI changes". Surface that distinction.
-        diffFileMissing = true;
-    }
-    if (diffFileMissing) {
-        const uiContext = {
-            changedFrontendFiles: [],
-            candidateUiPages: [],
-        };
-        return {
-            uiContext,
-            instructions: DIFF_FILE_MISSING_INSTRUCTIONS,
-        };
-    }
-    // Compute hasFlutterSdkDep once at the tool boundary; pass into isFrontendFile
-    // so .dart files are recognised as frontend in Flutter repos. See Confluence
-    // "Flutter support in Testbot" — this is one of two budget-driving call sites
-    // that must thread the flag (the other is analyzeChangesTool). Without it,
-    // a Flutter PR shows zero frontend files and never enters the UI pipeline.
-    const flutterSdk = hasFlutterSdkDep(repoPath);
-    const frontendFiles = changedFiles.filter((f) => isFrontendFile(f, { hasFlutterSdkDep: flutterSdk }) && !isTestFile(f));
-    if (frontendFiles.length === 0) {
-        const uiContext = {
-            changedFrontendFiles: [],
-            candidateUiPages: [],
-        };
-        return {
-            uiContext,
-            instructions: NO_UI_INSTRUCTIONS,
-        };
-    }
-    const candidateUiPages = await enumerateCandidateUiPages(repoPath, frontendFiles);
-    const uiContext = {
-        changedFrontendFiles: frontendFiles,
-        candidateUiPages,
-    };
-    const instructions = candidateUiPages.length === 0
-        ? NO_RESOLVABLE_URLS_INSTRUCTIONS
-        : buildCaptureInstructions(candidateUiPages, params.uiCredentials);
-    logger.info("ui_analyze_changes computed candidate UI pages", {
-        repositoryPath: repoPath,
-        frontendFileCount: frontendFiles.length,
-        candidateUiPageCount: candidateUiPages.length,
-        hasUiCredentials: !!params.uiCredentials,
-    });
-    return { uiContext, instructions };
-}
-const NO_UI_INSTRUCTIONS = `No UI changes detected in this PR. Proceed directly to \`skyramp_analyze_changes\`.`;
-const NO_RESOLVABLE_URLS_INSTRUCTIONS = `Frontend changes detected but no candidate URLs could be resolved (workspace baseUrl missing or no router files matched). Proceed to \`skyramp_analyze_changes\`; UI recommendations will be source-grounded only.`;
-const DIFF_FILE_MISSING_INSTRUCTIONS = `\`.skyramp_git_diff\` not found in the repository. This file is normally written by the testbot runtime before the prompt runs; without it, this tool cannot determine which files changed. **Do not skip blueprint capture on the assumption there are no UI changes** — that conclusion is not supported by the available data. If you have access to the changed-files list elsewhere (e.g. a \`gh pr diff\` command, an environment variable), determine UI candidates yourself and capture their blueprints before proceeding. Otherwise proceed to \`skyramp_analyze_changes\` and add an \`issuesFound\` info entry: "ui_analyze_changes could not read .skyramp_git_diff; recommendations are source-grounded only".`;
-function buildCaptureInstructions(pages, uiCredentials) {
-    // The `<ui-credentials>` tag is only emitted by the testbot orchestration
-    // prompt. When this tool is called from an IDE (Cursor, etc.) outside
-    // testbot, the agent has no such tag — embed the credentials inline
-    // instead. uiCredentials is "email:password" formatted by the caller.
-    const loginStep = uiCredentials
-        ? (isTestbotEnabled()
-            ? `\n0. **Log in first.** \`browser_navigate\` to the app's login URL, then use \`<ui-credentials>\` from the testbot prompt to authenticate (you have those credentials in context). Without authentication, the candidate URLs below may redirect to login and the captured blueprints will be of the login page, not the feature.\n`
-            : `\n0. **Log in first.** \`browser_navigate\` to the app's login URL, then authenticate with the credentials \`${uiCredentials}\` (formatted as \`email:password\`). Without authentication, the candidate URLs below may redirect to login and the captured blueprints will be of the login page, not the feature.\n`)
-        : "";
-    const pagesYaml = pages
-        .map((p, i) => `  ${i + 1}. ${p.url}  (sourcedFrom: ${p.sourcedFrom.join(", ") || "(none)"}, strategy: ${p.strategy})`)
-        .join("\n");
-    return `Frontend changes detected. **Before calling \`skyramp_analyze_changes\`, capture blueprints on the candidate UI pages below.** Those captures stay in your tool-result history and serve as element vocabulary when you write UI test recommendations — \`skyramp_analyze_changes\` returns the authoring rules; you bring the observed elements.
-${loginStep}
-**Candidate URLs:**
-${pagesYaml}
-
-**For each candidate URL:**
-  - \`browser_navigate\` to the URL
-  - \`browser_blueprint\` to capture the page
-
-You don't need to thread the blueprints back into a tool call — they're in your context once captured. **Then call \`skyramp_analyze_changes\`** to get the recommendation catalog and the UI authoring rules.
-
-If a candidate URL 404s or redirects unexpectedly, navigate from the workspace baseUrl and explore (admin apps mount routes under base prefixes the source extraction can't see). If the page rendered but lacks the changed feature (gated UI: modal, dropdown, accordion), do NOT iterate further during this step — UI recs will fall back to source-grounded prose for those, and the agent's later trace recording (Task 2) will navigate into the gate via capture-act-capture.
-
-If \`browser_blueprint\` fails on every candidate URL (app unreachable, all 404s), proceed to \`skyramp_analyze_changes\` and log an \`issuesFound\` info entry. Recommendations will be source-grounded; non-UI work is unaffected.`;
-}
-export const uiAnalyzeChangesInputSchema = {
-    repositoryPath: z
-        .string()
-        .describe("Absolute path to the repository root"),
-    uiCredentials: z
-        .string()
-        .optional()
-        .describe("Optional ui-credentials for authed apps. When present, the returned " +
-        "instructions include a login step before blueprint capture."),
-};
-export function registerUiAnalyzeChangesTool(server) {
-    server.registerTool("skyramp_ui_analyze_changes", {
-        description: [
-            "Run BEFORE skyramp_analyze_changes when the PR may contain frontend changes.",
-            "Enumerates candidate UI pages from the diff and instructs the agent to",
-            "capture browser_blueprints on each before calling analyze_changes. The",
-            "captures stay in the agent's tool-result history and serve as the",
-            "element vocabulary for UI rec reasoning.",
-            "",
-            "On backend-only PRs, returns a 'no UI work' instruction — caller can",
-            "proceed directly to skyramp_analyze_changes.",
-        ].join("\n"),
-        inputSchema: uiAnalyzeChangesInputSchema,
-        annotations: {
-            readOnlyHint: true, // reads .skyramp_git_diff + workspace.yml; no writes
-            idempotentHint: true, // pure function of repo state + uiCredentials
-            openWorldHint: false, // no external service calls
-        },
-    }, async (params) => {
-        const result = await runUiAnalyzeChanges(params);
-        const summary = JSON.stringify({ uiContext: result.uiContext }, null, 2);
-        return toolText(`<summary>
-${summary}
-</summary>
-
-<instructions>
-${result.instructions}
-</instructions>`);
-    });
-}

package/build/tools/test-management/uiAnalyzeChangesTool.test.js

@@ -1,147 +0,0 @@
-jest.mock("@skyramp/skyramp", () => ({
-    WorkspaceConfigManager: { create: jest.fn() },
-}));
-import { runUiAnalyzeChanges } from "./uiAnalyzeChangesTool.js";
-import * as workspaceAuth from "../../utils/workspaceAuth.js";
-import * as fs from "fs";
-import * as os from "os";
-import * as path from "path";
-function makeRepoWithDiff(changedFiles) {
-    const dir = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-uiac-test-"));
-    // Write .skyramp_git_diff with the requested file changes — match the
-    // format the runtime uses (raw `diff --git a/X b/X` block).
-    const diff = changedFiles
-        .map((f) => `diff --git a/${f} b/${f}\n+++ b/${f}\n@@ -0,0 +1,1 @@\n+x`)
-        .join("\n");
-    fs.writeFileSync(path.join(dir, ".skyramp_git_diff"), diff);
-    return dir;
-}
-describe("runUiAnalyzeChanges", () => {
-    let readSpy;
-    beforeEach(() => {
-        readSpy = jest.spyOn(workspaceAuth, "readWorkspaceConfigRaw");
-    });
-    afterEach(() => {
-        readSpy.mockRestore();
-    });
-    it("returns empty changedFrontendFiles for backend-only diffs", async () => {
-        readSpy.mockResolvedValue({ services: [] });
-        const repo = makeRepoWithDiff(["api/handlers/users.ts"]);
-        const result = await runUiAnalyzeChanges({ repositoryPath: repo });
-        expect(result.uiContext.changedFrontendFiles).toEqual([]);
-        expect(result.uiContext.candidateUiPages).toEqual([]);
-        expect(result.instructions).toMatch(/no UI changes/i);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("returns candidateUiPages and capture instructions for frontend diffs", async () => {
-        readSpy.mockResolvedValue({
-            services: [{ serviceName: "frontend", api: { baseUrl: "http://localhost:3000" } }],
-        });
-        const repo = makeRepoWithDiff(["frontend/src/components/Cart.tsx"]);
-        const result = await runUiAnalyzeChanges({ repositoryPath: repo });
-        expect(result.uiContext.changedFrontendFiles.length).toBeGreaterThan(0);
-        expect(result.uiContext.candidateUiPages.length).toBeGreaterThan(0);
-        expect(result.instructions).toMatch(/browser_navigate/);
-        expect(result.instructions).toMatch(/browser_blueprint/);
-        expect(result.instructions).toMatch(/skyramp_analyze_changes/);
-        // Captures stay in agent's tool-result history; we no longer thread them
-        // back through analyze_changes as a param.
-        expect(result.instructions).not.toMatch(/capturedBlueprints/);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("includes inline-credentials login step in non-testbot mode", async () => {
-        // SKYRAMP_FEATURE_TESTBOT is not set here, so the tool emits the
-        // credentials inline rather than referencing the testbot's
-        // <ui-credentials> context tag (which doesn't exist outside testbot).
-        delete process.env.SKYRAMP_FEATURE_TESTBOT;
-        readSpy.mockResolvedValue({
-            services: [{ serviceName: "frontend", api: { baseUrl: "http://localhost:3000" } }],
-        });
-        const repo = makeRepoWithDiff(["frontend/src/components/Cart.tsx"]);
-        const result = await runUiAnalyzeChanges({
-            repositoryPath: repo,
-            uiCredentials: "admin@example.com:admin",
-        });
-        expect(result.instructions).toMatch(/log in/i);
-        expect(result.instructions).toContain("admin@example.com:admin");
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("references the <ui-credentials> testbot context tag in testbot mode", async () => {
-        process.env.SKYRAMP_FEATURE_TESTBOT = "1";
-        try {
-            readSpy.mockResolvedValue({
-                services: [{ serviceName: "frontend", api: { baseUrl: "http://localhost:3000" } }],
-            });
-            const repo = makeRepoWithDiff(["frontend/src/components/Cart.tsx"]);
-            const result = await runUiAnalyzeChanges({
-                repositoryPath: repo,
-                uiCredentials: "admin@example.com:admin",
-            });
-            expect(result.instructions).toMatch(/log in/i);
-            expect(result.instructions).toContain("ui-credentials");
-            // Inline credentials should NOT leak in testbot mode (the
-            // <ui-credentials> tag is the secure carrier in that flow).
-            expect(result.instructions).not.toContain("admin@example.com:admin");
-            fs.rmSync(repo, { recursive: true, force: true });
-        }
-        finally {
-            delete process.env.SKYRAMP_FEATURE_TESTBOT;
-        }
-    });
-    it("instructs the agent to proceed to analyze_changes (source-grounded) when frontend baseUrl unresolvable", async () => {
-        readSpy.mockResolvedValue(null); // no workspace yml
-        const repo = makeRepoWithDiff(["frontend/src/components/Cart.tsx"]);
-        const result = await runUiAnalyzeChanges({ repositoryPath: repo });
-        expect(result.uiContext.changedFrontendFiles.length).toBeGreaterThan(0);
-        expect(result.uiContext.candidateUiPages).toEqual([]);
-        expect(result.instructions).toMatch(/source-grounded/i);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    // Flutter support — the headline case the Confluence plan validates.
-    // Without the .dart classifier change, a Flutter PR returns
-    // changedFrontendFiles: [] → "No UI changes detected" → agent skips
-    // browser_navigate entirely. With the change + a Flutter pubspec, the
-    // diff registers as frontend and the agent gets told to navigate.
-    it("recognises .dart files as frontend in a Flutter project (with pubspec.yaml)", async () => {
-        readSpy.mockResolvedValue({
-            services: [
-                {
-                    serviceName: "birdle-frontend",
-                    language: "typescript",
-                    framework: "playwright",
-                    api: { baseUrl: "http://localhost:8080" },
-                },
-            ],
-        });
-        const repo = makeRepoWithDiff(["lib/main.dart"]);
-        // Write a Flutter pubspec.yaml at the repo root so hasFlutterSdkDep returns true
-        fs.writeFileSync(path.join(repo, "pubspec.yaml"), `name: birdle\ndependencies:\n  flutter:\n    sdk: flutter\n`);
-        const result = await runUiAnalyzeChanges({ repositoryPath: repo });
-        expect(result.uiContext.changedFrontendFiles).toContain("lib/main.dart");
-        expect(result.uiContext.candidateUiPages.length).toBeGreaterThan(0);
-        expect(result.instructions).toMatch(/browser_navigate/);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("does NOT recognise .dart files as frontend without a Flutter pubspec.yaml", async () => {
-        // Pure Dart project (server/CLI) — pubspec.yaml absent or missing the SDK dep.
-        // .dart files should NOT enter the UI pipeline.
-        readSpy.mockResolvedValue({ services: [] });
-        const repo = makeRepoWithDiff(["bin/server.dart"]);
-        // No pubspec.yaml written at all
-        const result = await runUiAnalyzeChanges({ repositoryPath: repo });
-        expect(result.uiContext.changedFrontendFiles).toEqual([]);
-        expect(result.uiContext.candidateUiPages).toEqual([]);
-        expect(result.instructions).toMatch(/no UI changes/i);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-    it("does NOT recognise .dart files as frontend in a pure Dart project (pubspec without flutter SDK dep)", async () => {
-        // pubspec.yaml exists but declares a Dart server (e.g. shelf) — no flutter SDK
-        readSpy.mockResolvedValue({ services: [] });
-        const repo = makeRepoWithDiff(["bin/server.dart"]);
-        fs.writeFileSync(path.join(repo, "pubspec.yaml"), `name: my_dart_server\ndependencies:\n  shelf: ^1.4.0\n`);
-        const result = await runUiAnalyzeChanges({ repositoryPath: repo });
-        expect(result.uiContext.changedFrontendFiles).toEqual([]);
-        expect(result.uiContext.candidateUiPages).toEqual([]);
-        fs.rmSync(repo, { recursive: true, force: true });
-    });
-});

package/node_modules/playwright/node_modules/playwright-core/bundles/mcp/node_modules/.bin/node-which

@@ -1,52 +0,0 @@
-#!/usr/bin/env node
-var which = require("../")
-if (process.argv.length < 3)
-  usage()
-
-function usage () {
-  console.error('usage: which [-as] program ...')
-  process.exit(1)
-}
-
-var all = false
-var silent = false
-var dashdash = false
-var args = process.argv.slice(2).filter(function (arg) {
-  if (dashdash || !/^-/.test(arg))
-    return true
-
-  if (arg === '--') {
-    dashdash = true
-    return false
-  }
-
-  var flags = arg.substr(1).split('')
-  for (var f = 0; f < flags.length; f++) {
-    var flag = flags[f]
-    switch (flag) {
-      case 's':
-        silent = true
-        break
-      case 'a':
-        all = true
-        break
-      default:
-        console.error('which: illegal option -- ' + flag)
-        usage()
-    }
-  }
-  return false
-})
-
-process.exit(args.reduce(function (pv, current) {
-  try {
-    var f = which.sync(current, { all: all })
-    if (all)
-      f = f.join('\n')
-    if (!silent)
-      console.log(f)
-    return pv;
-  } catch (e) {
-    return 1;
-  }
-}, 0))

package/node_modules/playwright/node_modules/playwright-core/bundles/utils/node_modules/.bin/is-docker

@@ -1,5 +0,0 @@
-#!/usr/bin/env node
-'use strict';
-const isDocker = require('.');
-
-process.exitCode = isDocker() ? 0 : 2;

package/node_modules/playwright/node_modules/playwright-core/bundles/utils/node_modules/.bin/mime

@@ -1,46 +0,0 @@
-#!/usr/bin/env node
-
-'use strict';
-
-process.title = 'mime';
-let mime = require('.');
-let pkg = require('./package.json');
-let args = process.argv.splice(2);
-
-if (args.includes('--version') || args.includes('-v') || args.includes('--v')) {
-  console.log(pkg.version);
-  process.exit(0);
-} else if (args.includes('--name') || args.includes('-n') || args.includes('--n')) {
-  console.log(pkg.name);
-  process.exit(0);
-} else if (args.includes('--help') || args.includes('-h') || args.includes('--h')) {
-  console.log(pkg.name + ' - ' + pkg.description + '\n');
-  console.log(`Usage:
-
-  mime [flags] [path_or_extension]
-
-  Flags:
-    --help, -h                     Show this message
-    --version, -v                  Display the version
-    --name, -n                     Print the name of the program
-
-  Note: the command will exit after it executes if a command is specified
-  The path_or_extension is the path to the file or the extension of the file.
-
-  Examples:
-    mime --help
-    mime --version
-    mime --name
-    mime -v
-    mime src/log.js
-    mime new.py
-    mime foo.sh
-  `);
-  process.exit(0);
-}
-
-let file = args[0];
-let type = mime.getType(file);
-
-process.stdout.write(type + '\n');
-

package/node_modules/playwright/node_modules/playwright-core/bundles/utils/node_modules/.bin/yaml

@@ -1,11 +0,0 @@
-#!/usr/bin/env node
-
-import { UserError, cli, help } from './dist/cli.mjs'
-
-cli(process.stdin, error => {
-  if (error instanceof UserError) {
-    if (error.code === UserError.ARGS) console.error(`${help}\n`)
-    console.error(error.message)
-    process.exitCode = error.code
-  } else if (error) throw error
-})