@sellable/install 0.1.6 → 0.1.8

package/README.md

@@ -30,6 +30,28 @@ Auth is stored once at:
 
 Claude Code and Codex are configured to launch the same packaged MCP server.
 
+## Names
+
+Use the same public entrypoint in both hosts:
+
+- Claude Code: `/sellable:create-campaign`
+- Codex: `$sellable:create-campaign`
+- Codex Desktop plugin: `sellable@sellable`
+- Codex visible skill: `Sellable Create Campaign`
+- Internal MCP workflow prompt: `create-campaign-v2`
+
+Do not ask users to run `/sellable:create-campaign-v2`,
+`$sellable:create-campaign-v2`, or `$sellable:sellable:create-campaign`.
+`create-campaign-v2` is loaded internally by the skill.
+
+## Structured Questions
+
+Claude Code uses `AskUserQuestion`. Codex uses `request_user_input` when that
+tool is exposed in an interactive session. The installer enables Codex Default
+mode support by writing `default_mode_request_user_input = true` under
+`[features]` in `~/.codex/config.toml`. `codex exec` is non-interactive, so it
+cannot show the structured questionnaire UI.
+
 For Codex Desktop, the installer also writes a local Sellable plugin bundle into
 `~/.sellable/codex-marketplace`, includes the Sellable skill entrypoints, and
 enables it in `~/.codex/config.toml`.

package/bin/sellable-install.mjs

@@ -1,15 +1,21 @@
 #!/usr/bin/env node
 import { spawnSync } from "node:child_process";
-import { existsSync, mkdirSync, readFileSync, rmSync, writeFileSync } from "node:fs";
-import { dirname, join } from "node:path";
+import {
+  existsSync,
+  mkdirSync,
+  readFileSync,
+  rmSync,
+  writeFileSync,
+} from "node:fs";
 import { homedir } from "node:os";
+import { dirname, join } from "node:path";
 import { stdin as input, stdout as output } from "node:process";
 import { createInterface } from "node:readline/promises";
 
 const DEFAULT_API_URL = "https://app.sellable.dev";
 const DEFAULT_SERVER_PACKAGE =
   process.env.SELLABLE_MCP_PACKAGE || "@sellable/mcp";
