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

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

@@ -8,7 +8,7 @@ import { toolError } from "../../utils/utils.js";
 import { buildApprovedPlanItem } from "../../utils/planMatchKeys.js";
 import { SCENARIO_CATEGORIES, CATEGORY_PRIORITY, Novelty, PriorityTier } from "../../types/TestRecommendation.js";
 import { HttpMethod, TestType } from "../../types/TestTypes.js";
-import { CandidateSource, computeCandidateId, DiscriminatorKind } from "../../types/Recommendation.js";
+import { CandidateSource, computeCandidateId, scenarioMergeKey, DiscriminatorKind } from "../../types/Recommendation.js";
 import { selectPlan } from "../../recommendation/planRanker.js";
 import { validateDiscriminator } from "../../recommendation/discriminators.js";
 import { isAttackSurfaceSecurityBoundary } from "../../prompts/test-recommendation/recommendationShared.js";
@@ -220,6 +220,39 @@ function renderPlanItem(item, rank) {
     const discriminatorNote = item.verifiedDiscriminator ? `, verified discriminator: ${item.verifiedDiscriminator}` : "";
     return `${rank}. [${item.testType}] "${item.scenarioName}" (${item.category}${discriminatorNote}) — ${describeGenerationCall(item)}`;
 }
+/**
+ * Whether this run generates UI/E2E tests is a deterministic property of the
+ * approved plan, so it is decided here rather than left to the agent to resolve
+ * from prompt prose. The prompt owns HOW to record a UI trace and the runtime
+ * conditions the server cannot know (app unreachable, unintegrated component);
+ * this directive owns WHETHER and HOW MANY.
+ */
+function renderGenerationDirective(plan) {
+    const uiCount = plan.generate.filter((item) => item.testType === TestType.UI || item.testType === TestType.E2E).length;
+    const nonUICount = plan.generate.length - uiCount;
+    const tests = (n) => `${n} test${n === 1 ? "" : "s"}`;
+    if (plan.generate.length === 0) {
+        return [
+            "### Generation directive: GENERATE NOTHING",
+            "Create zero new tests of any type this run and proceed to the report. An empty `newTestsCreated` is the CORRECT " +
+                "result for this PR — there is no new observable surface to cover. Do not record a browser trace and do not invent " +
+                "a spec to have something to report; report your maintenance work and the ADDITIONAL candidates as recommendations instead.",
+        ];
+    }
+    if (uiCount === 0) {
+        return [
+            "### Generation directive: NO UI/E2E GENERATION",
+            `This plan allocates no UI/E2E generation. Generate the ${tests(nonUICount)} listed above and create zero UI tests ` +
+                "— do not record a browser trace and do not add a UI spec to fill `newTestsCreated`.",
+        ];
+    }
+    return [
+        `### Generation directive: UI/E2E GENERATION REQUIRED (${uiCount})`,
+        `This plan allocates ${tests(uiCount)} of type UI/E2E. You MUST attempt to record and generate each one — do not ` +
+            "downgrade them to recommendations while the app is reachable." +
+            (nonUICount > 0 ? ` Generate the ${tests(nonUICount)} of other types as well.` : ""),
+    ];
+}
 function renderPlanText(plan) {
     const lines = [];
     lines.push(`## Approved Test Plan (${plan.planId})`);
@@ -249,6 +282,8 @@ function renderPlanText(plan) {
             lines.push(`- ${demotion.candidateId}: ${demotion.reason}`);
         }
     }
+    lines.push("");
+    lines.push(...renderGenerationDirective(plan));
     return lines.join("\n");
 }
 // ── Tool registration ───────────────────────────────────────────────────────
