feat: upgrade OTLP infra, export Makefile, update TS infra (#36)

* fix: add makefile export

* feat: upgrade OTLP infra, export Makefile, update TS infra

* fix: address SonarQube points

Author: djMax

.gitignore

@@ -21,13 +21,10 @@ coverage
 *.tsbuildinfo
 .eslintcache
 
-# Added by coconfig
-.eslintignore
-.npmignore
-tsconfig.json
-tsconfig.build.json
-jest.config.js
-.commitlintrc.json
-vitest.config.ts
-.eslintrc.cjs
-.prettierrc.cjs
+# Managed by cpconfig
+/.commitlintrc.yaml
+/eslint.config.mts
+/tsconfig.json
+/tsconfig.build.json
+/.prettierrc.yaml
+/vitest.config.ts

__tests__/config.test.ts

@@ -4,7 +4,8 @@ import { fileURLToPath } from 'url';
 
 import { describe, expect, test } from 'vitest';
 
-import { ConfigurationSchema, insertConfigurationBefore, loadConfiguration } from '../src/config/index.js';
+import type { ConfigurationSchema} from '../src/config/index.js';
+import { insertConfigurationBefore, loadConfiguration } from '../src/config/index.js';
 import { shortstops } from '../src/config/shortstops.js';
 
 interface CustomConfig extends ConfigurationSchema {

__tests__/fake-serv.test.ts

@@ -5,9 +5,10 @@ import { fileURLToPath } from 'url';
 import { describe, expect, test } from 'vitest';
 import request from 'supertest';
 
+import type {
+  ServiceStartOptions} from '../src/index.js';
 import {
   listen,
-  ServiceStartOptions,
   startApp,
 } from '../src/index.js';
 
@@ -45,6 +46,7 @@ describe('fake-serv', () => {
     };
 
     const app = await startApp(options).catch((error) => {
+      // eslint-disable-next-line no-console
       console.error(error);
       throw error;
     });

__tests__/fake-serv/src/handlers/hello.ts

@@ -1,5 +1,6 @@
-import { repl$, ServiceHandler } from '../../../../src/index';
-import { FakeServLocals } from '../index';
+import type { ServiceHandler } from '../../../../src/index';
+import { repl$ } from '../../../../src/index';
+import type { FakeServLocals } from '../index';
 
 export const get: ServiceHandler<FakeServLocals> = async (req, res) => {
   res.json({ greeting: req.query.greeting || 'Hello World' });

__tests__/fake-serv/src/index.ts

@@ -4,7 +4,7 @@ import { useService } from '../../../src/index.js';
 export interface FakeServLocals extends ServiceLocals {
   services: {
     fakeServ: {
-      get_something(): { things: string[] } | Error;
+      get_something(): Promise<{ things: string[] } | Error>;
     };
   };
 }
@@ -17,7 +17,7 @@ export function service(): Service<FakeServLocals> {
       await base.start(app);
       app.locals.services = app.locals.services || {};
       app.locals.services.fakeServ = {
-        get_something() {
+        async get_something() {
           throw new Error('Should not be called.');
         },
       };

__tests__/fake-serv/src/routes/error.ts

@@ -1,4 +1,4 @@
-import { ServiceRouter } from '../../../../src/index';
+import type { ServiceRouter } from '../../../../src/index';
 import { ServiceError } from '../../../../src/error';
 
 export function route(router: ServiceRouter) {

__tests__/fake-serv/src/routes/index.ts

@@ -1,5 +1,5 @@
 import type { ServiceExpress, ServiceRouter } from '../../../../src/index';
-import { FakeServLocals } from '../index';
+import type { FakeServLocals } from '../index';
 
 export function route(router: ServiceRouter<FakeServLocals>, app: ServiceExpress<FakeServLocals>) {
   const worldRequests = app.locals.meter.createCounter('world_requests', {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openapi-typescript-infra/service",
-  "version": "4.10.2",
+  "version": "0.0.0",
   "description": "An opinionated framework for building configuration driven services - web, api, or  ob. Uses OpenAPI, pino logging, express, confit, Typescript and vitest.",
   "exports": {
     ".": {
@@ -10,7 +10,8 @@
     "./telemetry": {
       "import": "./build/telemetry/index.js",
       "types": "./build/telemetry/index.d.ts"
-    }
+    },
+    "./Makefile": "./Makefile"
   },
   "type": "module",
   "scripts": {
@@ -20,7 +21,7 @@
     "watch": "tsc -p tsconfig.json -w --preserveWatchOutput",
     "clean": "npx rimraf ./build",
     "prepublishOnly": "yarn build",
-    "postinstall": "coconfig"
+    "postinstall": "cpconfig"
   },
   "repository": {
     "type": "git",
@@ -30,7 +31,7 @@
     "start-service": "./build/bin/start-service.js"
   },
   "config": {
-    "coconfig": "@openapi-typescript-infra/coconfig"
+    "cpconfig": "@openapi-typescript-infra/cpconfig"
   },
   "engines": {
     "node": ">=22"
@@ -70,26 +71,26 @@
   "dependencies": {
     "@godaddy/terminus": "^4.12.1",
     "@opentelemetry/api": "^1.9.0",
-    "@opentelemetry/auto-instrumentations-node": "^0.66.0",
-    "@opentelemetry/exporter-prometheus": "^0.207.0",
-    "@opentelemetry/instrumentation-dns": "^0.51.0",
-    "@opentelemetry/instrumentation-express": "^0.56.0",
-    "@opentelemetry/instrumentation-generic-pool": "^0.51.0",
-    "@opentelemetry/instrumentation-graphql": "^0.55.0",
-    "@opentelemetry/instrumentation-http": "^0.207.0",
-    "@opentelemetry/instrumentation-net": "^0.51.0",
-    "@opentelemetry/instrumentation-pg": "^0.60.0",
-    "@opentelemetry/instrumentation-pino": "^0.54.0",
-    "@opentelemetry/instrumentation-redis": "^0.56.0",
-    "@opentelemetry/instrumentation-undici": "^0.18.0",
-    "@opentelemetry/resource-detector-container": "^0.7.10",
-    "@opentelemetry/resource-detector-gcp": "^0.42.0",
-    "@opentelemetry/sdk-node": "^0.207.0",
-    "@opentelemetry/semantic-conventions": "^1.37.0",
+    "@opentelemetry/auto-instrumentations-node": "^0.67.0",
+    "@opentelemetry/exporter-prometheus": "^0.208.0",
+    "@opentelemetry/instrumentation-dns": "^0.52.0",
+    "@opentelemetry/instrumentation-express": "^0.57.0",
+    "@opentelemetry/instrumentation-generic-pool": "^0.52.0",
+    "@opentelemetry/instrumentation-graphql": "^0.56.0",
+    "@opentelemetry/instrumentation-http": "^0.208.0",
+    "@opentelemetry/instrumentation-net": "^0.52.0",
+    "@opentelemetry/instrumentation-pg": "^0.61.0",
+    "@opentelemetry/instrumentation-pino": "^0.55.0",
+    "@opentelemetry/instrumentation-redis": "^0.57.0",
+    "@opentelemetry/instrumentation-undici": "^0.19.0",
+    "@opentelemetry/resource-detector-container": "^0.7.11",
+    "@opentelemetry/resource-detector-gcp": "^0.43.0",
+    "@opentelemetry/sdk-node": "^0.208.0",
+    "@opentelemetry/semantic-conventions": "^1.38.0",
     "@sesamecare-oss/confit": "^2.2.1",
     "@sesamecare-oss/opentelemetry-node-metrics": "^1.1.0",
     "ajv": "^8.17.1",
-    "clean-stack": "^5.3.0",
+    "clean-stack": "^6.0.0",
     "cookie-parser": "^1.4.7",
     "dotenv": "^17.2.3",
     "express": "^5.1.0",
@@ -100,27 +101,27 @@
     "moderndash": "^4.0.0",
     "opentelemetry-resource-detector-sync-api": "^0.30.0",
     "pino": "^10.1.0",
-    "read-package-up": "^11.0.0",
+    "read-package-up": "^12.0.0",
     "request-ip": "^3.3.0"
   },
   "devDependencies": {
     "@commitlint/cli": "^20.1.0",
     "@commitlint/config-conventional": "^20.0.0",
-    "@openapi-typescript-infra/coconfig": "^4.7.1",
+    "@openapi-typescript-infra/cpconfig": "^1.1.0",
     "@semantic-release/commit-analyzer": "^13.0.1",
     "@semantic-release/exec": "^7.1.0",
-    "@semantic-release/github": "^12.0.1",
+    "@semantic-release/github": "^12.0.2",
     "@semantic-release/release-notes-generator": "^14.1.0",
     "@types/cookie-parser": "^1.4.10",
     "@types/express": "^5.0.5",
     "@types/minimist": "^1.2.5",
-    "@types/node": "^24.10.0",
+    "@types/node": "^24.10.1",
     "@types/request-ip": "^0.0.41",
     "@types/supertest": "^6.0.3",
-    "@typescript-eslint/eslint-plugin": "^8.46.3",
-    "@typescript-eslint/parser": "^8.46.3",
-    "coconfig": "^1.6.2",
-    "eslint": "^8.57.1",
+    "@typescript-eslint/eslint-plugin": "^8.46.4",
+    "@typescript-eslint/parser": "^8.46.4",
+    "cpconfig": "^1.4.4",
+    "eslint": "^9.39.1",
     "eslint-config-prettier": "^10.1.8",
     "eslint-import-resolver-typescript": "^4.4.4",
     "eslint-plugin-import": "^2.32.0",
@@ -131,7 +132,7 @@
     "tsconfig-paths": "^4.2.0",
     "tsx": "^4.20.6",
     "typescript": "^5.9.3",
-    "vitest": "^4.0.7"
+    "vitest": "^4.0.9"
   },
   "resolutions": {
     "qs": "^6.11.0"

src/bin/start-service.ts

@@ -20,7 +20,7 @@ if (argv.telemetry) {
 }
 
 const noTelemetry = (argv.repl || isDev()) && !argv.telemetry;
-bootstrap({
+void bootstrap({
   ...argv,
   telemetry: !noTelemetry,
 }).then(({ app, codepath, server }) => {

src/bootstrap.ts

@@ -1,10 +1,10 @@
 import path from 'node:path';
 import assert from 'node:assert';
 
-import { config, DotenvConfigOptions } from 'dotenv';
+import { config } from 'dotenv';
 import { readPackageUp } from 'read-package-up';
 import type { NormalizedPackageJson } from 'read-package-up';
-import { NodeSDKConfiguration } from '@opentelemetry/sdk-node';
+import type { NodeSDKConfiguration } from '@opentelemetry/sdk-node';
 
 import type {
   AnyServiceLocals,
@@ -14,7 +14,7 @@ import type {
 } from './types.js';
 import { isDev } from './env.js';
 import { startWithTelemetry } from './telemetry/index.js';
-import { ConfigurationSchema } from './config/schema.js';
+import type { ConfigurationSchema } from './config/schema.js';
 
 interface BootstrapArguments {
   // The name of the service, else discovered via read-package-up
@@ -102,7 +102,7 @@ export async function bootstrap<
     entrypoint = './build/index.js';
   }
 
-  config({ quiet: true } as DotenvConfigOptions);
+  config({ quiet: true });
 
   const absoluteEntrypoint = path.resolve(rootDirectory, entrypoint);
   if (argv?.telemetry) {
@@ -128,7 +128,7 @@ export async function bootstrap<
   }
 
   // This needs to be required for TS on-the-fly to work
-  // eslint-disable-next-line global-require, import/no-dynamic-require, @typescript-eslint/no-var-requires
+
   const impl = await import(absoluteEntrypoint);
   const opts: ServiceStartOptions<SLocals, RLocals> = {
     name,
@@ -137,7 +137,7 @@ export async function bootstrap<
     service: impl.default || impl.service,
     codepath,
   };
-  // eslint-disable-next-line import/no-unresolved
+
   const { startApp, listen } = await import('./express-app/app.js');
   const app = await startApp<SLocals, RLocals>(opts);
   const server = argv?.nobind ? undefined : await listen(app);