@digitraffic/common 2026.5.27-1 → 2026.6.30-1

package/README.md

@@ -49,12 +49,14 @@ To update all dependencies to the latest versions:
 
 ```bash
 pnpm up --latest
+pnpm audit # -> fix high
 ```
 
 To update both normal dependencies and `peerDependencies` in one command, use:
 
 ```bash
 pnpm deps:update-all
+pnpm audit # -> fix high
 ```
 
 What `pnpm deps:update-all` does:

package/dist/__test__/utils/lead-time-logging.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/__test__/utils/lead-time-logging.test.js

@@ -0,0 +1,153 @@
+import { afterEach, describe, expect, test, vi } from "vitest";
+import { logger } from "../../aws/runtime/dt-logger-default.js";
+import { logLeadTime, logLeadTimes } from "../../utils/lead-time-logging.js";
+describe("lead-time-logging", () => {
+    afterEach(() => {
+        vi.restoreAllMocks();
+    });
+    test("logLeadTime logs expected payload", () => {
+        const infoSpy = vi
+            .spyOn(logger, "info")
+            .mockImplementation(() => undefined);
+        const errorSpy = vi
+            .spyOn(logger, "error")
+            .mockImplementation(() => undefined);
+        logLeadTime("lead-time", "test-target", 123);
+        expect(infoSpy).toHaveBeenCalledTimes(1);
+        expect(infoSpy).toHaveBeenCalledWith({
+            method: "LeadTimeLogging.logLeadTime",
+            customLeadTime: true,
+            customName: "lead-time",
+            customTarget: "test-target",
+            tookMs: 123,
+        });
+        expect(errorSpy).toHaveBeenCalledTimes(0);
+    });
+    test("logLeadTime merges extra fields", () => {
+        const infoSpy = vi
+            .spyOn(logger, "info")
+            .mockImplementation(() => undefined);
+        logLeadTime("mqtt-time", "mqtt", 88, {
+            customSource: "mqtt",
+            customRetriesCount: 2,
+        });
+        expect(infoSpy).toHaveBeenCalledTimes(1);
+        expect(infoSpy).toHaveBeenCalledWith({
+            method: "LeadTimeLogging.logLeadTime",
+            customLeadTime: true,
+            customName: "mqtt-time",
+            customTarget: "mqtt",
+            tookMs: 88,
+            customSource: "mqtt",
+            customRetriesCount: 2,
+        });
+    });
+    test("logLeadTime logs error if logger.info throws", () => {
+        const infoError = new Error("boom");
+        vi.spyOn(logger, "info").mockImplementation(() => {
+            throw infoError;
+        });
+        const errorSpy = vi
+            .spyOn(logger, "error")
+            .mockImplementation(() => undefined);
+        logLeadTime("process-time", "target", 10);
+        expect(errorSpy).toHaveBeenCalledTimes(1);
+        expect(errorSpy).toHaveBeenCalledWith({
+            method: "LeadTimeLogging.logLeadTime",
+            message: "Error logging lead time",
+            error: infoError,
+        });
+    });
+    test("logLeadTimes logs only lead-time when only leadTimeMs is safe", () => {
+        const infoSpy = vi
+            .spyOn(logger, "info")
+            .mockImplementation(() => undefined);
+        logLeadTimes({ target: "x", leadTimeMs: 10 });
+        expect(infoSpy).toHaveBeenCalledTimes(1);
+        expect(infoSpy).toHaveBeenNthCalledWith(1, {
+            method: "LeadTimeLogging.logLeadTime",
+            customLeadTime: true,
+            customName: "lead-time",
+            customTarget: "x",
+            tookMs: 10,
+        });
+    });
+    test("logLeadTimes logs only process-time when only processTimeMs is safe", () => {
+        const infoSpy = vi
+            .spyOn(logger, "info")
+            .mockImplementation(() => undefined);
+        logLeadTimes({ target: "x", processTimeMs: 20 });
+        expect(infoSpy).toHaveBeenCalledTimes(1);
+        expect(infoSpy).toHaveBeenNthCalledWith(1, {
+            method: "LeadTimeLogging.logLeadTime",
+            customLeadTime: true,
+            customName: "process-time",
+            customTarget: "x",
+            tookMs: 20,
+        });
+    });
+    test("logLeadTimes logs both in order with extra fields", () => {
+        const infoSpy = vi
+            .spyOn(logger, "info")
+            .mockImplementation(() => undefined);
+        const input = {
+            target: "device",
+            leadTimeMs: 11,
+            processTimeMs: 22,
+            extraFields: { customSource: "mqtt", customPartition: 3 },
+        };
+        logLeadTimes(input);
+        expect(infoSpy).toHaveBeenCalledTimes(2);
+        expect(infoSpy).toHaveBeenNthCalledWith(1, {
+            method: "LeadTimeLogging.logLeadTime",
+            customLeadTime: true,
+            customName: "lead-time",
+            customTarget: "device",
+            tookMs: 11,
+            customSource: "mqtt",
+            customPartition: 3,
+        });
+        expect(infoSpy).toHaveBeenNthCalledWith(2, {
+            method: "LeadTimeLogging.logLeadTime",
+            customLeadTime: true,
+            customName: "process-time",
+            customTarget: "device",
+            tookMs: 22,
+            customSource: "mqtt",
+            customPartition: 3,
+        });
+    });
+    test("logLeadTimes ignores non-safe values", () => {
+        const infoSpy = vi
+            .spyOn(logger, "info")
+            .mockImplementation(() => undefined);
+        logLeadTimes({
+            target: "x",
+            leadTimeMs: 1.5,
+            processTimeMs: Number.MAX_SAFE_INTEGER + 1,
+        });
+        logLeadTimes({
+            target: "x",
+            leadTimeMs: Number.NaN,
+            processTimeMs: Infinity,
+        });
+        logLeadTimes({ target: "x" });
+        expect(infoSpy).toHaveBeenCalledTimes(0);
+    });
+    test("logLeadTimes logs negative safe integer values", () => {
+        const infoSpy = vi
+            .spyOn(logger, "info")
+            .mockImplementation(() => undefined);
+        logLeadTimes({ target: "x", leadTimeMs: -5 });
+        expect(infoSpy).toHaveBeenCalledTimes(1);
+        expect(infoSpy).toHaveBeenNthCalledWith(1, {
+            method: "LeadTimeLogging.logLeadTime",
+            customLeadTime: true,
+            customName: "lead-time",
+            customTarget: "x",
+            tookMs: -5,
+        });
+    });
+});
+// clean comment
+//# sourceMappingURL=lead-time-logging.test.js.map

