Filter issues by changed files

Author: matejchalk

badges/coverage.svg

@@ -13,8 +13,8 @@ export async function listChangedFiles(
   ]
   const { files } = await git.diffSummary([
     `--diff-filter=${statuses.join('')}`,
-    refs.base.sha,
-    refs.head.sha
+    refs.base.ref,
+    refs.head.ref
   ])
   return files.filter(({ binary }) => !binary).map(({ file }) => file)
 }

dist/index.js

@@ -5,13 +5,13 @@ export type SourceFileIssue = Required<Issue> & { audit: AuditReport }
 export function filterRelevantIssues({
   currReport,
   prevReport,
-  reportsDiff
-  // changedFiles
+  reportsDiff,
+  changedFiles
 }: {
   currReport: Report
   prevReport: Report
   reportsDiff: ReportsDiff
-  // changedFiles: string[]
+  changedFiles: string[]
 }): SourceFileIssue[] {
   const audits = [...reportsDiff.audits.changed, ...reportsDiff.audits.added]
     .map(auditLink =>
@@ -28,7 +28,7 @@ export function filterRelevantIssues({
 
   return issues.filter(
     issue =>
-      // changedFiles.includes(issue.source?.file) &&
+      changedFiles.includes(issue.source?.file) &&
       !prevIssues.some(prevIssue => issuesAreEqual(prevIssue, issue))
   )
 }

dist/index.js.map

@@ -13,6 +13,7 @@ import {
 import { collect } from './collect'
 import { commentOnPR } from './comment'
 import { compare } from './compare'
+import { listChangedFiles } from './diff'
 import { parseInputs } from './inputs'
 import { filterRelevantIssues } from './issues'
 import type { PersistedCliFiles } from './persist'
@@ -44,8 +45,7 @@ export async function run(
     }
 
     if (isPullRequest(github.context.payload.pull_request)) {
-      const { base } = github.context.payload.pull_request
-      // const { base, head } = github.context.payload.pull_request
+      const { base, head } = github.context.payload.pull_request
       core.debug(`PR detected, preparing to compare base branch ${base.ref}`)
 
       let prevReport: string
@@ -92,18 +92,18 @@ export async function run(
       core.debug(`Commented on PR #${github.context.issue.number}`)
 
       if (inputs.annotations) {
+        await git.fetch('origin', base.ref, ['--depth=1'])
         const reportsDiff = await fs.readFile(diffJsonPath, 'utf8')
-        // const changedFiles = await listChangedFiles({ base, head }, git)
+        const changedFiles = await listChangedFiles({ base, head }, git)
         const issues = filterRelevantIssues({
           currReport: JSON.parse(currReport),
           prevReport: JSON.parse(prevReport),
-          reportsDiff: JSON.parse(reportsDiff)
-          // changedFiles
+          reportsDiff: JSON.parse(reportsDiff),
+          changedFiles
         })
         createAnnotationsFromIssues(issues)
         core.debug(
-          // `Found ${issues.length} relevant issues for ${changedFiles.length} changed files and created GitHub annotations`
-          `Found ${issues.length} relevant issues and created GitHub annotations`
+          `Found ${issues.length} relevant issues for ${changedFiles.length} changed files and created GitHub annotations`
         )
       }