@williamzujkowski/live-coding-music-mcp 4.0.0 → 4.1.1

package/dist/server/resources.js

@@ -31,6 +31,7 @@ import { sessionModule } from './tools/session.js';
 import { captureModule } from './tools/capture.js';
 import { aiModule } from './tools/ai.js';
 import { composeModule } from './tools/compose.js';
+import { TEMPO_BY_STYLE } from '../services/StyleRegistry.js';
 export const resources = [
     {
         uri: 'strudel://examples',
@@ -41,7 +42,7 @@ export const resources = [
     {
         uri: 'strudel://patterns',
         name: 'Saved patterns',
-        description: 'Patterns saved by the user via the `save` tool, with tags and timestamps',
+        description: 'Patterns saved by the user via the `pattern_store` tool, with tags and timestamps',
         mimeType: 'application/json',
     },
     {
@@ -66,23 +67,16 @@ export function isResourceUri(uri) {
  * compose.ts:TEMPO_BY_STYLE — both grow together when a new genre
  * lands. Future cleanup (#120 aftermath): single source of truth.
  */
-const STYLE_DEFAULTS = {
-    techno: 130,
-    house: 125,
-    dnb: 174,
-    ambient: 80,
-    trap: 140,
-    jungle: 160,
-    jazz: 110,
-    experimental: 120,
-    dubstep: 140,
-    trance: 138,
-    breakbeat: 130,
-    garage: 130,
-    electro: 128,
-    downtempo: 90,
-    idm: 115,
-};
+// The style list is no longer duplicated here.
+//
+// This was a hand-maintained copy, and it had drifted: it omitted
+// intelligent_dnb, trip_hop and boom_bap — the only three genres with
+// BOTH dedicated drums and a dedicated bassline — plus acid, dub and
+// funk. An agent using this resource to discover what the generator
+// accepts would never find trip_hop. The comment above it claimed it
+// was "kept in sync with compose.ts:TEMPO_BY_STYLE", but that table
+// moved to StyleRegistry and nothing guarded the copy (#341).
+const STYLE_DEFAULTS = TEMPO_BY_STYLE;
 const TOOL_MODULES = [
     editorModule,
     playbackModule,
@@ -128,14 +122,27 @@ async function readExamples(examplesDir) {
     return { count: items.length, examples: items };
 }
 async function readSavedPatterns(store) {
-    const patterns = await store.list();
+    // `listDetailed`, not `list`.
+    //
+    // #426 made the store keep going past a file it cannot read and report
+    // how many it skipped, because one truncated `.json` used to make
+    // every saved pattern list as zero. The `pattern_store` tool got that
+    // count; this resource did not, so it still reported `count: N` with
+    // unreadable files dropped in silence (#442).
+    const { patterns, skipped } = await store.listDetailed();
     const items = patterns.map(p => ({
         name: p.name,
         tags: p.tags,
         timestamp: p.timestamp,
         contentPreview: p.content.length > 200 ? p.content.slice(0, 200) + '...' : p.content,
     }));
-    return { count: items.length, patterns: items };
+    return {
+        count: items.length,
+        patterns: items,
+        // Only when there is something to say, so the common shape is
+        // unchanged for callers already parsing this.
+        ...(skipped > 0 ? { skipped, note: `${String(skipped)} file(s) could not be read and were skipped.` } : {}),
+    };
 }
 function readStyles() {
     const items = Object.entries(STYLE_DEFAULTS)