feat(metrics): add esmodule support (#1739)

Author: dreamorosi

Diff for: examples/cdk/functions/get-all-items.ts

@@ -1,5 +1,5 @@
 import { injectLambdaContext } from '@aws-lambda-powertools/logger/middleware';
-import { logMetrics } from '@aws-lambda-powertools/metrics';
+import { logMetrics } from '@aws-lambda-powertools/metrics/middleware';
 import { captureLambdaHandler } from '@aws-lambda-powertools/tracer';
 import { ScanCommand } from '@aws-sdk/lib-dynamodb';
 import middy from '@middy/core';

Diff for: examples/sam/src/get-all-items.ts

@@ -6,7 +6,7 @@ import {
 import middy from '@middy/core';
 import { tableName } from './common/constants';
 import { logger, tracer, metrics } from './common/powertools';
-import { logMetrics } from '@aws-lambda-powertools/metrics';
+import { logMetrics } from '@aws-lambda-powertools/metrics/middleware';
 import { injectLambdaContext } from '@aws-lambda-powertools/logger/middleware';
 import { captureLambdaHandler } from '@aws-lambda-powertools/tracer';
 import { docClient } from './common/dynamodb-client';

Diff for: layers/tests/e2e/layerPublisher.test.ts

@@ -7,12 +7,12 @@ import { App } from 'aws-cdk-lib';
 import { LayerVersion } from 'aws-cdk-lib/aws-lambda';
 import { LayerPublisherStack } from '../../src/layer-publisher-stack';
 import {
-  TestNodejsFunction,
   TestStack,
   TestInvocationLogs,
   invokeFunctionOnce,
   generateTestUniqueName,
 } from '@aws-lambda-powertools/testing-utils';
+import { TestNodejsFunction } from '@aws-lambda-powertools/testing-utils/resources/lambda';
 import {
   RESOURCE_NAME_PREFIX,
   SETUP_TIMEOUT,

Diff for: packages/commons/src/types/LambdaInterface.ts

@@ -15,4 +15,12 @@ interface LambdaInterface {
   handler: SyncHandler<Handler> | AsyncHandler<Handler>;
 }
 
-export { LambdaInterface };
+type HandlerMethodDecorator = (
+  target: LambdaInterface,
+  propertyKey: string | symbol,
+  descriptor:
+    | TypedPropertyDescriptor<SyncHandler<Handler>>
+    | TypedPropertyDescriptor<AsyncHandler<Handler>>
+) => void;
+
+export { LambdaInterface, HandlerMethodDecorator };

Diff for: packages/commons/src/types/index.ts

@@ -1,4 +1,13 @@
-export * from './middy.js';
-export * from './awsSdk.js';
-export * from './json.js';
-export * from './LambdaInterface.js';
+export {
+  MiddlewareLikeObj,
+  MiddyLikeRequest,
+  CleanupFunction,
+} from './middy.js';
+export { SdkClient, MiddlewareArgsLike } from './awsSdk.js';
+export { JSONPrimitive, JSONValue, JSONObject, JSONArray } from './json.js';
+export {
+  SyncHandler,
+  AsyncHandler,
+  LambdaInterface,
+  HandlerMethodDecorator,
+} from './LambdaInterface.js';

Diff for: packages/metrics/jest.config.js renamed to packages/metrics/jest.config.cjs

@@ -5,6 +5,9 @@ module.exports = {
   },
   runner: 'groups',
   preset: 'ts-jest',
+  moduleNameMapper: {
+    '^(\\.{1,2}/.*)\\.js$': '$1',
+  },
   transform: {
     '^.+\\.ts?$': 'ts-jest',
   },
@@ -14,7 +17,7 @@ module.exports = {
   roots: ['<rootDir>/src', '<rootDir>/tests'],
   testPathIgnorePatterns: ['/node_modules/'],
   testEnvironment: 'node',
-  coveragePathIgnorePatterns: ['/node_modules/'],
+  coveragePathIgnorePatterns: ['/node_modules/', '/types/'],
   coverageThreshold: {
     global: {
       statements: 100,

Diff for: packages/metrics/package.json

@@ -22,19 +22,54 @@
     "test:e2e:nodejs20x": "RUNTIME=nodejs20x jest --group=e2e",
     "test:e2e": "jest --group=e2e",
     "watch": "jest --group=unit --watch ",
-    "build": "tsc --build --force",
+    "build:cjs": "tsc --build --force && echo '{ \"type\": \"commonjs\" }' > lib/cjs/package.json",
+    "build:esm": "tsc --project tsconfig.esm.json && echo '{ \"type\": \"module\" }' > lib/esm/package.json",
+    "build": "npm run build:esm & npm run build:cjs",
     "lint": "eslint --ext .ts,.js --no-error-on-unmatched-pattern .",
     "lint-fix": "eslint --fix --ext .ts,.js --no-error-on-unmatched-pattern .",
     "prebuild": "rimraf ./lib",
-    "prepack": "node ../../.github/scripts/release_patch_package_json.js ."
+    "prepack": "rimraf ./lib/*.tsbuildinfo && node ../../.github/scripts/release_patch_package_json.js ."
   },
   "lint-staged": {
     "*.{js,ts}": "npm run lint-fix"
   },
   "homepage": "https://github.com/aws-powertools/powertools-lambda-typescript/tree/main/packages/metrics#readme",
   "license": "MIT-0",
-  "main": "./lib/index.js",
-  "types": "./lib/index.d.ts",
+  "type": "module",
+  "exports": {
+    ".": {
+      "require": {
+        "types": "./lib/cjs/index.d.ts",
+        "default": "./lib/cjs/index.js"
+      },
+      "import": {
+        "types": "./lib/esm/index.d.ts",
+        "default": "./lib/esm/index.js"
+      }
+    },
+    "./middleware": {
+      "import": "./lib/esm/middleware/middy.js",
+      "require": "./lib/cjs/middleware/middy.js"
+    },
+    "./types": {
+      "import": "./lib/esm/types/index.js",
+      "require": "./lib/cjs/types/index.js"
+    }
+  },
+  "typesVersions": {
+    "*": {
+      "middleware": [
+        "lib/cjs/middleware/middy.d.ts",
+        "lib/esm/middleware/middy.d.ts"
+      ],
+      "types": [
+        "lib/cjs/types/index.d.ts",
+        "lib/esm/types/index.d.ts"
+      ]
+    }
+  },
+  "types": "./lib/cjs/index.d.ts",
+  "main": "./lib/cjs/index.js",
   "devDependencies": {
     "@aws-lambda-powertools/testing-utils": "file:../testing",
     "@aws-sdk/client-cloudwatch": "^3.499.0",

Diff for: packages/metrics/src/Metrics.ts

@@ -1,30 +1,29 @@
 import type { Callback, Context, Handler } from 'aws-lambda';
 import { Console } from 'node:console';
 import { Utility } from '@aws-lambda-powertools/commons';
-import type { MetricsInterface } from './MetricsInterface';
-import {
-  type ConfigServiceInterface,
-  EnvironmentVariablesService,
-} from './config';
+import type { HandlerMethodDecorator } from '@aws-lambda-powertools/commons/types';
+import { EnvironmentVariablesService } from './config/EnvironmentVariablesService.js';
 import {
   MAX_DIMENSION_COUNT,
   MAX_METRICS_SIZE,
   DEFAULT_NAMESPACE,
   COLD_START_METRIC,
   MAX_METRIC_VALUES_SIZE,
-} from './constants';
+  MetricUnit as MetricUnits,
+  MetricResolution as MetricResolutions,
+} from './constants.js';
 import {
-  MetricsOptions,
-  Dimensions,
-  EmfOutput,
-  HandlerMethodDecorator,
-  StoredMetrics,
-  ExtraOptions,
-  MetricUnit,
-  MetricUnits,
-  MetricResolution,
-  MetricDefinition,
-} from './types';
+  type MetricsOptions,
+  type Dimensions,
+  type EmfOutput,
+  type StoredMetrics,
+  type ExtraOptions,
+  type MetricDefinition,
+  type ConfigServiceInterface,
+  type MetricsInterface,
+  type MetricUnit,
+  type MetricResolution,
+} from './types/index.js';
 
 /**
  * ## Intro
@@ -82,7 +81,7 @@ import {
  *   ⁣@metrics.logMetrics({ captureColdStartMetric: true, throwOnEmptyMetrics: true })
  *   public handler(_event: unknown, _context: unknown): Promise<void> {
  *     // ...
- *     metrics.addMetric('test-metric', MetricUnits.Count, 10);
+ *     metrics.addMetric('test-metric', MetricUnit.Count, 10);
  *     // ...
  *   }
  * }
@@ -98,13 +97,13 @@ import {
  * @example
  *
  * ```typescript
- * import { Metrics, MetricUnits } from '@aws-lambda-powertools/metrics';
+ * import { Metrics, MetricUnit } from '@aws-lambda-powertools/metrics';
  *
  * const metrics = new Metrics({ namespace: 'serverlessAirline', serviceName: 'orders' });
  *
  * export const handler = async (_event: unknown, __context: unknown): Promise<void> => {
  *   metrics.captureColdStartMetric();
- *   metrics.addMetric('test-metric', MetricUnits.Count, 10);
+ *   metrics.addMetric('test-metric', MetricUnit.Count, 10);
  *   metrics.publishStoredMetrics();
  * };
  * ```
@@ -199,15 +198,15 @@ class Metrics extends Utility implements MetricsInterface {
    * or when calling {@link Metrics.publishStoredMetrics}.
    *
    * You can add a metric by specifying the metric name, unit, and value. For convenience,
-   * we provide a set of constants for the most common units in {@link MetricUnits}.
+   * we provide a set of constants for the most common units in {@link MetricUnit}.
    *
    * @example
    * ```typescript
-   * import { Metrics, MetricUnits } from '@aws-lambda-powertools/metrics';
+   * import { Metrics, MetricUnit } from '@aws-lambda-powertools/metrics';
    *
    * const metrics = new Metrics({ namespace: 'serverlessAirline', serviceName: 'orders' });
    *
-   * metrics.addMetric('successfulBooking', MetricUnits.Count, 1);
+   * metrics.addMetric('successfulBooking', MetricUnit.Count, 1);
    * ```
    *
    * Optionally, you can specify the metric resolution, which can be either `High` or `Standard`.
@@ -216,11 +215,11 @@ class Metrics extends Utility implements MetricsInterface {
    *
    * @example
    * ```typescript
-   * import { Metrics, MetricUnits, MetricResolution } from '@aws-lambda-powertools/metrics';
+   * import { Metrics, MetricUnit, MetricResolution } from '@aws-lambda-powertools/metrics';
    *
    * const metrics = new Metrics({ namespace: 'serverlessAirline', serviceName: 'orders' });
    *
-   * metrics.addMetric('successfulBooking', MetricUnits.Count, 1, MetricResolution.High);
+   * metrics.addMetric('successfulBooking', MetricUnit.Count, 1, MetricResolution.High);
    * ```
    *
    * @param name - The metric name
@@ -232,7 +231,7 @@ class Metrics extends Utility implements MetricsInterface {
     name: string,
     unit: MetricUnit,
     value: number,
-    resolution: MetricResolution = MetricResolution.Standard
+    resolution: MetricResolution = MetricResolutions.Standard
   ): void {
     this.storeMetric(name, unit, value, resolution);
     if (this.isSingleMetric) this.publishStoredMetrics();
@@ -380,7 +379,7 @@ class Metrics extends Utility implements MetricsInterface {
    * @example
    *
    * ```typescript
-   * import { Metrics, MetricUnits } from '@aws-lambda-powertools/metrics';
+   * import { Metrics, MetricUnit } from '@aws-lambda-powertools/metrics';
    *
    * const metrics = new Metrics({ namespace: 'serverlessAirline', serviceName: 'orders' }); // Sets metric namespace, and service as a metric dimension
    *
@@ -424,7 +423,7 @@ class Metrics extends Utility implements MetricsInterface {
     ).map((metricDefinition) => ({
       Name: metricDefinition.name,
       Unit: metricDefinition.unit,
-      ...(metricDefinition.resolution === MetricResolution.High
+      ...(metricDefinition.resolution === MetricResolutions.High
         ? { StorageResolution: metricDefinition.resolution }
         : {}),
     }));
@@ -512,7 +511,7 @@ class Metrics extends Utility implements MetricsInterface {
    * ```typescript
    * const singleMetric = metrics.singleMetric();
    * singleMetric.addDimension('InnerDimension', 'true');
-   * singleMetric.addMetric('single-metric', MetricUnits.Percent, 50);
+   * singleMetric.addMetric('single-metric', MetricUnit.Percent, 50);
    * ```
    *
    * @returns the Metrics
@@ -738,4 +737,4 @@ class Metrics extends Utility implements MetricsInterface {
   }
 }
 
-export { Metrics, MetricUnits, MetricResolution };
+export { Metrics };

Diff for: packages/metrics/src/config/EnvironmentVariablesService.ts

@@ -1,4 +1,4 @@
-import type { ConfigServiceInterface } from './ConfigServiceInterface';
+import type { ConfigServiceInterface } from '../types/ConfigServiceInterface.js';
 import { EnvironmentVariablesService as CommonEnvironmentVariablesService } from '@aws-lambda-powertools/commons';
 
 class EnvironmentVariablesService

Diff for: packages/metrics/src/config/index.ts

@@ -4,10 +4,46 @@ const MAX_METRICS_SIZE = 100;
 const MAX_METRIC_VALUES_SIZE = 100;
 const MAX_DIMENSION_COUNT = 29;
 
+const MetricUnit = {
+  Seconds: 'Seconds',
+  Microseconds: 'Microseconds',
+  Milliseconds: 'Milliseconds',
+  Bytes: 'Bytes',
+  Kilobytes: 'Kilobytes',
+  Megabytes: 'Megabytes',
+  Gigabytes: 'Gigabytes',
+  Terabytes: 'Terabytes',
+  Bits: 'Bits',
+  Kilobits: 'Kilobits',
+  Megabits: 'Megabits',
+  Gigabits: 'Gigabits',
+  Terabits: 'Terabits',
+  Percent: 'Percent',
+  Count: 'Count',
+  BytesPerSecond: 'Bytes/Second',
+  KilobytesPerSecond: 'Kilobytes/Second',
+  MegabytesPerSecond: 'Megabytes/Second',
+  GigabytesPerSecond: 'Gigabytes/Second',
+  TerabytesPerSecond: 'Terabytes/Second',
+  BitsPerSecond: 'Bits/Second',
+  KilobitsPerSecond: 'Kilobits/Second',
+  MegabitsPerSecond: 'Megabits/Second',
+  GigabitsPerSecond: 'Gigabits/Second',
+  TerabitsPerSecond: 'Terabits/Second',
+  CountPerSecond: 'Count/Second',
+} as const;
+
+const MetricResolution = {
+  Standard: 60,
+  High: 1,
+} as const;
+
 export {
   COLD_START_METRIC,
   DEFAULT_NAMESPACE,
   MAX_METRICS_SIZE,
   MAX_METRIC_VALUES_SIZE,
   MAX_DIMENSION_COUNT,
+  MetricUnit,
+  MetricResolution,
 };

Diff for: packages/metrics/src/constants.ts

@@ -1,3 +1,2 @@
-export * from './Metrics';
-export * from './MetricsInterface';
-export * from './middleware';
+export { Metrics } from './Metrics.js';
+export { MetricUnit, MetricResolution } from './constants.js';

Diff for: packages/metrics/src/index.ts

@@ -1,6 +1,6 @@
 import { METRICS_KEY } from '@aws-lambda-powertools/commons';
-import type { Metrics } from '../Metrics';
-import type { ExtraOptions } from '../types';
+import type { Metrics } from '../Metrics.js';
+import type { ExtraOptions } from '../types/Metrics.js';
 import type {
   MiddlewareLikeObj,
   MiddyLikeRequest,