package/dist/aws/infra/canaries/database-checker.js

@@ -1,6 +1,5 @@
 import synthetics from "Synthetics";
 import { inDatabaseReadonly } from "../../../database/database.js";
-import { getEnvVariable } from "../../../utils/utils.js";
 import { logger } from "../../runtime/dt-logger-default.js";
 import { ProxyHolder } from "../../runtime/secrets/proxy-holder.js";
 import { RdsHolder } from "../../runtime/secrets/rds-holder.js";
@@ -66,10 +65,10 @@ export class DatabaseCountChecker {
         synthetics.getConfiguration().withFailedCanaryMetric(true);
     }
     static createForProxy() {
-        return new DatabaseCountChecker(() => new ProxyHolder(getEnvVariable("SECRET_ID")).setCredentials());
+        return new DatabaseCountChecker(() => ProxyHolder.create().setCredentials());
     }
     static createForRds() {
-        return new DatabaseCountChecker(() => new RdsHolder(getEnvVariable("SECRET_ID")).setCredentials());
+        return new DatabaseCountChecker(() => RdsHolder.create().setCredentials());
     }
     /**
      * Expect that the count is 1

package/dist/aws/runtime/secrets/proxy-holder.d.ts

@@ -1,9 +1,14 @@
+import type { RdsProxySecret } from "./dbsecret.js";
+import { SecretHolder } from "./secret-holder.js";
 /**
  * Holds credentials for RDS Proxy access.
  */
 export declare class ProxyHolder {
     private readonly secretHolder;
-    constructor(secretId: string);
+    private constructor();
+    /** Creates a new instance of ProxyHolder from given SecretHolder. */
+    static create(secretHolder: SecretHolder<RdsProxySecret>): ProxyHolder;
+    /** Creates a new instance of ProxyHolder with a new default SecretHolder(using env variable SECRET_ID). */
     static create(): ProxyHolder;
     setCredentials(): Promise<void>;
 }

package/dist/aws/runtime/secrets/proxy-holder.js

