Allow for missing ruleId.

[Ronoman]

This will be introduced if this ament_lint PR is merged. More commits will be added on this PR to support better de-duplication, depending on discussion in that PR's comments.

[ivanperez-keera]

I just posted a comment in the associated PR in ament_lint to try to move things along.

[Bckempa]

Provisionally nominating for inclusion in the next release, we're waiting on upstream for the the fix to ament_lint but that doesn't seem to preclude getting this one-liner merged.

[Bckempa]

Tested by building the current release ros2.repos workspace from source, using colcon test to generate test artifacts, then using ros2 run process-sarif visualize to parse the test logs. Then this PR's branch was loaded into the workspace and rebuilt without rerunning the tests. Finally the new version of process_sarif was used to parse the same artifacts with equivalent results.

Since this adds a defensive measure against future change but has no impact on current behavior I will approve and merge this PR.

Current release:

Result Kinds: {'UNKNOWN': 1609, 'PASS': 0, 'OPEN': 0, 'INFORMATIONAL': 0, 'REVIEW': 15822, 'FAIL': 0}
Result Levels: {'UNKNOWN': 1377, 'ERROR': 232, 'WARNING': 15822, 'NOTE': 0, 'NONE': 0}

This PR:

Result Kinds: {'UNKNOWN': 1609, 'PASS': 0, 'OPEN': 0, 'INFORMATIONAL': 0, 'REVIEW': 15822, 'FAIL': 0}
Result Levels: {'UNKNOWN': 1377, 'ERROR': 232, 'WARNING': 15822, 'NOTE': 0, 'NONE': 0}