@skyramp/mcp 0.4.1 → 0.4.2-rc.1

package/build/prompts/test-maintenance/actionsInstructions.js

@@ -1,3 +1,4 @@
+import { enhanceAssertionsMaintenanceCall } from "../../skills/enhanceAssertionsSkill.js";
 /**
  * Prompt language for skyramp_actions output — the LLM_INSTRUCTIONS block
  * emitted to the agent that applies UPDATE, REGENERATE, and DELETE actions.
@@ -5,6 +6,7 @@
  * Kept separate from actionsTool.ts so prompt text is co-located with other
  * maintenance prompts and can be reviewed/edited without touching tool logic.
  */
+import { regenerateCodeReuse } from "../reuse-hand-off.js";
 /** Strategy string for URL-path find-and-replace updates. */
 export function buildRenameStrategy() {
     return `For each file, find-and-replace all occurrences of oldPath with newPath. Do NOT regenerate or restructure the test — only update the URL paths.`;
@@ -13,23 +15,28 @@ export function buildRenameStrategy() {
 export function buildFileRenameStrategy() {
     return `After updating path content in each file, rename the file using 'mv' or equivalent. Use git mv if the repo tracks the file.`;
 }
-/** Strategy string for in-place UPDATE edits. */
-export function buildUpdateStrategy() {
+/** Strategy string for in-place UPDATE edits. `withPostEditSteps` is set when at
+ *  least one update_context entry carries post_edit_steps — the modularize → reuse
+ *  pair an updated test owes, which skyramp_execute_test refuses the file without. */
+export function buildUpdateStrategy(withPostEditSteps = false) {
     return `Apply each file in update_context yourself with the Edit tool. Read the file first, then make the changes described in context. Preserve all existing test logic — only add or adjust what context describes.
 
 When a file provides edit_sites, apply each site directly using its anchor as the Edit target — do not re-grep or re-page the file to relocate the sites; they were already found for you.
 
 UPDATE is an in-place edit to the existing baseline file — do not create a new test file to cover the same page or component.
 
-After editing, re-read each file to confirm the change landed, then call skyramp_enhance_assertions with each updated file path.`;
+After editing, re-read each file to confirm the change landed${withPostEditSteps ? `. For an entry that carries post_edit_steps, run them next, in the order given: they are the modularize → reuse steps the updated test now owes, and skyramp_execute_test refuses the file until they have run. Then ${enhanceAssertionsMaintenanceCall()}` : `, then ${enhanceAssertionsMaintenanceCall()}`}.`;
 }
 /** Strategy string for UPDATE entries that refresh visual-snapshot baselines (SKYR-4298). */
 export function buildRebaselineStrategy() {
     return `Some update_context entries carry rebaseline_snapshots: the diff changed how the page or element those toHaveScreenshot() baselines capture looks, so the committed PNGs are stale and must be replaced, not the spec. For those baselines do NOT edit the toHaveScreenshot() call, do NOT raise maxDiffPixelRatio, do NOT remove the assertion, and do NOT touch the PNG yourself. Refresh them at the final execution: call skyramp_execute_test for that spec with rebaselineSnapshots set to exactly the listed filenames — SmartPlaywright re-captures each one and the new PNG lands beside the spec, where the Testbot PR shows it as an image diff for review. An entry with edit_required: false (no context) has nothing to edit: skip the Edit/re-read/skyramp_enhance_assertions steps for that file — the refresh at final execution is its whole maintenance. When you call skyramp_execute_test for the refresh, set testFile to the entry's spec_file (NOT its file, which is the page object for a POM-backed update). The execution result names which baselines were refreshed and which were not; report only the refreshed ones as refreshed. Never pass rebaselineSnapshots on the phase: "before" run — that run is the evidence that the baseline is stale.`;
 }
-/** Strategy string for REGENERATE — call generation tool to overwrite the file. */
-export function buildRegenerateStrategy() {
-    return `For each file in regenerate_context, call the appropriate generation tool (skyramp_integration_test_generation or skyramp_contract_test_generation) with outputDir set to the file's directory and output set to the filename. The generation tool will overwrite the file. Do NOT use skyramp_ui_test_generation here — UI test regeneration requires a recorded trace (playwrightInput) and must be handled separately.`;
+/** Strategy string for REGENERATE — call generation tool to overwrite the file.
+ *  `withGenerationArgs` is set when at least one regenerate_context entry carries
+ *  generation_args — the codeReuse options that arm the modularize → reuse chain the
+ *  regenerated file owes. */
+export function buildRegenerateStrategy(withGenerationArgs = false) {
+    return `For each file in regenerate_context, call the appropriate generation tool (skyramp_integration_test_generation or skyramp_contract_test_generation) with outputDir set to the file's directory and output set to the filename${withGenerationArgs ? `, and, when the entry carries generation_args, every one of those arguments as well — they ask for the code reuse the regenerated file owes, and the generation result then carries its modularize → reuse steps` : ""}. The generation tool will overwrite the file. Do NOT use skyramp_ui_test_generation here — UI test regeneration requires a recorded trace (playwrightInput) and must be handled separately.`;
 }
 /** Strategy string for DELETE — remove obsolete test files whose covered endpoints/UI no longer exist. */
 export function buildDeleteStrategy() {
@@ -37,46 +44,60 @@ export function buildDeleteStrategy() {
 }
 /** Per-file instruction block for a single UPDATE recommendation. */
 export function buildUpdateFileInstruction(params) {
-    const { testFile, renames, suggestedNewFile, updateInstructions, rationale } = params;
+    const { testFile, renames, suggestedNewFile, updateInstructions, rationale, postEditSteps, } = params;
     const rebaselineSnapshots = params.rebaselineSnapshots ?? [];
-    const renameTable = renames.length > 0 ? [
-        `**Endpoint Rename Detected — Path Substitution Required:**\n`,
-        `| Old Path | New Path | Method |`,
-        `|----------|----------|--------|`,
-        ...renames.map((r) => `| \`${r.oldPath}\` | \`${r.newPath}\` | ${r.method} |`),
-        ``,
-        `**Action:** Find-and-replace all occurrences of the old path with the new path in this test file. Do NOT change any test logic, assertions, or structure — only update the URL paths.\n`,
-        ...(suggestedNewFile ? [
-            `**File Rename:** After updating the paths, rename this file:`,
-            `- From: \`${testFile.split("/").pop()}\``,
-            `- To: \`${suggestedNewFile.split("/").pop()}\`\n`,
-        ] : []),
-    ].join("\n") : "";
+    const renameTable = renames.length > 0
+        ? [
+            `**Endpoint Rename Detected — Path Substitution Required:**\n`,
+            `| Old Path | New Path | Method |`,
+            `|----------|----------|--------|`,
+            ...renames.map((r) => `| \`${r.oldPath}\` | \`${r.newPath}\` | ${r.method} |`),
+            ``,
+            `**Action:** Find-and-replace all occurrences of the old path with the new path in this test file. Do NOT change any test logic, assertions, or structure — only update the URL paths.\n`,
+            ...(suggestedNewFile
+                ? [
+                    `**File Rename:** After updating the paths, rename this file:`,
+                    `- From: \`${testFile.split("/").pop()}\``,
+                    `- To: \`${suggestedNewFile.split("/").pop()}\`\n`,
+                ]
+                : []),
+        ].join("\n")
+        : "";
     const changeBlock = updateInstructions
         ? `**What to change:**\n\n${updateInstructions}\n`
         : renames.length === 0 && rebaselineSnapshots.length === 0
             ? [
                 rationale ? `**Why:** ${rationale}\n` : "",
                 `**Action:** Update this test file based on the rationale above.\n`,
-            ].filter(Boolean).join("\n")
+            ]
+                .filter(Boolean)
+                .join("\n")
             : "";
     const rebaselineBlock = rebaselineSnapshots.length > 0
         ? [
             `**Visual baselines to refresh:** ${rebaselineSnapshots.map((s) => `\`${s}\``).join(", ")}`,
             rationale && !updateInstructions ? `**Why:** ${rationale}` : "",
             `**Action:** Do not edit these \`toHaveScreenshot()\` calls, their \`maxDiffPixelRatio\`, or the PNGs. At the final execution, call \`skyramp_execute_test\` for this spec with \`rebaselineSnapshots: ${JSON.stringify(rebaselineSnapshots)}\` so SmartPlaywright re-captures them; the refreshed PNGs are delivered with the Testbot PR as image diffs.\n`,
-        ].filter(Boolean).join("\n")
+        ]
+            .filter(Boolean)
+            .join("\n")
         : "";
-    return `\n### ${testFile}\n\n${renameTable}${changeBlock}${rebaselineBlock}`;
+    const stepsBlock = postEditSteps ? `\n${postEditSteps}\n` : "";
+    return `\n### ${testFile}\n\n${renameTable}${changeBlock}${rebaselineBlock}${stepsBlock}`;
 }
 /** Per-file instruction block for a single REGENERATE recommendation. */
 export function buildRegenerateFileInstruction(params) {
-    const { testFile, updateInstructions, outputDir, outputFile } = params;
-    const whatChanged = updateInstructions ? `**What changed:**\n\n${updateInstructions}\n\n` : "";
+    const { testFile, updateInstructions, outputDir, outputFile, testType, language, } = params;
+    const whatChanged = updateInstructions
+        ? `**What changed:**\n\n${updateInstructions}\n\n`
+        : "";
+    // The regenerated file goes through the same modularize → reuse chain a generated
+    // one does: the generation tool's result carries the steps when the call asks for
+    // codeReuse, so the clause here is what arms it.
     return [
         `\n### ${testFile}\n`,
         `**Action: REGENERATE** — the response shape changed too drastically for targeted edits.\n`,
         whatChanged,
-        `Call the appropriate generation tool (e.g. \`skyramp_integration_test_generation\`, \`skyramp_contract_test_generation\`) with \`outputDir: "${outputDir}"\` and \`output: "${outputFile}"\` to overwrite this file from scratch. Use the provided existing file content for context on the endpoint, auth pattern, and test structure — replicate the test type and language.\n`,
+        `Call the appropriate generation tool (e.g. \`skyramp_integration_test_generation\`, \`skyramp_contract_test_generation\`) with \`outputDir: "${outputDir}"\` and \`output: "${outputFile}"\`${regenerateCodeReuse(testType, language)?.clause ?? ""} to overwrite this file from scratch. Use the provided existing file content for context on the endpoint, auth pattern, and test structure — replicate the test type and language.\n`,
     ].join("\n");
 }

package/build/prompts/test-maintenance/drift-analysis-prompt.js

@@ -1,6 +1,6 @@
 import { buildActionDecisionTree, buildCheckAdditiveFields, buildCheckEndpointExistence, buildCheckResponseShape, buildCheckAuthAndAuthorization, buildCheckBehavioralContract, buildCheckAssignAction, buildDriftOutputChecklist, } from "./driftAnalysisSections.js";
 import { buildUiActionDecisionTree, buildUiCheckRouteExistence, buildUiCheckSelectors, buildUiCheckPageObjects, buildUiCheckBehavioralChanges, buildUiCheckAssignAction, buildUiDriftOutputChecklist, } from "./uiDriftAnalysisSections.js";
-import { RECOMMENDATIONS_INSTRUCTION, buildSymbolDiscoveryStep } from "./driftAnalysisShared.js";
+import { RECOMMENDATIONS_INSTRUCTION, buildSymbolDiscoveryStep, } from "./driftAnalysisShared.js";
 import { PromptPlan } from "../test-recommendation/promptPlan.js";
 /**
  * Render CONFIRMED failing tests from prior external-exec runs, plus the §4.5
@@ -117,7 +117,8 @@ For each test in \`<tests_to_assess>\`, read it to determine its type, then appl
 </ui_analysis_context>
 
 `;
-        parts.push(contextHeader + `<ui_drift_analysis_rules>\n${_uiPlan.render({ stateFile, tests: ui.tests, blueprintCaptured: ui.blueprintCaptured })}\n</ui_drift_analysis_rules>`);
+        parts.push(contextHeader +
+            `<ui_drift_analysis_rules>\n${_uiPlan.render({ stateFile, tests: ui.tests, blueprintCaptured: ui.blueprintCaptured })}\n</ui_drift_analysis_rules>`);
     }
     parts.push(RECOMMENDATIONS_INSTRUCTION);
     return parts.join("\n\n");

package/build/prompts/test-maintenance/driftAnalysisSections.js

@@ -3,7 +3,7 @@
  * mirroring the recommendationSections.ts pattern.
  */
 import { AUTH_MIDDLEWARE_PATTERNS_STR } from "../../utils/workspaceAuth.js";
-import { buildApiAssignAction, EMPTY_TEST_LIST_PLACEHOLDER, NO_PRELOADED_TESTS_NOTE } from "./driftAnalysisShared.js";
+import { buildApiAssignAction, EMPTY_TEST_LIST_PLACEHOLDER, NO_PRELOADED_TESTS_NOTE, } from "./driftAnalysisShared.js";
 // TODO: Replace the open-ended diff-line categories below with a two-tier structure:
 // Tier 1 — mechanical patterns (specific token + file-path criteria, zero judgment needed).
 // Tier 2 — residual open-ended scan for anything observable not covered by tier 1.
@@ -197,10 +197,12 @@ export function buildDriftOutputChecklist(stateFile, existingTests) {
     const finalStep = `Call \`skyramp_actions\` with \`stateFile: "${stateFile ?? "<stateFile>"}"\`.`;
     const hasTests = (existingTests?.length ?? 0) > 0;
     const testList = hasTests
-        ? existingTests.map((t) => {
+        ? existingTests
+            .map((t) => {
             const label = (t.source ?? "skyramp") === "external" ? " [external]" : "";
             return `- ${t.testFile}${label}`;
-        }).join("\n")
+        })
+            .join("\n")
         : EMPTY_TEST_LIST_PLACEHOLDER;
     const noTestsNote = !hasTests ? NO_PRELOADED_TESTS_NOTE : "";
     const existingTestSection = `**Existing tests (${existingTests?.length ?? 0} total) — assess ALL of the following:**

package/build/prompts/test-maintenance/driftAnalysisShared.js

@@ -6,8 +6,10 @@
  */
 export function buildSymbolDiscoveryStep(hasTests, repoPaths) {
     const repoList = repoPaths && repoPaths.length > 1
-        ? repoPaths.map(p => `\`${p}\``).join(", ")
-        : repoPaths?.[0] ? `\`${repoPaths[0]}\`` : "the repository";
+        ? repoPaths.map((p) => `\`${p}\``).join(", ")
+        : repoPaths?.[0]
+            ? `\`${repoPaths[0]}\``
+            : "the repository";
     // When no tests were pre-loaded this step is REQUIRED, not optional: in PR mode external
     // (user-written) tests are discovered by the agent, not pre-loaded, so an empty list means
     // "not yet discovered", not "none exist". Suppressing this step in that case is the bug that

package/build/prompts/test-maintenance/uiDriftAnalysisSections.js

@@ -8,7 +8,7 @@
  * (RTL/Flutter widget/vue-test-utils/angular-testbed) read the same text.
  * Component tests always use source-only analysis regardless of blueprintCaptured.
  */
-import { buildUiAssignAction, EMPTY_TEST_LIST_PLACEHOLDER, NO_PRELOADED_TESTS_NOTE } from "./driftAnalysisShared.js";
+import { buildUiAssignAction, EMPTY_TEST_LIST_PLACEHOLDER, NO_PRELOADED_TESTS_NOTE, } from "./driftAnalysisShared.js";
 /**
  * Build the UI Action Decision Tree section.
  * This is the main entry point for UI drift analysis.
@@ -103,7 +103,7 @@ Actions:
  * Check if page objects/fixtures referenced by the test still exist.
  */
 export function buildUiCheckPageObjects() {
-    return `Do the page objects or fixture files imported by this UI test still exist, and are their underlying selectors still valid?
+    return (`Do the page objects or fixture files imported by this UI test still exist, and are their underlying selectors still valid?
 
 **Always follow imports to find page objects — regardless of whether blueprints were captured.**
 Page objects can live anywhere in the repo (not just \`page-objects/\` or \`fixtures/\`). Do not assume a fixed directory. The only reliable way to find them is to follow the spec file's \`import\` statements.
@@ -127,7 +127,8 @@ Actions:
 - Page object import path changed in the diff → **UPDATE** (fix the import in the spec)
 - Page object file deleted, no replacement found → **UPDATE** (inline selectors or find alternative)
 - Selectors in page object unchanged and diff does not affect them → **IGNORE**
-- **All POM selectors are gone with no migration path in the diff** (the component's interaction model changed fundamentally — no ` + ` lines showing replacement selectors) → **REGENERATE**. Do not stay at VERIFY after reading the POM and confirming this — escalate once you have confirmed the failure.`;
+- **All POM selectors are gone with no migration path in the diff** (the component's interaction model changed fundamentally — no ` +
+        ` lines showing replacement selectors) → **REGENERATE**. Do not stay at VERIFY after reading the POM and confirming this — escalate once you have confirmed the failure.`);
 }
 /**
  * Check for component behavior changes that affect assertions.
@@ -164,7 +165,9 @@ export function buildUiDriftOutputChecklist(stateFile, tests) {
     const finalStep = `Call \`skyramp_actions\` with \`stateFile: "${stateFile ?? "<stateFile>"}"\`.`;
     const hasTests = (tests?.length ?? 0) > 0;
     const testList = hasTests
-        ? tests.map(t => `- ${t.testFile}${t.source === "external" ? " [external]" : ""}`).join("\n")
+        ? tests
+            .map((t) => `- ${t.testFile}${t.source === "external" ? " [external]" : ""}`)
+            .join("\n")
         : EMPTY_TEST_LIST_PLACEHOLDER;
     const noTestsNote = !hasTests ? NO_PRELOADED_TESTS_NOTE : "";
     const existingTestSection = `**Tests (${tests?.length ?? 0} total) — assess ALL of the following:**

package/build/prompts/test-recommendation/recommendationSections.js

@@ -3,7 +3,7 @@ import { sectionBody } from "../promptAssets.js";
 import { readPromptAsset } from "../promptAssets.js";
 import { fillPlaceholders } from "../../recommendation/verifierContracts.js";
 import { resolveServiceDetailsRef } from "../../utils/utils.js";
-import { getAuthScheme, isAuthorizationHeaderName } from "../../utils/workspaceAuth.js";
+import { getAuthScheme, isAuthorizationHeaderName, } from "../../utils/workspaceAuth.js";
 // Cached at module-load — flags are process-wide and cannot change per call.
 const CONSUMER_MODE_ENABLED = isContractConsumerModeEnabled();
 const SERVICE_REFS = resolveServiceDetailsRef();
@@ -83,15 +83,18 @@ export function buildToolWorkflows(authHeaderValue, authTypeValue = "", explicit
     let authGuidance;
     let authParams;
     if (noAuth) {
-        authGuidance = variant("## Auth verification required", { enrichStep: enrichStepLabel });
+        authGuidance = variant("## Auth verification required", {
+            enrichStep: enrichStepLabel,
+        });
         authParams = { authHeader: "" };
     }
     else if (isAuthorizationHeader) {
         // Use getAuthScheme (single source of truth) instead of capitalizing the raw slug.
         // Pass explicitScheme so custom schemes from workspace.yml (e.g. "Ghost", "Hawk") are used.
         const wsType = authTypeValue;
-        const resolvedScheme = (authTypeValue && authTypeValue !== "none")
-            ? (getAuthScheme(wsType, explicitScheme) ?? "<scheme e.g. Bearer, Token or empty>")
+        const resolvedScheme = authTypeValue && authTypeValue !== "none"
+            ? (getAuthScheme(wsType, explicitScheme) ??
+                "<scheme e.g. Bearer, Token or empty>")
             : "<scheme e.g. Bearer, Token or empty>";
         authParams = { authHeader: "Authorization", authScheme: resolvedScheme };
         // One ladder for both cases: only the first line differs, and step 3 states
@@ -105,7 +108,9 @@ export function buildToolWorkflows(authHeaderValue, authTypeValue = "", explicit
     }
     else {
         authParams = { authHeader: authHeaderValue };
-        authGuidance = variant("## Auth header, not Authorization", { authHeader: authHeaderValue });
+        authGuidance = variant("## Auth header, not Authorization", {
+            authHeader: authHeaderValue,
+        });
     }
     const hasWorkspaceAuthType = !!authTypeValue && authTypeValue !== "none";
     const authCallParams = serializeAuthCallParams(authParams);
@@ -114,9 +119,13 @@ export function buildToolWorkflows(authHeaderValue, authTypeValue = "", explicit
     // NOTE: scenario generation always uses full authCallParams so auth is embedded in the test file
     // and the executor sends the correct Authorization header at run time.
     const integrationAuthNote = hasWorkspaceAuthType
-        ? variant("## Integration auth, workspace handles it", { authConflictError: AUTH_CONFLICT_ERROR_MSG })
+        ? variant("## Integration auth, workspace handles it", {
+            authConflictError: AUTH_CONFLICT_ERROR_MSG,
+        })
         : variant("## Integration auth, header only", {
-            authHeaderOnlyParams: serializeAuthCallParams({ authHeader: authParams.authHeader }),
+            authHeaderOnlyParams: serializeAuthCallParams({
+                authHeader: authParams.authHeader,
+            }),
         });
     const authHeaderLine = noAuth
         ? variant("## Auth line, none configured", {})
@@ -129,6 +138,8 @@ export function buildToolWorkflows(authHeaderValue, authTypeValue = "", explicit
         pathParamUuidGuidance: buildPathParamGuidance(),
         frontendTestDir: SERVICE_REFS.frontendTestDirRef,
         enrichStep: enrichStepLabel,
-        contractModeSelection: variant(CONSUMER_MODE_ENABLED ? "## Contract modes, consumer enabled" : "## Contract modes, provider only", {}),
+        contractModeSelection: variant(CONSUMER_MODE_ENABLED
+            ? "## Contract modes, consumer enabled"
+            : "## Contract modes, provider only", {}),
     });
 }

package/build/prompts/test-recommendation/registerRecommendTestsPrompt.js

@@ -42,7 +42,9 @@ export function registerRecommendTestsPrompt(server) {
             const memData = getSessionData(sessionId);
             if (memData?.analysis) {
                 fullAnalysis = memData.analysis;
-                logger.debug("Loaded fullAnalysis from process memory", { sessionId });
+                logger.debug("Loaded fullAnalysis from process memory", {
+                    sessionId,
+                });
             }
         }
         if (!fullAnalysis) {
@@ -73,7 +75,8 @@ export function registerRecommendTestsPrompt(server) {
         // the user-facing param value "branch_diff". Map it explicitly so diff-mode detection
         // works correctly on state created before this deployment (2-hour TTL window).
         const rawScope = state.analysisScope;
-        const analysisScope = rawScope === "branch_diff" || rawScope === AnalysisScope.CurrentBranchDiff
+        const analysisScope = rawScope === "branch_diff" ||
+            rawScope === AnalysisScope.CurrentBranchDiff
             ? AnalysisScope.CurrentBranchDiff
             : AnalysisScope.FullRepo;
         const rejectionWarning = "";

package/build/prompts/test-recommendation/test-recommendation-prompt.js

@@ -1,21 +1,24 @@
 import { AnalysisScope, isDiff, } from "../../types/RepositoryAnalysis.js";
-import { WorkspaceAuthType, getDefaultAuthHeader, AUTH_MIDDLEWARE_PATTERNS_STR } from "../../utils/workspaceAuth.js";
+import { WorkspaceAuthType, getDefaultAuthHeader, AUTH_MIDDLEWARE_PATTERNS_STR, } from "../../utils/workspaceAuth.js";
 import { logger } from "../../utils/logger.js";
 import { buildArchitectPreamble, buildContextFetchingGuidance, buildReasoningProtocol, buildToolWorkflows, buildVerificationChecklist, } from "./recommendationSections.js";
 import { buildExecutionPlan, EXEC_STEP_ENRICH } from "./diffExecutionPlan.js";
 import { readPromptAsset } from "../promptAssets.js";
 import { fillPlaceholders } from "../../recommendation/verifierContracts.js";
-import { TASK_GENERATE, taskRef, } from "./recommendationShared.js";
+import { TASK_GENERATE, taskRef } from "./recommendationShared.js";
 // Re-export for backward compatibility (tests and external callers import this from this module)
 function formatTestLocations(locs) {
     const entries = Object.entries(locs || {});
     if (entries.length === 0)
         return "";
-    const rows = entries.map(([type, files]) => `| ${type.padEnd(12)} | ${files} |`).join("\n");
+    const rows = entries
+        .map(([type, files]) => `| ${type.padEnd(12)} | ${files} |`)
+        .join("\n");
     return ("\n**Existing test coverage (Skyramp + external):**\n" +
         "| Test type    | File (covers: endpoints)                                |\n" +
         "|--------------|---------------------------------------------------------|\n" +
-        rows + "\n");
+        rows +
+        "\n");
 }
 // ── Execution Plan (replaces pre-ranked + scenarios + heuristic sections) ──
 // Prevents bot-committed test files from being treated as application changes
@@ -38,7 +41,7 @@ export function buildRecommendationPrompt(analysis, analysisScope = AnalysisScop
     const diffContext = analysis.branchDiffContext;
     const openApiSpec = analysis.artifacts?.openApiSpecs?.[0];
     const filteredChangedFiles = diffContext
-        ? diffContext.changedFiles.filter(f => !SKYRAMP_TEST_FILE_PATTERN.test(f))
+        ? diffContext.changedFiles.filter((f) => !SKYRAMP_TEST_FILE_PATTERN.test(f))
         : [];
     const hasTraces = (analysis.artifacts?.traceFiles?.length ?? 0) > 0 ||
         (analysis.artifacts?.playwrightRecordings?.length ?? 0) > 0;
@@ -90,14 +93,14 @@ Use it for contract tests (\`apiSchema: "${openApiSpec.path}"\`) and to extract
     // ── PR History ──
     let prHistorySection = "";
     if (prContext && prContext.previousRecommendations.length > 0) {
-        const implemented = prContext.previousRecommendations.filter(r => r.status === "implemented");
+        const implemented = prContext.previousRecommendations.filter((r) => r.status === "implemented");
         let historyBody = "";
         if (implemented.length > 0) {
             const implLines = implemented
                 .map((r) => `  - ${r.testType} — ${r.endpoint}`)
                 .join("\n");
             const fileLines = prContext.implementedTestFiles.length > 0
-                ? `\nExisting test files (in working tree):\n${prContext.implementedTestFiles.map(f => "  - \`" + f + "\`").join("\n")}\n`
+                ? `\nExisting test files (in working tree):\n${prContext.implementedTestFiles.map((f) => "  - \`" + f + "\`").join("\n")}\n`
                 : "";
             historyBody += `### Previously Generated Tests
 ${implLines}${fileLines}`;