@@ -8,11 +8,13 @@ const RDS_PROXY_SECRET_KEYS = Object.values(RdsProxySecretKey);
  */
 export class ProxyHolder {
     secretHolder;
-    constructor(secretId) {
-        this.secretHolder = new SecretHolder(secretId, "", RDS_PROXY_SECRET_KEYS);
+    constructor(secretHolder) {
+        this.secretHolder = secretHolder;
     }
-    static create() {
-        return new ProxyHolder(getEnvVariable("SECRET_ID"));
+    static create(secretHolder) {
+        const holder = secretHolder ??
+            new SecretHolder(getEnvVariable("SECRET_ID"), "", RDS_PROXY_SECRET_KEYS);
+        return new ProxyHolder(holder);
     }
     async setCredentials() {
         const secret = await this.secretHolder.get();

package/dist/aws/runtime/secrets/rds-holder.d.ts

@@ -1,9 +1,12 @@
+import type { RdsSecret } from "./dbsecret.js";
+import { SecretHolder } from "./secret-holder.js";
 /**
  * Holds credentials for RDS access.
  */
 export declare class RdsHolder {
     private readonly secretHolder;
-    constructor(secretId: string);
+    private constructor();
+    static create(rdsHolder: SecretHolder<RdsSecret>): RdsHolder;
     static create(): RdsHolder;
     setCredentials(): Promise<void>;
 }

package/dist/aws/runtime/secrets/rds-holder.js

@@ -8,11 +8,13 @@ const RDS_SECRET_KEYS = Object.values(RdsSecretKey);
  */
 export class RdsHolder {
     secretHolder;
-    constructor(secretId) {
-        this.secretHolder = new SecretHolder(secretId, "", RDS_SECRET_KEYS);
+    constructor(secretHolder) {
+        this.secretHolder = secretHolder;
     }
-    static create() {
-        return new RdsHolder(getEnvVariable("SECRET_ID"));
+    static create(secretHolder) {
+        const holder = secretHolder ??
+            new SecretHolder(getEnvVariable("SECRET_ID"), "", RDS_SECRET_KEYS);
+        return new RdsHolder(holder);
     }
     async setCredentials() {
         const secret = await this.secretHolder.get();

package/dist/database/database.d.ts

@@ -1,4 +1,4 @@
-import { type IDatabase, type ITask } from "pg-promise";
+import type { IDatabase, ITask } from "pg-promise";
 export declare enum DatabaseEnvironmentKeys {
     DB_USER = "DB_USER",
     DB_PASS = "DB_PASS",

package/dist/database/database.js

@@ -1,4 +1,4 @@
-import pgpImport, {} from "pg-promise";
+import pgpImport from "pg-promise";
 import { logger } from "../aws/runtime/dt-logger-default.js";
 import { logException } from "../utils/logging.js";
 import { getEnvVariable, getEnvVariableOrElse } from "../utils/utils.js";

package/dist/utils/lead-time-logging.d.ts

@@ -0,0 +1,10 @@
+import type { CustomParams } from "../aws/runtime/dt-logger.js";
+export interface LeadTimeLogging {
+    readonly target: string;
+    readonly leadTimeMs?: number;
+    readonly processTimeMs?: number;
+    readonly extraFields?: CustomParams;
+}
+export type LeadTimeType = "lead-time" | "process-time" | "mqtt-time";
+export declare function logLeadTime(name: LeadTimeType, target: string, tookMs: number, extraFields?: CustomParams): void;
+export declare function logLeadTimes(ptl: LeadTimeLogging): void;

package/dist/utils/lead-time-logging.js

@@ -0,0 +1,33 @@
+import { logger } from "../aws/runtime/dt-logger-default.js";
+export function logLeadTime(name, target, tookMs, extraFields) {
+    const method = "LeadTimeLogging.logLeadTime";
+    try {
+        logger.info({
+            ...(extraFields ?? {}),
+            method,
+            customLeadTime: true,
+            customName: name,
+            customTarget: target,
+            tookMs,
+        });
+    }
+    catch (error) {
+        logger.error({
+            method,
+            message: "Error logging lead time",
+            error,
+        });
+    }
+}
+function isSafe(value) {
+    return Number.isSafeInteger(value);
+}
+export function logLeadTimes(ptl) {
+    if (isSafe(ptl.leadTimeMs)) {
+        logLeadTime("lead-time", ptl.target, ptl.leadTimeMs, ptl.extraFields);
+    }
+    if (isSafe(ptl.processTimeMs)) {
+        logLeadTime("process-time", ptl.target, ptl.processTimeMs, ptl.extraFields);
+    }
+}
+//# sourceMappingURL=lead-time-logging.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@digitraffic/common",
-  "version": "2026.5.27-1",
+  "version": "2026.6.30-1",
   "private": false,
   "description": "",
   "repository": {
@@ -36,6 +36,7 @@
     "./dist/marine/id_utils": "./dist/marine/id_utils.js",
     "./dist/index": "./dist/index.js",
     "./dist/utils/api-model": "./dist/utils/api-model.js",
+    "./dist/utils/lead-time-logging": "./dist/utils/lead-time-logging.js",
     "./dist/utils/logging": "./dist/utils/logging.js",
     "./dist/utils/base64": "./dist/utils/base64.js",
     "./dist/utils/date-utils": "./dist/utils/date-utils.js",
@@ -108,62 +109,62 @@
     "dist/**/*.d.ts"
   ],
   "devDependencies": {
-    "@aws-sdk/client-api-gateway": "3.1047.0",
-    "@aws-sdk/client-s3": "3.1047.0",
-    "@aws-sdk/client-secrets-manager": "3.1047.0",
-    "@aws-sdk/client-sns": "3.1047.0",
-    "@aws-sdk/lib-storage": "3.1047.0",
-    "@biomejs/biome": "2.4.15",
-    "@date-fns/tz": "1.4.1",
-    "@rushstack/heft": "1.2.17",
-    "@rushstack/heft-typescript-plugin": "1.3.12",
-    "@smithy/fetch-http-handler": "5.4.2",
-    "@smithy/node-http-handler": "4.7.2",
-    "@smithy/types": "4.14.1",
-    "@types/aws-lambda": "8.10.161",
+    "@aws-sdk/client-api-gateway": "3.1073.0",
+    "@aws-sdk/client-s3": "3.1073.0",
+    "@aws-sdk/client-secrets-manager": "3.1073.0",
+    "@aws-sdk/client-sns": "3.1073.0",
+    "@aws-sdk/lib-storage": "3.1073.0",
+    "@biomejs/biome": "2.5.0",
+    "@date-fns/tz": "1.5.0",
+    "@rushstack/heft": "1.2.19",
+    "@rushstack/heft-typescript-plugin": "1.3.14",
+    "@smithy/fetch-http-handler": "5.5.1",
+    "@smithy/node-http-handler": "4.8.1",
+    "@smithy/types": "4.15.0",
+    "@types/aws-lambda": "8.10.162",
     "@types/etag": "1.8.4",
     "@types/geojson": "7946.0.16",
     "@types/geojson-validation": "1.0.3",
     "@types/madge": "5.0.3",
     "@types/node": "24.12.4",
-    "@vitest/coverage-v8": "4.1.6",
-    "aws-cdk-lib": "2.254.0",
+    "@vitest/coverage-v8": "4.1.9",
+    "aws-cdk-lib": "2.260.0",
     "change-case": "5.4.4",
     "constructs": "10.6.0",
-    "date-fns": "4.1.0",
-    "es-toolkit": "1.46.1",
+    "date-fns": "4.4.0",
+    "es-toolkit": "1.47.1",
     "etag": "1.8.1",
     "geojson-validation": "1.0.2",
     "ky": "2.0.2",
-    "lefthook": "2.1.6",
+    "lefthook": "2.1.9",
     "madge": "8.0.0",
-    "pg-native": "3.7.0",
+    "pg-native": "3.8.0",
     "pg-promise": "12.6.2",
-    "pg-query-stream": "4.14.0",
+    "pg-query-stream": "4.16.0",
     "rimraf": "6.1.3",
-    "sort-package-json": "3.6.1",
+    "sort-package-json": "4.0.0",
     "typescript": "5.9.3",
     "velocityjs": "2.1.6",
-    "vitest": "4.1.6",
+    "vitest": "4.1.9",
     "zod": "4.4.3"
   },
   "peerDependencies": {
-    "@aws-sdk/client-api-gateway": "3.1047.0",
-    "@aws-sdk/client-s3": "3.1047.0",
-    "@aws-sdk/client-secrets-manager": "3.1047.0",
-    "@aws-sdk/client-sns": "3.1047.0",
-    "@aws-sdk/lib-storage": "3.1047.0",
-    "@date-fns/tz": "1.4.1",
-    "@smithy/fetch-http-handler": "5.4.2",
-    "@smithy/node-http-handler": "4.7.2",
-    "aws-cdk-lib": "2.254.0",
+    "@aws-sdk/client-api-gateway": "3.1073.0",
+    "@aws-sdk/client-s3": "3.1073.0",
+    "@aws-sdk/client-secrets-manager": "3.1073.0",
+    "@aws-sdk/client-sns": "3.1073.0",
+    "@aws-sdk/lib-storage": "3.1073.0",
+    "@date-fns/tz": "1.5.0",
+    "@smithy/fetch-http-handler": "5.5.1",
+    "@smithy/node-http-handler": "4.8.1",
+    "aws-cdk-lib": "2.260.0",
     "change-case": "5.4.4",
     "constructs": "10.6.0",
-    "date-fns": "4.1.0",
-    "es-toolkit": "1.46.1",
+    "date-fns": "4.4.0",
+    "es-toolkit": "1.47.1",
     "geojson-validation": "1.0.2",
     "ky": "2.0.2",
-    "pg-native": "3.7.0",
+    "pg-native": "3.8.0",
     "pg-promise": "12.6.2",
     "zod": "4.4.3"
   },