@juspay/neurolink 9.24.0 → 9.25.1

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## [9.25.1](https://github.com/juspay/neurolink/compare/v9.25.0...v9.25.1) (2026-03-15)
+
+### Bug Fixes
+
+- **(observability):** address code review findings from PR [#860](https://github.com/juspay/neurolink/issues/860) ([1655c7e](https://github.com/juspay/neurolink/commit/1655c7ef6991eb104834e10fa7d21516539ea16d))
+
+## [9.25.0](https://github.com/juspay/neurolink/compare/v9.24.0...v9.25.0) (2026-03-15)
+
+### Features
+
+- **(observability):** add OTEL instrumentation, observability exporters, and comprehensive test suite fixes ([66c4559](https://github.com/juspay/neurolink/commit/66c45592cfcf5ae5acbec0513231f48e87044dab))
+
 ## [9.24.0](https://github.com/juspay/neurolink/compare/v9.23.0...v9.24.0) (2026-03-14)
 
 ### Features

package/dist/adapters/tts/googleTTSHandler.js

@@ -10,6 +10,8 @@ import { TextToSpeechClient } from "@google-cloud/text-to-speech";
 import { TTSError, TTS_ERROR_CODES } from "../../utils/ttsProcessor.js";
 import { ErrorCategory, ErrorSeverity } from "../../constants/enums.js";
 import { logger } from "../../utils/logger.js";
+import { SpanSerializer, SpanType, SpanStatus, } from "../../observability/index.js";
+import { getMetricsAggregator } from "../../observability/index.js";
 export class GoogleTTSHandler {
     client = null;
     voicesCache = null;
@@ -68,18 +70,26 @@ export class GoogleTTSHandler {
                 retriable: false,
             });
         }
+        const span = SpanSerializer.createSpan(SpanType.TTS, "tts.google.listVoices", {
+            "tts.operation": "listVoices",
+            "tts.provider": "google",
+        });
         try {
             // Return cached voices if available, valid, and no language filter is specified
             if (this.voicesCache &&
                 Date.now() - this.voicesCache.timestamp <
                     GoogleTTSHandler.CACHE_TTL_MS &&
                 !languageCode) {
+                const endedSpan = SpanSerializer.endSpan(span, SpanStatus.OK);
+                getMetricsAggregator().recordSpan(endedSpan);
                 return this.voicesCache.voices;
             }
             // Call Google Cloud listVoices API
             const [response] = await this.client.listVoices(languageCode ? { languageCode } : {});
             if (!response.voices || response.voices.length === 0) {
                 logger.warn("Google Cloud TTS returned no voices");
+                const endedSpan = SpanSerializer.endSpan(span, SpanStatus.OK);
+                getMetricsAggregator().recordSpan(endedSpan);
                 return [];
             }
             const voices = [];
@@ -118,9 +128,14 @@ export class GoogleTTSHandler {
             if (!languageCode) {
                 this.voicesCache = { voices, timestamp: Date.now() };
             }
+            const endedSpan = SpanSerializer.endSpan(span, SpanStatus.OK);
+            getMetricsAggregator().recordSpan(endedSpan);
             return voices;
         }
         catch (err) {
+            // Record error span
+            const endedSpan = SpanSerializer.endSpan(span, SpanStatus.ERROR, err instanceof Error ? err.message : "Unknown error");
+            getMetricsAggregator().recordSpan(endedSpan);
             // Log error but return empty array for graceful degradation
             const message = err instanceof Error ? err.message : "Unknown error";
             logger.error(`Failed to fetch Google TTS voices: ${message}`);
@@ -144,6 +159,13 @@ export class GoogleTTSHandler {
                 retriable: false,
             });
         }
+        const voiceId = options.voice ?? "en-US-Neural2-C";
+        const span = SpanSerializer.createSpan(SpanType.TTS, "tts.google.synthesize", {
+            "tts.operation": "synthesize",
+            "tts.provider": "google",
+            "tts.voice": voiceId,
+            "tts.format": options.format ?? "mp3",
+        });
         const startTime = Date.now();
         try {
             const isSSML = text.startsWith("<speak>") && text.endsWith("</speak>");
@@ -160,7 +182,6 @@ export class GoogleTTSHandler {
                     retriable: false,
                 });
             }
-            const voiceId = options.voice ?? "en-US-Neural2-C";
             const languageCode = this.extractLanguageCode(voiceId);
             const audioEncoding = this.mapFormat(options.format ?? "mp3");
             const request = {
@@ -204,6 +225,8 @@ export class GoogleTTSHandler {
                         });
                     })();
             const latency = Date.now() - startTime;
