@kolbo/mcp 1.72.3 → 1.72.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kolbo/mcp",
-  "version": "1.72.3",
+  "version": "1.72.4",
   "description": "Kolbo AI MCP Server - Generate images, videos, music, speech, and sound effects from Claude Code",
   "main": "src/index.js",
   "bin": {

package/src/apps/widgets/generation.js

@@ -164,7 +164,8 @@ function renderChips(sc) {
   var h = modelChipHTML(modelLabel(sc), sc.model_icon);
   var s = sc.settings || {};
   if (sc.kind) h += chip(iconFor(sc.kind) + ' ' + sc.kind);
-  if (s.duration) h += chip(ICONS.clock + ' ' + fmtDur(s.duration));
+  if (s.duration) h += chip(ICONS.clock + ' ' + fmtDur(s.duration) + (s.shots > 1 ? ' · ' + s.shots + ' shots' : ''));
+  else if (s.shots > 1) h += chip(s.shots + ' shots');
   if (s.resolution) h += chip(esc(s.resolution));
   if (s.aspect_ratio) h += chip(esc(s.aspect_ratio));
   if (s.quality) h += chip(esc(s.quality) + ' quality');

package/src/tools/generate.js

@@ -125,7 +125,13 @@ const imageSettings = (a = {}) => ({
 // Same block for the video tools, which carried only { duration, resolution,
 // aspect_ratio } — a DNA-anchored video card showed no sign a DNA was in play.
 const videoSettings = (a = {}) => ({
-  duration: a.duration,
+  // Multi-shot elements calls usually set shot COUNT and let the model take the
+  // default per-shot length, so `duration` came through undefined and the card
+  // showed model/resolution/ratio but never how long the video actually is —
+  // the one number the user picked and is paying for. Fall back to what the API
+  // echoed back on start, then to per-shot × shots.
+  duration: a.duration ?? a.reported_duration ?? (a.shot_duration && a.shots ? a.shot_duration * a.shots : undefined),
+  ...(a.shots > 1 ? { shots: a.shots } : {}),
   resolution: a.resolution,
   aspect_ratio: a.aspect_ratio,
   ...(a.mode ? { mode: a.mode } : {}),
@@ -1093,7 +1099,12 @@ function registerGenerateTools(server, client, options = {}) {
 
       if (ui()) return uiGenerating({
         tool: 'generate_elements', kind: 'video', gen: startResponse, client, model, prompt,
-        settings: videoSettings({ duration, resolution, aspect_ratio, enhance_prompt, visual_dna_ids, preset_id }),
+        settings: videoSettings({
+          duration,
+          reported_duration: startResponse?.duration ?? startResponse?.result?.duration,
+          shots: multi_shot_count ?? (Array.isArray(multi_shots) ? multi_shots.length : undefined),
+          resolution, aspect_ratio, enhance_prompt, visual_dna_ids, preset_id,
+        }),
         reference_images: [
           ...(reference_images || []),
           ...(keyframes || []).map((keyframe) => keyframe.image_url),

package/src/tools/media.js

@@ -6,7 +6,7 @@
 const { z } = require('zod');
 const FormData = require('form-data');
 const { resolveToBuffer, DEFAULT_MAX_FILE_MB, compactList } = require('./_shared');
-const { UI, uiResult, listResult, appsEnabled } = require('../apps');
+const { UI, uiResult, listResult } = require('../apps');
 
 // How many tiles the media grid renders. A rendering limit only — the text
 // payload always carries the full page, and `total` reports the real library
@@ -65,8 +65,6 @@ function uploadTicketPayload(ticket) {
 }
 
 function registerMediaTools(server, client, options = {}) {
-  const ui = () => appsEnabled(server, options);
-
   // `opts.apps` is set only by kolbo-api's per-request server (see createServer
   // in ../index.js), which makes it a TRANSPORT signal — deliberately not
   // `appsEnabled()`, which also returns true for stdio hosts that advertise UI.
@@ -101,24 +99,27 @@ function registerMediaTools(server, client, options = {}) {
         expires_in_seconds: ticket.expires_in,
       };
 
-      if (ui()) {
-        // upload_ui_url: top-level page for Claude iOS/Android — in-iframe
-        // <input type=file> selections are dropped by WebKit (see upload widget).
-        const uploadUiUrl = ticket.upload_ui_url
-          || String(ticket.upload_url || '').replace(/\/upload\/?$/, '/upload-ui');
-        return uiResult(UI.upload, JSON.stringify(info, null, 2), {
-          widget: 'upload',
-          title: purpose || 'Upload media',
-          upload_url: ticket.upload_url,
-          upload_ui_url: uploadUiUrl,
-          token: ticket.token,
-          expires_at: Date.now() + (ticket.expires_in || 900) * 1000,
-          kinds: media_types && media_types.length ? media_types : undefined,
-          max_files: Math.min(Math.max(Number(max_files) || 10, 1), 20),
-          max_mb: ticket.max_file_mb || DEFAULT_MAX_FILE_MB,
-          ...(project_id ? { project_id } : {}),
-        });
-      }
+      // Always ship structuredContent. Kolbo Code does NOT advertise MCP Apps, so
+      // gating the grid payload on appsEnabled() sent it text only; the host then
+      // rebuilt items from the compactList text, whose field names are
+      // `filename`/`url` — not the `title`/`thumbnail` the grid renders — so every
+      // tile came out black and unlabelled. Same reasoning as listResult().
+      // upload_ui_url: top-level page for Claude iOS/Android — in-iframe
+      // <input type=file> selections are dropped by WebKit (see upload widget).
+      const uploadUiUrl = ticket.upload_ui_url
+        || String(ticket.upload_url || '').replace(/\/upload\/?$/, '/upload-ui');
+      return uiResult(UI.upload, JSON.stringify(info, null, 2), {
+        widget: 'upload',
+        title: purpose || 'Upload media',
+        upload_url: ticket.upload_url,
+        upload_ui_url: uploadUiUrl,
+        token: ticket.token,
+        expires_at: Date.now() + (ticket.expires_in || 900) * 1000,
+        kinds: media_types && media_types.length ? media_types : undefined,
+        max_files: Math.min(Math.max(Number(max_files) || 10, 1), 20),
+        max_mb: ticket.max_file_mb || DEFAULT_MAX_FILE_MB,
+        ...(project_id ? { project_id } : {}),
+      });
 
       // Text-only host (Claude Code, Codex CLI, Cursor): no iframe to render —
       // but these are exactly the hosts that CAN reach a filesystem, so hand
@@ -263,30 +264,33 @@ function registerMediaTools(server, client, options = {}) {
         note: 'Narrow with `type`, `category`, `project_id`, `folder_id`, or `search`; get_media returns one item in full.',
       });
 
-      if (ui()) {
-        // The SDK envelope reports `total_items` (see sdk/controller.js listMedia);
-        // reading `total` always came back undefined, so the grid claimed the page
-        // size was the whole library. Accept either, then fall back.
-        const totalItems = pagination
-          ? (pagination.total_items != null ? pagination.total_items : pagination.total)
-          : null;
-        const items = media.slice(0, GRID_CAP).map((m) => ({
-          id: m.id,
-          title: m.filename,
-          subtitle: m.media_type + (m.size ? ' · ' + Math.round(m.size / 1024) + 'KB' : ''),
-          thumbnail: m.media_type === 'image' ? m.url : (m.thumbnail_url || null),
-          media_type: m.media_type,
-          url: m.url,
-          use_hint: 'Use this media library asset in my next step:\nURL: {URL}\n(id: {ID})'
-        }));
-        return uiResult(UI.mediaGrid, text, {
-          widget: 'media-grid',
-          title: 'Media Library',
-          items,
-          total: totalItems != null ? totalItems : media.length,
-          shown: Math.min(media.length, GRID_CAP)
-        });
-      }
+      // Always ship structuredContent. Kolbo Code does NOT advertise MCP Apps, so
+      // gating the grid payload on appsEnabled() sent it text only; the host then
+      // rebuilt items from the compactList text, whose field names are
+      // `filename`/`url` — not the `title`/`thumbnail` the grid renders — so every
+      // tile came out black and unlabelled. Same reasoning as listResult().
+      // The SDK envelope reports `total_items` (see sdk/controller.js listMedia);
+      // reading `total` always came back undefined, so the grid claimed the page
+      // size was the whole library. Accept either, then fall back.
+      const totalItems = pagination
+        ? (pagination.total_items != null ? pagination.total_items : pagination.total)
+        : null;
+      const items = media.slice(0, GRID_CAP).map((m) => ({
+        id: m.id,
+        title: m.filename,
+        subtitle: m.media_type + (m.size ? ' · ' + Math.round(m.size / 1024) + 'KB' : ''),
+        thumbnail: m.media_type === 'image' ? m.url : (m.thumbnail_url || null),
+        media_type: m.media_type,
+        url: m.url,
+        use_hint: 'Use this media library asset in my next step:\nURL: {URL}\n(id: {ID})'
+      }));
+      return uiResult(UI.mediaGrid, text, {
+        widget: 'media-grid',
+        title: 'Media Library',
+        items,
+        total: totalItems != null ? totalItems : media.length,
+        shown: Math.min(media.length, GRID_CAP)
+      });
 
       return {
         content: [{

package/src/tools/moodboards.js

@@ -4,11 +4,10 @@
  * new OPTIONAL args only. Full rules: ../index.js top-of-file and CLAUDE.md. */
 
 const { z } = require('zod');
-const { UI, uiResult, appsEnabled } = require('../apps');
+const { UI, uiResult } = require('../apps');
 const { projectScopeReadField } = require('./_shared');
 
 function registerMoodboardTools(server, client, options = {}) {
-  const ui = () => appsEnabled(server, options);
   // ─── list_moodboards ───────────────────────────────────────
   server.tool(
     'list_moodboards',
@@ -29,25 +28,26 @@ function registerMoodboardTools(server, client, options = {}) {
         count: result.count || 0
       }, null, 2);
 
-      if (ui()) {
-        return uiResult(UI.mediaGrid, text, {
-          widget: 'media-grid',
-          title: 'Moodboards',
-          items: moodboards.slice(0, 24).map(mb => ({
-            id: mb.id,
-            title: mb.name,
-            // API returns thumbnail_url + images[] (sdk listMoodboards) — both
-            // previous keys were wrong, so the fallback never fired either.
-            thumbnail: mb.thumbnail_url || mb.thumbnail || (Array.isArray(mb.images) ? mb.images[0] : undefined),
-            media_type: 'image',
-            use_hint: 'Apply moodboard "{TITLE}" (moodboard_id: {ID}) to my next generation.'
-          })),
-          total: result.count || moodboards.length,
-          has_more: moodboards.length > 24
-        });
-      }
-
-      return { content: [{ type: 'text', text }] };
+      // Always ship structuredContent. Kolbo Code does NOT advertise MCP Apps, so
+      // gating the grid payload on appsEnabled() sent it text only; the host then
+      // rebuilt items from the compactList text, whose field names are
+      // `filename`/`url` — not the `title`/`thumbnail` the grid renders — so every
+      // tile came out black and unlabelled. Same reasoning as listResult().
+      return uiResult(UI.mediaGrid, text, {
+        widget: 'media-grid',
+        title: 'Moodboards',
+        items: moodboards.slice(0, 24).map(mb => ({
+          id: mb.id,
+          title: mb.name,
+          // API returns thumbnail_url + images[] (sdk listMoodboards) — both
+          // previous keys were wrong, so the fallback never fired either.
+          thumbnail: mb.thumbnail_url || mb.thumbnail || (Array.isArray(mb.images) ? mb.images[0] : undefined),
+          media_type: 'image',
+          use_hint: 'Apply moodboard "{TITLE}" (moodboard_id: {ID}) to my next generation.'
+        })),
+        total: result.count || moodboards.length,
+        has_more: moodboards.length > 24
+      });
     }
   );
 

package/src/tools/visual_dna.js

@@ -6,7 +6,7 @@
 const { z } = require('zod');
 const FormData = require('form-data');
 const { resolveToBuffer: sharedResolveToBuffer, VISUAL_DNA_MAX_BYTES, projectScopeReadField, compactList } = require('./_shared');
-const { UI, uiResult, listResult, appsEnabled } = require('../apps');
+const { UI, uiResult, listResult } = require('../apps');
 
 // Reference sheets are a blocking multi-panel render; the 120s client default
 // aborted them mid-flight while the server finished and charged anyway.
@@ -20,7 +20,6 @@ function resolveToBuffer(source, kind) {
 }
 
 function registerVisualDnaTools(server, client, options = {}) {
-  const ui = () => appsEnabled(server, options);
   // ─── create_visual_dna ─────────────────────────────────────
   server.tool(
     'create_visual_dna',
@@ -133,24 +132,25 @@ function registerVisualDnaTools(server, client, options = {}) {
         note: 'Narrow with `search`, `tags`, or `collection`, or pass `page`/`limit` for the rest; get_visual_dna returns one in full.',
       });
 
-      if (ui()) {
-        return uiResult(UI.mediaGrid, text, {
-          widget: 'media-grid',
-          title: 'Visual DNA Profiles',
-          items: dnas.slice(0, 24).map(d => ({
-            id: d.id,
-            title: d.name,
-            subtitle: (d.dna_type || '') + (Array.isArray(d.tags) && d.tags.length ? ' · ' + d.tags.slice(0, 3).join(', ') : ''),
-            thumbnail: d.thumbnail_url || d.thumbnail,
-            media_type: 'image',
-            use_hint: 'Use Visual DNA "{TITLE}" (id: {ID}) in my next generation for character/style consistency.'
-          })),
-          total,
-          has_more: result.has_more || dnas.length > 24
-        });
-      }
-
-      return { content: [{ type: 'text', text }] };
+      // Always ship structuredContent. Kolbo Code does NOT advertise MCP Apps, so
+      // gating the grid payload on appsEnabled() sent it text only; the host then
+      // rebuilt items from the compactList text, whose field names are
+      // `filename`/`url` — not the `title`/`thumbnail` the grid renders — so every
+      // tile came out black and unlabelled. Same reasoning as listResult().
+      return uiResult(UI.mediaGrid, text, {
+        widget: 'media-grid',
+        title: 'Visual DNA Profiles',
+        items: dnas.slice(0, 24).map(d => ({
+          id: d.id,
+          title: d.name,
+          subtitle: (d.dna_type || '') + (Array.isArray(d.tags) && d.tags.length ? ' · ' + d.tags.slice(0, 3).join(', ') : ''),
+          thumbnail: d.thumbnail_url || d.thumbnail,
+          media_type: 'image',
+          use_hint: 'Use Visual DNA "{TITLE}" (id: {ID}) in my next generation for character/style consistency.'
+        })),
+        total,
+        has_more: result.has_more || dnas.length > 24
+      });
     }
   );