@withpica/mcp-sdk 3.15.0 → 3.17.0

package/CHANGELOG.md

@@ -11,6 +11,37 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [3.17.0] - 2026-09-13
+
+### Fixed
+
+- **`credits.atomicUpdate()` now returns what its type declares.** It went
+  through `request()`, whose `return data.data || data` narrowed the route's
+  `{ data, credit_id, work_id, allocation }` envelope to the bare credit row —
+  so `allocation` (the work's writer allocation after the edit, which
+  `pica_credit_update` promises in its description) was always `undefined`,
+  and the tool's `allocation.status` completion hints were dead code. It now
+  uses `requestWithEnvelope` and returns `{ credit_id, work_id, credit,
+allocation }`. Callers that read credit fields off the top-level result
+  should read `result.credit` — the declared shape never promised them there.
+
+## [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

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> & {
@@ -1342,6 +1347,22 @@ export interface WorkCreditAtomicAddInput {
     share_pct?: number;
     notes?: string;
 }
+/**
+ * In-place edit of ONE credit (2026-09-11). Every field optional; only the
+ * fields present are written. `share_pct: null` records the share as
+ * UNKNOWN, never as 0. `instrument` applies to a recording (master) credit.
+ */
+export interface WorkCreditAtomicUpdateInput {
+    credit_type?: WorkCreditAtomicAddInput["credit_type"];
+    share_pct?: number | null;
+    credited_name?: string | null;
+    notes?: string | null;
+    instrument?: string | null;
+}
+export interface AtomicCreditUpdateResult extends AtomicCreditResult {
+    /** The work's writer allocation AFTER this change — closed / open / incomplete / no_writers / unscalable. */
+    allocation?: Record<string, unknown>;
+}
 export interface AtomicCreditResult {
     credit_id: string;
     credit?: Record<string, unknown>;
@@ -1370,6 +1391,20 @@ declare class CreditsResource extends BaseResource {
      * semantics — never leaks 404 vs 403 via latency).
      */
     atomicRemove(workId: string, creditId: string): Promise<AtomicCreditResult>;
+    /**
+     * Edit one credit in place — the verb between add and remove.
+     *
+     * Goes through `requestWithEnvelope`, not `request()`: the route answers
+     * `{ success, data: <credit row>, credit_id, work_id, allocation }` with
+     * `allocation` as a SIBLING of `data`, and `request()`'s
+     * `return data.data || data` narrowed that to the bare credit row — so
+     * every caller received an object that matched none of this method's
+     * declared return type, and `allocation` (which `pica_credit_update`
+     * promises in its description) was always undefined. Measured 2026-09-12
+     * on the #2210 preview; pinned by the holdout contract
+     * `scenarios/contracts/credit-update.spec.ts` leg 4.
+     */
+    atomicUpdate(workId: string, creditId: string, patch: WorkCreditAtomicUpdateInput): Promise<AtomicCreditUpdateResult>;
     /**
      * Send pending credits on a work to their recipients for attestation.
      * Lightweight alternative to pica_split_sheet_send — fans out the
@@ -1508,6 +1543,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 +1579,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
@@ -3208,7 +3278,12 @@ declare class ExportResource extends BaseResource {
     }): Promise<any>;
 }
 declare class DuplicatesResource extends BaseResource {
-    findDuplicates(entityType: "works" | "people" | "releases" | "recordings"): Promise<any>;
+    /**
+     * ⚠️ `audio_files` is findable but NOT mergeable — `merge` below keeps the
+     * narrower union deliberately. Re-uploaded audio is resolved by a person
+     * deleting a file, never by a merge (ops issue dcea642b, 2026-09-10).
+     */
+    findDuplicates(entityType: "works" | "people" | "releases" | "recordings" | "audio_files"): Promise<any>;
     merge(entityType: "work" | "person" | "recording" | "release", winnerId: string, loserIds: string[]): Promise<any>;
 }
 declare class EntityContextResource extends BaseResource {