Summary of Progress (October 21 – October 27)
This week, I focused on enhancing SARIF support by implementing new features, adjusting formatting, and ensuring schema adherence.
What I Worked On:
-
Key PRs:
- feat(errors): implement SARIF support as a MessageStyle- #17016
- fix(errors): adhere to SARIF 2.1.0 schema- #17020
- refactor(errors): move SARIF logic to sarif.d- #17025
- fix(errors): skip normal error output when SARIF is enabled- #17030
- feat(errors): integrate warnings and enhance SARIF formatting- #17032
-
Multiple Error Handling in SARIF Reporting:
I worked on enabling SARIF to handle multiple error messages within a single report. This involved modifying three files to accumulate errors insarif.d
using a global array that captured messages fromerrors.d
. After compilation,mars.d
called a function insarif.d
to generate the final report. While the integrated setup didn’t work as expected, a standalone version (sarifmultiple) was successful, indicating some more adjustments are still needed in integration.
Challenges:
- SARIF Report Hook Placement:
I’m considering where best to place the final report call. Should this function be called at the end ofcreateModules
inmars.d
, or is there a more optimal location?
Next Steps:
- I’ll work on ensuring consistent error accumulation across the compiler and adding SARIF support for successful executions.