create-restforge-skills 1.0.0 → 1.0.2

package/cli/mcp.js

@@ -1,94 +1,103 @@
-'use strict';
-
-/**
- * MCP registration for create-restforge-skills.
- *
- * The skill is inert without the RESTForge MCP server, so registration is the
- * default behavior. This module merges (never overwrites) a `restforge` entry
- * into a client's MCP config and backs the file up first.
- */
-
-const fs = require('fs');
-const path = require('path');
-
-/**
- * Build the MCP server entry.
- * - 'npx' (default): no separate global install — npx fetches on first run.
- * - 'global': uses the globally-installed `restforge-mcp` binary (faster start).
- */
-function buildEntry(mcpCommand) {
-  if (mcpCommand === 'global') {
-    return { command: 'restforge-mcp' };
-  }
-  return { command: 'npx', args: ['-y', '@restforgejs/mcp-server'] };
-}
-
-function readConfig(file) {
-  if (!fs.existsSync(file)) return { existed: false, data: {} };
-  const raw = fs.readFileSync(file, 'utf8').trim();
-  if (!raw) return { existed: true, data: {} };
-  let data;
-  try {
-    data = JSON.parse(raw);
-  } catch (err) {
-    const e = new Error(`config is not valid JSON: ${err.message}`);
-    e.code = 'EBADJSON';
-    throw e;
-  }
-  if (data === null || typeof data !== 'object' || Array.isArray(data)) {
-    const e = new Error('config root is not a JSON object');
-    e.code = 'EBADJSON';
-    throw e;
-  }
-  return { existed: true, data };
-}
-
-/**
- * Merge the `restforge` MCP server into `file`.
- * Returns { status, backup?, error? } where status is one of:
- *   registered | updated | already-registered | would-register | would-update | error
- * Never overwrites unrelated keys; backs up an existing file before writing.
- */
-function registerMcp(file, entry, opts) {
-  let cfg;
-  try {
-    cfg = readConfig(file);
-  } catch (err) {
-    if (err.code === 'EBADJSON') {
-      // Refuse to touch a config we cannot safely parse.
-      return { status: 'error', error: err.message };
-    }
-    throw err;
-  }
-
-  const data = cfg.data;
-  const servers =
-    data.mcpServers && typeof data.mcpServers === 'object' && !Array.isArray(data.mcpServers)
-      ? data.mcpServers
-      : {};
-
-  const existing = servers.restforge;
-  if (existing && JSON.stringify(existing) === JSON.stringify(entry)) {
-    return { status: 'already-registered' };
-  }
-
-  const willUpdate = Boolean(existing);
-  if (opts.dryRun) {
-    return { status: willUpdate ? 'would-update' : 'would-register' };
-  }
-
-  let backup;
-  if (cfg.existed) {
-    backup = file + '.bak';
-    fs.copyFileSync(file, backup);
-  }
-
-  servers.restforge = entry;
-  data.mcpServers = servers;
-  fs.mkdirSync(path.dirname(file), { recursive: true });
-  fs.writeFileSync(file, JSON.stringify(data, null, 2) + '\n', 'utf8');
-
-  return { status: willUpdate ? 'updated' : 'registered', backup };
-}
-
-module.exports = { buildEntry, registerMcp };
+'use strict';
+
+/**
+ * MCP registration for create-restforge-skills.
+ *
+ * The skill is inert without the RESTForge MCP server, so registration is the
+ * default behavior. This module merges (never overwrites) a `restforge` entry
+ * into a client's MCP config and backs the file up first.
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+// A cold `npx -y @restforgejs/mcp-server` (empty npm cache) took 13.3 s to answer
+// `initialize` on 2026-10-02 (issue #002), above the 10 s default startup window in
+// the Codex documentation. Codex reads `startup_timeout_sec` per MCP server, so its npx
+// entry gets a wider window. Claude Code and Cursor have no such key in their JSON config.
+const CODEX_NPX_STARTUP_TIMEOUT_SEC = 60;
+
+/**
+ * Build the MCP server entry.
+ * - 'npx' (default): no separate global install — npx fetches on first run.
+ * - 'global': uses the globally-installed `restforge-mcp` binary (faster start).
+ * - client 'codex' + 'npx': adds `startup_timeout_sec` for the cold npx start.
+ */
+function buildEntry(mcpCommand, client) {
+  if (mcpCommand === 'global') {
+    return { command: 'restforge-mcp' };
+  }
+  const entry = { command: 'npx', args: ['-y', '@restforgejs/mcp-server'] };
+  if (client === 'codex') entry.startup_timeout_sec = CODEX_NPX_STARTUP_TIMEOUT_SEC;
+  return entry;
+}
+
+function readConfig(file) {
+  if (!fs.existsSync(file)) return { existed: false, data: {} };
+  const raw = fs.readFileSync(file, 'utf8').trim();
+  if (!raw) return { existed: true, data: {} };
+  let data;
+  try {
+    data = JSON.parse(raw);
+  } catch (err) {
+    const e = new Error(`config is not valid JSON: ${err.message}`);
+    e.code = 'EBADJSON';
+    throw e;
+  }
+  if (data === null || typeof data !== 'object' || Array.isArray(data)) {
+    const e = new Error('config root is not a JSON object');
+    e.code = 'EBADJSON';
+    throw e;
+  }
+  return { existed: true, data };
+}
+
+/**
+ * Merge the `restforge` MCP server into `file`.
+ * Returns { status, backup?, error? } where status is one of:
+ *   registered | updated | already-registered | would-register | would-update | error
+ * Never overwrites unrelated keys; backs up an existing file before writing.
+ */
+function registerMcp(file, entry, opts) {
+  let cfg;
+  try {
+    cfg = readConfig(file);
+  } catch (err) {
+    if (err.code === 'EBADJSON') {
+      // Refuse to touch a config we cannot safely parse.
+      return { status: 'error', error: err.message };
+    }
+    throw err;
+  }
+
+  const data = cfg.data;
+  const servers =
+    data.mcpServers && typeof data.mcpServers === 'object' && !Array.isArray(data.mcpServers)
+      ? data.mcpServers
+      : {};
+
+  const existing = servers.restforge;
+  if (existing && JSON.stringify(existing) === JSON.stringify(entry)) {
+    return { status: 'already-registered' };
+  }
+
+  const willUpdate = Boolean(existing);
+  if (opts.dryRun) {
+    return { status: willUpdate ? 'would-update' : 'would-register' };
+  }
+
+  let backup;
+  if (cfg.existed) {
+    backup = file + '.bak';
+    fs.copyFileSync(file, backup);
+  }
+
+  servers.restforge = entry;
+  data.mcpServers = servers;
+  fs.mkdirSync(path.dirname(file), { recursive: true });
+  fs.writeFileSync(file, JSON.stringify(data, null, 2) + '\n', 'utf8');
+
+  return { status: willUpdate ? 'updated' : 'registered', backup };
+}
+
+module.exports = { buildEntry, registerMcp };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-restforge-skills",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "description": "Install the RESTForge Agent Skill and MCP server configuration into Claude Code, Cursor, and OpenAI Codex.",
   "type": "commonjs",
   "bin": {
@@ -8,6 +8,7 @@
   },
   "scripts": {
     "test": "node --test",
+    "prepublishOnly": "node --test test/package-contents.test.js && node scripts/audit-references.js",
     "audit-references": "node scripts/audit-references.js"
   },
   "files": [