@withpica/mcp-sdk 3.14.0 → 3.16.0

package/CHANGELOG.md

@@ -11,6 +11,36 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [3.16.0] - 2026-09-09
+
+### Added
+
+- `works.search()` accepts `has_lyrics` (boolean) and writes it to the query
+  string. Without it a caller could not count works holding a lyric sheet
+  without paging the whole catalogue, which is how a connector session came to
+  inspect one work, see `lyrics: null`, and report that lyrics were not being
+  stored for an org holding 178 of them (PICA FIX_LOG 2026-09-09).
+
+⚠️ **`@withpica/mcp-server` must not be published against 3.15.0 once its
+`pica_works_query` declares `has_lyrics`.** `URLSearchParams` drops unknown
+keys silently, so an older SDK would send no filter, the route would return the
+unfiltered catalogue, and the tool description tells the agent to read `total`
+as the answer — it would report every work as having lyrics. Publish this
+package first.
+
+## [3.15.0] - 2026-09-08
+
+### Added
+
+- `selections` resource (ADR-319 saved catalog selections) — `list`,
+  `create`, `inspect` (id + optional `resolve`/`limit`/`offset`), `update`,
+  `delete` over `/admin/selections`. Backs the new `pica_selections_*` MCP
+  tools (Task 8).
+- `health.catalogHealthFix` accepts an optional `selections` array
+  (`{ actionId, chosenPersonId }`) — the human choice a `resolve_person`
+  action needs. Type-only on this side (the body is forwarded as-is); the
+  route validates it.
+
 PR #2118 (`fix/avatar-import-path-2026-09-03`) is merged into the release branch, so its changes ship in the version below; publish in the order mcp-utils → mcp-sdk → mcp-server.
 
 ## [3.14.0] - 2026-09-04

package/dist/index.d.ts

