Multi-page PDF autofill resets index and misaligns field values" labels: bug

[mdfarid01]

---

name: 🐛 Bug Report
about: Create a report to help us improve FireForm.
title: "[BUG]: Multi-page PDF autofill resets index and misaligns field values"
labels: bug
assignees: ''

⚡️ Describe the Bug

In src/backend.py inside Fill.fill_form, the answer index i is reset for every PDF page.
For multi-page forms, page 2 starts filling again from the first extracted answer, causing incorrect field-value mapping.

👣 Steps to Reproduce

1. Use a fillable PDF with text fields on at least 2 pages.
2. Provide enough extracted values to fill all fields.
3. Run Fill.fill_form(...).
4. Open output PDF and compare values on page 2+ with expected order.

📉 Expected Behavior

Field filling should continue sequentially across all pages, not restart from the first value on each page.

🖥️ Environment Information

• OS: macOS / Linux (reproducible across environments)
• Docker/Compose Version: Docker Compose v2.x
• Ollama Model used: Mistral (via Ollama)

📸 Screenshots/Logs

Relevant code path:
src/backend.py in Fill.fill_form (around lines 151–159).
i = 0 is currently inside for page in pdf.pages:, so the index resets on every page.

🕵️ Possible Fix

Move i = 0 before iterating pages so index is global across the whole document.
Add a regression test with a 2-page fillable PDF fixture to verify correct mapping.

[its-arunchauhan]

Hi @mdfarid01,

I would like to work on this issue. I understand that the index i is resetting within the page loop in src/backend.py, causing the misalignment. I can move the initialization outside the loop and add a regression test with a multi-page PDF as suggested.

Could you please assign this to me?"

[mdfarid01]

Hi @vharkins1 @marcvergees

Could you please confirm if this is expected behavior? If it's a bug, I can submit a fix with a regression test.

Thanks!

[mdfarid01]

I opened a PR for this fix: #147

Summary:

• Fixed index reset across pages in src/filler.py by moving i = 0 outside the page loop.
• Added regression test tests/test_filler.py to ensure multi-page mapping continues correctly.

Validation:

• Before: page1=alpha, page2=alpha (incorrect)
• After: page1=alpha, page2=bravo (correct)

@vharkins1
@marcvergees

[marcvergees]

Done.