@mediagato/modelreins-channel 4.11.1 → 4.11.3

package/modelreins-channel.mjs

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 /**
  * @module modelreins-channel
- * @version 4.6.0
+ * @version 4.11.3
  *
  * ModelReins MCP Channel
  * ======================
@@ -134,6 +134,12 @@
  *                          you can do locally, delegate the rest to a headless
  *                          worker with more model capacity.
  *
+ *   modelreins_result    — Read back a job's actual output: full metadata plus
+ *                          the recorded output stream. This is the other half
+ *                          of dispatch — without it, dispatched work is a one
+ *                          way street. Works on running jobs too (partial
+ *                          output so far).
+ *
  *   modelreins_status    — Snapshot of the entire fleet: workers, recent jobs,
  *                          active schedules. Useful for situational awareness
  *                          before dispatching ("is worker-01 free right now?").
@@ -161,6 +167,10 @@
  *   Channel polls → pulls job → notification → Claude executes →
  *   Claude calls modelreins_complete → POST /jobs/{id}/output + PUT /jobs/{id}
  *   → dashboard updates in real time.
+ *
+ * Readback flow (the return trip):
+ *   Claude calls modelreins_result → GET /jobs/{id} + GET /jobs/{id}/output
+ *   → job metadata + recorded output rendered back into the session.
  */
 
 import { Server } from '@modelcontextprotocol/sdk/server/index.js';
