Merge branch 'master' into #91-default-value-provider

# Conflicts:
#	src/decorator/metadata/metadata.ts
#	src/mapper/mapper.ts

Author: simonmumenthaler

.lintstagedrc.yml

@@ -1,14 +1,11 @@
 "src/**/!(*.spec).ts":
   - prettier --write --config ./.prettierrc.yml
   - tslint --project ./tsconfig.json -t codeFrame --fix
-  - git add
 "(src/**/*.spec.ts|test/**/*.ts)":
   - prettier --write --config ./.prettierrc.yml
   # TODO LOW tslint will not work because of the following error
   # ✖ tslint --project ./tsconfig.jest.json -t codeFrame --fix found some errors. Please fix them and try committing again.
   #'/Users/michaelwittwer/dev/shiftcode/dynamo-easy/test/models/complex.model.ts' is not included in project.
   # - tslint --project ./tsconfig.jest.json -t codeFrame --fix
-  - git add
 "**/package.json":
   - sort-package-json
-  - git add

package-lock.json

@@ -47,7 +47,6 @@
     "lint:ci": "tslint --project tsconfig.json -t codeFrame",
     "prettier": "prettier --write --config ./.prettierrc.yml '{src,test}/**/*.ts'",
     "report-coverage": "cat ./coverage/lcov.info | coveralls",
-    "semantic-release-prepare": "ts-node tools/semantic-release-prepare",
     "test": "jest",
     "test:ci": "jest --coverage --no-cache",
     "test:watch": "jest --watch",
@@ -58,21 +57,21 @@
     "@commitlint/cli": "^8.1.0",
     "@commitlint/config-conventional": "^8.1.0",
     "@commitlint/prompt-cli": "^8.1.0",
-    "@types/jest": "^24.0.18",
+    "@types/jest": "^25.2.1",
     "@types/node": "8.10.40",
     "aws-sdk": "^2.401.0",
     "colors": "^1.3.3",
     "coveralls": "^3.0.6",
-    "husky": "^3.0.4",
-    "jest": "^24.9.0",
-    "lint-staged": "^9.2.3",
+    "husky": "^4.0.0",
+    "jest": "^25.2.7",
+    "lint-staged": "^10.0.0",
     "lodash.camelcase": "^4.3.0",
-    "prettier": "^1.18.2",
+    "prettier": "^2.0.2",
     "reflect-metadata": "^0.1.13",
     "rimraf": "^3.0.0",
-    "semantic-release": "^16.0.1",
+    "semantic-release": "^17.0.0",
     "sort-package-json": "^1.22.1",
-    "ts-jest": "^24.0.2",
+    "ts-jest": "^25.0.0",
     "ts-node": "^8.3.0",
     "tslint": "^5.19.0",
     "tslint-config-prettier": "^1.18.0",

package.json

@@ -5,12 +5,22 @@
   ],
   "ignoreDeps": [
     "@types/node",
-    "typescript"
+    "typescript",
+    "typedoc",
+    "typedoc-plugin-external-module-name"
   ],
   "packageRules": [
     {
-      "packagePatterns": [ "^@types" ],
-      "groupName": "@types"
+      "packagePatterns": [ "^@commitlint" ],
+      "groupName": "@commitlint"
+    },
+    {
+      "packagePatterns": ["jest"],
+      "groupName": "jest"
+    },
+    {
+      "packagePatterns": ["uuid"],
+      "groupName": "uuid"
     }
   ]
 }

renovate.json

