@skyramp/mcp 0.2.5 → 0.2.7

package/build/workspace/workspace.d.ts

@@ -0,0 +1,390 @@
+import { z } from "zod";
+export declare const WORKSPACE_DIR = ".skyramp";
+export declare const WORKSPACE_FILENAME = "workspace.yml";
+export declare const serviceSchema: z.ZodObject<{
+    serviceName: z.ZodString;
+    repo: z.ZodOptional<z.ZodString>;
+    language: z.ZodOptional<z.ZodEnum<["python", "typescript", "javascript", "java"]>>;
+    framework: z.ZodOptional<z.ZodEnum<["playwright", "pytest", "robot", "junit"]>>;
+    testDirectory: z.ZodOptional<z.ZodString>;
+    api: z.ZodOptional<z.ZodObject<{
+        schemaPath: z.ZodOptional<z.ZodString>;
+        authType: z.ZodOptional<z.ZodEnum<["bearer", "basic", "oauth", "apiKey", "cookie", "session", "token", "none"]>>;
+        authHeader: z.ZodOptional<z.ZodString>;
+        authScheme: z.ZodOptional<z.ZodString>;
+        baseUrl: z.ZodOptional<z.ZodString>;
+    }, "strict", z.ZodTypeAny, {
+        authHeader?: string | undefined;
+        authScheme?: string | undefined;
+        schemaPath?: string | undefined;
+        authType?: "none" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "cookie" | "session" | undefined;
+        baseUrl?: string | undefined;
+    }, {
+        authHeader?: string | undefined;
+        authScheme?: string | undefined;
+        schemaPath?: string | undefined;
+        authType?: "none" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "cookie" | "session" | undefined;
+        baseUrl?: string | undefined;
+    }>>;
+    runtimeDetails: z.ZodOptional<z.ZodObject<{
+        serverStartCommand: z.ZodOptional<z.ZodString>;
+        runtime: z.ZodEnum<["local", "docker", "k8s"]>;
+        dockerNetwork: z.ZodOptional<z.ZodString>;
+        k8sNamespace: z.ZodOptional<z.ZodString>;
+        k8sContext: z.ZodOptional<z.ZodString>;
+    }, "strict", z.ZodTypeAny, {
+        runtime: "local" | "docker" | "k8s";
+        dockerNetwork?: string | undefined;
+        k8sNamespace?: string | undefined;
+        k8sContext?: string | undefined;
+        serverStartCommand?: string | undefined;
+    }, {
+        runtime: "local" | "docker" | "k8s";
+        dockerNetwork?: string | undefined;
+        k8sNamespace?: string | undefined;
+        k8sContext?: string | undefined;
+        serverStartCommand?: string | undefined;
+    }>>;
+}, "strict", z.ZodTypeAny, {
+    serviceName: string;
+    language?: "python" | "typescript" | "javascript" | "java" | undefined;
+    framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
+    repo?: string | undefined;
+    testDirectory?: string | undefined;
+    api?: {
+        authHeader?: string | undefined;
+        authScheme?: string | undefined;
+        schemaPath?: string | undefined;
+        authType?: "none" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "cookie" | "session" | undefined;
+        baseUrl?: string | undefined;
+    } | undefined;
+    runtimeDetails?: {
+        runtime: "local" | "docker" | "k8s";
+        dockerNetwork?: string | undefined;
+        k8sNamespace?: string | undefined;
+        k8sContext?: string | undefined;
+        serverStartCommand?: string | undefined;
+    } | undefined;
+}, {
+    serviceName: string;
+    language?: "python" | "typescript" | "javascript" | "java" | undefined;
+    framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
+    repo?: string | undefined;
+    testDirectory?: string | undefined;
+    api?: {
+        authHeader?: string | undefined;
+        authScheme?: string | undefined;
+        schemaPath?: string | undefined;
+        authType?: "none" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "cookie" | "session" | undefined;
+        baseUrl?: string | undefined;
+    } | undefined;
+    runtimeDetails?: {
+        runtime: "local" | "docker" | "k8s";
+        dockerNetwork?: string | undefined;
+        k8sNamespace?: string | undefined;
+        k8sContext?: string | undefined;
+        serverStartCommand?: string | undefined;
+    } | undefined;
+}>;
+export declare const workspaceConfigSchema: z.ZodObject<{
+    workspace: z.ZodOptional<z.ZodObject<{
+        repoName: z.ZodOptional<z.ZodString>;
+        repoUrl: z.ZodOptional<z.ZodString>;
+    }, "strict", z.ZodTypeAny, {
+        repoName?: string | undefined;
+        repoUrl?: string | undefined;
+    }, {
+        repoName?: string | undefined;
+        repoUrl?: string | undefined;
+    }>>;
+    metadata: z.ZodOptional<z.ZodObject<{
+        schemaVersion: z.ZodString;
+        mcpVersion: z.ZodString;
+        executorVersion: z.ZodString;
+        createdAt: z.ZodString;
+        updatedAt: z.ZodString;
+    }, "strict", z.ZodTypeAny, {
+        createdAt: string;
+        schemaVersion: string;
+        mcpVersion: string;
+        executorVersion: string;
+        updatedAt: string;
+    }, {
+        createdAt: string;
+        schemaVersion: string;
+        mcpVersion: string;
+        executorVersion: string;
+        updatedAt: string;
+    }>>;
+    services: z.ZodOptional<z.ZodArray<z.ZodObject<{
+        serviceName: z.ZodString;
+        repo: z.ZodOptional<z.ZodString>;
+        language: z.ZodOptional<z.ZodEnum<["python", "typescript", "javascript", "java"]>>;
+        framework: z.ZodOptional<z.ZodEnum<["playwright", "pytest", "robot", "junit"]>>;
+        testDirectory: z.ZodOptional<z.ZodString>;
+        api: z.ZodOptional<z.ZodObject<{
+            schemaPath: z.ZodOptional<z.ZodString>;
+            authType: z.ZodOptional<z.ZodEnum<["bearer", "basic", "oauth", "apiKey", "cookie", "session", "token", "none"]>>;
+            authHeader: z.ZodOptional<z.ZodString>;
+            authScheme: z.ZodOptional<z.ZodString>;
+            baseUrl: z.ZodOptional<z.ZodString>;
+        }, "strict", z.ZodTypeAny, {
+            authHeader?: string | undefined;
+            authScheme?: string | undefined;
+            schemaPath?: string | undefined;
+            authType?: "none" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "cookie" | "session" | undefined;
+            baseUrl?: string | undefined;
+        }, {
+            authHeader?: string | undefined;
+            authScheme?: string | undefined;
+            schemaPath?: string | undefined;
+            authType?: "none" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "cookie" | "session" | undefined;
+            baseUrl?: string | undefined;
+        }>>;
+        runtimeDetails: z.ZodOptional<z.ZodObject<{
+            serverStartCommand: z.ZodOptional<z.ZodString>;
+            runtime: z.ZodEnum<["local", "docker", "k8s"]>;
+            dockerNetwork: z.ZodOptional<z.ZodString>;
+            k8sNamespace: z.ZodOptional<z.ZodString>;
+            k8sContext: z.ZodOptional<z.ZodString>;
+        }, "strict", z.ZodTypeAny, {
+            runtime: "local" | "docker" | "k8s";
+            dockerNetwork?: string | undefined;
+            k8sNamespace?: string | undefined;
+            k8sContext?: string | undefined;
+            serverStartCommand?: string | undefined;
+        }, {
+            runtime: "local" | "docker" | "k8s";
+            dockerNetwork?: string | undefined;
+            k8sNamespace?: string | undefined;
+            k8sContext?: string | undefined;
+            serverStartCommand?: string | undefined;
+        }>>;
+    }, "strict", z.ZodTypeAny, {
+        serviceName: string;
+        language?: "python" | "typescript" | "javascript" | "java" | undefined;
+        framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
+        repo?: string | undefined;
+        testDirectory?: string | undefined;
+        api?: {
+            authHeader?: string | undefined;
+            authScheme?: string | undefined;
+            schemaPath?: string | undefined;
+            authType?: "none" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "cookie" | "session" | undefined;
+            baseUrl?: string | undefined;
+        } | undefined;
+        runtimeDetails?: {
+            runtime: "local" | "docker" | "k8s";
+            dockerNetwork?: string | undefined;
+            k8sNamespace?: string | undefined;
+            k8sContext?: string | undefined;
+            serverStartCommand?: string | undefined;
+        } | undefined;
+    }, {
+        serviceName: string;
+        language?: "python" | "typescript" | "javascript" | "java" | undefined;
+        framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
+        repo?: string | undefined;
+        testDirectory?: string | undefined;
+        api?: {
+            authHeader?: string | undefined;
+            authScheme?: string | undefined;
+            schemaPath?: string | undefined;
+            authType?: "none" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "cookie" | "session" | undefined;
+            baseUrl?: string | undefined;
+        } | undefined;
+        runtimeDetails?: {
+            runtime: "local" | "docker" | "k8s";
+            dockerNetwork?: string | undefined;
+            k8sNamespace?: string | undefined;
+            k8sContext?: string | undefined;
+            serverStartCommand?: string | undefined;
+        } | undefined;
+    }>, "many">>;
+}, "strict", z.ZodTypeAny, {
+    metadata?: {
+        createdAt: string;
+        schemaVersion: string;
+        mcpVersion: string;
+        executorVersion: string;
+        updatedAt: string;
+    } | undefined;
+    workspace?: {
+        repoName?: string | undefined;
+        repoUrl?: string | undefined;
+    } | undefined;
+    services?: {
+        serviceName: string;
+        language?: "python" | "typescript" | "javascript" | "java" | undefined;
+        framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
+        repo?: string | undefined;
+        testDirectory?: string | undefined;
+        api?: {
+            authHeader?: string | undefined;
+            authScheme?: string | undefined;
+            schemaPath?: string | undefined;
+            authType?: "none" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "cookie" | "session" | undefined;
+            baseUrl?: string | undefined;
+        } | undefined;
+        runtimeDetails?: {
+            runtime: "local" | "docker" | "k8s";
+            dockerNetwork?: string | undefined;
+            k8sNamespace?: string | undefined;
+            k8sContext?: string | undefined;
+            serverStartCommand?: string | undefined;
+        } | undefined;
+    }[] | undefined;
+}, {
+    metadata?: {
+        createdAt: string;
+        schemaVersion: string;
+        mcpVersion: string;
+        executorVersion: string;
+        updatedAt: string;
+    } | undefined;
+    workspace?: {
+        repoName?: string | undefined;
+        repoUrl?: string | undefined;
+    } | undefined;
+    services?: {
+        serviceName: string;
+        language?: "python" | "typescript" | "javascript" | "java" | undefined;
+        framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
+        repo?: string | undefined;
+        testDirectory?: string | undefined;
+        api?: {
+            authHeader?: string | undefined;
+            authScheme?: string | undefined;
+            schemaPath?: string | undefined;
+            authType?: "none" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "cookie" | "session" | undefined;
+            baseUrl?: string | undefined;
+        } | undefined;
+        runtimeDetails?: {
+            runtime: "local" | "docker" | "k8s";
+            dockerNetwork?: string | undefined;
+            k8sNamespace?: string | undefined;
+            k8sContext?: string | undefined;
+            serverStartCommand?: string | undefined;
+        } | undefined;
+    }[] | undefined;
+}>;
+export type Service = z.infer<typeof serviceSchema>;
+export type ServiceApi = NonNullable<Service["api"]>;
+export type ServiceRuntimeDetails = NonNullable<Service["runtimeDetails"]>;
+export type WorkspaceConfig = z.infer<typeof workspaceConfigSchema>;
+export type WorkspaceSection = NonNullable<WorkspaceConfig["workspace"]>;
+export type MetadataSection = NonNullable<WorkspaceConfig["metadata"]>;
+/** Validates a workspace configuration object against the Zod schema. */
+export declare function validateWorkspaceConfig(config: unknown): z.SafeParseReturnType<{
+    metadata?: {
+        createdAt: string;
+        schemaVersion: string;
+        mcpVersion: string;
+        executorVersion: string;
+        updatedAt: string;
+    } | undefined;
+    workspace?: {
+        repoName?: string | undefined;
+        repoUrl?: string | undefined;
+    } | undefined;
+    services?: {
+        serviceName: string;
+        language?: "python" | "typescript" | "javascript" | "java" | undefined;
+        framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
+        repo?: string | undefined;
+        testDirectory?: string | undefined;
+        api?: {
+            authHeader?: string | undefined;
+            authScheme?: string | undefined;
+            schemaPath?: string | undefined;
+            authType?: "none" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "cookie" | "session" | undefined;
+            baseUrl?: string | undefined;
+        } | undefined;
+        runtimeDetails?: {
+            runtime: "local" | "docker" | "k8s";
+            dockerNetwork?: string | undefined;
+            k8sNamespace?: string | undefined;
+            k8sContext?: string | undefined;
+            serverStartCommand?: string | undefined;
+        } | undefined;
+    }[] | undefined;
+}, {
+    metadata?: {
+        createdAt: string;
+        schemaVersion: string;
+        mcpVersion: string;
+        executorVersion: string;
+        updatedAt: string;
+    } | undefined;
+    workspace?: {
+        repoName?: string | undefined;
+        repoUrl?: string | undefined;
+    } | undefined;
+    services?: {
+        serviceName: string;
+        language?: "python" | "typescript" | "javascript" | "java" | undefined;
+        framework?: "playwright" | "pytest" | "robot" | "junit" | undefined;
+        repo?: string | undefined;
+        testDirectory?: string | undefined;
+        api?: {
+            authHeader?: string | undefined;
+            authScheme?: string | undefined;
+            schemaPath?: string | undefined;
+            authType?: "none" | "token" | "bearer" | "basic" | "oauth" | "apiKey" | "cookie" | "session" | undefined;
+            baseUrl?: string | undefined;
+        } | undefined;
+        runtimeDetails?: {
+            runtime: "local" | "docker" | "k8s";
+            dockerNetwork?: string | undefined;
+            k8sNamespace?: string | undefined;
+            k8sContext?: string | undefined;
+            serverStartCommand?: string | undefined;
+        } | undefined;
+    }[] | undefined;
+}>;
+/** Creates a default workspace config with empty sections and timestamps. */
+export declare function createDefaultConfig(): WorkspaceConfig;
+/**
+ * High-level manager for .skyramp/workspace.yml.
+ *
+ * Provides three focused mutation methods:
+ *   1. initialize()     — create workspace file with repo info
+ *   2. updateMetadata() — update metadata section
+ *   3. addService()     — upsert a single service entry
+ */
+export declare class WorkspaceConfigManager {
+    private workspacePath;
+    private skyrampDir;
+    private configPath;
+    constructor(workspacePath: string);
+    /** Check if workspace config file exists */
+    exists(): Promise<boolean>;
+    /** Get the absolute path to the config file */
+    getConfigPath(): string;
+    /** Get the workspace root path */
+    getWorkspacePath(): string;
+    /** Read and parse workspace config from disk. */
+    read(): Promise<WorkspaceConfig>;
+    /**
+     * Initialize workspace — creates .skyramp/workspace.yml with repo info.
+     * Auto-detects repoName and repoUrl from git when not provided.
+     * Produces an empty services array and default metadata timestamps.
+     */
+    initialize(workspaceInfo?: WorkspaceSection): Promise<WorkspaceConfig>;
+    /**
+     * Update the metadata section of an existing workspace config.
+     * Only the provided fields are merged; metadata.updatedAt is refreshed
+     * automatically.
+     */
+    updateMetadata(metadata: Partial<MetadataSection>): Promise<WorkspaceConfig>;
+    /**
+     * Add a single service entry to the services array.
+     * If a service with the same identity already exists it is replaced
+     * (upsert semantics), otherwise the new entry is appended. Service identity
+     * is the composite (repo, serviceName) — a service's `repo` (absent → primary)
+     * is part of its key, so the same serviceName can coexist across repos in a
+     * unified multi-repo workspace.
+     * metadata.updatedAt is refreshed automatically.
+     */
+    addService(service: Service): Promise<WorkspaceConfig>;
+}

