@visionengine/remotion-file-bridge 1.0.1 → 1.0.2

package/dist/server.js

@@ -1,3 +1,4 @@
+import * as path from 'node:path';
 import { FastMCP } from 'fastmcp';
 import { z } from 'zod';
 import { deleteProjectFile, downloadProjectFile, listProjectFiles, readProjectFile, saveProjectFile } from './client.js';
@@ -44,21 +45,91 @@ server.addTool({
     }),
     execute: async (args) => JSON.stringify(await downloadProjectFile(args), null, 2),
 });
+const saveFileParameters = z.object({
+    userId: z.string().optional(),
+    sourceType: z.enum(['local_path', 'text', 'base64', 'remote_url']).describe('Choose local_path for local files. Choose text/base64/remote_url when content is provided in content field.'),
+    localPath: z.string().trim().min(1).optional().describe('Required when sourceType=local_path. Can be an absolute path or a path relative to WORKDIR.'),
+    content: z.string().optional().describe('Required for sourceType=text, base64, or remote_url. For remote_url, put URL in this field.'),
+    fileName: z.string().min(1).describe('Final file name in remote workspace, including extension, e.g. baidu_ai.png.'),
+    targetPath: z.string().optional().describe('Target directory path only, workspace-relative, e.g. public/images/photos. Do not include fileName.'),
+    contentType: z.string().optional(),
+    overwrite: z.boolean().optional(),
+}).superRefine((value, ctx) => {
+    if (value.sourceType === 'local_path' && !value.localPath) {
+        ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            path: ['localPath'],
+            message: 'localPath is required when sourceType=local_path',
+        });
+    }
+    if (value.sourceType !== 'local_path' && typeof value.content !== 'string') {
+        ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            path: ['content'],
+            message: `content is required when sourceType=${value.sourceType}`,
+        });
+    }
+});
+function normalizeTargetDirectory(targetPath, fileName) {
+    if (!targetPath?.trim()) {
+        return { targetPath };
+    }
+    const normalized = targetPath.trim().replaceAll('\\', '/').replace(/^\/+/, '').replace(/\/+/g, '/');
+    const parts = normalized.split('/').filter(Boolean);
+    if (parts.length === 0) {
+        return { targetPath: undefined };
+    }
+    const last = parts[parts.length - 1];
+    const normalizedFileName = path.posix.basename(fileName).toLowerCase();
+    const hasSameFileNameAsTail = last.toLowerCase() === normalizedFileName;
+    if (hasSameFileNameAsTail) {
+        parts.pop();
+    }
+    const normalizedTargetPath = parts.join('/') || undefined;
+    const looksLikeFilePath = /\.[^/.]+$/.test(last);
+    if (hasSameFileNameAsTail || looksLikeFilePath) {
+        return {
+            targetPath: normalizedTargetPath,
+            warning: {
+                code: 'target_path_should_be_directory',
+                message: 'targetPath should be a directory path and should not include fileName. The value was normalized before saving.',
+                source: 'save_file',
+                details: {
+                    fileName,
+                    originalTargetPath: targetPath,
+                    normalizedTargetPath,
+                },
+            },
+        };
+    }
+    return { targetPath: normalizedTargetPath };
+}
 server.addTool({
     name: 'save_file',
-    description: 'Use only when the user explicitly wants to save or upload content into the remote Remotion/VEC cloud workspace. Saves local files, text, base64 payloads, or remote URLs into the current user remote workspace.',
+    description: 'Use only when the user explicitly wants to save or upload content into the remote Remotion/VEC cloud workspace. Saves local files, text, base64 payloads, or remote URLs into the current user remote workspace. Always provide localPath when sourceType=local_path. targetPath should be a directory, and fileName should contain the final file name.',
     annotations: { title: 'Save Remotion Project File', readOnlyHint: false, openWorldHint: true },
-    parameters: z.object({
-        userId: z.string().optional(),
-        sourceType: z.enum(['local_path', 'text', 'base64', 'remote_url']),
-        localPath: z.string().optional(),
-        content: z.string().optional(),
-        fileName: z.string().min(1),
-        targetPath: z.string().optional(),
-        contentType: z.string().optional(),
-        overwrite: z.boolean().optional(),
-    }),
-    execute: async (args) => JSON.stringify(await saveProjectFile(args), null, 2),
+    parameters: saveFileParameters,
+    execute: async (args) => {
+        const normalized = normalizeTargetDirectory(args.targetPath, args.fileName);
+        const normalizedArgs = {
+            ...args,
+            targetPath: normalized.targetPath,
+        };
+        const result = await saveProjectFile(normalizedArgs);
+        if (!normalized.warning) {
+            return JSON.stringify(result, null, 2);
+        }
+        const metadata = result.metadata && typeof result.metadata === 'object'
+            ? result.metadata
+            : {};
+        return JSON.stringify({
+            ...result,
+            metadata: {
+                ...metadata,
+                warnings: [...(Array.isArray(metadata.warnings) ? metadata.warnings : []), normalized.warning],
+            },
+        }, null, 2);
+    },
 });
 server.addTool({
     name: 'delete_file',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@visionengine/remotion-file-bridge",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "VisionEngine Remotion File Bridge MCP Server",
   "main": "dist/index.js",
   "type": "module",