@withpica/mcp-sdk 3.7.1 → 3.9.0

package/dist/index.js

@@ -298,7 +298,15 @@ class BaseResource {
         // `count` / items.length so untouched callers keep their prior behaviour.
         const total = json.pagination?.total ?? json.total ?? json.count ?? items.length;
         const hasMore = json.pagination?.hasMore ?? json.hasMore ?? total > items.length;
-        return { data: items, total, hasMore };
+        // Carry the server's zero-result guidance through — silently dropping it
+        // here left agents staring at a bare empty list (ops_issue 689a8e9a).
+        const recovery_hint = typeof json.recovery_hint === "string" ? json.recovery_hint : undefined;
+        return {
+            data: items,
+            total,
+            hasMore,
+            ...(recovery_hint !== undefined && { recovery_hint }),
+        };
     }
 }
 class WorksResource extends BaseResource {
@@ -432,8 +440,12 @@ class WorksResource extends BaseResource {
      * release-context, not physical-asset provenance.
      */
     async listReleases(workId) {
+        // request() already unwraps the route's { success, data } envelope to the
+        // array, so reading `.data` again yielded undefined → `[]` on every call —
+        // the work→releases view looked empty even when the work was attached.
+        // Mirrors the recordings-side fix (cea22b5e); FIX_LOG 2026-07-16 sweep.
         const result = await this.request("GET", `/admin/works/${workId}/releases`);
-        return result?.data || [];
+        return Array.isArray(result) ? result : (result?.data ?? []);
     }
     /**
      * Assemble the full song dossier for one work — everything PICA holds about
@@ -811,6 +823,33 @@ class AudioFilesResource extends BaseResource {
     async get(id) {
         return this.request("GET", `/admin/audio-files/${id}`);
     }
+    /**
+     * Attach ALL orphan audio files whose suggested match is ISRC-high and
+     * non-ambiguous, in one call. Pairings are recomputed server-side at
+     * execution time — nothing is sent, so stale suggestions can't be applied
+     * and dismissed pairs stay excluded; `possible` (title-tier) matches are
+     * never bulk-attached. Wraps POST /admin/audio-files/bulk-assign-suggested
+     * (the route behind the /inspect "attach all high" strip; agent feedback
+     * 20f592ff asked for the same action tool-side).
+     *
+     * NB the route responds `{ success, result }` with no `data` key, so
+     * `request()`'s `.data || whole` unwrap yields the full envelope — read
+     * `.result` here rather than typing against a payload that isn't there
+     * (the ADR-265 double-unwrap lesson).
+     */
+    async bulkAssignSuggested(params) {
+        const res = await this.request("POST", "/admin/audio-files/bulk-assign-suggested", params?.dry_run ? { dry_run: true } : undefined);
+        return res.result;
+    }
+    /**
+     * Reject a suggested (audio_file → recording) match so the pair never
+     * resurfaces in suggestions or bulk attach. The file stays an orphan —
+     * only THIS pairing is suppressed. Repeat dismissals are no-op successes
+     * server-side. Wraps POST /admin/audio-files/{id}/dismiss-match.
+     */
+    async dismissMatch(id, recordingId) {
+        await this.request("POST", `/admin/audio-files/${id}/dismiss-match`, { recording_id: recordingId });
+    }
     async update(id, body) {
         return this.request("PATCH", `/admin/audio-files/${id}`, body);
     }
@@ -1235,8 +1274,20 @@ class RecordingsResource extends BaseResource {
         if (prefix.length < 5)
             return null;
         try {
-            const result = await this.request("GET", `/admin/isrc/prefix/${prefix}`);
-            return result.data ?? null;
+            // request() unwraps {success,data} to the registry ROW on a hit, but
+            // on a MISS the route returns data:null (falsy) and `data.data || data`
+            // falls back to the FULL envelope — so discriminate the shape instead
+            // of retyping (FIX_LOG 2026-07-16 double-unwrap sweep; same guard as
+            // AccessSimulateResource.simulate). Reading `.data` unconditionally
+            // made every hit return null, silently killing prefix_intel.
+            const result = (await this.request("GET", `/admin/isrc/prefix/${prefix}`));
+            if (result &&
+                typeof result === "object" &&
+                "success" in result &&
+                "data" in result) {
+                return (result.data ?? null);
+            }
+            return result ?? null;
         }
         catch {
             return null;
@@ -2713,6 +2764,12 @@ class CollaborationsResource extends BaseResource {
  * Cross-org discovery surface that lets the sender's agent invite a
  * collaborator by `@handle` without ever learning the recipient's
  * email (resolution happens server-side).
+ *
+ * Typed against the UNWRAPPED reality (ADR-265 lesson): BaseResource.request
+ * auto-unwraps the `{success,data}` envelope, so methods see `data` itself.
+ * Declaring `T = { data: X }` and returning `res.data` double-unwraps and
+ * resolves `undefined` on the live wire (FIX_LOG 2026-07-16 — the
+ * get_my_profile "reading 'public_handle'" crash).
  */
 class UsersResource extends BaseResource {
     /**
@@ -2723,27 +2780,26 @@ class UsersResource extends BaseResource {
      * proceed with an invite.
      */
     async findByHandle(handle) {
-        const res = await this.request("POST", "/admin/users/find-by-handle", { handle });
-        return res.data;
+        return this.request("POST", "/admin/users/find-by-handle", { handle });
     }
     async setHandle(handle) {
-        const res = await this.request("POST", "/admin/users/handle", { handle });
-        return res.data;
+        return this.request("POST", "/admin/users/handle", {
+            handle,
+        });
     }
     async clearHandle() {
         await this.request("DELETE", "/admin/users/handle");
     }
     async getMyProfile() {
-        const res = await this.request("GET", "/admin/users/profile");
-        return res.data;
+        return this.request("GET", "/admin/users/profile");
     }
     async setBio(bio) {
-        const res = await this.request("POST", "/admin/users/bio", { bio });
-        return res.data;
+        return this.request("POST", "/admin/users/bio", {
+            bio,
+        });
     }
     async setAvatarUrl(avatarUrl) {
-        const res = await this.request("POST", "/admin/users/avatar", { avatar_url: avatarUrl });
-        return res.data;
+        return this.request("POST", "/admin/users/avatar", { avatar_url: avatarUrl });
     }
     /**
      * Copy `people.biography` into `user_profiles.bio`. Returns the new
@@ -2753,8 +2809,7 @@ class UsersResource extends BaseResource {
      * `overwrite: true`.
      */
     async importBioFromPerson(params) {
-        const res = await this.request("POST", "/admin/users/import-bio-from-person", params ?? {});
-        return res.data;
+        return this.request("POST", "/admin/users/import-bio-from-person", params ?? {});
     }
 }
 class DirectoryResource extends BaseResource {