@withpica/mcp-sdk 3.3.1 → 3.5.0

package/dist/index.js

@@ -260,8 +260,13 @@ class BaseResource {
         }, timeoutMs);
         const json = await response.json();
         const items = (json.data || []);
-        const total = json.pagination?.total ?? json.count ?? items.length;
-        const hasMore = json.pagination?.hasMore ?? total > items.length;
+        // Two envelope shapes exist across admin routes: nested `pagination: {
+        // total, hasMore }` (works/people/multimedia) and flat sibling fields
+        // `total`/`hasMore` (e.g. GET /admin/recordings/search, ADR unified
+        // catalog search). Check nested first, then flat, then fall back to
+        // `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 };
     }
 }
@@ -307,6 +312,40 @@ class WorksResource extends BaseResource {
             qp.set("sort", params.sort);
         if (params.society_code)
             qp.set("society_code", params.society_code);
+        if (params.work_type)
+            qp.set("work_type", params.work_type);
+        if (params.work_status)
+            qp.set("work_status", params.work_status);
+        if (params.genre)
+            qp.set("genre", params.genre);
+        if (params.mood)
+            qp.set("mood", params.mood);
+        if (params.tempo_min !== undefined)
+            qp.set("tempo_min", String(params.tempo_min));
+        if (params.tempo_max !== undefined)
+            qp.set("tempo_max", String(params.tempo_max));
+        if (params.primary_artist)
+            qp.set("primary_artist", params.primary_artist);
+        if (params.artist)
+            qp.set("artist", params.artist);
+        if (params.label)
+            qp.set("label", params.label);
+        if (params.available_for_licensing !== undefined)
+            qp.set("available_for_licensing", String(params.available_for_licensing));
+        if (params.release_format)
+            qp.set("release_format", params.release_format);
+        if (params.is_featured !== undefined)
+            qp.set("is_featured", String(params.is_featured));
+        if (params.published !== undefined)
+            qp.set("published", String(params.published));
+        if (params.created_from)
+            qp.set("created_from", params.created_from);
+        if (params.created_to)
+            qp.set("created_to", params.created_to);
+        if (params.has_iswc !== undefined)
+            qp.set("has_iswc", String(params.has_iswc));
+        if (params.has_publisher !== undefined)
+            qp.set("has_publisher", String(params.has_publisher));
         const qs = qp.toString();
         return this.requestPaginated("GET", `/admin/works${qs ? `?${qs}` : ""}`);
     }
@@ -404,6 +443,14 @@ class PeopleResource extends BaseResource {
             qp.set("record_kind", params.record_kind);
         if (params.outreach_state)
             qp.set("outreach_state", params.outreach_state);
+        if (params.has_email !== undefined)
+            qp.set("has_email", String(params.has_email));
+        if (params.has_phone !== undefined)
+            qp.set("has_phone", String(params.has_phone));
+        if (params.roles && params.roles.length > 0)
+            qp.set("roles", params.roles.join(","));
+        if (params.tags && params.tags.length > 0)
+            qp.set("tags", params.tags.join(","));
         const qs = qp.toString();
         return this.requestPaginated("GET", `/admin/people${qs ? `?${qs}` : ""}`);
     }
