chore: removed js-yaml and re-enabled octokit plugins

package.json

@@ -87,7 +87,6 @@
     "@swc/jest": "0.2.36",
     "@types/console-log-level": "^1.4.5",
     "@types/jest": "29.5.12",
-    "@types/js-yaml": "^4.0.9",
     "@types/node": "20.14.10",
     "@types/node-rsa": "^1.1.4",
     "azure-functions-core-tools": "^4.0.6610",

src/github/github-client.ts

@@ -1,52 +1,30 @@
 import { Octokit } from "@octokit/core";
 import { requestLog } from "@octokit/plugin-request-log";
-// import { RequestOptions } from "@octokit/types";
+import { RequestOptions } from "@octokit/types";
 import { paginateRest } from "@octokit/plugin-paginate-rest";
 import { restEndpointMethods } from "@octokit/plugin-rest-endpoint-methods";
-// import { retry } from "@octokit/plugin-retry";
-// import { throttling } from "@octokit/plugin-throttling";
+import { retry } from "@octokit/plugin-retry";
+import { throttling } from "@octokit/plugin-throttling";
 import { createAppAuth } from "@octokit/auth-app";
 
 const defaultOptions = {
   authStrategy: createAppAuth,
-  // throttle: {
-  //   onAbuseLimit: (retryAfter: number, options: RequestOptions, octokit: Octokit) => {
-  //     octokit.log.warn(`Abuse limit hit with "${options.method} ${options.url}", retrying in ${retryAfter} seconds.`);
-  //     return true;
-  //   },
-  //   onRateLimit: (retryAfter: number, options: RequestOptions, octokit: Octokit) => {
-  //     octokit.log.warn(`Rate limit hit with "${options.method} ${options.url}", retrying in ${retryAfter} seconds.`);
-  //     return true;
-  //   },
-  //   onSecondaryRateLimit: (retryAfter: number, options: RequestOptions, octokit: Octokit) => {
-  //     octokit.log.warn(`Secondary rate limit hit with "${options.method} ${options.url}", retrying in ${retryAfter} seconds.`);
-  //     return true;
-  //   },
-  // },
+  throttle: {
+    onAbuseLimit: (retryAfter: number, options: RequestOptions, octokit: Octokit) => {
+      octokit.log.warn(`Abuse limit hit with "${options.method} ${options.url}", retrying in ${retryAfter} seconds.`);
+      return true;
+    },
+    onRateLimit: (retryAfter: number, options: RequestOptions, octokit: Octokit) => {
+      octokit.log.warn(`Rate limit hit with "${options.method} ${options.url}", retrying in ${retryAfter} seconds.`);
+      return true;
+    },
+    onSecondaryRateLimit: (retryAfter: number, options: RequestOptions, octokit: Octokit) => {
+      octokit.log.warn(`Secondary rate limit hit with "${options.method} ${options.url}", retrying in ${retryAfter} seconds.`);
+      return true;
+    },
+  },
 };
 
-// function requestLogging(octokit: Octokit) {
-//   octokit.hook.error("request", (error, options) => {
-//     if ("status" in error) {
-//       const { method, url, body } = octokit.request.endpoint.parse(options);
-//       const msg = `GitHub request: ${method} ${url} - ${error.status}`;
-//
-//       // @ts-expect-error log.debug is a pino log method and accepts a fields object
-//       octokit.log.debug(body || {}, msg);
-//     }
-//
-//     throw error;
-//   });
-//
-//   octokit.hook.after("request", (result, options) => {
-//     const { method, url, body } = octokit.request.endpoint.parse(options);
-//     const msg = `GitHub request: ${method} ${url} - ${result.status}`;
-//
-//     // @ts-expect-error log.debug is a pino log method and accepts a fields object
-//     octokit.log.debug(body || {}, msg);
-//   });
-// }
-
-export const customOctokit = Octokit.plugin(paginateRest, restEndpointMethods, requestLog).defaults((instanceOptions: object) => {
+export const customOctokit = Octokit.plugin(paginateRest, restEndpointMethods, requestLog, retry, throttling).defaults((instanceOptions: object) => {
   return Object.assign({}, defaultOptions, instanceOptions);
 });