github-repository-provider 7.24.8 → 7.24.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "github-repository-provider",
-  "version": "7.24.8",
+  "version": "7.24.9",
   "publishConfig": {
     "access": "public"
   },

package/src/github-branch.mjs

@@ -18,13 +18,16 @@ export class GithubBranch extends Branch {
    * @return {Promise<ConentEntry>} written content with sha values set
    */
   async writeEntry(entry) {
-    const json = await this.provider.fetchJSON(`repos/${this.slug}/git/blobs`, {
-      method: "POST",
-      body: JSON.stringify({
-        content: await entry.string,
-        encoding: "utf8"
-      })
-    });
+    const { json } = await this.provider.fetchJSON(
+      `repos/${this.slug}/git/blobs`,
+      {
+        method: "POST",
+        body: JSON.stringify({
+          content: await entry.string,
+          encoding: "utf8"
+        })
+      }
+    );
 
     entry.sha = json.sha;
 
@@ -36,7 +39,7 @@ export class GithubBranch extends Branch {
    * @param {string} sha
    */
   async baseTreeSha(sha) {
-    const json = await this.provider.fetchJSON(
+    const { json } = await this.provider.fetchJSON(
       `repos/${this.slug}/git/commits/${sha}`
     );
     return json.tree.sha;
@@ -66,24 +69,27 @@ export class GithubBranch extends Branch {
     const shaLatestCommit = await this.refId();
     const shaBaseTree = await this.baseTreeSha(shaLatestCommit);
 
-    let json = await this.provider.fetchJSON(`repos/${this.slug}/git/trees`, {
-      method: "POST",
-      body: JSON.stringify({
-        base_tree: shaBaseTree,
-        tree: updates.map(u => {
-          return {
-            path: u.name,
-            sha: u.sha,
-            type: "blob",
-            mode: "100" + u.mode.toString(8)
-          };
+    let { json } = await this.provider.fetchJSON(
+      `repos/${this.slug}/git/trees`,
+      {
+        method: "POST",
+        body: JSON.stringify({
+          base_tree: shaBaseTree,
+          tree: updates.map(u => {
+            return {
+              path: u.name,
+              sha: u.sha,
+              type: "blob",
+              mode: "100" + u.mode.toString(8)
+            };
+          })
         })
-      })
-    });
+      }
+    );
 
     const shaNewTree = json.sha;
 
-    json = await this.provider.fetchJSON(`repos/${this.slug}/git/commits`, {
+    const r = await this.provider.fetchJSON(`repos/${this.slug}/git/commits`, {
       method: "POST",
       body: JSON.stringify({
         message,
@@ -92,7 +98,7 @@ export class GithubBranch extends Branch {
       })
     });
 
-    const sha = json.sha;
+    const sha = r.json.sha;
 
     return await this.provider.fetchJSON(
       `repos/${this.slug}/git/refs/heads/${this.name}`,
@@ -111,7 +117,7 @@ export class GithubBranch extends Branch {
    * @param {string} name
    */
   async entry(name) {
-    const json = await this.provider.fetchJSON(
+    const { json } = await this.provider.fetchJSON(
       `repos/${this.slug}/contents/${name}?ref=${this.ref}`
     );
 
@@ -135,7 +141,7 @@ export class GithubBranch extends Branch {
    * @return {Object[]}
    */
   async tree(treeSha) {
-    const json = await this.provider.fetchJSON(
+    const { json } = await this.provider.fetchJSON(
       `repos/${this.slug}/git/trees/${treeSha}?recursive=1`
     );
     return json.tree;
@@ -187,7 +193,7 @@ class LazyBufferContentEntry extends BufferContentEntryMixin(ContentEntry) {
     }
 
     const branch = this.branch;
-    const json = await branch.provider.fetchJSON(
+    const { json } = await branch.provider.fetchJSON(
       `repos/${branch.slug}/contents/${this.name}?ref=${branch.ref}`
     );
 

package/src/github-owner.mjs

@@ -39,12 +39,12 @@ export class GithubOwner extends RepositoryGroup {
     );
 
     if (response.ok) {
-      this.provider.info(`Repository ${name} created`);
+      this.info(`Repository ${name} created`);
       options.defaultBranchName = "main";
       return this.addRepository(name, options);
     }
 
-    this.provider.error(`Repository ${name} creation error ${response.status}`);
+    this.error(`Repository ${name} creation error ${response.status}`);
   }
 
   /**

package/src/github-provider.mjs

@@ -117,18 +117,24 @@ export class GithubProvider extends MultiGroupProvider {
   }
 
   async fetchJSON(url, options) {
-    let i = 1;
-    while (true) {
+    for (let i = 1; ; i++) {
       try {
         const response = await this.fetch(url, options);
-        if (!response.ok) {
+        if (response.ok) {
+          return { response, json: await response.json() };
+        }
+
+        if (i >= 3 || response.status == 401) {
+          // none repeatable
           throw new Error(
-            `Unable to fetch ${response.url} (${response.status}) try #${i}`
+            `Unable to fetch ${response.url} (${response.status})`
           );
         }
-        return await response.json();
+        this.info(
+          `Unable to fetch ${response.url} (${response.status}) try #${i}`
+        );
       } catch (e) {
-        if (i++ >= 3) {
+        if (i >= 3) {
           throw e;
         }
         this.error(e);
@@ -140,9 +146,9 @@ export class GithubProvider extends MultiGroupProvider {
    * {@link https://developer.github.com/v3/repos/#list-repositories-for-the-authenticated-user}
    */
   async initializeRepositories() {
-    for (let page = 1; ; page++) {
-      try {
-        const json = await this.fetchJSON(
+    try {
+      for (let page = 1; ; page++) {
+        const { json } = await this.fetchJSON(
           `user/repos?page=${page}&per_page=100`,
           {
             headers: {
@@ -162,10 +168,8 @@ export class GithubProvider extends MultiGroupProvider {
             r
           );
         });
-      } catch (e) {
-        return;
       }
-    }
+    } catch {}
   }
 
   /**

package/src/github-pull-request.mjs

@@ -43,8 +43,8 @@ export class GithubPullRequest extends PullRequest {
 
       do {
         const provider = repository.provider;
-        const response = await provider.fetch(next);
-        for (const node of await response.json()) {
+        const { response, json } = await provider.fetchJSON(next);
+        for (const node of json) {
           const [source, dest] = await Promise.all(
             [node.head, node.base].map(r =>
               provider.branch([r.repo.full_name, r.ref].join("#"))
@@ -72,7 +72,7 @@ export class GithubPullRequest extends PullRequest {
       return p;
     }
 
-    const json = await destination.provider.fetchJSON(
+    const { json } = await destination.provider.fetchJSON(
       `repos/${destination.repository.slug}/pulls`,
       {
         method: "POST",

package/src/github-repository.mjs

@@ -37,16 +37,7 @@ export class GithubRepository extends Repository {
     let next = `repos/${this.slug}/${typeName}`;
 
     do {
-      const response = await this.provider.fetch(next);
-
-      if (!response.ok) {
-        this.error(
-          `Unable to fetch ${typeName} ${response.status} ${response.url}`
-        );
-        return;
-      }
-
-      const json = await response.json();
+      const { response, json } = await this.provider.fetchJSON(next);
       json.forEach(b => this[addMethodName](b.name, b));
       next = getHeaderLink(response.headers);
     } while (next);
@@ -112,7 +103,9 @@ export class GithubRepository extends Repository {
   async refId(ref) {
     ref = ref.replace(/^refs\//, "");
 
-    const json = await this.provider.fetchJSON(`repos/${this.slug}/git/ref/${ref}`);
+    const { json } = await this.provider.fetchJSON(
+      `repos/${this.slug}/git/ref/${ref}`
+    );
 
     // TODO why does this happen ?
     if (!json.object.sha) {
@@ -147,7 +140,7 @@ export class GithubRepository extends Repository {
       console.log(res);
       */
     } else {
-      const json = await this.provider.fetchJSON(
+      const { json } = await this.provider.fetchJSON(
         `repos/${this.slug}/git/ref/heads/${
           from === undefined ? this.defaultBranchName : from.name
         }`
@@ -204,9 +197,9 @@ export class GithubRepository extends Repository {
     let next = `repos/${this.slug}/hooks`;
 
     do {
-      const response = await this.provider.fetch(next);
+      const { response, json } = await this.provider.fetchJSON(next);
 
-      for (const h of await response.json()) {
+      for (const h of json) {
         const id = h.id;
         delete h.id;
         new this.hookClass(this, id, new Set(h.events), {