style(idempotency): apply stricter linting

Author: dreamorosi

packages/idempotency/src/idempotencyDecorator.ts

@@ -70,7 +70,7 @@ const idempotent = (
   ) {
     const childFunction = descriptor.value;
 
-    descriptor.value = async function (this: Handler, ...args: unknown[]) {
+    descriptor.value = function (this: Handler, ...args: unknown[]) {
       return makeIdempotent(childFunction, options).bind(this)(...args);
     };
 

packages/idempotency/src/middleware/makeHandlerIdempotent.ts

@@ -112,7 +112,7 @@ const makeHandlerIdempotent = (
    *
    * @param request - The Middy request object
    */
-  const before = async (request: MiddyLikeRequest): Promise<unknown> => {
+  const before = (request: MiddyLikeRequest): unknown => {
     const idempotencyConfig = options.config
       ? options.config
       : new IdempotencyConfig({});

packages/idempotency/src/persistence/DynamoDBPersistenceLayer.ts

@@ -168,9 +168,9 @@ class DynamoDBPersistenceLayer extends BasePersistenceLayer {
        * |       (in_progress_expiry)                                          (expiry)
        *
        * Conditions to successfully save a record:
-       * * The idempotency key does not exist:
-       *   - first time that this invocation key is used
-       *   - previous invocation with the same key was deleted due to TTL
+       * - The idempotency key does not exist:
+       * - first time that this invocation key is used
+       * - previous invocation with the same key was deleted due to TTL
        */
       const idempotencyKeyDoesNotExist = 'attribute_not_exists(#id)';
       // * The idempotency key exists but it is expired

packages/idempotency/tests/e2e/idempotentDecorator.test.FunctionCode.ts

@@ -2,7 +2,7 @@ import type { LambdaInterface } from '@aws-lambda-powertools/commons/types';
 import { Logger } from '@aws-lambda-powertools/logger';
 import type { Context } from 'aws-lambda';
 import { IdempotencyConfig } from '../../src/IdempotencyConfig.js';
-import { idempotent } from '../../src/idempotencyDecorator';
+import { idempotent } from '../../src/idempotencyDecorator.js';
 import { DynamoDBPersistenceLayer } from '../../src/persistence/DynamoDBPersistenceLayer.js';
 
 const IDEMPOTENCY_TABLE_NAME =
@@ -44,10 +44,7 @@ class DefaultLambda implements LambdaInterface {
     persistenceStore: dynamoDBPersistenceLayerCustomized,
     config: config,
   })
-  public async handlerCustomized(
-    event: { foo: string },
-    context: Context
-  ): Promise<string> {
+  public handlerCustomized(event: { foo: string }, context: Context) {
     config.registerLambdaContext(context);
     logger.info('Processed event', { details: event.foo });
 
@@ -62,10 +59,10 @@ class DefaultLambda implements LambdaInterface {
       eventKeyJmesPath: 'foo',
     }),
   })
-  public async handlerExpired(
+  public handlerExpired(
     event: { foo: string; invocation: number },
     context: Context
-  ): Promise<{ foo: string; invocation: number }> {
+  ) {
     logger.addContext(context);
 
     logger.info('Processed event', { details: event.foo });
@@ -77,10 +74,7 @@ class DefaultLambda implements LambdaInterface {
   }
 
   @idempotent({ persistenceStore: dynamoDBPersistenceLayer })
-  public async handlerParallel(
-    event: { foo: string },
-    context: Context
-  ): Promise<string> {
+  public async handlerParallel(event: { foo: string }, context: Context) {
     logger.addContext(context);
 
     await new Promise((resolve) => setTimeout(resolve, 1500));
@@ -99,7 +93,7 @@ class DefaultLambda implements LambdaInterface {
   public async handlerTimeout(
     event: { foo: string; invocation: number },
     context: Context
-  ): Promise<{ foo: string; invocation: number }> {
+  ) {
     logger.addContext(context);
 
     if (event.invocation === 0) {
@@ -145,7 +139,7 @@ class LambdaWithKeywordArgument implements LambdaInterface {
     config: config,
     dataIndexArgument: 1,
   })
-  public async process(id: string, foo: string): Promise<string> {
+  public process(id: string, foo: string) {
     logger.info('Got test event', { id, foo });
 
     return `idempotent result: ${foo}`;

packages/idempotency/tests/e2e/makeHandlerIdempotent.test.FunctionCode.ts

@@ -16,14 +16,12 @@ const logger = new Logger();
 /**
  * Test handler with sequential execution.
  */
-export const handler = middy(
-  async (event: { foo: string }, context: Context) => {
-    logger.addContext(context);
-    logger.info('foo', { details: event.foo });
+export const handler = middy((event: { foo: string }, context: Context) => {
+  logger.addContext(context);
+  logger.info('foo', { details: event.foo });
 
-    return event.foo;
-  }
-).use(
+  return event.foo;
+}).use(
   makeHandlerIdempotent({
     persistenceStore: dynamoDBPersistenceLayer,
   })
@@ -97,7 +95,7 @@ export const handlerTimeout = middy(
  * was processed by looking at the value in the stored idempotency record.
  */
 export const handlerExpired = middy(
-  async (event: { foo: string; invocation: number }, context: Context) => {
+  (event: { foo: string; invocation: number }, context: Context) => {
     logger.addContext(context);
 
     logger.info('Processed event', { details: event.foo });

packages/idempotency/tests/e2e/makeIdempotent.test.FunctionCode.ts

@@ -90,7 +90,7 @@ export const handlerCustomized = async (
  * Test idempotent Lambda handler with JMESPath expression to extract event key.
  */
 export const handlerLambda = makeIdempotent(
-  async (event: { body: string }, context: Context) => {
+  (event: { body: string }, context: Context) => {
     logger.addContext(context);
     const body = JSON.parse(event.body);
     logger.info('foo', { details: body.foo });

packages/idempotency/tests/unit/IdempotencyConfig.test.ts

@@ -87,7 +87,7 @@ describe('Class: IdempotencyConfig', () => {
   });
 
   describe('Method: registerLambdaContext', () => {
-    it('stores the provided context', async () => {
+    it('stores the provided context', () => {
       // Prepare
       const config = new IdempotencyConfig({});
 

packages/idempotency/tests/unit/IdempotencyHandler.test.ts

@@ -68,7 +68,7 @@ describe('Class IdempotencyHandler', () => {
       },
     ])(
       'throws when the record is in progress and within expiry window ($case)',
-      async ({ keys, expectedErrorMsg }) => {
+      ({ keys, expectedErrorMsg }) => {
         // Prepare
         const stubRecord = new IdempotencyRecord({
           ...keys,
@@ -89,7 +89,7 @@ describe('Class IdempotencyHandler', () => {
       }
     );
 
-    it('throws when the record is in progress and outside expiry window', async () => {
+    it('throws when the record is in progress and outside expiry window', () => {
       // Prepare
       const stubRecord = new IdempotencyRecord({
         idempotencyKey: 'idempotencyKey',
@@ -109,7 +109,7 @@ describe('Class IdempotencyHandler', () => {
       expect(mockResponseHook).not.toHaveBeenCalled();
     });
 
-    it('throws when the idempotency record is expired', async () => {
+    it('throws when the idempotency record is expired', () => {
       // Prepare
       const stubRecord = new IdempotencyRecord({
         idempotencyKey: 'idempotencyKey',

packages/idempotency/tests/unit/deepSort.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from 'vitest';
-import { deepSort } from '../../src/deepSort';
+import { deepSort } from '../../src/deepSort.js';
 
 describe('Function: deepSort', () => {
   it('can sort string correctly', () => {

packages/idempotency/tests/unit/idempotencyDecorator.test.ts

@@ -15,10 +15,7 @@ describe('Given a class with a function to decorate', () => {
       @idempotent({
         persistenceStore: new PersistenceLayerTestClass(),
       })
-      public async handler(
-        _event: unknown,
-        _context: Context
-      ): Promise<string> {
+      public handler(_event: unknown, _context: Context) {
         return this.privateMethod();
       }
 
@@ -48,10 +45,7 @@ describe('Given a class with a function to decorate', () => {
         config: idempotencyConfig,
         keyPrefix: 'my-custom-prefix',
       })
-      public async handler(
-        _event: unknown,
-        _context: Context
-      ): Promise<boolean> {
+      public handler(_event: unknown, _context: Context) {
         return true;
       }
     }