@skyramp/mcp 0.4.1 → 0.4.2-rc.2

package/build/utils/language-helper.js

@@ -1,28 +1,66 @@
+import { SKYRAMP_SDK_VERSION } from "./skyrampSdkVersion.js";
+/** The SDK spec the generated JavaScript and TypeScript tests need. */
+const npmSdkSpec = `@skyramp/skyramp@${SKYRAMP_SDK_VERSION}`;
+const pythonDependencyGuidance = `
+Persist each missing approved package in the development manifest:
+- Use \`uv add --dev <package>\` for a project using \`[dependency-groups].dev\`.
+- Use \`poetry add --group dev <package>\` for a Poetry project.
+- Add it to \`requirements-dev.txt\`, then run \`python3 -m pip install -r requirements-dev.txt\`.
+If no Python manifest exists at the test project root, create requirements-dev.txt and seed it with the missing approved package. Do not create it alongside pyproject.toml.
+If the repository commits poetry.lock or uv.lock, update it in the same change. Use \`poetry lock\` or \`uv lock\`; both keep the versions already locked.
+`;
+const javascriptDependencyGuidance = `
+Inspect package.json before installing anything.
+- If a required package is already declared, install from the existing manifest without changing its version.
+- If no JavaScript/TypeScript manifest exists at the test project root, create package.json with only \`{ "devDependencies": {} }\` before adding the missing approved package.
+- If an approved package is missing, add it as a devDependency with the repository's package manager. The npm equivalent is \`npm install --save-dev --legacy-peer-deps <package>\`.
+- Never add, upgrade, or downgrade Playwright or @playwright/test when either is already declared.
+- If the repository commits a lockfile, update it in the same change. Use \`npm install --package-lock-only\`, \`yarn install --mode update-lockfile\`, \`pnpm install --lockfile-only\`, or \`bun install --lockfile-only\` for the package manager the repository uses.
+- Do not change \`allowScripts\`, \`pnpm.onlyBuiltDependencies\`, or \`trustedDependencies\`.
+`;
+const javaDependencyGuidance = `
+Inspect existing Java build signals before adding a dependency. Use an existing manifest when present.
+- For Maven wrapper or settings signals, create a minimal pom.xml only when it is absent, and add approved dependencies with <scope>test</scope>.
+- For Gradle wrapper or settings signals, create the matching build.gradle or build.gradle.kts only when it is absent, and use testImplementation or testRuntimeOnly.
+- Do not introduce a competing build system. If neither build system is identified, keep the standalone runner instead of inventing one.
+- If the repository commits gradle.lockfile, update it in the same change with the repository's Gradle wrapper (\`./gradlew dependencies --write-locks\`), or \`gradle\` only when there is no wrapper.
+`;
+function getJavascriptSteps(extension) {
+    return `${javascriptDependencyGuidance}${sdkDeclaration("javascript")}
+# Execute the test file
+npx playwright test <test-file>.spec.${extension} --reporter=list
+`;
+}
+/** How to declare the Skyramp SDK, per language. Kept out of the shared
+ *  dependency guidance because that guidance is also returned for `mock`, and
+ *  a mock is deployed rather than run as a test — it needs no SDK. */
+const SDK_DECLARATION = {
+    python: "Declare `skyramp` in the same development manifest.",
+    javascript: `If @skyramp/skyramp is not already declared, add ${npmSdkSpec} as a devDependency. If it is declared, leave its version alone and report the mismatch if the verifier refuses it.`,
+    java: "Declare `dev.skyramp:skyramp-library` with test scope. An existing manifest resolves the version through its own dependency management; a manifest you created has none, so give a concrete version there.",
+};
+/** The SDK line for a language, or nothing when the language has none. */
+function sdkDeclaration(language) {
+    const key = language === "typescript" ? "javascript" : language;
+    const line = SDK_DECLARATION[key];
+    return line ? `\n${line}\n` : "";
+}
 export function getLanguageSteps(params) {
     const { language, testType } = params;
     if (testType === "mock") {
         switch (language) {
             case "python":
-                return `
-# Install skyramp dependency
-pip3 install skyramp
-
+                return `${pythonDependencyGuidance}
 # Run the mock server
 python3 <mock-file>.py
 `;
             case "typescript":
-                return `
-# Install dependencies
-npm install @skyramp/skyramp
-
+                return `${javascriptDependencyGuidance}
 # Run the mock server
 npx ts-node <mock-file>.ts
 `;
             case "javascript":
-                return `
-# Install dependencies
-npm install @skyramp/skyramp
-
+                return `${javascriptDependencyGuidance}
 # Run the mock server
 node <mock-file>.js
 `;
@@ -30,63 +68,40 @@ node <mock-file>.js
                 return "";
         }
     }
-    let pythonSteps = "";
+    let steps = "";
     switch (language) {
         case "python":
-            pythonSteps = `
-# Install skyramp dependency
-pip3 install skyramp
-      `;
+            steps = pythonDependencyGuidance + sdkDeclaration("python");
             // e2e and ui tests
             if (testType === "e2e" || testType === "ui") {
-                pythonSteps += `
-# Install playwright dependencies
-pip3 install pytest-playwright
+                steps += `
 # Execute the test file
 python3 -m pytest --browser chromium <test-file>.py
         `;
             }
             // framework pytest
             if (params.framework === "pytest") {
-                pythonSteps += `
-# Install pytest dependency
-pip3 install pytest
-
+                steps += `
 # Execute test file
 python3 -m pytest <test-file>.py
       `;
             }
             // robot framework
             if (params.framework === "robot") {
-                pythonSteps += `
-# Install robotframework dependency
-pip3 install robotframework
-
+                steps += `
 # Execution of test file
 python3 -m robot <robot-file>.robot
       `;
             }
             break;
         case "javascript":
-            pythonSteps = `
-# install dependencies for package.json
-npm install @skyramp/skyramp @playwright/test
-
-# Execution of test file
-npx playwright test <test-file>.spec.js --reporter=list
-      `;
+            steps = getJavascriptSteps("js");
             break;
         case "typescript":
-            pythonSteps = `
-# install dependencies for package.json
-npm install @skyramp/skyramp @playwright/test
-
-# Execution of test file
-npx playwright test <test-file>.spec.ts --reporter=list
-      `;
+            steps = getJavascriptSteps("ts");
             break;
         case "java":
-            pythonSteps = `
+            steps = `${javaDependencyGuidance}${sdkDeclaration("java")}
 # Prerequisites
 # Compile the test file
 javac -cp "./lib/*" -d target <test-file>.java
@@ -104,11 +119,11 @@ java -jar "./lib/junit-platform-console-standalone-\${JUNIT_PLATFORM_VER}.jar" \
         default:
             return "";
     }
-    return pythonSteps + appendExecutionInstruction();
+    return steps + appendExecutionInstruction();
 }
 function appendExecutionInstruction() {
     return `
-- Notify the user that they can also use ** Execute with Skyramp ** tool to run the test.If this is your first test you are trying to execute with Skyramp, it may take a little bit longer than usual because we need to build the image for the first time.
+- Notify the user that they can also use the Execute with Skyramp tool to run the test.
 - Before I can execute the test for you, I need to confirm the following information:
   Authentication: Please provide me with your authentication token (by default we assume a Bearer Token). If you don't require any token for your service please just say: 'No token required'
 `;

package/build/utils/pathMatching.js

@@ -49,7 +49,8 @@ export const isBareIdPath = (path) => {
     return (segs.length > 0 &&
         segs.every((seg) => {
             const canonical = canonicalSegment(seg);
-            return canonical.startsWith("{") && canonical.slice(1, -1).toLowerCase() === "id";
+            return (canonical.startsWith("{") &&
+                canonical.slice(1, -1).toLowerCase() === "id");
         }));
 };
 // Segment-boundary suffix match: avoids "/orders" matching "/preorders".

package/build/utils/pathSignatures.js

@@ -27,7 +27,9 @@ export function buildPathSignatures(file) {
     // import of "./Foo" — special-case the compound extension.
     const isDts = /\.d\.ts$/i.test(file);
     const ext = isDts ? ".d.ts" : path.extname(file);
-    const basename = isDts ? path.basename(file, ".d.ts") : path.basename(file, ext);
+    const basename = isDts
+        ? path.basename(file, ".d.ts")
+        : path.basename(file, ext);
     const quotes = [`'`, `"`, "`"];
     const stems = [`/${basename}`, `/${basename}${ext}`];
     // An index module is normally imported by its directory (`from './_constants'`,
@@ -43,7 +45,9 @@ export function buildPathSignatures(file) {
 /** True for `index.<ext>` modules, including `index.d.ts` (the same test buildPathSignatures uses). */
 export function isIndexModule(file) {
     const isDts = /\.d\.ts$/i.test(file);
-    return (isDts ? path.basename(file, ".d.ts") : path.basename(file, path.extname(file))) === "index";
+    return ((isDts
+        ? path.basename(file, ".d.ts")
+        : path.basename(file, path.extname(file))) === "index");
 }
 /**
  * True when `importerContent` (the source of `importerFile`) contains an import

package/build/utils/planMatchKeys.d.ts

@@ -11,7 +11,7 @@
  *    matching (contract tests, or a fallback when names drift between
  *    register and generate).
  */
-import { PlanScenario } from "../types/RepositoryAnalysis.js";
+import { PlannedScenario } from "../recommendation/types.js";
 import type { PlannedTest } from "../recommendation/types.js";
 /** Query shape built from a generation-tool call or a reported test entry. */
 export interface PlanMatchQuery {
@@ -34,7 +34,7 @@ export declare function normalizeMatchPath(rawPath: string | undefined): string;
 /** Name key (if a name exists) plus one endpoint key per step. BOTH kinds carry
  *  the planned test's test type, or a contract call matches an integration-only
  *  planned test sharing its name. A slug holds no `:`. */
-export declare function computeMatchKeys(scenario: PlanScenario): string[];
+export declare function computeMatchKeys(scenario: PlannedScenario): string[];
 /**
  * True when `query` matches `keys`:
  *  - by testType + scenarioName slug (`name:<testType>:<slug>`), or

package/build/utils/planMatchKeys.js

@@ -51,7 +51,9 @@ export function isOpaqueIdSegment(segment) {
     // The sign belongs to the value: an edge-case step writes `/orders/-1` to
     // probe a negative id, and without the sign that segment became the resource
     // name, so the key could never match real order coverage.
-    return UUID_RE.test(segment) || /^-?\d+$/.test(segment) || /^[0-9a-f]{16,}$/i.test(segment);
+    return (UUID_RE.test(segment) ||
+        /^-?\d+$/.test(segment) ||
+        /^[0-9a-f]{16,}$/i.test(segment));
 }
 /**
  * Normalize a path so param and id segments collapse to `:p`.
@@ -76,22 +78,20 @@ export function normalizeMatchPath(rawPath) {
     })
         .join("/");
 }
-/** The scenario is read back from the stored plan, so `steps` is an array only on
- *  the validated path. `?? []` keeps a malformed plan from costing the call. */
-function inferTestType(scenario) {
-    const steps = Array.isArray(scenario?.steps) ? scenario.steps : [];
-    return (scenario?.testType ?? (steps.length === 1 ? "contract" : "integration")).toLowerCase();
-}
 /** Name key (if a name exists) plus one endpoint key per step. BOTH kinds carry
  *  the planned test's test type, or a contract call matches an integration-only
  *  planned test sharing its name. A slug holds no `:`. */
 export function computeMatchKeys(scenario) {
     const keys = [];
-    const testType = inferTestType(scenario);
-    const slug = slugifyName(scenario?.scenarioName);
+    const testType = scenario.testType.toLowerCase();
+    const slug = slugifyName(scenario.scenarioName);
     if (slug)
         keys.push(`name:${testType}:${slug}`);
-    for (const step of Array.isArray(scenario?.steps) ? scenario.steps : []) {
+    // The steps come back off the run state file too, so the array the type declares
+    // holds only on the validated path. A corrupted one is no evidence about the
+    // scenario, and a throw here failed generation instead of allowing the call.
+    const steps = Array.isArray(scenario.steps) ? scenario.steps : [];
+    for (const step of steps) {
         if (!step?.method || !step?.path)
             continue;
         keys.push(`ep:${testType}:${step.method.toUpperCase()}:${normalizeMatchPath(step.path)}`);
@@ -127,8 +127,12 @@ function nameAgrees(keys, query) {
     const nameSlug = query.scenarioName ? slugifyName(query.scenarioName) : "";
     if (!nameSlug)
         return false;
-    const wanted = query.testType ? `name:${query.testType.toLowerCase()}:${nameSlug}` : undefined;
-    return keys.some((key) => wanted !== undefined ? key === wanted : key.startsWith("name:") && key.endsWith(`:${nameSlug}`));
+    const wanted = query.testType
+        ? `name:${query.testType.toLowerCase()}:${nameSlug}`
+        : undefined;
+    return keys.some((key) => wanted !== undefined
+        ? key === wanted
+        : key.startsWith("name:") && key.endsWith(`:${nameSlug}`));
 }
 /** Whether the plan holds this query's endpoint, ignoring the name. Shared by
  *  the name channel — which may no longer answer alone on an endpoint query —
@@ -222,12 +226,14 @@ export function nameMatchedTypeMismatches(plannedTests, query) {
         // The name channel, minus the type: a key ending in this slug means the plan
         // holds the scenario, and the segment before it is the type it holds it as.
         const named = keys.filter((key) => key.startsWith("name:") && key.endsWith(`:${nameSlug}`));
-        return named.length > 0 && !named.some((key) => key === `name:${calledType}:${nameSlug}`);
+        return (named.length > 0 &&
+            !named.some((key) => key === `name:${calledType}:${nameSlug}`));
     });
 }
 /** True when `query` matches at least one planned test of the registered plan.
  *  The keys are computed per call rather than stored with the plan, so there is
  *  one description of a planned test and no second shape to drift from it. */
 export function matchesPlanCandidates(plannedTests, query) {
-    return plannedTests.some((plannedTest) => !!plannedTest?.scenario && matchesMatchKeys(computeMatchKeys(plannedTest.scenario), query));
+    return plannedTests.some((plannedTest) => !!plannedTest?.scenario &&
+        matchesMatchKeys(computeMatchKeys(plannedTest.scenario), query));
 }

package/build/utils/pom-catalog-parse.js

@@ -36,7 +36,9 @@ function selectorsFrom(description) {
         }
         // A bare token is only a selector if the whole backticked run is one — otherwise
         // it is prose, or an attribute selector already harvested above.
-        if (!sawAttr && BARE_TOKEN_RE.test(token) && token.length >= MIN_BARE_TOKEN_LENGTH) {
+        if (!sawAttr &&
+            BARE_TOKEN_RE.test(token) &&
+            token.length >= MIN_BARE_TOKEN_LENGTH) {
             out.add(token);
         }
     }
@@ -125,7 +127,10 @@ function bareMemberName(member) {
  */
 export function findMissedReuse(specContent, members) {
     const locators = extractRawLocatorTokens(specContent);
-    const compiled = members.map((m) => ({ m, matchers: m.selectors.map(toMatcher) }));
+    const compiled = members.map((m) => ({
+        m,
+        matchers: m.selectors.map(toMatcher),
+    }));
     const missed = [];
     for (const locator of locators) {
         const hit = compiled.find(({ matchers }) => matchers.some((re) => re.test(locator)));

package/build/utils/pom-scope/import-expansion.js

@@ -19,7 +19,12 @@ export async function expandImports(selected, allPoms) {
             if (!ref || !ref.startsWith("."))
                 continue;
             const base = path.resolve(path.dirname(file), ref);
-            for (const cand of [base, `${base}.js`, `${base}.ts`, base.replace(/\.js$/, ".ts")]) {
+            for (const cand of [
+                base,
+                `${base}.js`,
+                `${base}.ts`,
+                base.replace(/\.js$/, ".ts"),
+            ]) {
                 if (pomSet.has(cand) && !seen.has(cand)) {
                     seen.add(cand);
                     out.push({ file: cand, referrer: file });

package/build/utils/pom-scope/index.js

@@ -1,6 +1,6 @@
 import { readFile, stat } from "fs/promises";
 import * as path from "path";
-import { extractSelectors, DEFAULT_TESTID_ATTRS } from "./selector-extractor.js";
+import { extractSelectors, DEFAULT_TESTID_ATTRS, } from "./selector-extractor.js";
 import { globPomFiles, globAllSourceFiles } from "./pom-files.js";
 import { scorePoms } from "./scoring.js";
 import { expandImports } from "./import-expansion.js";
@@ -9,17 +9,29 @@ import { stripComments } from "./strip.js";
 const DISCOVERY_FILE_CAP = 3000;
 export const SELECTION_CAP = 10;
 export const FINAL_CAP = 25;
-const PLAYWRIGHT_CONFIG_NAMES = ["playwright.config.ts", "playwright.config.js", "playwright.config.mjs", "playwright.config.cjs"];
+const PLAYWRIGHT_CONFIG_NAMES = [
+    "playwright.config.ts",
+    "playwright.config.js",
+    "playwright.config.mjs",
+    "playwright.config.cjs",
+];
 const TESTID_ATTRIBUTE_RE = /testIdAttribute\s*:\s*["']([^"']+)["']/;
 export async function findRoot(startDir) {
     let dir = startDir;
     for (let i = 0; i < 8; i++) {
-        for (const marker of ["package.json", "tsconfig.json", "jsconfig.json", ".git"]) {
+        for (const marker of [
+            "package.json",
+            "tsconfig.json",
+            "jsconfig.json",
+            ".git",
+        ]) {
             try {
                 await stat(path.join(dir, marker));
                 return dir;
             }
-            catch { /* keep walking */ }
+            catch {
+                /* keep walking */
+            }
         }
         const parent = path.dirname(dir);
         if (parent === dir)
@@ -39,7 +51,9 @@ export async function readTestIdAttribute(root) {
             if (m)
                 return m[1];
         }
-        catch { /* try next filename */ }
+        catch {
+            /* try next filename */
+        }
     }
     return undefined;
 }
@@ -49,7 +63,9 @@ export async function readTestIdAttribute(root) {
  * the exact-tie case in index.test.ts.
  */
 export function tierAndCap(candidates, maxScore) {
-    const kept = [...candidates].sort((a, b) => b.score - a.score).slice(0, FINAL_CAP);
+    const kept = [...candidates]
+        .sort((a, b) => b.score - a.score)
+        .slice(0, FINAL_CAP);
     const isTier1 = (c) => c.score > maxScore / 4 || c.rareHits >= 2;
     const tier1 = kept.filter(isTier1);
     const tier2 = kept.filter((c) => !isTier1(c));
@@ -57,16 +73,24 @@ export function tierAndCap(candidates, maxScore) {
 }
 /** Deterministic POM pre-scoping. tier1/tier2 both [] ⇒ caller must fall back to the unscoped prompt. */
 export async function selectScopedPoms(opts) {
-    const empty = (selectors = 0, pomFiles = 0, discovery = "none") => ({ tier1: [], tier2: [], diagnostics: { selectors, pomFiles, scoped: 0, discovery } });
+    const empty = (selectors = 0, pomFiles = 0, discovery = "none") => ({
+        tier1: [],
+        tier2: [],
+        diagnostics: { selectors, pomFiles, scoped: 0, discovery },
+    });
     let spec = opts.specContent ?? "";
     // Prefer the on-disk file (the agent may have rewritten it); fall back to the passed prompt content.
     try {
         spec = await readFile(opts.testFile, "utf8");
     }
-    catch { /* use provided specContent */ }
+    catch {
+        /* use provided specContent */
+    }
     const root = await findRoot(path.dirname(opts.testFile));
     const testIdAttribute = await readTestIdAttribute(root);
-    const extraTestIdAttrs = testIdAttribute && !DEFAULT_TESTID_ATTRS.includes(testIdAttribute) ? [testIdAttribute] : [];
+    const extraTestIdAttrs = testIdAttribute && !DEFAULT_TESTID_ATTRS.includes(testIdAttribute)
+        ? [testIdAttribute]
+        : [];
     const selectors = extractSelectors(spec, extraTestIdAttrs);
     if (selectors.length === 0)
         return empty();
@@ -111,19 +135,33 @@ export async function selectScopedPoms(opts) {
     const candidates = expanded.flatMap(({ file, referrer }) => {
         const own = rankedByFile.get(file);
         if (own)
-            return [{ file: own.file, score: own.score, rareHits: own.rareHits, matchedTokens: own.matchedTokens }];
+            return [
+                {
+                    file: own.file,
+                    score: own.score,
+                    rareHits: own.rareHits,
+                    matchedTokens: own.matchedTokens,
+                },
+            ];
         // Invariant: every expansion addition without an own rank carries the referrer that
         // pulled it in (expandImports always tags non-selected files this way). Skip
         // defensively rather than assert, in case that invariant is ever violated.
         const referrerEntry = referrer ? rankedByFile.get(referrer) : undefined;
         if (!referrerEntry)
             return [];
-        return [{ file, score: referrerEntry.score, rareHits: 0, matchedTokens: [] }];
+        return [
+            { file, score: referrerEntry.score, rareHits: 0, matchedTokens: [] },
+        ];
     });
     const { tier1, tier2 } = tierAndCap(candidates, maxScore);
     return {
         tier1,
         tier2,
-        diagnostics: { selectors: selectors.length, pomFiles: pomFiles.length, scoped: tier1.length + tier2.length, discovery },
+        diagnostics: {
+            selectors: selectors.length,
+            pomFiles: pomFiles.length,
+            scoped: tier1.length + tier2.length,
+            discovery,
+        },
     };
 }

package/build/utils/pom-scope/scoring.js

@@ -20,12 +20,16 @@ export async function scorePoms(selectors, files) {
         try {
             contents.set(f, (await readFile(f, "utf8")).replace(/\\/g, ""));
         }
-        catch { /* unreadable file: skip */ }
+        catch {
+            /* unreadable file: skip */
+        }
     }));
     // Selectors arrive pre-normalized from extractSelectors — only file contents need stripping.
     const fileMatches = new Map(); // selector -> files containing it
     for (const sel of selectors) {
-        const matched = [...contents.entries()].filter(([, c]) => c.includes(sel)).map(([f]) => f);
+        const matched = [...contents.entries()]
+            .filter(([, c]) => c.includes(sel))
+            .map(([f]) => f);
         if (matched.length > 0)
             fileMatches.set(sel, matched);
     }
@@ -33,7 +37,13 @@ export async function scorePoms(selectors, files) {
     for (const [sel, matched] of fileMatches) {
         const df = matched.length;
         for (const f of matched) {
-            const s = scores.get(f) ?? { file: f, score: 0, hits: 0, rareHits: 0, matchedTokens: [] };
+            const s = scores.get(f) ?? {
+                file: f,
+                score: 0,
+                hits: 0,
+                rareHits: 0,
+                matchedTokens: [],
+            };
             s.score += 1 / df;
             s.hits += 1;
             if (df <= RARE_DF)

package/build/utils/pom-scope/selector-extractor.js

@@ -5,7 +5,12 @@
  */
 // Default test-id attribute family. Playwright's own default is data-testid, but
 // data-test/data-qa/data-cy are common conventions in the wild — match them all.
-export const DEFAULT_TESTID_ATTRS = ["data-test", "data-testid", "data-qa", "data-cy"];
+export const DEFAULT_TESTID_ATTRS = [
+    "data-test",
+    "data-testid",
+    "data-qa",
+    "data-cy",
+];
 // Full regex-escape for attr names — a config-supplied testIdAttribute could contain
 // a stray metachar, which must not be allowed to distort the pattern.
 import { escapeRegExp as esc } from "../regex.js";
@@ -20,7 +25,10 @@ const attrAnchor = (attrs) => new RegExp(`\\[(?:${attrs.map(esc).join("|")})[$^*
 // locator values, shared by both extractSelectors and extractRawLocatorTokens.
 const GET_BY_TEST_ID = /getByTestId\(\s*["']([^"']+)["']/g;
 const GET_BY_TEXT_LABEL_PLACEHOLDER = /getBy(?:Text|Label|Placeholder)\(\s*["']([^"']+)["']/g;
-const LOCATOR_PATTERNS = [GET_BY_TEST_ID, GET_BY_TEXT_LABEL_PLACEHOLDER];
+const LOCATOR_PATTERNS = [
+    GET_BY_TEST_ID,
+    GET_BY_TEXT_LABEL_PLACEHOLDER,
+];
 // getByRole("role", ...) and bare `name: "..."` object keys are useful evidence for
 // POM *detection* (scoring which page objects a test's actions touch), but they are
 // NOT raw locator values: a role name isn't a selector, and `name:` matches ANY bare
@@ -28,7 +36,12 @@ const LOCATOR_PATTERNS = [GET_BY_TEST_ID, GET_BY_TEXT_LABEL_PLACEHOLDER];
 // finding 1). Only extractSelectors (the detection path) includes these.
 const GET_BY_ROLE = /getByRole\(\s*["']([^"']+)["']/g;
 const BARE_NAME_FIELD = /name:\s*["']([^"']+)["']/g;
-const OTHER_PATTERNS = [GET_BY_TEST_ID, GET_BY_ROLE, BARE_NAME_FIELD, GET_BY_TEXT_LABEL_PLACEHOLDER];
+const OTHER_PATTERNS = [
+    GET_BY_TEST_ID,
+    GET_BY_ROLE,
+    BARE_NAME_FIELD,
+    GET_BY_TEXT_LABEL_PLACEHOLDER,
+];
 // Raw .locator("...")/.frameLocator("...") strings that are NOT test-id-attribute based
 // (e.g. ".step-deduplication", "iframe#asset-list").
 // Quote-type-aware alternates: a double-quoted arg may contain unescaped single quotes

package/build/utils/pom-scope/strip.d.ts

@@ -1,3 +1,11 @@
+/** Every line- and block-comment span in `src`, in source order. Exported for callers
+ * that must decide whether a `//` OPENS a comment rather than sits inside one: a `//`
+ * on its own line inside `/* … *\/` opens nothing, and deleting that line can take the
+ * block's terminator with it. */
+export declare function commentSpans(src: string): {
+    start: number;
+    end: number;
+}[];
 /** Blanks line/block comments only, preserving string/template literal contents (e.g. so an
  * import specifier's quoted path stays readable). Each blanked character becomes a space
  * (newlines preserved) so remaining code characters keep their original offset. */