github-repository-provider 7.24.9 → 7.25.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "github-repository-provider",
-  "version": "7.24.9",
+  "version": "7.25.2",
   "publishConfig": {
     "access": "public"
   },
@@ -32,17 +32,17 @@
   "dependencies": {
     "content-entry": "^3.0.2",
     "fetch-link-util": "^1.0.4",
-    "fetch-rate-limit-util": "^1.1.6",
+    "fetch-rate-limit-util": "^2.1.0",
     "matching-iterator": "^2.0.0",
     "node-fetch": "3.1.0",
     "one-time-execution-method": "^2.0.9",
-    "repository-provider": "^26.0.3"
+    "repository-provider": "^26.1.1"
   },
   "devDependencies": {
     "ava": "^3.15.0",
     "c8": "^7.11.0",
     "documentation": "^13.2.5",
-    "repository-provider-test-support": "^1.9.0",
+    "repository-provider-test-support": "^1.9.2",
     "semantic-release": "^18.0.1"
   },
   "engines": {

package/src/github-branch.mjs

@@ -34,17 +34,6 @@ export class GithubBranch extends Branch {
     return entry;
   }
 
-  /**
-   * {@link https://developer.github.com/v3/git/commits/#get-a-commit}
-   * @param {string} sha
-   */
-  async baseTreeSha(sha) {
-    const { json } = await this.provider.fetchJSON(
-      `repos/${this.slug}/git/commits/${sha}`
-    );
-    return json.tree.sha;
-  }
-
   /**
    * {@link https://developer.github.com/v3/git/trees/#create-a-tree}
    * {@link https://developer.github.com/v3/git/commits/#create-a-commit}
@@ -135,6 +124,17 @@ export class GithubBranch extends Branch {
     }
   }
 
+  /**
+   * {@link https://developer.github.com/v3/git/commits/#get-a-commit}
+   * @param {string} sha
+   */
+  async baseTreeSha(sha) {
+    const { json } = await this.provider.fetchJSON(
+      `repos/${this.slug}/git/commits/${sha}`
+    );
+    return json.tree.sha;
+  }
+
   /**
    * @see https://developer.github.com/v3/git/trees/
    * @param {string } treeSha

package/src/github-provider.mjs

@@ -1,6 +1,6 @@
 import fetch from "node-fetch";
 import { replaceWithOneTimeExecutionMethod } from "one-time-execution-method";
-import { rateLimitHandler, defaultWaitDecide } from "fetch-rate-limit-util";
+import { stateActionHandler } from "fetch-rate-limit-util";
 
 import { MultiGroupProvider } from "repository-provider";
 import { GithubRepository } from "./github-repository.mjs";
@@ -9,7 +9,7 @@ import { GithubOwner } from "./github-owner.mjs";
 import { GithubPullRequest } from "./github-pull-request.mjs";
 export { GithubRepository, GithubBranch, GithubOwner, GithubPullRequest };
 
-const domain = "github.com";
+const host = "github.com";
 
 /**
  * <!-- skip-example -->
@@ -42,31 +42,40 @@ export class GithubProvider extends MultiGroupProvider {
    * @return {string} default instance environment name prefix
    */
   static get instanceIdentifier() {
-    return "GITHUB_";
+    return "GITHUB_"; // "GH_" "GH_ENTERPRISE_"
   }
 
   static get attributes() {
     return {
       ...super.attributes,
+      host: {
+        type: "string",
+        env: ["{{instanceIdentifier}}HOST", "GH_HOST"],
+        default: "github.com"
+      },
       ssh: {
         type: "url",
-        default: `git@${domain}:`
+        default: `git@${host}:`
       },
       url: {
         type: "url",
         env: ["{{instanceIdentifier}}SERVER_URL"],
         set: value => (value.endsWith("/") ? value : value + "/"),
-        default: `https://${domain}/`
+        default: `https://${host}/`
       },
       api: {
         type: "url",
         env: ["{{instanceIdentifier}}API_URL"],
         set: value => value.replace(/\/$/, ""),
-        default: `https://api.${domain}`
+        default: `https://api.${host}`
       },
       "authentication.token": {
         type: "string",
-        env: ["{{instanceIdentifier}}TOKEN", "GH_TOKEN"],
+        // @see https://cli.github.com/manual/gh_help_environment
+        env: [
+          "{{instanceIdentifier}}TOKEN",
+          "GH_TOKEN" // declare GH_ as identifier
+        ],
         additionalAttributes: { "authentication.type": "token" },
         private: true,
         mandatory: true
@@ -89,31 +98,13 @@ export class GithubProvider extends MultiGroupProvider {
   }
 
   fetch(url, options = {}) {
-    return rateLimitHandler(
-      () =>
-        fetch(new URL(url, this.api), {
-          ...options,
-          headers: {
-            authorization: `token ${this.authentication.token}`,
-            ...options.headers
-          }
-        }),
-      (millisecondsToWait, rateLimitRemaining, nthTry, response) => {
-        this.rateLimitRemaining = rateLimitRemaining;
-
-        const msecs = defaultWaitDecide(
-          millisecondsToWait,
-          rateLimitRemaining,
-          nthTry,
-          response
-        );
-
-        if (msecs > 0) {
-          this.warn(`Rate limit reached: waiting for ${msecs / 1000}s`);
-        }
-        return msecs;
+    return stateActionHandler(fetch, new URL(url, this.api), {
+      ...options,
+      headers: {
+        authorization: `token ${this.authentication.token}`,
+        ...options.headers
       }
-    );
+    });
   }
 
   async fetchJSON(url, options) {
@@ -186,9 +177,9 @@ export class GithubProvider extends MultiGroupProvider {
     return super.repositoryBases.concat([
       this.url,
       "git+" + this.url,
-      `git+ssh://${domain}`,
-      `git://${domain}/`,
-      `git@${domain}:`
+      `git+ssh://${host}`,
+      `git://${host}/`,
+      `git@${host}:`
     ]);
   }