@labelgrid/core 0.2.1 → 0.2.3

package/CHANGELOG.md

@@ -8,6 +8,32 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 `@labelgrid/core` is primarily an internal shared client for the LabelGrid MCP
 server and CLI — there are no API-stability promises before 1.0.
 
+## [Unreleased]
+
+## [0.2.3] - 2026-09-30
+
+### Changed
+
+- Public API `error_code` and `details` fields are preserved on normalized
+  errors, including typed 404 and 5xx responses.
+- Normalized errors also preserve the API's `blocking_issues` field, which
+  lists the review issues that block a confirm-review request.
+
+## [0.2.2] - 2026-08-20
+
+### Added
+
+- Catalog entities now declare whether their delete endpoint accepts a
+  replacement, allowing clients to expose reassignment only for writers and
+  publishers.
+- `LabelGridClient.delete()` accepts optional query parameters while preserving
+  the existing request URL byte-for-byte when none are supplied.
+
+### Changed
+
+- Writer and publisher delete guidance now names every reference that can block
+  deletion.
+
 ## [0.2.1] - 2026-08-05
 
 ### Changed

package/dist/api/http.d.ts

@@ -15,6 +15,9 @@ export type ApiError = {
     suggestion?: string;
     retry_after_seconds?: number;
     errors?: unknown;
+    details?: unknown;
+    /** Actionable release-review issues returned when confirm-review is blocked. */
+    blocking_issues?: unknown;
     /** Structured validation detail passed through verbatim from the API (422). */
     errors_structured?: unknown;
 };
@@ -72,7 +75,12 @@ export declare class LabelGridClient {
         idempotency?: boolean;
         idempotencyKey?: string;
     }): Promise<ApiResult<T>>;
