github-repository-provider 7.30.20 → 7.30.23

package/README.md

@@ -38,17 +38,15 @@ console.log(entry.name);
         *   [Parameters](#parameters)
     *   [commit](#commit)
         *   [Parameters](#parameters-1)
-    *   [commitForSha](#commitforsha)
-        *   [Parameters](#parameters-2)
     *   [entry](#entry)
-        *   [Parameters](#parameters-3)
+        *   [Parameters](#parameters-2)
     *   [removeEntries](#removeentries)
-        *   [Parameters](#parameters-4)
+        *   [Parameters](#parameters-3)
 *   [GithubOwner](#githubowner)
     *   [createRepository](#createrepository)
-        *   [Parameters](#parameters-5)
+        *   [Parameters](#parameters-4)
     *   [deleteRepository](#deleterepository)
-        *   [Parameters](#parameters-6)
+        *   [Parameters](#parameters-5)
     *   [attributeMapping](#attributemapping)
 *   [GithubProvider](#githubprovider)
     *   [Examples](#examples)
@@ -58,16 +56,18 @@ console.log(entry.name);
     *   [instanceIdentifier](#instanceidentifier)
 *   [GithubPullRequest](#githubpullrequest)
     *   [\_merge](#\_merge)
-        *   [Parameters](#parameters-7)
+        *   [Parameters](#parameters-6)
     *   [update](#update)
     *   [validMergeMethods](#validmergemethods)
     *   [list](#list)
-        *   [Parameters](#parameters-8)
+        *   [Parameters](#parameters-7)
     *   [open](#open)
-        *   [Parameters](#parameters-9)
+        *   [Parameters](#parameters-8)
 *   [GithubRepository](#githubrepository)
     *   [defaultBranchName](#defaultbranchname)
     *   [commits](#commits)
+        *   [Parameters](#parameters-9)
+    *   [commitForSha](#commitforsha)
         *   [Parameters](#parameters-10)
     *   [tree](#tree)
         *   [Parameters](#parameters-11)
@@ -82,8 +82,10 @@ console.log(entry.name);
     *   [update](#update-1)
     *   [refId](#refid)
         *   [Parameters](#parameters-13)
-    *   [deletePullRequest](#deletepullrequest)
+    *   [setRefId](#setrefid)
         *   [Parameters](#parameters-14)
+    *   [deletePullRequest](#deletepullrequest)
+        *   [Parameters](#parameters-15)
     *   [initializeHooks](#initializehooks)
 
 ## GithubBranch
@@ -115,16 +117,6 @@ Returns **[Promise](https://developer.mozilla.org/docs/Web/JavaScript/Reference/
 *   `entries` **[Array](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Array)\<ContentEntry>** 
 *   `options` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** 
 
-### commitForSha
-
-<https://developer.github.com/v3/git/commits/#get-a-commit>
-
-#### Parameters
-
-*   `sha` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
-
-Returns **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** response
-
 ### entry
 
 <https://developer.github.com/v3/repos/contents/#get-repository-content>
@@ -288,6 +280,16 @@ Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/G
 
 Returns **AsyncIterator\<Commit>** 
 
+### commitForSha
+
+<https://developer.github.com/v3/git/commits/#get-a-commit>
+
+#### Parameters
+
+*   `sha` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
+
+Returns **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** response
+
 ### tree
 
 *   **See**: <https://developer.github.com/v3/git/trees/>
@@ -354,6 +356,18 @@ Get sha of a ref.
 
 Returns **[Promise](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Promise)<[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)>** sha of the ref
 
+### setRefId
+
+<https://docs.github.com/en/github-ae@latest/rest/git/refs#update-a-reference>
+
+#### Parameters
+
+*   `ref` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
+*   `sha` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
+*   `options` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** 
+
+Returns **any** 
+
 ### deletePullRequest
 
 <https://developer.github.com/v3/pulls/#update-a-pull-request>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "github-repository-provider",
-  "version": "7.30.20",
+  "version": "7.30.23",
   "publishConfig": {
     "access": "public"
   },
@@ -36,7 +36,7 @@
     "matching-iterator": "^2.0.6",
     "node-fetch": "^3.2.6",
     "one-time-execution-method": "^3.0.3",
-    "repository-provider": "^31.2.7"
+    "repository-provider": "^31.2.9"
   },
   "devDependencies": {
     "ava": "^4.3.0",
@@ -45,7 +45,7 @@
     "etag-cache-leveldb": "^1.2.4",
     "leveldown": "^6.1.1",
     "levelup": "^5.1.1",
-    "repository-provider-test-support": "^2.2.5",
+    "repository-provider-test-support": "^2.2.6",
     "semantic-release": "^19.0.3"
   },
   "engines": {

package/src/github-branch.mjs

@@ -11,7 +11,6 @@ import {
  */
 export class GithubBranch extends Branch {
   #entries = new Map();
-  #commits = new Map();
 
   /**
    * Writes content into the branch
@@ -44,10 +43,6 @@ export class GithubBranch extends Branch {
    * @param {Object} options
    */
   async commit(message, entries, options) {
-    const updates = await Promise.all(
-      entries.map(entry => this.writeEntry(entry))
-    );
-
     /*
      * 1st. commit on empty tree
      * https://stackoverflow.com/questions/9765453/is-gits-semi-secret-empty-tree-object-reliable-and-why-is-there-not-a-symbolic/9766506#9766506
@@ -56,42 +51,20 @@ export class GithubBranch extends Branch {
      * sha256:6ef19b41225c5369f1c104d45d8d85efa9b057b53b14b4b9b939dd74decc5321
      */
 
-    const shaLatestCommit = await this.refId();
-    const commit = await this.commitForSha(shaLatestCommit);
-    const tree = await this.owner.addTree(updates, commit.tree.sha);
-
-    let r = await this.provider.fetchJSON(`${this.api}/git/commits`, {
-      method: "POST",
-      body: JSON.stringify({
-        message,
-        tree: tree.sha,
-        parents: [shaLatestCommit]
-      })
-    });
-
-    this.#commits.set(r.json.sha, r.json);
-
-    return this.owner.setRefId(this.ref, r.json.sha, options);
-  }
-
-  /**
-   * {@link https://developer.github.com/v3/git/commits/#get-a-commit}
-   * @param {string} sha
-   * @return {Object} response
-   */
-  async commitForSha(sha) {
-    const commit = this.#commits.get(sha);
-    if (commit) {
-      return commit;
-    }
-
-    const { json } = await this.provider.fetchJSON(
-      `${this.api}/git/commits/${sha}`
+    const updates = await Promise.all(
+      entries.map(entry => this.writeEntry(entry))
     );
 
-    this.#commits.set(sha, json);
+    const shaLatestCommit = await this.refId();
+    const latestCommit = await this.owner.commitForSha(shaLatestCommit);
+    const tree = await this.owner.addTree(updates, latestCommit.tree.sha);
+    const commit = await this.owner.addCommit(
+      tree.sha,
+      [shaLatestCommit],
+      message
+    );
 
-    return json;
+    return this.owner.setRefId(this.ref, commit.sha, options);
   }
 
   /**
@@ -126,7 +99,7 @@ export class GithubBranch extends Branch {
   }
 
   async *entries(patterns) {
-    const commit = await this.commitForSha(await this.refId());
+    const commit = await this.owner.commitForSha(await this.refId());
 
     for (const entry of matcher(
       await this.owner.tree(commit.tree.sha),

package/src/github-repository.mjs

@@ -14,6 +14,7 @@ const conflictErrorActions = {
 export class GithubRepository extends Repository {
   #refs = new Map();
   #trees = new Map();
+  #commits = new Map();
 
   static get attributeMapping() {
     return {
@@ -62,6 +63,8 @@ export class GithubRepository extends Repository {
       const { response, json } = await this.provider.fetchJSON(next);
 
       for (const c of json) {
+        this.#commits.set(c.sha, json);
+
         yield {
           sha: c.sha,
           message: c.message,
@@ -73,6 +76,40 @@ export class GithubRepository extends Repository {
     } while (next);
   }
 
+  async addCommit(tree, parents, message) {
+    let r = await this.provider.fetchJSON(`${this.api}/git/commits`, {
+      method: "POST",
+      body: JSON.stringify({
+        tree,
+        parents,
+        message
+      })
+    });
+
+    this.#commits.set(r.json.sha, r.json);
+    return r.json;
+  }
+
+  /**
+   * {@link https://developer.github.com/v3/git/commits/#get-a-commit}
+   * @param {string} sha
+   * @return {Object} response
+   */
+  async commitForSha(sha) {
+    const commit = this.#commits.get(sha);
+    if (commit) {
+      return commit;
+    }
+
+    const { json } = await this.provider.fetchJSON(
+      `${this.api}/git/commits/${sha}`
+    );
+
+    this.#commits.set(sha, json);
+
+    return json;
+  }
+
   /**
    * @see https://developer.github.com/v3/git/trees/
    * @param {string} sha