@@ -273,7 +283,7 @@ const STUCK_JOB_MS = 30 * 60 * 1000;  // 30 min — generous Claude completion w
 // without user interaction.
 
 const mcp = new Server(
-    { name: 'modelreins', version: '4.6.0' },
+    { name: 'modelreins', version: '4.11.3' },
     {
         capabilities: {
             // Enable the Claude-specific channel notification method.
@@ -306,6 +316,11 @@ const mcp = new Server(
             'DISPATCHING TO OTHER WORKERS:',
             'Use modelreins_dispatch to hand off subtasks. This is powerful: you can split a large job,',
             'keep the local/interactive parts for yourself, and delegate batch work to headless workers.',
+            '',
+            'READING RESULTS BACK:',
+            'Use modelreins_result to read a job\'s actual output — dispatch alone is a one way street.',
+            'After a dispatched job completes (check with modelreins_status), fetch its real result',
+            'before reporting on it or building on it. Works mid-run too for partial output.',
         ].join('\n'),
     }
 );
@@ -397,6 +412,37 @@ mcp.setRequestHandler(ListToolsRequestSchema, async () => ({
             },
         },
 
+        // ── modelreins_result ─────────────────────────────────────────────────
+        {
+            name: 'modelreins_result',
+            description: [
+                'Read back a job\'s actual result: full metadata (status, worker, timing, exit code)',
+                'plus the recorded output stream. This is the other half of modelreins_dispatch —',
+                'without it, dispatched work is a one way street.',
+                '',
+                'Use after modelreins_status shows a dispatched job as done/failed, to get the real',
+                'output instead of just a status flag. Also works on running jobs — returns whatever',
+                'partial output the worker has streamed so far.',
+                '',
+                'Output is tail-truncated at max_chars (the end of a job\'s output carries the',
+                'summary/conclusion). Truncation is always reported explicitly, never silent.',
+            ].join('\n'),
+            inputSchema: {
+                type: 'object',
+                properties: {
+                    job_id: {
+                        type: 'string',
+                        description: 'The job ID to read back (from modelreins_dispatch or modelreins_status).',
+                    },
+                    max_chars: {
+                        type: 'number',
+                        description: 'Max characters of output to return (tail-biased). Default 30000, max 200000.',
+                    },
+                },
+                required: ['job_id'],
+            },
+        },
+
         // ── modelreins_status ─────────────────────────────────────────────────
         {
             name: 'modelreins_status',
@@ -628,6 +674,74 @@ mcp.setRequestHandler(CallToolRequestSchema, async (req) => {
         return { content: [{ type: 'text', text: lines.join('\n') }] };
     }
 
+    // ── modelreins_result ─────────────────────────────────────────────────────
+
+    if (name === 'modelreins_result') {
+        const jobId = String(args.job_id).trim();
+        const maxChars = Math.max(1000, Math.min(args.max_chars ?? 30000, 200000));
+
+        const [jobRes, outRes] = await Promise.all([
+            api('GET', `/jobs/${jobId}`),
+            api('GET', `/jobs/${jobId}/output?limit=500`),
+        ]);
+
+        // A missing job and an unreachable server are NOT the same finding —
+        // same discipline as the workers block in modelreins_status.
+        if (jobRes.status === 404) {
+            return { content: [{ type: 'text', text: `Job #${jobId} not found (404). Check the ID — jobs are tenant-scoped, so a job dispatched under another token is invisible here.` }] };
+        }
+        if (jobRes.status < 200 || jobRes.status >= 300) {
+            return { content: [{ type: 'text', text: `COULD NOT READ job #${jobId} — HTTP ${jobRes.status}${jobRes.data?.error ? ` (${jobRes.data.error})` : ''}. This is a failed check, not a missing job.` }] };
+        }
+
+        const job = jobRes.data.job || {};
+        const lines = [`## Job #${jobId} — ${job.status || 'unknown'}`];
+        const meta = [];
+        if (job.assigned_to) meta.push(`worker: ${job.assigned_to}`);
+        if (job.model_preference) meta.push(`model: ${job.model_preference}`);
+        if (job.exit_code !== null && job.exit_code !== undefined) meta.push(`exit: ${job.exit_code}`);
+        if (job.created_at) meta.push(`created: ${job.created_at}`);
+        if (job.completed_at) meta.push(`completed: ${job.completed_at}`);
+        if (meta.length) lines.push(meta.join(' · '));
+        if (job.error) lines.push(`error: ${job.error}`);
+        if (job.prompt) {
+            const p = String(job.prompt);
+            lines.push('', `### Prompt${p.length > 300 ? ' (first 300 chars)' : ''}`, p.slice(0, 300));
+        }
+
+        if (outRes.status < 200 || outRes.status >= 300) {
+            lines.push('', `### Output — COULD NOT READ (HTTP ${outRes.status}${outRes.data?.error ? `: ${outRes.data.error}` : ''})`);
+            lines.push('This is a failed read, not an empty result.');
+            return { content: [{ type: 'text', text: lines.join('\n') }] };
+        }
+
+        const rows = outRes.data?.output || [];
+        const systemRows = rows.filter(r => r.stream === 'system');
+        const bodyRows = rows.filter(r => r.stream !== 'system');
+        // Join with '' — workers stream token/text DELTAS that carry their own
+        // whitespace (verified against job output from both the Node worker
+        // daemon and LlamaStash). Joining with '\n' renders one token per line.
+        const body = bodyRows.map(r => r.content).join('');
+
+        if (rows.length === 0) {
+            lines.push('', '### Output', 'No output rows recorded for this job. The worker never posted output — for a done job that usually means it completed via a bare status update without a summary.');
+        } else {
+            let shown = body;
+            let truncNote = '';
+            if (body.length > maxChars) {
+                shown = body.slice(-maxChars);
+                truncNote = ` — TRUNCATED, showing last ${maxChars} of ${body.length} chars (raise max_chars for more)`;
+            }
+            lines.push('', `### Output (${bodyRows.length} chunk${bodyRows.length === 1 ? '' : 's'}, ${body.length} chars${truncNote})`, shown);
+            if (systemRows.length) {
+                lines.push('', `### System notes (${systemRows.length})`);
+                for (const r of systemRows.slice(-10)) lines.push(`  ${r.content}`);
+            }
+        }
+
+        return { content: [{ type: 'text', text: lines.join('\n') }] };
+    }
+
     // ── modelreins_status ─────────────────────────────────────────────────────
 
     if (name === 'modelreins_status') {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mediagato/modelreins-channel",
-  "version": "4.11.1",
+  "version": "4.11.3",
   "description": "ModelReins MCP Channel — turn any Claude Code session into a ModelReins worker",
   "type": "module",
   "main": "modelreins-channel.mjs",
@@ -10,13 +10,7 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.12.0"
   },
-  "keywords": [
-    "modelreins",
-    "mcp",
-    "claude-code",
-    "channel",
-    "ai-orchestration"
-  ],
+  "keywords": ["modelreins", "mcp", "claude-code", "channel", "ai-orchestration"],
   "license": "MIT",
   "homepage": "https://modelreins.com"
 }