@parall/parel-channel 1.57.1 → 1.58.0

package/dist/channel-prompt.d.ts

@@ -36,8 +36,12 @@ export interface ChatPromptArgs {
     chatId: string;
     /** Display name of the chat; undefined when unresolved (best-effort). */
     chatName?: string;
-    /** Sender user id (usr_); display-name enrichment is a follow-up. */
+    /** Sender user id (usr_). */
     senderId?: string;
+    /** Sender display name; rendered before the id when resolved (best-effort). */
+    senderName?: string;
+    /** Chat shape (direct/group); the type segment is omitted when unresolved. */
+    chatType?: string;
     /**
      * The inbound message id (msg_); anchors the forwarded-message hint's
      * explicit --from so coalesced turns don't lean on the single trigger id.

package/dist/channel-prompt.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"channel-prompt.d.ts","sourceRoot":"","sources":["../src/channel-prompt.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AA+BH,MAAM,WAAW,gBAAgB;IAC/B,EAAE,EAAE,MAAM,CAAC;IACX,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,WAAW,cAAc;IAC7B,MAAM,EAAE,MAAM,CAAC;IACf,yEAAyE;IACzE,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,qEAAqE;IACrE,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB;;;OAGG;IACH,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB;;;;OAIG;IACH,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,iFAAiF;IACjF,IAAI,EAAE,MAAM,CAAC;IACb,iFAAiF;IACjF,WAAW,CAAC,EAAE,gBAAgB,EAAE,CAAC;IACjC,8FAA8F;IAC9F,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB;AAmBD;;;;;GAKG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,cAAc,GAAG,MAAM,CAqB5D;AAED,MAAM,WAAW,iBAAiB;IAChC,gEAAgE;IAChE,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,4EAA4E;IAC5E,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,sBAAsB,CAAC,EAAE,MAAM,CAAC;IAChC,wBAAwB,CAAC,EAAE,MAAM,CAAC;IAClC,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,UAAU,EAAE,MAAM,CAAC;IACnB,iDAAiD;IACjD,IAAI,EAAE,MAAM,CAAC;IACb;;;;;OAKG;IACH,UAAU,CAAC,EAAE,OAAO,CAAC;CACtB;AAED,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,iBAAiB,GAAG,MAAM,CA2BlE"}
+{"version":3,"file":"channel-prompt.d.ts","sourceRoot":"","sources":["../src/channel-prompt.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AA+BH,MAAM,WAAW,gBAAgB;IAC/B,EAAE,EAAE,MAAM,CAAC;IACX,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,WAAW,cAAc;IAC7B,MAAM,EAAE,MAAM,CAAC;IACf,yEAAyE;IACzE,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,6BAA6B;IAC7B,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,+EAA+E;IAC/E,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,8EAA8E;IAC9E,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB;;;OAGG;IACH,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB;;;;OAIG;IACH,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,iFAAiF;IACjF,IAAI,EAAE,MAAM,CAAC;IACb,iFAAiF;IACjF,WAAW,CAAC,EAAE,gBAAgB,EAAE,CAAC;IACjC,8FAA8F;IAC9F,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB;AAmBD;;;;;GAKG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,cAAc,GAAG,MAAM,CAwC5D;AAgBD,MAAM,WAAW,iBAAiB;IAChC,gEAAgE;IAChE,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,4EAA4E;IAC5E,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,sBAAsB,CAAC,EAAE,MAAM,CAAC;IAChC,wBAAwB,CAAC,EAAE,MAAM,CAAC;IAClC,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,UAAU,EAAE,MAAM,CAAC;IACnB,iDAAiD;IACjD,IAAI,EAAE,MAAM,CAAC;IACb;;;;;OAKG;IACH,UAAU,CAAC,EAAE,OAAO,CAAC;CACtB;AAED,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,iBAAiB,GAAG,MAAM,CA2BlE"}

package/dist/channel-prompt.js

@@ -73,16 +73,31 @@ function attachmentLines(attachments) {
  * not re-created here.
  */
 export function buildChatPrompt(args) {
-    const chatLabel = args.chatName
-        ? `${sanitizeMeta(args.chatId)} — ${sanitizeMeta(args.chatName)}`
-        : sanitizeMeta(args.chatId);
-    const lines = [`[Chat: ${chatLabel}]`];
-    if (args.senderId)
-        lines.push(`[From: ${sanitizeMeta(args.senderId)}]`);
+    // agent-core's event-format.ts message branch is the SSOT for this frame:
+    // same lines, same order, same send-hint. The only sanctioned deltas are
+    // absent-field fallbacks (no chat name → bare URI, no sender name → bare
+    // id, no chat type → segment omitted) — agent-core's host always has them.
+    const lines = [`[Event: message.new]`];
+    const chatUri = `prll://${args.chatId}`;
+    const chatLabel = args.chatName ? `"${sanitizeMeta(args.chatName)}" (${chatUri})` : chatUri;
+    const typeSeg = args.chatType ? ` | type: ${sanitizeMeta(args.chatType)}` : '';
+    lines.push(`[Chat: ${chatLabel}${typeSeg}]`);
+    if (args.senderId) {
+        lines.push(args.senderName
+            ? `[From: ${sanitizeMeta(args.senderName)} (prll://${args.senderId})]`
+            : `[From: prll://${args.senderId}]`);
+    }
+    if (args.messageId)
+        lines.push(`[Message ID: prll://${args.messageId}]`);
     if (args.threadRootId)
-        lines.push(`[Thread: ${sanitizeMeta(args.threadRootId)}]`);
+        lines.push(`[Thread: prll://${sanitizeMeta(args.threadRootId)}]`);
     if (args.deliveryReason)
         lines.push(`[Delivery: ${sanitizeMeta(args.deliveryReason)}]`);
+    if (args.deliveryReason === 'watcher' && args.threadRootId) {
+        // agent-core's threadWatcherHint, verbatim — the cooperative nudge under
+        // the server's agent-loop ceiling (agent-dm-loop-prevention.md Layer 4).
+        lines.push(`[Hint: watcher — you were not addressed in this reply. Reply only if you add something new; otherwise run \`parall no-reply\`. If this thread is not relevant to you, or you have nothing more to contribute, run \`parall messages unwatch prll://${sanitizeMeta(args.threadRootId)}\` so later replies stop waking you.]`);
+    }
     if (args.noReply)
         lines.push(`[Hint: no_reply]`);
     if (isForwardedMessageBody(args.text)) {
@@ -94,7 +109,21 @@ export function buildChatPrompt(args) {
     }
     lines.push(...attachmentLines(args.attachments));
     lines.push('', args.text);
-    return lines.join('\n');
+    return lines.join('\n') + chatSendHint(args);
+}
+/**
+ * agent-core's buildSendMessageHint chat branch, verbatim: every chat event
+ * ends with the concrete send command. The standing system prompt teaches the
+ * CLI-only reply contract too, but a per-event reminder is what the standard
+ * runtimes ship — and weak models demonstrably drop the send step without it.
+ */
+function chatSendHint(args) {
+    if (args.noReply)
+        return '';
+    if (args.threadRootId) {
+        return `\n<system-reminder>To reply in this thread, run \`parall messages send prll://${args.chatId} --thread-root-id ${sanitizeMeta(args.threadRootId)} --text-file - <<'EOF'\` … \`EOF\` — the quoted heredoc keeps \`$\`, backticks and apostrophes literal (plain \`--text "$1,000"\` sends \`,000\`). Your plain text output is not delivered to the chat.</system-reminder>`;
+    }
+    return `\n<system-reminder>To reply, run \`parall messages send prll://${args.chatId} --text-file - <<'EOF'\` … \`EOF\` — the quoted heredoc keeps \`$\`, backticks and apostrophes literal (plain \`--text "$1,000"\` sends \`,000\`). Your plain text output is not delivered to the chat.</system-reminder>`;
 }
 export function buildChannelPrompt(args) {
     const providerLabel = sanitizeMeta(args.provider ?? 'external IM');

package/dist/typed-dispatch.js

@@ -342,7 +342,16 @@ async function planApprovalDecided(ctx, approvalId, chatId, actor) {
     if (!approval?.id)
         return STALE;
     const statusLabel = approval.status === 'approved' ? 'Approved' : 'Rejected';
-    const execInfo = approval.execution_status ? ` | execution: ${approval.execution_status}` : '';
+    // A failed execution names its cause on the same line — the agent must not
+    // need a second read to tell a version conflict from anything else.
+    // agent-core's formatApprovalExecution and the server's Go materializer
+    // render the identical suffix; keep the three in sync.
+    const execError = approval.execution_error
+        ? ` | error: ${sanitizeMeta(approval.execution_error)}`
+        : '';
+    const execInfo = approval.execution_status
+        ? ` | execution: ${approval.execution_status}${execError}`
+        : '';
     const resolvedChat = chatId ?? approval.chat_id;
     const lines = [`[Event: approval.decided]`];
     lines.push(`[Approval: prll://${approval.id}]`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@parall/parel-channel",
-  "version": "1.57.1",
+  "version": "1.58.0",
   "description": "Parall channel plugin for the parel runtime — lets a parel agent treat Parall as a managed_ws channel (receive dispatches, reply via the Parall API)",
   "license": "MIT",
   "repository": {

package/src/channel-prompt.ts

@@ -67,8 +67,12 @@ export interface ChatPromptArgs {
   chatId: string;
   /** Display name of the chat; undefined when unresolved (best-effort). */
   chatName?: string;
-  /** Sender user id (usr_); display-name enrichment is a follow-up. */
+  /** Sender user id (usr_). */
   senderId?: string;
+  /** Sender display name; rendered before the id when resolved (best-effort). */
+  senderName?: string;
+  /** Chat shape (direct/group); the type segment is omitted when unresolved. */
+  chatType?: string;
   /**
    * The inbound message id (msg_); anchors the forwarded-message hint's
    * explicit --from so coalesced turns don't lean on the single trigger id.
@@ -113,13 +117,32 @@ function attachmentLines(attachments: PromptAttachment[] | undefined): string[]
  * not re-created here.
  */
 export function buildChatPrompt(args: ChatPromptArgs): string {
-  const chatLabel = args.chatName
-    ? `${sanitizeMeta(args.chatId)} — ${sanitizeMeta(args.chatName)}`
-    : sanitizeMeta(args.chatId);
-  const lines = [`[Chat: ${chatLabel}]`];
-  if (args.senderId) lines.push(`[From: ${sanitizeMeta(args.senderId)}]`);
-  if (args.threadRootId) lines.push(`[Thread: ${sanitizeMeta(args.threadRootId)}]`);
+  // agent-core's event-format.ts message branch is the SSOT for this frame:
+  // same lines, same order, same send-hint. The only sanctioned deltas are
+  // absent-field fallbacks (no chat name → bare URI, no sender name → bare
+  // id, no chat type → segment omitted) — agent-core's host always has them.
+  const lines = [`[Event: message.new]`];
+  const chatUri = `prll://${args.chatId}`;
+  const chatLabel = args.chatName ? `"${sanitizeMeta(args.chatName)}" (${chatUri})` : chatUri;
+  const typeSeg = args.chatType ? ` | type: ${sanitizeMeta(args.chatType)}` : '';
+  lines.push(`[Chat: ${chatLabel}${typeSeg}]`);
+  if (args.senderId) {
+    lines.push(
+      args.senderName
+        ? `[From: ${sanitizeMeta(args.senderName)} (prll://${args.senderId})]`
+        : `[From: prll://${args.senderId}]`,
+    );
+  }
+  if (args.messageId) lines.push(`[Message ID: prll://${args.messageId}]`);
+  if (args.threadRootId) lines.push(`[Thread: prll://${sanitizeMeta(args.threadRootId)}]`);
   if (args.deliveryReason) lines.push(`[Delivery: ${sanitizeMeta(args.deliveryReason)}]`);
+  if (args.deliveryReason === 'watcher' && args.threadRootId) {
+    // agent-core's threadWatcherHint, verbatim — the cooperative nudge under
+    // the server's agent-loop ceiling (agent-dm-loop-prevention.md Layer 4).
+    lines.push(
+      `[Hint: watcher — you were not addressed in this reply. Reply only if you add something new; otherwise run \`parall no-reply\`. If this thread is not relevant to you, or you have nothing more to contribute, run \`parall messages unwatch prll://${sanitizeMeta(args.threadRootId)}\` so later replies stop waking you.]`,
+    );
+  }
   if (args.noReply) lines.push(`[Hint: no_reply]`);
   if (isForwardedMessageBody(args.text)) {
     // Embed the concrete --from anchor: parel snapshots one invocation
@@ -132,7 +155,21 @@ export function buildChatPrompt(args: ChatPromptArgs): string {
   }
   lines.push(...attachmentLines(args.attachments));
   lines.push('', args.text);
-  return lines.join('\n');
+  return lines.join('\n') + chatSendHint(args);
+}
+
+/**
+ * agent-core's buildSendMessageHint chat branch, verbatim: every chat event
+ * ends with the concrete send command. The standing system prompt teaches the
+ * CLI-only reply contract too, but a per-event reminder is what the standard
+ * runtimes ship — and weak models demonstrably drop the send step without it.
+ */
+function chatSendHint(args: ChatPromptArgs): string {
+  if (args.noReply) return '';
+  if (args.threadRootId) {
+    return `\n<system-reminder>To reply in this thread, run \`parall messages send prll://${args.chatId} --thread-root-id ${sanitizeMeta(args.threadRootId)} --text-file - <<'EOF'\` … \`EOF\` — the quoted heredoc keeps \`$\`, backticks and apostrophes literal (plain \`--text "$1,000"\` sends \`,000\`). Your plain text output is not delivered to the chat.</system-reminder>`;
+  }
+  return `\n<system-reminder>To reply, run \`parall messages send prll://${args.chatId} --text-file - <<'EOF'\` … \`EOF\` — the quoted heredoc keeps \`$\`, backticks and apostrophes literal (plain \`--text "$1,000"\` sends \`,000\`). Your plain text output is not delivered to the chat.</system-reminder>`;
 }
 
 export interface ChannelPromptArgs {

package/src/typed-dispatch.ts

@@ -508,6 +508,7 @@ interface ApprovalRow {
   title?: string;
   status?: string;
   execution_status?: string;
+  execution_error?: string | null;
   chat_id?: string;
   decided_by?: string;
   decision_reason?: string | null;
@@ -532,7 +533,16 @@ async function planApprovalDecided(
   if (!approval?.id) return STALE;
 
   const statusLabel = approval.status === 'approved' ? 'Approved' : 'Rejected';
-  const execInfo = approval.execution_status ? ` | execution: ${approval.execution_status}` : '';
+  // A failed execution names its cause on the same line — the agent must not
+  // need a second read to tell a version conflict from anything else.
+  // agent-core's formatApprovalExecution and the server's Go materializer
+  // render the identical suffix; keep the three in sync.
+  const execError = approval.execution_error
+    ? ` | error: ${sanitizeMeta(approval.execution_error)}`
+    : '';
+  const execInfo = approval.execution_status
+    ? ` | execution: ${approval.execution_status}${execError}`
+    : '';
   const resolvedChat = chatId ?? approval.chat_id;
 
   const lines = [`[Event: approval.decided]`];