build(rollup): set module resolving and packages excluded

set preferBuiltins to true and exclude amqlib and amqp-connection-manager because they should be
installed by the package using rc

Author: juanmarin-co

package-lock.json

@@ -19,7 +19,7 @@
     "node": ">=6.0.0"
   },
   "scripts": {
-    "lint": "tslint  --project tsconfig.json -t codeFrame 'src/**/*.ts' 'test/**/*.ts'",
+    "lint": "tslint --project tsconfig.json -t codeFrame 'src/**/*.ts' 'test/**/*.ts'",
     "prebuild": "rimraf dist",
     "build": "tsc --module commonjs && rollup -c rollup.config.ts && typedoc --out docs --target es6 --theme minimal --mode file src",
     "start": "rollup -c rollup.config.ts -w",
@@ -51,13 +51,12 @@
   },
   "jest": {
     "transform": {
-      ".(ts|tsx)": "ts-jest"
+      ".ts": "ts-jest"
     },
     "testEnvironment": "node",
-    "testRegex": "(/__tests__/.*|\\.(test|spec))\\.(ts|tsx|js)$",
+    "testRegex": "(/__tests__/.*|\\.(test|spec))\\.(ts|js)$",
     "moduleFileExtensions": [
       "ts",
-      "tsx",
       "js"
     ],
     "coveragePathIgnorePatterns": [
@@ -88,6 +87,8 @@
   "devDependencies": {
     "@commitlint/cli": "^8.1.0",
     "@commitlint/config-conventional": "^8.1.0",
+    "@types/amqp-connection-manager": "^2.0.4",
+    "@types/amqplib": "^0.5.13",
     "@types/jest": "^24.0.15",
     "@types/node": "^12.6.8",
     "colors": "^1.3.3",
@@ -122,7 +123,7 @@
   },
   "dependencies": {
     "amqp-connection-manager": "^3.0.0",
-    "amqplib": "^0.5.3",
+    "amqplib": "^0.5.5",
     "lodash": "^4.17.15",
     "uuid": "^3.3.2"
   }

package.json

@@ -12,13 +12,16 @@ const libraryName = 'reactive-commons'
 export default {
   input: `src/${libraryName}.ts`,
   output: [
-    { file: pkg.main, name: camelCase(libraryName), format: 'umd', sourcemap: true },
-    { file: pkg.module, format: 'es', sourcemap: true },
+    { file: pkg.main, name: camelCase(libraryName), format: 'cjs', sourcemap: true },
+    { file: pkg.module, format: 'esm', sourcemap: true }
   ],
   // Indicate here external modules you don't wanna include in your bundle (i.e.: 'lodash')
-  external: [],
+  external: [
+    'amqplib',
+    'amqp-connection-manager'
+  ],
   watch: {
-    include: 'src/**',
+    include: 'src/**'
   },
   plugins: [
     // Allow json resolution
@@ -30,9 +33,9 @@ export default {
     // Allow node_modules resolution, so you can use 'external' to control
     // which external modules to include in the bundle
     // https://github.com/rollup/rollup-plugin-node-resolve#usage
-    resolve(),
+    resolve({ preferBuiltins: true }),
 
     // Resolve source maps to the original source
-    sourceMaps(),
-  ],
+    sourceMaps()
+  ]
 }

rollup.config.ts

@@ -3,7 +3,7 @@
     "moduleResolution": "node",
     "target": "es5",
     "module":"es2015",
-    "lib": ["es2015", "es2016", "es2017", "dom"],
+    "lib": ["es2015", "es2016", "es2017"],
     "strict": true,
     "sourceMap": true,
     "declaration": true,