@opennous/mcp 0.61.0 → 0.63.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@opennous/mcp",
-  "version": "0.61.0",
-  "description": "Nous — the Context Graph for AI Agents.",
+  "version": "0.63.0",
+  "description": "Nous \u2014 the Context Graph for AI Agents.",
   "license": "Apache-2.0",
   "repository": {
     "type": "git",
@@ -21,7 +21,8 @@
     "start": "node src/index.js",
     "dev:http": "node --watch src/http.js",
     "start:http": "node src/http.js",
-    "typecheck": "echo 'no ts in mcp — skipping'"
+    "typecheck": "echo 'no ts in mcp \u2014 skipping'",
+    "test": "node --test tests/*.test.mjs"
   },
   "keywords": [
     "mcp",

package/src/server.js

@@ -29,7 +29,7 @@ const APP_URL = () => (process.env.NOUS_APP_URL || "https://app.opennous.cloud")
 const OAUTH_CONNECT = { gmail: "gmail", gmail_oauth: "gmail", google: "gmail", "google-mail": "gmail", googlemail: "gmail", linkedin: "linkedin" };
 const connectLink = (slug) => `${APP_URL()}/settings?section=integrations&connect=${slug}`;
 
-export const SERVER_VERSION = "0.60.0";
+export const SERVER_VERSION = "0.62.0";
 
 // ─── helpers ──────────────────────────────────────────────────────────────────
 
@@ -91,6 +91,24 @@ const pct     = (c) => `${Math.round((c ?? 0) * 100)}%`;
 
 // ─── factory ──────────────────────────────────────────────────────────────────
 
+// A closed deal as the agent may supply it. A bare domain is the original shape and still
+// works; the object form carries the money and the date, which the forecast needs. Widened
+// rather than replaced so nothing that already calls this breaks.
+//
+// Module scope, not inside createServer(). It was declared below its own first use, and a
+// `const` in the temporal dead zone throws on every request — the hosted server crashed on
+// each call with "Cannot access 'CLOSED_DEAL' before initialization". Out here it is also
+// built once instead of per request, since createServer() runs for every incoming call.
+const CLOSED_DEAL = z.union([
+  z.string(),
+  z.object({
+    domain: z.string().describe("Company domain, e.g. 'acme.com' — no scheme."),
+    amount: z.number().optional().describe("Deal value in major units (48000, not 4800000)."),
+    currency: z.string().optional().describe("ISO code or symbol. Defaults to the workspace currency."),
+    closed_at: z.string().optional().describe("When it actually closed, ISO date. Drives every cycle-time measurement, so pass the real date rather than today's."),
+  }),
+]);
+
 export function createServer() {
   const server = new McpServer({
     name: "nous",
@@ -282,6 +300,17 @@ export function createServer() {
         lines.push(`ICP FIT: ${rec.icp.score}/100 — ${label}${rec.icp.reason ? `  (${rec.icp.reason})` : ""}`);
         lines.push("");
       }
+      if (rec.scorecard) {
+        // The latest call scorecard the app computed (Coaching v2). The after-call skill writes the
+        // coaching review out of THIS — it does not re-score. Each dimension carries its exact moment.
+        const sc = rec.scorecard;
+        lines.push(`LAST CALL SCORE: ${sc.overall}/100 · ${sc.next_step_secured ? "next step secured" : "no next step secured"}${sc.occurred_at ? `  (${relAge(sc.occurred_at)})` : ""}`);
+        for (const d of (sc.scores ?? [])) {
+          lines.push(`  [${d.score}] ${d.label}${d.note ? ` — ${d.note}` : ""}`);
+          if (d.moment) lines.push(`      "${d.moment}"`);
+        }
+        lines.push("");
+      }
       if (rec.facts?.length) {
         // Atomic memory — the durable, decision-relevant facts learned about them.
         lines.push(`FACTS (${rec.facts.length} — durable memory about them):`);
@@ -390,8 +419,12 @@ export function createServer() {
     "Insights about YOUR OWN business (product/positioning/market/buyer) go to record_insight, not here. " +
     "CLOSING THE LOOP — when you RECOMMEND an action, write the recommendation down before it happens: " +
     "{kind:'state',property:'decision.proposed',value:{decision_id:'<your id>',proposal:'send a follow-up " +
-    "naming the security objection',rationale:'the security review is the real blocker at this stage', " +
+    "naming the security objection',proposal_body:'<the exact draft>',rationale:'the security review " +
+    "is the real blocker at this stage', " +
     "evidence_ids:['<claim/note id you reasoned from>'],action_type:'email_send',category:'outreach'}}. " +
+    "`proposal_body` is the draft itself — hashed and discarded, never stored, and the only way to tell " +
+    "later whether the human sent it as written or rewrote it first. Omit it and the decision reports as " +
+    "sent-verbatim whatever happened. " +
     "Keep `proposal` (WHAT you are doing) apart from `rationale` (WHY you think it works), and list the " +
     "facts you actually reasoned from in `evidence_ids` — that is what lets Nous learn which KINDS of " +
     "evidence are worth acting on, not just which actions worked. When the human answers, " +
@@ -745,19 +778,6 @@ export function createServer() {
 
 
 
-  // ===========================================================================
-  // A closed deal as the agent may supply it. A bare domain is the original shape and still
-  // works; the object form carries the money and the date, which the forecast needs. Widened
-  // rather than replaced so nothing that already calls this breaks.
-  const CLOSED_DEAL = z.union([
-    z.string(),
-    z.object({
-      domain: z.string().describe("Company domain, e.g. 'acme.com' — no scheme."),
-      amount: z.number().optional().describe("Deal value in major units (48000, not 4800000)."),
-      currency: z.string().optional().describe("ISO code or symbol. Defaults to the workspace currency."),
-      closed_at: z.string().optional().describe("When it actually closed, ISO date. Drives every cycle-time measurement, so pass the real date rather than today's."),
-    }),
-  ]);