fix: use correct build for publish (#230)

Author: Lodin

.gitignore

@@ -32,3 +32,4 @@ node_modules/
 /ts/*/*.d.ts
 /ts/*/*.js.map
 /ts/*/*.js
+/ts/*/*.tgz

package-lock.json

@@ -2,7 +2,7 @@
   "name": "@vaadin/observability-kit-monorepo",
   "private": true,
   "description": "Observability Kit monorepo",
-  "main": "index.js",
+  "main": "",
   "type": "module",
   "engines": {
     "node": ">=16.19.0",
@@ -13,6 +13,7 @@
     "build": "nx run-many -t build --all",
     "lint": "nx run-many -t lint --all",
     "test": "echo \"Error: no test specified\" && exit 1",
+    "prepare": "npm run build",
     "typecheck": "nx run-many -t typecheck --all"
   },
   "repository": {
@@ -32,6 +33,7 @@
     "esbuild": "^0.17.19",
     "eslint": "^8.38.0",
     "eslint-config-vaadin": "^1.0.0-alpha.12",
+    "glob": "^10.2.7",
     "karma": "^6.4.2",
     "karma-chrome-launcher": "^3.2.0",
     "karma-coverage": "^2.2.0",

package.json

@@ -0,0 +1,26 @@
+import { readFile } from 'node:fs/promises';
+import { sep } from 'node:path';
+import { fileURLToPath, pathToFileURL } from 'url';
+import { build } from 'esbuild';
+import { glob } from 'glob';
+import type { PackageJson } from 'type-fest';
+
+const root = pathToFileURL(process.cwd() + sep);
+const [packageJsonFile, srcFiles] = await Promise.all([
+  readFile(new URL('package.json', root), 'utf8'),
+  glob('src/**/*.ts'),
+]);
+
+const packageJson: PackageJson = JSON.parse(packageJsonFile);
+
+await build({
+  define: {
+    __VERSION__: `'${packageJson.version ?? '0.0.0'}'`,
+  },
+  entryPoints: srcFiles.map((file) => new URL(file, root)).map(fileURLToPath),
+  minify: true,
+  outdir: fileURLToPath(root),
+  sourcemap: 'linked',
+  sourcesContent: true,
+  tsconfig: fileURLToPath(new URL('./tsconfig.build.json', root)),
+});

scripts/build.ts

@@ -14,8 +14,11 @@
     "*.{d.ts.map,d.ts,js.map,js}"
   ],
   "scripts": {
-    "build": "tsc -p tsconfig.build.json",
+    "build": "concurrently npm:build:code npm:build:dts",
+    "build:code": "tsx ../../scripts/build.ts",
+    "build:dts": "tsc -p tsconfig.build.json",
     "lint": "eslint src/**/*.ts",
+    "postversion": "npm run build",
     "typecheck": "tsc --noEmit"
   },
   "exports": {

ts/observability-kit-client/package.json

@@ -5,14 +5,16 @@ import { SemanticAttributes } from '@opentelemetry/semantic-conventions';
 
 export class FrontendErrorInstrumentation extends InstrumentationBase {
   readonly component: string = 'frontend-error';
-  moduleName = this.component;
+  moduleName: string;
   readonly version: string = '1';
   #listenerControllers = new Map<keyof WindowEventMap, AbortController>();
 
   constructor(config: InstrumentationConfig = { enabled: true }) {
     // Avoid issue when `enable` is called in the super constructor and fails
     // because of private methods
     super('@vaadin/frontend-error-instrumentation', '2.1-SNAPSHOT', { enabled: false });
+    this.moduleName = this.component;
+
     if (config.enabled) {
       this.enable();
     }

ts/observability-kit-client/src/FrontendErrorInstrumentation.ts

@@ -1,6 +1,8 @@
 export * from './HillaEndpointExporter.js';
 export * from './init.js';
 
+declare const __VERSION__: string;
+
 declare global {
   interface VaadinRegistration {
     is: string;
@@ -20,5 +22,5 @@ window.Vaadin ??= {};
 window.Vaadin.registrations ??= [];
 window.Vaadin.registrations.push({
   is: '@hilla/observability-kit-client',
-  version: /* updated-by-script */ '2.1-alpha.0',
+  version: __VERSION__,
 });

ts/observability-kit-client/src/index.ts

@@ -1,8 +1,10 @@
 {
   "extends": "./tsconfig.json",
   "compilerOptions": {
+    "alwaysStrict": false,
     "rootDir": "src",
-    "outDir": "."
+    "outDir": ".",
+    "emitDeclarationOnly": true
   },
   "include": ["src"]
 }