@@ -985,6 +985,11 @@ declare class WorksResource extends BaseResource {
         has_iswc?: boolean;
         /** Filter to works that have (true) or lack (false) a publisher_name. */
         has_publisher?: boolean;
+        /**
+         * Filter to works that have (true) or lack (false) a stored lyric sheet
+         * (works.lyrics; blank counts as lacking).
+         */
+        has_lyrics?: boolean;
     }): Promise<PaginatedResult<Work>>;
     get(id: string): Promise<Work>;
     create(data: Partial<Work> & {
@@ -1508,6 +1513,31 @@ interface IdentifyResult {
         confidence: number;
     };
 }
+export interface AudioFileTranscriptSegment {
+    start: number;
+    end: number;
+    text: string;
+    confidence?: number;
+}
+export interface AudioFileTranscript {
+    audio_file_id: string;
+    has_transcript: boolean;
+    /** absent when has_transcript is false */
+    work_id?: string | null;
+    recording_id?: string | null;
+    /** false when the transcript reaches no work — nothing else can show it */
+    reaches_work?: boolean;
+    transcript: {
+        full_text: string;
+        language: string | null;
+        confidence: number | null;
+        segment_count: number;
+        segments: AudioFileTranscriptSegment[];
+        processor: string | null;
+        model_version: string | null;
+        created_at: string | null;
+    } | null;
+}
 declare class AudioFilesResource extends BaseResource {
     list(params?: {
         work_id?: string;
@@ -1519,6 +1549,16 @@ declare class AudioFilesResource extends BaseResource {
         offset?: number;
     }): Promise<AudioFile[]>;
     get(id: string): Promise<AudioFile>;
+    /**
+     * The stored Whisper transcript for one audio file, linked to a work or not.
+     *
+     * Until 2026-09-07 a transcript was readable only as `works.lyrics` after
+     * propagation, so a transcript on an unlinked file had no reader at all —
+     * 218 of 558 on prod. `has_transcript: false` is a normal answer, not an
+     * error, and `reaches_work` says whether this transcript can be seen
+     * anywhere else.
+     */
+    getTranscript(id: string): Promise<AudioFileTranscript>;
     /**
      * Attach ALL orphan audio files whose suggested match is ISRC-high and
      * non-ambiguous, in one call. Pairings are recomputed server-side at
@@ -2330,11 +2370,17 @@ declare class HealthResource extends BaseResource {
     /**
      * ADR-273 — apply accepted fix actions from a prior plan. Requires planToken
      * from catalogHealthPlan; accept is a list of actionIds (or "all_safe_auto").
+     * `selections` carries the human choice a resolve_person action needs
+     * ({ actionId, chosenPersonId }); a selection also accepts its action.
      * Returns honest per-action met/unmet plus a fresh verdict.
      */
     catalogHealthFix(body: {
         planToken: string;
         accept: string[];
+        selections?: Array<{
+            actionId: string;
+            chosenPersonId: string;
+        }>;
     }): Promise<CatalogHealthFixResult>;
 }
 declare class DashboardResource extends BaseResource {
@@ -4536,6 +4582,104 @@ declare class StatementsResource extends BaseResource {
         rows: unknown[];
     }): Promise<Record<string, unknown>>;
 }
+/**
+ * ADR-319 — saved catalog selections. `mcp-sdk` is a standalone published
+ * package with no access to the app's path aliases, so this is a hand-kept
+ * mirror of `lib/services/catalog-selection/types.ts` rather than an
+ * import. Keep the two in sync by hand when the shape changes.
+ *
+ * `evaluable`/`substrate_rows`/`org_rows` on {@link CoverageEntry} are the
+ * whole point of the feature (ADR-319 Decision 3): a zero-member resolution
+ * with `substrate_rows === 0` on a clause means "PICA does not know", not
+ * "there are none" — the never-invent rule applied to a selection.
+ */
+export type SubstrateUnit = "works" | "credit_rows" | "audio_analysis_rows";
+export interface CoverageEntry {
+    clause: string;
+    /**
+     * False when the resolver had nothing to run the clause against — today,
+     * a role class the vocabulary no longer carries, which a stored predicate
+     * can still name. The clause is narrowed to nothing rather than dropped,
+     * and this flag is what says the empty answer is PICA's ignorance rather
+     * than the catalogue's.
+     */
+    evaluable: boolean;
+    substrate_rows: number;
+    /**
+     * What `substrate_rows` counted. NOT the same unit as `org_rows`, which is
+     * always works: a role clause counts credits and a trait clause counts
+     * audio analyses, so the two numbers are not two halves of one fraction.
+     */
+    substrate_unit: SubstrateUnit;
+    org_rows: number;
+}
+export interface PlayableRef {
+    source: "spotify" | "youtube" | "apple" | "audio_file";
+    ref: string;
+}
+export interface SelectionMember {
+    work_id: string;
+    title: string;
+    primary_artist: string | null;
+    playable: PlayableRef | null;
+    has_lyrics: boolean;
+}
+export interface ResolveResult {
+    members: SelectionMember[];
+    total: number;
+    coverage: CoverageEntry[];
+    /** True when `total` exceeded the page size and members is a first page. */
+    truncated: boolean;
+}
+export interface CatalogSelection {
+    id: string;
+    organisation_id: string;
+    name: string;
+    description: string | null;
+    predicate: Record<string, unknown>;
+    renderer: "listen" | "worklist";
+    visibility: "private" | "shared" | "public";
+    created_by: string | null;
+    created_at: string;
+    updated_at: string;
+    last_resolved_at: string | null;
+    last_resolved_count: number | null;
+    is_deleted: boolean;
+}
+export interface CreateSelectionInput {
+    name: string;
+    description?: string;
+    /** `{ version: 1, all: [...clauses] }` — validated server-side. */
+    predicate: Record<string, unknown>;
+    renderer?: "listen" | "worklist";
+}
+export interface UpdateSelectionInput {
+    name?: string;
+    description?: string;
+    predicate?: Record<string, unknown>;
+}
+declare class SelectionsResource extends BaseResource {
+    list(params?: {
+        limit?: number;
+        offset?: number;
+    }): Promise<CatalogSelection[]>;
+    create(data: CreateSelectionInput): Promise<CatalogSelection>;
+    /**
+     * Per-id read. `resolve: true` also resolves the predicate — the route
+     * returns `{ selection, resolution }` instead of `{ selection }`, and
+     * `resolution` is omitted below when the caller didn't ask for it.
+     */
+    inspect(id: string, opts?: {
+        resolve?: boolean;
+        limit?: number;
+        offset?: number;
+    }): Promise<{
+        selection: CatalogSelection;
+        resolution?: ResolveResult;
+    }>;
+    update(id: string, data: UpdateSelectionInput): Promise<CatalogSelection>;
+    delete(id: string): Promise<void>;
+}
 declare class ShareLinksResource extends BaseResource {
     list(params?: {
         work_id?: string;
@@ -4859,6 +5003,8 @@ export declare class PicaClient {
     workForHire: WorkForHireResource;
     royalties: RoyaltiesResource;
     statements: StatementsResource;
+    /** ADR-319 — saved catalog selections. */
+    selections: SelectionsResource;
     shareLinks: ShareLinksResource;
     consent: ConsentResource;
     custody: CustodyResource;