Flatten directory structure, simplify tsconfig

Author: Lukas Stracke

packages/angular/angular.json

@@ -13,7 +13,7 @@
           "builder": "@angular-devkit/build-angular:ng-packagr",
           "options": {
             "tsConfig": "lib/tsconfig.lib.json",
-            "project": "lib/ng-package.json"
+            "project": "ng-package.json"
           },
           "configurations": {
             "production": {

packages/angular/lib/package.json

@@ -5,7 +5,11 @@
     "declarationMap": false
   },
   "angularCompilerOptions": {
+    // As per Angular 10, the recommendation from the library creation guide
+    // is to disable compilation for Ivy in production builds to ensure
+    // compatibility with Angular 10.
+    // For Angular 11-13 applications, ngcc and the Angular linker ensure that
+    // the library code is processed to match the specific Angular runtime version.
     "enableIvy": false,
-    // "compilationMode": "partial"
   }
 }

packages/angular/lib/src/public-api copy.ts

@@ -1,8 +1,8 @@
 {
-  "$schema": "../node_modules/ng-packagr/ng-package.schema.json",
-  "dest": "../dist",
+  "$schema": "node_modules/ng-packagr/ng-package.schema.json",
+  "dest": "dist",
   "lib": {
-    "entryFile": "src/index.ts",
+    "entryFile": "lib/src/index.ts",
     "umdModuleIds": {
       "@sentry/browser": "Sentry",
       "@sentry/utils": "utils"
@@ -12,7 +12,9 @@
     "@sentry/browser",
     "@sentry/utils",
     "@sentry/types",
-    "tslib"
+    "tslib",
+    "rxjs"
   ],
-  "assets": ["README.md"]
+  "assets": ["README.md"],
+  "keepLifecycleScripts": true
 }

packages/angular/lib/tsconfig.lib.prod.json

@@ -23,41 +23,27 @@
   "private": true,
 
   "dependencies": {
-    "@angular/animations": "~10.2.5",
-    "@angular/common": "~10.2.5",
-    "@angular/compiler": "~10.2.5",
-    "@angular/core": "~10.2.5",
-    "@angular/forms": "~10.2.5",
-    "@angular/platform-browser": "~10.2.5",
-    "@angular/platform-browser-dynamic": "~10.2.5",
-    "@angular/router": "~10.2.5",
     "@sentry/browser": "^6.18.0",
     "@sentry/types": "^6.18.0",
     "@sentry/utils": "^6.18.0",
     "rxjs": "~6.6.0",
-    "tslib": "^2.0.0",
-    "zone.js": "~0.10.2"
+    "tslib": "^2.0.0"
   },
   "devDependencies": {
+    "@angular/common": "~10.2.5",
+    "@angular/compiler": "~10.2.5",
+    "@angular/core": "~10.2.5",
+    "@angular/router": "~10.2.5",
     "@angular-devkit/build-angular": "~0.1002.4",
     "@angular/cli": "~10.2.4",
     "@angular/compiler-cli": "~10.2.5",
-    "@types/jasmine": "~3.5.0",
-    "@types/jasminewd2": "~2.0.3",
     "@types/node": "^12.11.1",
     "codelyzer": "^6.0.0",
-    "jasmine-core": "~3.6.0",
-    "jasmine-spec-reporter": "~5.0.0",
-    "karma": "~5.0.0",
-    "karma-chrome-launcher": "~3.1.0",
-    "karma-coverage-istanbul-reporter": "~3.0.2",
-    "karma-jasmine": "~4.0.0",
-    "karma-jasmine-html-reporter": "~1.5.0",
     "ng-packagr": "^10.1.0",
-    "protractor": "~7.0.0",
     "ts-node": "~8.3.0",
     "tslint": "~6.1.0",
-    "typescript": "~4.0.2"
+    "typescript": "~4.0.2",
+    "zone.js": "~0.10.2"
   },
   "scripts": {
     "build": "run-p build:esm #**not necessary** build:cjs",

packages/angular/lib/ng-package.json renamed to packages/angular/ng-package.json

@@ -1,25 +1,11 @@
 {
-  "compileOnSave": false,
+  "extends": "../../tsconfig.json",
+
+  "include": ["lib/src/**/*"],
+
   "compilerOptions": {
-    "baseUrl": "./",
-    "outDir": "./dist/out-tsc",
-    "sourceMap": true,
-    "declaration": false,
-    "downlevelIteration": true,
+    // package-specific options
     "experimentalDecorators": true,
-    "moduleResolution": "node",
-    "importHelpers": true,
-    "target": "es2015",
-    "module": "es2020",
-    "lib": [
-      "es2018",
-      "dom"
-    ],
-    "paths": {
-      "sentry-angular": [
-        "dist/sentry-angular/sentry-angular",
-        "dist/sentry-angular"
-      ]
-    }
+    "baseUrl": "./",
   }
 }

packages/angular/package.json

@@ -0,0 +1,12 @@
+{
+  "extends": "./tsconfig.json",
+
+  "include": ["test/**/*"],
+
+  "compilerOptions": {
+    // should include all types from `./tsconfig.json` plus types for all test frameworks used
+    "types": ["jest"]
+
+    // other package-specific, test-specific options
+  }
+}