chore: if and throw for null types

ubq-testing · Aug 24, 2024 · 0dcc361 · 0dcc361
1 parent d908e06
commit 0dcc361
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/handlers/watch-user-activity.ts b/src/handlers/watch-user-activity.ts
@@ -22,8 +22,12 @@ export async function watchUserActivity(context: Context) {
 
 async function updateReminders(context: Context, repo: ListForOrg["data"][0]) {
     const { logger, octokit, payload } = context;
+    const owner = payload.repository.owner?.login;
+    if (!owner) {
+        throw new Error("No owner found in the payload");
+    }
     const issues = (await octokit.paginate(octokit.rest.issues.listForRepo, {
-        owner: payload.repository.owner.login,
+        owner,
         repo: repo.name,
         per_page: 100,
         state: "open",

diff --git a/src/helpers/get-watched-repos.ts b/src/helpers/get-watched-repos.ts
@@ -8,7 +8,11 @@ export async function getWatchedRepos(context: Context) {
     },
   } = context;
   const repoNames = new Set<string>();
-  const orgRepos = await getReposForOrg(context, context.payload.repository.owner.login);
+  const owner = context.payload.repository.owner?.login;
+  if (!owner) {
+    throw new Error("No owner found in the payload");
+  }
+  const orgRepos = await getReposForOrg(context, owner);
   orgRepos.forEach((repo) => repoNames.add(repo.name.toLowerCase()));
 
   for (const repo of optOut) {