Merge pull request #6751 from LedgerHQ/chore/fix-bot-user-checking

chore: update github bot user checking
LedgerHQ · Apr 25, 2024 · 6ba2746 · 6ba2746
2 parents 7b1a7c5 + 1192762
commit 6ba2746
Show file tree

Hide file tree

Showing 8 changed files with 246 additions and 170 deletions.
diff --git a/tools/actions/build-checks/build/main.js b/tools/actions/build-checks/build/main.js
@@ -39783,7 +39783,8 @@ var desktopMetafilesKeys = {
   preloader: "metafile.preloader.json",
   renderer: "metafile.renderer.json",
   rendererWorker: "metafile.renderer.worker.json",
-  webviewPreloader: "metafile.webviewPreloader.json"
+  webviewPreloader: "metafile.webviewPreloader.json",
+  webviewDappPreloader: "metafile.webviewDappPreloader.json"
 };
 var mobileMetafileKeys = ["main.ios.jsbundle", "main.android.jsbundle"];
 function getMetafileBundleSize(metafile, slug) {

diff --git a/tools/actions/turbo-affected/build/main.js b/tools/actions/turbo-affected/build/main.js
@@ -18602,6 +18602,8 @@ var package_default = {
     doc: "pnpm --filter docs",
     mobile: "pnpm --filter live-mobile",
     common: "pnpm --filter live-common",
+    "device-core": "pnpm --filter device-core",
+    "device-react": "pnpm --filter device-react",
     "web-tools": "pnpm --filter web-tools",
     "live-env": "pnpm --filter live-env",
     portfolio: "pnpm --filter live-portfolio",
@@ -18656,6 +18658,7 @@ var package_default = {
     "dummy-live-app": "pnpm --filter dummy-live-app",
     "dummy-wallet-app": "pnpm --filter dummy-wallet-app",
     "dummy-ptx-app": "pnpm --filter dummy-ptx-app",
+    "ethereum-provider": "pnpm --filter ethereum-provider",
     ui: "pnpm --filter ui",
     "ui:react": "pnpm --filter react-ui",
     "ui:native": "pnpm --filter native-ui",

diff --git a/tools/actions/turborepo-gh-cache/build/server.js b/tools/actions/turborepo-gh-cache/build/server.js
diff --git a/tools/actions/turborepo-s3-cache/build/server.js b/tools/actions/turborepo-s3-cache/build/server.js
diff --git a/tools/github-bot/src/commands/generate-screenshots.ts b/tools/github-bot/src/commands/generate-screenshots.ts
@@ -68,14 +68,18 @@ export function generateScreenshots(app: Probot) {
     const number = payload.check_run.pull_requests[0]?.number;
     const login = payload.sender.login;
 
-    const res = await context.octokit.orgs.checkMembershipForUser({
-      org: "LedgerHQ",
-      username: login,
-    });
+    if (!number) return;
 
-    if (res.status >= 300) return;
+    try {
+      const res = await context.octokit.orgs.checkMembershipForUser({
+        org: "LedgerHQ",
+        username: login,
+      });
 
-    if (!number) return;
+      if (res.status >= 300) return;
+    } catch (error) {
+      // Ignore error
+    }
 
     await triggerWorkflow({
       context,

diff --git a/tools/github-bot/src/commands/regen-doc.ts b/tools/github-bot/src/commands/regen-doc.ts
@@ -69,14 +69,18 @@ export function regenDoc(app: Probot) {
     const number = payload.check_run.pull_requests[0]?.number;
     const login = payload.sender.login;
 
-    const res = await context.octokit.orgs.checkMembershipForUser({
-      org: "LedgerHQ",
-      username: login,
-    });
+    if (!number) return;
 
-    if (res.status >= 300) return;
+    try {
+      const res = await context.octokit.orgs.checkMembershipForUser({
+        org: "LedgerHQ",
+        username: login,
+      });
 
-    if (!number) return;
+      if (res.status >= 300) return;
+    } catch (error) {
+      // Ignore error
+    }
 
     await triggerWorkflow({
       context,

diff --git a/tools/github-bot/src/commands/regen-pods.ts b/tools/github-bot/src/commands/regen-pods.ts
@@ -69,14 +69,18 @@ export function regenPods(app: Probot) {
     const number = payload.check_run.pull_requests[0]?.number;
     const login = payload.sender.login;
 
-    const res = await context.octokit.orgs.checkMembershipForUser({
-      org: "LedgerHQ",
-      username: login,
-    });
+    if (!number) return;
 
-    if (res.status >= 300) return;
+    try {
+      const res = await context.octokit.orgs.checkMembershipForUser({
+        org: "LedgerHQ",
+        username: login,
+      });
 
-    if (!number) return;
+      if (res.status >= 300) return;
+    } catch (error) {
+      // Ignore error
+    }
 
     await triggerWorkflow({
       context,

diff --git a/tools/github-bot/src/commands/tools.ts b/tools/github-bot/src/commands/tools.ts
@@ -33,6 +33,9 @@ export const commands = (
     const command = comment.body.match(matcher);
     const login = payload.sender.login;
 
+    if (!isPR(issue)) return;
+    if (!command || (command && command[1] !== name)) return;
+
     try {
       const res = await octokit.orgs.checkMembershipForUser({
         org: "LedgerHQ",
@@ -66,9 +69,6 @@ export const commands = (
       return;
     }
 
-    if (!isPR(issue)) return;
-    if (!command || (command && command[1] !== name)) return;
-
     await octokit.rest.reactions.createForIssueComment({
       ...context.repo(),
       comment_id: comment.id,