@@ -20,7 +20,7 @@ export interface IndexData {
  */
 export function initOrUpdateIndex(indexType: IndexType, indexData: IndexData, target: any, propertyKey: string): void {
   const properties: Array<PropertyMetadata<any>> = Reflect.getMetadata(KEY_PROPERTY, target.constructor) || []
-  const existingProperty = properties.find(property => property.name === propertyKey)
+  const existingProperty = properties.find((property) => property.name === propertyKey)
 
   let propertyMetadata: Partial<PropertyMetadata<any>>
   switch (indexType) {

src/decorator/impl/index/util.ts

@@ -14,9 +14,9 @@ export function PartitionKey(): PropertyDecorator {
       // check for existing properties marked as partition key
       const properties: Array<PropertyMetadata<any>> = Reflect.getMetadata(KEY_PROPERTY, target.constructor) || []
       if (properties && properties.length) {
-        const existingPartitionKeys = properties.filter(property => property.key && property.key.type === 'HASH')
+        const existingPartitionKeys = properties.filter((property) => property.key && property.key.type === 'HASH')
         if (existingPartitionKeys.length) {
-          if (properties.find(property => property.name === propertyKey)) {
+          if (properties.find((property) => property.name === propertyKey)) {
             // just ignore this and go on, somehow the partition key gets defined two times
             logger.warn(
               `this is the second execution to define the partitionKey for property ${propertyKey}`,

src/decorator/impl/key/partition-key.decorator.ts

@@ -28,7 +28,7 @@ export function Model(opts: ModelData = {}): ClassDecorator {
         : Reflect.getMetadata(KEY_PROPERTY, constructor)) || []
 
     // get partition key
-    const partitionKeys = properties.filter(property => property.key && property.key.type === 'HASH')
+    const partitionKeys = properties.filter((property) => property.key && property.key.type === 'HASH')
 
     const partitionKeyName: string | null = partitionKeys.length ? partitionKeys[0].nameDb : null
 
@@ -40,7 +40,7 @@ export function Model(opts: ModelData = {}): ClassDecorator {
     const indexes: Map<string, SecondaryIndex<any>> = new Map([...globalSecondaryIndexes, ...localSecondaryIndexes])
 
     const transientProperties = properties.length
-      ? properties.filter(property => property.transient === true).map(property => property.name)
+      ? properties.filter((property) => property.transient === true).map((property) => property.name)
       : []
 
     const metaData: ModelMetadata<any> = {
@@ -80,7 +80,7 @@ function testForLSI<T>(property: PropertyMetadata<T>): property is PropertyMetad
 function getGlobalSecondaryIndexes(properties: Array<PropertyMetadata<any>>): Map<string, SecondaryIndex<any>> {
   return properties.filter(testForGSI).reduce((map, property): Map<string, SecondaryIndex<any>> => {
     let gsi: SecondaryIndex<any>
-    Object.keys(property.keyForGSI).forEach(indexName => {
+    Object.keys(property.keyForGSI).forEach((indexName) => {
       if (map.has(indexName)) {
         gsi = map.get(indexName)
       } else {
@@ -119,7 +119,7 @@ function getLocalSecondaryIndexes(
   return properties.filter(testForLSI).reduce((map, property): Map<string, SecondaryIndex<any>> => {
     let lsi: SecondaryIndex<any>
 
-    property.sortKeyForLSI.forEach(indexName => {
+    property.sortKeyForLSI.forEach((indexName) => {
       if (map.has(indexName)) {
         throw new Error(modelErrors.lsiMultipleSk(indexName, property.nameDb))
       }

src/decorator/impl/model/model.decorator.ts

@@ -20,12 +20,12 @@ export function initOrUpdateProperty(
 
   // Update the attribute array
   let properties: Array<PropertyMetadata<any>> = Reflect.getMetadata(KEY_PROPERTY, target.constructor) || []
-  const existingProperty = properties.find(p => p.name === propertyKey)
+  const existingProperty = properties.find((p) => p.name === propertyKey)
   if (existingProperty) {
     // create new property with merged options
     property = { ...existingProperty, ...propertyMetadata }
     // remove existing from array
-    properties = properties.filter(p => p !== existingProperty)
+    properties = properties.filter((p) => p !== existingProperty)
   } else {
     // add new options
     property = createNewProperty(propertyMetadata, target, propertyKey)

src/decorator/impl/property/init-or-update-property.function.ts

@@ -32,7 +32,7 @@ export class Metadata<T> {
     modelOpts: ModelMetadata<M>,
     propertyName: keyof M,
   ): PropertyMetadata<M> | undefined {
-    return modelOpts.properties.find(property => property.name === propertyName || property.nameDb === propertyName)
+    return modelOpts.properties.find((property) => property.name === propertyName || property.nameDb === propertyName)
   }
 
   constructor(modelConstructor: ModelConstructor<T>) {
@@ -95,7 +95,7 @@ export class Metadata<T> {
         throw new Error(`there is no index defined for name ${indexName}`)
       }
     } else {
-      const property = filterByFirst(this.modelOptions, p => !!(p.key && p.key.type === 'HASH'))
+      const property = filterByFirst(this.modelOptions, (p) => !!(p.key && p.key.type === 'HASH'))
 
       if (property) {
         return property.name
@@ -123,7 +123,7 @@ export class Metadata<T> {
         throw new Error(`there is no index defined for name ${indexName}`)
       }
     } else {
-      const property = filterByFirst(this.modelOptions, p => !!(p.key && p.key.type === 'RANGE'))
+      const property = filterByFirst(this.modelOptions, (p) => !!(p.key && p.key.type === 'RANGE'))
       return property ? property.name : null
     }
   }

src/decorator/metadata/metadata.ts

@@ -20,14 +20,14 @@ export function batchGetItemsFetchAll(
   backoffTimer: IterableIterator<number>,
   throttleTimeSlot: number,
 ): Promise<DynamoDB.BatchGetItemOutput> {
-  return dynamoDBWrapper.batchGetItems(params).then(response => {
+  return dynamoDBWrapper.batchGetItems(params).then((response) => {
     if (hasUnprocessedKeys(response)) {
       // in case of unprocessedKeys do a follow-up requests
       return (
         Promise.resolve(response.UnprocessedKeys)
           // delay before doing the follow-up request
           .then(promiseDelay(backoffTimer.next().value * throttleTimeSlot))
-          .then(UnprocessedKeys => {
+          .then((UnprocessedKeys) => {
             const nextParams = { ...params, RequestItems: UnprocessedKeys }
             // call recursively batchGetItemsFetchAll with the returned UnprocessedItems params
             return batchGetItemsFetchAll(dynamoDBWrapper, nextParams, backoffTimer, throttleTimeSlot)
@@ -56,7 +56,7 @@ export function hasUnprocessedKeys(
   if (!response.UnprocessedKeys) {
     return false
   }
-  return Object.values(response.UnprocessedKeys).some(t => !!t && t.Keys && t.Keys.length > 0)
+  return Object.values(response.UnprocessedKeys).some((t) => !!t && t.Keys && t.Keys.length > 0)
 }
 
 /**
@@ -68,8 +68,8 @@ export function combineBatchGetResponses(response1: DynamoDB.BatchGetItemOutput)
     const tableNames: string[] = Object.keys(response1.Responses || {})
 
     Object.keys(response2.Responses || {})
-      .filter(tn => !tableNames.includes(tn))
-      .forEach(tn => tableNames.push(tn))
+      .filter((tn) => !tableNames.includes(tn))
+      .forEach((tn) => tableNames.push(tn))
 
     const Responses = tableNames.reduce(
       (u, tableName) => ({