Ignore error getting authenticated user

Author: tjenkinson

dist/index.js

@@ -15236,37 +15236,40 @@ function run() {
                         });
                     };
                     approvePR = function () { return __awaiter(_this, void 0, void 0, function () {
-                        var authenticatedUser, existingReviews, e_2, existingReview, review;
+                        var maybeAuthenticatedUser, e_2, authenticatedUser_1, existingReviews, existingReview, review;
                         return __generator(this, function (_a) {
                             switch (_a.label) {
-                                case 0: return [4 /*yield*/, octokit.rest.users.getAuthenticated()];
+                                case 0:
+                                    maybeAuthenticatedUser = null;
+                                    _a.label = 1;
                                 case 1:
-                                    authenticatedUser = (_a.sent())
-                                        .data;
-                                    core.debug("Authenticated user: ".concat(authenticatedUser.id));
-                                    existingReviews = [];
-                                    _a.label = 2;
+                                    _a.trys.push([1, 3, , 4]);
+                                    return [4 /*yield*/, octokit.rest.users.getAuthenticated()];
                                 case 2:
-                                    _a.trys.push([2, 4, , 5]);
-                                    return [4 /*yield*/, octokit.rest.pulls.listReviews({
-                                            owner: context.repo.owner,
-                                            repo: context.repo.repo,
-                                            pull_number: pr.number,
-                                        })];
+                                    maybeAuthenticatedUser = (_a.sent())
+                                        .data;
+                                    return [3 /*break*/, 4];
                                 case 3:
-                                    existingReviews = (_a.sent()).data;
-                                    return [3 /*break*/, 5];
-                                case 4:
                                     e_2 = _a.sent();
-                                    core.warning('Error checking for existing reviews. Assuming none');
+                                    core.warning('Error fetching authenticated user');
                                     if (core.isDebug() && (e_2 instanceof Error || typeof e_2 === 'string')) {
                                         core.warning(e_2);
                                     }
-                                    return [3 /*break*/, 5];
+                                    return [3 /*break*/, 4];
+                                case 4:
+                                    if (!maybeAuthenticatedUser) return [3 /*break*/, 7];
+                                    authenticatedUser_1 = maybeAuthenticatedUser;
+                                    core.debug("Authenticated user: ".concat(authenticatedUser_1.id));
+                                    return [4 /*yield*/, octokit.rest.pulls.listReviews({
+                                            owner: context.repo.owner,
+                                            repo: context.repo.repo,
+                                            pull_number: pr.number,
+                                        })];
                                 case 5:
+                                    existingReviews = (_a.sent()).data;
                                     existingReview = existingReviews.find(function (_a) {
                                         var user = _a.user, state = _a.state;
-                                        return (user === null || user === void 0 ? void 0 : user.id) === authenticatedUser.id && state === 'PENDING';
+                                        return (user === null || user === void 0 ? void 0 : user.id) === authenticatedUser_1.id && state === 'PENDING';
                                     });
                                     if (!existingReview) return [3 /*break*/, 7];
                                     core.info("Found an existing pending review. Deleting it");

src/run.ts

@@ -209,41 +209,48 @@ export async function run(): Promise<Result> {
     });
 
   const approvePR = async () => {
-    const authenticatedUser = (await octokit.rest.users.getAuthenticated())
-      .data;
+    let maybeAuthenticatedUser:
+      | Awaited<
+          ReturnType<(typeof octokit)['rest']['users']['getAuthenticated']>
+        >['data']
+      | null = null;
 
-    core.debug(`Authenticated user: ${authenticatedUser.id}`);
-
-    let existingReviews: Awaited<
-      ReturnType<(typeof octokit)['rest']['pulls']['listReviews']>
-    >['data'] = [];
     try {
-      existingReviews = (
+      maybeAuthenticatedUser = (await octokit.rest.users.getAuthenticated())
+        .data;
+    } catch (e) {
+      core.warning('Error fetching authenticated user');
+      if (core.isDebug() && (e instanceof Error || typeof e === 'string')) {
+        core.warning(e);
+      }
+    }
+
+    if (maybeAuthenticatedUser) {
+      const authenticatedUser = maybeAuthenticatedUser;
+      core.debug(`Authenticated user: ${authenticatedUser.id}`);
+
+      const existingReviews = (
         await octokit.rest.pulls.listReviews({
           owner: context.repo.owner,
           repo: context.repo.repo,
           pull_number: pr.number,
         })
       ).data;
-    } catch (e) {
-      core.warning('Error checking for existing reviews. Assuming none');
-      if (core.isDebug() && (e instanceof Error || typeof e === 'string')) {
-        core.warning(e);
-      }
-    }
-    const existingReview = existingReviews.find(
-      ({ user, state }) =>
-        user?.id === authenticatedUser.id && state === 'PENDING'
-    );
 
-    if (existingReview) {
-      core.info(`Found an existing pending review. Deleting it`);
-      await octokit.rest.pulls.deletePendingReview({
-        owner: context.repo.owner,
-        repo: context.repo.repo,
-        pull_number: pr.number,
-        review_id: existingReview.id,
-      });
+      const existingReview = existingReviews.find(
+        ({ user, state }) =>
+          user?.id === authenticatedUser.id && state === 'PENDING'
+      );
+
+      if (existingReview) {
+        core.info(`Found an existing pending review. Deleting it`);
+        await octokit.rest.pulls.deletePendingReview({
+          owner: context.repo.owner,
+          repo: context.repo.repo,
+          pull_number: pr.number,
+          review_id: existingReview.id,
+        });
+      }
     }
 
     const review = await octokit.rest.pulls.createReview({