@opennous/mcp 0.44.1 → 0.44.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opennous/mcp",
-  "version": "0.44.1",
+  "version": "0.44.2",
   "description": "Nous — the Context Graph for AI Agents.",
   "license": "Apache-2.0",
   "repository": {

package/src/client.js

@@ -71,8 +71,8 @@ function currentApiUrl() {
 export function validateConfig() {
   if (!resolvedEnv("NOUS_API_KEY") && !fileApiKey()) {
     throw new Error(
-      "No Nous API key found. Run the /nous-login command (or `npx @opennous/cli login`) to sign in, " +
-      "or set NOUS_API_KEY."
+      "No Nous API key found. Run `npx @opennous/cli login` to sign in (or `npx @opennous/cli init` " +
+      "to set up from scratch), or set NOUS_API_KEY."
     );
   }
 }

package/src/index.js

@@ -17,10 +17,10 @@ import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"
 import { validateConfig } from "./client.js";
 import { createServer } from "./server.js";
 
-// Advisory only — don't hard-exit if there's no key yet. The user may install
-// the plugin and then run /nous-login; the server must already be running so the
-// key (resolved per-call from env or ~/.nous/config.json) is picked up without a
-// restart.
+// Advisory only — don't hard-exit if there's no key yet. The user may register
+// the MCP first and sign in after (`npx @opennous/cli login`); the server must
+// already be running so the key (resolved per-call from env or ~/.nous/config.json)
+// is picked up without a restart.
 try {
   validateConfig();
 } catch (err) {

package/src/server.js

@@ -1156,6 +1156,12 @@ export function createServer() {
           ...imp.map((s) => `  • ${s.section}${s.source_path ? `  ← ${s.source_path}` : ""}`),
         ];
         if (r.skipped?.length) lines.push("", `Skipped (unknown/empty): ${r.skipped.join(", ")}`);
+        // Section-check nudges from the server — the ICP file synced but is missing
+        // canonical sections (buyer, fit, triggers, …). Surface them so the agent
+        // rounds the file out and re-syncs, instead of the gaps passing silently.
+        for (const w of (r.warnings ?? [])) {
+          lines.push("", `⚠ ${w.message}`);
+        }
         const sig = r.signals ?? [];
         if (r.model_status === "created" && sig.length) {
           lines.push("", `Built the ICP scoring model — ${sig.length} signal${sig.length === 1 ? "" : "s"}.`);
@@ -1205,9 +1211,15 @@ export function createServer() {
           "No ICP scoring model yet. Sync the user's ICP file with sync_icp first (or build one with " +
           "build_icp_model / train_icp_model), then call this to write it back." }] };
       }
-      const note = r.has_outcomes
+      // Lift can only be learned from WINS, so a win-less cohort (losses only) is
+      // still seed estimates — don't claim it's "trained on real closed deals".
+      const wonCount = r.calibration?.won ?? 0;
+      const lostCount = r.calibration?.lost ?? 0;
+      const note = wonCount > 0
         ? "This model is trained on real closed deals (lift + calibration shown)."
-        : "This model is seeded from the ICP only — add closed deals with train_icp_model to sharpen it.";
+        : lostCount > 0
+          ? `Still seed estimates — ${lostCount} closed-lost recorded but no closed-won yet to learn lift from. Add closed-won with train_icp_model to sharpen it.`
+          : "This model is seeded from the ICP only — add closed deals with train_icp_model to sharpen it.";
       return { content: [{ type: "text", text:
         `Write the block below into ${r.target_path} with your file editor — replace any existing block ` +
         `between the nous:icp markers, or append it if there's none (create the file/section if absent). ` +