@north-light/crouter 0.3.314 → 0.3.315

package/dist/core/human/answer-text.d.ts

@@ -1,3 +1,5 @@
 import type { PageAnswer } from './answer.js';
-/** Render a PageAnswer as the deterministic Markdown report delivered to its asking agent. */
-export declare function renderAnswerText(answer: PageAnswer, responsePath: string): string;
+/** Render a PageAnswer as the deterministic Markdown report delivered to its asking agent.
+ *  The person sees this same prose in their own chat surface, so it carries the answer
+ *  itself and nothing addressed only to the agent. */
+export declare function renderAnswerText(answer: PageAnswer): string;

package/dist/core/human/answer-text.js

@@ -80,19 +80,21 @@ function slotLines(slot, totalSlots) {
         lines.push(...commentLines(comment));
     return lines;
 }
-/** Render a PageAnswer as the deterministic Markdown report delivered to its asking agent. */
-export function renderAnswerText(answer, responsePath) {
+/** Render a PageAnswer as the deterministic Markdown report delivered to its asking agent.
+ *  The person sees this same prose in their own chat surface, so it carries the answer
+ *  itself and nothing addressed only to the agent. */
+export function renderAnswerText(answer) {
     if (answer.kind === 'acknowledgement') {
         return [
             `The user saw **${answer.title}** and dismissed it. The page asked for nothing, so the acknowledgement is the whole answer.`,
             '',
-            `Acknowledged ${answer.completedAt}. Response record: ${responsePath}`,
+            `Acknowledged ${answer.completedAt}.`,
         ].join('\n');
     }
     const answered = answer.slots.filter((slot) => slot.state === 'answered');
     const lines = [`The user answered **${answer.title}**.`];
     for (const slot of answered)
         lines.push('', ...slotLines(slot, answer.slots.length));
-    lines.push('', `Answered ${answer.completedAt}. Typed responses: ${responsePath}`);
+    lines.push('', `Answered ${answer.completedAt}.`);
     return lines.join('\n');
 }

package/dist/daemon/human/finish.js

@@ -2,6 +2,11 @@ import { getRequest, getRequestByInboxTicket } from '../../core/human/requests.j
 import { buildPageSummary } from '../../core/human/summary.js';
 import { validateFinalPageResponses } from '../../core/human/tickets.js';
 import { composeHumanCompletion } from '../../core/human/completion.js';
+import { describePageAnswer } from '../../core/human/answer.js';
+import { renderAnswerText } from '../../core/human/answer-text.js';
+import { readPageFeedback } from '../../core/human/feedback.js';
+import { pageDocumentSha256 } from '../../core/human/feedback-companion.js';
+import { pagePath } from '../../core/human/convention.js';
 import { retireCompanion } from '../companion-retire.js';
 import { settleHumanRequest } from './settle.js';
 function requestForTicket(ticketId) {
@@ -38,10 +43,73 @@ function completionAction(request, result) {
         throw new Error(`corrupt frozen completion action: ${request.request_id}`, { cause: error });
     }
 }
-function notification(request, disposition, bodyJson) {
+function firstLine(text, maxLen = 80) {
+    const line = text.split('\n')[0] ?? '';
+    return line.length > maxLen ? `${line.slice(0, maxLen - 1)}…` : line;
+}
+function pageTitle(request) {
+    try {
+        return manifest(request).title;
+    }
+    catch {
+        return undefined;
+    }
+}
+/** The settle-time feedback addendum — answer and cancellation alike: a change
+ * notice when the final page differs from the version first commented on, and
+ * each still-open item's quote and note. Silent when the page is unchanged and
+ * nothing remains open; never a manufactured prose diff. */
+function feedbackAddendum(request) {
+    let sections;
+    try {
+        const feedback = readPageFeedback(request.request_dir);
+        if (feedback.companionNodeId === undefined)
+            return '';
+        sections = [];
+        if (feedback.firstCommentedSha256 !== undefined && pageDocumentSha256(request.request_dir) !== feedback.firstCommentedSha256) {
+            sections.push(`The page changed during feedback: the person commented on it and its companion conversation (node ${feedback.companionNodeId}) revised it in place. ` +
+                `Reread the final page at ${pagePath(request.request_dir)} before interpreting this outcome.`);
+        }
+        const open = feedback.comments.filter((comment) => comment.status === 'open');
+        if (open.length > 0) {
+            const items = open.map((comment) => [
+                ...comment.quote.split('\n').map((line) => `> ${line}`),
+                comment.note,
+            ].join('\n'));
+            sections.push([
+                `Feedback left unresolved at settlement — the companion conversation (node ${feedback.companionNodeId}) had not dealt with these items:`,
+                ...items,
+            ].join('\n\n'));
+        }
+    }
+    catch {
+        // The outcome itself is the deliverable; a broken feedback record adds nothing.
+        return '';
+    }
+    return sections.length === 0 ? '' : `\n\n${sections.join('\n\n')}`;
+}
+/** The person's answer as prose the asking agent reads directly. Short bodies
+ * ride inline; the mail layer spills a long one to a file and leaves a preview
+ * plus its path. */
+function answerBody(request, result) {
+    const answer = describePageAnswer(manifest(request), result.responses, { completedAt: result.completedAt });
+    return renderAnswerText(answer) + feedbackAddendum(request);
+}
+/** A dismissal is a rejection, not a routing event: say so plainly so the agent
+ * acts on the "no" instead of re-raising the same ask. */
+function cancelBody(request, dismissed, reason) {
+    const title = pageTitle(request);
+    const named = title === undefined ? `the human interaction ${request.request_id}` : `“${title}”`;
+    const why = reason !== undefined && reason !== '' ? reason : 'no reason was given';
+    const body = dismissed
+        ? `The user dismissed ${named} without answering — a deliberate “no, I'm not answering this.” Treat it as a rejection: no response will arrive, no partial answer exists, and the same ask must not be raised again.`
+        : `${title === undefined ? `The human interaction ${request.request_id}` : `The page “${title}”`} was canceled — ${why}. No response will arrive, and any partial answers the user had entered were discarded unsent.`;
+    return body + feedbackAddendum(request);
+}
+function notification(request, disposition, body, result) {
     if (request.requester_node_id === null)
         return undefined;
-    return { label: disposition === 'human-answer' ? 'human request answered' : 'human request canceled', disposition, body_json: bodyJson };
+    return { label: firstLine(body), disposition, body_json: { body, result } };
 }
 function retire(request) {
     return request.companion_node_id === null ? undefined : { retire: async (nodeId, outcome) => { await retireCompanion(nodeId, outcome); } };
@@ -60,7 +128,9 @@ export async function resolvePageTicket(ticketId, responses, opts = {}) {
         request_id: request.request_id,
         outcome: 'answered',
         result_json: JSON.stringify(result),
-        ...(notification(request, 'human-answer', result) === undefined ? {} : { notification: notification(request, 'human-answer', result) }),
+        ...(notification(request, 'human-answer', answerBody(request, result), result) === undefined
+            ? {}
+            : { notification: notification(request, 'human-answer', answerBody(request, result), result) }),
         ...(completionAction(request, result) === undefined ? {} : { completion_action: completionAction(request, result) }),
         ...(retire(request) === undefined ? {} : { post_commit: retire(request) }),
     });
@@ -84,7 +154,9 @@ export async function cancelHumanTicket(ticketId, opts) {
         request_id: request.request_id,
         outcome: opts.disposition === 'dismissed' ? 'dismissed' : 'canceled',
         result_json: JSON.stringify(result),
-        ...(notification(request, 'human-canceled', result) === undefined ? {} : { notification: notification(request, 'human-canceled', result) }),
+        ...(notification(request, 'human-canceled', cancelBody(request, opts.disposition === 'dismissed', opts.reason), result) === undefined
+            ? {}
+            : { notification: notification(request, 'human-canceled', cancelBody(request, opts.disposition === 'dismissed', opts.reason), result) }),
         ...(completionAction(request, result) === undefined ? {} : { completion_action: completionAction(request, result) }),
         ...(retire(request) === undefined ? {} : { post_commit: retire(request) }),
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.314",
+  "version": "0.3.315",
   "description": "crtr — agent runtime with memory, plugins, and marketplaces",
   "type": "module",
   "main": "dist/index.js",

package/runtime.lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.314",
+  "version": "0.3.315",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@north-light/crouter",
-      "version": "0.3.314",
+      "version": "0.3.315",
       "hasInstallScript": true,
       "license": "MIT",
       "workspaces": [
@@ -5206,17 +5206,17 @@
     },
     "packages/crouter-api": {
       "name": "@north-light/crouter-api",
-      "version": "0.3.314",
+      "version": "0.3.315",
       "license": "UNLICENSED"
     },
     "packages/crouter-env-docker": {
       "name": "@north-light/crouter-env-docker",
-      "version": "0.3.314",
+      "version": "0.3.315",
       "license": "UNLICENSED"
     },
     "packages/crouter-sdk": {
       "name": "@north-light/crouter-sdk",
-      "version": "0.3.314",
+      "version": "0.3.315",
       "license": "UNLICENSED",
       "dependencies": {
         "@north-light/crouter-api": "^0.3.295"