fix(knowledge): take the page count from the parse that produced the text

The density check counted pages with a second, independent read of the file. The
two could disagree: a count that failed reported no pages, the check fell back to
treating the document as a single page, and a long scan carrying only a header
looked dense enough to skip OCR and be indexed as that header.

`parseBuffer` already reports the page count from the parse that produced the
text, so the two can no longer diverge, and the redundant second open of the file
goes away with it.

Author: waleedlatif1

apps/sim/lib/knowledge/documents/document-processor.ts

@@ -321,11 +321,16 @@ async function readEmbeddedPdfText(
 > {
   try {
     const buffer = await downloadFileWithTimeout(fileUrl, userId)
-    const [parsed, pageCount] = await Promise.all([
-      parseBuffer(buffer, 'pdf'),
-      getPdfPageCount(buffer),
-    ])
+    const parsed = await parseBuffer(buffer, 'pdf')
 
+    /**
+     * The page count comes from the same parse as the text, rather than a second
+     * independent read of the file. Counting separately lets the two disagree: a
+     * count that failed would report no pages, the density check would fall back to
+     * treating the document as a single page, and a long scan carrying only a header
+     * would look dense enough to skip OCR and be indexed as that header.
+     */
+    const pageCount = parsed.metadata?.pageCount ?? 0
     const verdict = assessPdfTextLayer(parsed.content, pageCount, parsed.metadata?.truncated)
     if (!verdict.usable) {
       logger.info('PDF text layer not usable, routing to OCR', {

apps/sim/lib/knowledge/documents/pdf-ocr-triage.test.ts

@@ -72,6 +72,28 @@ describe('PDF OCR triage', () => {
     expect(fetchMock).not.toHaveBeenCalled()
   })
 
+  /**
+   * The density check reads its page count from the same parse as the text. A long
+   * scan that yields only a header must stay sparse against its real page count —
+   * counting separately allowed a failed count to present it as a single dense page.
+   */
+  it('takes the page count from the parse, so a header-only scan stays sparse', async () => {
+    // Enough to clear the floor as a single page, nowhere near enough for 80.
+    const headerOnly = 'CONFIDENTIAL - Vendor Master Agreement - Page header. '.repeat(6)
+    mockParseBuffer.mockResolvedValue({ content: headerOnly, metadata: { pageCount: 80 } })
+    const fetchMock = vi.fn().mockResolvedValue(
+      new Response(JSON.stringify({ pages: [{ markdown: 'Recognised' }], usage_info: {} }), {
+        status: 200,
+        headers: { 'Content-Type': 'application/json' },
+      })
+    )
+    vi.stubGlobal('fetch', fetchMock)
+
+    const result = await parse()
+
+    expect(result.metadata.processingMethod).toBe('mistral-ocr')
+  })
+
   it('falls through to OCR when the PDF is a scan', async () => {
     mockParseBuffer.mockResolvedValue({ content: '', metadata: {} })
     const fetchMock = vi.fn().mockResolvedValue(