repository-provider 26.4.1 → 26.4.5

package/README.md

@@ -166,7 +166,6 @@ console.log(await readme.getString());
         *   [Parameters](#parameters-35)
     *   [removeEntries](#removeentries)
         *   [Parameters](#parameters-36)
-    *   [entryClass](#entryclass-2)
     *   [createPullRequest](#createpullrequest)
         *   [Parameters](#parameters-37)
     *   [createBranch](#createbranch)
@@ -185,35 +184,37 @@ console.log(await readme.getString());
     *   [Parameters](#parameters-43)
     *   [repository](#repository)
         *   [Parameters](#parameters-44)
-    *   [repositoryGroup](#repositorygroup)
+    *   [branch](#branch-1)
         *   [Parameters](#parameters-45)
-    *   [repositoryGroups](#repositorygroups)
+    *   [repositoryGroup](#repositorygroup)
         *   [Parameters](#parameters-46)
-    *   [createRepositoryGroup](#createrepositorygroup)
+    *   [repositoryGroups](#repositorygroups)
         *   [Parameters](#parameters-47)
-    *   [addRepositoryGroup](#addrepositorygroup)
+    *   [createRepositoryGroup](#createrepositorygroup)
         *   [Parameters](#parameters-48)
+    *   [addRepositoryGroup](#addrepositorygroup)
+        *   [Parameters](#parameters-49)
 *   [NamedObject](#namedobject)
-    *   [Parameters](#parameters-49)
+    *   [Parameters](#parameters-50)
     *   [Properties](#properties-7)
     *   [equals](#equals-3)
-        *   [Parameters](#parameters-50)
+        *   [Parameters](#parameters-51)
     *   [displayName](#displayname)
     *   [toJSON](#tojson-2)
 *   [Project](#project)
-    *   [Parameters](#parameters-51)
-*   [PullRequest](#pullrequest)
     *   [Parameters](#parameters-52)
+*   [PullRequest](#pullrequest)
+    *   [Parameters](#parameters-53)
     *   [Properties](#properties-8)
     *   [fullName](#fullname)
     *   [url](#url-2)
     *   [repository](#repository-1)
     *   [provider](#provider-2)
     *   [equals](#equals-4)
-        *   [Parameters](#parameters-53)
+        *   [Parameters](#parameters-54)
     *   [delete](#delete-1)
     *   [merge](#merge)
-        *   [Parameters](#parameters-54)
+        *   [Parameters](#parameters-55)
     *   [decline](#decline)
     *   [reviews](#reviews)
     *   [identifier](#identifier)
@@ -221,9 +222,9 @@ console.log(await readme.getString());
     *   [defaultListStates](#defaultliststates)
     *   [validMergeMethods](#validmergemethods)
     *   [list](#list-1)
-        *   [Parameters](#parameters-55)
-    *   [open](#open)
         *   [Parameters](#parameters-56)
+    *   [open](#open)
+        *   [Parameters](#parameters-57)
 *   [title](#title)
 *   [body](#body)
 *   [state](#state)
@@ -233,19 +234,19 @@ console.log(await readme.getString());
 *   [ContentEntry](#contententry)
     *   [Properties](#properties-9)
 *   [Ref](#ref)
-    *   [Parameters](#parameters-57)
+    *   [Parameters](#parameters-58)
     *   [equals](#equals-5)
-        *   [Parameters](#parameters-58)
+        *   [Parameters](#parameters-59)
     *   [ref](#ref-1)
     *   [refId](#refid)
-        *   [Parameters](#parameters-59)
-    *   [entries](#entries)
         *   [Parameters](#parameters-60)
+    *   [entries](#entries)
+        *   [Parameters](#parameters-61)
     *   [asyncIterator](#asynciterator)
     *   [maybeEntry](#maybeentry)
-        *   [Parameters](#parameters-61)
-    *   [entry](#entry)
         *   [Parameters](#parameters-62)
+    *   [entry](#entry)
+        *   [Parameters](#parameters-63)
     *   [owner](#owner)
     *   [fullName](#fullname-1)
     *   [fullCondensedName](#fullcondensedname)
@@ -260,28 +261,28 @@ console.log(await readme.getString());
     *   [attributes](#attributes-1)
 *   [isProtected](#isprotected-1)
 *   [RepositoryGroup](#repositorygroup-1)
-    *   [Parameters](#parameters-63)
+    *   [Parameters](#parameters-64)
     *   [Properties](#properties-10)
     *   [attributeMapping](#attributemapping-1)
 *   [type](#type)
 *   [url](#url-3)
 *   [RepositoryOwner](#repositoryowner)
-    *   [Parameters](#parameters-64)
-*   [Repository](#repository-2)
     *   [Parameters](#parameters-65)
+*   [Repository](#repository-2)
+    *   [Parameters](#parameters-66)
     *   [Properties](#properties-11)
     *   [fullName](#fullname-2)
     *   [slug](#slug)
     *   [provider](#provider-3)
     *   [identifier](#identifier-2)
     *   [equals](#equals-6)
-        *   [Parameters](#parameters-66)
-    *   [entry](#entry-1)
         *   [Parameters](#parameters-67)
-    *   [entries](#entries-1)
+    *   [entry](#entry-1)
         *   [Parameters](#parameters-68)
-    *   [maybeEntry](#maybeentry-1)
+    *   [entries](#entries-1)
         *   [Parameters](#parameters-69)
+    *   [maybeEntry](#maybeentry-1)
+        *   [Parameters](#parameters-70)
     *   [urls](#urls)
     *   [url](#url-4)
     *   [cloneURL](#cloneurl)
@@ -293,42 +294,42 @@ console.log(await readme.getString());
     *   [isDisabled](#isdisabled-1)
     *   [isTemplate](#istemplate)
     *   [defaultBranch](#defaultbranch)
-    *   [branch](#branch-1)
-        *   [Parameters](#parameters-70)
-    *   [branches](#branches-1)
+    *   [branch](#branch-2)
         *   [Parameters](#parameters-71)
-    *   [createBranch](#createbranch-1)
+    *   [branches](#branches-1)
         *   [Parameters](#parameters-72)
-    *   [addBranch](#addbranch)
+    *   [createBranch](#createbranch-1)
         *   [Parameters](#parameters-73)
-    *   [deleteBranch](#deletebranch)
+    *   [addBranch](#addbranch)
         *   [Parameters](#parameters-74)
-    *   [tag](#tag)
+    *   [deleteBranch](#deletebranch)
         *   [Parameters](#parameters-75)
-    *   [tags](#tags-1)
+    *   [tag](#tag)
         *   [Parameters](#parameters-76)
-    *   [addTag](#addtag)
+    *   [tags](#tags-1)
         *   [Parameters](#parameters-77)
+    *   [addTag](#addtag)
+        *   [Parameters](#parameters-78)
     *   [delete](#delete-2)
     *   [createPullRequest](#createpullrequest-1)
-        *   [Parameters](#parameters-78)
-    *   [addPullRequest](#addpullrequest)
         *   [Parameters](#parameters-79)
+    *   [addPullRequest](#addpullrequest)
+        *   [Parameters](#parameters-80)
     *   [pullRequests](#pullrequests-1)
     *   [pullRequest](#pullrequest-1)
-        *   [Parameters](#parameters-80)
-    *   [deletePullRequest](#deletepullrequest)
         *   [Parameters](#parameters-81)
-    *   [addHook](#addhook)
+    *   [deletePullRequest](#deletepullrequest)
         *   [Parameters](#parameters-82)
-    *   [createHook](#createhook)
+    *   [addHook](#addhook)
         *   [Parameters](#parameters-83)
+    *   [createHook](#createhook)
+        *   [Parameters](#parameters-84)
     *   [hooks](#hooks-1)
     *   [hook](#hook-1)
-        *   [Parameters](#parameters-84)
+        *   [Parameters](#parameters-85)
     *   [type](#type-1)
     *   [refId](#refid-1)
-        *   [Parameters](#parameters-85)
+        *   [Parameters](#parameters-86)
     *   [toJSON](#tojson-3)
     *   [attributes](#attributes-2)
 *   [defaultBranchName](#defaultbranchname)
@@ -337,23 +338,23 @@ console.log(await readme.getString());
 *   [Review](#review)
 *   [SingleGroupProvider](#singlegroupprovider)
     *   [repository](#repository-3)
-        *   [Parameters](#parameters-86)
-    *   [repositoryGroup](#repositorygroup-2)
         *   [Parameters](#parameters-87)
-    *   [repositoryGroups](#repositorygroups-1)
+    *   [repositoryGroup](#repositorygroup-2)
         *   [Parameters](#parameters-88)
+    *   [repositoryGroups](#repositorygroups-1)
+        *   [Parameters](#parameters-89)
 *   [Tag](#tag-1)
-    *   [Parameters](#parameters-89)
+    *   [Parameters](#parameters-90)
     *   [refType](#reftype-1)
     *   [isWritable](#iswritable-1)
 *   [asArray](#asarray)
-    *   [Parameters](#parameters-90)
-*   [stripBaseName](#stripbasename)
     *   [Parameters](#parameters-91)
-*   [stripBaseNames](#stripbasenames)
+*   [stripBaseName](#stripbasename)
     *   [Parameters](#parameters-92)
-*   [generateBranchName](#generatebranchname)
+*   [stripBaseNames](#stripbasenames)
     *   [Parameters](#parameters-93)
+*   [generateBranchName](#generatebranchname)
+    *   [Parameters](#parameters-94)
 
 ## Application
 
@@ -403,7 +404,7 @@ Object.definedProperties(new aClass(),{ with_default: { value: 77 }})
 *   `object` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** target object
 *   `options` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** as passed to object constructor (optional, default `{}`)
 *   `properties` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** object properties (optional, default `{}`)
-*   `attributes` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)**  (optional, default `object.constructor.attributes`)
+*   `attributes` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)**  (optional, default `object.constructor.attributes||[]`)
 
 ## defaultValues
 
@@ -484,8 +485,8 @@ Returns **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/G
 
 ### Parameters
 
-*   `options` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** 
-*   `additionalProperties` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** 
+*   `options` **any** 
+*   `additionalProperties` **any** 
 
 ### update
 
@@ -967,7 +968,7 @@ Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/G
 
 Are we the default branch.
 
-Returns **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)** true if name is the repository default branch
+Returns **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)** true if name matches the repository default branch
 
 ### delete
 
@@ -1014,12 +1015,6 @@ Remove entries form the branch.
 
 *   `entries` **AsyncIterator<[ContentEntry](#contententry)>** 
 
-### entryClass
-
-By default we use the providers implementation.
-
-Returns **Class** as defined in the repository
-
 ### createPullRequest
 
 Create a pull request.
@@ -1116,6 +1111,16 @@ Lookup a repository in the provider and all of its repository groups.
 
 Returns **[Promise](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Promise)<[Repository](#repository)>** 
 
+### branch
+
+Lookup a branch.
+
+#### Parameters
+
+*   `name` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** of the branch
+
+Returns **[Promise](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Promise)<[Branch](#branch)>** 
+
 ### repositoryGroup
 
 Lookup a repository group.
@@ -1187,7 +1192,7 @@ Returns **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/
 
 ### displayName
 
-Beatified name use for human displaying only.
+Beautified name use for human displaying only.
 
 Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** human readable name
 
@@ -2043,7 +2048,7 @@ Loops over names and executes stripBaseName.
 *   `repositoryBases` **[Array](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Array)<[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)>** all possible bases
 *   `whenFound` **[Function](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Statements/function)?** to be called with the found base name
 
-Returns **[Array](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Array)<[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)>** names without base
+Returns **([Array](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Array)<[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)> | [undefined](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/undefined))** names without base
 
 ## generateBranchName
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "repository-provider",
-  "version": "26.4.1",
+  "version": "26.4.5",
   "publishConfig": {
     "access": "public"
   },
@@ -30,15 +30,15 @@
     "lint:tsc": "tsc --allowJs --checkJs --noEmit -module nodenext -t es2022 ./src/*.mjs"
   },
   "dependencies": {
-    "matching-iterator": "^2.0.0"
+    "matching-iterator": "^2.0.2"
   },
   "devDependencies": {
     "ava": "^4.0.1",
     "c8": "^7.11.0",
     "documentation": "^13.2.5",
-    "repository-provider-test-support": "^1.9.9",
+    "repository-provider-test-support": "^1.11.1",
     "semantic-release": "^19.0.2",
-    "typescript": "^4.6.0-dev.20220125"
+    "typescript": "^4.6.0-dev.20220201"
   },
   "engines": {
     "node": ">=14.18.3"

package/src/attribute.mjs

@@ -34,69 +34,67 @@ export function definePropertiesFromOptions(
   object,
   options = {},
   properties = {},
-  attributes = object.constructor.attributes
+  attributes = object.constructor.attributes || []
 ) {
   const applyLater = {};
 
-  if (attributes !== undefined) {
-    Object.entries(attributes).forEach(([name, attribute]) => {
-      const path = name.split(/\./);
-      const first = path.shift();
-      const property = properties[first];
-
-      let value = options[name];
-      if (value === undefined) {
-        if (attribute.getDefault) {
-          value = attribute.getDefault(attribute, object);
-        } else if (
-          attribute.default !== undefined &&
-          attribute.default !== getAttribute(object, name)
-        ) {
-          value = attribute.default;
-        }
+  Object.entries(attributes).forEach(([name, attribute]) => {
+    const path = name.split(/\./);
+    const first = path.shift();
+    const property = properties[first];
+
+    let value = options[name];
+    if (value === undefined) {
+      if (attribute.getDefault) {
+        value = attribute.getDefault(attribute, object);
+      } else if (
+        attribute.default !== undefined &&
+        attribute.default !== getAttribute(object, name)
+      ) {
+        value = attribute.default;
       }
+    }
 
-      if (attribute.set) {
-        value = attribute.set(value);
-      } else {
-        switch (attribute.type) {
-          case "boolean":
-            if (value !== undefined) {
-              value =
-                value === 0 || value === "0" || value === false ? false : true;
-            }
-            break;
-        }
+    if (attribute.set) {
+      value = attribute.set(value);
+    } else {
+      switch (attribute.type) {
+        case "boolean":
+          if (value !== undefined) {
+            value =
+              value === 0 || value === "0" || value === false ? false : true;
+          }
+          break;
       }
+    }
 
-      if (path.length) {
-        const remaining = path.join(".");
-        if (property) {
-          setAttribute(property.value, remaining, value);
-        } else {
-          const slice = {};
-          setAttribute(slice, remaining, value);
-          properties[first] = { configurable: true, value: slice };
-        }
+    if (path.length) {
+      const remaining = path.join(".");
+      if (property) {
+        setAttribute(property.value, remaining, value);
       } else {
-        if (value !== undefined) {
-          const op = Object.getOwnPropertyDescriptor(
-            object.constructor.prototype,
-            first
+        const slice = {};
+        setAttribute(slice, remaining, value);
+        properties[first] = { configurable: true, value: slice };
+      }
+    } else {
+      if (value !== undefined) {
+        const op = Object.getOwnPropertyDescriptor(
+          object.constructor.prototype,
+          first
+        );
+
+        if ((op && op.set) || (property && property.set)) {
+          applyLater[first] = value;
+        } else {
+          properties[first] = Object.assign(
+            { value, writable: attribute.writable },
+            property
           );
-
-          if ((op && op.set) || (property && property.set)) {
-            applyLater[first] = value;
-          } else {
-            properties[first] = Object.assign(
-              { value, writable: attribute.writable },
-              property
-            );
-          }
         }
       }
-    });
-  }
+    }
+  });
 
   Object.defineProperties(object, properties);
   Object.assign(object, applyLater);
@@ -114,8 +112,7 @@ export function defaultValues(attributes, object) {
 
       if (attribute.default !== undefined) {
         a.push([name, attribute.default]);
-      }
-      else if (attribute.getDefault !== undefined) {
+      } else if (attribute.getDefault !== undefined) {
         a.push([name, attribute.getDefault(attribute, object)]);
       }
 

package/src/base-object.mjs

@@ -63,6 +63,11 @@ export class BaseObject {
     return {};
   }
 
+  /**
+   * Creates an instance of BaseObject.
+   * @param {*} options
+   * @param {*} additionalProperties
+   */
   constructor(options, additionalProperties) {
     definePropertiesFromOptions(
       this,
@@ -161,7 +166,7 @@ export class BaseObject {
    * By default we use the owners implementation.
    * @return {Class} as defined in the owner
    */
-  get hookClass() {
+   get hookClass() {
     return this.owner.hookClass;
   }
 }

package/src/branch.mjs

@@ -1,17 +1,8 @@
 import { Ref } from "./ref.mjs";
 import { PullRequest } from "./pull-request.mjs";
 import { Repository } from "./repository.mjs";
+import { Commit } from "./commit.mjs";
 
-/**
- * @typedef {Object} Commit
- * @property {string} message
- * @property {ContentEntry[]} updates
- */
-
-/**
- * @typedef {Object} CommitResult
- * @property {string} ref
- */
 
 /**
  * Abstract branch.
@@ -49,7 +40,7 @@ export class Branch extends Ref {
 
   /**
    * Are we the default branch.
-   * @return {boolean} true if name is the repository default branch
+   * @return {boolean} true if name matches the repository default branch
    */
   get isDefault() {
     return this.name === this.repository.defaultBranchName;

package/src/commit.mjs

@@ -1,3 +1,10 @@
+
+
+/**
+ * @typedef {Object} CommitResult
+ * @property {string} ref
+ */
+
 /**
  * @property {Repository} repository
  * @property {string} message

package/src/multi-group-provider.mjs

@@ -2,6 +2,7 @@ import { matcher } from "matching-iterator";
 import { BaseProvider } from "./base-provider.mjs";
 import { stripBaseNames } from "./util.mjs";
 import { Repository } from "./repository.mjs";
+import { Branch } from "./branch.mjs";
 import { RepositoryGroup } from "./repository-group.mjs";
 
 /**
@@ -34,6 +35,11 @@ export class MultiGroupProvider extends BaseProvider {
     }
   }
 
+  /**
+   * Lookup a branch.
+   * @param {string} name of the branch
+   * @return {Promise<Branch>}
+   */
   async branch(name) {
     const { base, group, repository, branch } = this.parseName(name);
 
@@ -44,9 +50,7 @@ export class MultiGroupProvider extends BaseProvider {
         if (rg !== undefined) {
           const r = await rg.repository(repository);
           if (r !== undefined) {
-            return r.branch(
-              branch === undefined ? r.defaultBranchName : branch
-            );
+            return r.branch( r.defaultBranchName || branch);
           }
         }
       }
@@ -78,7 +82,7 @@ export class MultiGroupProvider extends BaseProvider {
 
     yield* matcher(
       this._repositoryGroups.values(),
-      stripBaseNames(patterns,this.provider.repositoryBases),
+      stripBaseNames(patterns, this.provider.repositoryBases),
       {
         caseSensitive: this.areGroupNamesCaseSensitive,
         name: "name"

package/src/named-object.mjs

@@ -27,7 +27,7 @@ export class NamedObject extends BaseObject {
   }
 
   /**
-   * Beatified name use for human displaying only.
+   * Beautified name use for human displaying only.
    * @return {string} human readable name
    */
   get displayName() {

package/src/ref.mjs

@@ -61,7 +61,7 @@ export class Ref extends NamedObject {
 
   /**
    * List entries of the branch.
-   * @param {string[]} matchingPatterns
+   * @param {string[]} [matchingPatterns]
    * @return {AsyncIterator<ContentEntry>} all matching entries in the branch
    */
   async *entries(matchingPatterns) {}

package/src/util.mjs

@@ -42,7 +42,7 @@ export function stripBaseName(name, repositoryBases, whenFound) {
  * @param {*} repositoryBases
  * @param {string[]} repositoryBases all possible bases
  * @param {Function} [whenFound] to be called with the found base name
- * @return {string[]} names without base
+ * @return {string[]|undefined} names without base
  */
 export function stripBaseNames(names, repositoryBases, whenFound) {
   return names === undefined