We log every time we are unable to rearrange the infobox and lead paragraph so the lead paragraph comes first. We only log for English Wikipedia but would like to reduce the amount of logging so we can track any major changes to major templates that our code depends on. It seems we log events for pages where the lead paragraph is already above the infobox which adds unnecessary noise.
When visiting https://en.wikipedia.org/wiki/P38_mitogen-activated_protein_kinases, even though the infoboxes on this page are wrapped, they do not appear at the top of the page. Despite this, we log that the page is not mobile friendly "message:Found infobox wrapped with container on P38 mitogen-activated protein kinases (rev:840755052) "
We should only log pages where the infobox appears above the first paragraph and cannot be moved. I believe, if this was addressed, we would see no errors logged (or near zero) and possibly reconsider our daily chore in the chores list.
In the 2nd if statement the method hasNoNonEmptyPrecedingParagraphs( $xPath, $infobox ) is not taken into account in logging. Adding a test and adding a check should be straightforward.
This should not make any difference, but there is a slight risk that this may impact the lead paragraph algorithm.
Please verify nothing has broken by looking at a variety of articles on http://reading-web-staging.wmflabs.org/ and confirming the lead paragraph shows above the infobox on mobile.
Sign off steps
A developer should look at logstash and should notice a decrease in number of events logged complaining about unmoveable paragraphs.