Review fixes

Author: vladimir-tikhonov-nutrient

README.md

@@ -43,7 +43,7 @@ Once configured, you (or your AI agent) can process documents through natural la
 
 ## Installation
 
-Once this extension is listed, install it from Claude Desktop Settings -> Extensions. Until the directory listing is live, use the manual setup below.
+Install it from Claude Desktop Settings -> Extensions if you are using Claude Desktop. If you are developing locally or using it before the directory listing is live, use the manual setup below.
 
 ### 1. Create a Nutrient Account
 

manifest.json

@@ -65,10 +65,6 @@
   ],
   "compatibility": {
     "claude_desktop": ">=0.10.0",
-    "platforms": [
-      "darwin",
-      "win32"
-    ],
     "runtimes": {
       "node": ">=18.0.0"
     }

package.json

@@ -42,8 +42,9 @@
     "build": "tsc && shx chmod +x dist/index.js",
     "format": "prettier --write .",
     "lint": "eslint .",
-    "mcpb:validate": "npx -y @anthropic-ai/mcpb validate manifest.json",
-    "mcpb:pack": "pnpm run build && node scripts/build-mcpb.mjs",
+    "manifest:sync-version": "node scripts/sync-manifest-version.mjs",
+    "mcpb:validate": "pnpm run manifest:sync-version && npx -y @anthropic-ai/mcpb validate manifest.json",
+    "mcpb:pack": "pnpm run manifest:sync-version && pnpm run build && node scripts/build-mcpb.mjs",
     "pretest": "tsc --project tsconfig.test.json --noEmit",
     "test": "vitest run",
     "test:ci": "vitest run --exclude tests/build-api-examples.test.ts --exclude tests/signing-api-examples.test.ts",

scripts/build-mcpb.mjs

@@ -13,6 +13,9 @@ const rootDir = path.resolve(__dirname, '..')
 const outputPath = path.resolve(process.argv[2] ?? path.join(rootDir, 'dist', 'nutrient-dws.mcpb'))
 const cleanEnv = { ...process.env }
 
+// pnpm injects npm_config_* environment variables that make npm print warnings
+// and can influence staging installs. Strip the known noisy ones for a clean,
+// reproducible npm install in the temporary bundle directory.
 for (const key of [
   'npm_config_supported_architectures',
   'npm_config_npm_globalconfig',

scripts/sync-manifest-version.mjs

@@ -0,0 +1,19 @@
+#!/usr/bin/env node
+
+import { readFile, writeFile } from 'node:fs/promises'
+import path from 'node:path'
+import { fileURLToPath } from 'node:url'
+
+const __filename = fileURLToPath(import.meta.url)
+const __dirname = path.dirname(__filename)
+const rootDir = path.resolve(__dirname, '..')
+const packageJsonPath = path.join(rootDir, 'package.json')
+const manifestJsonPath = path.join(rootDir, 'manifest.json')
+
+const packageJson = JSON.parse(await readFile(packageJsonPath, 'utf8'))
+const manifestJson = JSON.parse(await readFile(manifestJsonPath, 'utf8'))
+
+if (manifestJson.version !== packageJson.version) {
+  manifestJson.version = packageJson.version
+  await writeFile(manifestJsonPath, `${JSON.stringify(manifestJson, null, 2)}\n`, 'utf8')
+}