Merge pull request #2 from felickz/start-line

feat: include line number in location path for most recent instance
theztefan · Jan 16, 2025 · c2a1852 · c2a1852
2 parents a41009c + 7c74de5
commit c2a1852
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -64736,7 +64736,9 @@ class CodeScanning extends Printable {
             a.rule?.security_severity_level || a.rule?.severity || "",
             this.cweFromTags(a),
             a.tool?.name || "",
-            a.most_recent_instance?.location.path || "",
+            a.most_recent_instance?.location?.path
+                ? `${a.most_recent_instance.location.path}#L${a.most_recent_instance.location.start_line}`
+                : "",
             a.html_url,
         ]);
     }

diff --git a/src/context/CodeScanning.ts b/src/context/CodeScanning.ts
@@ -68,7 +68,9 @@ export class CodeScanning extends Printable implements Feature {
       a.rule?.security_severity_level || a.rule?.severity || "",
       this.cweFromTags(a),
       a.tool?.name || "",
-      a.most_recent_instance?.location.path || "",
+      a.most_recent_instance?.location?.path
+        ? `${a.most_recent_instance.location.path}#L${a.most_recent_instance.location.start_line}`
+        : "",
       a.html_url,
     ]);
   }