github-repository-provider 7.30.17 → 7.30.20

package/README.md

@@ -38,19 +38,17 @@ console.log(entry.name);
         *   [Parameters](#parameters)
     *   [commit](#commit)
         *   [Parameters](#parameters-1)
-    *   [entry](#entry)
-        *   [Parameters](#parameters-2)
     *   [commitForSha](#commitforsha)
+        *   [Parameters](#parameters-2)
+    *   [entry](#entry)
         *   [Parameters](#parameters-3)
-    *   [tree](#tree)
-        *   [Parameters](#parameters-4)
     *   [removeEntries](#removeentries)
-        *   [Parameters](#parameters-5)
+        *   [Parameters](#parameters-4)
 *   [GithubOwner](#githubowner)
     *   [createRepository](#createrepository)
-        *   [Parameters](#parameters-6)
+        *   [Parameters](#parameters-5)
     *   [deleteRepository](#deleterepository)
-        *   [Parameters](#parameters-7)
+        *   [Parameters](#parameters-6)
     *   [attributeMapping](#attributemapping)
 *   [GithubProvider](#githubprovider)
     *   [Examples](#examples)
@@ -60,17 +58,21 @@ console.log(entry.name);
     *   [instanceIdentifier](#instanceidentifier)
 *   [GithubPullRequest](#githubpullrequest)
     *   [\_merge](#\_merge)
-        *   [Parameters](#parameters-8)
+        *   [Parameters](#parameters-7)
     *   [update](#update)
     *   [validMergeMethods](#validmergemethods)
     *   [list](#list)
-        *   [Parameters](#parameters-9)
+        *   [Parameters](#parameters-8)
     *   [open](#open)
-        *   [Parameters](#parameters-10)
+        *   [Parameters](#parameters-9)
 *   [GithubRepository](#githubrepository)
     *   [defaultBranchName](#defaultbranchname)
     *   [commits](#commits)
+        *   [Parameters](#parameters-10)
+    *   [tree](#tree)
         *   [Parameters](#parameters-11)
+    *   [addTree](#addtree)
+        *   [Parameters](#parameters-12)
     *   [initializeBranches](#initializebranches)
     *   [initializeTags](#initializetags)
     *   [url](#url)
@@ -79,9 +81,9 @@ console.log(entry.name);
     *   [api](#api)
     *   [update](#update-1)
     *   [refId](#refid)
-        *   [Parameters](#parameters-12)
-    *   [deletePullRequest](#deletepullrequest)
         *   [Parameters](#parameters-13)
+    *   [deletePullRequest](#deletepullrequest)
+        *   [Parameters](#parameters-14)
     *   [initializeHooks](#initializehooks)
 
 ## GithubBranch
@@ -113,14 +115,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)** 
 
-### entry
-
-<https://developer.github.com/v3/repos/contents/#get-repository-content>
-
-#### Parameters
-
-*   `name` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
-
 ### commitForSha
 
 <https://developer.github.com/v3/git/commits/#get-a-commit>
@@ -131,15 +125,13 @@ Returns **[Promise](https://developer.mozilla.org/docs/Web/JavaScript/Reference/
 
 Returns **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** response
 
-### tree
+### entry
 
-*   **See**: <https://developer.github.com/v3/git/trees/>
+<https://developer.github.com/v3/repos/contents/#get-repository-content>
 
 #### Parameters
 
-*   `sha` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
-
-Returns **[Array](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Array)<[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)>** 
+*   `name` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
 
 ### removeEntries
 
@@ -296,6 +288,27 @@ Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/G
 
 Returns **AsyncIterator\<Commit>** 
 
+### tree
+
+*   **See**: <https://developer.github.com/v3/git/trees/>
+
+#### Parameters
+
+*   `sha` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
+
+Returns **[Array](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Array)<[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)>** 
+
+### addTree
+
+*   **See**: <https://developer.github.com/v3/git/trees/>
+
+#### Parameters
+
+*   `updates` **[Array](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Array)<[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)>** 
+*   `base` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** base tree sha
+
+Returns **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** newly created tree
+
 ### initializeBranches
 
 <https://developer.github.com/v3/repos/branches/#list-branches>

package/package.json

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

package/src/github-branch.mjs

@@ -11,7 +11,6 @@ import {
  */
 export class GithubBranch extends Branch {
   #entries = new Map();
-  #trees = new Map();
   #commits = new Map();
 
   /**
@@ -59,30 +58,13 @@ export class GithubBranch extends Branch {
 
     const shaLatestCommit = await this.refId();
     const commit = await this.commitForSha(shaLatestCommit);
-
-    let { json } = await this.provider.fetchJSON(`${this.api}/git/trees`, {
-      method: "POST",
-      body: JSON.stringify({
-        base_tree: commit.tree.sha,
-        tree: updates.map(u => {
-          return {
-            path: u.name,
-            sha: u.sha,
-            type: "blob",
-            mode: "100" + u.mode.toString(8)
-          };
-        })
-      })
-    });
-
-    const treeSHA = json.sha;
-    this.#trees.set(treeSHA, json);
+    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: treeSHA,
+        tree: tree.sha,
         parents: [shaLatestCommit]
       })
     });
@@ -112,28 +94,6 @@ export class GithubBranch extends Branch {
     return json;
   }
 
-  /**
-   * @see https://developer.github.com/v3/git/trees/
-   * @param {string} sha
-   * @return {Object[]}
-   */
-  async tree(sha) {
-    let tree = this.#trees.get(sha);
-    if (tree) {
-      return tree;
-    }
-
-    const { json } = await this.provider.fetchJSON(
-      `${this.api}/git/trees/${sha}?recursive=1`
-    );
-
-    tree = json.tree;
-
-    this.#trees.set(sha, tree);
-
-    return tree;
-  }
-
   /**
    * {@link https://developer.github.com/v3/repos/contents/#get-repository-content}
    * @param {string} name
@@ -168,9 +128,13 @@ export class GithubBranch extends Branch {
   async *entries(patterns) {
     const commit = await this.commitForSha(await this.refId());
 
-    for (const entry of matcher(await this.tree(commit.tree.sha), patterns, {
-      name: "path"
-    })) {
+    for (const entry of matcher(
+      await this.owner.tree(commit.tree.sha),
+      patterns,
+      {
+        name: "path"
+      }
+    )) {
       switch (entry.type) {
         case "tree":
           yield new BaseCollectionEntry(entry.path);

package/src/github-repository.mjs

@@ -12,6 +12,9 @@ const conflictErrorActions = {
  * Repository on GitHub.
  */
 export class GithubRepository extends Repository {
+  #refs = new Map();
+  #trees = new Map();
+
   static get attributeMapping() {
     return {
       ...super.attributeMapping,
@@ -70,6 +73,55 @@ export class GithubRepository extends Repository {
     } while (next);
   }
 
+  /**
+   * @see https://developer.github.com/v3/git/trees/
+   * @param {string} sha
+   * @return {Object[]}
+   */
+  async tree(sha) {
+    let tree = this.#trees.get(sha);
+    if (tree) {
+      return tree;
+    }
+
+    const { json } = await this.provider.fetchJSON(
+      `${this.api}/git/trees/${sha}?recursive=1`
+    );
+
+    tree = json.tree;
+
+    this.#trees.set(sha, tree);
+
+    return tree;
+  }
+
+  /**
+   * @see https://developer.github.com/v3/git/trees/
+   * @param {Object[]} updates
+   * @param {string} base base tree sha
+   * @returns {Object} newly created tree
+   */
+  async addTree(updates, base) {
+    let { json } = await this.provider.fetchJSON(`${this.api}/git/trees`, {
+      method: "POST",
+      body: JSON.stringify({
+        base_tree: base,
+        tree: updates.map(u => {
+          return {
+            path: u.name,
+            sha: u.sha,
+            type: "blob",
+            mode: "100" + u.mode.toString(8)
+          };
+        })
+      })
+    });
+
+    this.#trees.set(json.sha, json);
+
+    return json;
+  }
+
   async #initializeSlot(typeName, addMethodName) {
     let next = `${this.api}/${typeName}`;
 
@@ -140,8 +192,6 @@ export class GithubRepository extends Repository {
     });
   }
 
-  #ref;
-
   /**
    * Get sha of a ref.
    * {@link https://developer.github.com/v3/git/refs/}
@@ -151,13 +201,9 @@ export class GithubRepository extends Repository {
   async refId(ref) {
     ref = ref.replace(/^refs\//, "");
 
-    if (this.#ref) {
-      const sha = this.#ref.get(ref);
-      if (sha) {
-        return sha;
-      }
-    } else {
-      this.#ref = new Map();
+    let sha = this.#refs.get(ref);
+    if (sha) {
+      return sha;
     }
 
     const { response, json } = await this.provider.fetchJSON(
@@ -169,18 +215,26 @@ export class GithubRepository extends Repository {
       throw new Error(`No refId for '${this.fullName}' '${ref}'`);
     }
 
-    const sha = json.object.sha;
+    sha = json.object.sha;
 
-    this.#ref.set(ref, sha);
+    this.#refs.set(ref, sha);
 
     return sha;
   }
 
-  // TODO belongs into Ref ?
+  /**
+   * {@link https://docs.github.com/en/github-ae@latest/rest/git/refs#update-a-reference}
+   * @param {string} ref
+   * @param {string} sha
+   * @param {Object} options
+   * @returns
+   *
+   * @TODO: belongs into Ref ?
+   */
   async setRefId(ref, sha, options) {
-    //console.log("NEW HEAD", sha, ref);
+    ref = ref.replace(/^refs\//, "");
 
-    const r = await this.provider.fetchJSON(`${this.api}/git/${ref}`, {
+    const r = await this.provider.fetchJSON(`${this.api}/git/refs/${ref}`, {
       method: "PATCH",
       body: JSON.stringify({
         ...options,
@@ -188,13 +242,10 @@ export class GithubRepository extends Repository {
       })
     });
 
-    if (r.response.ok && this.#ref) {
-      this.#ref.set(ref, sha);
+    if (r.response.ok) {
+      this.#refs.set(ref, sha);
+      return r.json;
     }
-
-    //console.log(r.response.ok, r.response.status, r.json);
-
-    return r.json;
   }
 
   async createBranch(name, from, options) {
@@ -225,7 +276,6 @@ export class GithubRepository extends Repository {
           tree: sha
         })
       });
-      console.log(res);
       */
     }