chore(deps): update to typescript 4.9

BREAKING CHANGE: tslib^2.0.0 is now the minimum required version

Author: Michael Wittwer

.github/workflows/main.yml

@@ -21,7 +21,6 @@ jobs:
  node:
  - 16
  - 18
- - 19
  name: Node.js v${{ matrix.node }}
  runs-on: ubuntu-latest
  steps:

package-lock.json

@@ -78,12 +78,12 @@
  "tsutils": "^3.17.1",
  "typedoc": "0.14.0",
  "typedoc-plugin-external-module-name": "^2.1.0",
- "typescript": ">=2.9.1 <4.0.0"
+ "typescript": "^4.9.4"
  },
  "peerDependencies": {
  "aws-sdk": "^2.401.0",
  "reflect-metadata": "^0.1.12",
- "tslib": "^1.10.0"
+ "tslib": "^2.0.0"
  },
  "engines": {
  "node": ">=18"

package.json

@@ -18,7 +18,7 @@ import { BatchGetSingleTableResponse } from './batch-get-single-table.response'
 /**
  * Request class for BatchGetItem operation which supports a single model class only.
  */
-export class BatchGetSingleTableRequest<T, T2 = T> extends BaseRequest<
+export class BatchGetSingleTableRequest<T, T2 extends Partial<T> = T> extends BaseRequest<
  T,
  T2,
  DynamoDB.BatchGetItemInput,

src/dynamo/request/batchgetsingletable/batch-get-single-table.request.ts

@@ -15,7 +15,12 @@ import { GetResponse } from './get.response'
 /**
  * Request class for the GetItem operation.
  */
-export class GetRequest<T, T2 = T> extends StandardRequest<T, T2, DynamoDB.GetItemInput, GetRequest<T, T2>> {
+export class GetRequest<T, T2 extends Partial<T> = T> extends StandardRequest<
+ T,
+ T2,
+ DynamoDB.GetItemInput,
+ GetRequest<T, T2>
+> {
  private readonly logger: Logger
 
  constructor(dynamoDBWrapper: DynamoDbWrapper, modelClazz: ModelConstructor<T>, partitionKey: any, sortKey?: any) {

src/dynamo/request/get/get.request.ts

@@ -15,7 +15,7 @@ import { WriteRequest } from '../write.request'
 /**
  * Request class for the UpdateItem operation.
  */
-export class UpdateRequest<T, T2 = void> extends WriteRequest<
+export class UpdateRequest<T, T2 extends T | Partial<T> | void = void> extends WriteRequest<
  T,
  T2,
  DynamoDB.UpdateItemInput,

src/dynamo/request/update/update.request.ts

@@ -44,7 +44,7 @@ export function toDb<T>(item: T, modelConstructor?: ModelConstructor<T>): Attrib
  * initialize possible properties with default value providers
  */
  if (metadata) {
- metadata.getPropertiesWithDefaultValueProvider().forEach(propertyMetadata => {
+ metadata.getPropertiesWithDefaultValueProvider().forEach((propertyMetadata) => {
  const currentVal = Reflect.get(<any>item, propertyMetadata.name)
  if (currentVal === undefined || currentVal === null) {
  // tslint:disable-next-line:no-non-null-assertion
@@ -184,7 +184,7 @@ export function createToKeyFn<T>(modelConstructor: ModelConstructor<T>): (item:
  throw new Error(`there is no value for property ${propMeta.name.toString()} but is ${propMeta.key.type} key`)
  }
  const propertyValue = getPropertyValue(item, propMeta.name)
- key[propMeta.nameDb] = <Attribute>toDbOne(propertyValue, propMeta)
+ ;(key as any)[propMeta.nameDb] = <Attribute>toDbOne(propertyValue, propMeta)
  return key
  }, <Attributes<T>>{})
  }
@@ -220,15 +220,15 @@ export function createKeyAttributes<T>(
 
  if (hasSortKey(metadata)) {
  if (sortKey === null || sortKey === undefined) {
- throw new Error(`please provide the sort key for attribute ${metadata.getSortKey()}`)
+ throw new Error(`please provide the sort key for attribute ${String(metadata.getSortKey())}`)
  }
  const sortKeyProp = metadata.getSortKey()
  const sortKeyMetadata = metadata.forProperty(sortKeyProp)
  if (!sortKeyMetadata) {
  throw new Error('metadata for sort key must be defined')
  }
 
- keyAttributeMap[sortKeyMetadata.nameDb] = <Attribute>toDbOne(sortKey, sortKeyMetadata)
+ ;(keyAttributeMap as any)[sortKeyMetadata.nameDb] = <Attribute>toDbOne(sortKey, sortKeyMetadata)
  }
 
  return keyAttributeMap

src/mapper/mapper.ts

@@ -21,9 +21,9 @@ export function getPropertyPath<T>(
  propertyKey: keyof T,
 ): string {
  if (modelConstructorOrPropertyMetadata && modelConstructorOrPropertyMetadata.name) {
- return `[${modelConstructorOrPropertyMetadata.name}::${propertyKey}]`
+ return `[${String(modelConstructorOrPropertyMetadata.name)}::${String(propertyKey)}]`
  } else {
- return `[unknown::${propertyKey}]`
+ return `[unknown::${String(propertyKey)}]`
  }
 }