@@ -766,6 +813,18 @@ class AgreementsResource extends BaseResource {
             queryParams.set("includeWorkCounts", "true");
         if (params?.awaiting_signature)
             queryParams.set("awaiting_signature", "true");
+        if (params?.start_date_from)
+            queryParams.set("start_date_from", params.start_date_from);
+        if (params?.start_date_to)
+            queryParams.set("start_date_to", params.start_date_to);
+        if (params?.end_date_from)
+            queryParams.set("end_date_from", params.end_date_from);
+        if (params?.end_date_to)
+            queryParams.set("end_date_to", params.end_date_to);
+        if (params?.expiring_within_days !== undefined)
+            queryParams.set("expiring_within_days", String(params.expiring_within_days));
+        if (params?.has_end_date !== undefined)
+            queryParams.set("has_end_date", String(params.has_end_date));
         if (params?.limit !== undefined)
             queryParams.set("limit", String(params.limit));
         if (params?.offset !== undefined)
@@ -1001,19 +1060,35 @@ class RecordingsResource extends BaseResource {
         return this.request("GET", `/admin/recordings?limit=${limit}`);
     }
     /**
-     * Search recordings. The recordings API doesn't support text search,
-     * so we fetch all and filter client-side for text queries.
+     * Search recordings. A text `query` delegates to the server-side unified
+     * catalog matcher (`GET /admin/recordings/search`, backed by
+     * `chainService.searchFlat`) which matches title + artist_name + isrc —
+     * replacing the old fetch-200-and-filter-client-side-on-title/isrc
+     * approach, which both capped results at 200 rows and never matched on
+     * artist_name (e.g. a query for an artist whose name doesn't appear in
+     * the recording title, like "gwazai" against `artist_name: 'Gwazai'`,
+     * returned zero results).
+     *
+     * Without a `query`, behaviour is unchanged: forward filter/sort params
+     * to the plain list route.
      *
      * ADR-270 Phase (b): sort, streaming_activity_gt, streaming_activity_lt
      * are forwarded as query params to the server-side filter/sort path.
      */
     async search(params) {
-        // Best-effort ceiling: recordings API has no text search, so we fetch up to 200
-        // and filter client-side. Will need server-side search when recordings exceed this.
-        const fetchLimit = params.query ? 200 : params.limit || 50;
+        if (params.query) {
+            // GET /admin/recordings/search — server-side text match, no 200-row
+            // cap. Route default limit is 25 when omitted; set explicitly so SDK
+            // behaviour doesn't depend on the route's default changing silently.
+            const qp = new URLSearchParams();
+            qp.set("query", params.query);
+            qp.set("limit", String(params.limit || 25));
+            qp.set("offset", String(params.offset || 0));
+            return this.requestPaginated("GET", `/admin/recordings/search?${qp.toString()}`);
+        }
         const qp = new URLSearchParams();
-        qp.set("limit", String(fetchLimit));
-        if (params.offset !== undefined && !params.query)
+        qp.set("limit", String(params.limit || 50));
+        if (params.offset !== undefined)
             qp.set("offset", String(params.offset));
         if (params.sort !== undefined)
             qp.set("sort", params.sort);
@@ -1025,21 +1100,20 @@ class RecordingsResource extends BaseResource {
             qp.set("missing_field_count_gt", String(params.missing_field_count_gt));
         if (params.missing_field_count_lt !== undefined)
             qp.set("missing_field_count_lt", String(params.missing_field_count_lt));
+        if (params.has_isrc !== undefined)
+            qp.set("has_isrc", String(params.has_isrc));
+        if (params.version_type)
+            qp.set("version_type", params.version_type);
+        if (params.recorded_from)
+            qp.set("recorded_from", params.recorded_from);
+        if (params.recorded_to)
+            qp.set("recorded_to", params.recorded_to);
+        if (params.created_from)
+            qp.set("created_from", params.created_from);
+        if (params.created_to)
+            qp.set("created_to", params.created_to);
         const qs = qp.toString();
-        const result = await this.requestPaginated("GET", `/admin/recordings?${qs}`);
-        if (params.query) {
-            const q = params.query.toLowerCase();
-            const filtered = result.data.filter((r) => (r.title && r.title.toLowerCase().includes(q)) ||
-                (r.isrc && r.isrc.toLowerCase().includes(q)));
-            const limit = params.limit || 20;
-            const start = params.offset || 0;
-            return {
-                data: filtered.slice(start, start + limit),
-                total: filtered.length,
-                hasMore: start + limit < filtered.length,
-            };
-        }
-        return result;
+        return this.requestPaginated("GET", `/admin/recordings?${qs}`);
     }
     async get(id) {
         return this.request("GET", `/admin/recordings/${id}`);
@@ -2906,6 +2980,10 @@ class ReleasesResource extends BaseResource {
             parts.push(`upc=${encodeURIComponent(params.upc)}`);
         if (params?.spotify_album_id)
             parts.push(`spotify_album_id=${encodeURIComponent(params.spotify_album_id)}`);
+        if (params?.release_date_from)
+            parts.push(`release_date_from=${encodeURIComponent(params.release_date_from)}`);
+        if (params?.release_date_to)
+            parts.push(`release_date_to=${encodeURIComponent(params.release_date_to)}`);
         const qs = parts.length > 0 ? `?${parts.join("&")}` : "";
         return this.request("GET", `/admin/releases${qs}`);
     }
@@ -3545,13 +3623,16 @@ export class PicaClient {
     shareSend;
     accessSimulate;
     approvals;
-    /**
-     * Get accurate catalog stats via SQL counts (no pagination limits)
-     */
-    async catalogStats() {
+    async catalogStats(params) {
         const baseUrl = this.works["baseUrl"];
         const apiKey = this.works["apiKey"];
-        const res = await fetch(`${baseUrl}/admin/catalog/stats`, {
+        const qp = new URLSearchParams();
+        if (params?.entity)
+            qp.set("entity", params.entity);
+        if (params?.group_by)
+            qp.set("group_by", params.group_by);
+        const suffix = qp.toString() ? `?${qp.toString()}` : "";
+        const res = await fetch(`${baseUrl}/admin/catalog/stats${suffix}`, {
             headers: {
                 Authorization: `Bearer ${apiKey}`,
                 "Content-Type": "application/json",