engine-dj-mcp 0.15.0 → 0.17.0

package/dist/tools/search.js

@@ -32,6 +32,12 @@ export const FIELD_SQL = {
     label: "t.label",
     year: "t.year",
     rating: "t.rating",
+    // Engine stores 0, 20, 40, 60, 80, 100 -- measured: four stars set in Engine
+    // came back as 80. `rating` hands that back as it is; `rating_stars` is the
+    // same thing in the units a person uses, so a caller never has to know the
+    // factor. A value no Engine writes (a third-party tagger's 55) rounds to the
+    // nearest star here and stays exact in `rating`.
+    rating_stars: "CAST(ROUND(COALESCE(t.rating, 0) / 20.0) AS INTEGER)",
     length: "t.length",
     path: "t.path",
     filename: "t.filename",
@@ -296,13 +302,16 @@ export async function searchTracks(qp, raw) {
             filterParams.push(input.key.mode === "minor" ? "%A" : "%B");
         }
     }
+    // In stars, which is what README has always documented and what a person
+    // means. Compared against the raw column, `min: 4` matched 20, 40, 60, 80
+    // and 100 -- every rated track -- and `max: 3` matched only unrated ones.
     if (input.rating?.min !== undefined) {
         filterWhere.push("t.rating >= ?");
-        filterParams.push(input.rating.min);
+        filterParams.push(input.rating.min * 20);
     }
     if (input.rating?.max !== undefined) {
         filterWhere.push("t.rating <= ?");
-        filterParams.push(input.rating.max);
+        filterParams.push(input.rating.max * 20);
     }
     if (input.played?.never)
         filterWhere.push("(t.timeLastPlayed IS NULL OR t.isPlayed = 0)");

package/dist/tools/write-track-metadata.d.ts

@@ -0,0 +1,26 @@
+import { z } from "zod";
+import { type TrackUpdate } from "../store/track-metadata-plan.js";
+export declare const UpdateTrackMetadataInput: z.ZodObject<{
+    updates: z.ZodArray<z.ZodObject<{
+        id: z.ZodNumber;
+        genre: z.ZodOptional<z.ZodString>;
+        comment: z.ZodOptional<z.ZodString>;
+        label: z.ZodOptional<z.ZodString>;
+        year: z.ZodOptional<z.ZodNumber>;
+        rating_stars: z.ZodOptional<z.ZodNumber>;
+        rating_raw: z.ZodOptional<z.ZodNumber>;
+        expect: z.ZodOptional<z.ZodObject<{
+            genre: z.ZodOptional<z.ZodString>;
+            comment: z.ZodOptional<z.ZodString>;
+            label: z.ZodOptional<z.ZodString>;
+            year: z.ZodOptional<z.ZodNumber>;
+            rating_raw: z.ZodOptional<z.ZodNumber>;
+        }, z.core.$strict>>;
+    }, z.core.$strict>>;
+}, z.core.$strip>;
+export declare function runUpdateTrackMetadata(mdbPath: string, uuid: string, args: {
+    updates: TrackUpdate[];
+}, backupDir: string): Promise<import("../errors.js").EngineError | (import("../store/track-metadata.js").TrackMetadataResult & {
+    library: import("../store/write.js").LibraryRef;
+    backup_path?: string;
+})>;

package/dist/tools/write-track-metadata.js

@@ -0,0 +1,42 @@
+// src/tools/write-track-metadata.ts
+import { z } from "zod";
+import { updateTrackMetadata } from "../store/track-metadata.js";
+import { MAX_UPDATES } from "../store/track-metadata-plan.js";
+/**
+ * Types and the per-call cap only (spec §7.1). Every other rule -- ranges,
+ * which depend on whether `expect` is present, and the rating_raw restore
+ * rule -- lives in the store, so it comes back as a structured
+ * invalid_argument instead of a bare SDK validation message. `.strict()` makes
+ * a misnamed field such as `rating` an error rather than silently ignored.
+ */
+const Expect = z
+    .object({
+    genre: z.string().optional(),
+    comment: z.string().optional(),
+    label: z.string().optional(),
+    year: z.number().int().optional(),
+    rating_raw: z.number().int().optional(),
+})
+    .strict();
+export const UpdateTrackMetadataInput = z.object({
+    updates: z
+        .array(z
+        .object({
+        id: z.number().int().positive(),
+        genre: z.string().optional(),
+        comment: z.string().optional(),
+        label: z.string().optional(),
+        year: z.number().int().optional(),
+        rating_stars: z.number().int().optional(),
+        rating_raw: z.number().int().optional(),
+        expect: Expect.optional(),
+    })
+        .strict())
+        .max(MAX_UPDATES)
+        .describe("One entry per track: its id from search_tracks or get_tracks, and only the fields to change. " +
+        '"" clears a text field. rating_stars is 0-5. rating_raw and expect exist for replaying an undo; ' +
+        "you do not need them for an ordinary edit."),
+});
+export async function runUpdateTrackMetadata(mdbPath, uuid, args, backupDir) {
+    return updateTrackMetadata(mdbPath, uuid, { updates: args.updates }, { backupDir });
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "engine-dj-mcp",
-  "version": "0.15.0",
+  "version": "0.17.0",
   "description": "MCP server for an Engine DJ library: search and audit it, read cues and beatgrids, and build playlists when you ask. Not affiliated with inMusic or Denon DJ.",
   "keywords": [
     "mcp",