@withpica/mcp-sdk 3.7.0 → 3.7.2

package/dist/index.js

@@ -432,8 +432,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
@@ -1235,8 +1239,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;
@@ -2174,11 +2190,12 @@ class BulkResource extends BaseResource {
             tags,
         });
     }
-    async linkArtist(workIds, personId, artistName) {
+    async linkArtist(workIds, personId, artistName, dryRun) {
         return this.request("POST", "/admin/works/bulk-link-artist", {
             workIds,
             personId,
             artistName,
+            dryRun: dryRun === true,
         });
     }
     /**
@@ -2712,6 +2729,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 {
     /**
@@ -2722,27 +2745,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
@@ -2752,8 +2774,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 {