fix: prefer output.pdf when selecting compile PDF (#26) (#27)

Overleaf's CLSI always names the main compile output 'output.pdf'.
The previous find(f => f.type === 'pdf') could match figure PDFs or
*-eps-converted-to.pdf intermediates that appear earlier in the array.

Now explicitly matches path === 'output.pdf' first, with a fallback
to type === 'pdf' for non-standard CLSI responses.

Fixes #26

Author: aloth

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aloth/olcli",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "Command-line interface for Overleaf — Sync, manage, and compile LaTeX projects from your terminal",
   "type": "module",
   "bin": {

src/client.ts

@@ -697,7 +697,11 @@ export class OverleafClient {
       throw new Error(`Compilation failed: ${data.status}`);
     }
 
-    const pdfFile = data.outputFiles?.find((f: any) => f.type === 'pdf');
+    // Match by path 'output.pdf' — Overleaf's CLSI always names the main
+    // compile output 'output.pdf'. Matching only on type === 'pdf' can pick up
+    // figure PDFs or *-eps-converted-to.pdf intermediates. See #26.
+    const pdfFile = data.outputFiles?.find((f: any) => f.path === 'output.pdf')
+      || data.outputFiles?.find((f: any) => f.type === 'pdf');
     if (!pdfFile) {
       throw new Error('No PDF output found');
     }
@@ -2059,7 +2063,10 @@ export class OverleafClient {
     this.applySetCookieHeaders(response.headers['set-cookie'] as string[] | undefined);
 
     const data = response.body as any;
-    const pdfFile = data.outputFiles?.find((f: any) => f.type === 'pdf');
+    // Prefer 'output.pdf' (the main compile output) over any other PDF.
+    // See #26 — projects with figure PDFs could return the wrong file.
+    const pdfFile = data.outputFiles?.find((f: any) => f.path === 'output.pdf')
+      || data.outputFiles?.find((f: any) => f.type === 'pdf');
 
     // Overleaf's CDN requires ?clsiserverid=<id> for build-output downloads.
     // Without it every output (pdf/log/bbl/...) 404s. See issue #22.