Yep - your overall interpretation is correct, but I would tighten the terminology.
For a forest trust, SIDFilteringQuarantined : False by itself does not mean that SID filtering is disabled. The quarantine mechanism represented by TRUST_ATTRIBUTE_QUARANTINED_DOMAIN applies to external trusts. Forest trusts have a different mechanism for handling SID history, which is why your initial forest trust can show SIDFilteringQuarantined : False and still pass the ANSSI check.
/EnableSIDHistory:Yes is the important change in your test. It permits SID history to be used across the forest trust and, in security terms, relaxes the SID-filtering protection that would otherwise prevent foreign SIDs from being carried across the trust. It is therefore reasonable to describe it as enabling SIDHistory across the forest trust, rather than literally saying that it toggles a separate "SID filtering" switch.
So, yes, the controls differ by trust type. For an external trust, quarantine/SID filtering is controlled through the quarantine setting, such as /Quarantine:Yes. For a forest trust, the relevant control is whether SID history is enabled, such as /EnableSIDHistory:Yes. They protect against the same general class of SID-history/SID injection abuse, but they are not literally the same trust attribute.
The wording, "Quarantine is disabled or SID history is enabled," is consistent with that model: it has two independent conditions that can make the trust fail the security check. For an external trust, the quarantine condition is relevant - for a forest trust, the SID-history condition is the relevant one.
I would change your client wording slightly. Saying "the outbound forest trust is not enforcing SID filtering because SID history is enabled" is understandable, but technically it implies there is a conventional SID-filtering flag that has simply been turned off. A more precise finding would be: "SID history is enabled across the outbound forest trust, allowing SID history to be honored across the trust and thereby weakening the SID-filtering protection against unauthorized SID injection. Disable SID history unless it is required for a documented migration or interoperability scenario."
And yep - legitimate cross-forest migrations are the main thing to investigate before disabling it. SID history can be used during migrations so users/groups retain access to resources in the source forest after their accounts are migrated. If the client has an active migration, legacy cross-forest authorization, or an application specifically depending on SID history, disabling it can break access. The key question is not simply "Do you use a forest trust?" but "Is SID history currently being used across this trust, and is there a documented business requirement for it?"
If the above response helps answer your question, remember to "Accept Answer" so that others in the community facing similar issues can easily find the solution. Your contribution is highly appreciated.
hth
Marcin