repository-provider 31.2.6 → 31.2.9

package/README.md

@@ -200,7 +200,6 @@ console.log(await readme.getString());
     *   [fullName](#fullname-1)
     *   [toJSON](#tojson-1)
     *   [attributes](#attributes-1)
-*   [avatarURL](#avatarurl)
 *   [homePageURL](#homepageurl)
 *   [OwnedObject](#ownedobject)
     *   [Parameters](#parameters-45)
@@ -268,6 +267,7 @@ console.log(await readme.getString());
 *   [type](#type-1)
 *   [url](#url-3)
 *   [url](#url-4)
+*   [avatarURL](#avatarurl)
 *   [RepositoryOwner](#repositoryowner)
     *   [Parameters](#parameters-57)
 *   [Repository](#repository-3)
@@ -328,7 +328,6 @@ console.log(await readme.getString());
     *   [type](#type-2)
     *   [refId](#refid-1)
         *   [Parameters](#parameters-78)
-    *   [toJSON](#tojson-2)
     *   [attributes](#attributes-3)
 *   [defaultBranchName](#defaultbranchname)
 *   [issuesURL](#issuesurl-2)
@@ -1193,12 +1192,6 @@ Provided name and all defined attributes.
 
 options
 
-## avatarURL
-
-Avatar.
-
-Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
-
 ## homePageURL
 
 The url of home page.
@@ -1591,6 +1584,12 @@ URL of the repository
 
 Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
 
+## avatarURL
+
+Avatar.
+
+Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
+
 ## RepositoryOwner
 
 Mixin to define a class able to handle a collection of repositories.
@@ -1920,10 +1919,6 @@ 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
 
-### toJSON
-
-Provide name and all defined attributes
-
 ### attributes
 
 options

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "repository-provider",
-  "version": "31.2.6",
+  "version": "31.2.9",
   "publishConfig": {
     "access": "public"
   },
@@ -36,9 +36,9 @@
     "ava": "^4.3.0",
     "c8": "^7.11.3",
     "documentation": "^13.2.5",
-    "repository-provider-test-support": "^2.2.2",
-    "semantic-release": "^19.0.2",
-    "typescript": "^4.7.3"
+    "repository-provider-test-support": "^2.2.6",
+    "semantic-release": "^19.0.3",
+    "typescript": "^4.7.4"
   },
   "engines": {
     "node": ">=16.15.1"

package/src/named-object.mjs

@@ -20,11 +20,6 @@ export class NamedObject extends BaseObject {
         type: "string",
         isKey: true
       },
-      /**
-       * Avatar.
-       * @return {string}
-       */
-      avatarURL: { type: "url" },
 
       /**
        * The url of home page.

package/src/ref.mjs

@@ -39,11 +39,10 @@ export class Ref extends OwnedObject {
 
   /**
    * Get sha of a ref.
-   * @param {string} ref
    * @return {Promise<string>} sha of the ref
    */
-  async refId(ref = this.ref) {
-    return this.owner.refId(ref);
+  async refId() {
+    return this.owner.refId(this.ref);
   }
 
   /**

package/src/repository-group.mjs

@@ -45,6 +45,12 @@ export class RepositoryGroup extends RepositoryOwner(OwnedObject) {
        */
       url: { type: "url" },
 
+      /**
+       * Avatar.
+       * @return {string}
+       */
+      avatarURL: { type: "url" },
+
       isAdmin: { type: "boolean", default: false }
     };
   }

package/src/repository-owner.mjs

@@ -81,7 +81,15 @@ export function RepositoryOwner(base) {
       );
     }
 
-    async _lookup(type, name, split, defaultItem) {
+    /**
+     * Lookup entity of a given type and name.
+     * @param {string} type 
+     * @param {string} name 
+     * @param {function} split 
+     * @param {Object} defaultItem 
+     * @returns {NamedObject} from a repository
+     */
+    async lookup(type, name, split, defaultItem) {
       if (name !== undefined) {
         await this.initializeRepositories();
 
@@ -100,7 +108,15 @@ export function RepositoryOwner(base) {
       }
     }
 
-    async *_list(type, patterns, split, defaultItem) {
+    /**
+     * List entities for a given type and pattern.
+     * @param {string} type 
+     * @param {string|string[]} patterns 
+     * @param {function} split 
+     * @param {Object} defaultItem
+     * @return {AsyncIterator<NamedObject>} matching type and pattern
+     */
+    async *list(type, patterns, split, defaultItem) {
       await this.initializeRepositories();
 
       patterns = stripBaseNames(patterns, this.provider.repositoryBases);
@@ -175,7 +191,7 @@ export function RepositoryOwner(base) {
      * @return {Promise<Branch|undefined>}
      */
     async branch(name) {
-      return this._lookup(
+      return this.lookup(
         "branch",
         name,
         name => name.split(/#/),
@@ -189,7 +205,7 @@ export function RepositoryOwner(base) {
      * @return {AsyncIterator<Branch>} all matching branches of the owner
      */
     async *branches(patterns) {
-      yield* this._list(
+      yield* this.list(
         "branches",
         patterns,
         pattern => pattern.split(/#/),
@@ -198,51 +214,51 @@ export function RepositoryOwner(base) {
     }
 
     async tag(name) {
-      return this._lookup("tag", name, name => name.split(/#/));
+      return this.lookup("tag", name, name => name.split(/#/));
     }
 
     async *tags(patterns) {
-      yield* this._list("tags", patterns, pattern => pattern.split(/#/));
+      yield* this.list("tags", patterns, pattern => pattern.split(/#/));
     }
 
     async pullRequest(name) {
-      return this._lookup("pullRequest", name);
+      return this.lookup("pullRequest", name);
     }
 
     async *pullRequests(patterns) {
-      yield* this._list("pullRequests", patterns);
+      yield* this.list("pullRequests", patterns);
     }
 
     async project(name) {
-      return this._lookup("project", name);
+      return this.lookup("project", name);
     }
 
     async *projects(patterns) {
-      yield* this._list("projects", patterns);
+      yield* this.list("projects", patterns);
     }
 
     async application(name) {
-      return this._lookup("application", name);
+      return this.lookup("application", name);
     }
 
     async *applications(patterns) {
-      yield* this._list("applications", patterns);
+      yield* this.list("applications", patterns);
     }
 
     async milestone(name) {
-      return this._lookup("milestone", name);
+      return this.lookup("milestone", name);
     }
 
     async *milestones(patterns) {
-      yield* this._list("milestones", patterns);
+      yield* this.list("milestones", patterns);
     }
 
     async hook(name) {
-      return this._lookup("hook", name);
+      return this.lookup("hook", name);
     }
 
     async *hooks(patterns) {
-      yield* this._list("hooks", patterns);
+      yield* this.list("hooks", patterns);
     }
   };
 }

package/src/repository.mjs

@@ -465,16 +465,6 @@ export class Repository extends OwnedObject {
    */
   async refId(ref) {}
 
-  /**
-   * Provide name and all defined attributes
-   */
-  toJSON() {
-    return optionJSON(this, {
-      name: this.name,
-      fullName: this.fullName,
-    });
-  }
-
   initialize() {}
 
   initializeHooks() {