dropbox-mcp-server 2.2.0 → 2.2.1

package/dist/index.js

@@ -5,7 +5,7 @@ import { CallToolRequestSchema, ListToolsRequestSchema } from "@modelcontextprot
 import { Dropbox } from "dropbox";
 import "dotenv/config";
 // ------------------------------------------------------------------
-// 1. Rate Limiter
+// 1. Rate Limiter (Fixed: Removed dangerous Proxy)
 // ------------------------------------------------------------------
 const MAX_CONCURRENT = 1;
 const MIN_GAP_MS = 1000;
@@ -59,17 +59,6 @@ async function limited(fn) {
         release();
     }
 }
-function throttleClient(client) {
-    return new Proxy(client, {
-        get(target, prop, receiver) {
-            const val = Reflect.get(target, prop, receiver);
-            if (typeof val === "function") {
-                return (...args) => limited(() => val.apply(target, args));
-            }
-            return val;
-        },
-    });
-}
 // ------------------------------------------------------------------
 // 2. Error Handling
 // ------------------------------------------------------------------
@@ -95,86 +84,105 @@ function isBadRequestError(err) {
 // ------------------------------------------------------------------
 const DROPBOX_APP_KEY = process.env.DROPBOX_APP_KEY;
 const DROPBOX_REFRESH_TOKEN = process.env.DROPBOX_REFRESH_TOKEN;
-const DROPBOX_SHARED_URL = process.env.DROPBOX_SHARED_URL; // optional
+const DROPBOX_SHARED_URL = process.env.DROPBOX_SHARED_URL;
+const DROPBOX_APP_SECRET = process.env.DROPBOX_APP_SECRET; // Highly recommended for token refresh
 if (!DROPBOX_APP_KEY || !DROPBOX_REFRESH_TOKEN) {
-    console.error("Missing required environment variables:");
-    console.error("  - DROPBOX_APP_KEY");
-    console.error("  - DROPBOX_REFRESH_TOKEN");
-    console.error("Note: DROPBOX_SHARED_URL is optional.");
+    console.error("Missing required environment variables: DROPBOX_APP_KEY, DROPBOX_REFRESH_TOKEN");
     process.exit(1);
 }
-const rawClient = new Dropbox({
+// Instantiate direct client (no proxy)
+const client = new Dropbox({
     clientId: DROPBOX_APP_KEY,
+    clientSecret: DROPBOX_APP_SECRET, // Needed if your Dropbox app is configured with a secret
     refreshToken: DROPBOX_REFRESH_TOKEN,
 });
-const client = throttleClient(rawClient);
-const server = new Server({ name: "dropbox-agent-mcp", version: "2.0.0" }, { capabilities: { tools: {} } });
+const server = new Server({ name: "dropbox-agent-mcp", version: "2.0.1" }, { capabilities: { tools: {} } });
+// ------------------------------------------------------------------
+// 4. Helpers (Fixed Pathing & Node 18+ Blobs)
+// ------------------------------------------------------------------
+import officeParser from 'officeparser';
+import * as fs from 'fs/promises';
+import * as path from 'path';
+import * as os from 'os';
 // ------------------------------------------------------------------
-// 4. Helpers
+// 4. Helpers 
 // ------------------------------------------------------------------
-/** Normalise a path for use with a shared link: strip leading "/" and handle root. */
-function normalizeSharedPath(path) {
-    if (!path || path === "/")
+function formatDropboxPath(pathStr) {
+    if (!pathStr || pathStr === "/" || pathStr.trim() === "")
         return "";
-    return path.replace(/^\/+/, "");
+    return pathStr.startsWith("/") ? pathStr : `/${pathStr}`;
 }
-/** Check if a filename likely belongs to a text file. */
-function isTextFile(pathOrName) {
+function isSupportedFile(pathOrName) {
     const fileName = pathOrName.split('/').pop() || '';
     const ext = fileName.split('.').pop()?.toLowerCase() || '';
-    const textExtensions = [
+    if (fileName === ext)
+        return true;
+    const supportedExtensions = [
         'csv', 'tsv', 'txt', 'json', 'md', 'log',
         'xml', 'yaml', 'yml', 'html', 'htm', 'css',
-        'js', 'ts', 'py', 'java', 'c', 'cpp', 'h'
+        'js', 'ts', 'py', 'java', 'c', 'cpp', 'h',
+        'pdf', 'docx', 'pptx', 'xlsx', 'xls', 'doc', 'ppt', 'odt', 'odp', 'ods'
     ];
-    return textExtensions.includes(ext);
+    return supportedExtensions.includes(ext);
 }
-/**
- * Download a file from Dropbox.
- * - If DROPBOX_SHARED_URL is set, uses shared link APIs with relative paths.
- * - Otherwise, uses the regular filesDownload with absolute paths.
- * Returns UTF‑8 text, or throws if binary.
- */
 async function downloadHelper(identifier) {
     let result;
     let fileName;
-    // If the identifier is a full URL, we always treat it as a shared link.
     if (identifier.startsWith("http://") || identifier.startsWith("https://")) {
-        const response = await client.sharingGetSharedLinkFile({ url: identifier });
+        const response = await limited(() => client.sharingGetSharedLinkFile({ url: identifier }));
         result = response.result;
         fileName = result.name || identifier.split('/').pop();
     }
-    // Otherwise, decide based on DROPBOX_SHARED_URL.
     else if (DROPBOX_SHARED_URL) {
-        // Shared link mode – identifier is a relative path.
-        const path = normalizeSharedPath(identifier);
-        const response = await client.sharingGetSharedLinkFile({
+        const reqPath = formatDropboxPath(identifier);
+        const response = await limited(() => client.sharingGetSharedLinkFile({
             url: DROPBOX_SHARED_URL,
-            path: path,
-        });
+            path: reqPath,
+        }));
         result = response.result;
         fileName = result.name || identifier;
     }
     else {
-        // Regular mode – identifier is an absolute path.
-        const response = await client.filesDownload({ path: identifier });
+        const reqPath = formatDropboxPath(identifier);
+        const response = await limited(() => client.filesDownload({ path: reqPath }));
         result = response.result;
         fileName = result.name || identifier;
     }
-    if (!result.fileBinary) {
+    const fileRef = fileName || identifier;
+    if (!isSupportedFile(fileRef)) {
+        throw new Error(`File "${fileRef}" appears to be an unsupported file format.`);
+    }
+    let fileBuffer;
+    if (result.fileBinary) {
+        fileBuffer = Buffer.from(result.fileBinary);
+    }
+    else if (result.fileBlob) {
+        const arrayBuffer = await result.fileBlob.arrayBuffer();
+        fileBuffer = Buffer.from(arrayBuffer);
+    }
+    else {
         throw new Error("Download succeeded but no file content was returned.");
     }
-    const fileRef = fileName || identifier;
-    if (!isTextFile(fileRef)) {
-        throw new Error(`File "${fileRef}" appears to be a binary file. ` +
-            `This tool only supports text‑based files (CSV, JSON, TXT, etc.).`);
+    const ext = fileRef.split('.').pop()?.toLowerCase() || '';
+    const binaryExtensions = ['pdf', 'docx', 'pptx', 'xlsx', 'xls', 'doc', 'ppt', 'odt', 'odp', 'ods'];
+    if (binaryExtensions.includes(ext)) {
+        const tempFilePath = path.join(os.tmpdir(), `temp_${Date.now()}_${Math.random().toString(36).substring(7)}.${ext}`);
+        try {
+            await fs.writeFile(tempFilePath, fileBuffer);
+            // officeParser is called from the default import
+            const ast = await officeParser.parseOffice(fileBuffer);
+            const result = await ast.to('md');
+            return result.value || "Document parsed, but it appears to be empty.";
+        }
+        catch (error) {
+            throw new Error(`Failed to parse binary document: ${error.message}`);
+        }
+        finally {
+            await fs.unlink(tempFilePath).catch(() => { });
+        }
     }
-    return Buffer.from(result.fileBinary).toString('utf-8');
+    return fileBuffer.toString('utf-8');
 }
-/**
- * Parse a CSV header line, handling quoted fields and trimming.
- * Returns an array of clean column names.
- */
 function parseCsvHeader(headerLine) {
     const fields = [];
     let current = '';
@@ -202,47 +210,28 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
     tools: [
         {
             name: "list_folder",
-            description: DROPBOX_SHARED_URL
-                ? "List folders and files inside the shared Dropbox folder. Path must be relative to the shared root (e.g., '' for root, 'Data' for /Data)."
-                : "List folders and files in a specific Dropbox path (absolute, e.g., '/Data' or '' for root).",
+            description: "List folders and files. Use '' for root, or '/FolderName' for subfolders.",
             inputSchema: {
                 type: "object",
-                properties: {
-                    path: {
-                        type: "string",
-                        description: DROPBOX_SHARED_URL
-                            ? "Relative path inside the shared folder (e.g., '' or 'Data')."
-                            : "Absolute path to list (e.g., '' for root or '/Data')."
-                    }
-                },
+                properties: { path: { type: "string" } },
                 required: ["path"]
             }
         },
         {
             name: "search_files",
-            description: "Search for files by name, extension, or content across the entire Dropbox account.",
+            description: "Search for files by name, extension, or content.",
             inputSchema: {
                 type: "object",
-                properties: {
-                    query: { type: "string", description: "The search query (e.g., 'sales report' or '.csv')" }
-                },
+                properties: { query: { type: "string" } },
                 required: ["query"]
             }
         },
         {
             name: "get_file_metadata",
-            description: DROPBOX_SHARED_URL
-                ? "Retrieve metadata (size, modified date) for a file inside the shared folder. Path is relative to the shared root."
-                : "Retrieve metadata (size, modified date) for a file at an absolute Dropbox path.",
+            description: "Retrieve size/modified date for a file.",
             inputSchema: {
                 type: "object",
-                properties: {
-                    path: {
-                        type: "string", description: DROPBOX_SHARED_URL
-                            ? "Relative path to the file inside the shared folder."
-                            : "Exact Dropbox file path (e.g., '/Data/file.csv')."
-                    }
-                },
+                properties: { path: { type: "string" } },
                 required: ["path"]
             }
         },
@@ -251,64 +240,48 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
             description: "Display the schema (columns/headers) of a CSV or TSV file.",
             inputSchema: {
                 type: "object",
-                properties: {
-                    identifier: {
-                        type: "string", description: DROPBOX_SHARED_URL
-                            ? "Relative path to the table file (e.g., 'data.csv') or a shared link URL."
-                            : "Dropbox path (e.g., '/Data/file.csv') or shared link URL."
-                    }
-                },
+                properties: { identifier: { type: "string" } },
                 required: ["identifier"]
             }
         },
         {
             name: "read_data",
-            description: "Download and read the content of a text file (CSV, JSON, TXT, etc.).",
+            description: "Download and read the content of a text file. Truncated if over 100k chars.",
             inputSchema: {
                 type: "object",
-                properties: {
-                    identifier: {
-                        type: "string", description: DROPBOX_SHARED_URL
-                            ? "Relative path (e.g., 'data.csv') or a shared link URL."
-                            : "Dropbox path (e.g., '/Data/file.csv') or shared link URL."
-                    }
-                },
+                properties: { identifier: { type: "string" } },
                 required: ["identifier"]
             }
         }
     ]
 }));
 // ------------------------------------------------------------------
-// 6. Tool Handlers
+// 6. Tool Handlers (Using direct limited wrapper)
 // ------------------------------------------------------------------
 server.setRequestHandler(CallToolRequestSchema, async (request) => {
     try {
         const name = request.params.name;
         const args = request.params.arguments || {};
-        // --- list_folder ---
         if (name === "list_folder") {
-            let path = args.path || "";
+            const rawPath = args.path || "";
             let requestArgs;
             if (DROPBOX_SHARED_URL) {
-                path = normalizeSharedPath(path);
                 requestArgs = {
-                    path: path,
+                    path: formatDropboxPath(rawPath),
                     shared_link: { url: DROPBOX_SHARED_URL }
                 };
             }
             else {
-                // Absolute path: if "/" then empty string for root
-                requestArgs = { path: path === "/" ? "" : path };
+                requestArgs = { path: formatDropboxPath(rawPath) };
             }
-            const response = await client.filesListFolder(requestArgs);
+            const response = await limited(() => client.filesListFolder(requestArgs));
             const entries = response.result.entries.map((e) => `[${e['.tag'].toUpperCase()}] ${e.name} (Path: ${e.path_display})`);
             return {
                 content: [{ type: "text", text: entries.join("\n") || "Folder is empty." }]
             };
         }
-        // --- search_files ---
         if (name === "search_files") {
-            const response = await client.filesSearchV2({ query: args.query });
+            const response = await limited(() => client.filesSearchV2({ query: args.query }));
             const matches = response.result.matches.map((m) => {
                 const meta = m.metadata.metadata;
                 return `[${meta['.tag'].toUpperCase()}] ${meta.name} (Path: ${meta.path_display})`;
@@ -317,102 +290,46 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 content: [{ type: "text", text: matches.join("\n") || "No matches found." }]
             };
         }
-        // --- get_file_metadata ---
         if (name === "get_file_metadata") {
             let response;
-            const rawPath = args.path || "";
+            const path = formatDropboxPath(args.path || "");
             if (DROPBOX_SHARED_URL) {
-                const path = normalizeSharedPath(rawPath);
-                response = await client.sharingGetSharedLinkMetadata({
+                response = await limited(() => client.sharingGetSharedLinkMetadata({
                     url: DROPBOX_SHARED_URL,
                     path: path
-                });
+                }));
             }
             else {
-                response = await client.filesGetMetadata({ path: rawPath });
+                response = await limited(() => client.filesGetMetadata({ path: path }));
             }
             return {
                 content: [{ type: "text", text: JSON.stringify(response.result, null, 2) }]
             };
         }
-        // --- read_data ---
         if (name === "read_data") {
-            try {
-                const text = await downloadHelper(args.identifier);
-                const truncated = text.length > 100000
-                    ? text.substring(0, 100000) + "\n... (truncated)"
-                    : text;
-                return { content: [{ type: "text", text: truncated }] };
-            }
-            catch (err) {
-                if (isNotFoundError(err)) {
-                    return {
-                        content: [{ type: "text", text: `File not found at "${args.identifier}".` }],
-                        isError: true
-                    };
-                }
-                if (isPermissionError(err)) {
-                    return {
-                        content: [{ type: "text", text: `Permission denied for "${args.identifier}".` }],
-                        isError: true
-                    };
-                }
-                if (isBadRequestError(err)) {
-                    return {
-                        content: [{ type: "text", text: `Invalid request. Check the path/identifier: "${args.identifier}".` }],
-                        isError: true
-                    };
-                }
-                if (err.message && err.message.includes("binary file")) {
-                    return { content: [{ type: "text", text: err.message }], isError: true };
-                }
-                throw err;
-            }
+            const text = await downloadHelper(args.identifier);
+            const truncated = text.length > 100000
+                ? text.substring(0, 100000) + "\n... [WARNING: FILE TRUNCATED TO 100K CHARACTERS]"
+                : text;
+            return { content: [{ type: "text", text: truncated }] };
         }
-        // --- describe_table ---
         if (name === "describe_table") {
-            try {
-                const text = await downloadHelper(args.identifier);
-                const lines = text.split(/\r?\n/).filter(line => line.trim().length > 0);
-                if (lines.length === 0) {
-                    return { content: [{ type: "text", text: "File is empty." }] };
-                }
-                let headers;
-                let totalRows = lines.length - 1;
-                const firstLine = lines[0];
-                const commaCount = (firstLine.match(/,/g) || []).length;
-                if (commaCount > 0) {
-                    headers = parseCsvHeader(firstLine);
-                    if (headers.length < 2) {
-                        headers = lines.map((_, i) => `Column ${i + 1}`);
-                        totalRows = lines.length;
-                    }
-                }
-                else {
-                    headers = lines.map((_, i) => `Column ${i + 1}`);
-                    totalRows = lines.length;
-                }
-                const schemaStr = `Columns detected:\n- ${headers.join('\n- ')}\n\nTotal rows: ${totalRows}`;
-                return { content: [{ type: "text", text: schemaStr }] };
+            // Note: This still downloads the full file to read the first line.
+            // Be careful running this on files >50MB as it may cause timeouts.
+            const text = await downloadHelper(args.identifier);
+            const lines = text.split(/\r?\n/).filter(line => line.trim().length > 0);
+            if (lines.length === 0)
+                return { content: [{ type: "text", text: "File is empty." }] };
+            let headers;
+            const firstLine = lines[0];
+            if ((firstLine.match(/,/g) || []).length > 0) {
+                headers = parseCsvHeader(firstLine);
             }
-            catch (err) {
-                if (isNotFoundError(err)) {
-                    return {
-                        content: [{ type: "text", text: `File not found at "${args.identifier}".` }],
-                        isError: true
-                    };
-                }
-                if (isPermissionError(err)) {
-                    return {
-                        content: [{ type: "text", text: `Permission denied for "${args.identifier}".` }],
-                        isError: true
-                    };
-                }
-                if (err.message && err.message.includes("binary file")) {
-                    return { content: [{ type: "text", text: err.message }], isError: true };
-                }
-                throw err;
+            else {
+                headers = lines.map((_, i) => `Column ${i + 1}`);
             }
+            const schemaStr = `Columns detected:\n- ${headers.join('\n- ')}\n\nTotal rows: ${lines.length - 1}`;
+            return { content: [{ type: "text", text: schemaStr }] };
         }
         throw new Error(`Unknown tool: ${name}`);
     }

package/index.ts

@@ -6,7 +6,7 @@ import { Dropbox } from "dropbox";
 import "dotenv/config";
 
 // ------------------------------------------------------------------
-// 1. Rate Limiter
+// 1. Rate Limiter (Fixed: Removed dangerous Proxy)
 // ------------------------------------------------------------------
 const MAX_CONCURRENT = 1;
 const MIN_GAP_MS = 1000;
@@ -62,18 +62,6 @@ async function limited<T>(fn: () => Promise<T>): Promise<T> {
     }
 }
 
-function throttleClient<T extends object>(client: T): T {
-    return new Proxy(client, {
-        get(target, prop, receiver) {
-            const val = Reflect.get(target, prop, receiver);
-            if (typeof val === "function") {
-                return (...args: any[]) => limited(() => val.apply(target, args));
-            }
-            return val;
-        },
-    });
-}
-
 // ------------------------------------------------------------------
 // 2. Error Handling
 // ------------------------------------------------------------------
@@ -100,101 +88,121 @@ function isBadRequestError(err: any): boolean {
 // ------------------------------------------------------------------
 const DROPBOX_APP_KEY = process.env.DROPBOX_APP_KEY;
 const DROPBOX_REFRESH_TOKEN = process.env.DROPBOX_REFRESH_TOKEN;
-const DROPBOX_SHARED_URL = process.env.DROPBOX_SHARED_URL; // optional
+const DROPBOX_SHARED_URL = process.env.DROPBOX_SHARED_URL;
+const DROPBOX_APP_SECRET = process.env.DROPBOX_APP_SECRET; // Highly recommended for token refresh
 
 if (!DROPBOX_APP_KEY || !DROPBOX_REFRESH_TOKEN) {
-    console.error("Missing required environment variables:");
-    console.error("  - DROPBOX_APP_KEY");
-    console.error("  - DROPBOX_REFRESH_TOKEN");
-    console.error("Note: DROPBOX_SHARED_URL is optional.");
+    console.error("Missing required environment variables: DROPBOX_APP_KEY, DROPBOX_REFRESH_TOKEN");
     process.exit(1);
 }
 
-const rawClient = new Dropbox({
+// Instantiate direct client (no proxy)
+const client = new Dropbox({
     clientId: DROPBOX_APP_KEY,
+    clientSecret: DROPBOX_APP_SECRET, // Needed if your Dropbox app is configured with a secret
     refreshToken: DROPBOX_REFRESH_TOKEN,
 });
-const client = throttleClient(rawClient);
 
 const server = new Server(
-    { name: "dropbox-agent-mcp", version: "2.0.0" },
+    { name: "dropbox-agent-mcp", version: "2.0.1" },
     { capabilities: { tools: {} } }
 );
 
 // ------------------------------------------------------------------
-// 4. Helpers
+// 4. Helpers (Fixed Pathing & Node 18+ Blobs)
 // ------------------------------------------------------------------
 
-/** Normalise a path for use with a shared link: strip leading "/" and handle root. */
-function normalizeSharedPath(path: string): string {
-    if (!path || path === "/") return "";
-    return path.replace(/^\/+/, "");
+import officeParser from 'officeparser';
+import * as fs from 'fs/promises';
+import * as path from 'path';
+import * as os from 'os';
+
+// ------------------------------------------------------------------
+// 4. Helpers 
+// ------------------------------------------------------------------
+
+function formatDropboxPath(pathStr: string): string {
+    if (!pathStr || pathStr === "/" || pathStr.trim() === "") return "";
+    return pathStr.startsWith("/") ? pathStr : `/${pathStr}`;
 }
 
-/** Check if a filename likely belongs to a text file. */
-function isTextFile(pathOrName: string): boolean {
+function isSupportedFile(pathOrName: string): boolean {
     const fileName = pathOrName.split('/').pop() || '';
     const ext = fileName.split('.').pop()?.toLowerCase() || '';
-    const textExtensions = [
+    if (fileName === ext) return true;
+
+    const supportedExtensions = [
         'csv', 'tsv', 'txt', 'json', 'md', 'log',
         'xml', 'yaml', 'yml', 'html', 'htm', 'css',
-        'js', 'ts', 'py', 'java', 'c', 'cpp', 'h'
+        'js', 'ts', 'py', 'java', 'c', 'cpp', 'h',
+        'pdf', 'docx', 'pptx', 'xlsx', 'xls', 'doc', 'ppt', 'odt', 'odp', 'ods'
     ];
-    return textExtensions.includes(ext);
+    return supportedExtensions.includes(ext);
 }
 
-/**
- * Download a file from Dropbox.
- * - If DROPBOX_SHARED_URL is set, uses shared link APIs with relative paths.
- * - Otherwise, uses the regular filesDownload with absolute paths.
- * Returns UTF‑8 text, or throws if binary.
- */
 async function downloadHelper(identifier: string): Promise<string> {
     let result: any;
     let fileName: string | undefined;
 
-    // If the identifier is a full URL, we always treat it as a shared link.
     if (identifier.startsWith("http://") || identifier.startsWith("https://")) {
-        const response = await client.sharingGetSharedLinkFile({ url: identifier });
+        const response = await limited(() => client.sharingGetSharedLinkFile({ url: identifier }));
         result = response.result;
         fileName = result.name || identifier.split('/').pop();
-    }
-    // Otherwise, decide based on DROPBOX_SHARED_URL.
-    else if (DROPBOX_SHARED_URL) {
-        // Shared link mode – identifier is a relative path.
-        const path = normalizeSharedPath(identifier);
-        const response = await client.sharingGetSharedLinkFile({
+    } else if (DROPBOX_SHARED_URL) {
+        const reqPath = formatDropboxPath(identifier);
+        const response = await limited(() => client.sharingGetSharedLinkFile({
             url: DROPBOX_SHARED_URL,
-            path: path,
-        });
+            path: reqPath,
+        }));
         result = response.result;
         fileName = result.name || identifier;
     } else {
-        // Regular mode – identifier is an absolute path.
-        const response = await client.filesDownload({ path: identifier });
+        const reqPath = formatDropboxPath(identifier);
+        const response = await limited(() => client.filesDownload({ path: reqPath }));
         result = response.result;
         fileName = result.name || identifier;
     }
 
-    if (!result.fileBinary) {
+    const fileRef = fileName || identifier;
+    if (!isSupportedFile(fileRef)) {
+        throw new Error(`File "${fileRef}" appears to be an unsupported file format.`);
+    }
+
+    let fileBuffer: Buffer;
+    if (result.fileBinary) {
+        fileBuffer = Buffer.from(result.fileBinary);
+    } else if (result.fileBlob) {
+        const arrayBuffer = await result.fileBlob.arrayBuffer();
+        fileBuffer = Buffer.from(arrayBuffer);
+    } else {
         throw new Error("Download succeeded but no file content was returned.");
     }
 
-    const fileRef = fileName || identifier;
-    if (!isTextFile(fileRef)) {
-        throw new Error(
-            `File "${fileRef}" appears to be a binary file. ` +
-            `This tool only supports text‑based files (CSV, JSON, TXT, etc.).`
+    const ext = fileRef.split('.').pop()?.toLowerCase() || '';
+    const binaryExtensions = ['pdf', 'docx', 'pptx', 'xlsx', 'xls', 'doc', 'ppt', 'odt', 'odp', 'ods'];
+
+    if (binaryExtensions.includes(ext)) {
+        const tempFilePath = path.join(
+            os.tmpdir(),
+            `temp_${Date.now()}_${Math.random().toString(36).substring(7)}.${ext}`
         );
+
+        try {
+            await fs.writeFile(tempFilePath, fileBuffer);
+            // officeParser is called from the default import
+            const ast = await officeParser.parseOffice(fileBuffer);
+
+            const result = await ast.to('md');
+            return (result.value as string) || "Document parsed, but it appears to be empty.";
+        } catch (error: any) {
+            throw new Error(`Failed to parse binary document: ${error.message}`);
+        } finally {
+            await fs.unlink(tempFilePath).catch(() => { });
+        }
     }
 
-    return Buffer.from(result.fileBinary).toString('utf-8');
+    return fileBuffer.toString('utf-8');
 }
-
-/**
- * Parse a CSV header line, handling quoted fields and trimming.
- * Returns an array of clean column names.
- */
 function parseCsvHeader(headerLine: string): string[] {
     const fields: string[] = [];
     let current = '';
@@ -223,47 +231,28 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
     tools: [
         {
             name: "list_folder",
-            description: DROPBOX_SHARED_URL
-                ? "List folders and files inside the shared Dropbox folder. Path must be relative to the shared root (e.g., '' for root, 'Data' for /Data)."
-                : "List folders and files in a specific Dropbox path (absolute, e.g., '/Data' or '' for root).",
+            description: "List folders and files. Use '' for root, or '/FolderName' for subfolders.",
             inputSchema: {
                 type: "object",
-                properties: {
-                    path: {
-                        type: "string",
-                        description: DROPBOX_SHARED_URL
-                            ? "Relative path inside the shared folder (e.g., '' or 'Data')."
-                            : "Absolute path to list (e.g., '' for root or '/Data')."
-                    }
-                },
+                properties: { path: { type: "string" } },
                 required: ["path"]
             }
         },
         {
             name: "search_files",
-            description: "Search for files by name, extension, or content across the entire Dropbox account.",
+            description: "Search for files by name, extension, or content.",
             inputSchema: {
                 type: "object",
-                properties: {
-                    query: { type: "string", description: "The search query (e.g., 'sales report' or '.csv')" }
-                },
+                properties: { query: { type: "string" } },
                 required: ["query"]
             }
         },
         {
             name: "get_file_metadata",
-            description: DROPBOX_SHARED_URL
-                ? "Retrieve metadata (size, modified date) for a file inside the shared folder. Path is relative to the shared root."
-                : "Retrieve metadata (size, modified date) for a file at an absolute Dropbox path.",
+            description: "Retrieve size/modified date for a file.",
             inputSchema: {
                 type: "object",
-                properties: {
-                    path: {
-                        type: "string", description: DROPBOX_SHARED_URL
-                            ? "Relative path to the file inside the shared folder."
-                            : "Exact Dropbox file path (e.g., '/Data/file.csv')."
-                    }
-                },
+                properties: { path: { type: "string" } },
                 required: ["path"]
             }
         },
@@ -272,28 +261,16 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
             description: "Display the schema (columns/headers) of a CSV or TSV file.",
             inputSchema: {
                 type: "object",
-                properties: {
-                    identifier: {
-                        type: "string", description: DROPBOX_SHARED_URL
-                            ? "Relative path to the table file (e.g., 'data.csv') or a shared link URL."
-                            : "Dropbox path (e.g., '/Data/file.csv') or shared link URL."
-                    }
-                },
+                properties: { identifier: { type: "string" } },
                 required: ["identifier"]
             }
         },
         {
             name: "read_data",
-            description: "Download and read the content of a text file (CSV, JSON, TXT, etc.).",
+            description: "Download and read the content of a text file. Truncated if over 100k chars.",
             inputSchema: {
                 type: "object",
-                properties: {
-                    identifier: {
-                        type: "string", description: DROPBOX_SHARED_URL
-                            ? "Relative path (e.g., 'data.csv') or a shared link URL."
-                            : "Dropbox path (e.g., '/Data/file.csv') or shared link URL."
-                    }
-                },
+                properties: { identifier: { type: "string" } },
                 required: ["identifier"]
             }
         }
@@ -301,30 +278,27 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
 }));
 
 // ------------------------------------------------------------------
-// 6. Tool Handlers
+// 6. Tool Handlers (Using direct limited wrapper)
 // ------------------------------------------------------------------
 server.setRequestHandler(CallToolRequestSchema, async (request) => {
     try {
         const name = request.params.name;
         const args = request.params.arguments || {};
 
-        // --- list_folder ---
         if (name === "list_folder") {
-            let path = (args.path as string) || "";
+            const rawPath = (args.path as string) || "";
             let requestArgs: any;
 
             if (DROPBOX_SHARED_URL) {
-                path = normalizeSharedPath(path);
                 requestArgs = {
-                    path: path,
+                    path: formatDropboxPath(rawPath),
                     shared_link: { url: DROPBOX_SHARED_URL }
                 };
             } else {
-                // Absolute path: if "/" then empty string for root
-                requestArgs = { path: path === "/" ? "" : path };
+                requestArgs = { path: formatDropboxPath(rawPath) };
             }
 
-            const response = await client.filesListFolder(requestArgs);
+            const response = await limited(() => client.filesListFolder(requestArgs));
             const entries = response.result.entries.map((e: any) =>
                 `[${e['.tag'].toUpperCase()}] ${e.name} (Path: ${e.path_display})`
             );
@@ -333,11 +307,10 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             };
         }
 
-        // --- search_files ---
         if (name === "search_files") {
-            const response = await client.filesSearchV2({ query: args.query as string });
+            const response = await limited(() => client.filesSearchV2({ query: args.query as string }));
             const matches = response.result.matches.map((m: any) => {
-                const meta = (m.metadata as any).metadata;
+                const meta = m.metadata.metadata;
                 return `[${meta['.tag'].toUpperCase()}] ${meta.name} (Path: ${meta.path_display})`;
             });
             return {
@@ -345,106 +318,48 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             };
         }
 
-        // --- get_file_metadata ---
         if (name === "get_file_metadata") {
             let response;
-            const rawPath = (args.path as string) || "";
+            const path = formatDropboxPath((args.path as string) || "");
 
             if (DROPBOX_SHARED_URL) {
-                const path = normalizeSharedPath(rawPath);
-                response = await client.sharingGetSharedLinkMetadata({
+                response = await limited(() => client.sharingGetSharedLinkMetadata({
                     url: DROPBOX_SHARED_URL,
                     path: path
-                });
+                }));
             } else {
-                response = await client.filesGetMetadata({ path: rawPath });
+                response = await limited(() => client.filesGetMetadata({ path: path }));
             }
-
             return {
                 content: [{ type: "text", text: JSON.stringify(response.result, null, 2) }]
             };
         }
 
-        // --- read_data ---
         if (name === "read_data") {
-            try {
-                const text = await downloadHelper(args.identifier as string);
-                const truncated = text.length > 100000
-                    ? text.substring(0, 100000) + "\n... (truncated)"
-                    : text;
-                return { content: [{ type: "text", text: truncated }] };
-            } catch (err: any) {
-                if (isNotFoundError(err)) {
-                    return {
-                        content: [{ type: "text", text: `File not found at "${args.identifier}".` }],
-                        isError: true
-                    };
-                }
-                if (isPermissionError(err)) {
-                    return {
-                        content: [{ type: "text", text: `Permission denied for "${args.identifier}".` }],
-                        isError: true
-                    };
-                }
-                if (isBadRequestError(err)) {
-                    return {
-                        content: [{ type: "text", text: `Invalid request. Check the path/identifier: "${args.identifier}".` }],
-                        isError: true
-                    };
-                }
-                if (err.message && err.message.includes("binary file")) {
-                    return { content: [{ type: "text", text: err.message }], isError: true };
-                }
-                throw err;
-            }
+            const text = await downloadHelper(args.identifier as string);
+            const truncated = text.length > 100000
+                ? text.substring(0, 100000) + "\n... [WARNING: FILE TRUNCATED TO 100K CHARACTERS]"
+                : text;
+            return { content: [{ type: "text", text: truncated }] };
         }
 
-        // --- describe_table ---
         if (name === "describe_table") {
-            try {
-                const text = await downloadHelper(args.identifier as string);
-                const lines = text.split(/\r?\n/).filter(line => line.trim().length > 0);
-                if (lines.length === 0) {
-                    return { content: [{ type: "text", text: "File is empty." }] };
-                }
-
-                let headers: string[];
-                let totalRows = lines.length - 1;
-                const firstLine = lines[0];
-                const commaCount = (firstLine.match(/,/g) || []).length;
-
-                if (commaCount > 0) {
-                    headers = parseCsvHeader(firstLine);
-                    if (headers.length < 2) {
-                        headers = lines.map((_, i) => `Column ${i + 1}`);
-                        totalRows = lines.length;
-                    }
-                } else {
-                    headers = lines.map((_, i) => `Column ${i + 1}`);
-                    totalRows = lines.length;
-                }
-
-                const schemaStr =
-                    `Columns detected:\n- ${headers.join('\n- ')}\n\nTotal rows: ${totalRows}`;
-                return { content: [{ type: "text", text: schemaStr }] };
-            } catch (err: any) {
-                if (isNotFoundError(err)) {
-                    return {
-                        content: [{ type: "text", text: `File not found at "${args.identifier}".` }],
-                        isError: true
-                    };
-                }
-                if (isPermissionError(err)) {
-                    return {
-                        content: [{ type: "text", text: `Permission denied for "${args.identifier}".` }],
-                        isError: true
-                    };
-                }
-                if (err.message && err.message.includes("binary file")) {
-                    return { content: [{ type: "text", text: err.message }], isError: true };
-                }
-                throw err;
+            // Note: This still downloads the full file to read the first line.
+            // Be careful running this on files >50MB as it may cause timeouts.
+            const text = await downloadHelper(args.identifier as string);
+            const lines = text.split(/\r?\n/).filter(line => line.trim().length > 0);
+            if (lines.length === 0) return { content: [{ type: "text", text: "File is empty." }] };
+
+            let headers: string[];
+            const firstLine = lines[0];
+            if ((firstLine.match(/,/g) || []).length > 0) {
+                headers = parseCsvHeader(firstLine);
+            } else {
+                headers = lines.map((_, i) => `Column ${i + 1}`);
             }
+
+            const schemaStr = `Columns detected:\n- ${headers.join('\n- ')}\n\nTotal rows: ${lines.length - 1}`;
+            return { content: [{ type: "text", text: schemaStr }] };
         }
 
         throw new Error(`Unknown tool: ${name}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dropbox-mcp-server",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "type": "module",
   "bin": {
     "dropbox-mcp-server": "dist/index.js"
@@ -13,7 +13,11 @@
     "@modelcontextprotocol/sdk": "^1.0.0",
     "dotenv": "^16.4.5",
     "dropbox": "^10.34.0",
-    "dropbox-mcp-server": "^2.0.0"
+    "dropbox-mcp-server": "^2.0.0",
+    "fs": "^0.0.1-security",
+    "officeparser": "^7.3.0",
+    "os": "^0.1.2",
+    "path": "^0.12.7"
   },
   "devDependencies": {
     "typescript": "^5.9.3"