fix: repair get_component_html, upgrade SDK 0.5 -> 1.29 and refresh toolchain (#11)

Two pre-existing bugs made get_component_html — the reason this server exists —
fail on most real design systems. Both were found while verifying the
dependency upgrade against a real Storybook 10 build, and both reproduce on
main with the old SDK and Puppeteer.

1. The render check required more than 100 characters of HTML

   A rendered button is 76 characters. Badges, chips, icons and inputs are
   smaller still, so the poll loop never accepted them, ran its full 15 x 1s,
   and blew past the 10s caller timeout. Every compact component — the bulk of
   a design system — returned "Operation timed out after 10000ms".

   Content now counts as rendered when it is non-empty and is not Storybook's
   own no-preview placeholder. The loop is also bounded by a deadline instead
   of an attempt count that could exceed the caller's budget, and polls every
   100ms instead of sleeping a full second before even looking.

2. Concurrent jobs corrupted each other

   The job queue runs maxConcurrent = 2, but PuppeteerClient shared a single
   page: two simultaneous fetches navigated the same tab and whichever started
   second tore the first one's document away. On top of that,
   getPuppeteerClient() had an async-init race — two callers both saw a null
   field, both launched a browser, and the second assignment orphaned the
   first mid-request. Either way one of any two parallel jobs failed with a
   spurious connection error.

   Each fetch now gets its own page (closed in a finally, so tabs cannot leak)
   and the launch caches its in-flight promise rather than the resolved client.

Verified against Storybook 10.5.4: before, one job timed out at 10s and the
other failed; after, two concurrent extractions both complete in ~2.4s with
correct HTML.

Toolchain, which is what prompted all of the above:

- @modelcontextprotocol/sdk 0.5.0 -> 1.29.0. The server was two years behind
  on a pre-1.0 SDK; it now negotiates protocol 2025-06-18. No API changes were
  needed, the low-level Server surface is unchanged.
- Node 18 dropped (EOL) — engines, CI matrix now 20/22/24, publish workflow
  and tsup target follow.
- eslint 8 -> 10 with a flat config matching the other repos, typescript-eslint
  6 -> 8, vitest 3 -> 4, puppeteer 24 -> 25, node-html-parser 6 -> 9, zod 3 -> 4,
  @types/node 20 -> 26.
- eslint 10's new rules found real defects, fixed rather than silenced: five
  throw sites discarded the caught error instead of chaining it via `cause`,
  and isSSLCertificateError() ran String() over an unknown value, so a thrown
  plain object stringified to "[object Object]" and every certificate check
  silently missed.
- Server version was hardcoded in index.ts and had already drifted (1.1.0 there
  vs 1.1.1 in package.json); it is now injected from package.json at build.
- Coverage: added the lcov reporter and coverage.all, so CI can upload and the
  denominator is the whole source tree rather than whatever tests imported.
  Thresholds are set to today's real numbers, which are ~6%. They were 80 but
  never enforced, because CI ran `vitest run` without --coverage.
- server.json plus mcpName and the registry publish workflow, so the package
  appears in the MCP Registry like mcp-gsheets and openclaw-mcp.
- README documents Storybook 7-10 support, verified by building Storybook
  10.5.4 and extracting from it.

21 tests pass. The new ones pin the length regression directly: a 76-character
button must read as rendered, and a long no-preview placeholder must not.

Author: freema

.eslintrc.json

@@ -16,7 +16,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        node-version: [18, 20, 22]
+        node-version: [20, 22, 24]
 
     steps:
       - name: Checkout
@@ -41,14 +41,14 @@ jobs:
         run: npm run typecheck
 
       - name: Test
-        run: npm run test:run
+        run: npm run test:coverage
 
       - name: Build
         run: npm run build
 
       - name: Upload coverage
-        if: matrix.node-version == 20 && hashFiles('coverage/lcov.info') != ''
-        uses: codecov/codecov-action@v4
+        if: matrix.node-version == 20
+        uses: codecov/codecov-action@v7
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
           files: ./coverage/lcov.info

.github/workflows/ci.yml

@@ -0,0 +1,59 @@
+name: Publish to MCP Registry
+
+on:
+  release:
+    types: [published]
+  workflow_dispatch:
+
+jobs:
+  publish:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      id-token: write
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Sync server.json version with release tag
+        run: |
+          VERSION="${GITHUB_REF_NAME#v}"
+          if [ -z "$VERSION" ] || [ "$VERSION" = "$GITHUB_REF_NAME" ]; then
+            VERSION=$(node -p "require('./package.json').version")
+          fi
+          echo "Publishing version: $VERSION"
+          node -e "
+            const fs = require('fs');
+            const s = JSON.parse(fs.readFileSync('server.json', 'utf8'));
+            s.version = '$VERSION';
+            for (const p of s.packages || []) p.version = '$VERSION';
+            fs.writeFileSync('server.json', JSON.stringify(s, null, 2) + '\n');
+          "
+          cat server.json
+
+      - name: Wait for npm package to be available
+        run: |
+          VERSION=$(node -p "require('./server.json').version")
+          echo "Waiting for mcp-design-system-extractor@$VERSION on npm..."
+          for i in $(seq 1 60); do
+            if npm view "mcp-design-system-extractor@$VERSION" version 2>/dev/null | grep -q "^$VERSION$"; then
+              echo "Package available."
+              exit 0
+            fi
+            echo "Not yet (attempt $i/60), sleeping 10s..."
+            sleep 10
+          done
+          echo "Timed out waiting for npm package."
+          exit 1
+
+      - name: Install mcp-publisher
+        run: |
+          curl -L "https://github.com/modelcontextprotocol/registry/releases/latest/download/mcp-publisher_linux_amd64.tar.gz" \
+            | tar xz mcp-publisher
+          chmod +x mcp-publisher
+
+      - name: Login to MCP Registry (GitHub OIDC)
+        run: ./mcp-publisher login github-oidc
+
+      - name: Publish to MCP Registry
+        run: ./mcp-publisher publish

.github/workflows/publish-mcp-registry.yml

@@ -21,7 +21,7 @@ jobs:
       - name: Use Node.js 18.x
         uses: actions/setup-node@v4
         with:
-          node-version: 18
+          node-version: 20
           registry-url: 'https://registry.npmjs.org/'
           cache: 'npm'
           always-auth: true

.github/workflows/publish.yml

@@ -257,9 +257,22 @@ Connects to Storybook via `/index.json` and `/iframe.html` endpoints. Uses Puppe
 
 ## Requirements
 
-- Node.js 18+
+- Node.js 20+
 - Chrome/Chromium (for Puppeteer)
-- Running Storybook instance
+- Running Storybook instance (see below for supported versions)
+
+### Supported Storybook versions
+
+**Storybook 7, 8, 9 and 10.** The server reads the story index from
+`/index.json`, falling back to `/stories.json`, and renders stories through
+`/iframe.html?id=<storyId>` — endpoints that have been stable across all four
+major versions.
+
+Storybook 6 and earlier are not supported: they predate `/index.json` and use
+a different story-id scheme.
+
+Both a dev server (`npm run storybook`) and a built static Storybook served
+over HTTP will work.
 
 ## Development
 

README.md

@@ -0,0 +1,65 @@
+import eslint from '@eslint/js';
+import tseslint from 'typescript-eslint';
+import prettierConfig from 'eslint-config-prettier';
+import prettierPlugin from 'eslint-plugin-prettier';
+import globals from 'globals';
+
+export default tseslint.config(
+  {
+    ignores: ['dist/**', 'node_modules/**', '**/*.js', 'scripts/**'],
+  },
+  {
+    files: ['src/**/*.ts'],
+    extends: [
+      eslint.configs.recommended,
+      ...tseslint.configs.recommended,
+      ...tseslint.configs.recommendedTypeChecked,
+      prettierConfig,
+    ],
+    plugins: {
+      prettier: prettierPlugin,
+    },
+    languageOptions: {
+      globals: {
+        ...globals.node,
+        ...globals.es2022,
+      },
+      parserOptions: {
+        project: './tsconfig.json',
+        sourceType: 'module',
+      },
+    },
+    rules: {
+      'prettier/prettier': 'error',
+      '@typescript-eslint/explicit-function-return-type': 'off',
+      '@typescript-eslint/explicit-module-boundary-types': 'off',
+      '@typescript-eslint/no-explicit-any': 'off',
+      '@typescript-eslint/no-unsafe-assignment': 'off',
+      '@typescript-eslint/no-unsafe-member-access': 'off',
+      '@typescript-eslint/no-unsafe-call': 'off',
+      '@typescript-eslint/no-unsafe-return': 'off',
+      '@typescript-eslint/no-unsafe-argument': 'off',
+      '@typescript-eslint/ban-ts-comment': 'off',
+      '@typescript-eslint/no-unused-vars': [
+        'error',
+        {
+          argsIgnorePattern: '^_',
+          varsIgnorePattern: '^_',
+        },
+      ],
+      '@typescript-eslint/no-floating-promises': 'error',
+      '@typescript-eslint/no-misused-promises': 'error',
+      '@typescript-eslint/await-thenable': 'error',
+      '@typescript-eslint/no-unnecessary-type-assertion': 'error',
+      '@typescript-eslint/no-empty-object-type': 'off',
+      '@typescript-eslint/require-await': 'off',
+      '@typescript-eslint/prefer-nullish-coalescing': 'off',
+      '@typescript-eslint/prefer-optional-chain': 'warn',
+      'no-console': ['warn', { allow: ['error', 'warn'] }],
+      'prefer-const': 'error',
+      'no-var': 'error',
+      eqeqeq: ['error', 'always'],
+      curly: ['error', 'all'],
+    },
+  },
+);