@foam-ai/node-cliengo 0.1.0-alpha.17 → 0.1.0-alpha.19

package/dist/node-cliengo/src/http/fastify.d.ts

@@ -23,6 +23,9 @@
  *   - All hooks wrapped in try/catch — never crashes the request pipeline.
  *   - onError/onResponse use async hooks (Fastify 5 forward-compat).
  *     onRequest uses callback style intentionally for context.with() propagation.
+ *   - Plugin sets Symbol.for('skip-override') (same as fastify-plugin) so hooks
+ *     fire globally across all encapsulated contexts. Without this, hooks only
+ *     fire for routes in the same plugin scope — a silent production footgun.
  */
 import { type Tracer } from '@opentelemetry/api';
 import type { FastifyPluginAsync } from '../types';

package/dist/node-cliengo/src/http/fastify.js

@@ -24,6 +24,9 @@
  *   - All hooks wrapped in try/catch — never crashes the request pipeline.
  *   - onError/onResponse use async hooks (Fastify 5 forward-compat).
  *     onRequest uses callback style intentionally for context.with() propagation.
+ *   - Plugin sets Symbol.for('skip-override') (same as fastify-plugin) so hooks
+ *     fire globally across all encapsulated contexts. Without this, hooks only
+ *     fire for routes in the same plugin scope — a silent production footgun.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getRequestTraceContext = getRequestTraceContext;
@@ -55,7 +58,7 @@ function getRequestTraceContext(request) {
 }
 /* eslint-disable @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-assignment */
 function createFastifyPlugin(tracer) {
-    return async (fastify) => {
+    const plugin = async (fastify) => {
         // onRequest uses callback style intentionally — otelContext.with(ctx, done)
         // propagates the shadow span as the active OTel context via AsyncLocalStorage
         // to all downstream hooks and the route handler. Async hooks would lose the
@@ -143,5 +146,11 @@ function createFastifyPlugin(tracer) {
             }
         });
     };
+    // Equivalent to wrapping with fastify-plugin — breaks Fastify's encapsulation
+    // so hooks fire for ALL routes, not just those registered in the same scope.
+    // Without this, hooks silently no-op for routes in child contexts.
+    plugin[Symbol.for('skip-override')] = true;
+    plugin[Symbol.for('fastify.display-name')] = '@foam-ai/node-cliengo';
+    return plugin;
 }
 /* eslint-enable @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-assignment */

package/dist/node-cliengo/src/init.js

@@ -110,8 +110,14 @@ function createInertInstance(serviceName) {
         buildTraceparent: trace_bridge_1.buildTraceparent,
         injectSnsAttributes: sns_1.injectSnsAttributes,
         injectJobData: job_1.injectJobData,
-        wrapSqsConsumer: async (_s, _m, fn) => fn(),
-        wrapJobConsumer: async (_s, _j, fn) => fn(),
+        wrapSqsConsumer: async (...args) => {
+            const fn = typeof args[0] === 'string' ? args[2] : args[3];
+            return fn();
+        },
+        wrapJobConsumer: async (...args) => {
+            const fn = typeof args[0] === 'string' ? args[2] : args[3];
+            return fn();
+        },
         extractParentContext: () => {
             const { ROOT_CONTEXT } = require('@opentelemetry/api');
             return ROOT_CONTEXT;
@@ -381,8 +387,18 @@ function init(serviceName, options = {}) {
         buildTraceparent: trace_bridge_1.buildTraceparent,
         injectSnsAttributes: sns_1.injectSnsAttributes,
         injectJobData: job_1.injectJobData,
-        wrapSqsConsumer: (s, m, fn) => (0, sqs_1.wrapSqsConsumer)(tracer, s, m, fn, metrics),
-        wrapJobConsumer: (s, j, fn) => (0, job_1.wrapJobConsumer)(tracer, s, j, fn, metrics),
+        wrapSqsConsumer: (...args) => {
+            if (typeof args[0] === 'string') {
+                return (0, sqs_1.wrapSqsConsumer)(tracer, args[0], args[1], args[2], metrics);
+            }
+            return (0, sqs_1.wrapSqsConsumer)(tracer, args[1], args[2], args[3], metrics);
+        },
+        wrapJobConsumer: (...args) => {
+            if (typeof args[0] === 'string') {
+                return (0, job_1.wrapJobConsumer)(tracer, args[0], args[1], args[2], metrics);
+            }
+            return (0, job_1.wrapJobConsumer)(tracer, args[1], args[2], args[3], metrics);
+        },
         extractParentContext: trace_bridge_1.extractParentContext,
         shutdown,
     };

package/dist/node-cliengo/src/types.d.ts

@@ -96,7 +96,11 @@ export interface FoamInstance {
     injectJobData<T extends Record<string, unknown>>(data: T): T & {
         traceparent: string;
     };
+    /** @deprecated Pass (spanName, msg, fn) — the tracer arg is no longer needed. */
+    wrapSqsConsumer(tracer: Tracer, spanName: string, msg: SqsMessage, fn: () => Promise<void>): Promise<void>;
     wrapSqsConsumer(spanName: string, msg: SqsMessage, fn: () => Promise<void>): Promise<void>;
+    /** @deprecated Pass (spanName, jobData, fn) — the tracer arg is no longer needed. */
+    wrapJobConsumer(tracer: Tracer, spanName: string, jobData: Record<string, unknown>, fn: () => Promise<void>): Promise<void>;
     wrapJobConsumer(spanName: string, jobData: Record<string, unknown>, fn: () => Promise<void>): Promise<void>;
     extractParentContext(traceparent: string): Context;
     shutdown(): Promise<void>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@foam-ai/node-cliengo",
-  "version": "0.1.0-alpha.17",
+  "version": "0.1.0-alpha.19",
   "description": "Unified observability (traces, logs, metrics) for Cliengo Node.js services, connecting New Relic APM with Foam's OTel collector.",
   "main": "dist/node-cliengo/src/index.js",
   "types": "dist/node-cliengo/src/index.d.ts",