ForumMagnum / ForumMagnum / 25463845060 / 1

48%

master: 47%

Pull #12497

github

Document the ES indexer's index-everything-filter-at-query-time convention

Add a paragraph to the IndexConfig.filters doc comment so the next person
(or AI agent) touching the search code knows that ForumMagnum indexes
every row regardless of visibility flags and does the filtering here at
query time, not in the SQL export methods.

I half-implemented the alternative pattern earlier on this branch (added
WHERE rejected IS NOT TRUE to the bulk-export queries) and reverted it
in fb9c12f because the per-document update path always re-indexes, so
the filter just creates drift between recreateIndex and steady-state
behavior. Captured the rationale here so the same trap doesn't catch
the next contributor.

4806 of 6715 branches covered (71.57%)

Branch coverage included in aggregate %.

125057 of 373121 relevant lines covered (33.52%)

34.62 hits per line