@poncho-ai/harness 0.59.0 → 0.59.2

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @poncho-ai/harness@0.59.0 build /home/runner/work/poncho-ai/poncho-ai/packages/harness
+> @poncho-ai/harness@0.59.2 build /home/runner/work/poncho-ai/poncho-ai/packages/harness
 > node scripts/embed-docs.js && tsup src/index.ts --format esm --dts
 
 [embed-docs] Generated poncho-docs.ts with 4 topics
@@ -8,9 +8,9 @@
 CLI tsup v8.5.1
 CLI Target: es2022
 ESM Build start
-ESM dist/index.js            567.16 KB
 ESM dist/isolate-F2PPSUL6.js 53.82 KB
-ESM ⚡️ Build success in 256ms
+ESM dist/index.js            567.57 KB
+ESM ⚡️ Build success in 251ms
 DTS Build start
-DTS ⚡️ Build success in 8437ms
+DTS ⚡️ Build success in 7447ms
 DTS dist/index.d.ts 104.68 KB

package/CHANGELOG.md

@@ -1,5 +1,34 @@
 # @poncho-ai/harness
 
+## 0.59.2
+
+### Patch Changes
+
+- [`ac0faae`](https://github.com/cesr/poncho-ai/commit/ac0faae54365afda5ef518b0a306a8cde5978ca8) Thanks [@cesr](https://github.com/cesr)! - conversations.rename now does a targeted title-column UPDATE instead of a
+  whole-row get→mutate→update. The read-modify-write raced a streaming turn's
+  per-step draft persist: a rename landing mid-run wrote the stale blob back
+  and silently reverted the turn's persisted progress.
+
+## 0.59.1
+
+### Patch Changes
+
+- [`299f574`](https://github.com/cesr/poncho-ai/commit/299f574a2f2f0d4873f42bbcffdf604e9cc4c29c) Thanks [@cesr](https://github.com/cesr)! - Mark in-flight assistant drafts with `metadata.incomplete = true`.
+
+  The orchestrator's per-step draft persist (`persistDraft`) and the
+  approval/device checkpoint and continuation writes now stamp the trailing
+  assistant message `metadata.incomplete = true`; the three terminal writes
+  (normal finalize, cancelled, errored) clear it. This lets a consumer that
+  reconciles a persisted snapshot against a live event stream (e.g. a
+  WebSocket layer) strip the in-flight draft from the authoritative snapshot
+  and rebuild that turn from the event log instead — so the snapshot and the
+  replayed events never both carry the in-flight turn, eliminating
+  reconnect-time duplication. Additive + backwards-compatible: consumers that
+  ignore the flag are unaffected.
+
+- Updated dependencies [[`299f574`](https://github.com/cesr/poncho-ai/commit/299f574a2f2f0d4873f42bbcffdf604e9cc4c29c)]:
+  - @poncho-ai/sdk@1.15.1
+
 ## 0.59.0
 
 ### Minor Changes

package/dist/index.js

@@ -3966,11 +3966,15 @@ var SqlStorageEngine = class {
       );
     },
     rename: async (conversationId, title) => {
-      const conv = await this.conversations.get(conversationId);
-      if (!conv) return void 0;
-      conv.title = normalizeTitle2(title);
-      await this.conversations.update(conv);
-      return conv;
+      const normalized = normalizeTitle2(title);
+      await this.executor.run(
+        rewrite(
+          `UPDATE conversations SET title = $1, updated_at = $2 WHERE id = $3`,
+          this.dialect
+        ),
+        [normalized, (/* @__PURE__ */ new Date()).toISOString(), conversationId]
+      );
+      return this.conversations.get(conversationId);
     },
     delete: async (conversationId) => {
       const row = await this.executor.get(
@@ -14477,7 +14481,7 @@ var runConversationTurn = async (opts) => {
   let runContinuationMessages;
   let cancelHarnessMessages;
   let checkpointedRun = false;
-  const buildMessages = () => {
+  const buildMessages = (incomplete = true) => {
     const draftSections = cloneSections(draft.sections);
     if (draft.currentTools.length > 0) {
       draftSections.push({ type: "tools", content: [...draft.currentTools] });
@@ -14496,10 +14500,15 @@ var runConversationTurn = async (opts) => {
       {
         role: "assistant",
         content: draft.assistantResponse,
-        metadata: buildAssistantMetadata(draft, draftSections, {
-          id: assistantId,
-          timestamp: turnTimestamp
-        })
+        metadata: {
+          ...buildAssistantMetadata(draft, draftSections, {
+            id: assistantId,
+            timestamp: turnTimestamp
+          }),
+          // Only stamp the flag when true; finalize omits it so completed
+          // assistants stay clean (no `incomplete: false` noise on the row).
+          ...incomplete ? { incomplete: true } : {}
+        }
       }
     ];
   };
@@ -14721,7 +14730,7 @@ var runConversationTurn = async (opts) => {
     flushTurnDraft(draft);
     latestRunId = execution.latestRunId || latestRunId;
     if (!checkpointedRun && !runContinuationMessages) {
-      conversation.messages = buildMessages();
+      conversation.messages = buildMessages(false);
       applyTurnMetadata(
         conversation,
         {
@@ -14779,7 +14788,7 @@ var runConversationTurn = async (opts) => {
     const aborted = opts.abortSignal?.aborted === true;
     if (aborted || runCancelled) {
       if (draft.assistantResponse.length > 0 || draft.toolTimeline.length > 0 || draft.sections.length > 0) {
-        conversation.messages = buildMessages();
+        conversation.messages = buildMessages(false);
         applyTurnMetadata(
           conversation,
           {
@@ -14828,7 +14837,7 @@ var runConversationTurn = async (opts) => {
       }
     }
     if (draft.assistantResponse.length > 0 || draft.toolTimeline.length > 0 || draft.sections.length > 0) {
-      conversation.messages = buildMessages();
+      conversation.messages = buildMessages(false);
       conversation.updatedAt = Date.now();
       await opts.conversationStore.update(conversation);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@poncho-ai/harness",
-  "version": "0.59.0",
+  "version": "0.59.2",
   "description": "Agent execution runtime - conversation loop, tool dispatch, streaming",
   "repository": {
     "type": "git",
@@ -34,7 +34,7 @@
     "mustache": "^4.2.0",
     "yaml": "^2.4.0",
     "zod": "^3.22.0",
-    "@poncho-ai/sdk": "1.15.0"
+    "@poncho-ai/sdk": "1.15.1"
   },
   "peerDependencies": {
     "esbuild": ">=0.17.0",

package/src/orchestrator/run-conversation-turn.ts

@@ -157,7 +157,15 @@ export const runConversationTurn = async (
   let cancelHarnessMessages: Message[] | undefined;
   let checkpointedRun = false;
 
-  const buildMessages = (): Message[] => {
+  // `incomplete: true` (the default) marks the trailing assistant message as
+  // an in-flight DRAFT — content for a turn that hasn't finished. A consumer
+  // (e.g. PonchOS's WS snapshot) uses this to strip the draft from the
+  // authoritative snapshot: the in-flight turn is delivered by the event
+  // stream instead, so the snapshot and the event log never both carry it
+  // (no reconnect duplication). The three TERMINAL writes (normal finalize,
+  // cancelled, errored) pass `incomplete: false` — at that point the turn is
+  // done and the assistant message is authoritative.
+  const buildMessages = (incomplete = true): Message[] => {
     const draftSections = cloneSections(draft.sections);
     if (draft.currentTools.length > 0) {
       draftSections.push({ type: "tools", content: [...draft.currentTools] });
@@ -179,10 +187,15 @@ export const runConversationTurn = async (
       {
         role: "assistant" as const,
         content: draft.assistantResponse,
-        metadata: buildAssistantMetadata(draft, draftSections, {
-          id: assistantId,
-          timestamp: turnTimestamp,
-        }),
+        metadata: {
+          ...buildAssistantMetadata(draft, draftSections, {
+            id: assistantId,
+            timestamp: turnTimestamp,
+          }),
+          // Only stamp the flag when true; finalize omits it so completed
+          // assistants stay clean (no `incomplete: false` noise on the row).
+          ...(incomplete ? { incomplete: true } : {}),
+        },
       },
     ];
   };
@@ -442,7 +455,7 @@ export const runConversationTurn = async (
     latestRunId = execution.latestRunId || latestRunId;
 
     if (!checkpointedRun && !runContinuationMessages) {
-      conversation.messages = buildMessages();
+      conversation.messages = buildMessages(false); // terminal: turn complete
       applyTurnMetadata(
         conversation,
         {
@@ -515,7 +528,7 @@ export const runConversationTurn = async (
         draft.toolTimeline.length > 0 ||
         draft.sections.length > 0
       ) {
-        conversation.messages = buildMessages();
+        conversation.messages = buildMessages(false); // terminal: cancelled
         applyTurnMetadata(
           conversation,
           {
@@ -571,7 +584,7 @@ export const runConversationTurn = async (
       draft.toolTimeline.length > 0 ||
       draft.sections.length > 0
     ) {
-      conversation.messages = buildMessages();
+      conversation.messages = buildMessages(false); // terminal: errored
       conversation.updatedAt = Date.now();
       await opts.conversationStore.update(conversation);
     }

package/src/storage/sql-dialect.ts

@@ -557,11 +557,21 @@ export abstract class SqlStorageEngine implements StorageEngine {
       conversationId: string,
       title: string,
     ): Promise<Conversation | undefined> => {
-      const conv = await this.conversations.get(conversationId);
-      if (!conv) return undefined;
-      conv.title = normalizeTitle(title);
-      await this.conversations.update(conv);
-      return conv;
+      // Targeted column update — deliberately NOT get→mutate→update().
+      // The whole-row read-modify-write races a streaming turn's per-step
+      // draft persist: rename reads the row at T0, the turn persists step
+      // N's draft at T1, rename writes T0's stale blob back at T2 and
+      // silently reverts the turn's progress. Title lives in its own
+      // column, so touch only that (+ updated_at for sidebar ordering).
+      const normalized = normalizeTitle(title);
+      await this.executor.run(
+        rewrite(
+          `UPDATE conversations SET title = $1, updated_at = $2 WHERE id = $3`,
+          this.dialect,
+        ),
+        [normalized, new Date().toISOString(), conversationId],
+      );
+      return this.conversations.get(conversationId);
     },
 
     delete: async (conversationId: string): Promise<boolean> => {