@skyramp/mcp 0.3.0 → 0.3.1-ldw-beta

package/build/tools/test-management/analyzeChangesTool.d.ts

@@ -16,11 +16,12 @@ export declare const analyzeChangesInputSchema: {
     scope: z.ZodOptional<z.ZodDefault<z.ZodEnum<["full_repo", "branch_diff"]>>>;
     baseBranch: z.ZodOptional<z.ZodString>;
     testDirectory: z.ZodOptional<z.ZodString>;
-    topN: z.ZodOptional<z.ZodDefault<z.ZodNumber>>;
+    topN: z.ZodDefault<z.ZodOptional<z.ZodNumber>>;
     maxGenerate: z.ZodOptional<z.ZodNumber>;
     prNumber: z.ZodOptional<z.ZodNumber>;
     repository: z.ZodOptional<z.ZodString>;
     testsRepoDir: z.ZodOptional<z.ZodEffects<z.ZodString, string, string>>;
+    includeUncommitted: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
 };
 export declare const NO_UI_INSTRUCTIONS = "No UI changes detected \u2014 no blueprint capture needed.";
 export declare const NO_RESOLVABLE_URLS_INSTRUCTIONS = "Frontend changes detected but no candidate URLs could be resolved (workspace baseUrl missing or no router files matched). UI recommendations will be source-grounded only.";

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

