Skip to content

Commit 3782ca0

Browse files
committed
only add if tool is present
1 parent 7ff4ef8 commit 3782ca0

File tree

3 files changed

+4
-3
lines changed

3 files changed

+4
-3
lines changed

src/codemodder/codeql.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,5 +59,5 @@ def from_sarif(cls, sarif_file: str | Path, truncate_rule_id: bool = False) -> S
5959
sarif_result, sarif_run, truncate_rule_id
6060
)
6161
result_set.add_result(codeql_result)
62-
result_set.store_tool_data(sarif_run["tool"])
62+
result_set.store_tool_data(sarif_run.get("tool", {}))
6363
return result_set

src/codemodder/result.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,8 @@ def add_result(self, result: Result):
192192
self.setdefault(result.rule_id, {}).setdefault(loc.file, []).append(result)
193193

194194
def store_tool_data(self, tool_data: dict):
195-
self.tools.append(tool_data)
195+
if tool_data:
196+
self.tools.append(tool_data)
196197

197198
def results_for_rule_and_file(
198199
self, context: CodemodExecutionContext, rule_id: str, file: Path

src/codemodder/semgrep.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ def from_sarif(cls, sarif_file: str | Path, truncate_rule_id: bool = False) -> S
8888
result, sarif_run, truncate_rule_id
8989
)
9090
result_set.add_result(sarif_result)
91-
result_set.store_tool_data(sarif_run["tool"])
91+
result_set.store_tool_data(sarif_run.get("tool", {}))
9292
return result_set
9393

9494

0 commit comments

Comments
 (0)