github-repository-provider 7.26.20 → 7.26.23

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "github-repository-provider",
-  "version": "7.26.20",
+  "version": "7.26.23",
   "publishConfig": {
     "access": "public"
   },
@@ -32,17 +32,17 @@
   "dependencies": {
     "content-entry": "^5.0.0",
     "fetch-link-util": "^1.0.8",
-    "fetch-rate-limit-util": "^2.9.5",
+    "fetch-rate-limit-util": "^2.10.0",
     "matching-iterator": "^2.0.4",
-    "node-fetch": "^3.2.3",
+    "node-fetch": "^3.2.4",
     "one-time-execution-method": "^2.0.13",
-    "repository-provider": "^28.3.0"
+    "repository-provider": "^28.3.5"
   },
   "devDependencies": {
     "ava": "^4.2.0",
     "c8": "^7.11.2",
     "documentation": "^13.2.5",
-    "repository-provider-test-support": "^2.1.5",
+    "repository-provider-test-support": "^2.1.8",
     "semantic-release": "^19.0.2"
   },
   "engines": {

package/src/github-provider.mjs

@@ -92,7 +92,7 @@ export class GithubProvider extends MultiGroupProvider {
     };
   }
 
-  fetch(url, options = {}, responseHandler) {
+  fetch(url, options = {}, responseHandler, actions) {
     return stateActionHandler(
       fetch,
       new URL(url, this.api),
@@ -104,15 +104,20 @@ export class GithubProvider extends MultiGroupProvider {
         }
       },
       responseHandler,
-      undefined,
+      actions,
       (url, ...args) => this.trace(url.toString(), ...args)
     );
   }
 
-  fetchJSON(url, options) {
-    return this.fetch(url, options, async response => {
-      return { response, json: await response.json() };
-    });
+  fetchJSON(url, options, actions) {
+    return this.fetch(
+      url,
+      options,
+      async response => {
+        return { response, json: await response.json() };
+      },
+      actions
+    );
   }
 
   /**

package/src/github-repository.mjs

@@ -1,6 +1,12 @@
 import { replaceWithOneTimeExecutionMethod } from "one-time-execution-method";
 import { Repository } from "repository-provider";
 import { getHeaderLink } from "fetch-link-util";
+import { defaultStateActions, errorHandler } from "fetch-rate-limit-util";
+
+const confictErrorActions = {
+  ...defaultStateActions,
+  409: errorHandler
+};
 
 /**
  * Repository on GitHub.
@@ -39,7 +45,7 @@ export class GithubRepository extends Repository {
 
   /**
    * {@link https://docs.github.com/en/rest/reference/commits#list-commits}
-   * @param {Object} options 
+   * @param {Object} options
    * @returns {AsyncIterator<Commit>}
    */
   async *commits(options) {
@@ -48,12 +54,12 @@ export class GithubRepository extends Repository {
     do {
       const { response, json } = await this.provider.fetchJSON(next);
 
-      for(const c of json) {
+      for (const c of json) {
         yield {
           sha: c.sha,
           message: c.message,
           author: c.author,
-          committer: c.committer,
+          committer: c.committer
         };
       }
       next = getHeaderLink(response.headers);
@@ -140,8 +146,11 @@ export class GithubRepository extends Repository {
       this._ref = new Map();
     }
 
+    // TODO: 409 -> none repeatable
     const { response, json } = await this.provider.fetchJSON(
-      `repos/${this.slug}/git/ref/${ref}`
+      `repos/${this.slug}/git/ref/${ref}`,
+      undefined,
+      confictErrorActions
     );
 
     // TODO why does this happen ?
@@ -181,7 +190,9 @@ export class GithubRepository extends Repository {
       console.log(res);
       */
     } else {
-      sha = await this.refId(`heads/${from ? from.name : this.defaultBranchName}` );
+      sha = await this.refId(
+        `heads/${from ? from.name : this.defaultBranchName}`
+      );
     }
 
     const res = await this.provider.fetch(`repos/${this.slug}/git/refs`, {
@@ -236,9 +247,7 @@ export class GithubRepository extends Repository {
       const { response, json } = await this.provider.fetchJSON(next);
 
       for (const h of json) {
-        const id = h.id;
-        delete h.id;
-        new this.hookClass(this, id, new Set(h.events), {
+        new this.hookClass(this, h.name, new Set(h.events), {
           ...h,
           ...h.config
         });