@nooticr/mcp 1.26.27 → 1.26.29

package/dist/shared/tools.js

@@ -14,7 +14,7 @@ import { OUTPUT_SCHEMAS, anyObject } from "./output-schemas.js";
 import { deriveHashtags, derivedNote } from "./hashtags.js";
 import { createTaskStore, registerSlowTool } from "./tasks.js";
 import { COMMENT_CATEGORIES, COMMENT_SENTIMENTS, platformFromUrl, reviewGuidance, toEvidence, } from "./comment-review.js";
-import { costSentence, EVIDENCE_PLANS, fetchBillingNote, frameIndex, framesToBlocks, planCalls, scoreDraftGuidance, } from "./evidence.js";
+import { costSentence, EVIDENCE_PLANS, fetchBillingNote, frameIndex, framesToBlocks, planCalls, scoreDraftGuidance, KNOWN_PLATFORMS, platformFailureGuidance, } from "./evidence.js";
 import { confirmSpend, declinedResult, searchMentionsCost, SEARCH_PLATFORMS, DISCOVERABLE_PLATFORMS, } from "./spend.js";
 import { BackendWatchStore, MemoryWatchStore, registerWatchlist, } from "./watchlist.js";
 import { registerJobTools } from "./jobs.js";
@@ -25,7 +25,7 @@ import { loadingPlansJson } from "./loading-plans.js";
 import { registerHandoff } from "./handoff.js";
 import { registerCollabTools } from "./collab.js";
 /** Current MCP server version — bumped on every deploy for traceability. */
-export const MCP_SERVER_VERSION = "1.26.27";
+export const MCP_SERVER_VERSION = "1.26.29";
 /** MCP Apps extension identifier */
 const UI_EXTENSION = "io.modelcontextprotocol/ui";
 /** MIME type for MCP Apps HTML resources */
@@ -342,6 +342,21 @@ function toolError(prefix, err) {
             isError: true,
         };
     }
