@sellable/install 0.1.210 → 0.1.212

package/bin/sellable-install.mjs

@@ -72,8 +72,9 @@ const CODEX_PLUGIN_COMPAT_VERSIONS = [
   "0.1.40",
   "0.1.41",
 ];
-const INSTALL_PACKAGE_SPEC =
+const RAW_INSTALL_PACKAGE_SPEC =
   process.env.SELLABLE_INSTALL_PACKAGE_SPEC || "@sellable/install@latest";
+const INSTALL_PACKAGE_SPEC = normalizeWindowsPackageSpec(RAW_INSTALL_PACKAGE_SPEC);
 
 const useColor = Boolean(output.isTTY) && process.env.NO_COLOR === undefined;
 const C = {
@@ -89,6 +90,15 @@ const C = {
 
 let VERBOSE = false;
 
+function normalizeWindowsPackageSpec(value) {
+  if (typeof value !== "string") return value;
+  if (/^[A-Za-z]:\//.test(value)) return value.replace(/\//g, "\\");
+  if (/^\/[A-Za-z]\//.test(value)) {
+    return `${value[1]}:${value.slice(2)}`.replace(/\//g, "\\");
+  }
+  return value;
+}
+
 function logVerbose(line) {
   if (VERBOSE) console.log(line);
 }
@@ -263,7 +273,7 @@ function parseArgs(argv) {
     } else if (arg === "--api-url") {
       opts.apiUrl = next();
     } else if (arg === "--mcp-package") {
-      opts.mcpPackage = next();
+      opts.mcpPackage = normalizeWindowsPackageSpec(next());
     } else if (arg === "--local-command") {
       opts.localCommand = next();
     } else if (arg === "--hosted-url") {
@@ -771,7 +781,54 @@ function codexSkillOpenAiYaml(displayName, description) {
 `;
 }
 
-function createCampaignSkillMd() {
+function installedHostContract(host, commandName) {
+  if (host === "codex") {
+    return `## Installed Host Contract
+
+This installed skill is running in Codex. When the shared workflow body or
+fallback text mentions both Claude Code and Codex for internal parity, choose
+the Codex instruction for customer-facing language and host functions.
+
+- Customer-facing command: \`$sellable:${commandName}\`
+- Structured question function: \`request_user_input\`
+- Bootstrap host label: \`host: "Codex"\`
+- Install/reload blocker label: Codex install/reload problem
+- Reload instruction: fully quit and reopen Codex Desktop, then start a new thread
+
+Do not tell Codex users to run \`/sellable:${commandName}\`, use
+\`AskUserQuestion\`, or restart Claude Code. Do not describe this run as Claude Code.`;
+  }
+  if (host === "claude") {
+    return `## Installed Host Contract
+
+This installed command is running in Claude Code. When the shared workflow body
+or fallback text mentions both Claude Code and Codex for internal parity, choose
+the Claude Code instruction for customer-facing language and host functions.
+
+- Customer-facing command: \`/sellable:${commandName}\`
+- Structured question function: \`AskUserQuestion\`
+- Bootstrap host label: \`host: "Claude Code"\`
+- Install/reload blocker label: Claude Code install/reload problem
+- Reload instruction: fully quit and reopen Claude Code, then start a new session
+
+Do not tell Claude Code users to run \`$sellable:${commandName}\`, use
+\`request_user_input\`, or restart Codex Desktop. Do not describe this run as Codex.`;
+  }
+  return "";
+}
+
+function stampInstalledHost(markdown, host, commandName) {
+  const preamble = installedHostContract(host, commandName);
+  if (!preamble || markdown.includes("## Installed Host Contract")) {
+    return markdown;
+  }
+  return String(markdown).replace(
+    /(^# .+\n\n)/m,
+    `$1${preamble}\n\n`
+  );
+}
+
+function createCampaignSkillMd(host = "shared") {
   // Single source of truth: ../skill-templates/create-campaign.md, which is
   // copied from the canonical mcp/sellable/skills/create-campaign/SKILL.md at
   // publish time via scripts/sync-skill-templates.mjs (prepublishOnly hook).
@@ -786,12 +843,16 @@ function createCampaignSkillMd() {
       "create-campaign.md"
     );
     if (existsSync(templatePath)) {
-      return readFileSync(templatePath, "utf8");
+      return stampInstalledHost(
+        readFileSync(templatePath, "utf8"),
+        host,
+        "create-campaign"
+      );
     }
   } catch {
     // fall through to hardcoded fallback below
   }
-  return `---
+  return stampInstalledHost(`---
 name: create-campaign
 description: Create a Sellable campaign through the approval-gated workflow.
 allowed-tools:
@@ -814,17 +875,11 @@ person/company this campaign is for, then I’ll turn that into a campaign brief
 before anything is created.
 \`\`\`
 
-Exception: if \`bootstrap_create_campaign.modelQuality.status === "warn"\` and
-\`bootstrap_create_campaign.modelQuality.metadataStale !== true\`, the first
-visible campaign message must be the model-quality warning from
-\`modelQuality.message\`. Ask the user to switch to the configured minimum model
-or explicitly continue anyway before identity setup, research, lead filtering,
-message generation, or launch review. If \`metadataStale === true\`, continue
-normally and do not ask the user to switch.
-
-If \`bootstrap_create_campaign.modelQuality.metadataStale === true\`, continue
-normally. Do not ask the user to switch models; this is an accepted Codex host
-metadata mismatch.
+Exception: if \`bootstrap_create_campaign.modelQuality.status === "warn"\`, the
+first visible campaign message must be the model-quality warning from
+\`modelQuality.message\`. Only trust that warning when bootstrap received active
+turn/runtime metadata or an explicit user-confirmed model. Do not warn from
+config defaults, stale host labels, or inferred model names.
 
 If a linked/local skill file is stale or missing, silently use the installed
 \`sellable@sellable\` plugin copy. Do not tell the user about the stale link,
@@ -842,6 +897,56 @@ clear business decisions, tradeoffs, and approval gates. Use product language:
 - "I can create a draft shell for you to watch with approval gates before
   sourcing", not mutation jargon
 
+## Active Model Metadata
+
+Before calling \`bootstrap_create_campaign\`, collect the active host model data
+only from a source that can describe this turn:
+
+- Codex: if \`mcp__node_repl__js\` is available, inspect only
+  \`nodeRepl.requestMeta["x-codex-turn-metadata"].model\` and
+  \`nodeRepl.requestMeta["x-codex-turn-metadata"].reasoning_effort\`. Pass those
+  values as \`model\` and \`reasoningEffort\`, with
+  \`modelMetadataSource: "codex_turn_metadata"\`.
+- Codex fallback/cross-check: if active turn metadata is unavailable, you may
+  inspect \`~/.codex/config.toml\` for \`model\` and
+  \`model_reasoning_effort\`, but pass
+  \`modelMetadataSource: "codex_config_fallback"\` and do not treat it as a
+  reason to ask the user to switch models. Active turn metadata wins if it
+  differs from config.
+- Claude Code / Opus: use active Claude runtime metadata only if the current
+  host exposes model and effort for this same session, with
+  \`modelMetadataSource: "claude_runtime_metadata"\`. Current Claude Code MCP
+  tool calls may not include model or effort metadata; when they do not, omit
+  \`model\`, \`reasoningEffort\`, and \`modelMetadataSource\`.
+- Claude Code session-context self-report: if your current Claude session
+  context explicitly states both the exact model ID and effort/thinking level,
+  report it internally in this shape and pass it to bootstrap with
+  \`modelMetadataSource: "claude_session_context"\`:
+
+  \`\`\`text
+  - Model (name): ...
+  - Model (ID): ...
+  - Reasoning effort: ...
+  - Source: active Claude Code session context
+  \`\`\`
+
+  Use this only when the values are explicitly present in the current session
+  context. Do not infer an ID from the friendly name, do not infer effort from
+  \`alwaysThinkingEnabled\`, and do not show this self-report to the user during
+  normal campaign setup.
+- Do not run a nested \`claude -p\`, inspect \`~/.claude/settings.json\`, or read
+  Claude CLI defaults as proof of the user's active Claude Code session. Those
+  checks can validate a new child session or saved defaults, but not this
+  session's actual model and effort.
+- If the user explicitly provides active Claude \`/status\` or \`/model\` output
+  that includes both model and effort, pass it with
+  \`modelMetadataSource: "user_confirmed"\`. If it is missing either model or
+  effort, treat the metadata as unknown and continue.
+
+Never invent the model or reasoning effort. Never pass config defaults as active
+metadata. If bootstrap returns \`modelQuality.status === "unknown"\`, continue
+without asking the user to switch models.
+
 When explaining source decisions, show the concrete counts behind the
 logic: lanes searched, timeframe, raw result counts, finalist posts or preview
 rows, sampled people, sampled fits as n/N (%), estimated usable people, and the
@@ -1263,13 +1368,19 @@ updates.
    - Do not call \`mcp__sellable__get_campaigns\`.
    - Do not call \`mcp__sellable__get_campaign\` to hunt for IDs.
    - Do not call \`mcp__sellable__create_campaign({ campaignId: ... })\` unless the user supplied that id.
-5. Call \`mcp__sellable__bootstrap_create_campaign({ flowVersion: "v2", campaignId?, host?, model?, reasoningEffort? })\`.
-   Pass the current host, model, and reasoning when the host exposes them.
+5. Call \`mcp__sellable__bootstrap_create_campaign({ flowVersion: "v2", campaignId?, host?, model?, reasoningEffort?, modelMetadataSource? })\`.
+   Pass model metadata only when collected by the Active Model Metadata rules
+   above. For Codex active turn metadata, pass
+   \`modelMetadataSource: "codex_turn_metadata"\`. For explicit Claude session
+   context, pass \`modelMetadataSource: "claude_session_context"\`. For explicit
+   user-confirmed Claude \`/status\` or \`/model\` output, pass
+   \`modelMetadataSource: "user_confirmed"\` only when it includes both model and
+   effort.
 6. If \`safeToProceed !== true\`, stop and show \`blockingErrors\` + \`nextStep\`.
-7. If \`modelQuality.status === "warn"\` and \`modelQuality.metadataStale !== true\`,
-   show \`modelQuality.message\` before any setup/research and wait for the user
-   to switch or explicitly continue. If \`metadataStale === true\`, continue
-   normally and do not tell the user to switch.
+7. If \`modelQuality.status === "warn"\`, show \`modelQuality.message\` before
+   any setup/research and wait for the user to switch or explicitly continue. If
+   \`modelQuality.status === "unknown"\`, continue; do not tell the user to
+   switch models.
 
 ## Execute Workflow
 
@@ -1350,10 +1461,10 @@ updates.
 If subskill lookup fails, use
 \`mcp__sellable__search_subskill_prompts({ query: "create-campaign-v2" })\`,
 then retry \`get_subskill_prompt\`.
-`;
+`, host, "create-campaign");
 }
 
-function createAbTestSkillMd() {
+function createAbTestSkillMd(host = "shared") {
   try {
     const here = dirname(fileURLToPath(import.meta.url));
     const templatePath = join(
@@ -1363,12 +1474,16 @@ function createAbTestSkillMd() {
       "create-ab-test.md"
     );
     if (existsSync(templatePath)) {
-      return readFileSync(templatePath, "utf8");
+      return stampInstalledHost(
+        readFileSync(templatePath, "utf8"),
+        host,
+        "create-ab-test"
+      );
     }
   } catch {
     // fall through to hardcoded fallback below
   }
-  return `---
+  return stampInstalledHost(`---
 name: create-ab-test
 description: Create a Sellable campaign A/B test from a clean source lead list.
 visibility: public
@@ -1384,11 +1499,11 @@ allowed-tools:
 # Sellable Create A/B Test
 
 Use \`prepare_campaign_ab_test\` to create clean A/B split lead lists and review-copy campaigns. Do not call \`export_table_csv\` from generated campaign workflow tables as lead source data, and do not call \`start_campaign\`.
-`;
+`, host, "create-ab-test");
 }
 
-function genericSellableSkillMd({ name, title, description }) {
-  return `---
+function genericSellableSkillMd({ name, title, description, host = "shared" }) {
+  return stampInstalledHost(`---
 name: ${name}
 description: ${yamlString(description)}
 allowed-tools:
@@ -1423,11 +1538,11 @@ Desktop, then start a new thread.
 ## MCP Prompt Fallback
 
 If subskill lookup fails, use \`mcp__sellable__search_subskill_prompts({ query: "${name}" })\`, then retry \`get_subskill_prompt\`.
-`;
+`, host, name);
 }
 
-function foundationSkillMd() {
-  return `---
+function foundationSkillMd(host = "shared") {
+  return stampInstalledHost(`---
 name: foundation
 description: ${yamlString(FOUNDATION_SKILL_DESCRIPTION)}
 allowed-tools:
@@ -1486,11 +1601,11 @@ Desktop, then start a new thread.
 If exact subskill lookup fails, use
 \`mcp__sellable__search_subskill_prompts({ query: "foundation", includePublic: true, includeInternal: true })\`,
 then retry \`get_subskill_prompt\`.
-`;
+`, host, "foundation");
 }
 
-function contentSkillMd() {
-  return `---
+function contentSkillMd(host = "shared") {
+  return stampInstalledHost(`---
 name: content
 description: Add transcripts and rough ideas, cluster recurring themes, ideate post seeds, and hand draft requests to create-post.
 allowed-tools:
@@ -1556,11 +1671,11 @@ Desktop, then start a new thread.
 If exact subskill lookup fails, use
 \`mcp__sellable__search_subskill_prompts({ query: "content", includePublic: true, includeInternal: true })\`,
 then retry \`get_subskill_prompt\`.
-`;
+`, host, "content");
 }
 
-function createPostSkillMd() {
-  return `---
+function createPostSkillMd(host = "shared") {
+  return stampInstalledHost(`---
 name: create-post
 description: Capture rough LinkedIn post ideas, develop a valuable premise, research current hooks and audience tension, then save validated drafts in the user's voice.
 allowed-tools:
@@ -1630,7 +1745,7 @@ Desktop, then start a new thread.
 If exact subskill lookup fails, use
 \`mcp__sellable__search_subskill_prompts({ query: "create-post", includePublic: true, includeInternal: true })\`,
 then retry \`get_subskill_prompt\`.
-`;
+`, host, "create-post");
 }
 
 function createCampaignSoulMd() {
@@ -1827,7 +1942,7 @@ function codexPluginSkills() {
       dir: "sellable-create-campaign",
       displayName: "Sellable Create Campaign",
       description: "Create a Sellable campaign with approval gates",
-      skillMd: createCampaignSkillMd(),
+      skillMd: createCampaignSkillMd("codex"),
       soulMd: createCampaignSoulMd(),
     },
     {
@@ -2051,7 +2166,7 @@ function claudeCommands() {
       filename: join("sellable", "create-campaign.md"),
       description: "Create a Sellable campaign through the approval-gated workflow.",
       argumentHint: "[campaign goal, company, or offer]",
-      skillMd: createCampaignSkillMd(),
+      skillMd: createCampaignSkillMd("claude"),
       allowedTools: ["AskUserQuestion", "Task", ...CREATE_CAMPAIGN_ALLOWED_TOOLS],
     },
     {

package/package.json

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

package/skill-templates/create-campaign.md

@@ -179,17 +179,11 @@ person/company this campaign is for, then I’ll turn that into a campaign brief
 before we move into lead sourcing.
 ```
 
-Exception: if `bootstrap_create_campaign.modelQuality.status === "warn"` and
-`bootstrap_create_campaign.modelQuality.metadataStale !== true`, the first
-visible campaign message must be the model-quality warning from
-`modelQuality.message`. Ask the user to switch to the configured minimum model
-or explicitly continue anyway before identity setup, research, lead filtering,
-message generation, or launch review. If `metadataStale === true`, continue
-normally and do not ask the user to switch.
-
-If `bootstrap_create_campaign.modelQuality.metadataStale === true`, continue
-normally. Do not ask the user to switch models; this is an accepted Codex host
-metadata mismatch.
+Exception: if `bootstrap_create_campaign.modelQuality.status === "warn"`, the
+first visible campaign message must be the model-quality warning from
+`modelQuality.message`. Only trust that warning when bootstrap received active
+turn/runtime metadata or an explicit user-confirmed model. Do not warn from
+config defaults, stale host labels, or inferred model names.
 
 If a linked/local skill file is stale or missing, silently use the installed
 `sellable@sellable` plugin copy. Do not tell the user about the stale link,
@@ -207,6 +201,55 @@ clear business decisions, tradeoffs, and approval gates. Use product language:
 - "I can create a draft shell for you to watch with approval gates before
   sourcing", not mutation jargon
 
+## Active Model Metadata
+
+Before calling `bootstrap_create_campaign`, collect the active host model data
+only from a source that can describe this turn:
+
+- Codex: if `mcp__node_repl__js` is available, inspect only
+  `nodeRepl.requestMeta["x-codex-turn-metadata"].model` and
+  `nodeRepl.requestMeta["x-codex-turn-metadata"].reasoning_effort`. Pass those
+  values as `model` and `reasoningEffort`, with
+  `modelMetadataSource: "codex_turn_metadata"`.
+- Codex fallback/cross-check: if active turn metadata is unavailable, you may
+  inspect `~/.codex/config.toml` for `model` and `model_reasoning_effort`, but
+  pass `modelMetadataSource: "codex_config_fallback"` and do not treat it as a
+  reason to ask the user to switch models. Active turn metadata wins if it
+  differs from config.
+- Claude Code / Opus: use active Claude runtime metadata only if the current
+  host exposes model and effort for this same session, with
+  `modelMetadataSource: "claude_runtime_metadata"`. Current Claude Code MCP
+  tool calls may not include model or effort metadata; when they do not, omit
+  `model`, `reasoningEffort`, and `modelMetadataSource`.
+- Claude Code session-context self-report: if your current Claude session
+  context explicitly states both the exact model ID and effort/thinking level,
+  report it internally in this shape and pass it to bootstrap with
+  `modelMetadataSource: "claude_session_context"`:
+
+  ```text
+  - Model (name): ...
+  - Model (ID): ...
+  - Reasoning effort: ...
+  - Source: active Claude Code session context
+  ```
+
+  Use this only when the values are explicitly present in the current session
+  context. Do not infer an ID from the friendly name, do not infer effort from
+  `alwaysThinkingEnabled`, and do not show this self-report to the user during
+  normal campaign setup.
+- Do not run a nested `claude -p`, inspect `~/.claude/settings.json`, or read
+  Claude CLI defaults as proof of the user's active Claude Code session. Those
+  checks can validate a new child session or saved defaults, but not this
+  session's actual model and effort.
+- If the user explicitly provides active Claude `/status` or `/model` output
+  that includes both model and effort, pass it with
+  `modelMetadataSource: "user_confirmed"`. If it is missing either model or
+  effort, treat the metadata as unknown and continue.
+
+Never invent the model or reasoning effort. Never pass config defaults as active
+metadata. If bootstrap returns `modelQuality.status === "unknown"`, continue
+without asking the user to switch models.
+
 Approval and safety copy should be tasteful. State what the current approval
 covers once, in one short sentence, then move on. Do not append repeated
 "nothing starts / no leads import / no sending" disclaimers to routine progress
@@ -938,13 +981,19 @@ updates.
    - Do not call `mcp__sellable__get_campaigns`.
    - Do not call `mcp__sellable__get_campaign` to hunt for IDs.
    - Do not call `mcp__sellable__create_campaign({ campaignId: ... })` unless the user supplied that id.
-6. Call `mcp__sellable__bootstrap_create_campaign({ flowVersion: "v2", campaignId?, host?, model?, reasoningEffort? })`.
-   Pass the current host, model, and reasoning when the host exposes them.
+6. Call `mcp__sellable__bootstrap_create_campaign({ flowVersion: "v2", campaignId?, host?, model?, reasoningEffort?, modelMetadataSource? })`.
+   Pass model metadata only when collected by the Active Model Metadata rules
+   above. For Codex active turn metadata, pass
+   `modelMetadataSource: "codex_turn_metadata"`. For explicit Claude session
+   context, pass `modelMetadataSource: "claude_session_context"`. For explicit
+   user-confirmed Claude `/status` or `/model` output, pass
+   `modelMetadataSource: "user_confirmed"` only when it includes both model and
+   effort.
 7. If `safeToProceed !== true`, stop and show `blockingErrors` + `nextStep`.
-8. If `modelQuality.status === "warn"` and `modelQuality.metadataStale !== true`,
-   show `modelQuality.message` before any setup/research and wait for the user
-   to switch or explicitly continue. If `metadataStale === true`, continue
-   normally and do not tell the user to switch.
+8. If `modelQuality.status === "warn"`, show `modelQuality.message` before any
+   setup/research and wait for the user to switch or explicitly continue. If
+   `modelQuality.status === "unknown"`, continue; do not tell the user to
+   switch models.
 
 ## Execute Workflow