@imenam/simple-scraper 1.0.15 → 1.0.16

package/README.md

@@ -332,7 +332,7 @@ The capture area is resolved by precedence: `selector` → `clip` → `full_page
 | `full_page` | boolean | | Capture the full scrollable page height (default: false, ignored when `selector` or `clip` is provided) |
 | `format` | `png` \| `jpeg` | | Image format (default: `png`) |
 | `output` | `inline` \| `file` \| `both` | ✅ | `inline` embeds the image in the response, `file` saves to disk and returns the path, `both` does both |
-| `path` | string | | Absolute or relative path for the saved file. Defaults to `./screenshots/screenshot-<timestamp>.<format>` |
+| `path` | string | | **Absolute** path for the saved file — relative paths are rejected, since they would resolve against the server's working directory, which the client sets unpredictably. Missing parent directories are created. Defaults to `<server_root>/screenshots/screenshot-<timestamp>.<format>` |
 
 ---
 

package/dist/index.js

@@ -592,7 +592,13 @@ server.tool('screenshot', 'Take a screenshot of a page. Provide either session_i
     full_page: z.boolean().optional().describe('Capture the full scrollable page height (default: false, ignored when selector or clip is provided)'),
     format: z.enum(['png', 'jpeg']).optional().describe('Image format (default: png)'),
     output: z.enum(['inline', 'file', 'both']).describe('Return format: "inline" embeds the image in the response, "file" saves to disk and returns the path, "both" does both'),
-    path: z.string().optional().describe('Absolute or relative file path for the saved image (used when output is "file" or "both"). Relative paths are resolved from the MCP server\'s working directory. Defaults to <server_root>/screenshots/<timestamp>.<format>'),
+    path: z
+        .string()
+        .refine(p => path.isAbsolute(p), {
+        message: 'path must be absolute — a relative path would resolve against the MCP server\'s working directory, which is set by the client and not predictable.',
+    })
+        .optional()
+        .describe('Absolute file path for the saved image (used when output is "file" or "both"). Must be absolute; missing parent directories are created. Defaults to <server_root>/screenshots/screenshot-<timestamp>.<format>'),
 }, async ({ session_id, url, wait_for, timeout, selector, clip, viewport, full_page, format, output, path: filePath }) => {
     const effectiveTimeout = timeout ?? getDefaultTimeout();
     const fmt = format ?? 'png';
@@ -646,7 +652,8 @@ server.tool('screenshot', 'Take a screenshot of a page. Provide either session_i
             imageBuffer = Buffer.from(await page.screenshot({ type: fmt, fullPage: full_page ?? false }));
         }
         const base64 = imageBuffer.toString('base64');
-        // Resolve file path
+        // Resolve file path. filePath is guaranteed absolute by the schema; resolve()
+        // is kept only to normalise separators and any ".." segments.
         const resolvedPath = (() => {
             if (filePath)
                 return path.resolve(filePath);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@imenam/simple-scraper",
-  "version": "1.0.15",
+  "version": "1.0.16",
   "description": "MCP server for web scraping and JavaScript execution using Puppeteer",
   "type": "module",
   "main": "dist/index.js",