Update @rollup/plugin-typescript to 11.1.0 + Fix dist directory layout

@rollup/plugin-typescript v11.1.0 respects outDir for code emission, which makes it impossible to emit to separate cjs and esm subdirectories with correct source paths in the source maps. This is fixed by emitting both output js files directly to dist, and emitting the types to dist/types. Package imports in react-hook-tracer-demo can now still be command-clicked to navigate to their source.

Author: Oblosys

packages/react-hook-tracer/package.json

@@ -12,9 +12,9 @@
     "logging",
     "devtool"
   ],
-  "main": "dist/cjs/index.js",
-  "module": "dist/esm/index.js",
-  "typings": "dist/esm/src/index.d.ts",
+  "main": "dist/index.cjs.js",
+  "module": "dist/index.esm.js",
+  "typings": "dist/types/src/index.d.ts",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/Oblosys/react-hook-tracer.git",
@@ -25,10 +25,10 @@
     "url": "https://github.com/Oblosys/react-hook-tracer/issues"
   },
   "files": [
-    "dist/cjs/index.js",
-    "dist/esm/index.js",
-    "dist/esm/src/**/*.d.ts",
-    "dist/esm/src/**/*.d.ts.map",
+    "dist/index.cjs.js",
+    "dist//index.esm.js",
+    "dist/types/src/**/*.d.ts",
+    "dist/types/src/**/*.d.ts.map",
     "!**/test/*",
     "!**/__mocks__/*",
     "!**/*.test.*"
@@ -51,7 +51,7 @@
   "devDependencies": {
     "@rollup/plugin-commonjs": "^23.0.2",
     "@rollup/plugin-node-resolve": "^15.0.1",
-    "@rollup/plugin-typescript": "^9.0.2",
+    "@rollup/plugin-typescript": "^11.1.0",
     "@testing-library/dom": "^8.19.0",
     "@testing-library/jest-dom": "^5.16.5",
     "@testing-library/react": "^13.4.0",

packages/react-hook-tracer/rollup.config.ts

@@ -23,11 +23,14 @@ export default (args: { configIncludeDeclarationMap?: boolean }) => {
     typescript({
       declaration: true,
       declarationMap: includeDeclarationMap,
-
-      // NOTE: outDir does not affect the emited code, but fixes the sources path in generated .d.ts.map files.
-      // Without this, the relative path to the source assumes the map files are under dist rather than dist/esm.
-      // (Using sourceRoot: '..' won't fix this, as setting sourceRoot removes makes the 'sources' in the map absolute.)
-      outDir: 'dummy/dummy',
+      outDir: 'dist/types',
+      // outDir 'dist/types' produces the same directory layout as 'types', as the 'dist' is stripped, but adds an extra
+      // '../' to the sources path in the source maps.
+      //
+      // For example, in dist/types/src/Tracer.d.ts.map
+      // outDir: 'types' yields the incorrect "sources":["../../src/Tracer.ts"]
+      // whereas outDir: 'dist/types' yields "sources":["../../../src/Tracer.ts"]
+      // The output js file is not affected as it is specified in defineConfig below.
     }),
     // No terser, as rollup-plugin-terser has peer-dep warnings and seems unmaintained.
     // It also only reduces the tgz by 20%.

yarn.lock

@@ -2506,6 +2506,25 @@ __metadata:
   languageName: node
   linkType: hard
 
+"@rollup/plugin-typescript@npm:^11.1.0":
+  version: 11.1.0
+  resolution: "@rollup/plugin-typescript@npm:11.1.0"
+  dependencies:
+    "@rollup/pluginutils": ^5.0.1
+    resolve: ^1.22.1
+  peerDependencies:
+    rollup: ^2.14.0||^3.0.0
+    tslib: "*"
+    typescript: ">=3.7.0"
+  peerDependenciesMeta:
+    rollup:
+      optional: true
+    tslib:
+      optional: true
+  checksum: 64ed5abbae392032fb190a5d3a29beb73f968919aed964ed5545e2732de6a1670f97131785e9376c06f784ae61545c5ad47352c126f3830aaa16b669bee2e9b9
+  languageName: node
+  linkType: hard
+
 "@rollup/plugin-typescript@npm:^9.0.2":
   version: 9.0.2
   resolution: "@rollup/plugin-typescript@npm:9.0.2"
@@ -12031,7 +12050,7 @@ __metadata:
   dependencies:
     "@rollup/plugin-commonjs": ^23.0.2
     "@rollup/plugin-node-resolve": ^15.0.1
-    "@rollup/plugin-typescript": ^9.0.2
+    "@rollup/plugin-typescript": ^11.1.0
     "@testing-library/dom": ^8.19.0
     "@testing-library/jest-dom": ^5.16.5
     "@testing-library/react": ^13.4.0