@fluojs/metrics 1.0.0-beta.4 → 1.0.1

package/README.ko.md

@@ -91,7 +91,7 @@ MetricsModule.forRoot({
 });
 ```
 
-`endpointMiddleware`는 class-based `@fluojs/http` middleware constructor를 받으며 metrics scrape endpoint에만 바인딩됩니다. middleware function이나 global middleware declaration은 이 option의 패키지 계약이 아닙니다.
+`endpointMiddleware`는 class-based `@fluojs/http` middleware constructor를 받으며 metrics scrape endpoint에만 바인딩됩니다. middleware function이나 global middleware declaration은 이 option의 패키지 계약이 아닙니다. HTTP 계측이 활성화된 경우 endpoint middleware가 던진 실패도 내장 HTTP request/error collector에 기록됩니다.
 
 ### Framework metric과 app metric이 하나의 registry를 공유하기
 
@@ -171,7 +171,7 @@ MetricsModule.forRoot({
 - `path`의 기본값은 `'/metrics'`이며, `path: false`로 스크레이프 엔드포인트를 완전히 비활성화할 수 있습니다.
 - scrape response는 active Registry의 Prometheus content type과 Registry contents를 사용합니다.
 - `defaultMetrics`의 기본값은 `true`이며, `defaultMetrics: false`로 해당 Registry의 Prometheus 기본 프로세스/Node.js collector를 끌 수 있습니다.
-- `endpointMiddleware`는 class-based route-scoped middleware를 스크레이프 엔드포인트에만 바인딩합니다.
+- `endpointMiddleware`는 class-based route-scoped middleware를 스크레이프 엔드포인트에만 바인딩합니다. HTTP 계측이 활성화된 경우 endpoint middleware 실패는 내장 HTTP collector에 집계됩니다.
 - HTTP 메트릭은 `http: true` 또는 `http` 옵션 객체를 전달한 경우에만 설치되며, 설치된 뒤에는 기본적으로 템플릿 기반 경로 라벨 정규화를 사용합니다.
 - 내장 HTTP collector와 플랫폼 텔레메트리 Gauge는 같은 Registry를 공유하는 모듈 인스턴스 사이에서 framework-owned이고 예상 label schema를 가진 경우에만 재사용되며, 커스텀 애플리케이션 메트릭 이름 충돌은 Prometheus의 중복 이름 실패 동작을 유지합니다.
 - raw path 라벨은 `allowUnsafeRawPathLabelMode: true`를 명시한 bounded internal route에서만 사용해야 합니다.

package/README.md

@@ -91,7 +91,7 @@ MetricsModule.forRoot({
 });
 ```
 
-`endpointMiddleware` accepts class-based `@fluojs/http` middleware constructors and binds them only to the metrics scrape endpoint. Middleware functions or global middleware declarations are not the package contract for this option.
+`endpointMiddleware` accepts class-based `@fluojs/http` middleware constructors and binds them only to the metrics scrape endpoint. Middleware functions or global middleware declarations are not the package contract for this option. When HTTP instrumentation is enabled, failures thrown by endpoint middleware are recorded in the built-in HTTP request and error collectors.
 
 ### Share one registry for framework and app metrics
 
@@ -171,7 +171,7 @@ MetricsModule.forRoot({
 - `path` defaults to `'/metrics'`, and `path: false` disables the scrape endpoint entirely.
 - The scrape response uses the active registry's Prometheus content type and registry contents.
 - `defaultMetrics` defaults to `true`, and `defaultMetrics: false` disables Prometheus default process and Node.js collectors for that registry.
-- `endpointMiddleware` binds class-based route-scoped middleware only to the scrape endpoint.
+- `endpointMiddleware` binds class-based route-scoped middleware only to the scrape endpoint; with HTTP instrumentation enabled, endpoint middleware failures are counted by the built-in HTTP collectors.
 - HTTP metrics are installed only when `http: true` or an `http` options object is provided, and then default to template-normalized path labels.
 - Built-in HTTP collectors and platform telemetry gauges are reused when module instances share one registry only if they are framework-owned and have the expected label schema; custom application metric name collisions keep Prometheus' duplicate-name failure behavior.
 - Raw path labels require `allowUnsafeRawPathLabelMode: true` and should stay limited to bounded internal routes.

package/dist/http-metrics-middleware.js

@@ -125,6 +125,7 @@ function getOrCreateHttpCounter(registry, config) {
     if (!FRAMEWORK_HTTP_COUNTERS.has(existing)) {
       throw new Error(`Metric name "${config.name}" is already registered by the application. Built-in HTTP metrics require framework-owned collectors.`);
     }
+    assertHttpMetricLabelSchema(existing, config);
     return existing;
   }
   const counter = createPrometheusCounter(registry, {
@@ -141,6 +142,7 @@ function getOrCreateHttpHistogram(registry, config) {
     if (!FRAMEWORK_HTTP_HISTOGRAMS.has(existing)) {
       throw new Error(`Metric name "${config.name}" is already registered by the application. Built-in HTTP metrics require framework-owned collectors.`);
     }
+    assertHttpMetricLabelSchema(existing, config);
     return existing;
   }
   const histogram = createPrometheusHistogram(registry, {
@@ -151,6 +153,13 @@ function getOrCreateHttpHistogram(registry, config) {
   FRAMEWORK_HTTP_HISTOGRAMS.add(histogram);
   return histogram;
 }
+function assertHttpMetricLabelSchema(metric, config) {
+  const registeredLabels = (metric.labelNames ?? []).join(',');
+  const expectedLabels = config.labelNames.join(',');
+  if (registeredLabels !== expectedLabels) {
+    throw new Error(`Metric name "${config.name}" is already registered with labels [${registeredLabels}]. Built-in HTTP metrics require labels [${expectedLabels}].`);
+  }
+}
 function normalizePathToTemplate(path, params) {
   if (!path) {
     return '/';

package/dist/metrics-module.js

@@ -54,7 +54,7 @@ export class MetricsModule {
       });
     }
     const endpointMiddleware = metricsPath ? (options.endpointMiddleware ?? []).map(middlewareClass => forRoutes(middlewareClass, metricsPath)) : [];
-    const middleware = [...endpointMiddleware, ...(httpOptions ? [new HttpMetricsMiddleware(registry, httpOptions)] : []), ...(options.middleware ?? [])];
+    const middleware = [...(httpOptions ? [new HttpMetricsMiddleware(registry, httpOptions)] : []), ...endpointMiddleware, ...(options.middleware ?? [])];
     const providers = [{
       provide: MetricsService,
       useValue: metricsService

package/package.json

@@ -8,7 +8,7 @@
     "monitoring",
     "observability"
   ],
-  "version": "1.0.0-beta.4",
+  "version": "1.0.1",
   "private": false,
   "license": "MIT",
   "repository": {
@@ -36,9 +36,9 @@
   ],
   "dependencies": {
     "prom-client": "^15.1.3",
-    "@fluojs/di": "^1.0.0-beta.7",
-    "@fluojs/http": "^1.0.0-beta.10",
-    "@fluojs/runtime": "^1.0.0-beta.12"
+    "@fluojs/di": "^1.0.1",
+    "@fluojs/http": "^1.0.0",
+    "@fluojs/runtime": "^1.0.1"
   },
   "devDependencies": {
     "@types/node": "^22.0.0",