feat(reflect-metadata): remove import

- reflect-metadata should only exist once in the whole code base of a project so the consumer of our library must import the reflect-metadata project, even if he might not use it in his code base. But if he uses the reflect-metadata in his code or uses another library like InversifyJS we must ensure the Reflect global variable only exists once, to prevent from possible hard to debug issues

Author: michaelwittwer

.lintstagedrc.yml

@@ -3,6 +3,9 @@ linters:
     - prettier --write --config ./.prettierrc.yml
     - tslint --project ./tsconfig.json -t codeFrame --fix
     - git add
+  "(src/**/*.spec.ts,test/**/*.ts)":
+    - prettier --write --config ./.prettierrc.yml
+    - tslint --project ./tsconfig.jest.json -t codeFrame --fix
   "**/package.json":
     - sort-package-json
     - git add

README.md

@@ -8,12 +8,6 @@
 [![styled with prettier](https://img.shields.io/badge/styled_with-prettier-ff69b4.svg)](https://github.com/prettier/prettier)
 [![All Contributors](https://img.shields.io/badge/all_contributors-3-orange.svg)](#contributors)
 
-This is the documentation for the new version of dynamo-easy released as a scoped package using the namespace 
-@shiftcoders (@shiftcoders/dynamo-easy). If you still want to use the 0.0.x version you can 
-by installing the old version `npm install dynamo-easy` (https://github
-.com/shiftcode/dynamo-easy/releases/tag/v0.10
-.1) 
-
 Abstracts away the complexity of the low level aws dynamosdk. Provides an easy to use fluent api to for requests and supports typescript decorators,
 to define some metadata for your models. You don't need to care about the mapping of javascript types to their dynamo types any more. We got you covered.
 
@@ -22,6 +16,37 @@ Checkout the full technical api documentation [here](https://shiftcode.github.io
 Built with :heart: by [shiftcode](https://www.shiftcode.ch).
 
 # Get Started
+
+## Metadata Reflection API
+
+> ⚠ The reflect-metadata polyfill should be imported only once in your entire application because the Reflect object is 
+mean to be a global singleton.
+
+Required always. Use reflect-metadata as polyfill.
+
+```
+npm install reflect-metadata --save
+```
+
+The type definitions for reflect-metadata are included in the npm package. 
+You need to add the following reference to the types field in your tsconfig.json:
+
+```
+"types": ["reflect-metadata"]
+```
+
+Finally, import reflect-metadata in some entry file in your application.
+
+```
+import "reflect-metadata"
+```
+
+Also make sure to install the other peer dependencies of @shiftcoders/dynamo-easy.
+
+## First Sample
+
+When all the setup work is done, define your first model and create a dynamo store to execute actions on the dynamoDB.
+
 ```typescript
 @Model()
 class Person{
@@ -57,8 +82,6 @@ dynamoStore.scan()
 
 ```
 
-Want to use this library with Angular (>4) [checkout our angular-service](TODO).
-
 # Decorators
 Decorators are used to add some metadata to our model classes, relevant to our javascript-to-dynamo mapper.
 

package-lock.json

@@ -47,7 +47,6 @@
     "@types/jest": "^23.3.1",
     "@types/lodash": "^4.14.116",
     "@types/node": "^8.0.26",
-    "@types/reflect-metadata": "^0.1.0",
     "@types/uuid": "^3.4.1",
     "aws-sdk": "^2.286.1",
     "colors": "^1.1.2",
@@ -60,6 +59,7 @@
     "lodash": "^4.17.11",
     "lodash.camelcase": "^4.3.0",
     "prettier": "^1.15.2",
+    "reflect-metadata": "^0.1.12",
     "rimraf": "^2.6.2",
     "rxjs": "^6.3.3",
     "semantic-release": "^15.12.0",

package.json

@@ -1,10 +1,3 @@
-//
-// Reflect Metadata
-// FIXME: Remove reflect-metadata as hard dep (lib using project needs to import it)
-// - possible singleton override issues and it's a shim eg. runtime dependent in future --> es7
-//
-import 'reflect-metadata'
-
 //
 // Export public api of the library
 //

src/dynamo-easy.ts

@@ -1,5 +1,6 @@
 import { ObjectMapper } from './object.mapper'
 
+// TODO remove test comment
 describe('object mapper', () => {
   describe('to db', () => {
     it('should work', () => {

src/mapper/for-type/object.mapper.spec.ts

@@ -10,6 +10,8 @@ import {
 } from '../../src/dynamo-easy'
 import { NestedObject } from './nested-object.model'
 
+// TODO remove test comment
+
 @Model({ tableName: 'complex_model' })
 export class ComplexModel {
   @PartitionKey()

test/models/complex.model.ts

@@ -1,8 +1,15 @@
 {
   "extends": "./tsconfig.json",
   "compilerOptions": {
-    "module": "commonjs",
-    "declaration": false,
-    "allowJs": true           // allow js to be compiled with ts for jest es2015 module import (lodash-es)
-  }
+    "declaration": false,         /* Generates corresponding '.d.ts' file. */
+    "types": [                    /* Type declaration files to be included in compilation. */
+      "node",
+      "reflect-metadata",
+      "jest"
+    ]
+  },
+  "include": [
+    "src/**/*.spec.ts",
+    "test/**/*"
+  ]
 }

tsconfig.jest.json

@@ -47,7 +47,10 @@
     // "paths": {},                           /* A series of entries which re-map imports to lookup locations relative to the 'baseUrl'. */
     // "rootDirs": [],                        /* List of root folders whose combined content represents the structure of the project at runtime. */
     // "typeRoots": [],                       /* List of folders to include type definitions from. */
-    // "types": [],                           /* Type declaration files to be included in compilation. */
+     "types": [                               /* Type declaration files to be included in compilation. */
+       "node",
+       "reflect-metadata"
+     ],
     // "allowSyntheticDefaultImports": true,  /* Allow default imports from modules with no default export. This does not affect code emit, just typechecking. */
     // "esModuleInterop": true                /* Enables emit interoperability between CommonJS and ES Modules via creation of namespace objects for all imports. Implies 'allowSyntheticDefaultImports'. */
     // "preserveSymlinks": true,              /* Do not resolve the real path of symlinks. */