@kolbo/mcp 1.82.1 → 1.82.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kolbo/mcp",
-  "version": "1.82.1",
+  "version": "1.82.3",
   "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

@@ -280,8 +280,12 @@ function renderChips(sc) {
   var s = sc.settings || {};
   var kind = displayKind(sc);
   if (kind) h += chip(iconFor(kind) + ' ' + kind);
-  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');
+  // Shot count when we know it, otherwise just say multishot is on — the
+  // provider picks the count in that case, and showing nothing made an
+  // enabled toggle look like it had not applied.
+  var shotLabel = s.shots > 1 ? (s.shots + ' shots') : (s.multi_shot ? 'multishot' : '');
+  if (s.duration) h += chip(ICONS.clock + ' ' + fmtDur(s.duration) + (shotLabel ? ' · ' + shotLabel : ''));
+  else if (shotLabel) h += chip(shotLabel);
   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');
@@ -441,6 +445,47 @@ function iconFor(kind) {
 /* ---------- generating ---------- */
 function isBatch(sc) { return !!(sc && sc.generation_ids && sc.generation_ids.length > 1); }
 
+// Columns for a batch grid. Video caps at 2 per row: a 16:9 cell in a 4-up
+// grid lands around 200px inside a chat card, which is too small to judge a
+// shot and far too small to scrub once <video controls> appears. Images stay
+// 4-up — they read fine as contact-sheet tiles.
+function gridCols(count, kind) {
+  return Math.min(count, kind === 'video' ? 2 : 4);
+}
+
+// Batch prompts routinely share a long identical preamble (a locked global
+// look, style rules, duration). Clamping from the left then renders EVERY
+// caption as the same string — four cells all reading "Total: 8s [GLOBAL LOOK
+// - LOCKED]…" identify nothing. Drop the shared prefix so the caption shows
+// the part that actually differs. Full text stays in the tooltip.
+function batchCaptions(prompts) {
+  var list = (prompts || []).map(function (p) { return String(p == null ? '' : p); });
+  if (list.length < 2) return list;
+  var first = list[0], n = first.length;
+  for (var i = 1; i < list.length && n > 0; i++) {
+    var j = 0;
+    while (j < n && j < list[i].length && list[i].charAt(j) === first.charAt(j)) j++;
+    n = j;
+  }
+  // Short shared runs are just coincidence ("A ", "The "), not a preamble.
+  if (n < 12) return list;
+  // Back up to a word boundary so a label never starts mid-word.
+  var cut = n;
+  while (cut > 0 && !/\s/.test(first.charAt(cut - 1))) cut--;
+  if (!cut) cut = n;
+  return list.map(function (p) {
+    var rest = p.slice(cut).trim();
+    // A prompt identical to the prefix has nothing distinct to show — keep it
+    // whole rather than rendering an empty chip.
+    return rest ? '… ' + rest : p;
+  });
+}
+function capAt(sc, i) {
+  if (!sc || !sc.prompts || !sc.prompts[i]) return '';
+  if (!sc._caps) sc._caps = batchCaptions(sc.prompts);
+  return sc._caps[i] || sc.prompts[i];
+}
+
 function renderGenerating(sc) {
   setPhaseChip('Generating', true);
   var n = Math.min(sc.count || 1, isBatch(sc) ? 8 : 4);
@@ -449,11 +494,11 @@ function renderGenerating(sc) {
   var cells = '';
   for (var i = 0; i < n; i++) {
     var cap = (sc.prompts && sc.prompts[i])
-      ? '<span class="k-skel-cap" title="' + esc(sc.prompts[i]) + '">' + esc(sc.prompts[i]) + '</span>' : '';
+      ? '<span class="k-skel-cap" title="' + esc(sc.prompts[i]) + '">' + esc(capAt(sc, i)) + '</span>' : '';
     cells += '<div class="k-skel ' + shape + '" data-cell="' + i + '">' + cap + '</div>';
   }
   // Grid class caps at n4 — the auto-fill rule handles any larger batch count.
-  el('stage').innerHTML = '<div class="k-gen-grid n' + Math.min(n, 4) + '">' + cells + '</div>';
+  el('stage').innerHTML = '<div class="k-gen-grid n' + gridCols(n, kind) + '">' + cells + '</div>';
   renderStopButton(sc);
   schedulePoll(sc);
 }
@@ -709,7 +754,7 @@ function handleBatchStatus(sc, st) {
     }
     scenes.push({
       scene_number: i + 1,
-      title: (sc.prompts && sc.prompts[i]) || '',
+      title: capAt(sc, i),
       image_urls: sc.kind === 'video' ? [] : urls,
       video_urls: sc.kind === 'video' ? urls : []
     });
@@ -743,7 +788,7 @@ function fillBatchCell(sc, i, g) {
   cell.setAttribute('data-done', '1');
   cell.classList.add('done');
   var cap = (sc.prompts && sc.prompts[i])
-    ? '<span class="k-skel-cap" title="' + esc(sc.prompts[i]) + '">' + esc(sc.prompts[i]) + '</span>' : '';
+    ? '<span class="k-skel-cap" title="' + esc(sc.prompts[i]) + '">' + esc(capAt(sc, i)) + '</span>' : '';
   if (sc.prompts && sc.prompts[i]) cell.title = sc.prompts[i];
   // The controls attribute is not optional here: this cell is a FINISHED result
   // the user is meant to watch, and without it the tile was a muted poster with
@@ -939,7 +984,7 @@ function renderBatchGrid(sc) {
   var items = sceneItems(sc);
   if (!items.length) return renderError('No completed results received');
   var shape = items[0].type === 'video' ? 'video' : 'square';
-  el('stage').innerHTML = '<div class="k-gen-grid n' + Math.min(items.length, 4) + '">' +
+  el('stage').innerHTML = '<div class="k-gen-grid n' + gridCols(items.length, items[0].type) + '">' +
     items.map(function (it, i) {
       return '<div class="k-skel done ' + shape + '" data-focus="' + i + '"' +
         (it.label ? ' title="' + esc(it.label) + '"' : '') + '>' +

package/src/tools/generate.js

@@ -186,6 +186,9 @@ const videoSettings = (a = {}) => ({
   // 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 } : {}),
+  // Multishot with no explicit count: the provider decides how many, so there
+  // is no number to show — but the card still has to say the toggle applied.
+  ...(a.multi_shot ? { multi_shot: true } : {}),
   resolution: a.resolution,
   aspect_ratio: a.aspect_ratio,
   ...(a.mode ? { mode: a.mode } : {}),
@@ -1374,7 +1377,14 @@ function registerGenerateTools(server, client, options = {}) {
         settings: videoSettings({
           duration,
           reported_duration: startResponse?.duration ?? startResponse?.result?.duration,
-          shots: multi_shot_count ?? (Array.isArray(multi_shots) ? multi_shots.length : undefined),
+          // `multi_shots` is a boolean (z.boolean()); the old
+          // `Array.isArray(multi_shots) ? multi_shots.length : undefined`
+          // branch could never be true, so enabling multishot WITHOUT an
+          // explicit count left shots undefined and the card silently showed
+          // no multishot indicator at all. Carry the flag so the chip can
+          // render even when the shot count is the provider's to decide.
+          shots: multi_shot_count,
+          multi_shot: multi_shots === true || undefined,
           resolution, aspect_ratio, enhance_prompt, visual_dna_ids, preset_id,
         }),
         // Already bucketed and deduped above, so a URL handed to `files`
@@ -1396,7 +1406,14 @@ function registerGenerateTools(server, client, options = {}) {
         settings: videoSettings({
           duration,
           reported_duration: result.result?.duration,
-          shots: multi_shot_count ?? (Array.isArray(multi_shots) ? multi_shots.length : undefined),
+          // `multi_shots` is a boolean (z.boolean()); the old
+          // `Array.isArray(multi_shots) ? multi_shots.length : undefined`
+          // branch could never be true, so enabling multishot WITHOUT an
+          // explicit count left shots undefined and the card silently showed
+          // no multishot indicator at all. Carry the flag so the chip can
+          // render even when the shot count is the provider's to decide.
+          shots: multi_shot_count,
+          multi_shot: multi_shots === true || undefined,
           resolution, aspect_ratio, enhance_prompt, visual_dna_ids, preset_id,
         }),
         reference_images: [...urlsOf('image'), ...(keyframes || []).map((keyframe) => keyframe.image_url)],

package/src/tools/visual_dna.js

@@ -102,7 +102,7 @@ function registerVisualDnaTools(server, client, options = {}) {
     {
       scope: z.enum(['all', 'personal', 'global', 'organization']).optional().describe('Default: "personal" — the user\'s own DNAs (plus a shared project\'s when project_id is set). "global" = the ~1000 system cast/preset DNAs, for browsing when the user needs a character and has none of their own. "organization" = org-shared. "all" = everything, rarely wanted.'),
       search: z.string().optional().describe('Search by name, tags, or description (case-insensitive). Matches at WORD STARTS, so "man" finds "Man"/"Manager" but not "woman" or "romantic". Name matches are ranked first.'),
-      collection: z.string().optional().describe('Filter global presets by collection: cast, influencers, props, locations, styles, glamour, street'),
+      collection: z.string().optional().describe('Filter global presets by collection: cast, influencers, props, locations, styles, street'),
       tags: z.string().optional().describe('Comma-separated tags to filter by (OR logic)'),
       page: z.number().optional().describe('Page number, 1-indexed. Default: 1. Needed to reach the global cast beyond the first page.'),
       limit: z.number().optional().describe('Results per page, max 100. Default: 50'),