bambu-printer-mcp 1.1.14 → 1.1.15

package/CONTRIBUTORS.md

@@ -15,7 +15,7 @@ Thank you to everyone who builds, tests, reports problems, and shares real print
 | [John Randall (johntrandall)](https://github.com/johntrandall) | Captures A1 job manifests and contributes the SD-root/project-file fix in [#14](https://github.com/DMontgomery40/bambu-printer-mcp/pull/14). |
 | [Kyle Taylor (kyletaylored)](https://github.com/kyletaylored) | Contributes Claude Desktop extension packaging, prompted setup, and the writable-temp-directory fix in [#11](https://github.com/DMontgomery40/bambu-printer-mcp/pull/11). |
 | [Quinn (quinnpertuit)](https://github.com/quinnpertuit) | Investigates P2S transfer routing and contributes model-capability and serial-prefix analysis in [#9](https://github.com/DMontgomery40/bambu-printer-mcp/pull/9). The release uses the alternative P2S implementation from #15. |
-| [Vail (VailElla)](https://github.com/VailElla) | Contributes the native X2D transport, local eMMC investigation, and hardware evidence in [#10](https://github.com/DMontgomery40/bambu-printer-mcp/pull/10). The optional macOS native route is integrated with shared file/state/preflight checks, complete AMS mappings, helper cancellation, and clean-install coverage. Prior hardware evidence is credited separately from this release's mocked tests. |
+| [Vail (VailElla)](https://github.com/VailElla) | Contributes the native X2D transport, local eMMC investigation, and hardware evidence in [#10](https://github.com/DMontgomery40/bambu-printer-mcp/pull/10). The optional macOS native route is integrated with shared file/state/preflight checks, complete AMS mappings, helper cancellation, and clean-install coverage. In [#26](https://github.com/DMontgomery40/bambu-printer-mcp/pull/26), replaces misleading partial FTPS listings with read-only, failure-reporting queries and verifies a real X2D empty-root listing; nonempty directories and failure paths use mocked coverage. |
 
 ## Project contributors
 

package/README.md

@@ -603,6 +603,8 @@ High-signal fields:
 
 List files stored on the printer's SD card. Scans the `cache/`, `timelapse/`, and `logs/` directories and returns both a flat list and a directory-grouped breakdown.
 
+This is a read-only query: it never creates directories. Optional directories absent from a successful root listing return empty lists. Authentication, TLS, permission, and transfer failures are reported as errors rather than empty or partial results.
+
 ```json
 {
   "host": "192.168.1.100",

package/dist/printers/bambu.js

@@ -1065,19 +1065,32 @@ export class BambuImplementation {
         };
     }
     async getFiles(host, serial, token) {
-        const printer = new BambuPrinter(host, serial, token);
+        const client = new FTPClient(15000);
         const directories = ["cache", "timelapse", "logs"];
         const filesByDirectory = {};
-        await printer.manipulateFiles(async (context) => {
+        try {
+            await client.access({
+                host,
+                port: 990,
+                user: "bblp",
+                password: token,
+                secure: "implicit",
+                secureOptions: ftpsSecureOptions(host),
+            });
+            await this.waitForTlsSession(client);
+            // Discover absent optional directories without creating them or treating
+            // ambiguous FTP 550 responses (including permission errors) as emptiness.
+            const root = await client.list("/");
             for (const directory of directories) {
-                try {
-                    filesByDirectory[directory] = await context.readDir(directory);
-                }
-                catch {
-                    filesByDirectory[directory] = [];
-                }
+                const present = root.some(entry => entry.name === directory && (entry.isDirectory || entry.isSymbolicLink));
+                filesByDirectory[directory] = present
+                    ? (await client.list(`/${directory}`)).map(entry => entry.name)
+                    : [];
             }
-        });
+        }
+        finally {
+            client.close();
+        }
         const files = Object.entries(filesByDirectory).flatMap(([directory, names]) => names.map((name) => `${directory}/${name}`));
         return {
             files,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bambu-printer-mcp",
-  "version": "1.1.14",
+  "version": "1.1.15",
   "description": "MCP server for Bambu Lab 3D printers with STL manipulation, slicing, and print management",
   "main": "dist/index.js",
   "repository": {

package/src/printers/bambu.ts

@@ -1257,19 +1257,32 @@ export class BambuImplementation {
   }
 
   async getFiles(host: string, serial: string, token: string) {
-    const printer = new BambuPrinter(host, serial, token);
+    const client = new FTPClient(15_000);
     const directories = ["cache", "timelapse", "logs"];
     const filesByDirectory: Record<string, string[]> = {};
 
-    await printer.manipulateFiles(async (context) => {
+    try {
+      await client.access({
+        host,
+        port: 990,
+        user: "bblp",
+        password: token,
+        secure: "implicit",
+        secureOptions: ftpsSecureOptions(host),
+      });
+      await this.waitForTlsSession(client);
+      // Discover absent optional directories without creating them or treating
+      // ambiguous FTP 550 responses (including permission errors) as emptiness.
+      const root = await client.list("/");
       for (const directory of directories) {
-        try {
-          filesByDirectory[directory] = await context.readDir(directory);
-        } catch {
-          filesByDirectory[directory] = [];
-        }
+        const present = root.some(entry => entry.name === directory && (entry.isDirectory || entry.isSymbolicLink));
+        filesByDirectory[directory] = present
+          ? (await client.list(`/${directory}`)).map(entry => entry.name)
+          : [];
       }
-    });
+    } finally {
+      client.close();
+    }
 
     const files = Object.entries(filesByDirectory).flatMap(([directory, names]) =>
       names.map((name) => `${directory}/${name}`)