@letsrunit/mcp-server 0.10.0 → 0.11.0

package/dist/index.js

@@ -7,14 +7,14 @@ import { MemorySink, Journal } from '@letsrunit/journal';
 import { join, dirname, resolve, isAbsolute } from 'path';
 import { z } from 'zod';
 import { loadConfiguration } from '@cucumber/cucumber/api';
-import { locatorParameter, valueParameter, keysParameter, booleanParameter, enumParameter, sanitizeStepDefinition, parseFeature } from '@letsrunit/gherkin';
+import { locatorParameter, valueParameter, keysParameter, booleanParameter, enumParameter, sanitizeStepDefinition, scenarioIdFromGherkin } from '@letsrunit/gherkin';
 import { scrubHtml, screenshotElement, screenshot, unifiedHtmlDiff, fuzzyLocator, suppressInterferences, waitForIdle, waitAfterInteraction, setFieldValue } from '@letsrunit/playwright';
 import { expect } from '@playwright/test';
 import { eventually, splitUrl, pathRegexp, sleep, textToHtml, asFilename, clean } from '@letsrunit/utils';
 import { readFileSync, existsSync } from 'fs';
 import { mkdir, writeFile, glob } from 'fs/promises';
 import { pathToFileURL } from 'url';
-import { openStore, findLastRun, findArtifacts, computeStepId, computeScenarioId } from '@letsrunit/store';
+import { openStore, findLastTest, findArtifacts } from '@letsrunit/store';
 import { execSync } from 'child_process';
 
 var __create = Object.create;
@@ -74933,7 +74933,7 @@ function expectOrNot(actual, toBe) {
 // ../bdd/src/steps/assert.ts
 var WAIT_TIMEOUT = 5e3;
 Then(
-  "The page {contains|does not contain} {locator}",
+  "the page {contains|does not contain} {locator}",
   async function(visible, selector) {
     const el = await fuzzyLocator(this.page, selector);
     await expectOrNot(el, visible).toBeVisible({ timeout: WAIT_TIMEOUT });
@@ -79358,11 +79358,11 @@ function registerDiff(server2, sessions2) {
   server2.registerTool(
     "letsrunit_diff",
     {
-      description: "Diff the current live page against the HTML snapshot from the last passing run of a scenario. Pass the scenarioId returned by letsrunit_run. Returns a unified HTML diff and paths to baseline screenshots. By default only considers baseline runs from the current git ancestry (gitTreeOnly: true).",
+      description: "Diff the current live page against the HTML snapshot from the last passing test of a scenario. Pass the scenarioId returned by letsrunit_run. Returns a unified HTML diff and paths to baseline screenshots. By default only considers baseline tests from the current git ancestry (gitTreeOnly: true).",
       inputSchema: {
         sessionId: z.string().describe("Session ID returned by letsrunit_session_start"),
         scenarioId: z.string().describe("Scenario UUID returned by letsrunit_run"),
-        gitTreeOnly: z.boolean().optional().describe("Restrict baseline to runs from the current git ancestry (default: true)")
+        gitTreeOnly: z.boolean().optional().describe("Restrict baseline to tests from the current git ancestry (default: true)")
       }
     },
     async (input) => {
@@ -79376,16 +79376,16 @@ function registerDiff(server2, sessions2) {
           return err("Could not open the letsrunit store. Run cucumber with the store formatter first.");
         }
         const allowedCommits = input.gitTreeOnly ?? true ? resolveAllowedCommits() : void 0;
-        const run = findLastRun(db, input.scenarioId, "passed", allowedCommits ?? void 0);
-        if (!run) {
+        const test = findLastTest(db, input.scenarioId, "passed", allowedCommits ?? void 0);
+        if (!test) {
           return err(
-            allowedCommits ? "No passing run found for this scenario in the current git ancestry. Try gitTreeOnly: false or run cucumber first." : "No passing run found for this scenario."
+            allowedCommits ? "No passing test found for this scenario in the current git ancestry. Try gitTreeOnly: false or run cucumber first." : "No passing test found for this scenario."
           );
         }
-        const artifacts = findArtifacts(db, run.id);
+        const artifacts = findArtifacts(db, test.id);
         const htmlArtifact = [...artifacts].reverse().find((a) => a.filename.endsWith(".html"));
         if (!htmlArtifact) {
-          return err("No HTML snapshot found in the baseline run. Ensure the store formatter is configured.");
+          return err("No HTML snapshot found in the baseline test. Ensure the store formatter is configured.");
         }
         const storedHtml = readFileSync(join(artifactDir, htmlArtifact.filename), "utf-8");
         const session = sessions2.get(input.sessionId);
@@ -79396,8 +79396,8 @@ function registerDiff(server2, sessions2) {
           JSON.stringify({
             diff,
             baseline: {
-              runId: run.id,
-              commit: run.gitCommit,
+              testId: test.id,
+              commit: test.gitCommit,
               screenshots
             }
           })
@@ -79468,11 +79468,6 @@ Scenario: Steps
 }
 
 // src/tools/run.ts
-function scenarioIdFromGherkin(gherkin) {
-  const { steps } = parseFeature(gherkin);
-  const stepIds = steps.map((s) => computeStepId(s));
-  return computeScenarioId(stepIds);
-}
 function registerRun(server2, sessions2) {
   server2.registerTool(
     "letsrunit_run",