Fix package sorting in scan summary report

index.ts

@@ -30,11 +30,14 @@ async function run(): Promise<void> {
     ];
 
     if (!config.skipSummary) {
-      presenters.push(new SummaryReportPresenter());
+      presenters.push(new SummaryReportPresenter(core.summary));
     }
 
     const useCase = new RunScanUseCase(scanner, presenters, inputProvider);
+
+    core.summary.emptyBuffer().clear();
     await useCase.execute();
+    await core.summary.write({ overwrite: true });
 
 } catch (error) {
     if (error instanceof Error) {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "secure-inline-scan-action",
-  "version": "6.3.0",
+  "version": "6.3.1",
   "description": "This actions performs image analysis on locally built container image and posts the result of the analysis to Sysdig Secure.",
   "main": "index.js",
   "scripts": {

src/application/use-cases/RunScanUseCase.ts

@@ -6,7 +6,6 @@ import { IReportPresenter } from '../ports/IReportPresenter';
 import { FilterOptions } from '../../domain/services/filtering';
 import { Severity } from '../../domain/scanresult';
 import { ScanConfig } from '../ports/ScanConfig';
-import { ScanExecutionError } from '../errors/ScanExecutionError';
 import { ChecksumVerificationError } from '../errors/ChecksumVerificationError';
 
 export class RunScanUseCase {

src/domain/scanresult/Severity.ts

@@ -29,6 +29,10 @@ export class Severity {
     return Severity.values.find(s => s.value === value) || Severity.Unknown;
   }
 
+  public static getValues(): Severity[] {
+    return Severity.values;
+  }
+
   public asNumber(): number {
     return this.value;
   }

src/domain/services/sorting.ts

@@ -0,0 +1,14 @@
+import { Package, Severity } from "../scanresult";
+
+export function sortPackagesByVulnSeverity(packages: Package[]): Package[] {
+    return packages.sort((a, b) => {
+        for (const severity of Severity.getValues()) {
+            const aCount = a.getVulnerabilities().filter(v => v.severity.isEqualTo(severity)).length;
+            const bCount = b.getVulnerabilities().filter(v => v.severity.isEqualTo(severity)).length;
+            if (aCount !== bCount) {
+                return bCount - aCount;
+            }
+        }
+        return 0;
+    });
+}

src/infrastructure/github/ISummary.ts

@@ -0,0 +1,7 @@
+export interface ISummary {
+    addHeading(text: string, level?: number): ISummary;
+    addTable(rows: any[]): ISummary;
+    addCodeBlock(code: string, lang?: string): ISummary;
+    addList(items: string[], ordered?: boolean): ISummary;
+    addRaw(text: string, addEOL?: boolean): ISummary;
+}