github-repository-provider 7.24.4 → 7.24.5

package/package.json

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

package/src/github-branch.mjs

@@ -36,15 +36,10 @@ export class GithubBranch extends Branch {
    * @param {string} sha
    */
   async baseTreeSha(sha) {
-    const res = await this.provider.fetch(
+    const json = await this.provider.fetchJSON(
       `repos/${this.slug}/git/commits/${sha}`
     );
-    if (res.ok) {
-      const json = await res.json();
-      return json.tree.sha;
-    }
-
-    throw new Error(`Unable to decode ${res.url}`);
+    return json.tree.sha;
   }
 
   /**
@@ -116,16 +111,11 @@ export class GithubBranch extends Branch {
    * @param {string} name
    */
   async entry(name) {
-    const res = await this.provider.fetch(
+    const json = await this.provider.fetchJSON(
       `repos/${this.slug}/contents/${name}?ref=${this.ref}`
     );
 
-    if (res.ok) {
-      const json = await res.json();
-      return new this.entryClass(name, Buffer.from(json.content, "base64"));
-    }
-
-    throw new Error(res.status);
+    return new this.entryClass(name, Buffer.from(json.content, "base64"));
   }
 
   /** @inheritdoc */
@@ -158,7 +148,7 @@ export class GithubBranch extends Branch {
       } catch (e) {
         // errno: 'ERR_STREAM_PREMATURE_CLOSE',
         // code: 'ERR_STREAM_PREMATURE_CLOSE',
-        
+
         this.error(e);
       }
     }
@@ -214,11 +204,10 @@ class LazyBufferContentEntry extends BufferContentEntryMixin(ContentEntry) {
     }
 
     const branch = this.branch;
-    const res = await branch.provider.fetch(
+    const json = await branch.provider.fetchJSON(
       `repos/${branch.slug}/contents/${this.name}?ref=${branch.ref}`
     );
 
-    const json = await res.json();
     this._buffer = Buffer.from(json.content, "base64");
     return this._buffer;
   }

package/src/github-provider.mjs

@@ -121,8 +121,7 @@ export class GithubProvider extends MultiGroupProvider {
       try {
         const response = await this.fetch(url, options);
         if (!response.ok) {
-          this.error(`Unable to fetch ${response.status} ${response.url}`);
-          throw new Error(`Unable to fetch ${response.status} ${response.url}`);
+          throw new Error(`Unable to fetch ${response.url} (${response.status})`);
         }
         return await response.json();
       } catch (e) {

package/src/github-pull-request.mjs

@@ -72,7 +72,7 @@ export class GithubPullRequest extends PullRequest {
       return p;
     }
 
-    const res = await destination.provider.fetch(
+    const json = await destination.provider.fetchJSON(
       `repos/${destination.repository.slug}/pulls`,
       {
         method: "POST",
@@ -84,13 +84,7 @@ export class GithubPullRequest extends PullRequest {
       }
     );
 
-    const json = await res.json();
-
-    if (res.ok) {
-      return new this(source, destination, json.number, json);
-    }
-
-    throw new Error(json.errors.map(e => e.message).join(";"));
+    return new this(source, destination, json.number, json);
   }
 
   /**