TS: emit type declarations for .ts files with external exports, and publish them under dist/types

This change makes it possible to easily expose types from internal .ts files.
This is achieved by adding a TS compilation step which generates .d.ts files from
.ts ones.
Additionally, a copying step is needed for existing .d.ts files, which are not
automatically moved to the outDir by tsc.
The dist types will still only include those exported by the index files.

Other tried approaches (to e.g. avoid the manual copying step) that were not viable:
- generating a single .d.ts bundle is not supported by tsc, and other tools/plugins
  are either deprecated or (in the case of `rollup-plugin-dts`) failed to process our
  source code due to unsupported JS syntax
- relying on the `rollup-typescript` plugin to emit declarations felt overly messy
  to configure and potentially unreliable as it generated declarations for .js files as well
- renaming existing .d.ts to .ts (alongside .js ones) is not an option since it results
in an empty output for the corresponding modules on Rollup compilation, as the .js
source files are ignored.

Author: larabr

package-lock.json

@@ -21,33 +21,32 @@
   },
   "exports": {
     ".": {
-      "types": "./openpgp.d.ts",
+      "types": "./dist/types/index.d.ts",
       "browser": "./dist/openpgp.min.mjs",
       "import": "./dist/node/openpgp.mjs",
       "require": "./dist/node/openpgp.min.cjs"
     },
     "./lightweight": {
-      "types": "./openpgp.d.ts",
+      "types": "./dist/types/index.d.ts",
       "browser": "./dist/lightweight/openpgp.min.mjs"
     }
   },
-  "types": "openpgp.d.ts",
+  "types": "dist/types/index.d.ts",
   "type": "module",
   "directories": {
     "lib": "src"
   },
   "files": [
     "dist/",
-    "lightweight/",
-    "openpgp.d.ts",
-    "**/*.d.ts"
+    "lightweight/"
   ],
   "scripts": {
     "build": "rollup --config",
+    "build-types": "rm -rf dist/types && tsc --project tsconfig.dist.json && cpy 'src/**/*.d.ts' dist/types",
     "build-test": "npm run build -- --config-build-only=test",
-    "prepare": "npm run build",
+    "prepare": "npm run build && npm run build-types",
     "test": "mocha --timeout 120000 test/unittests.js",
-    "test-type-definitions": "tsc --project test/typescript/tsconfig.test.json && tsx test/typescript/definitions.ts",
+    "test-type-definitions": "npm run build-types && tsc --project test/typescript/tsconfig.test.json && tsx test/typescript/definitions.ts",
     "benchmark-time": "node test/benchmarks/time.js",
     "benchmark-memory-usage": "node test/benchmarks/memory_usage.js",
     "prebrowsertest": "npm run build-test",
@@ -96,6 +95,7 @@
     "c8": "^10.1.3",
     "chai": "^6.2.1",
     "chai-as-promised": "^8.0.2",
+    "cpy-cli": "^6.0.0",
     "eckey-utils": "^0.7.14",
     "eslint": "^9.38.0",
     "eslint-import-resolver-typescript": "^4.4.4",

package.json

@@ -78,7 +78,8 @@ const nodeBuild = {
       exportConditions: ['node'] // needed for resolution of noble-curves import of '@noble/crypto' in Node 18
     }),
     typescript({
-      compilerOptions: { outDir: './dist/node' } // temporary output location, needed to avoid js files being overwritten under `src`
+      compilerOptions: { outDir: './dist/node' }, // temporary output location, needed to avoid js files being overwritten under `src`
+      rootDir: 'src' // ignore .ts files outside of src folder
     }),
     commonjs(),
     replace({
@@ -102,7 +103,8 @@ const fullBrowserBuild = {
       browser: true
     }),
     typescript({
-      compilerOptions: { outDir: './dist' } // temporary output location, needed to avoid js files being overwritten under `src`
+      compilerOptions: { outDir: './dist' }, // temporary output location, needed to avoid js files being overwritten under `src`
+      rootDir: 'src' // ignore .ts files outside of src folder
     }),
     commonjs({
       ignore: nodeBuiltinModules.concat(nodeDependencies)
@@ -129,7 +131,8 @@ const lightweightBrowserBuild = {
       browser: true
     }),
     typescript({
-      compilerOptions: { outDir: './dist/lightweight' }
+      compilerOptions: { outDir: './dist/lightweight' },
+      rootDir: 'src' // ignore .ts files outside of src folder
     }),
     commonjs({
       ignore: nodeBuiltinModules.concat(nodeDependencies)

rollup.config.js

@@ -10,8 +10,10 @@
  */
 
 import type { WebStream as GenericWebStream, NodeWebStream as GenericNodeWebStream } from '@openpgp/web-stream-tools';
-import enums from './src/enums';
-import config, { type Config, type PartialConfig } from './src/config';
+import enums from './enums';
+import config, { type Config, type PartialConfig } from './config';
+
+export { GrammarError } from './packet/grammar';
 
 export { enums, config, Config, PartialConfig };
 

openpgp.d.ts src/index.d.tsopenpgp.d.ts renamed to src/index.d.ts

@@ -0,0 +1,12 @@
+// tsconfig taking care of generating declaration files to be released
+// under 'dist/types'
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "declaration": true,
+    "emitDeclarationOnly": true,
+    "declarationDir": "./dist/types",
+    "rootDir": "src"
+  },
+  "files": ["src/index.d.ts"]
+}