@@ -285,14 +320,27 @@ export function registerRegisterTestPlanTool(server) {
             const allScenarios = stateData.repositoryAnalysis?.scenarios ?? [];
             const { candidates: agentCandidates, demotions } = buildAgentCandidates(params.candidates ?? [], diffText);
             const serverCandidates = recoverServerCandidates(stateData.approvedPlan, allScenarios);
-            // Merge by candidateId; a pre-seeded server candidate wins over an agent
-            // submission that happens to hash to the same id (identical scenario
-            // content) — source: "server" is preserved per the unified-plan design.
+            // Merge by scenario-name identity, NOT the full content-hashed candidateId:
+            // the server (analyze_changes) and the agent frequently draft their own
+            // independent steps[] for "the same" scenario (identical scenarioName),
+            // which hash to different candidateIds — deduping on the full id let
+            // both survive as apparent duplicates (SKYR-4026). A pre-seeded server
+            // candidate wins over an agent submission for the same scenario name —
+            // source: "server" is preserved per the unified-plan design.
+            //
+            // Uses scenarioMergeKey (untruncated), not scenarioNameSlug — the
+            // latter's 48-char cap is fine for a short candidateId prefix combined
+            // with a content hash, but bare as a Map key it would silently collapse
+            // two distinct long scenario names sharing a common prefix. A missing
+            // scenarioName (schema requires one for agent candidates; only a
+            // malformed server-recovered scenario could lack one) falls back to
+            // the already-unique candidateId rather than a shared literal.
+            const mergeKey = (c) => scenarioMergeKey(c.scenario.scenarioName) || c.candidateId;
             const merged = new Map();
             for (const candidate of agentCandidates)
-                merged.set(candidate.candidateId, candidate);
+                merged.set(mergeKey(candidate), candidate);
             for (const candidate of serverCandidates)
-                merged.set(candidate.candidateId, candidate);
+                merged.set(mergeKey(candidate), candidate);
             const allCandidates = [...merged.values()];
             // An empty union would persist an authoritative plan with an empty
             // GENERATE list, which the generation gate then enforces — bricking
@@ -304,7 +352,7 @@ export function registerRegisterTestPlanTool(server) {
                 return errorResult;
             }
             const budgetContext = resolveBudgetContext(stateData);
-            const result = selectPlan(allCandidates, { ...budgetContext, demotions });
+            const result = selectPlan(allCandidates, { ...budgetContext, demotions, diffText });
             const approvedPlan = {
                 planId: crypto.randomUUID(),
                 createdAt: new Date().toISOString(),

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/FrontendIntegration.d.ts

@@ -9,6 +9,9 @@
 export declare enum IntegrationReason {
     /** Framework route/page/entrypoint file — reachable by convention, grep skipped. */
     RouteOrEntrypoint = "route-or-entrypoint",
+    /** i18n/locale content file — often loaded via a runtime-templated path rather
+     *  than a static quoted import, so the grep-based check is skipped (SKYR-3977). */
+    I18nLocaleContent = "i18n-locale-content",
     /** At least one production file imports/references the component's module path. */
     Imported = "imported",
     /** No production importer found — the component has no DOM presence in the running app. */

package/build/types/FrontendIntegration.js

@@ -10,6 +10,9 @@ export var IntegrationReason;
 (function (IntegrationReason) {
     /** Framework route/page/entrypoint file — reachable by convention, grep skipped. */
     IntegrationReason["RouteOrEntrypoint"] = "route-or-entrypoint";
+    /** i18n/locale content file — often loaded via a runtime-templated path rather
+     *  than a static quoted import, so the grep-based check is skipped (SKYR-3977). */
+    IntegrationReason["I18nLocaleContent"] = "i18n-locale-content";
     /** At least one production file imports/references the component's module path. */
     IntegrationReason["Imported"] = "imported";
     /** No production importer found — the component has no DOM presence in the running app. */

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/Recommendation.d.ts

@@ -74,6 +74,26 @@ export interface Budgeter {
     readonly name: string;
     select(ranked: Candidate[], ctx: BudgetContext): SelectionResult;
 }
+/**
+ * Slug of a scenario name alone (no content hash) — stable across independent
+ * drafts of "the same" scenario even when their steps[] differ (e.g. the
+ * server's auto-drafted version vs. the agent's own re-drafted version of a
+ * scenario with an identical name). Used to dedupe by scenario identity where
+ * `computeCandidateId`'s content-sensitive hash would wrongly treat two drafts
+ * of the same named scenario as distinct (SKYR-4026).
+ */
+export declare function scenarioNameSlug(scenarioName: string | undefined): string;
+/**
+ * Merge-dedup identity key for a scenario name — like {@link scenarioNameSlug}
+ * but NOT truncated and with no shared fallback for a missing name. The 48-char
+ * truncation in `scenarioNameSlug` is fine for `computeCandidateId` (combined
+ * with a content hash for uniqueness), but used bare as a Map key it would
+ * silently collapse two distinct scenarios sharing a long common prefix, and
+ * collapse every candidate with no scenarioName onto the same key. Returns ""
+ * for a missing name so the caller can fall back to something already unique
+ * (e.g. candidateId) instead of a shared literal.
+ */
+export declare function scenarioMergeKey(scenarioName: string | undefined): string;
 /**
  * Deterministic candidate id: a slug of the scenario name plus an 8-char content
  * hash. Pure and reproducible — the same scenario always yields the same id, so

package/build/types/Recommendation.js

@@ -17,6 +17,37 @@ export var CandidateSource;
     CandidateSource["AGENT"] = "agent";
     CandidateSource["SERVER"] = "server";
 })(CandidateSource || (CandidateSource = {}));
+/**
+ * Slug of a scenario name alone (no content hash) — stable across independent
+ * drafts of "the same" scenario even when their steps[] differ (e.g. the
+ * server's auto-drafted version vs. the agent's own re-drafted version of a
+ * scenario with an identical name). Used to dedupe by scenario identity where
+ * `computeCandidateId`'s content-sensitive hash would wrongly treat two drafts
+ * of the same named scenario as distinct (SKYR-4026).
+ */
+export function scenarioNameSlug(scenarioName) {
+    return ((scenarioName ?? "")
+        .toLowerCase()
+        .replace(/[^a-z0-9]+/g, "-")
+        .replace(/^-+|-+$/g, "")
+        .slice(0, 48) || "scenario");
+}
+/**
+ * Merge-dedup identity key for a scenario name — like {@link scenarioNameSlug}
+ * but NOT truncated and with no shared fallback for a missing name. The 48-char
+ * truncation in `scenarioNameSlug` is fine for `computeCandidateId` (combined
+ * with a content hash for uniqueness), but used bare as a Map key it would
+ * silently collapse two distinct scenarios sharing a long common prefix, and
+ * collapse every candidate with no scenarioName onto the same key. Returns ""
+ * for a missing name so the caller can fall back to something already unique
+ * (e.g. candidateId) instead of a shared literal.
+ */
+export function scenarioMergeKey(scenarioName) {
+    return (scenarioName ?? "")
+        .toLowerCase()
+        .replace(/[^a-z0-9]+/g, "-")
+        .replace(/^-+|-+$/g, "");
+}
 /**
  * Deterministic candidate id: a slug of the scenario name plus an 8-char content
  * hash. Pure and reproducible — the same scenario always yields the same id, so
@@ -26,12 +57,7 @@ export var CandidateSource;
  * cosmetic re-serialization does not change the id while a real edit does.
  */
 export function computeCandidateId(scenario) {
-    const slugSource = scenario.scenarioName ?? "";
-    const slug = slugSource
-        .toLowerCase()
-        .replace(/[^a-z0-9]+/g, "-")
-        .replace(/^-+|-+$/g, "")
-        .slice(0, 48) || "scenario";
+    const slug = scenarioNameSlug(scenario.scenarioName);
     const canonical = {
         scenarioName: scenario.scenarioName ?? "",
         category: scenario.category ?? "",

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 {
@@ -542,6 +559,7 @@ export declare const endpointMethodSchema: z.ZodObject<{
 }, "strip", z.ZodTypeAny, {
     method: string;
     description: string;
+    sourceFile: string;
     queryParams: {
         type: string;
         name: string;
@@ -550,7 +568,6 @@ export declare const endpointMethodSchema: z.ZodObject<{
         constraints?: string | undefined;
     }[];
     authRequired: boolean;
-    sourceFile: string;
     interactions: {
         type: "error" | "success" | "edge-case";
         description: string;
@@ -581,6 +598,7 @@ export declare const endpointMethodSchema: z.ZodObject<{
 }, {
     method: string;
     description: string;
+    sourceFile: string;
     queryParams: {
         type: string;
         name: string;
@@ -589,7 +607,6 @@ export declare const endpointMethodSchema: z.ZodObject<{
         constraints?: string | undefined;
     }[];
     authRequired: boolean;
-    sourceFile: string;
     interactions: {
         type: "error" | "success" | "edge-case";
         description: string;
@@ -784,6 +801,7 @@ export declare const enrichedEndpointSchema: z.ZodObject<{
     }, "strip", z.ZodTypeAny, {
         method: string;
         description: string;
+        sourceFile: string;
         queryParams: {
             type: string;
             name: string;
@@ -792,7 +810,6 @@ export declare const enrichedEndpointSchema: z.ZodObject<{
             constraints?: string | undefined;
         }[];
         authRequired: boolean;
-        sourceFile: string;
         interactions: {
             type: "error" | "success" | "edge-case";
             description: string;
@@ -823,6 +840,7 @@ export declare const enrichedEndpointSchema: z.ZodObject<{
     }, {
         method: string;
         description: string;
+        sourceFile: string;
         queryParams: {
             type: string;
             name: string;
@@ -831,7 +849,6 @@ export declare const enrichedEndpointSchema: z.ZodObject<{
             constraints?: string | undefined;
         }[];
         authRequired: boolean;
-        sourceFile: string;
         interactions: {
             type: "error" | "success" | "edge-case";
             description: string;
@@ -873,6 +890,7 @@ export declare const enrichedEndpointSchema: z.ZodObject<{
     methods: {
         method: string;
         description: string;
+        sourceFile: string;
         queryParams: {
             type: string;
             name: string;
@@ -881,7 +899,6 @@ export declare const enrichedEndpointSchema: z.ZodObject<{
             constraints?: string | undefined;
         }[];
         authRequired: boolean;
-        sourceFile: string;
         interactions: {
             type: "error" | "success" | "edge-case";
             description: string;
@@ -923,6 +940,7 @@ export declare const enrichedEndpointSchema: z.ZodObject<{
     methods: {
         method: string;
         description: string;
+        sourceFile: string;
         queryParams: {
             type: string;
             name: string;
@@ -931,7 +949,6 @@ export declare const enrichedEndpointSchema: z.ZodObject<{
             constraints?: string | undefined;
         }[];
         authRequired: boolean;
-        sourceFile: string;
         interactions: {
             type: "error" | "success" | "edge-case";
             description: string;
@@ -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;
@@ -2057,6 +2113,7 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
             }, "strip", z.ZodTypeAny, {
                 method: string;
                 description: string;
+                sourceFile: string;
                 queryParams: {
                     type: string;
                     name: string;
@@ -2065,7 +2122,6 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
                     constraints?: string | undefined;
                 }[];
                 authRequired: boolean;
-                sourceFile: string;
                 interactions: {
                     type: "error" | "success" | "edge-case";
                     description: string;
@@ -2096,6 +2152,7 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
             }, {
                 method: string;
                 description: string;
+                sourceFile: string;
                 queryParams: {
                     type: string;
                     name: string;
@@ -2104,7 +2161,6 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
                     constraints?: string | undefined;
                 }[];
                 authRequired: boolean;
-                sourceFile: string;
                 interactions: {
                     type: "error" | "success" | "edge-case";
                     description: string;
@@ -2146,6 +2202,7 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
             methods: {
                 method: string;
                 description: string;
+                sourceFile: string;
                 queryParams: {
                     type: string;
                     name: string;
@@ -2154,7 +2211,6 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
                     constraints?: string | undefined;
                 }[];
                 authRequired: boolean;
-                sourceFile: string;
                 interactions: {
                     type: "error" | "success" | "edge-case";
                     description: string;
@@ -2196,6 +2252,7 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
             methods: {
                 method: string;
                 description: string;
+                sourceFile: string;
                 queryParams: {
                     type: string;
                     name: string;
@@ -2204,7 +2261,6 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
                     constraints?: string | undefined;
                 }[];
                 authRequired: boolean;
-                sourceFile: string;
                 interactions: {
                     type: "error" | "success" | "edge-case";
                     description: string;
@@ -2250,6 +2306,7 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
             methods: {
                 method: string;
                 description: string;
+                sourceFile: string;
                 queryParams: {
                     type: string;
                     name: string;
@@ -2258,7 +2315,6 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
                     constraints?: string | undefined;
                 }[];
                 authRequired: boolean;
-                sourceFile: string;
                 interactions: {
                     type: "error" | "success" | "edge-case";
                     description: string;
@@ -2304,6 +2360,7 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
             methods: {
                 method: string;
                 description: string;
+                sourceFile: string;
                 queryParams: {
                     type: string;
                     name: string;
@@ -2312,7 +2369,6 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
                     constraints?: string | undefined;
                 }[];
                 authRequired: boolean;
-                sourceFile: string;
                 interactions: {
                     type: "error" | "success" | "edge-case";
                     description: string;
@@ -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;
@@ -2700,6 +2795,7 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
             methods: {
                 method: string;
                 description: string;
+                sourceFile: string;
                 queryParams: {
                     type: string;
                     name: string;
@@ -2708,7 +2804,6 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
                     constraints?: string | undefined;
                 }[];
                 authRequired: boolean;
-                sourceFile: string;
                 interactions: {
                     type: "error" | "success" | "edge-case";
                     description: string;
@@ -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;
@@ -2902,6 +3008,7 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
             methods: {
                 method: string;
                 description: string;
+                sourceFile: string;
                 queryParams: {
                     type: string;
                     name: string;
@@ -2910,7 +3017,6 @@ export declare const repositoryAnalysisSchema: z.ZodObject<{
                     constraints?: string | undefined;
                 }[];
                 authRequired: boolean;
-                sourceFile: string;
                 interactions: {
                     type: "error" | "success" | "edge-case";
                     description: 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(),
 });