@expressivecss/mcp-server 0.3.0 → 0.3.1

package/server.js

@@ -5,7 +5,7 @@ import path from 'node:path';
 import { fileURLToPath, pathToFileURL } from 'node:url';
 import { createHash, randomUUID } from 'node:crypto';
 import { spawn } from 'node:child_process';
-import { JSDOM } from 'jsdom';
+import { createRequire } from 'node:module';
 import * as z from 'zod/v4';
 import { McpServer, ResourceTemplate } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
@@ -104,7 +104,7 @@ const LegacyPatternList = [
     id: 'legacy-card-content',
     severity: 'high',
     description: '`.card-content` was removed; use the component’s documented child structure.',
-    pattern: /\bcard-content\b/g,
+    pattern: /(?<![-\w])card-content(?![-\w])/g,
   },
   {
     id: 'legacy-nav-wrapper',
@@ -1060,6 +1060,8 @@ function inspectSemanticRules(snippet, maxIssues = MAX_STATIC_ISSUES, deadline =
   if (markupStructureExceedsLimit(snippet)) return markInspectionTruncated([], 'markup structure limit reached');
   const prefix = '<!doctype html><body>';
   const includeNodeLocations = snippet.length * (snippet.split('<').length - 1) <= MAX_LOCATED_MARKUP_WORK;
+  // Loaded on first use: jsdom takes ~0.3 s to load, and most server starts never inspect markup.
+  const { JSDOM } = createRequire(import.meta.url)('jsdom');
   const dom = new JSDOM(`${prefix}${snippet}</body>`, { includeNodeLocations });
   const { document } = dom.window;
   // Elements the parser creates without a start tag have no source offset, so they report the start.

package/smoke.mjs

@@ -220,9 +220,18 @@ async function verifyResponseBudgets(matchingDir, versionedDir) {
     const result = await call(catalogRequest);
     assert.notEqual(result.structuredContent.responseBudget.delivery, 'complete');
   });
-  await withServer(20_000, async ({ call }) => {
+  // A partial listing needs room for one recovery record per omitted entry, and
+  // that accounting grows with the catalogue. Derive the budget from the
+  // complete listing so it stays below it while the accounting still fits; a
+  // fixed figure turns into the documented error once enough components exist.
+  let listingBytes;
+  await withServer(1_048_576, async ({ call }) => {
+    listingBytes = bytes(await call(request('component_catalog')));
+  });
+  await withServer(listingBytes - 3_000, async ({ call }) => {
     const listing = await call(request('component_catalog'));
     assert.equal(listing.structuredContent.responseBudget.delivery, 'partial');
+    assert.ok(listing.structuredContent.responseBudget.omissions.length > 1, 'the listing omits several entries');
     assert.equal(listing.structuredContent.count + listing.structuredContent.responseBudget.omissions.length, guideData.guides.length);
     for (const recovery of listing.structuredContent.responseBudget.recoveries) {
       assert.equal(recovery.action, 'retry');
@@ -230,6 +239,8 @@ async function verifyResponseBudgets(matchingDir, versionedDir) {
       assert.equal(recovered.structuredContent.entries[0].slug, recovery.request.arguments.query);
       assert.equal(recovered.structuredContent.responseBudget.delivery, 'complete');
     }
+  });
+  await withServer(20_000, async ({ call }) => {
     const duplicate = await call(request('component_syntax_expert', { projectRoot: matchingDir, components: ['cards', 'cards', 'unknown-name', 'autocomplete'], sections: ['options', 'methods'] }));
     assert.equal(duplicate.structuredContent.missing[0].requested, 'unknown-name');
     for (const omitted of duplicate.structuredContent.responseBudget.omissions.filter((row) => row.slug === 'cards')) assert.ok([0, 1].includes(omitted.index));
@@ -958,6 +969,12 @@ const transport = new StdioClientTransport({
   assert.match(long.stdout, /long\.html:1:1 nav-needs-label/u);
   assert.doesNotMatch(long.stdout, /inspection-truncated/u);
   assert.equal(lint([outsideFile]).status, 0);
+  // The retired class name matches only as a whole class, not inside documented names that contain it.
+  const cardContentMarkup = path.join(outsideDir, 'card-content.html');
+  await writeFile(cardContentMarkup, '<style>article { --md-comp-card-content-padding: 24px; }</style>\n<div class="expanding-card-content">Details</div>\n<div class="card-content">Old</div>\n');
+  const cardContent = lint([cardContentMarkup]);
+  assert.equal(cardContent.stdout.match(/legacy-card-content/gu)?.length, 1, cardContent.stdout);
+  assert.match(cardContent.stdout, /card-content\.html:3:\d+ legacy-card-content/u);
   // Files the bounded reader refuses, and inspections that stop early, fail instead of passing silently.
   const linkedMarkup = path.join(outsideDir, 'linked.html');
   const crowdedMarkup = path.join(outsideDir, 'crowded.html');