repository-provider 32.7.9 → 32.7.10

package/README.md

@@ -1,6 +1,6 @@
 [![npm](https://img.shields.io/npm/v/repository-provider.svg)](https://www.npmjs.com/package/repository-provider)
 [![License](https://img.shields.io/badge/License-BSD%203--Clause-blue.svg)](https://opensource.org/licenses/BSD-3-Clause)
-[![Open Bundle](https://bundlejs.com/badge-light.svg)](https://bundlejs.com/?q=repository-provider)
+[![bundlejs](https://deno.bundlejs.com/?q=repository-provider\&badge=detailed)](https://bundlejs.com/?q=repository-provider)
 [![downloads](http://img.shields.io/npm/dm/repository-provider.svg?style=flat-square)](https://npmjs.org/package/repository-provider)
 [![GitHub Issues](https://img.shields.io/github/issues/arlac77/repository-provider.svg?style=flat-square)](https://github.com/arlac77/repository-provider/issues)
 [![Build Status](https://img.shields.io/endpoint.svg?url=https%3A%2F%2Factions-badge.atrox.dev%2Farlac77%2Frepository-provider%2Fbadge\&style=flat)](https://actions-badge.atrox.dev/arlac77/repository-provider/goto)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "repository-provider",
-  "version": "32.7.9",
+  "version": "32.7.10",
   "publishConfig": {
     "access": "public"
   },

package/src/attribute-extras.mjs

@@ -32,10 +32,6 @@ export function definePropertiesFromOptions(
   const applyLater = {};
 
   Object.entries(attributes).forEach(([name, attribute]) => {
-    const path = name.split(/\./);
-    const first = path.shift();
-    const property = properties[first];
-
     let value = getAttribute(options, name);
 
     if (value === undefined) {
@@ -67,8 +63,13 @@ export function definePropertiesFromOptions(
       }
     }
 
+    const path = name.split(/\./);
+    const first = path.shift();
+    const property = properties[first];  
+
     if (path.length) {
       const remaining = path.join(".");
+
       if (property) {
         setAttribute(property.value, remaining, value);
       } else {

package/src/base-object.mjs

@@ -4,11 +4,12 @@ import {
 } from "./attribute-extras.mjs";
 import { description_attribute, id_attribute } from "./attributes.mjs";
 
-/**
- * @param {Object} options
- * @param {Object} additionalProperties
- */
-export class BaseObject {
+  /**
+   * Creates an instance of BaseObject.
+   * @param {Object|undefined} options
+   * @param {Object|undefined} additionalProperties
+   */
+  export class BaseObject {
   /**
    * @return {string} type we represent
    */
@@ -56,18 +57,13 @@ export class BaseObject {
     return {};
   }
 
-  /**
-   * Creates an instance of BaseObject.
-   * @param {Object} options
-   * @param {Object} additionalProperties
-   */
   constructor(options, additionalProperties) {
     this.updateAttributes(options, additionalProperties);
   }
 
   /**
    * Takes values from options.
-   * @param {Object} options
+   * @param {Object|undefined} options
    */
   updateAttributes(options, additionalProperties) {
     definePropertiesFromOptions(

package/src/base-provider.mjs

@@ -12,7 +12,8 @@ import {
   url_attribute,
   name_attribute,
   description_attribute,
-  priority_attribute
+  priority_attribute,
+  default_attribute
 } from "./attributes.mjs";
 
 /**
@@ -119,9 +120,9 @@ export class BaseProvider extends BaseObject {
        * To forward info/warn and error messages to
        */
       messageDestination: {
+        ...default_attribute,
         type: "object",
         default: console,
-        mandatory: false,
         writable: true,
         private: true
       }
@@ -300,7 +301,7 @@ export class BaseProvider extends BaseObject {
    * List provider objects of a given type.
    *
    * @param {string} type name of the method to deliver typed iterator projects,milestones,hooks,repositories,branches,tags
-   * @param {string|string[]} patterns group / repository filter
+   * @param {string[]|string} patterns group / repository filter
    */
   async *list(type, patterns) {
     if (patterns === undefined) {

package/src/branch.mjs

@@ -201,7 +201,7 @@ export class Branch extends Ref {
    * Create a new {@link Branch} by cloning a given source branch.
    * Simply calls Repository.createBranch() with the receiver as source branch
    * @param {string} name the new branch
-   * @param {Object|undefined} options
+   * @param {Object|undefined} options passed through
    * @return {Promise<Branch>} newly created branch (or already present old one with the same name)
    */
   async createBranch(name, options) {

package/src/repository-owner.mjs

@@ -1,11 +1,12 @@
 import { matcher } from "matching-iterator";
 import { Branch } from "./branch.mjs";
 import { Repository } from "./repository.mjs";
+import { NamedObject } from "./named-object.mjs";
 import { asArray, stripBaseName, stripBaseNames } from "./util.mjs";
 
 /**
  * Mixin to define a class able to handle a collection of repositories.
- * @param {Class} base to be extendet
+ * @param {Object} base to be extendet
  */
 export function RepositoryOwner(base) {
   return class RepositoryOwner extends base {
@@ -108,7 +109,7 @@ export function RepositoryOwner(base) {
     /**
      * List entities for a given type and pattern.
      * @param {string} type
-     * @param {string|string[]} patterns
+     * @param {string[]|string} patterns
      * @param {function} [split]
      * @param {Object} [defaultItem]
      * @return {AsyncIterator<NamedObject>} matching type and pattern

package/src/repository.mjs

@@ -23,7 +23,7 @@ import {
  * @class Repository
  * @param {RepositoryOwner} owner
  * @param {string} name (#branch) will be removed
- * @param {Object} options
+ * @param {Object|undefined} options
  * @param {string} [options.description] human readable description
  * @param {string} [options.id] internal id
  *
@@ -362,7 +362,7 @@ export class Repository extends OwnedObject {
   }
 
   /**
-   * The @{link PullRequest} for a given name.
+   * The @see {@link PullRequest} for a given name.
    * @param {string} name
    * @return {Promise<PullRequest>}
    */
@@ -465,7 +465,7 @@ export class Repository extends OwnedObject {
   /**
    * Get sha of a ref.
    * @param {string} ref
-   * @return {Promise<string>} sha of the ref
+   * @return {Promise<string|undefined>} sha of the ref
    */
   async refId(ref) {}