+    // A network that ERRORED is not a network that came back empty, and the two
+    // want opposite responses: an empty result means try a different query, an
+    // error means try a different network or tell the user. Nothing said which,
+    // so a host did the one thing that cannot work — re-ran the same call with
+    // the words changed, three times, each a paid upstream call. The backend
+    // names the platform in these messages ("... failed: reddit: ..."), so the
+    // split is recoverable here without every one of the 26 call sites having to
+    // pass its arguments down.
+    const platform = KNOWN_PLATFORMS.find((p) => new RegExp(`(^|[^a-z])${p}([^a-z]|$)`, "i").test(msg));
+    if (platform) {
+        return {
+            content: [{ type: "text", text: `${prefix}. ${platformFailureGuidance({ platform, message: msg })}` }],
+            isError: true,
+        };
+    }
     return { content: [{ type: "text", text: `${prefix}: ${msg}` }], isError: true };
 }
 /**
@@ -377,16 +392,34 @@ async function runEvidence(tool, args, client) {
             extra = { unavailable: plan.also.via };
         }
     }
-    // The guidance leads. A model reads the first text block in a result, and
-    // that is where the account of what to produce has to be — everything below
-    // it is material with no instruction attached.
+    // The guidance rides in BOTH channels, and the structured one is the one
+    // that arrives.
+    //
+    // This used to say "the guidance leads: a model reads the first text block in
+    // a result, and that is where the account of what to produce has to be". That
+    // is what the MCP spec allows, and it is not what hosts do. Claude Code
+    // discards every `content` text block whenever a result carries
+    // `structuredContent`, keeps the non-text blocks, and shows the model the
+    // serialised JSON instead — unconditionally, with no setting, and without
+    // consulting `outputSchema`. Every tool here declares one, so every guidance
+    // string this file writes was landing nowhere.
+    //
+    // Measured before changing it, over 59 real runs of the quest suite: 0
+    // guidance phrases across 175 tool results. `analyze_post` handed back frames,
+    // a transcript and 2,000 characters ending in "call show_analysis when you are
+    // done"; the model wrote the analysis and delivered it in chat, and
+    // show_analysis was called in 1 run of 12.
+    //
+    // The text block stays for hosts that honour it (and for the images, which
+    // must sit in `content`). `guidance` on the payload is what reaches the rest.
+    // See docs/testing/tool-chaining-quests.md.
     const billing = nothingToFetch
         ? "Nothing was fetched for this call, so nothing was charged."
         : fetchBillingNote(tool);
-    const content = [
-        { type: "text", text: `${plan.guidance(args)}\n\n${billing}` },
-    ];
+    const guidance = `${plan.guidance(args)}\n\n${billing}`;
+    const content = [{ type: "text", text: guidance }];
     const out = {
+        guidance,
         // Kept, and constant. It is not an echo of an argument — the job tools in
         // jobs.ts set the same key with no argument to echo — it marks a payload as
         // material the caller still has to read. Dropping it would break anything
@@ -525,7 +558,12 @@ export function createMcpServer(rawMakeClient, opts) {
         // a search_mentions result render in the monitoring view.
         "answer_my_audience",
         "show_audience_replies",
-        "track_competitor",
+        "track_creator",
+        // The comparison track_creator computes per creator and never puts side
+        // by side, plus the free view that draws the table once a read is written.
+        "compare_creators",
+        "watchlist_standings",
+        "show_standings",
         "who_should_i_work_with",
         "why_did_this_underperform",
         "what_should_i_make_next",
@@ -533,6 +571,10 @@ export function createMcpServer(rawMakeClient, opts) {
         // hits rather than comments, so it gets its own view rather than the
         // monitoring one.
         "search_spoken_mentions",
+        // Prospect discovery: the same keyword search as discover_social_posts, but
+        // widened into the shapes a complaint takes rather than the topic it is
+        // about. Returns `posts`, so the gallery view draws it.
+        "find_people_with_problem",
         // Own-account intelligence (own-account.ts). list_own_apps stays
         // view-less like watch_creator — it lists metadata, nothing to draw.
         // get_scheduled_posts and get_post_performance return a `posts` array
@@ -902,7 +944,11 @@ export function createMcpServer(rawMakeClient, opts) {
             "Work out their niche, recurring themes, hook formula, what over- and under-performs and who " +
             "their audience is, reading the spread of the numbers rather than only the best post, and " +
             `name the posts you reason from. ${costSentence("analyze_creator_profile")} ` +
-            "Use for the teardown itself; find_hook_pattern fetches the same posts and asks only for the formula.",
+            "Use for the teardown itself; find_hook_pattern fetches the same posts and asks only for the " +
+            "formula. NOT for keeping an eye on a rival over time: \"track X\", \"how are they doing " +
+            "lately\", \"what have they shipped since I last looked\" are track_creator, which scores " +
+            "each post against that creator's own median and remembers where you left off. This tool " +
+            "returns the posts and no verdict, and remembers nothing between calls.",
         _meta: {
             ui: { resourceUri: uiResource("analyze_creator_profile") },
             "ui/resourceUri": uiResource("analyze_creator_profile"),
@@ -1155,9 +1201,14 @@ export function createMcpServer(rawMakeClient, opts) {
             const res = await client.callTool("get_post_comments", { ...args });
             const structured = (res.structured ?? {});
             const comments = toEvidence(args.url, structured.comments);
+            const guidance = reviewGuidance(args.url, comments.length);
             return {
-                content: [{ type: "text", text: reviewGuidance(args.url, comments.length) }],
+                content: [{ type: "text", text: guidance }],
                 structuredContent: {
+                    // See runEvidence: a text block alone reaches no host that renders
+                    // structuredContent, and this is the tool show_comment_review's whole
+                    // chain hangs off.
+                    guidance,
                     mode: "evidence",
                     url: args.url,
                     platform: structured.platform ?? null,
@@ -1418,6 +1469,62 @@ export function createMcpServer(rawMakeClient, opts) {
             },
         };
     });
+    server.registerTool("show_standings", {
+        title: "Show Standings",
+        description: "Draw the standings you read out of compare_creators or watchlist_standings. Free, and makes " +
+            "no requests — it only renders what you pass it: one row per creator with their window, their " +
+            "own median, how often they beat it and how hard, and the post that did best. Pass `ranking` " +
+            "to say which axis you ordered on, and `tooThin` for the creators whose window was too short " +
+            "to rank — a table that hides that is the one mistake this view can make on your behalf. " +
+            "Call this after you have decided what the numbers mean, not instead of deciding.",
+        _meta: {
+            ui: { resourceUri: uiResource("show_standings") },
+            "ui/resourceUri": uiResource("show_standings"),
+            "openai/outputTemplate": appsSdkResource("show_standings"),
+        },
+        annotations: { readOnlyHint: true, destructiveHint: false, idempotentHint: true, openWorldHint: false },
+        outputSchema: OUTPUT_SCHEMAS.show_standings,
+        inputSchema: z
+            .object({
+            creators: z
+                .array(anyObject())
+                .min(1)
+                .max(25)
+                .describe("The creator rows, in the order you decided on — the same shape compare_creators returned " +
+                "(handle, platform, window, baseline, hitRate, medianWinRatio, best, ...)."),
+            metric: z.string().optional().describe("Which stat the standings are on (views, likes, ...)."),
+            ranking: z
+                .string()
+                .optional()
+                .describe('What you ordered on and why — "how often they land one" and "how big it is when they do" ' +
+                "usually disagree, so the reader needs to know which they are looking at."),
+            verdict: z.string().optional().describe("Your read, in a sentence or two."),
+            tooThin: z
+                .array(z.string())
+                .optional()
+                .describe("Handles whose window was too short to rank. Drawn as unranked rather than bottom, " +
+                "because missing from a comparison is not the same as losing it."),
+        })
+            .strict(),
+    }, async (args) => {
+        return {
+            content: [
+                {
+                    type: "text",
+                    text: `Showing standings for ${args.creators.length} creator${args.creators.length === 1 ? "" : "s"}.` +
+                        (args.verdict ? ` ${args.verdict}` : ""),
+                },
+            ],
+            structuredContent: {
+                creators: args.creators,
+                metric: args.metric ?? "views",
+                ranking: args.ranking ?? null,
+                verdict: args.verdict ?? null,
+                tooThin: args.tooThin ?? [],
+                standings: true,
+            },
+        };
+    });
     server.registerTool("show_comparison", {
         title: "Show Comparison",
         description: "Display a comparison you wrote after compare_posts fetched the first post and you fetched " +
@@ -1860,16 +1967,24 @@ export function createMcpServer(rawMakeClient, opts) {
     // argument: the draft is the caller's own text, so the backend was only ever
     // being paid to hold it up against a standard, and the standard is what
     // comes back instead. Nothing here can fail, which is why nothing is caught.
-    async (args) => ({
-        content: [{ type: "text", text: scoreDraftGuidance(args.draft, String(args.platform ?? "")) }],
-        structuredContent: {
-            draft: args.draft,
-            platform: args.platform ?? "tiktok",
-            // Same shape the free tools use, so a caller totting up a session's spend
-            // does not have to special-case this one.
-            mcpCredits: { cost: 0 },
-        },
-    }));
+    async (args) => {
+        const guidance = scoreDraftGuidance(args.draft, String(args.platform ?? ""));
+        return {
+            content: [{ type: "text", text: guidance }],
+            structuredContent: {
+                // The rubric IS this tool's answer — it fetches nothing and scores
+                // nothing itself. In a text block it reached no host that renders
+                // structuredContent, which made the tool return its own input and a
+                // credit note. See runEvidence.
+                guidance,
+                draft: args.draft,
+                platform: args.platform ?? "tiktok",
+                // Same shape the free tools use, so a caller totting up a session's spend
+                // does not have to special-case this one.
+                mcpCredits: { cost: 0 },
+            },
+        };
+    });
     server.registerTool("repurpose_post", {
         title: "Repurpose Post",
         description: "The source post, its transcript and its stats, for you to rewrite for other surfaces — X " +
@@ -2186,7 +2301,7 @@ export function createMcpServer(rawMakeClient, opts) {
         }
     });
     registerPrompts(server);
-    // One store for both. track_competitor keeps its "since I last looked" marker
+    // One store for both. track_creator keeps its "since I last looked" marker
     // on the same watchlist entries, in its own field — two stores would mean a
     // creator you watch and a creator you track were different people.
     //

package/dist/shared/ui-template.js

@@ -2782,7 +2782,7 @@ export const NOOTICR_UI_TEMPLATE = `<!DOCTYPE html>
   /**
    * The one line a competitor sweep or an underperformance check actually
    * exists to produce — ratio to the creator's own median, and the verdict
-   * that ratio earns. track_competitor and why_did_this_underperform attach
+   * that ratio earns. track_creator and why_did_this_underperform attach
    * "standing" to every post they score (see performance.ts), and until this
    * existed postCard rendered the same view/like/comment pills any gallery
    * gets, so the computed comparison a person paid for never reached the
@@ -3661,6 +3661,64 @@ export const NOOTICR_UI_TEMPLATE = `<!DOCTYPE html>
     if(d.posts&&Array.isArray(d.posts)){
       if(!d.posts.length){app.innerHTML='<div class="empty-state fade-in"><div class="icon">🔍</div><div class="text">No posts found</div></div>';return;}
       app.innerHTML=galleryWrap(d.posts.map(function(p){return postCard(p,false,true);}).join(""),d.posts.length)+pickBarHtml();initPicks();return;}
+    // Standings: creators on one normalised axis. A table, because the question
+    // is comparative — and every row carries the window it was scored over. A
+    // hit rate whose denominator is invisible cannot be told apart from one
+    // that means something, which is why these numbers were computed per
+    // creator and never put side by side before (issue #30).
+    if(Array.isArray(d.creators)&&(d.standings||d.tool==="compare_creators"||d.tool==="watchlist_standings")){
+      var thin=Array.isArray(d.tooThin)?d.tooThin:[];
+      var floor=typeof d.thinWindow==="number"?d.thinWindow:8;
+      var above=typeof d.aboveRatio==="number"?d.aboveRatio:1.25;
+      var met=esc(d.metric||"views");
+      var rows=d.creators.map(function(c){
+        var un=c.unavailable;
+        // A window shorter than the floor, or one the model flagged, is drawn
+        // unranked. Bottom of a table is a claim; unranked is the truth.
+        var short=!un&&(thin.indexOf(String(c.handle))>=0||(typeof c.window==="number"&&c.window<floor));
+        var med=c.baseline&&c.baseline.median!=null?fmtNum(c.baseline.median):"-";
+        var hit=c.hitRate!=null?Math.round(c.hitRate*100)+"%":"-";
+        var win=c.medianWinRatio!=null?(Math.round(c.medianWinRatio*100)/100)+"x":"-";
+        var best=c.best&&(c.best.title||c.best.caption)?String(c.best.title||c.best.caption):"";
+        return '<tr style="border-top:1px solid var(--border-soft)'+(un?";opacity:.6":"")+'">'
+          +'<td style="padding:9px 10px;font-weight:600;white-space:nowrap">@'+esc(c.handle)
+          +(c.platform?'<span style="font-weight:400;color:var(--muted)"> · '+esc(c.platform)+"</span>":"")
+          +(short?'<span style="margin-left:6px;font-size:10px;font-weight:700;color:var(--warn)">too thin to rank</span>':"")
+          +"</td>"
+          +(un
+            ?'<td colspan="4" style="padding:9px 10px;color:var(--muted);font-size:12.5px">'+esc(String(un))+"</td>"
+            :'<td style="padding:9px 10px;text-align:right;font-variant-numeric:tabular-nums">'+fmtNum(c.window||0)+"</td>"
+              +'<td style="padding:9px 10px;text-align:right;font-variant-numeric:tabular-nums">'+med+"</td>"
+              +'<td style="padding:9px 10px;text-align:right;font-variant-numeric:tabular-nums">'+hit+"</td>"
+              +'<td style="padding:9px 10px;text-align:right;font-variant-numeric:tabular-nums">'+win+"</td>")
+          +"</tr>"
+          +(best&&!un
+            ?'<tr><td colspan="5" style="padding:0 10px 9px 10px;color:var(--muted);font-size:12px">best: '+esc(best.slice(0,120))+"</td></tr>"
+            :"");
+      }).join("");
+      app.innerHTML='<div class="card card-wide fade-in"><div class="card-body">'
+        +'<div style="font-size:16px;font-weight:700">Standings</div>'
+        +'<div class="faint" style="font-size:11.5px;color:var(--muted);margin:2px 0 10px">'
+        +d.creators.length+" creator"+(d.creators.length===1?"":"s")+" · each against their OWN median "+met
+        +(d.watching!=null?" · watchlist of "+fmtNum(d.watching):"")+"</div>"
+        +(d.ranking?'<div style="font-size:12.5px;margin:0 0 8px">Ordered on: '+esc(String(d.ranking))+"</div>":"")
+        +'<div style="overflow-x:auto"><table style="width:100%;border-collapse:collapse;font-size:13px">'
+        +'<thead><tr style="text-align:left;color:var(--muted);font-size:11px;text-transform:uppercase;letter-spacing:.05em">'
+        +'<th style="padding:0 10px 6px">Creator</th>'
+        +'<th style="padding:0 10px 6px;text-align:right">Window</th>'
+        +'<th style="padding:0 10px 6px;text-align:right">Their median</th>'
+        +'<th style="padding:0 10px 6px;text-align:right">Beat it</th>'
+        +'<th style="padding:0 10px 6px;text-align:right">By</th>'
+        +"</tr></thead><tbody>"+rows+"</tbody></table></div>"
+        +(d.verdict?'<div style="font-size:13.5px;line-height:1.6;margin-top:12px">'+esc(String(d.verdict))+"</div>":"")
+        +'<div class="faint" style="font-size:11.5px;color:var(--muted);margin-top:10px;line-height:1.6">'
+        +'"Beat it" is the share of the window at or above '+above+"x that creator's own median. "
+        +"A raw view count mostly measures follower count, so nothing here compares raw numbers. "
+        +"A window under "+floor+" posts is one post either way, and no ratio here says whether a "
+        +"baseline is itself moving — that needs a second point in time."
+        +"</div></div></div>";
+      return;}
+
     // Creators
     if(d.creators&&Array.isArray(d.creators)){
       if(!d.creators.length){app.innerHTML='<div class="empty-state fade-in"><div class="icon">👤</div><div class="text">No creators found</div></div>';return;}
@@ -3924,8 +3982,18 @@ export const NOOTICR_UI_TEMPLATE = `<!DOCTYPE html>
 
     // Single post
     if(d.post||d.platform){app.innerHTML=postCard(d.post||d,true);return;}
-    // Fallback
-    app.innerHTML='<div class="json-block fade-in">'+esc(JSON.stringify(d,null,2))+"</div>";
+    // Fallback. The guidance field is prose written for the model, not data —
+    // dumped into the JSON block it reads as a wall of escaped text, and it is
+    // the longest string in most payloads. Drawn above the block and dropped
+    // from it. This is also what finally gives score_draft a view worth
+    // looking at: its rubric IS its answer, and the card used to show the
+    // echoed draft and a credit cost with the rubric nowhere on screen.
+    var rest=d,lead="";
+    if(d&&typeof d==="object"&&typeof d.guidance==="string"){
+      lead='<div class="fade-in" style="white-space:pre-wrap;line-height:1.5;margin-bottom:12px">'+esc(d.guidance)+"</div>";
+      rest={};for(var gk in d){if(gk!=="guidance")rest[gk]=d[gk];}
+    }
+    app.innerHTML=lead+'<div class="json-block fade-in">'+esc(JSON.stringify(rest,null,2))+"</div>";
     setTimeout(reportSize,50);
   }
 

package/dist/shared/watchlist.js

@@ -33,11 +33,18 @@
  * registers, and only one registration under a name can win — but renaming
  * either side would break whichever hosts had already learned the old one,
  * for a tool whose whole promise is "what you called before still works". So
- * nothing about the surface moved. `watch_creator`, `unwatch_creator`,
- * `catch_up_watchlist` and `track_competitor` keep their names, arguments and
+ * nothing about the surface moved for THAT change. `watch_creator`,
+ * `unwatch_creator` and `catch_up_watchlist` keep their names, arguments and
  * behaviour; only the bytes moved, and the backend's own twins stay
  * unregistered here because they would be a second, disconnected list.
  *
+ * `track_creator` is the one exception, and for an unrelated reason: it was
+ * `track_competitor`, and that name was measured to be unreachable for the
+ * way most people ask for it — see its registration in jobs.ts. The backend
+ * still registers `track_competitor` for its own dashboard copilot, which is
+ * the ordinary state of two repos with two release cycles rather than a
+ * collision: the names no longer overlap, so nothing has to win.
+ *
  * The stores above are still live and still matter. They are the fallback for
  * an account with no workspace or a backend too old to have the tools, and
  * they are what the one-time migration reads from. Deleting one because "the
@@ -155,9 +162,10 @@ export class FileWatchStore {
  * and on ChatGPT had two, and a server-side scheduler could read neither.
  *
  * The swap is deliberately at `WatchStore` rather than at the tool layer.
- * `watch_creator`, `unwatch_creator`, `catch_up_watchlist` and
- * `track_competitor` keep their names, their arguments and their behaviour;
- * only where the bytes live changes. That is the whole reason the collision
+ * `watch_creator`, `unwatch_creator` and `catch_up_watchlist` keep their
+ * names, their arguments and their behaviour; only where the bytes live
+ * changes. (`track_creator` was later renamed from `track_competitor` for a
+ * reason that has nothing to do with storage — see above.) That is the whole reason the collision
  * with the backend's identically-named tools was not worth solving by
  * renaming anything: a host that already learned `watch_creator` keeps
  * working, and it now writes somewhere the other host can see.

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@nooticr/mcp",
   "mcpName": "com.nooticr/mcp",
-  "version": "1.26.27",
+  "version": "1.26.29",
   "description": "MCP server for nooticr - fetch, discover and understand TikTok/Instagram/YouTube/X/Reddit/LinkedIn/Weibo posts with AI (media metadata, niche discovery, brand monitoring, hook & viral analysis)",
   "homepage": "https://mcp.nooticr.com",
   "repository": {
@@ -38,9 +38,10 @@
     "test:e2e:visual": "playwright test tests/e2e/agentic-visual.e2e.ts tests/e2e/agentic-visual-full-app.e2e.ts",
     "test:e2e-smoke": "bash scripts/run-mechanical-e2e-smoke.sh",
     "test:e2e-smoke:fixture": "NOOTICR_E2E_BACKEND=fixture bash scripts/run-mechanical-e2e-smoke.sh",
-    "test:agentic-e2e": "bash scripts/run-agentic-evals.sh",
+    "test:quests": "bash scripts/run-quests.sh",
     "contract:host": "sh scripts/host-contract.sh",
-    "verify": "npm run build && npx tsc --noEmit && (cd cloudflare && npx tsc --noEmit) && npx vitest run && npm run contract:host && npm run conformance:mcpjam && npx playwright test",
+    "chain:map": "bash scripts/run-chain-map.sh",
+    "verify": "npm run build && npx tsc --noEmit && (cd cloudflare && npx tsc --noEmit) && npx vitest run && npm run contract:host && npm run chain:map && npm run conformance:mcpjam && npx playwright test",
     "contract:manifest": "node scripts/contract-check.mjs --from=../nooticr-server/platform-capabilities.json",
     "test:mutation": "node scripts/mutation-guard.mjs",
     "test:invariants": "node scripts/invariant-guard.mjs",