Feature request: export LoggerInterface in `@aws-lambda-powertools/logger/types`

[sandrokeil]

Expected Behavior

The LoggerInterface should be able to be imported, so I can decorate the logger with my own implementation.

import { LoggerInterface } from '@aws-lambda-powertools/logger/types';

Current Behavior

The current export of logger types is missing the LoggerInterface from Logger.d.ts. See

powertools-lambda-typescript/packages/logger/src/types/index.ts

Lines 9 to 18 in 72803e2

	export type {
	LogItemMessage,
	LogItemExtraInput,
	LambdaFunctionContext,
	UnformattedAttributes,
	PowertoolsLogData,
	ConstructorOptions,
	InjectLambdaContextOptions,
	CustomJsonReplacerFn,
	} from './Logger.js';

Code snippet

import { LoggerInterface } from '@aws-lambda-powertools/logger/types';

Steps to Reproduce

1. install powertools
2. try to import the LoggerInterface
3. TS2305 (no exported member) error should occur

Possible Solution

Export also LoggerInterface here

powertools-lambda-typescript/packages/logger/src/types/index.ts

Lines 9 to 18 in 72803e2

	export type {
	LogItemMessage,
	LogItemExtraInput,
	LambdaFunctionContext,
	UnformattedAttributes,
	PowertoolsLogData,
	ConstructorOptions,
	InjectLambdaContextOptions,
	CustomJsonReplacerFn,
	} from './Logger.js';

Powertools for AWS Lambda (TypeScript) version

latest

AWS Lambda function runtime

20.x

Packaging format used

npm

Execution logs

No response

[boring-cyborg]

Thanks for opening your first issue here! We'll come back to you as soon as we can.
In the meantime, check out the #typescript channel on our Powertools for AWS Lambda Discord: Invite link

[dreamorosi]

Hi @sandrokeil - thank you for opening the issue.

Could you please share a bit more about the use case for having the LoggerInterface exposed?

[sandrokeil]

@dreamorosi I want to create a logger class which implements the LoggerInterface. For this I want to use the decorator design pattern. I want to add the functionality of a so called FingersCrossed logger. If the LoggerInterface is exposed, I can implement any new functionality to the logging methods and don't need to change my code because I can reference the interface instead of the concrete implementation.

Any concerns about to export the LoggerInterface?

FingersCrosserLogger

A very interesting wrapper. It takes a handler (in my case the powertools logger) as a parameter and will accumulate log records of all levels until a record exceeds the defined severity level. At which point it delivers all records, including those of lower severity, to the handler it wraps. This means that until an error actually happens you will not see anything in your logs, but when it happens you will have the full information, including debug and info records. This provides you with all the information you need, but only when you need it.

[dreamorosi]

Hey, thanks for the additional info.

I don't have any major concern with exposing the interface, I'm only curious to learn how you'd use it.

I remember removing it from the exported types because I though having the actual class would be enough to extend the behavior, for example:

import { Logger } from "@aws-lambda-powertools/logger"

class MyLogger extends Logger {}

By using the interface with implements you'd have to replace the entirety of the Logger:

import type { LoggerInterface } from "@aws-lambda-powertools/logger/types"

class MyLogger implements LoggerInterface {}

Am I misunderstanding the intent?

I would really appreciate if you could share a concrete code example, even if it's pseudo code.

[dreamorosi]

Either way - based on our definition of bug this should be classified as a feature request, since it doesn't have an impact on the runtime behavior of the tool.

I'm going to change the type to Feature request and label as improvement.