Fetch alerts with open status only

kunalnagarco · Mar 12, 2024 · c706b8c · c706b8c
1 parent a67a900
commit c706b8c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 16 deletions.
diff --git a/src/entities/alert.ts b/src/entities/alert.ts
@@ -33,13 +33,3 @@ export const toAlert = (
     : undefined,
   createdAt: dependabotAlert.created_at,
 })
-
-export const isActiveAlert = (dependabotAlert: DependabotAlert): boolean => {
-  if (
-    dependabotAlert.dismissed_at === null &&
-    dependabotAlert.fixed_at === null
-  ) {
-    return true
-  }
-  return false
-}
diff --git a/src/fetch-alerts.ts b/src/fetch-alerts.ts
@@ -1,6 +1,6 @@
 import { Octokit } from '@octokit/rest'
 
-import { Alert, isActiveAlert, toAlert } from './entities'
+import { Alert, toAlert } from './entities'
 
 export const fetchAlerts = async (
   gitHubPersonalAccessToken: string,
@@ -18,13 +18,12 @@ export const fetchAlerts = async (
   const response = await octokit.dependabot.listAlertsForRepo({
     owner: repositoryOwner,
     repo: repositoryName,
+    state: 'open',
     severity,
     per_page: count,
   })
-  const alerts: Alert[] = response.data
-    .filter((dependabotAlert) => isActiveAlert(dependabotAlert))
-    .map((dependabotAlert) =>
-      toAlert(dependabotAlert, repositoryName, repositoryOwner),
-    )
+  const alerts: Alert[] = response.data.map((dependabotAlert) =>
+    toAlert(dependabotAlert, repositoryName, repositoryOwner),
+  )
   return alerts
 }