package/build/workspace/workspace.test.d.ts

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

package/node_modules/playwright/lib/mcp/skyramp/traceRecordingBackend.js

@@ -1235,6 +1235,14 @@ Cell row ${params.row}, column ${params.column} has "${actual}", expected "${par
         locator: { kind: "text", body: text, options: { exact: false } }
       };
     }
+    const colonTextMatch = refLine.match(/\[ref=\w+\](?:\s*\[[^\]]*\])*\s*:\s*(.+?)\s*$/);
+    if (colonTextMatch && colonTextMatch[1].trim()) {
+      const text = colonTextMatch[1].trim();
+      return {
+        selector: `internal:text="${text}"i`,
+        locator: { kind: "text", body: text, options: { exact: false } }
+      };
+    }
     return null;
   }
   /**
@@ -1671,8 +1679,12 @@ Cell row ${params.row}, column ${params.column} has "${actual}", expected "${par
       }
     }
     if (!isFlutter) return;
-    const placeholder = page.locator("flt-semantics-placeholder").first();
-    const clicked = await placeholder.click({ force: true }).then(() => true).catch(() => false);
+    const clicked = await page.evaluate(() => {
+      const el = document.querySelector("flt-semantics-placeholder");
+      if (!el) return false;
+      el.dispatchEvent(new MouseEvent("click", { isTrusted: true, bubbles: true, cancelable: true }));
+      return true;
+    }).catch(() => false);
     if (!clicked) {
       traceDebug("Flutter detected but placeholder click failed");
       return;

package/package.json

@@ -1,11 +1,15 @@
 {
   "name": "@skyramp/mcp",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "main": "build/index.js",
   "exports": {
     ".": "./build/index.js",
     "./tool-phases": "./build/tool-phases.js",
     "./personas": "./build/prompts/personas.js",
+    "./workspace": {
+      "types": "./build/workspace/index.d.ts",
+      "default": "./build/workspace/index.js"
+    },
     "./build/*": "./build/*"
   },
   "type": "module",
@@ -22,10 +26,11 @@
     "pretty": "npx prettier --write .",
     "lint": "eslint src --ext .ts",
     "lint:fix": "eslint src --ext .ts --fix",
-    "test": "npx jest"
+    "test": "node --experimental-vm-modules node_modules/.bin/jest"
   },
   "files": [
     "build/**/*.js",
+    "build/**/*.d.ts",
     "README.md"
   ],
   "bundleDependencies": [
@@ -55,7 +60,7 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.24.3",
     "@playwright/test": "^1.55.0",
-    "@skyramp/skyramp": "1.3.27",
+    "@skyramp/skyramp": "1.3.28",
     "dockerode": "^5.0.0",
     "fast-glob": "^3.3.3",
     "js-yaml": "^4.1.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>`);
-    });
-}