@@ -283,8 +283,8 @@ export const analyzeChangesInputSchema = {
         .describe("Directory containing existing tests (auto-detected if omitted)"),
     topN: z
         .number()
-        .default(MAX_RECOMMENDATIONS)
         .optional()
+        .default(MAX_RECOMMENDATIONS)
         .describe(`Number of ranked test recommendations to generate. Defaults to ${MAX_RECOMMENDATIONS}.`),
     maxGenerate: z
         .number()
@@ -305,6 +305,11 @@ export const analyzeChangesInputSchema = {
         .refine((v) => path.isAbsolute(v), { message: "testsRepoDir must be an absolute path" })
         .optional()
         .describe("Absolute path to a separate test repository clone. When set, existing test discovery scans this directory instead of repositoryPath. Used in cross-repo test delivery mode where tests live in a separate repo."),
+    includeUncommitted: z
+        .boolean()
+        .optional()
+        .default(false)
+        .describe("When true, diffs the base ref against the working tree (captures uncommitted and unstaged changes). Use for local-dev workflows. Defaults to false (CI mode — committed changes only)."),
 };
 // ── UI blueprint-capture instructions ──
 // Moved here from the former skyramp_ui_analyze_changes pre-flight tool. These
@@ -387,7 +392,7 @@ export function registerAnalyzeChangesTool(server) {
             if (analysisScope === AnalysisScope.CurrentBranchDiff) {
                 await sendProgress(10, 100, "Computing branch diff...");
                 try {
-                    diffData = await computeBranchDiff(params.repositoryPath, params.baseBranch);
+                    diffData = await computeBranchDiff(params.repositoryPath, params.baseBranch, params.includeUncommitted ?? false);
                     logger.info("Branch diff computed", {
                         currentBranch: diffData.currentBranch,
                         baseBranch: diffData.baseBranch,
@@ -1177,6 +1182,56 @@ export function registerAnalyzeChangesTool(server) {
                 affectedServices: classifiedEndpoints.affectedServices,
                 summary: "",
             } : undefined;
+            // ── Route discovery context for LLM grounding and state persistence ──
+            // fullAnalysis lives only in inMemorySessionStore (for MCP resources
+            // and registerRecommendTestsPrompt). The disk state carries only the
+            // slim fields that downstream tools (health, execute, actions) need.
+            // routerMountContext and candidateRouteFiles are computed here so they
+            // can be persisted to the state file for downstream tools (health, drift).
+            // Without them, analyzeTestHealth would work only off the static catalog
+            // which has wrong paths for nested resources and unsupported frameworks.
+            const routeLikeUnmatchedFiles = [];
+            for (const file of classifiedEndpoints?.unmatchedFiles ?? []) {
+                const routeLike = SOURCE_EXTS.test(file) &&
+                    (ROUTE_FILE_PATTERN.test(file) || ROUTE_FILE_BASENAME_PATTERN.test(path.basename(file)));
+                if (routeLike && !(await isGraphQLFile(file, params.repositoryPath))) {
+                    routeLikeUnmatchedFiles.push(file);
+                }
+            }
+            const shouldIncludeCandidateRouteFiles = analysisScope !== AnalysisScope.CurrentBranchDiff ||
+                rawRelatedEndpointCount === 0 ||
+                scannedEndpoints.length === 0 ||
+                routeLikeUnmatchedFiles.length > 0;
+            let candidateRouteFiles;
+            if (shouldIncludeCandidateRouteFiles) {
+                candidateRouteFiles = [];
+                for (const file of findCandidateRouteFiles(params.repositoryPath)) {
+                    if (!(await isGraphQLFile(file, params.repositoryPath))) {
+                        candidateRouteFiles.push(file);
+                    }
+                }
+            }
+            // Write the full diff to a temp file before building state so the path
+            // can be persisted and read by analyzeTestHealthTool for per-line detection.
+            let diffFilePath;
+            if (diffData?.diffContent) {
+                diffFilePath = path.join(os.tmpdir(), `skyramp-diff-${sessionId}.diff`);
+                await fs.promises.writeFile(diffFilePath, diffData.diffContent, { encoding: "utf-8", mode: 0o600 });
+            }
+            const routeDiscovery = {
+                candidateFiles: [
+                    ...(diffData?.changedFiles ?? []),
+                    ...(candidateRouteFiles ?? []),
+                ].filter((file, index, files) => files.indexOf(file) === index),
+                staticHints: scannedEndpoints.map((ep) => ({
+                    path: ep.path,
+                    methods: ep.methods,
+                    sourceFile: ep.sourceFile,
+                })),
+                openApiPaths: specPaths ? [...specPaths] : [],
+                routerMountContext,
+                ...(diffFilePath ? { diffFilePath } : {}),
+            };
             const fullAnalysis = {
                 metadata: {
                     repositoryName: path.basename(params.repositoryPath),
@@ -1233,6 +1288,7 @@ export function registerAnalyzeChangesTool(server) {
                     hasCoverageReports: false,
                     relevantExternalTestPaths,
                 },
+                routeDiscovery,
                 ...(diffContext ? { branchDiffContext: diffContext } : {}),
             };
             // Store RecommendationState in memory so it's compatible with skyramp_recommend_tests if needed
@@ -1242,42 +1298,6 @@ export function registerAnalyzeChangesTool(server) {
                 analysis: fullAnalysis,
             };
             storeSessionData(sessionId, recommendationState);
-            // ── Step 11: Build UnifiedAnalysisState and save ──
-            // fullAnalysis lives only in inMemorySessionStore (for MCP resources
-            // and registerRecommendTestsPrompt). The disk state carries only the
-            // slim fields that downstream tools (health, execute, actions) need.
-            // routerMountContext and candidateRouteFiles are computed here so they
-            // can be persisted to the state file for downstream tools (health, drift).
-            // Without them, analyzeTestHealth would work only off the static catalog
-            // which has wrong paths for nested resources and unsupported frameworks.
-            const routeLikeUnmatchedFiles = [];
-            for (const file of classifiedEndpoints?.unmatchedFiles ?? []) {
-                const routeLike = SOURCE_EXTS.test(file) &&
-                    (ROUTE_FILE_PATTERN.test(file) || ROUTE_FILE_BASENAME_PATTERN.test(path.basename(file)));
-                if (routeLike && !(await isGraphQLFile(file, params.repositoryPath))) {
-                    routeLikeUnmatchedFiles.push(file);
-                }
-            }
-            const shouldIncludeCandidateRouteFiles = analysisScope !== AnalysisScope.CurrentBranchDiff ||
-                rawRelatedEndpointCount === 0 ||
-                scannedEndpoints.length === 0 ||
-                routeLikeUnmatchedFiles.length > 0;
-            let candidateRouteFiles;
-            if (shouldIncludeCandidateRouteFiles) {
-                candidateRouteFiles = [];
-                for (const file of findCandidateRouteFiles(params.repositoryPath)) {
-                    if (!(await isGraphQLFile(file, params.repositoryPath))) {
-                        candidateRouteFiles.push(file);
-                    }
-                }
-            }
-            // Write the full diff to a temp file before building state so the path
-            // can be persisted and read by analyzeTestHealthTool for per-line detection.
-            let diffFilePath;
-            if (diffData?.diffContent) {
-                diffFilePath = path.join(os.tmpdir(), `skyramp-diff-${sessionId}.diff`);
-                await fs.promises.writeFile(diffFilePath, diffData.diffContent, { encoding: "utf-8", mode: 0o600 });
-            }
             // Compute UI context from the diff's changed files using the shared
             // `isFrontendFile` classifier. Persisting this in the stateFile lets
             // skyramp_analyze_test_health and the recommendation prompt consume the
@@ -1376,6 +1396,7 @@ export function registerAnalyzeChangesTool(server) {
                     sessionId,
                     routerMountContext,
                     candidateRouteFiles,
+                    routeDiscovery,
                     relevantExternalTestPaths,
                 },
             };
@@ -1473,6 +1494,8 @@ export function registerAnalyzeChangesTool(server) {
                     modifiedEndpointCount: classifiedEndpoints?.changedEndpoints.length ?? 0,
                     removedEndpointCount: classifiedEndpoints?.removedEndpoints.length ?? 0,
                     endpointCount: skeletonEndpoints.reduce((acc, ep) => acc + ep.methods.length, 0),
+                    newEndpoints: (classifiedEndpoints?.newEndpoints ?? []).flatMap((ep) => ep.methods.map((m) => `${m} ${ep.path}`)),
+                    modifiedEndpoints: (classifiedEndpoints?.changedEndpoints ?? []).flatMap((ep) => ep.methods.map((m) => `${m} ${ep.path}`)),
                 },
                 // Surface uiContext inline so the testbot prompt can iterate
                 // candidateUiPages and inspect changedFrontendFiles without

package/build/tools/trace/startTraceCollectionTool.js

@@ -5,7 +5,7 @@ import { SkyrampClient } from "@skyramp/skyramp";
 import openProxyTerminalTracked from "../../utils/proxy-terminal.js";
 import { getEntryPoint } from "../../utils/telemetry.js";
 import { logger } from "../../utils/logger.js";
-import { basePlaywrightSchema, baseSchema, SESSION_STORAGE_FILENAME, } from "../../types/TestTypes.js";
+import { basePlaywrightSchema, baseSchema, RuntimeEnvironment, SESSION_STORAGE_FILENAME, } from "../../types/TestTypes.js";
 import { AnalyticsService } from "../../services/AnalyticsService.js";
 import { resolveSessionPaths } from "./resolveSessionPaths.js";
 import { setSavedSessionPath } from "./sessionState.js";
@@ -67,8 +67,8 @@ For detailed documentation visit: https://www.skyramp.dev/docs/load-test/advance
             playwrightSaveStoragePath: basePlaywrightSchema.shape.playwrightSaveStoragePath,
             playwrightViewportSize: basePlaywrightSchema.shape.playwrightViewportSize,
             runtime: z
-                .string()
-                .default("docker")
+                .literal(RuntimeEnvironment.DOCKER)
+                .default(RuntimeEnvironment.DOCKER)
                 .describe("Runtime environment for trace collection. Currently only 'docker' is supported and is used as the default."),
             include: z
                 .array(z.string())

package/build/types/OneClickCommands.d.ts

@@ -4,7 +4,7 @@
  * and executed by the coding assistant following the workflow instructions.
  */
 /** Unique identifier for a one-click command */
-export type OneClickCommandId = "test_given_endpoint_comprehensively" | "full_repo_scan_recommend_generate_and_execute_top_n_tests";
+export type OneClickCommandId = "test_given_endpoint_comprehensively" | "full_repo_scan_recommend_generate_and_execute_top_n_tests" | "local_dev_test_changes";
 /** Metadata for intent recognition by coding assistants */
 export interface CommandIntentMetadata {
     /** Context indicators describing when to use and when NOT to use this workflow */

package/build/types/RepositoryAnalysis.d.ts

@@ -145,6 +145,22 @@ export interface BranchDiffContext {
     affectedServices: string[];
     summary?: string;
 }
+export interface RouteDiscoveryInfo {
+    /** Files the LLM should inspect to infer endpoint methods and paths from source. */
+    candidateFiles: string[];
+    /** Static scanner output. Best-effort hints only, not an authoritative catalog. */
+    staticHints: Array<{
+        path: string;
+        methods: string[];
+        sourceFile: string;
+    }>;
+    /** OpenAPI paths from workspace config. Authoritative when specFetchSucceeded is true. */
+    openApiPaths: string[];
+    /** Router/module entry points that help the LLM reconstruct full mounted paths. */
+    routerMountContext: string[];
+    /** Full diff file path when available; read this to ground endpoint extraction in changed code. */
+    diffFilePath?: string;
+}
 export interface RepositoryAnalysis {
     metadata: AnalysisMetadata;
     projectClassification: ProjectClassification;
@@ -155,6 +171,7 @@ export interface RepositoryAnalysis {
     authentication: AuthenticationInfo;
     infrastructure: InfrastructureInfo;
     existingTests: ExistingTestInfo;
+    routeDiscovery?: RouteDiscoveryInfo;
     branchDiffContext?: BranchDiffContext;
 }
 export interface AnalysisMetadata {
@@ -1423,6 +1440,45 @@ export declare const branchDiffContextSchema: z.ZodObject<{
     }[] | undefined;
     summary?: string | undefined;
 }>;
+export declare const routeDiscoveryInfoSchema: z.ZodObject<{
+    candidateFiles: z.ZodArray<z.ZodString, "many">;
+    staticHints: z.ZodArray<z.ZodObject<{
+        path: z.ZodString;
+        methods: z.ZodArray<z.ZodString, "many">;
+        sourceFile: z.ZodString;
+    }, "strip", z.ZodTypeAny, {
+        path: string;
+        sourceFile: string;
+        methods: string[];
+    }, {
+        path: string;
+        sourceFile: string;
+        methods: string[];
+    }>, "many">;
+    openApiPaths: z.ZodArray<z.ZodString, "many">;
+    routerMountContext: z.ZodArray<z.ZodString, "many">;
+    diffFilePath: z.ZodOptional<z.ZodString>;
+}, "strip", z.ZodTypeAny, {
+    candidateFiles: string[];
+    staticHints: {
+        path: string;
+        sourceFile: string;
+        methods: string[];
+    }[];
+    openApiPaths: string[];
+    routerMountContext: string[];
+    diffFilePath?: string | undefined;
+}, {
+    candidateFiles: string[];
+    staticHints: {
+        path: string;
+        sourceFile: string;
+        methods: string[];
+    }[];
+    openApiPaths: string[];
+    routerMountContext: string[];
+    diffFilePath?: string | undefined;
+}>;
 export declare const analysisMetadataSchema: z.ZodObject<{
     repositoryName: z.ZodString;
     analysisDate: z.ZodString;
@@ -2440,6 +2496,45 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
         estimatedCoverage?: number | undefined;
         relevantExternalTestPaths?: string[] | undefined;
     }>;
+    routeDiscovery: z.ZodOptional<z.ZodObject<{
+        candidateFiles: z.ZodArray<z.ZodString, "many">;
+        staticHints: z.ZodArray<z.ZodObject<{
+            path: z.ZodString;
+            methods: z.ZodArray<z.ZodString, "many">;
+            sourceFile: z.ZodString;
+        }, "strip", z.ZodTypeAny, {
+            path: string;
+            sourceFile: string;
+            methods: string[];
+        }, {
+            path: string;
+            sourceFile: string;
+            methods: string[];
+        }>, "many">;
+        openApiPaths: z.ZodArray<z.ZodString, "many">;
+        routerMountContext: z.ZodArray<z.ZodString, "many">;
+        diffFilePath: z.ZodOptional<z.ZodString>;
+    }, "strip", z.ZodTypeAny, {
+        candidateFiles: string[];
+        staticHints: {
+            path: string;
+            sourceFile: string;
+            methods: string[];
+        }[];
+        openApiPaths: string[];
+        routerMountContext: string[];
+        diffFilePath?: string | undefined;
+    }, {
+        candidateFiles: string[];
+        staticHints: {
+            path: string;
+            sourceFile: string;
+            methods: string[];
+        }[];
+        openApiPaths: string[];
+        routerMountContext: string[];
+        diffFilePath?: string | undefined;
+    }>>;
     branchDiffContext: z.ZodOptional<z.ZodObject<{
         currentBranch: z.ZodString;
         baseBranch: z.ZodString;
@@ -2768,6 +2863,17 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
         estimatedCoverage?: number | undefined;
         relevantExternalTestPaths?: string[] | undefined;
     };
+    routeDiscovery?: {
+        candidateFiles: string[];
+        staticHints: {
+            path: string;
+            sourceFile: string;
+            methods: string[];
+        }[];
+        openApiPaths: string[];
+        routerMountContext: string[];
+        diffFilePath?: string | undefined;
+    } | undefined;
     branchDiffContext?: {
         currentBranch: string;
         baseBranch: string;
@@ -2970,6 +3076,17 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
         estimatedCoverage?: number | undefined;
         relevantExternalTestPaths?: string[] | undefined;
     };
+    routeDiscovery?: {
+        candidateFiles: string[];
+        staticHints: {
+            path: string;
+            sourceFile: string;
+            methods: string[];
+        }[];
+        openApiPaths: string[];
+        routerMountContext: string[];
+        diffFilePath?: string | undefined;
+    } | undefined;
     branchDiffContext?: {
         currentBranch: string;
         baseBranch: string;

package/build/types/RepositoryAnalysis.js

@@ -128,17 +128,30 @@ export const branchDiffContextSchema = z.object({
             changeType: z.enum(["added", "modified", "removed"]),
         })),
     })),
-    removedEndpoints: z.array(z.object({
+    removedEndpoints: z
+        .array(z.object({
         path: z.string(),
         methods: z.array(z.object({
             method: z.string(),
             sourceFile: z.string(),
             changeType: z.literal("removed"),
         })),
-    })).optional(),
+    }))
+        .optional(),
     affectedServices: z.array(z.string()),
     summary: z.string().optional(),
 });
+export const routeDiscoveryInfoSchema = z.object({
+    candidateFiles: z.array(z.string()),
+    staticHints: z.array(z.object({
+        path: z.string(),
+        methods: z.array(z.string()),
+        sourceFile: z.string(),
+    })),
+    openApiPaths: z.array(z.string()),
+    routerMountContext: z.array(z.string()),
+    diffFilePath: z.string().optional(),
+});
 export const analysisMetadataSchema = z.object({
     repositoryName: z.string(),
     analysisDate: z.string(),
@@ -239,5 +252,6 @@ export const repositoryAnalysisSchema = z.object({
         estimatedCoverage: z.number().optional(),
         relevantExternalTestPaths: z.array(z.string()).optional(),
     }),
+    routeDiscovery: routeDiscoveryInfoSchema.optional(),
     branchDiffContext: branchDiffContextSchema.optional(),
 });

package/build/types/TestExecution.d.ts

@@ -42,6 +42,7 @@ export interface TestExecutionOptions {
     token?: string;
     timeout?: number;
     playwrightSaveStoragePath?: string;
+    dockerNetwork?: string;
     useHostNetwork?: boolean;
 }
 /**

package/build/types/TestTypes.d.ts

@@ -15,10 +15,19 @@ export declare enum TestType {
     INTEGRATION = "integration",
     E2E = "e2e",
     UI = "ui",
-    MOCK = "mock",
     UNIT = "unit",
     UNKNOWN = "unknown"
 }
+/**
+ * Mock is not a test type — mock files are deployed via apply_mock(),
+ * not executed as tests. Kept as a standalone constant for mock generation.
+ */
+export declare const MOCK_TYPE: "mock";
+export declare enum MockProtocol {
+    Rest = "rest",
+    Grpc = "grpc",
+    Kafka = "kafka"
+}
 export declare enum HttpMethod {
     GET = "GET",
     POST = "POST",
@@ -31,7 +40,16 @@ export declare enum HttpMethod {
 export declare enum RuntimeEnvironment {
     LOCAL = "local",
     DOCKER = "docker",
-    KUBERNETES = "kubernetes"
+    KUBERNETES = "k8s"
+}
+export declare const runtimeEnvironmentSchema: z.ZodNativeEnum<typeof RuntimeEnvironment>;
+export declare const runtimeEnvironmentInputSchema: z.ZodEffects<z.ZodNativeEnum<typeof RuntimeEnvironment>, RuntimeEnvironment, unknown>;
+export declare function normalizeRuntimeEnvironment(value: RuntimeEnvironment | "kubernetes"): RuntimeEnvironment;
+export interface TestConfig {
+    testType: TestType;
+    toolName: string;
+    description: string;
+    additionalFields?: Record<string, z.ZodSchema>;
 }
 export declare const languageSchema: z.ZodObject<{
     language: z.ZodNativeEnum<typeof ProgrammingLanguage>;
@@ -45,7 +63,7 @@ export declare const languageSchema: z.ZodObject<{
 }>;
 export declare const baseSchema: z.ZodObject<{
     deployDashboard: z.ZodDefault<z.ZodBoolean>;
-    runtime: z.ZodDefault<z.ZodNativeEnum<typeof RuntimeEnvironment>>;
+    runtime: z.ZodDefault<z.ZodEffects<z.ZodNativeEnum<typeof RuntimeEnvironment>, RuntimeEnvironment, unknown>>;
     dockerNetwork: z.ZodDefault<z.ZodString>;
     dockerWorkerPort: z.ZodDefault<z.ZodNumber>;
     k8sNamespace: z.ZodDefault<z.ZodString>;
@@ -87,7 +105,7 @@ export declare const baseSchema: z.ZodObject<{
     outputDir: string;
     prompt: string;
     deployDashboard?: boolean | undefined;
-    runtime?: RuntimeEnvironment | undefined;
+    runtime?: unknown;
     dockerNetwork?: string | undefined;
     dockerWorkerPort?: number | undefined;
     k8sNamespace?: string | undefined;
@@ -125,7 +143,7 @@ export declare const basePlaywrightSchema: z.ZodObject<{
 }>;
 export declare const baseTraceSchema: z.ZodObject<{
     deployDashboard: z.ZodDefault<z.ZodBoolean>;
-    runtime: z.ZodDefault<z.ZodNativeEnum<typeof RuntimeEnvironment>>;
+    runtime: z.ZodDefault<z.ZodEffects<z.ZodNativeEnum<typeof RuntimeEnvironment>, RuntimeEnvironment, unknown>>;
     dockerNetwork: z.ZodDefault<z.ZodString>;
     dockerWorkerPort: z.ZodDefault<z.ZodNumber>;
     k8sNamespace: z.ZodDefault<z.ZodString>;
@@ -174,7 +192,7 @@ export declare const baseTraceSchema: z.ZodObject<{
     prompt: string;
     trace: string;
     deployDashboard?: boolean | undefined;
-    runtime?: RuntimeEnvironment | undefined;
+    runtime?: unknown;
     dockerNetwork?: string | undefined;
     dockerWorkerPort?: number | undefined;
     k8sNamespace?: string | undefined;
@@ -192,7 +210,7 @@ export declare const baseTraceSchema: z.ZodObject<{
 }>;
 export declare const baseTestSchema: {
     deployDashboard: z.ZodDefault<z.ZodBoolean>;
-    runtime: z.ZodDefault<z.ZodNativeEnum<typeof RuntimeEnvironment>>;
+    runtime: z.ZodDefault<z.ZodEffects<z.ZodNativeEnum<typeof RuntimeEnvironment>, RuntimeEnvironment, unknown>>;
     dockerNetwork: z.ZodDefault<z.ZodString>;
     dockerWorkerPort: z.ZodDefault<z.ZodNumber>;
     k8sNamespace: z.ZodDefault<z.ZodString>;

package/build/types/TestTypes.js

@@ -17,10 +17,20 @@ export var TestType;
     TestType["INTEGRATION"] = "integration";
     TestType["E2E"] = "e2e";
     TestType["UI"] = "ui";
-    TestType["MOCK"] = "mock";
     TestType["UNIT"] = "unit";
     TestType["UNKNOWN"] = "unknown";
 })(TestType || (TestType = {}));
+/**
+ * Mock is not a test type — mock files are deployed via apply_mock(),
+ * not executed as tests. Kept as a standalone constant for mock generation.
+ */
+export const MOCK_TYPE = "mock";
+export var MockProtocol;
+(function (MockProtocol) {
+    MockProtocol["Rest"] = "rest";
+    MockProtocol["Grpc"] = "grpc";
+    MockProtocol["Kafka"] = "kafka";
+})(MockProtocol || (MockProtocol = {}));
 export var HttpMethod;
 (function (HttpMethod) {
     HttpMethod["GET"] = "GET";
@@ -35,8 +45,13 @@ export var RuntimeEnvironment;
 (function (RuntimeEnvironment) {
     RuntimeEnvironment["LOCAL"] = "local";
     RuntimeEnvironment["DOCKER"] = "docker";
-    RuntimeEnvironment["KUBERNETES"] = "kubernetes";
+    RuntimeEnvironment["KUBERNETES"] = "k8s";
 })(RuntimeEnvironment || (RuntimeEnvironment = {}));
+export const runtimeEnvironmentSchema = z.nativeEnum(RuntimeEnvironment);
+export const runtimeEnvironmentInputSchema = z.preprocess((value) => (value === "kubernetes" ? RuntimeEnvironment.KUBERNETES : value), runtimeEnvironmentSchema);
+export function normalizeRuntimeEnvironment(value) {
+    return value === "kubernetes" ? RuntimeEnvironment.KUBERNETES : value;
+}
 export const languageSchema = z.object({
     language: z
         .nativeEnum(ProgrammingLanguage)
@@ -51,10 +66,9 @@ export const baseSchema = z.object({
         .boolean()
         .default(false)
         .describe("Whether to deploy the Skyramp dashboard for test monitoring"),
-    runtime: z
-        .nativeEnum(RuntimeEnvironment)
+    runtime: runtimeEnvironmentInputSchema
         .default(RuntimeEnvironment.LOCAL)
-        .describe("Runtime environment (docker, kubernetes, local)"),
+        .describe("Runtime environment (local, docker, or k8s; accepts kubernetes as an alias)"),
     dockerNetwork: z
         .string()
         .default("")

package/build/utils/analyze-openapi.js

@@ -1,5 +1,10 @@
 import { SkyrampClient } from "@skyramp/skyramp";
 import { IsValidKeyValueList } from "./utils.js";
+import { logger } from "./logger.js";
+function isOpenApiRuntimeFailure(error) {
+    const message = error instanceof Error ? error.message : String(error);
+    return /(?:ECONNREFUSED|ECONNRESET|ENOTFOUND|EAI_AGAIN|ETIMEDOUT|socket hang up|connection refused|unavailable|timed? out)/i.test(message);
+}
 // Analyze OpenAPI to check for required path params
 export async function analyzeOpenAPIWithGivenEndpoint(apiSchemaInput, uriInput, pathParamsInput) {
     const client = new SkyrampClient();
@@ -7,7 +12,19 @@ export async function analyzeOpenAPIWithGivenEndpoint(apiSchemaInput, uriInput,
         apiSchema: apiSchemaInput,
         uri: uriInput,
     };
-    const pathParamRequired = await client.analyzeOpenapi(analyzeOpenapiOptions);
+    let pathParamRequired;
+    try {
+        pathParamRequired = await client.analyzeOpenapi(analyzeOpenapiOptions);
+    }
+    catch (e) {
+        if (isOpenApiRuntimeFailure(e)) {
+            logger.warning("Skipping OpenAPI path-param validation because schema analysis is unavailable", {
+                error: e instanceof Error ? e.message : String(e),
+            });
+            return null;
+        }
+        throw e;
+    }
     if (IsValidKeyValueList(pathParamRequired)) {
         const requiredPathParams = pathParamRequired
             ? pathParamRequired.split(",").map((p) => p.trim())

package/build/utils/analyze-openapi.test.d.ts

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

package/build/utils/analyze-openapi.test.js

@@ -0,0 +1,19 @@
+import { jest, describe, it, expect, beforeEach } from "@jest/globals";
+const analyzeOpenapi = jest.fn();
+jest.unstable_mockModule("@skyramp/skyramp", () => ({
+    SkyrampClient: jest.fn(() => ({ analyzeOpenapi })),
+}));
+const { analyzeOpenAPIWithGivenEndpoint } = await import("./analyze-openapi.js");
+describe("analyzeOpenAPIWithGivenEndpoint", () => {
+    beforeEach(() => {
+        analyzeOpenapi.mockReset();
+    });
+    it("skips path-param validation for runtime connectivity failures", async () => {
+        analyzeOpenapi.mockRejectedValue(new Error("gRPC unavailable: connection refused"));
+        await expect(analyzeOpenAPIWithGivenEndpoint("schema.yaml", "/users/{id}", "")).resolves.toBeNull();
+    });
+    it("rethrows schema errors instead of silently disabling validation", async () => {
+        analyzeOpenapi.mockRejectedValue(new Error("invalid apiSchema path: missing.yaml"));
+        await expect(analyzeOpenAPIWithGivenEndpoint("missing.yaml", "/users/{id}", "")).rejects.toThrow("invalid apiSchema path");
+    });
+});

package/build/utils/branchDiff.d.ts

@@ -15,4 +15,20 @@ export interface BranchDiffData {
  * Always uses the `b/` form so renames return the new path.
  */
 export declare function parseChangedFilesFromDiff(rawDiff: string): string[];
-export declare function computeBranchDiff(repositoryPath: string, providedBaseBranch?: string): Promise<BranchDiffData>;
+/**
+ * Normalize and validate a caller-provided base branch/ref before passing it
+ * to git. Reject option-like or ambiguous values so the ref is always treated
+ * as a revision name, not as a command-line flag.
+ */
+export declare function normalizeProvidedBaseRef(providedBaseBranch?: string): string | undefined;
+/**
+ * Compute the diff between a base reference and the current state.
+ *
+ * @param repositoryPath  Absolute path to the git repository.
+ * @param providedBaseBranch  A branch name, tag, or commit SHA to diff against.
+ *   When omitted, the remote default branch is auto-detected.
+ * @param includeUncommitted  When true, diffs against the working tree so
+ *   uncommitted/unstaged changes are included. Defaults to false (CI mode
+ *   that only considers committed changes via the three-dot form).
+ */
+export declare function computeBranchDiff(repositoryPath: string, providedBaseBranch?: string, includeUncommitted?: boolean): Promise<BranchDiffData>;