mulmoclaude 1.2.0 → 1.3.0

package/src/plugins/chart/index.ts

@@ -46,8 +46,6 @@ const chartPlugin: ToolPlugin<PresentChartData> = {
   viewComponent: wrapWithScope("chart", View as unknown as Component),
   previewComponent: wrapWithScope("chart", Preview as unknown as Component),
 };
-
-export default chartPlugin;
 export { TOOL_NAME };
 
 export const REGISTRATION: PluginRegistration = {

package/src/plugins/editImages/index.ts

@@ -33,8 +33,6 @@ const editImagesPlugin: ToolPlugin<ImageToolData> = {
   viewComponent: wrapWithScope("image", View),
   previewComponent: wrapWithScope("image", Preview),
 };
-
-export default editImagesPlugin;
 export { TOOL_NAME };
 
 export const REGISTRATION: PluginRegistration = {

package/src/plugins/generateImage/index.ts

@@ -55,8 +55,6 @@ const generateImagePlugin: ToolPlugin<ImageToolData> = {
   viewComponent: wrapWithScope("image", View),
   previewComponent: wrapWithScope("image", Preview),
 };
-
-export default generateImagePlugin;
 export { TOOL_NAME };
 
 export const REGISTRATION: PluginRegistration = {

package/src/plugins/manageSkills/index.ts

@@ -47,8 +47,6 @@ const manageSkillsPlugin: ToolPlugin<ManageSkillsData> = {
   viewComponent: wrapWithScope("skills", View),
   previewComponent: wrapWithScope("skills", Preview),
 };
-
-export default manageSkillsPlugin;
 export { TOOL_NAME };
 
 export const REGISTRATION: PluginRegistration = {

package/src/plugins/markdown/index.ts

@@ -52,8 +52,6 @@ const markdownPlugin: ToolPlugin<MarkdownToolData> = {
   viewComponent: wrapWithScope("markdown", View as unknown as Component),
   previewComponent: wrapWithScope("markdown", Preview as unknown as Component),
 };
-
-export default markdownPlugin;
 export { TOOL_NAME };
 
 export const REGISTRATION: PluginRegistration = {

package/src/plugins/photoLocations/index.ts

@@ -30,8 +30,6 @@ const photoLocationsPlugin: ToolPlugin<PhotoLocationsActionData> = {
   viewComponent: wrapWithScope("photoLocations", View),
 };
 
-export default photoLocationsPlugin;
-
 export const REGISTRATION: PluginRegistration = {
   toolName: META.toolName,
   entry: photoLocationsPlugin,

package/src/plugins/presentCollection/index.ts

@@ -17,8 +17,6 @@ const presentCollectionPlugin: ToolPlugin<PresentCollectionData, PresentCollecti
   viewComponent: wrapWithScope("presentCollection", PresentCollectionView),
   previewComponent: wrapWithScope("presentCollection", PresentCollectionPreview),
 };
-
-export default presentCollectionPlugin;
 export { TOOL_NAME };
 
 export const REGISTRATION: PluginRegistration = {

package/src/plugins/presentForm/index.ts

@@ -24,8 +24,6 @@ const presentFormPlugin: ToolPlugin<FormData, FormData, FormArgs> = {
   viewComponent: wrapWithScope("form", pkg.viewComponent),
   previewComponent: wrapWithScope("form", pkg.previewComponent),
 };
-
-export default presentFormPlugin;
 export { TOOL_NAME };
 
 export const REGISTRATION: PluginRegistration = {

package/src/plugins/presentHtml/index.ts

@@ -51,8 +51,6 @@ const presentHtmlPlugin: ToolPlugin<PresentHtmlData> = {
   viewComponent: wrapWithScope("html", View as unknown as Component),
   previewComponent: wrapWithScope("html", Preview as unknown as Component),
 };
-
-export default presentHtmlPlugin;
 export { TOOL_NAME };
 
 export const REGISTRATION: PluginRegistration = {

package/src/plugins/presentMulmoScript/definition.ts

@@ -1,122 +1,17 @@
-import type { ToolDefinition } from "gui-chat-protocol";
 import { META } from "./meta";
 import type { ResolvedRoute } from "../meta-types";
+import { TOOL_DEFINITION } from "@mulmoclaude/mulmoscript-plugin";
 
+// presentMulmoScript's tool schema and save/reopen/update logic now live in
+// the shared @mulmoclaude/mulmoscript-plugin package (single source of truth,
+// also consumable by MulmoTerminal — plans/feat-mulmoscript-plugin.md). This
+// built-in is a thin host adapter: it keeps MulmoClaude's host-specific
+// routing META and View while sourcing the schema from the package.
 export const TOOL_NAME = META.toolName;
-export type MulmoScriptEndpoints = { readonly [K in keyof typeof META.apiRoutes]: ResolvedRoute };
-
-const toolDefinition: ToolDefinition = {
-  type: "function",
-  name: META.toolName,
-  description: `Save and present a MulmoScript story or presentation as a visual storyboard in the canvas.
-
-Two modes — provide EXACTLY ONE of \`script\` or \`filePath\`:
-
-1. **Create new** — pass \`script\` (full MulmoScript JSON). Server saves it to disk and presents it.
-2. **Re-display existing** — pass \`filePath\` (workspace-relative path returned by a previous call, e.g. "stories/my-story-1700000000000.json"). Much cheaper than re-sending the full script. Use whenever the user wants to revisit a presentation that was already created in this workspace.
-
-Optional \`autoGenerateMovie: true\` kicks off movie generation in the background, so the final video is ready by the time the user opens the canvas. Movie generation is expensive (multiple image + audio API calls + video encoding) — only set this when the user has explicitly asked for the movie. Default \`false\`.
 
-Provider rules for new scripts:
-- \`speechParams.speakers.<name>.provider\`: \`"gemini"\` — pairs with Gemini voices like \`"Kore"\`, \`"Aoede"\`, \`"Puck"\`. Do NOT use \`"google"\` here — that routes to Google Cloud TTS, where Gemini-class voices fail with "This voice requires a model name to be specified." unless an explicit \`model\` is set.
-- \`imageParams.provider\`: \`"google"\`
-- \`movieParams.provider\`: \`"google"\`
-- Do NOT add a top-level \`provider\` field to \`speechParams\` — provider belongs per-speaker only.
-
-Required structure:
-
-{
-  "$mulmocast": { "version": "1.1" },
-  "title": "The Life of a Star",
-  "description": "A short educational explainer about stellar evolution",
-  "lang": "en",
-  "speechParams": {
-    "speakers": {
-      "Presenter": {
-        "provider": "gemini",
-        "voiceId": "Kore",
-        "displayName": { "en": "Presenter" }
-      }
-    }
-  },
-  "imageParams": { "provider": "google", "model": "gemini-3.1-flash-image-preview" },
-  "movieParams": { "provider": "google", "model": "veo-3.1-generate" },
-  "beats": [
-    {
-      "speaker": "Presenter",
-      "text": "Narration spoken aloud for this beat.",
-      "imagePrompt": "Detailed description — AI generates the image"
-    },
-    {
-      "speaker": "Presenter",
-      "text": "Bullet point beat.",
-      "image": { "type": "textSlide", "slide": { "title": "Slide Title", "bullets": ["Point one", "Point two"] } }
-    },
-    {
-      "speaker": "Presenter",
-      "text": "Markdown beat.",
-      "image": { "type": "markdown", "markdown": "## Heading\\n\\nBody text here." }
-    },
-    {
-      "speaker": "Presenter",
-      "text": "Chart beat — use for data, comparisons, trends.",
-      "image": { "type": "chart", "title": "Chart Title", "chartData": { "type": "bar", "data": { "labels": ["A", "B", "C"], "datasets": [{ "label": "Series", "data": [10, 20, 30] }] } } }
-    },
-    {
-      "speaker": "Presenter",
-      "text": "Diagram beat — use for flows, architectures, relationships.",
-      "image": { "type": "mermaid", "title": "Diagram Title", "code": { "kind": "text", "text": "graph TD\\n  A[Start] --> B[Process] --> C[End]" } }
-    },
-    {
-      "speaker": "Presenter",
-      "text": "Rich interactive beat — use for custom layouts, animations, or anything that benefits from HTML/CSS.",
-      "image": { "type": "html_tailwind", "html": "<div class=\\"flex items-center justify-center h-full text-4xl font-bold text-blue-600\\">Hello World</div>" }
-    },
-    {
-      "speaker": "Presenter",
-      "text": "AI video beat.",
-      "moviePrompt": "Detailed description — AI generates the video clip"
-    }
-  ]
-}
-
-Beat visual options (choose one per beat):
-- "imagePrompt": "..."  → top-level string field — AI generates an image from the prompt
-- "moviePrompt": "..."  → top-level string field — AI generates a video clip from the prompt
-- "image": { "type": "textSlide", "slide": { "title", "subtitle"?, "bullets"? } }
-- "image": { "type": "markdown", "markdown": "..." }
-- "image": { "type": "chart", "title": "...", "chartData": { "type": "bar"|"line"|"pie"|..., "data": { "labels": [...], "datasets": [...] } } }  ← PREFER for data/numbers/comparisons. chartData is a full Chart.js config: labels/datasets go under "data", not at the top level.
-- "image": { "type": "mermaid", "title": "...", "code": { "kind": "text", "text": "..." } }  ← PREFER for flows/diagrams/relationships
-- "image": { "type": "html_tailwind", "html": "...", "script"?: "..." }  ← PREFER for rich layouts, animations, custom visuals
-
-IMPORTANT: "imagePrompt" and "moviePrompt" are plain string fields on the beat, NOT nested under "image".`,
-  parameters: {
-    type: "object",
-    properties: {
-      script: {
-        type: "object",
-        description:
-          "Complete MulmoScript JSON for a NEW presentation. Must include $mulmocast, speechParams, imageParams, movieParams, and beats array. Always populate the top-level 'description' field with a concise 1–2 sentence summary of the presentation. Do NOT pass alongside `filePath`.",
-        additionalProperties: true,
-      },
-      filename: {
-        type: "string",
-        description:
-          "Optional filename without extension. Defaults to a slug of the script title. Only meaningful with `script`; ignored when `filePath` is given.",
-      },
-      filePath: {
-        type: "string",
-        description:
-          "Workspace-relative path to an EXISTING MulmoScript JSON file (e.g. 'stories/my-story-1700000000000.json'). Use this to re-display a script previously saved in this workspace, instead of resending the full JSON. Do NOT pass alongside `script`.",
-      },
-      autoGenerateMovie: {
-        type: "boolean",
-        description:
-          "When true, the server starts movie generation in the background after save/load. The user does NOT need to open the canvas — progress streams via the existing session channel. Default false. Only set true when the user has explicitly asked for the movie; generation is expensive.",
-      },
-    },
-    required: [],
-  },
-};
+/** Resolved-URL view of the mulmoScript plugin's routes. Plugin code reads
+ *  `endpoints.<route>.{method, url}` to drive `apiCall`. Auto-derived from META. */
+export type MulmoScriptEndpoints = { readonly [K in keyof typeof META.apiRoutes]: ResolvedRoute };
 
-export default toolDefinition;
+export { TOOL_DEFINITION };
+export default TOOL_DEFINITION;

package/src/plugins/presentMulmoScript/index.ts

@@ -1,21 +1,73 @@
+// MulmoClaude's thin built-in adapter for the shared mulmoscript plugin
+// (plans/feat-mulmoscript-plugin.md phase 2). View / Preview /
+// TOOL_DEFINITION come from @mulmoclaude/mulmoscript-plugin; the View
+// reaches host backends via useRuntime().dispatch → the built-in
+// "mulmoScript" dispatch handler (server/plugins/mulmoscript-server.ts)
+// and hears generation progress on the plugin pubsub channel. This adapter
+// keeps MulmoClaude's existing tool-call create path (POST
+// /api/mulmoScript/save) and injects the two host-transport capabilities
+// the generic runtime can't carry: the active chat session id (sidebar
+// generation indicator) and the bearer-authenticated media download.
+import { computed, defineComponent, h, markRaw, provide, type Component } from "vue";
 import type { PluginRegistration, ToolPlugin } from "../../tools/types";
 import type { ToolResult } from "gui-chat-protocol";
-import type { MulmoScript } from "mulmocast";
+import { View, Preview, MULMOSCRIPT_HOST_ADAPTER_KEY, type MulmoScriptData, type MulmoScriptHostAdapter } from "@mulmoclaude/mulmoscript-plugin/vue";
+// Lib mode doesn't auto-inject the package's compiled styles; the consumer
+// must import them — same as @mulmoclaude/{markdown,form,chart,html}-plugin.
+import "@mulmoclaude/mulmoscript-plugin/style.css";
 import toolDefinition, { TOOL_NAME, type MulmoScriptEndpoints } from "./definition";
 import { pluginEndpoints } from "../api";
 import { wrapWithScope } from "../scope";
-import View from "./View.vue";
-import Preview from "./Preview.vue";
-import { apiCall } from "../../utils/api";
+import { apiCall, apiFetchRaw } from "../../utils/api";
 import { makeUuid } from "../../utils/id";
+import { useActiveSession } from "../../composables/useActiveSession";
 
-export interface MulmoScriptData {
-  script: MulmoScript;
-  filePath: string;
+// Re-exported from the shared package so anything importing the result-data
+// shape from "./index" keeps working while the type stays single-sourced.
+export type { MulmoScriptData };
+
+// Bearer-authenticated media download over the host's existing
+// download routes. A plain <video src> / <a href download> can't attach
+// the Authorization header, and the routes stay behind the standard
+// /api/* bearer guard by explicit review decision — so the package View
+// fetches bytes through this injected capability instead.
+async function fetchMediaBlob(query: { moviePath?: string; pdfPath?: string }): Promise<Blob> {
+  const endpoints = pluginEndpoints<MulmoScriptEndpoints>("mulmoScript");
+  const target = query.pdfPath
+    ? { url: endpoints.downloadPdf.url, query: { pdfPath: query.pdfPath } }
+    : { url: endpoints.downloadMovie.url, query: { moviePath: query.moviePath ?? "" } };
+  const res = await apiFetchRaw(target.url, { method: "GET", query: target.query });
+  if (!res.ok) {
+    throw new Error(`HTTP ${res.status}`);
+  }
+  return res.blob();
+}
+
+/** Provide the package's host-adapter injection around the View —
+ *  mounted INSIDE wrapWithScope's PluginScopedRoot, forwarding every
+ *  prop / attr / slot through verbatim (same shape as wrapWithScope). */
+function withHostAdapter<TInner extends Component>(inner: TInner): TInner {
+  return markRaw(
+    defineComponent({
+      name: "MulmoScriptHostAdapter",
+      inheritAttrs: false,
+      setup(_props, { attrs, slots }) {
+        const activeSessionRef = useActiveSession();
+        const adapter: MulmoScriptHostAdapter = {
+          chatSessionId: computed(() => activeSessionRef?.value?.id),
+          fetchMediaBlob,
+        };
+        provide(MULMOSCRIPT_HOST_ADAPTER_KEY, adapter);
+        return () => h(inner, attrs, slots);
+      },
+    }),
+  ) as unknown as TInner;
 }
 
 const presentMulmoScriptPlugin: ToolPlugin<MulmoScriptData> = {
-  toolDefinition,
+  // gui-chat-protocol type is externalized but yarn-4's dual-@vue can
+  // make the package's nominal types distinct; coerce once here.
+  toolDefinition: toolDefinition as ToolPlugin<MulmoScriptData>["toolDefinition"],
 
   // Pass-through: the agent (MCP) and GUI dispatcher both end up at the
   // same backend route, which dispatches between create-new (`script`)
@@ -42,12 +94,10 @@ const presentMulmoScriptPlugin: ToolPlugin<MulmoScriptData> = {
 
   isEnabled: () => true,
   generatingMessage: "Generating MulmoScript storyboard…",
-  viewComponent: wrapWithScope("mulmoScript", View),
-  previewComponent: wrapWithScope("mulmoScript", Preview),
+  viewComponent: wrapWithScope("mulmoScript", withHostAdapter(View as unknown as Component)),
+  previewComponent: wrapWithScope("mulmoScript", Preview as unknown as Component),
 };
 
-export default presentMulmoScriptPlugin;
-
 export const REGISTRATION: PluginRegistration = {
   toolName: TOOL_NAME,
   entry: presentMulmoScriptPlugin,

package/src/plugins/presentSVG/index.ts

@@ -39,8 +39,6 @@ const presentSvgPlugin: ToolPlugin<PresentSvgData> = {
   viewComponent: wrapWithScope("svg", View),
   previewComponent: wrapWithScope("svg", Preview),
 };
-
-export default presentSvgPlugin;
 export { TOOL_NAME };
 
 export const REGISTRATION: PluginRegistration = {

package/src/plugins/skill/index.ts

@@ -27,9 +27,6 @@ export { TOOL_NAME };
 
 export { View, Preview };
 
-const skillPlugin = { plugin };
-export default skillPlugin;
-
 export const REGISTRATION: PluginRegistration = {
   toolName: TOOL_NAME,
   entry: plugin,

package/src/plugins/spreadsheet/engine/parser.ts

@@ -38,106 +38,3 @@ export function indexToColumn(index: number): string {
   }
   return col;
 }
-
-/**
- * Parse a cell reference to its components
- * Supports: A1, $A$1, $A1, A$1, Sheet1!A1, 'My Sheet'!A1
- *
- * @param ref - Cell reference string
- * @returns Parsed cell reference object
- */
-export function parseCellRef(ref: string): CellRef {
-  let cellRef = ref;
-  let sheetName: string | undefined;
-
-  // Check for cross-sheet reference (e.g., 'Sheet Name'!B2 or Sheet1!B2)
-  const sheetMatch = ref.match(/^(?:'([^']+)'|([^!]+))!(.+)$/);
-  if (sheetMatch) {
-    sheetName = sheetMatch[1] || sheetMatch[2]; // Quoted or unquoted sheet name
-    cellRef = sheetMatch[3]; // Cell reference part
-  }
-
-  // Parse absolute references ($A$1)
-  const absoluteRow = cellRef.includes("$") && cellRef.match(/\$\d+/);
-  const absoluteCol = cellRef.includes("$") && cellRef.match(/\$[A-Z]+/);
-
-  // Remove $ symbols
-  const cleanRef = cellRef.replace(/\$/g, "");
-  const match = cleanRef.match(/^([A-Z]+)(\d+)$/);
-
-  if (!match) {
-    throw new Error(`Invalid cell reference: ${ref}`);
-  }
-
-  const col = columnToIndex(match[1]);
-  const row = parseInt(match[2]) - 1; // 1-indexed to 0-indexed
-
-  const result: CellRef = { row, col };
-
-  if (sheetName) {
-    result.sheet = sheetName;
-  }
-
-  if (absoluteRow || absoluteCol) {
-    result.absolute = {
-      row: !!absoluteRow,
-      col: !!absoluteCol,
-    };
-  }
-
-  return result;
-}
-
-/**
- * Parse a range reference to its components
- * Supports: A1:B10, $A$1:$B$10, Sheet1!A1:B10
- *
- * @param range - Range reference string
- * @returns Parsed range reference object
- */
-export function parseRangeRef(range: string): RangeRef {
-  // Use non-greedy match to improve performance
-  const colonIndex = range.lastIndexOf(":");
-  if (colonIndex === -1) {
-    throw new Error(`Invalid range reference: ${range}`);
-  }
-
-  const start = parseCellRef(range.substring(0, colonIndex));
-  const end = parseCellRef(range.substring(colonIndex + 1));
-
-  return { start, end };
-}
-
-/**
- * Convert a cell reference object back to A1 notation
- *
- * @param ref - Cell reference object
- * @returns A1 notation string
- */
-export function cellRefToA1(ref: CellRef): string {
-  const col = indexToColumn(ref.col);
-  const row = ref.row + 1; // 0-based to 1-based
-
-  let result = "";
-
-  if (ref.absolute?.col) {
-    result += "$";
-  }
-  result += col;
-
-  if (ref.absolute?.row) {
-    result += "$";
-  }
-  result += row;
-
-  if (ref.sheet) {
-    // Quote sheet name if it contains spaces
-    if (ref.sheet.includes(" ")) {
-      result = `'${ref.sheet}'!${result}`;
-    } else {
-      result = `${ref.sheet}!${result}`;
-    }
-  }
-
-  return result;
-}

package/src/plugins/spreadsheet/index.ts

@@ -34,8 +34,6 @@ const spreadsheetPlugin: ToolPlugin<SpreadsheetToolData> = {
   viewComponent: wrapWithScope("spreadsheet", View),
   previewComponent: wrapWithScope("spreadsheet", Preview),
 };
-
-export default spreadsheetPlugin;
 export { TOOL_NAME };
 
 export const REGISTRATION: PluginRegistration = {

package/src/plugins/textResponse/index.ts

@@ -26,9 +26,6 @@ export { samples } from "./samples";
 
 export { View, Preview };
 
-const textResponsePlugin = { plugin };
-export default textResponsePlugin;
-
 export const REGISTRATION: PluginRegistration = {
   toolName: TOOL_NAME,
   entry: plugin,

package/src/plugins/wiki/index.ts

@@ -63,8 +63,6 @@ const wikiPlugin: PluginEntry = {
   previewComponent: wrapWithScope("wiki", Preview),
 };
 
-export default wikiPlugin;
-
 export const REGISTRATION: PluginRegistration = {
   toolName: TOOL_NAME,
   entry: wikiPlugin,

package/src/types/notification.ts

@@ -25,16 +25,6 @@ export const NOTIFICATION_KINDS = {
 
 export type NotificationKind = (typeof NOTIFICATION_KINDS)[keyof typeof NOTIFICATION_KINDS];
 
-export const NOTIFICATION_ICONS: Record<NotificationKind, string> = {
-  todo: "check_circle",
-  scheduler: "event",
-  agent: "smart_toy",
-  journal: "auto_stories",
-  push: "notifications",
-  bridge: "chat",
-  system: "warning",
-};
-
 export const NOTIFICATION_ACTION_TYPES = {
   navigate: "navigate",
   none: "none",

package/src/utils/api.ts

@@ -226,10 +226,6 @@ export function apiPut<T = unknown>(path: string, body?: unknown, extra: Omit<Ap
   return apiCall<T>(path, { ...extra, method: "PUT", body });
 }
 
-export function apiPatch<T = unknown>(path: string, body?: unknown, extra: Omit<ApiOptions, "method" | "body"> = {}): Promise<ApiResult<T>> {
-  return apiCall<T>(path, { ...extra, method: "PATCH", body });
-}
-
 export function apiDelete<T = unknown>(path: string, body?: unknown, extra: Omit<ApiOptions, "method" | "body"> = {}): Promise<ApiResult<T>> {
   return apiCall<T>(path, { ...extra, method: "DELETE", body });
 }