@poncho-ai/harness 0.50.0 → 0.50.1

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @poncho-ai/harness@0.50.0 build /home/runner/work/poncho-ai/poncho-ai/packages/harness
+> @poncho-ai/harness@0.50.1 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            530.75 KB
 ESM dist/isolate-BNQ6P3HI.js 51.41 KB
-ESM dist/index.js            530.56 KB
-ESM ⚡️ Build success in 259ms
+ESM ⚡️ Build success in 224ms
 DTS Build start
-DTS ⚡️ Build success in 6896ms
+DTS ⚡️ Build success in 6878ms
 DTS dist/index.d.ts 89.28 KB

package/CHANGELOG.md

@@ -1,5 +1,30 @@
 # @poncho-ai/harness
 
+## 0.50.1
+
+### Patch Changes
+
+- [#131](https://github.com/cesr/poncho-ai/pull/131) [`9e0ccd8`](https://github.com/cesr/poncho-ai/commit/9e0ccd8307ab056f94b7efc3e25a0cb71ee75625) Thanks [@cesr](https://github.com/cesr)! - harness: strip `poncho-upload://` scheme in `S3UploadStore.get` / `.delete`
+
+  `createUploadStore({ provider: "s3" })` wraps `S3UploadStore` in
+  `CachedUploadStore`, whose `put` returns a `poncho-upload://<key>` ref
+  and stores the underlying S3 object at the bare `<key>`. On read,
+  `CachedUploadStore.get` checks an in-memory cache (10-minute TTL); on
+  miss it falls through to `S3UploadStore.get(<ref>)`. Pre-fix, the S3
+  store treated the scheme-prefixed ref as a literal S3 key and hit the
+  backend with `poncho-upload://<key>` — guaranteed `NoSuchKey`.
+
+  In practice this meant a chat message with an attached image worked on
+  the turn it was uploaded (cache hit) and then started showing as
+  "[Attached file: … — file is no longer available]" on every follow-up
+  turn ~10 minutes later (cache miss → S3 NoSuchKey → outer catch in the
+  harness resolver). The same path worked for the local-fs store, which
+  strips the scheme in both `get` and `delete`.
+
+  `S3UploadStore.get` now strips the scheme before issuing
+  `GetObjectCommand`. `S3UploadStore.delete` already stripped `https://`
+  and now strips `poncho-upload://` too.
+
 ## 0.50.0
 
 ### Minor Changes

package/dist/index.js

@@ -2612,16 +2612,17 @@ var S3UploadStore = class {
       }
       return Buffer.from(await response.arrayBuffer());
     }
+    const key = urlOrKey.startsWith(PONCHO_UPLOAD_SCHEME) ? urlOrKey.slice(PONCHO_UPLOAD_SCHEME.length) : urlOrKey;
     await this.ensureClient();
     const result = await this.client.send(
-      new this.s3Sdk.GetObjectCommand({ Bucket: this.bucket, Key: urlOrKey })
+      new this.s3Sdk.GetObjectCommand({ Bucket: this.bucket, Key: key })
     );
-    if (!result.Body) throw new Error(`uploads: empty body for S3 key ${urlOrKey}`);
+    if (!result.Body) throw new Error(`uploads: empty body for S3 key ${key}`);
     return Buffer.from(await result.Body.transformToByteArray());
   }
   async delete(urlOrKey) {
     await this.ensureClient();
-    const key = urlOrKey.startsWith("https://") ? new URL(urlOrKey).pathname.slice(1) : urlOrKey;
+    const key = urlOrKey.startsWith("https://") ? new URL(urlOrKey).pathname.slice(1) : urlOrKey.startsWith(PONCHO_UPLOAD_SCHEME) ? urlOrKey.slice(PONCHO_UPLOAD_SCHEME.length) : urlOrKey;
     await this.client.send(
       new this.s3Sdk.DeleteObjectCommand({ Bucket: this.bucket, Key: key })
     );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@poncho-ai/harness",
-  "version": "0.50.0",
+  "version": "0.50.1",
   "description": "Agent execution runtime - conversation loop, tool dispatch, streaming",
   "repository": {
     "type": "git",

package/src/upload-store.ts

@@ -349,11 +349,19 @@ export class S3UploadStore implements UploadStore {
       }
       return Buffer.from(await response.arrayBuffer());
     }
+    // Strip the `poncho-upload://` scheme — LocalUploadStore does this for
+    // both get/delete but S3 used to treat the full URI as a literal S3
+    // key, so resolving a persisted file part (data: "poncho-upload://…")
+    // on a follow-up turn 404'd against R2 and the harness emitted a
+    // "file is no longer available" placeholder.
+    const key = urlOrKey.startsWith(PONCHO_UPLOAD_SCHEME)
+      ? urlOrKey.slice(PONCHO_UPLOAD_SCHEME.length)
+      : urlOrKey;
     await this.ensureClient();
     const result = await this.client.send(
-      new this.s3Sdk.GetObjectCommand({ Bucket: this.bucket, Key: urlOrKey }),
+      new this.s3Sdk.GetObjectCommand({ Bucket: this.bucket, Key: key }),
     );
-    if (!result.Body) throw new Error(`uploads: empty body for S3 key ${urlOrKey}`);
+    if (!result.Body) throw new Error(`uploads: empty body for S3 key ${key}`);
     return Buffer.from(await result.Body.transformToByteArray());
   }
 
@@ -361,7 +369,9 @@ export class S3UploadStore implements UploadStore {
     await this.ensureClient();
     const key = urlOrKey.startsWith("https://")
       ? new URL(urlOrKey).pathname.slice(1)
-      : urlOrKey;
+      : urlOrKey.startsWith(PONCHO_UPLOAD_SCHEME)
+        ? urlOrKey.slice(PONCHO_UPLOAD_SCHEME.length)
+        : urlOrKey;
     await this.client.send(
       new this.s3Sdk.DeleteObjectCommand({ Bucket: this.bucket, Key: key }),
     );