repository-provider 28.3.12 → 28.3.15

package/README.md

@@ -940,9 +940,10 @@ Abstract branch.
 
 ### Parameters
 
-*   `repository` **[Repository](#repository)** 
-*   `name` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)**  (optional, default `repository.defaultBranchName`)
+*   `owner`  
+*   `name` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)**  (optional, default `owner.defaultBranchName`)
 *   `options` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** 
+*   `repository` **[Repository](#repository)** 
 
 ### Properties
 
@@ -1182,7 +1183,7 @@ Check for equality.
 
 *   `other` **[NamedObject](#namedobject)** 
 
-Returns **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)** true if names are equal and have the same owner
+Returns **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)** true if names are equal and have the same provider
 
 ### fullName
 
@@ -1190,7 +1191,7 @@ Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/G
 
 ### toJSON
 
-Provide name and all defined attributes.
+Provided name and all defined attributes.
 
 ## OwnedObject
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "repository-provider",
-  "version": "28.3.12",
+  "version": "28.3.15",
   "publishConfig": {
     "access": "public"
   },
@@ -36,7 +36,7 @@
     "ava": "^4.2.0",
     "c8": "^7.11.2",
     "documentation": "^13.2.5",
-    "repository-provider-test-support": "^2.1.9",
+    "repository-provider-test-support": "^2.1.10",
     "semantic-release": "^19.0.2",
     "typescript": "^4.6.3"
   },

package/src/branch.mjs

@@ -21,8 +21,8 @@ export class Branch extends Ref {
     return "_addBranch";
   }
 
-  constructor(repository, name = repository.defaultBranchName, options) {
-    super(repository, name, options);
+  constructor(owner, name = owner.defaultBranchName, options) {
+    super(owner, name, options);
   }
 
   /**
@@ -31,8 +31,8 @@ export class Branch extends Ref {
    */
   get url() {
     return this.isDefault
-      ? this.repository.url
-      : `${this.repository.url}#${this.name}`;
+      ? this.owner.url
+      : `${this.owner.url}#${this.name}`;
   }
 
   /**
@@ -60,7 +60,7 @@ export class Branch extends Ref {
    * @return {boolean} true if name matches the repository default branch
    */
   get isDefault() {
-    return this.name === this.repository.defaultBranchName;
+    return this.name === this.owner.defaultBranchName;
   }
 
   /**
@@ -69,7 +69,7 @@ export class Branch extends Ref {
    * @return {Promise<any>}
    */
   async delete() {
-    return this.repository.deleteBranch(this.name);
+    return this.owner.deleteBranch(this.name);
   }
 
   /**
@@ -191,6 +191,6 @@ export class Branch extends Ref {
    * @return {Promise<Branch>} newly created branch (or already present old one with the same name)
    */
   async createBranch(name, options) {
-    return this.repository.createBranch(name, this, options);
+    return this.owner.createBranch(name, this, options);
   }
 }

package/src/multi-group-provider.mjs

