Fix MCP server API migration and improve logging

- Update MCP server to use registerTool() API with new schema format
  (fixes compatibility with @modelcontextprotocol/sdk ^1.9.0)
- Fix Kokoro buffer handling to properly return ArrayBuffer
  (ensures type consistency in audio generation)
- Improve FFmpeg logging with Pino structured format
  (aligns with codebase logging conventions)

Author: justintanner

src/server/routers/mcp.ts

@@ -5,7 +5,12 @@ import z from "zod";
 
 import { ShortCreator } from "../../short-creator/ShortCreator";
 import { logger } from "../../logger";
-import { renderConfig, sceneInput } from "../../types/shorts";
+import {
+  renderConfig,
+  sceneInput,
+  type RenderConfig,
+  type SceneInput,
+} from "../../types/shorts";
 
 export class MCPRouter {
   router: express.Router;
@@ -19,50 +24,52 @@ export class MCPRouter {
     this.mcpServer = new McpServer({
       name: "Short Creator",
       version: "0.0.1",
-      capabilities: {
-        resources: {},
-        tools: {},
-      },
     });
 
     this.setupMCPServer();
     this.setupRoutes();
   }
 
   private setupMCPServer() {
-    this.mcpServer.tool(
+    this.mcpServer.registerTool(
       "get-video-status",
-      "Get the status of a video (ready, processing, failed)",
       {
-        videoId: z.string().describe("The ID of the video"),
+        description: "Get the status of a video (ready, processing, failed)",
+        inputSchema: z.object({
+          videoId: z.string().describe("The ID of the video"),
+        }) as any,
       },
-      async ({ videoId }) => {
+      async (args: any) => {
+        const { videoId } = args;
         const status = this.shortCreator.status(videoId);
         return {
           content: [
             {
-              type: "text",
+              type: "text" as const,
               text: status,
             },
           ],
         };
       },
     );
 
-    this.mcpServer.tool(
+    this.mcpServer.registerTool(
       "create-short-video",
-      "Create a short video from a list of scenes",
       {
-        scenes: z.array(sceneInput).describe("Each scene to be created"),
-        config: renderConfig.describe("Configuration for rendering the video"),
+        description: "Create a short video from a list of scenes",
+        inputSchema: z.object({
+          scenes: z.array(sceneInput),
+          config: renderConfig,
+        }) as any,
       },
-      async ({ scenes, config }) => {
+      async (args: any) => {
+        const { scenes, config } = args;
         const videoId = await this.shortCreator.addToQueue(scenes, config);
 
         return {
           content: [
             {
-              type: "text",
+              type: "text" as const,
               text: videoId,
             },
           ],

src/short-creator/libraries/FFmpeg.ts

@@ -6,7 +6,7 @@ export class FFMpeg {
   static async init(): Promise<FFMpeg> {
     return import("@ffmpeg-installer/ffmpeg").then((ffmpegInstaller) => {
       ffmpeg.setFfmpegPath(ffmpegInstaller.path);
-      logger.info("FFmpeg path set to:", ffmpegInstaller.path);
+      logger.info({ path: ffmpegInstaller.path }, "FFmpeg path set to");
       return new FFMpeg();
     });
   }

src/short-creator/libraries/Kokoro.ts

@@ -58,7 +58,8 @@ export class Kokoro {
     header.writeUInt32LE(36 + totalDataLength, 4);
     header.writeUInt32LE(totalDataLength, 40);
 
-    return Buffer.concat([header, ...dataParts]);
+    const result = Buffer.concat([header, ...dataParts]);
+    return result.buffer.slice(result.byteOffset, result.byteOffset + result.byteLength);
   }
 
   static async init(dtype: kokoroModelPrecision): Promise<Kokoro> {