repository-provider 32.3.17 → 32.3.19

package/README.md

@@ -64,6 +64,12 @@ console.log(await readme.getString());
     *   [Parameters](#parameters-6)
 *   [mapAttributesInverse](#mapattributesinverse)
     *   [Parameters](#parameters-7)
+*   [description](#description)
+*   [id](#id)
+*   [uuid](#uuid)
+*   [body](#body)
+*   [title](#title)
+*   [priority](#priority)
 *   [BaseObject](#baseobject)
     *   [Parameters](#parameters-8)
     *   [update](#update)
@@ -79,9 +85,6 @@ console.log(await readme.getString());
     *   [attributes](#attributes)
     *   [writableAttributes](#writableattributes)
     *   [attributeMapping](#attributemapping)
-*   [description](#description)
-*   [id](#id)
-*   [uuid](#uuid)
 *   [MessageDestination](#messagedestination)
     *   [Properties](#properties-1)
 *   [BaseProvider](#baseprovider)
@@ -135,7 +138,6 @@ console.log(await readme.getString());
         *   [Parameters](#parameters-25)
     *   [initialize](#initialize)
         *   [Parameters](#parameters-26)
-*   [priority](#priority)
 *   [name](#name-1)
 *   [messageDestination](#messagedestination-1)
 *   [Branch](#branch)
@@ -217,11 +219,9 @@ console.log(await readme.getString());
     *   [fullName](#fullname-2)
     *   [url](#url-2)
     *   [repository](#repository-1)
-    *   [equals](#equals-4)
-        *   [Parameters](#parameters-50)
     *   [delete](#delete-1)
     *   [merge](#merge)
-        *   [Parameters](#parameters-51)
+        *   [Parameters](#parameters-50)
     *   [decline](#decline)
     *   [reviews](#reviews)
     *   [identifier](#identifier-1)
@@ -229,11 +229,9 @@ console.log(await readme.getString());
     *   [defaultListStates](#defaultliststates)
     *   [validMergeMethods](#validmergemethods)
     *   [list](#list-1)
-        *   [Parameters](#parameters-52)
+        *   [Parameters](#parameters-51)
     *   [open](#open)
-        *   [Parameters](#parameters-53)
-*   [title](#title)
-*   [body](#body)
+        *   [Parameters](#parameters-52)
 *   [state](#state)
 *   [locked](#locked)
 *   [merged](#merged)
@@ -244,12 +242,12 @@ console.log(await readme.getString());
     *   [ref](#ref-1)
     *   [refId](#refid)
     *   [entries](#entries)
-        *   [Parameters](#parameters-54)
+        *   [Parameters](#parameters-53)
     *   [asyncIterator](#asynciterator)
     *   [maybeEntry](#maybeentry)
-        *   [Parameters](#parameters-55)
+        *   [Parameters](#parameters-54)
     *   [entry](#entry)
-        *   [Parameters](#parameters-56)
+        *   [Parameters](#parameters-55)
     *   [repository](#repository-2)
     *   [fullName](#fullname-3)
     *   [fullCondensedName](#fullcondensedname-1)
@@ -263,28 +261,25 @@ console.log(await readme.getString());
     *   [attributes](#attributes-2)
 *   [isProtected](#isprotected-1)
 *   [RepositoryGroup](#repositorygroup-1)
-    *   [Parameters](#parameters-57)
+    *   [Parameters](#parameters-56)
     *   [Properties](#properties-9)
     *   [attributeMapping](#attributemapping-1)
 *   [type](#type-1)
-*   [url](#url-3)
-*   [url](#url-4)
-*   [avatarURL](#avatarurl)
 *   [RepositoryOwner](#repositoryowner)
-    *   [Parameters](#parameters-58)
+    *   [Parameters](#parameters-57)
 *   [Repository](#repository-3)
-    *   [Parameters](#parameters-59)
+    *   [Parameters](#parameters-58)
     *   [Properties](#properties-10)
     *   [fullName](#fullname-4)
     *   [slug](#slug)
     *   [entry](#entry-1)
-        *   [Parameters](#parameters-60)
+        *   [Parameters](#parameters-59)
     *   [entries](#entries-1)
-        *   [Parameters](#parameters-61)
+        *   [Parameters](#parameters-60)
     *   [maybeEntry](#maybeentry-1)
-        *   [Parameters](#parameters-62)
+        *   [Parameters](#parameters-61)
     *   [commits](#commits)
-        *   [Parameters](#parameters-63)
+        *   [Parameters](#parameters-62)
     *   [cloneURL](#cloneurl)
     *   [issuesURL](#issuesurl-1)
     *   [homePageURL](#homepageurl-2)
@@ -295,62 +290,62 @@ console.log(await readme.getString());
     *   [delete](#delete-2)
     *   [defaultBranch](#defaultbranch)
     *   [branch](#branch-2)
-        *   [Parameters](#parameters-64)
+        *   [Parameters](#parameters-63)
     *   [hasBranches](#hasbranches)
     *   [branches](#branches-1)
-        *   [Parameters](#parameters-65)
+        *   [Parameters](#parameters-64)
     *   [createBranch](#createbranch-1)
-        *   [Parameters](#parameters-66)
+        *   [Parameters](#parameters-65)
     *   [addBranch](#addbranch)
-        *   [Parameters](#parameters-67)
+        *   [Parameters](#parameters-66)
     *   [deleteBranch](#deletebranch)
-        *   [Parameters](#parameters-68)
+        *   [Parameters](#parameters-67)
     *   [tag](#tag)
-        *   [Parameters](#parameters-69)
+        *   [Parameters](#parameters-68)
     *   [tags](#tags-1)
-        *   [Parameters](#parameters-70)
+        *   [Parameters](#parameters-69)
     *   [addTag](#addtag)
-        *   [Parameters](#parameters-71)
+        *   [Parameters](#parameters-70)
     *   [createPullRequest](#createpullrequest-1)
-        *   [Parameters](#parameters-72)
+        *   [Parameters](#parameters-71)
     *   [addPullRequest](#addpullrequest)
-        *   [Parameters](#parameters-73)
+        *   [Parameters](#parameters-72)
     *   [pullRequests](#pullrequests-1)
     *   [pullRequest](#pullrequest-1)
-        *   [Parameters](#parameters-74)
+        *   [Parameters](#parameters-73)
     *   [deletePullRequest](#deletepullrequest)
-        *   [Parameters](#parameters-75)
+        *   [Parameters](#parameters-74)
     *   [addHook](#addhook)
-        *   [Parameters](#parameters-76)
+        *   [Parameters](#parameters-75)
     *   [createHook](#createhook)
-        *   [Parameters](#parameters-77)
+        *   [Parameters](#parameters-76)
     *   [hooks](#hooks-1)
     *   [hook](#hook-1)
-        *   [Parameters](#parameters-78)
+        *   [Parameters](#parameters-77)
     *   [type](#type-2)
     *   [refId](#refid-1)
-        *   [Parameters](#parameters-79)
+        *   [Parameters](#parameters-78)
     *   [attributes](#attributes-3)
 *   [defaultBranchName](#defaultbranchname)
 *   [issuesURL](#issuesurl-2)
 *   [Review](#review)
 *   [SingleGroupProvider](#singlegroupprovider)
     *   [repository](#repository-4)
-        *   [Parameters](#parameters-80)
+        *   [Parameters](#parameters-79)
     *   [repositoryGroup](#repositorygroup-2)
-        *   [Parameters](#parameters-81)
+        *   [Parameters](#parameters-80)
     *   [repositoryGroups](#repositorygroups-1)
-        *   [Parameters](#parameters-82)
+        *   [Parameters](#parameters-81)
 *   [Tag](#tag-1)
     *   [refType](#reftype-1)
 *   [asArray](#asarray)
-    *   [Parameters](#parameters-83)
+    *   [Parameters](#parameters-82)
 *   [stripBaseName](#stripbasename)
-    *   [Parameters](#parameters-84)
+    *   [Parameters](#parameters-83)
 *   [stripBaseNames](#stripbasenames)
-    *   [Parameters](#parameters-85)
+    *   [Parameters](#parameters-84)
 *   [generateBranchName](#generatebranchname)
-    *   [Parameters](#parameters-86)
+    *   [Parameters](#parameters-85)
 
 ## Application
 
@@ -486,6 +481,31 @@ Filters out null, undefined and empty strings
 
 Returns **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** keys renamed after mapping
 
+## description
+
+The description of the object content.
+
+## id
+
+Unique id within the provider.
+
+## uuid
+
+Unique id.
+
+## body
+
+The description of the pull request.
+
+## title
+
+The one line description of the pull request.
+
+## priority
+
+In case there are several providers able to support a given source which one sould be used ?
+this defines the order
+
 ## BaseObject
 
 ### Parameters
@@ -562,24 +582,6 @@ Map attributes between external and internal representation.
 
 Returns **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** 
 
-## description
-
-The description of the repository content.
-
-Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
-
-## id
-
-Unique id within the provider.
-
-Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
-
-## uuid
-
-Unique id.
-
-Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
-
 ## MessageDestination
 
 Type: [Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)
@@ -859,11 +861,6 @@ Creates a new provider for a given set of options.
 
 Returns **[BaseProvider](#baseprovider)** newly created provider or undefined if options are not sufficient to construct a provider
 
-## priority
-
-In case there are several providers able to support a given source which one sould be used ?
-this defines the order
-
 ## name
 
 Name of the provider.
@@ -1303,16 +1300,6 @@ Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/G
 
 Returns **[Repository](#repository)** destination repository
 
-### equals
-
-Check for equality
-
-#### Parameters
-
-*   `other` **[PullRequest](#pullrequest)** 
-
-Returns **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)** true if number and repository are equal
-
 ### delete
 
 *   **See**: [Repository#deletePullRequest](#repositorydeletepullrequest)
@@ -1387,18 +1374,6 @@ Open a pull request
 *   `destination` **[Branch](#branch)** 
 *   `options` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** 
 
-## title
-
-The one line description of the pull request.
-
-Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
-
-## body
-
-The description of the pull request.
-
-Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
-
 ## state
 
 state of the pull request.
@@ -1589,24 +1564,6 @@ Map attributes between external and internal representation.
 
 Type of the repository group either User or Organization.
 
-Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
-
-## url
-
-api url
-
-## url
-
-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.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "repository-provider",
-  "version": "32.3.17",
+  "version": "32.3.19",
   "publishConfig": {
     "access": "public"
   },
@@ -31,14 +31,14 @@
     "lint:tsc": "tsc --allowJs --checkJs --noEmit -t esnext -m esnext ./src/*.mjs"
   },
   "dependencies": {
-    "matching-iterator": "^2.0.10"
+    "matching-iterator": "^2.0.11"
   },
   "devDependencies": {
     "ava": "^5.1.0",
     "browser-ava": "^1.3.13",
     "c8": "^7.12.0",
     "documentation": "^14.0.1",
-    "repository-provider-test-support": "^2.2.25",
+    "repository-provider-test-support": "^2.2.27",
     "semantic-release": "^19.0.5",
     "typescript": "^4.9.4"
   },

package/src/attribute.mjs

@@ -135,6 +135,37 @@ export function defaultValues(attributes, object) {
   );
 }
 
+/**
+ * Split property path into tokens
+ * @param {string} string 
+ * @return {Iterator<string>}
+ */
+function* tokens(string) {
+  let identifier = "";
+
+  for (const c of string) {
+    switch (c) {
+      case "\t":
+      case " ": break;
+      case ".":
+      case "[":
+      case "]":
+        if (identifier.length) {
+          yield identifier;
+          identifier = "";
+        }
+        yield c;
+        break;
+      default:
+        identifier += c;
+    }
+  }
+
+  if (identifier.length) {
+    yield identifier;
+  }
+}
+
 /**
  * Set Object attribute.
  * The name may be a property path like 'a.b.c'.
@@ -143,7 +174,6 @@ export function defaultValues(attributes, object) {
  * @param {any} value
  */
 export function setAttribute(object, name, value) {
-
   let lastObject = object;
   let lastKey;
 
@@ -169,37 +199,6 @@ export function setAttribute(object, name, value) {
   lastObject[lastKey] = value;
 }
 
-/**
- * Split property path into tokens
- * @param {string} string 
- * @return {Iterator<string>}
- */
-function* tokens(string) {
-  let identifier = "";
-
-  for (const c of string) {
-    switch (c) {
-      case "\t":
-      case " ": break;
-      case ".":
-      case "[":
-      case "]":
-        if (identifier.length) {
-          yield identifier;
-          identifier = "";
-        }
-        yield c;
-        break;
-      default:
-        identifier += c;
-    }
-  }
-
-  if (identifier.length) {
-    yield identifier;
-  }
-}
-
 /**
  * Deliver attribute value.
  * The name may be a property path like 'a.b.c'.

package/src/attributes.mjs

@@ -61,3 +61,4 @@ export const priority = {
 export const active = { type: "boolean", default: true, writable: true };
 export const size = { type: "integer" };
 export const language = { type: "string" };
+export const type = { type: "string" };

package/src/branch.mjs

@@ -19,6 +19,10 @@ export class Branch extends Ref {
   static get addMethodName() {
     return "_addBranch";
   }
+
+  static get deleteMethodName() {
+    return "_deleteBranch";  	
+  }
   
   static get collectionName() {
     return "branches";

package/src/hook.mjs

@@ -20,4 +20,8 @@ export class Hook extends OwnedObject {
   static get addMethodName() {
     return "_addHook";
   }
+
+  static get delteteMethodName() {
+  	return "_deleteHook";
+  }
 }

package/src/multi-group-provider.mjs

@@ -104,7 +104,7 @@ export class MultiGroupProvider extends BaseProvider {
    * @return {RepositoryGroup}
    */
   addRepositoryGroup(name, options) {
-    return this.#repositoryGroups.get(this.normalizeGroupName(name, true)) || new this.repositoryGroupClass(this, name, options);
+    return this.#repositoryGroups.get(this.normalizeGroupName(name, true)) || new this.repositoryGroupClass(this, name, options);
   }
 
   _addRepositoryGroup(repositoryGroup) {

package/src/owned-object.mjs

@@ -13,15 +13,20 @@ export class OwnedObject extends NamedObject {
     return "_add" + this.name;
   }
 
-  constructor(owner, name, options, additionalProperties) {
-    super(name, options, {
-      ...additionalProperties,
-      owner: { value: owner }
-    });
+  static get deleteMethodName() {
+    return "_delete" + this.name;
+  }
 
+  constructor(owner, name, options, additionalProperties) {
+    super(name, options, additionalProperties);
+    this.owner = owner;
     owner[this.constructor.addMethodName](this);
   }
 
+  delete() {
+    this.owner[this.constructor.deleteMethodName](this);
+  }
+
   /**
    * Check for equality.
    * @param {OwnedObject} other

package/src/pull-request.mjs

@@ -31,6 +31,10 @@ export class PullRequest extends OwnedObject {
     return "_addPullRequest";
   }
 
+  static get deleteMethodName() {
+    return "_deletePullRequest";	
+  }
+  
   static get type() {
     return "pull-request";
   }
@@ -39,14 +43,6 @@ export class PullRequest extends OwnedObject {
     return "pullRequests";
   }
 
-  /**
-   * All valid states
-   * @return {Set<string>} valid states
-   */
-  static get validStates() {
-    return new Set(["OPEN", "MERGED", "CLOSED"]);
-  }
-
   /**
    * States to list pull request by default
    * @return {Set<string>} states to list by default
@@ -103,6 +99,7 @@ export class PullRequest extends OwnedObject {
       state: {
         ...state,
         default: "OPEN",
+        values: new Set(["OPEN", "MERGED", "CLOSED"]),
         writable: true
       },
 
@@ -143,7 +140,7 @@ export class PullRequest extends OwnedObject {
       state: {
         set(value) {
           value = value.toUpperCase();
-          if (this.constructor.validStates.has(value)) {
+          if (this.constructor.attributes.state.values.has(value)) {
             state = value;
           } else throw new Error(`Invalid Pull Request state ${value}`);
         },

package/src/ref.mjs

@@ -93,7 +93,7 @@ export class Ref extends OwnedObject {
    * Ref owner.
    * By default we provide the repository owner
    * @see {@link Repository#owner}
-   * @return {string}
+   * @return {Repository}
    */
   get repository() {
     return this.owner;

package/src/repository-group.mjs

@@ -1,7 +1,7 @@
 import { RepositoryOwner } from "./repository-owner.mjs";
 import { OwnedObject } from "./owned-object.mjs";
 import { BaseProvider } from "./base-provider.mjs";
-import { url, boolean_attribute } from "./attributes.mjs";
+import { url, boolean_attribute, type } from "./attributes.mjs";
 
 /**
  * Abstract repository collection.
@@ -23,6 +23,10 @@ export class RepositoryGroup extends RepositoryOwner(OwnedObject) {
     return "_addRepositoryGroup";
   }
 
+  static get deleteMethodName() {
+    return "_deleteRepositoryGroup";  	
+  }
+
   static get type() {
     return "repository-group";
   }
@@ -37,18 +41,10 @@ export class RepositoryGroup extends RepositoryOwner(OwnedObject) {
 
       /**
        * Type of the repository group either User or Organization.
-       * @return {string}
        */
-      type: { type: "string" },
-
+      type,
       url,
-
-      /**
-       * Avatar.
-       * @return {string}
-       */
-      avatarURL: { ...url },
-
+      avatarURL: url,
       isAdmin: boolean_attribute
     };
   }
@@ -60,11 +56,6 @@ export class RepositoryGroup extends RepositoryOwner(OwnedObject) {
     return {};
   }
 
-  get provider()
-  {
-    return this.owner;
-  }
-
   get isAdmin() {
     return false;
   }

package/src/repository.mjs

@@ -8,13 +8,13 @@ import { url, size, language, boolean_attribute } from "./attributes.mjs";
 
 /**
  * Abstract repository
- * @param {Owner} owner
+ * @param {RepositoryOwner} owner
  * @param {string} name (#branch) will be removed
  * @param {Object} options
  * @param {string} [options.description] human readable description
  * @param {string} [options.id] internal id
  *
- * @property {Owner} owner
+ * @property {RepositoryOwner} owner
  * @property {string} name without (#branch)
  * @property {string} [description] from options.description
  * @property {string} [id] from options.id
@@ -28,6 +28,11 @@ export class Repository extends OwnedObject {
     return "_addRepository";
   }
 
+  static get deleteMethodName()
+  {
+     return "_deleteRepository";  	
+  }
+
   static get collectionName() {
     return "repositories";
   }

package/src/tag.mjs

@@ -9,6 +9,10 @@ export class Tag extends Ref {
     return "_addTag";
   }
 
+  static get deleteMethodName() {
+    return "_deleteTag";  	
+  }
+
   /**
    * @return {string} tags
    */