-    delete<T>(path: string): Promise<ApiResult<T>>;
+    /**
+     * A delete carries its parameters in the query string, never in a body: RFC
+     * 9110 gives a DELETE body no defined semantics and intermediaries drop it.
+     * Omitting `query` sends exactly the URL it always did.
+     */
+    delete<T>(path: string, query?: Record<string, unknown>): Promise<ApiResult<T>>;
     /**
      * Sends a multipart/form-data POST with a single file field plus optional
      * extra string fields. A missing/unreadable file yields a FILE_NOT_FOUND

package/dist/api/http.js

@@ -56,8 +56,8 @@ function buildQuery(query) {
     return parts.length > 0 ? `?${parts.join('&')}` : '';
 }
 /**
- * Extracts a code/message/errors triple from any of the four backend error body
- * shapes: `{message}`, `{error: string}`, `{errors}`, `{error: {code, message}}`.
+ * Extracts typed fields from the backend's top-level and nested error shapes,
+ * including public `error_code` and `details` fields.
  */
 function extractServerError(body) {
     if (typeof body === 'string') {
@@ -69,11 +69,22 @@ function extractServerError(body) {
     const record = body;
     const errors = record.errors;
     const errorsStructured = record.errors_structured;
+    const details = record.details;
+    const blockingIssues = record.blocking_issues;
+    const topLevelCode = typeof record.error_code === 'string'
+        ? record.error_code
+        : typeof record.code === 'string'
+            ? record.code
+            : undefined;
     // Shape: { error: { code, message } }
     if (record.error !== null && typeof record.error === 'object') {
         const nested = record.error;
         return {
-            code: typeof nested.code === 'string' ? nested.code : undefined,
+            code: typeof nested.error_code === 'string'
+                ? nested.error_code
+                : typeof nested.code === 'string'
+                    ? nested.code
+                    : topLevelCode,
             message: typeof nested.message === 'string'
                 ? nested.message
                 : typeof nested.error === 'string'
@@ -81,24 +92,30 @@ function extractServerError(body) {
                     : undefined,
             errors,
             errors_structured: errorsStructured,
+            details,
+            blocking_issues: blockingIssues,
         };
     }
     // Shape: { error: 'string' }
     if (typeof record.error === 'string') {
         return {
-            code: typeof record.code === 'string' ? record.code : undefined,
+            code: topLevelCode,
             message: record.error,
             errors,
             errors_structured: errorsStructured,
+            details,
+            blocking_issues: blockingIssues,
         };
     }
     // Shapes: { message } and/or { errors } and/or top-level { code }
     const parts = {
-        code: typeof record.code === 'string' ? record.code : undefined,
+        code: topLevelCode,
         message: typeof record.message === 'string' ? record.message : undefined,
         field: typeof record.field === 'string' ? record.field : undefined,
         errors,
         errors_structured: errorsStructured,
+        details,
+        blocking_issues: blockingIssues,
     };
     // Derive a message from the first validation error when none was given.
     if (parts.message === undefined && errors !== null && typeof errors === 'object') {
@@ -129,6 +146,8 @@ function normalizeError(res, body) {
         status,
         ...(server.field !== undefined ? { field: server.field } : {}),
         ...(server.errors !== undefined ? { errors: server.errors } : {}),
+        ...(server.details !== undefined ? { details: server.details } : {}),
+        ...(server.blocking_issues !== undefined ? { blocking_issues: server.blocking_issues } : {}),
         ...extra,
     });
     switch (status) {
@@ -139,7 +158,7 @@ function normalizeError(res, body) {
         case 403:
             return withCommon(server.code ?? 'FORBIDDEN', server.message ?? 'Forbidden.');
         case 404:
-            return withCommon('NOT_FOUND', server.message ?? 'The requested resource was not found.');
+            return withCommon(server.code ?? 'NOT_FOUND', server.message ?? 'The requested resource was not found.');
         case 409:
             return withCommon(server.code ?? 'CONFLICT', server.message ?? 'The request conflicts with the current state.');
         case 422:
@@ -156,7 +175,7 @@ function normalizeError(res, body) {
         }
         default:
             if (status >= 500) {
-                return withCommon('SERVER_ERROR', server.message ?? 'The server encountered an error.');
+                return withCommon(server.code ?? 'SERVER_ERROR', server.message ?? 'The server encountered an error.');
             }
             return withCommon(server.code ?? 'ERROR', server.message ?? `Request failed with status ${status}.`);
     }
@@ -357,8 +376,13 @@ export class LabelGridClient {
             idempotencyKey: opts?.idempotencyKey,
         });
     }
-    delete(path) {
-        return this.send('DELETE', path);
+    /**
+     * A delete carries its parameters in the query string, never in a body: RFC
+     * 9110 gives a DELETE body no defined semantics and intermediaries drop it.
+     * Omitting `query` sends exactly the URL it always did.
+     */
+    delete(path, query) {
+        return this.send('DELETE', path, { query });
     }
     /**
      * Sends a multipart/form-data POST with a single file field plus optional

package/dist/entities.d.ts

@@ -23,5 +23,19 @@ export type EntitySpec = {
     fieldsDoc: string;
     /** One-line doc of the server-side delete refusals. */
     deleteNote: string;
+    /**
+     * Whether this entity's DELETE endpoint accepts `replace_with` — the id of the
+     * entity that takes over every credit held by the one being deleted, rewritten
+     * before the delete. Only the writer and publisher endpoints have it. Stated
+     * for every entity rather than defaulted, so a new entity cannot inherit an
+     * answer nobody gave.
+     */
+    acceptsDeleteReplacement: boolean;
 };
 export declare const ENTITIES: Record<EntityName, EntitySpec>;
+/**
+ * The entities whose DELETE accepts `replace_with`, derived from the registry so
+ * the tool schema, its description and its refusal message cannot disagree with
+ * the table or with each other.
+ */
+export declare const REPLACEMENT_ENTITIES: readonly EntityName[];

package/dist/entities.js

@@ -19,35 +19,47 @@ export const ENTITIES = {
         filtersDoc: 'label: no documented filters.',
         fieldsDoc: 'label — required: name, default_email; optional: support email, website/platform URLs, default copyright lines, isrc_base.',
         deleteNote: 'label: refused while the label still has releases — remove or reassign its releases first.',
+        acceptsDeleteReplacement: false,
     },
     artist: {
         path: '/artists',
         filtersDoc: 'artist: artist_name.',
         fieldsDoc: 'artist — required: artist_name; optional: full_name, email, location, bios, isni, default_language, platform profile URLs.',
         deleteNote: 'artist: refused while still referenced by releases or tracks.',
+        acceptsDeleteReplacement: false,
     },
     writer: {
         path: '/writers',
         filtersDoc: 'writer: name, ipi.',
         fieldsDoc: 'writer — required: first_name, last_name; optional: middle_name, display_credits, email, country, pro, ipi, isni, publisher_id (or publisher_name/publisher_pro/publisher_ipi).',
-        deleteNote: 'writer: refused while still referenced by tracks.',
+        deleteNote: 'writer: refused while still referenced by tracks or artists, unless replace_with reassigns those credits.',
+        acceptsDeleteReplacement: true,
     },
     publisher: {
         path: '/publishers',
         filtersDoc: 'publisher: name, ipi.',
         fieldsDoc: 'publisher — required: name; optional: ipi, pro, isni, controlled_publisher.',
-        deleteNote: 'publisher: refused while still referenced by writers.',
+        deleteNote: 'publisher: refused while still referenced by tracks or label default publishers, unless replace_with reassigns those credits.',
+        acceptsDeleteReplacement: true,
     },
     release: {
         path: '/releases',
         filtersDoc: 'release: label_id, is_live (1 = live only), barcode_number (UPC/EAN), cat.',
         fieldsDoc: 'release — required on create: content_type, label_id, artists, titles, cat (catalog number), artwork_ai_usage, primary_genre_id; many optional fields (dates, copyright lines, genres, per-outlet URLs).',
         deleteNote: 'release: only a never-submitted draft can be deleted.',
+        acceptsDeleteReplacement: false,
     },
     track: {
         path: '/tracks',
         filtersDoc: 'track: release_id, isrc.',
         fieldsDoc: 'track — required on create: release_id, disc, track_num, composition_type, artists, audio_ai_usage, composition_ai_usage, commercial_samples, audio_language, contributors, and recording_country (ISO 3166-1 alpha-2, e.g. "US"); optional: titles, isrc, iswc, writers, publishers, splits, and more.',
         deleteNote: 'track: refused once the release is no longer an editable draft.',
+        acceptsDeleteReplacement: false,
     },
 };
+/**
+ * The entities whose DELETE accepts `replace_with`, derived from the registry so
+ * the tool schema, its description and its refusal message cannot disagree with
+ * the table or with each other.
+ */
+export const REPLACEMENT_ENTITIES = ENTITY_NAMES.filter((name) => ENTITIES[name].acceptsDeleteReplacement);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@labelgrid/core",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "description": "Shared LabelGrid public-API client: HTTP transport, uploads, content types, the catalog-entity registry, and redacting logging",
   "type": "module",
   "keywords": ["labelgrid", "music-distribution", "api-client"],