@@ -10,11 +10,8 @@ import { RepositoryGroup } from "./repository-group.mjs";
  *
  */
 export class MultiGroupProvider extends BaseProvider {
-  constructor(options) {
-    super(options, {
-      _repositoryGroups: { value: new Map() }
-    });
-  }
+
+  #repositoryGroups = new Map();
 
   /**
    * Lookup a repository in the provider and all of its repository groups.
@@ -68,7 +65,7 @@ export class MultiGroupProvider extends BaseProvider {
     if (this.supportsBase(base)) {
       name = stripBaseNames(name, this.provider.repositoryBases);
       await this.initializeRepositories();
-      return this._repositoryGroups.get(this.normalizeGroupName(name, true));
+      return this.#repositoryGroups.get(this.normalizeGroupName(name, true));
     }
   }
 
@@ -81,7 +78,7 @@ export class MultiGroupProvider extends BaseProvider {
     await this.initializeRepositories();
 
     yield* matcher(
-      this._repositoryGroups.values(),
+      this.#repositoryGroups.values(),
       stripBaseNames(patterns, this.provider.repositoryBases),
       {
         caseSensitive: this.areGroupNamesCaseSensitive,
@@ -110,10 +107,10 @@ export class MultiGroupProvider extends BaseProvider {
   addRepositoryGroup(name, options) {
     const normalizedName = this.normalizeGroupName(name, true);
 
-    let repositoryGroup = this._repositoryGroups.get(normalizedName);
+    let repositoryGroup = this.#repositoryGroups.get(normalizedName);
     if (repositoryGroup === undefined) {
       repositoryGroup = new this.repositoryGroupClass(this, name, options);
-      this._repositoryGroups.set(normalizedName, repositoryGroup);
+      this.#repositoryGroups.set(normalizedName, repositoryGroup);
     }
     return repositoryGroup;
   }

package/src/named-object.mjs

@@ -20,7 +20,7 @@ export class NamedObject extends BaseObject {
   /**
    * Check for equality.
    * @param {NamedObject} other
-   * @return {boolean} true if names are equal and have the same owner
+   * @return {boolean} true if names are equal and have the same provider
    */
   equals(other) {
     return (
@@ -37,7 +37,7 @@ export class NamedObject extends BaseObject {
   }
  
   /**
-   * Provide name and all defined attributes.
+   * Provided name and all defined attributes.
    */
   toJSON() {
     return optionJSON(this, {

package/src/repository-owner.mjs

@@ -9,11 +9,8 @@ import { asArray, stripBaseName, stripBaseNames } from "./util.mjs";
  */
 export function RepositoryOwner(base) {
   return class RepositoryOwner extends base {
-    constructor(...args) {
-      super(...args);
 
-      Object.defineProperties(this, { _repositories: { value: new Map() } });
-    }
+    #repositories = new Map();
 
     /**
      * Normalizes a repository name.
@@ -51,7 +48,7 @@ export function RepositoryOwner(base) {
 
       await this.initializeRepositories();
 
-      return this._repositories.get(this.normalizeRepositoryName(name, true));
+      return this.#repositories.get(this.normalizeRepositoryName(name, true));
     }
 
     /**
@@ -75,7 +72,7 @@ export function RepositoryOwner(base) {
 
       await this.initializeRepositories();
       yield* matcher(
-        this._repositories.values(),
+        this.#repositories.values(),
         stripBaseNames(patterns, this.provider.repositoryBases),
         {
           caseSensitive: this.areRepositoryNamesCaseSensitive,
@@ -91,7 +88,7 @@ export function RepositoryOwner(base) {
         name = stripBaseName(name, this.provider.repositoryBases);
 
         const [repoName, typeName] = split ? split(name) : name.split("/");
-        const repository = this._repositories.get(repoName);
+        const repository = this.#repositories.get(repoName);
 
         if (repository) {
           if (typeName === undefined && defaultItem) {
@@ -113,10 +110,10 @@ export function RepositoryOwner(base) {
           ? split(pattern)
           : pattern.split("/");
 
-        for (const name of matcher(this._repositories.keys(), repoPattern, {
+        for (const name of matcher(this.#repositories.keys(), repoPattern, {
           caseSensitive: this.areRepositoriesCaseSensitive
         })) {
-          const repository = this._repositories.get(name);
+          const repository = this.#repositories.get(name);
 
           if (typePattern === undefined && defaultItem) {
             const item = await defaultItem(repository);
@@ -150,7 +147,7 @@ export function RepositoryOwner(base) {
      */
     addRepository(name, options) {
       const normalizedName = this.normalizeRepositoryName(name, true);
-      let repository = this._repositories.get(normalizedName);
+      let repository = this.#repositories.get(normalizedName);
       if (repository === undefined) {
         repository = new this.repositoryClass(this, name, options);
       }
@@ -160,7 +157,7 @@ export function RepositoryOwner(base) {
     _addRepository(repository)
     {
       const normalizedName = this.normalizeRepositoryName(repository.name, true);
-      this._repositories.set(normalizedName, repository);
+      this.#repositories.set(normalizedName, repository);
     }
 
     /**
@@ -169,7 +166,7 @@ export function RepositoryOwner(base) {
      * @return {Promise<any>}
      */
     async deleteRepository(name) {
-      this._repositories.delete(this.normalizeRepositoryName(name, true));
+      this.#repositories.delete(this.normalizeRepositoryName(name, true));
     }
 
     initializeRepositories() {}