Skip to content

Exclude ALDI Süd assortment pages; release 0.1.67 - #67

Merged
lesecuritae merged 2 commits into
mainfrom
fix/issue-66-aldi-assortment-20261023
Oct 23, 2026
Merged

lesecuritae merged 2 commits into
mainfrom
fix/issue-66-aldi-assortment-20261023

Conversation

@lesecuritae

Copy link
Copy Markdown
Owner

ALDI Süd includes permanent-range pages headed “Aus unserem Sortiment” in its weekly brochures. The server previously imported their products as special offers. Filter these pages before fetching product hotspots, using the official single-page endpoint when the other half of a spread still contains offers.

Preserve normal offers without a crossed-out price, pages without extracted text, and body text that merely mentions the assortment. The native direct Android HTML provider does not consume these brochure pages; the behavior fix is confined to the server parser.

Validation:

  • Current official KW43 brochure: 168 entries before, 144 after; exactly the 24 assortment-only entries on pages 26–29 removed, all other offer IDs preserved.
  • 498 local Python/JavaScript tests passed; Ruff and Bandit passed.
  • Five regression cases fail against the original parser and pass with the fix; mixed spreads, line breaks, case and nonbreaking spaces covered.
  • No VM or emulator was required for this parser change.

Prepare release 0.1.67. References #66; the issue will be answered and closed after the release artifacts are verified.

@lesecuritae
lesecuritae merged commit 2e0865b into main Oct 23, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant