The rules for handling false positives for the markdown text editor were a bit messy and all over the place, especially the tests. The rules have now been cleaned up, simplified, and reorganized. This also fixes a false positive with the text editor that occurs at PL-2, it was easier to simplify the rules then add another rule.
Real payloads are now being used for tests, this makes the tests smaller and should hopefully prevent any issues stemming from not using real payloads.
About 100 lines of code has been removed.
Rule count has been reduced from 184 down to 179.
All rules for the markdown text editor are in one area instead of being split between file manager and text editor.
theseion
commented
2 weeks ago
The rules for handling false positives for the markdown text editor were a bit messy and all over the place, especially the tests. The rules have now been cleaned up, simplified, and reorganized. This also fixes a false positive with the text editor that occurs at PL-2, it was easier to simplify the rules then add another rule.