@extension.dev/mcp 10.10.4 → 10.10.5

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
       "name": "extension-mcp",
       "source": "./",
       "description": "MCP tools for browser extension development: scaffold from 50+ templates, run the dev server with HMR, inspect the live DOM and logs, and publish store-ready builds for Chrome, Edge, and Firefox.",
-      "version": "10.10.4",
+      "version": "10.10.5",
       "category": "development",
       "author": {
         "name": "Cezar Augusto"

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "extension-mcp",
   "description": "MCP tools for browser extension development: scaffold from 50+ templates, run the dev server with HMR, inspect the live DOM and logs, and publish store-ready builds for Chrome, Edge, and Firefox. Ships /extension, /extension-add, /extension-debug, and /extension-publish commands.",
-  "version": "10.10.4",
+  "version": "10.10.5",
   "author": {
     "name": "Cezar Augusto",
     "email": "hello@extension.dev",

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # Changelog
 
+## 10.10.5
+
+- `extension_build` and `extension_analyze` report store review risks
+  read off the built package, as `value.reviewRisks` plus one warning
+  each with the fix: access to every website, code the package did not
+  ship (`eval`, `new Function`, scripts or imports loaded from a URL), a
+  Firefox manifest without `data_collection_permissions`, and API
+  permissions no shipped script uses. They never block a build. On a
+  dev build only the manifest checks run, because the hot-reload runtime
+  and the injected `management` permission are not what a store
+  receives (measured on seven published extensions' dev builds against a
+  production build of an official template).
+
+- The bundled `/extension-publish` command now takes an extension through
+  store review with this server's own tools instead of sending people to
+  the store consoles by hand: it validates, builds, checks for review
+  risks, rehearses with `extension_submit` (dry run), and submits only on
+  the user's yes through the approval link. It covers Chrome, Firefox,
+  Edge and Safari, and keeps whatever manifest version the project
+  declares rather than migrating it during a submission.
+
 ## 10.10.4
 
 - `--project <workspace>/<project>` (env `EXTENSION_DEV_PROJECT`) pins a

package/claude/commands/extension-publish.md

@@ -1,62 +1,39 @@
 ---
-description: Prepare an extension for store submission (Chrome Web Store, Firefox Add-ons)
-argument-hint: "[chrome|firefox|both]"
+description: Get an extension through store review (Chrome Web Store, Firefox AMO, Edge Add-ons, App Store for Safari)
+argument-hint: "[chrome|firefox|edge|safari|all]"
 ---
 
-Prepare the current extension for store submission. The user said: $ARGUMENTS
+Take the current extension to store review. The user said: $ARGUMENTS
 
 ## Parse arguments
 
-Default to `both` (Chrome + Firefox). If the user specifies `chrome` or `firefox`, target only that store.
+Default to `chrome` and `firefox`. `all` means chrome, firefox, edge and safari. Accept any comma-separated mix.
 
 ## Steps
 
-1. **Validate the manifest**
-   If MCP tool `extension_manifest_validate` is available, use it with the target browsers.
-   Otherwise, read `src/manifest.json` and check:
-   - Has `name`, `version`, `description`
-   - Has appropriate `manifest_version` for each target
-   - Permissions are minimal (no unnecessary permissions)
-   - Has icons (at least 16x16, 48x48, 128x128)
-
-2. **Build for each target browser**
-
-   ```bash
-   npx extension build --browser=chrome --zip
-   npx extension build --browser=firefox --zip
-   ```
-
-3. **Inspect the builds**
-   If MCP tool `extension_analyze` is available, use it for each browser build.
-   Check:
-   - Total size under 10MB (store limit)
-   - No source maps in production build
-   - Has manifest.json in dist
-   - Has icons
-
-4. **Report store readiness**
-
-   ### Chrome Web Store
-   - Zip location: `dist/chrome/<name>.zip`
-   - Submit at: https://chrome.google.com/webstore/devconsole
-   - Checklist:
-     - [ ] manifest_version: 3
-     - [ ] Icons: 128x128 PNG
-     - [ ] Description under 132 characters (for listing)
-     - [ ] Screenshots: 1280x800 or 640x400
-     - [ ] Privacy policy URL (if using sensitive permissions)
-
-   ### Firefox Add-ons (AMO)
-   - Zip location: `dist/firefox/<name>.zip`
-   - Submit at: https://addons.mozilla.org/developers/
-   - Checklist:
-     - [ ] manifest_version: 2 (recommended for broadest compat) or 3
-     - [ ] No Chrome-only APIs without polyfill
-     - [ ] Source code zip if using a bundler: `npx extension build --browser=firefox --zip --zip-source`
-     - [ ] AMO requires source code review for minified/bundled code
-
-5. **Flag issues** that would cause store rejection:
-   - `<all_urls>` host permission without justification
-   - `activeTab` + `scripting` without clear use case
-   - Remote code loading (eval, Function constructor, remote scripts)
-   - Excessive permissions for the extension's functionality
+1. **Check the manifest.** Call `extension_manifest_validate` for each target browser. Fix every `buildBlocking` finding before going on; report warnings.
+
+2. **Build store bundles.** Call `extension_build` with `zip: true` for each target. If it refuses on a blocking manifest error or returns compiler errors in `value.errors`, fix and rebuild.
+
+3. **Look for review risks.** Call `extension_analyze` on each build and flag anything a reviewer rejects:
+   - `<all_urls>` or broad host permissions without a reason in the listing
+   - remote code (`eval`, `new Function`, scripts loaded from a URL)
+   - permissions the code never uses
+   - Firefox: a missing `browser_specific_settings.gecko.data_collection_permissions` declaration, which AMO now requires for new add-ons and updates
+   - a bundle over 10 MB, source maps in the production build, missing 128px icon
+
+4. **Pick the build to submit.** Store review runs from a build on extension.dev, not from local files. Call `extension_release_status` with `include: "releases"` to find the sha. If the user has not shipped this version yet, say so and offer `extension_publish` first.
+
+5. **Rehearse.** Call `extension_submit` with the browsers and `buildSha`, leaving `dryRun` at its default (`true`). Show the per-store credential rows. A store whose credentials are not healthy is fixed in the extension.dev console (the response names the page); drop it from `browsers` or stop.
+
+6. **Submit only on a clear yes.** Ask the user to confirm the exact stores, build and channel. Then call `extension_submit` with `dryRun: false`.
+   - It answers `approval-required` with an `approvalUrl`: give the user that link. A workspace owner approves exactly this submission on extension.dev.
+   - When they say it is approved, call `extension_submit` again with the same arguments plus the returned `approvalId`. It runs once.
+
+7. **Report.** Call `extension_release_status` for the recorded outcome and review state per store, and give the user each store's listing or dashboard link from the response.
+
+## Rules
+
+- Never submit without the user's confirmation in step 6, and never retry a real submission with a different build or store list under an old `approvalId`; request a new approval.
+- Keep the manifest version the project already declares. Extension.js templates use `chromium:manifest_version: 3` and `firefox:manifest_version: 2`, and AMO accepts both; do not migrate a manifest as part of a submission.
+- Store credentials are never arguments and never asked for in chat; they live in the extension.dev console.

package/dist/module.js

@@ -236,7 +236,7 @@ __webpack_require__.d(workspace_create_namespaceObject, {
     handler: ()=>workspace_create_handler,
     schema: ()=>workspace_create_schema
 });
-var package_namespaceObject = JSON.parse('{"rE":"10.10.4","El":{"OP":"4.1.31"}}');
+var package_namespaceObject = JSON.parse('{"rE":"10.10.5","El":{"OP":"4.1.31"}}');
 const ENVELOPE_SCHEMA = 1;
 const collectWarnings = (warnings)=>{
     if (!warnings) return [];
@@ -3769,6 +3769,229 @@ function readZipEntryNames(zipPath) {
         } catch  {}
     }
 }
+const BROAD_PATTERNS = new Set([
+    "<all_urls>",
+    "*://*/*",
+    "http://*/*",
+    "https://*/*",
+    "*://*/"
+]);
+const API_PERMISSIONS = {
+    alarms: [
+        "alarms"
+    ],
+    bookmarks: [
+        "bookmarks"
+    ],
+    contextMenus: [
+        "contextMenus",
+        "menus"
+    ],
+    menus: [
+        "menus",
+        "contextMenus"
+    ],
+    cookies: [
+        "cookies"
+    ],
+    declarativeNetRequest: [
+        "declarativeNetRequest"
+    ],
+    downloads: [
+        "downloads"
+    ],
+    history: [
+        "history"
+    ],
+    identity: [
+        "identity"
+    ],
+    idle: [
+        "idle"
+    ],
+    management: [
+        "management"
+    ],
+    notifications: [
+        "notifications"
+    ],
+    offscreen: [
+        "offscreen"
+    ],
+    scripting: [
+        "scripting"
+    ],
+    sidePanel: [
+        "sidePanel"
+    ],
+    storage: [
+        "storage"
+    ],
+    tabGroups: [
+        "tabGroups"
+    ],
+    tabs: [
+        "tabs"
+    ],
+    topSites: [
+        "topSites"
+    ],
+    tts: [
+        "tts"
+    ],
+    webNavigation: [
+        "webNavigation"
+    ],
+    webRequest: [
+        "webRequest"
+    ]
+};
+const REMOTE_CODE_PATTERNS = [
+    [
+        "eval()",
+        /\beval\s*\(/
+    ],
+    [
+        "new Function()",
+        /\bnew\s+Function\s*\(/
+    ],
+    [
+        "importScripts from a URL",
+        /importScripts\(\s*["'`]https?:/
+    ],
+    [
+        "import() from a URL",
+        /\bimport\(\s*["'`]https?:/
+    ]
+];
+const REMOTE_SCRIPT_TAG = /<script[^>]+src\s*=\s*["']https?:/i;
+const MAX_SCAN_BYTES = 8388608;
+function strings(value) {
+    return Array.isArray(value) ? value.filter((v)=>"string" == typeof v) : [];
+}
+function hostPatterns(manifest) {
+    const found = new Set();
+    for (const pattern of strings(manifest.host_permissions))found.add(pattern);
+    for (const perm of strings(manifest.permissions))if (perm.includes("://") || "<all_urls>" === perm) found.add(perm);
+    const scripts = Array.isArray(manifest.content_scripts) ? manifest.content_scripts : [];
+    for (const entry of scripts)for (const match of strings(entry?.matches))found.add(match);
+    return [
+        ...found
+    ];
+}
+function readText(distPath, rel) {
+    try {
+        return node_fs.readFileSync(node_path.join(distPath, rel), "utf8");
+    } catch  {
+        return "";
+    }
+}
+function reviewRisks(input) {
+    const risks = [];
+    const development = input.development ?? isDevelopmentBuild(input.files);
+    const broad = hostPatterns(input.manifest).filter((p)=>BROAD_PATTERNS.has(p));
+    if (broad.length) risks.push({
+        code: "BROAD_HOST_ACCESS",
+        message: `The extension asks for access to every website (${broad.join(", ")}). Chrome Web Store and AMO reviewers ask why, and the review takes longer.`,
+        fix: "List only the sites the extension works on, or use activeTab so access is granted per click; if every site is the point, say why in the listing.",
+        permissions: broad
+    });
+    const scripts = input.files.filter((f)=>/\.(m?js|cjs)$/.test(f.path) && !f.path.endsWith(".map"));
+    const pages = input.files.filter((f)=>/\.html?$/.test(f.path));
+    const hits = new Map();
+    const note = (label, file)=>{
+        if (!hits.has(label)) hits.set(label, new Set());
+        hits.get(label).add(file);
+    };
+    let scanned = 0;
+    let source = "";
+    for (const file of scripts){
+        if (scanned >= MAX_SCAN_BYTES) break;
+        const text = readText(input.distPath, file.path);
+        scanned += text.length;
+        source += `\n${text}`;
+        for (const [label, pattern] of REMOTE_CODE_PATTERNS)if (pattern.test(text)) note(label, file.path);
+    }
+    for (const file of pages)if (REMOTE_SCRIPT_TAG.test(readText(input.distPath, file.path))) note("a <script> loaded from a URL", file.path);
+    if (hits.size && !development) {
+        const labels = [
+            ...hits.keys()
+        ];
+        risks.push({
+            code: "REMOTE_CODE",
+            message: `The package contains ${labels.join(", ")}. Manifest V3 stores reject extensions that run code they did not ship, and reviewers flag these patterns even inside a bundled library.`,
+            fix: "Remove the call, or confirm it only ever runs code that ships in the package and be ready to say so in review.",
+            files: [
+                ...new Set([
+                    ...hits.values()
+                ].flatMap((set)=>[
+                        ...set
+                    ]))
+            ].slice(0, 10)
+        });
+    }
+    if (isGeckoFamily(input.browser)) {
+        const gecko = input.manifest.browser_specific_settings?.gecko;
+        if (!gecko || !("data_collection_permissions" in gecko)) risks.push({
+            code: "FIREFOX_DATA_COLLECTION_MISSING",
+            message: "The Firefox manifest has no browser_specific_settings.gecko.data_collection_permissions. AMO requires it for new add-ons and for updates.",
+            fix: 'Add "browser_specific_settings": { "gecko": { "data_collection_permissions": { "required": ["none"] } } }, or list what the extension does collect.'
+        });
+    }
+    if (source && scanned < MAX_SCAN_BYTES && !development) {
+        const unused = strings(input.manifest.permissions).filter((perm)=>{
+            const apis = API_PERMISSIONS[perm];
+            if (!apis) return false;
+            return !apis.some((api)=>new RegExp(`\\.${api}\\b`).test(source));
+        });
+        if (unused.length) risks.push({
+            code: "UNUSED_PERMISSION",
+            message: `The manifest asks for ${unused.join(", ")}, but no shipped script uses ${1 === unused.length ? "that API" : "those APIs"}. Reviewers reject permissions an extension does not need.`,
+            fix: `Remove ${unused.join(", ")} from permissions, or keep ${1 === unused.length ? "it" : "them"} if the call happens in code this scan cannot see.`,
+            permissions: unused
+        });
+    }
+    return risks;
+}
+function isDevelopmentBuild(files) {
+    return files.some((f)=>f.path.endsWith(".map") || /hot-update\./.test(f.path));
+}
+function reviewRiskWarnings(risks) {
+    return risks.map((risk)=>`Store review: ${risk.message} ${risk.fix}`);
+}
+function listFiles(dir, base = "") {
+    let entries;
+    try {
+        entries = node_fs.readdirSync(dir, {
+            withFileTypes: true
+        });
+    } catch  {
+        return [];
+    }
+    const out = [];
+    for (const entry of entries){
+        const rel = base ? `${base}/${entry.name}` : entry.name;
+        if (entry.isDirectory()) out.push(...listFiles(node_path.join(dir, entry.name), rel));
+        else out.push({
+            path: rel
+        });
+    }
+    return out;
+}
+function reviewDist(distPath, browser) {
+    let manifest;
+    try {
+        manifest = JSON.parse(node_fs.readFileSync(node_path.join(distPath, "manifest.json"), "utf8"));
+    } catch  {
+        return [];
+    }
+    return reviewRisks({
+        distPath,
+        browser,
+        manifest,
+        files: listFiles(distPath)
+    });
+}
 const build_COMMAND = "extension_build";
 function readBuildSummary(projectPath, browser, since) {
     const file = session_paths_buildSummaryPath(projectPath, browser);
@@ -4210,6 +4433,7 @@ async function build_handler(args) {
         const buildWarningsTruncated = buildWarnings.length && "number" == typeof engineSummary?.warnings_count && engineSummary.warnings_count > buildWarnings.length ? engineSummary.warnings_count : void 0;
         const distDir = node_path.resolve(args.projectPath, "dist", browser);
         const entrypoints = builtEntrypoints(distDir);
+        const risks = reviewDist(distDir, browser);
         const contamination = carrierContamination(node_path.resolve(args.projectPath, "dist"));
         const uncheckedNote = contamination.unchecked.length ? `Could not read the entry table of ${contamination.unchecked.join(", ")}, so those archives were not checked for the live-preview carrier. Unpack and check them yourself before submitting.` : null;
         if (contamination.paths.length) return envelope_envelope({
@@ -4293,6 +4517,9 @@ async function build_handler(args) {
                 ...entrypoints.length ? {
                     entrypoints
                 } : {},
+                ...risks.length ? {
+                    reviewRisks: risks
+                } : {},
                 ...void 0 !== buildWarningsTruncated ? {
                     buildWarningsTruncated
                 } : {},
@@ -4313,6 +4540,7 @@ async function build_handler(args) {
                 ...warnings,
                 ...preflight?.warnings ?? [],
                 ...buildWarnings,
+                ...reviewRiskWarnings(risks),
                 ...zipNotes,
                 uncheckedNote,
                 derivedBundleIdNote,
@@ -13069,9 +13297,17 @@ async function analyze_handler(args) {
     }
     const note = "development" === buildType ? `This dist contains ${formatBytes(sourcemapSize)} of sourcemaps and looks like a dev build; run extension_build for production sizes. shippableSize excludes sourcemaps.` : void 0;
     const archiveNote = archiveSize > 0 ? `This dist contains ${formatBytes(archiveSize)} of .zip archive(s) (store packaging output, written into dist by zip builds). shippableSize excludes them so the packaged copy does not double-count the files it contains.` : void 0;
+    const risks = reviewRisks({
+        distPath,
+        browser,
+        manifest,
+        files,
+        development: "development" === buildType
+    });
     const result = {
         browser,
         distPath,
+        reviewRisks: risks,
         entrypoints,
         buildType,
         totalSize,
@@ -13120,6 +13356,7 @@ async function analyze_handler(args) {
         status: "analyzed",
         value: result,
         warnings: [
+            ...reviewRiskWarnings(risks),
             ...sizeWarnings,
             note,
             archiveNote

package/dist/src/lib/store-review.d.ts

@@ -0,0 +1,23 @@
+export type ReviewRiskCode = "BROAD_HOST_ACCESS" | "REMOTE_CODE" | "FIREFOX_DATA_COLLECTION_MISSING" | "UNUSED_PERMISSION";
+export interface ReviewRisk {
+    code: ReviewRiskCode;
+    message: string;
+    fix: string;
+    files?: string[];
+    permissions?: string[];
+}
+export declare function reviewRisks(input: {
+    distPath: string;
+    browser: string;
+    manifest: Record<string, unknown>;
+    files: Array<{
+        path: string;
+        type?: string;
+    }>;
+    development?: boolean;
+}): ReviewRisk[];
+export declare function isDevelopmentBuild(files: Array<{
+    path: string;
+}>): boolean;
+export declare function reviewRiskWarnings(risks: ReviewRisk[]): string[];
+export declare function reviewDist(distPath: string, browser: string): ReviewRisk[];

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@extension.dev/mcp",
   "type": "module",
-  "version": "10.10.4",
+  "version": "10.10.5",
   "description": "MCP server that lets AI agents (Claude Code, Claude Desktop, Cursor, Copilot, Codex) build, run, inspect, and publish browser extensions. 31 tools for scaffolding, live DOM inspection, log streaming, and store-ready builds across Chrome, Edge, Firefox, Safari, and every Chromium- or Gecko-based browser (Brave, Opera, Vivaldi, Yandex, Waterfox, LibreWolf). Powered by extension.dev and Extension.js.",
   "mcpName": "io.github.extensiondev/mcp",
   "license": "Apache-2.0",

package/server.json

@@ -7,13 +7,13 @@
     "source": "github"
   },
   "websiteUrl": "https://extension.dev",
-  "version": "10.10.4",
+  "version": "10.10.5",
   "packages": [
     {
       "registryType": "npm",
       "registryBaseUrl": "https://registry.npmjs.org",
       "identifier": "@extension.dev/mcp",
-      "version": "10.10.4",
+      "version": "10.10.5",
       "runtimeHint": "npx",
       "transport": {
         "type": "stdio"