@lalalic/markcut 3.2.7 → 3.2.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lalalic/markcut",
-  "version": "3.2.7",
+  "version": "3.2.9",
   "description": "Markdown-to-video engine. Describe scenes in markdown, get a rendered video.",
   "bin": {
     "markcut": "bin/markcut"

package/skills/markcut/SKILL.md

@@ -78,10 +78,20 @@ npx @lalalic/markcut spots --waypoints "lat,lng;lat,lng" # discover POIs along a
 
 ---
 
+## MCP Markdown Review
+
+Use the model-visible MCP tool `markcut.preview({ path })` when a human needs to review a Markdown artifact. This is a generic Markdown review surface and is separate from the video-specific `npx @lalalic/markcut preview <file>` CLI.
+
+- `path` must be an absolute path to a Markdown file.
+- The user sees the rendered Markdown and can either **Approve** or submit feedback with **Request changes**.
+- Markcut does not edit the file. The calling agent owns all source changes.
+- On `approved`, continue the workflow.
+- On `changes_requested`, apply the feedback to the source Markdown, then call `markcut.preview({ path })` again for another review round.
+
 ## review
 use the review contract defined in [./review.md](./review.md) to guide the review process.
 review as early as possible in the video production process to catch issues before they propagate.
-* review md file
+* review md file with `markcut.preview({ path })` when the MCP tool is available
 * review compiled.json
 * review the rendered video
 

package/src/mcp/bridge.mjs

@@ -7,7 +7,7 @@ function send(message){window.parent.postMessage(message,'*');}
 function rpcRequest(method,params){const id=++nextRequestId;send({jsonrpc:'2.0',id,method,params});return new Promise((resolve,reject)=>pendingRequests.set(id,{resolve,reject}));}
 function applyToolResult(params){const result=params?.result||params||{},data=result.structuredContent||result.structured_content||{};if(typeof data.path==='string')pathLabel.textContent=data.path;if(typeof data.markdown==='string'){article.innerHTML=data.markdown.split(/\\r?\\n/).map(line=>{const escaped=line.replace(/[&<>"']/g,c=>({'&':'&amp;','<':'&lt;','>':'&gt;','"':'&quot;',"'":'&#39;'}[c]));if(line.startsWith('### '))return '<h3>'+escaped.slice(4)+'</h3>';if(line.startsWith('## '))return '<h2>'+escaped.slice(3)+'</h2>';if(line.startsWith('# '))return '<h1>'+escaped.slice(2)+'</h1>';if(line.startsWith('- '))return '<li>'+escaped.slice(2)+'</li>';return line.trim()?'<p>'+escaped+'</p>':''}).join('\\n');}}
 window.addEventListener('message',event=>{if(event.source!==window.parent)return;const message=event.data;if(!message||typeof message!=='object'||message.jsonrpc!=='2.0')return;if(Object.prototype.hasOwnProperty.call(message,'id')&&pendingRequests.has(message.id)){const pending=pendingRequests.get(message.id);pendingRequests.delete(message.id);if(message.error)pending.reject(new Error(message.error.message||'MCP request failed'));else pending.resolve(message.result);return;}if(message.method==='ui/notifications/tool-result')applyToolResult(message.params);});
-async function initializeBridge(){try{await rpcRequest('ui/initialize',{protocolVersion:'2026-01-26',capabilities:{},clientInfo:{name:'markcut-preview',version:'1'}});send({jsonrpc:'2.0',method:'ui/notifications/initialized'});status.textContent='Ready for review.';}catch(error){status.textContent=error.message||'MCP initialization failed.';}}
+async function initializeBridge(){try{await rpcRequest('ui/initialize',{appInfo:{name:'Markcut Preview',version:'1.0.0'},appCapabilities:{},protocolVersion:'2026-01-26'});send({jsonrpc:'2.0',method:'ui/notifications/initialized'});status.textContent='Ready for review.';}catch(error){status.textContent=error.message||'MCP initialization failed.';}}
 async function submit(decision){const value=feedback.value.trim();if(decision==='changes_requested'&&!value){status.textContent='Feedback is required.';feedback.focus();return;}const result={decision};if(value&&decision==='changes_requested')result.feedback=value;status.textContent='Submitting...';try{await rpcRequest('tools/call',{name:'${submitToolName}',arguments:result});send({jsonrpc:'2.0',method:'ui/message',params:{role:'user',content:[{type:'text',text:JSON.stringify(result)}]}});status.textContent=decision==='approved'?'Approved.':'Changes requested.';}catch(error){status.textContent=error.message||'Unable to submit review.';}}
 initializeBridge();
 document.getElementById('approve').onclick=()=>submit('approved');document.getElementById('changes').onclick=()=>submit('changes_requested');

package/tests/mcp-preview.test.ts

@@ -53,8 +53,26 @@ describe("MarkCut MCP Markdown preview", () => {
   it("performs the MCP Apps handshake and awaits a successful submit", async () => {
     const harness = createBridgeHarness();
     harness.run();
-    expect(harness.messages[0]).toMatchObject({ id: 1, method: "ui/initialize" });
-    harness.respond({ id: 1, result: { protocolVersion: "2026-01-26" } });
+    expect(harness.messages[0]).toEqual({
+      jsonrpc: "2.0",
+      id: 1,
+      method: "ui/initialize",
+      params: {
+        appInfo: { name: "Markcut Preview", version: "1.0.0" },
+        appCapabilities: {},
+        protocolVersion: "2026-01-26",
+      },
+    });
+    expect(harness.messages.some((message) => message.method === "ui/notifications/initialized")).toBe(false);
+    harness.respond({
+      id: 1,
+      result: {
+        protocolVersion: "2026-01-26",
+        hostInfo: { name: "test-host", version: "1" },
+        hostCapabilities: {},
+        hostContext: {},
+      },
+    });
     await tick();
     expect(harness.messages[1]).toMatchObject({ method: "ui/notifications/initialized" });