+            const endedSpan = SpanSerializer.endSpan(span, SpanStatus.OK);
+            getMetricsAggregator().recordSpan(endedSpan);
             return {
                 buffer,
                 format: options.format ?? "mp3",
@@ -216,6 +239,8 @@ export class GoogleTTSHandler {
             };
         }
         catch (err) {
+            const endedSpan = SpanSerializer.endSpan(span, SpanStatus.ERROR, err instanceof Error ? err.message : String(err));
+            getMetricsAggregator().recordSpan(endedSpan);
             if (err instanceof TTSError) {
                 throw err;
             }

package/dist/adapters/video/vertexVideoHandler.js

@@ -70,7 +70,14 @@ const DEFAULT_LOCATION = "us-central1";
  * ```
  */
 export function isVertexVideoConfigured() {
+    // Same credential detection as googleVertex.ts hasGoogleCredentials().
+    // GoogleAuth (used by getAccessToken) also supports ADC from
+    // `gcloud auth application-default login` automatically, so we only
+    // gate on the explicit env vars here — if none are set, we still
+    // allow the call through and let GoogleAuth resolve ADC at runtime.
+    // This avoids duplicating GoogleAuth's discovery logic.
     return !!(process.env.GOOGLE_APPLICATION_CREDENTIALS ||
+        process.env.GOOGLE_APPLICATION_CREDENTIALS_NEUROLINK ||
         process.env.GOOGLE_SERVICE_ACCOUNT_KEY ||
         (process.env.GOOGLE_AUTH_CLIENT_EMAIL &&
             process.env.GOOGLE_AUTH_PRIVATE_KEY));
@@ -270,21 +277,10 @@ function calculateDimensions(resolution, aspectRatio) {
  * ```
  */
 export async function generateVideoWithVertex(image, prompt, options = {}, region) {
-    // Validate configuration
-    if (!isVertexVideoConfigured()) {
-        throw new VideoError({
-            code: VIDEO_ERROR_CODES.PROVIDER_NOT_CONFIGURED,
-            message: "Vertex AI credentials not configured. Set GOOGLE_APPLICATION_CREDENTIALS environment variable",
-            category: ErrorCategory.CONFIGURATION,
-            severity: ErrorSeverity.HIGH,
-            retriable: false,
-            context: {
-                provider: "vertex",
-                feature: "video-generation",
-                suggestion: "Set GOOGLE_APPLICATION_CREDENTIALS to the path of your service account JSON file",
-            },
-        });
-    }
+    // Credential validation is deferred to getAccessToken() which uses
+    // GoogleAuth — it handles env vars, service accounts, AND ADC from
+    // `gcloud auth application-default login` automatically.
+    // Same pattern as googleVertex.ts — no synchronous pre-check needed.
     const config = await getVertexConfig();
     const project = config.project;
     const location = region || config.location;
@@ -312,7 +308,12 @@ export async function generateVideoWithVertex(image, prompt, options = {}, regio
         // Get auth token
         const accessToken = await getAccessToken();
         // Construct API request - predictLongRunning endpoint
-        const endpoint = `https://${location}-aiplatform.googleapis.com/v1/projects/${project}/locations/${location}/publishers/google/models/${VEO_MODEL}:predictLongRunning`;
+        // Global endpoint uses aiplatform.googleapis.com (no region prefix),
+        // same pattern as googleVertex.ts createVertexSettings
+        const apiHost = location === "global"
+            ? "aiplatform.googleapis.com"
+            : `${location}-aiplatform.googleapis.com`;
+        const endpoint = `https://${apiHost}/v1/projects/${project}/locations/${location}/publishers/google/models/${VEO_MODEL}:predictLongRunning`;
         // Request body structure (verified working from video.js reference)
         const requestBody = {
             instances: [
@@ -727,7 +728,12 @@ export async function generateTransitionWithVertex(firstFrame, lastFrame, prompt
  */
 async function pollOperation(modelOrEndpoint, operationName, accessToken, project, location, timeoutMs) {
     const startTime = Date.now();
-    const pollEndpoint = `https://${location}-aiplatform.googleapis.com/v1/projects/${project}/locations/${location}/publishers/google/models/${modelOrEndpoint}:fetchPredictOperation`;
+    // Global endpoint uses aiplatform.googleapis.com (no region prefix),
+    // same pattern as the predictLongRunning endpoint at line 374
+    const pollHost = location === "global"
+        ? "aiplatform.googleapis.com"
+        : `${location}-aiplatform.googleapis.com`;
+    const pollEndpoint = `https://${pollHost}/v1/projects/${project}/locations/${location}/publishers/google/models/${modelOrEndpoint}:fetchPredictOperation`;
     while (Date.now() - startTime < timeoutMs) {
         const result = await makePollRequest(pollEndpoint, operationName, accessToken);
         if (result.done) {

package/dist/cli/commands/config.d.ts

@@ -60,9 +60,9 @@ declare const ConfigSchema: z.ZodObject<{
         }, {
             model?: string | undefined;
             projectId?: string | undefined;
-            location?: string | undefined;
             credentials?: string | undefined;
             privateKey?: string | undefined;
+            location?: string | undefined;
             serviceAccountKey?: string | undefined;
             clientEmail?: string | undefined;
         }>>;
@@ -200,9 +200,9 @@ declare const ConfigSchema: z.ZodObject<{
         vertex?: {
             model?: string | undefined;
             projectId?: string | undefined;
-            location?: string | undefined;
             credentials?: string | undefined;
             privateKey?: string | undefined;
+            location?: string | undefined;
             serviceAccountKey?: string | undefined;
             clientEmail?: string | undefined;
         } | undefined;
@@ -569,9 +569,9 @@ declare const ConfigSchema: z.ZodObject<{
         vertex?: {
             model?: string | undefined;
             projectId?: string | undefined;
-            location?: string | undefined;
             credentials?: string | undefined;
             privateKey?: string | undefined;
+            location?: string | undefined;
             serviceAccountKey?: string | undefined;
             clientEmail?: string | undefined;
         } | undefined;

package/dist/cli/commands/observability.d.ts

@@ -0,0 +1,53 @@
+#!/usr/bin/env node
+/**
+ * NeuroLink CLI Observability Commands
+ *
+ * Commands for monitoring and managing observability features:
+ * - status: Show telemetry status
+ * - metrics: Show metrics summary
+ * - exporters: List configured exporters
+ * - costs: Show cost breakdown
+ */
+import type { CommandModule } from "yargs";
+/**
+ * Observability command arguments
+ */
+type ObservabilityCommandArgs = {
+    format?: "text" | "json" | "table";
+    quiet?: boolean;
+};
+type StatusArgs = ObservabilityCommandArgs;
+type MetricsArgs = ObservabilityCommandArgs & {
+    detailed?: boolean;
+};
+type ExportersArgs = ObservabilityCommandArgs;
+type CostsArgs = ObservabilityCommandArgs & {
+    byModel?: boolean;
+    byProvider?: boolean;
+};
+/**
+ * Observability Command Factory
+ */
+export declare class ObservabilityCommandFactory {
+    /**
+     * Create the observability command group
+     */
+    static createObservabilityCommands(): CommandModule<object, object>;
+    /**
+     * Create the status subcommand
+     */
+    static createStatusCommand(): CommandModule<object, StatusArgs>;
+    /**
+     * Create the metrics subcommand
+     */
+    static createMetricsCommand(): CommandModule<object, MetricsArgs>;
+    /**
+     * Create the exporters subcommand
+     */
+    static createExportersCommand(): CommandModule<object, ExportersArgs>;
+    /**
+     * Create the costs subcommand
+     */
+    static createCostsCommand(): CommandModule<object, CostsArgs>;
+}
+export {};