-const CODEX_PLUGIN_VERSION = "0.1.6";
+const CODEX_PLUGIN_VERSION = "0.1.8";
 const INSTALL_PACKAGE_SPEC = `@sellable/install@${CODEX_PLUGIN_VERSION}`;
 
 function usage() {
@@ -109,7 +115,9 @@ function redact(value) {
 
 function run(command, args, opts = {}) {
   const rendered = [command, ...args].join(" ");
-  console.log(`+ ${rendered.replace(opts.token || "__NO_TOKEN__", "[redacted-token]")}`);
+  console.log(
+    `+ ${rendered.replace(opts.token || "__NO_TOKEN__", "[redacted-token]")}`
+  );
   if (opts.dryRun) return { status: 0, stdout: "", stderr: "" };
   const result = spawnSync(command, args, {
     encoding: "utf8",
@@ -186,8 +194,12 @@ async function promptForMissingAuth(opts) {
   }
 
   console.log("");
-  console.log("Sellable needs one API token to connect Claude/Codex to your workspace.");
-  console.log(`Open ${opts.apiUrl}/settings, create a token, then paste the values below.`);
+  console.log(
+    "Sellable needs one API token to connect Claude/Codex to your workspace."
+  );
+  console.log(
+    `Open ${opts.apiUrl}/settings, create a token, then paste the values below.`
+  );
   console.log(`Auth will be stored once at ${authPath()}.`);
   console.log("");
 
@@ -211,7 +223,11 @@ async function promptForMissingAuth(opts) {
 }
 
 function writeJson(path, data, opts) {
-  const redacted = JSON.stringify({ ...data, token: redact(data.token) }, null, 2);
+  const redacted = JSON.stringify(
+    { ...data, token: redact(data.token) },
+    null,
+    2
+  );
   console.log(`Writing ${path}: ${redacted}`);
   if (opts.dryRun) return;
   mkdirSync(dirname(path), { recursive: true, mode: 0o700 });
@@ -246,12 +262,38 @@ function upsertTomlTable(content, tableName, block) {
   const normalizedBlock = block.trimEnd();
 
   if (tablePattern.test(content)) {
-    return content.replace(tablePattern, (_, prefix) => `${prefix}${normalizedBlock}`);
+    return content.replace(
+      tablePattern,
+      (_, prefix) => `${prefix}${normalizedBlock}`
+    );
   }
 
   return `${content.trimEnd()}\n\n${normalizedBlock}\n`;
 }
 
+function upsertTomlBoolean(content, tableName, key, value) {
+  const line = `${key} = ${value ? "true" : "false"}`;
+  const tablePattern = new RegExp(
+    `(^|\\n)(\\[${escapeRegExp(tableName)}\\]\\n)([\\s\\S]*?)(?=\\n\\[[^\\n]+\\]|$)`
+  );
+
+  if (!tablePattern.test(content)) {
+    return `${content.trimEnd()}\n\n[${tableName}]\n${line}\n`;
+  }
+
+  return content.replace(tablePattern, (_, prefix, header, body) => {
+    const keyPattern = new RegExp(
+      `(^|\\n)\\s*${escapeRegExp(key)}\\s*=.*`,
+      "m"
+    );
+    const updatedBody = keyPattern.test(body)
+      ? body.replace(keyPattern, `$1${line}`)
+      : `${body.trimEnd()}${body.trimEnd() ? "\n" : ""}${line}\n`;
+
+    return `${prefix}${header}${updatedBody.trimEnd()}`;
+  });
+}
+
 function writeFile(path, content, opts, mode = 0o644) {
   console.log(`Writing ${path}`);
   if (opts.dryRun) return;
@@ -410,6 +452,43 @@ ${allowedToolsYaml(CREATE_CAMPAIGN_ALLOWED_TOOLS)}
 
 Use this as the customer-facing entrypoint for Sellable campaign creation.
 
+## Names To Use
+
+Use these exact public names so Claude Code and Codex do not drift:
+
+- Claude Code command: \`/sellable:create-campaign\`
+- Codex skill command: \`$sellable:create-campaign\`
+- Codex Desktop plugin: \`sellable@sellable\`
+- Codex visible skill: \`Sellable Create Campaign\`
+- Codex skill frontmatter name: \`create-campaign\`
+- MCP server name: \`sellable\`
+- Internal workflow prompt: \`create-campaign-v2\`
+
+Do not tell users to run \`/sellable:create-campaign-v2\`,
+\`$sellable:create-campaign-v2\`, or \`$sellable:sellable:create-campaign\`.
+\`create-campaign-v2\` is only the internal subskill loaded through
+\`mcp__sellable__get_subskill_prompt({ subskillName: "create-campaign-v2" })\`.
+
+## Structured Questions
+
+Use the host-native structured question gate for intake and approval:
+
+- Claude Code: \`AskUserQuestion\`
+- Codex: \`request_user_input\` when exposed in an interactive session. The
+  installer enables this in Default mode with
+  \`[features].default_mode_request_user_input = true\`.
+
+Do not silently ask Codex intake or approval questions as plain chat when
+\`request_user_input\` is unavailable in an interactive session. Stop and tell
+the user: \`Codex has not exposed request_user_input in this session. Can I
+update ~/.codex/config.toml with [features].default_mode_request_user_input =
+true? After that, fully quit and reopen Codex, then rerun
+$sellable:create-campaign.\` If they decline, tell them to switch to
+Plan/collaboration mode and rerun \`$sellable:create-campaign\`. Plain chat
+questions are only acceptable in non-interactive \`codex exec\`
+smoke/rehearsal runs because structured user input is unavailable by design
+there.
+
 ## Bootstrap
 
 MCP tool access is required. First call \`mcp__sellable__get_auth_status({})\`
@@ -557,7 +636,12 @@ function installCodexDesktopPlugin(opts) {
   const home = codexHome();
   const configPath = join(home, "config.toml");
   const marketplaceRoot = join(homedir(), ".sellable", "codex-marketplace");
-  const marketplacePath = join(marketplaceRoot, ".agents", "plugins", "marketplace.json");
+  const marketplacePath = join(
+    marketplaceRoot,
+    ".agents",
+    "plugins",
+    "marketplace.json"
+  );
   const pluginRoot = join(marketplaceRoot, "plugins", "sellable");
   const cacheRoot = join(home, "plugins", "cache", "sellable", "sellable");
   const pluginCache = join(cacheRoot, CODEX_PLUGIN_VERSION);
@@ -592,7 +676,11 @@ function installCodexDesktopPlugin(opts) {
     `${JSON.stringify(manifest, null, 2)}\n`,
     opts
   );
-  writeFile(join(pluginRoot, ".mcp.json"), `${JSON.stringify(mcp, null, 2)}\n`, opts);
+  writeFile(
+    join(pluginRoot, ".mcp.json"),
+    `${JSON.stringify(mcp, null, 2)}\n`,
+    opts
+  );
   writeCodexPluginSkills(pluginRoot, opts);
 
   if (!opts.dryRun) {
@@ -603,12 +691,18 @@ function installCodexDesktopPlugin(opts) {
     `${JSON.stringify(manifest, null, 2)}\n`,
     opts
   );
-  writeFile(join(pluginCache, ".mcp.json"), `${JSON.stringify(mcp, null, 2)}\n`, opts);
+  writeFile(
+    join(pluginCache, ".mcp.json"),
+    `${JSON.stringify(mcp, null, 2)}\n`,
+    opts
+  );
   writeCodexPluginSkills(pluginCache, opts);
 
   if (!opts.dryRun) {
     mkdirSync(home, { recursive: true, mode: 0o700 });
-    let content = existsSync(configPath) ? readFileSync(configPath, "utf8") : "";
+    let content = existsSync(configPath)
+      ? readFileSync(configPath, "utf8")
+      : "";
     content = upsertTomlTable(
       content,
       "marketplaces.sellable",
@@ -629,11 +723,22 @@ enabled = true`
       `[plugins."sellable@sellable-local"]
 enabled = false`
     );
+    content = upsertTomlBoolean(
+      content,
+      "features",
+      "default_mode_request_user_input",
+      true
+    );
     writeFileSync(configPath, `${content.trimEnd()}\n`, { mode: 0o600 });
   } else {
     console.log(`+ upsert [marketplaces.sellable] in ${configPath}`);
     console.log(`+ enable [plugins."sellable@sellable"] in ${configPath}`);
-    console.log(`+ disable stale [plugins."sellable@sellable-local"] in ${configPath}`);
+    console.log(
+      `+ disable stale [plugins."sellable@sellable-local"] in ${configPath}`
+    );
+    console.log(
+      `+ enable [features].default_mode_request_user_input in ${configPath}`
+    );
   }
 
   console.log("Codex Desktop plugin installed:");
@@ -656,7 +761,9 @@ function writeAuth(opts) {
   };
 
   if (opts.authFromExistingConfig) {
-    console.log(`Leaving existing Sellable auth config unchanged: ${authPath()}`);
+    console.log(
+      `Leaving existing Sellable auth config unchanged: ${authPath()}`
+    );
     return;
   }
 
@@ -684,7 +791,8 @@ function mcpCommand(opts) {
 
 function installClaude(opts) {
   if (!commandExists("claude")) {
-    const message = "Claude CLI not found. Install/login to Claude Code, then rerun: sellable --host claude";
+    const message =
+      "Claude CLI not found. Install/login to Claude Code, then rerun: sellable --host claude";
     if (opts.host === "all") {
       console.log(`Skipping Claude Code: ${message}`);
       return false;
@@ -692,7 +800,11 @@ function installClaude(opts) {
     throw new Error(message);
   }
   if (opts.server === "hosted") {
-    run("claude", ["mcp", "add", "--transport", "http", "sellable", opts.hostedUrl], opts);
+    run(
+      "claude",
+      ["mcp", "add", "--transport", "http", "sellable", opts.hostedUrl],
+      opts
+    );
     return true;
   }
   const [command, args] = mcpCommand(opts);
@@ -701,13 +813,18 @@ function installClaude(opts) {
     dryRun: opts.dryRun,
     allowFail: true,
   });
-  run("claude", ["mcp", "add", "--transport", "stdio", "sellable", "--", command, ...args], opts);
+  run(
+    "claude",
+    ["mcp", "add", "--transport", "stdio", "sellable", "--", command, ...args],
+    opts
+  );
   return true;
 }
 
 function installCodex(opts) {
   if (!commandExists("codex")) {
-    const message = "Codex CLI not found. Install/login to Codex, then rerun: sellable --host codex";
+    const message =
+      "Codex CLI not found. Install/login to Codex, then rerun: sellable --host codex";
     if (opts.host === "all") {
       console.log(`Skipping Codex: ${message}`);
       return false;
@@ -733,14 +850,20 @@ function installCodex(opts) {
 function verify(opts) {
   const stored = readStoredAuth();
   if (!stored?.token || !stored?.workspaceId) {
-    throw new Error(`Sellable auth config missing or incomplete: ${authPath()}`);
+    throw new Error(
+      `Sellable auth config missing or incomplete: ${authPath()}`
+    );
   }
   console.log(`Sellable auth config present: ${authPath()}`);
   if (opts.host === "claude" || opts.host === "all") {
-    console.log(commandExists("claude") ? "Claude CLI present" : "Claude CLI missing");
+    console.log(
+      commandExists("claude") ? "Claude CLI present" : "Claude CLI missing"
+    );
   }
   if (opts.host === "codex" || opts.host === "all") {
-    console.log(commandExists("codex") ? "Codex CLI present" : "Codex CLI missing");
+    console.log(
+      commandExists("codex") ? "Codex CLI present" : "Codex CLI missing"
+    );
     const pluginPath = join(
       codexHome(),
       "plugins",
@@ -762,8 +885,25 @@ function verify(opts) {
       "sellable-create-campaign",
       "SKILL.md"
     );
-    console.log(existsSync(pluginPath) ? "Codex Desktop plugin present" : "Codex Desktop plugin missing");
-    console.log(existsSync(skillPath) ? "Codex skill bundle present" : "Codex skill bundle missing");
+    console.log(
+      existsSync(pluginPath)
+        ? "Codex Desktop plugin present"
+        : "Codex Desktop plugin missing"
+    );
+    console.log(
+      existsSync(skillPath)
+        ? "Codex skill bundle present"
+        : "Codex skill bundle missing"
+    );
+    const configPath = join(codexHome(), "config.toml");
+    const configContent = existsSync(configPath)
+      ? readFileSync(configPath, "utf8")
+      : "";
+    console.log(
+      configContent.includes("default_mode_request_user_input = true")
+        ? "Codex Default-mode request_user_input enabled"
+        : "Codex Default-mode request_user_input missing"
+    );
   }
 }
 
@@ -771,11 +911,17 @@ function printNextSteps(installedHosts) {
   console.log("");
   console.log("Next steps:");
   if (installedHosts.length > 0) {
-    console.log(`1. Fully quit and reopen ${installedHosts.join(" and ")} so MCP tools reload.`);
+    console.log(
+      `1. Fully quit and reopen ${installedHosts.join(" and ")} so MCP tools reload.`
+    );
     console.log("2. Start a new thread and choose Sellable Create Campaign.");
-    console.log("3. If tools do not appear, run: sellable --verify-only --host all");
+    console.log(
+      "3. If tools do not appear, run: sellable --verify-only --host all"
+    );
   } else {
-    console.log("1. Install Claude Code or Codex, then rerun this installer for that host.");
+    console.log(
+      "1. Install Claude Code or Codex, then rerun this installer for that host."
+    );
     console.log("2. Verify auth later with: sellable --verify-only --host all");
   }
 }
@@ -806,7 +952,9 @@ async function main() {
       }
     }
     if (opts.dryRun) {
-      console.log("Dry run complete; verification skipped because no files were written.");
+      console.log(
+        "Dry run complete; verification skipped because no files were written."
+      );
     } else {
       verify(opts);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sellable/install",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "type": "module",
   "description": "One-command installer for Sellable MCP in Claude Code and Codex",
   "bin": {