Add a minimal message option for cleaner logs

[tigawanna]

Add a minimal messgae option for cleaner logs

ex: with minimal enabled

  return logger({
    pino: pino(
      {
        level: envVariables.LOG_LEVEL || "info",
      },
      envVariables.NODE_ENV === "production" ? undefined : pretty({
        colorize: true,
      }),
    ),
    http: {
      reqId: () => crypto.randomUUID(),
      minimalMessage:true,
    },
  });

[10:37:26.771] INFO (223659): ✅  /api/v1/inventory GET  200 OK  application/json 29ms

With custom bindings

  return logger({
    pino: pino(
      {
        level: envVariables.LOG_LEVEL || "info",
      },
      envVariables.NODE_ENV === "production" ? undefined : pretty({
        colorize: true,
      }),
    ),
    http: {
      reqId: () => crypto.randomUUID(),
      minimalMessage:(b,c)=>{
        return {
          extra:"i want to log this too",
          ...b,
        }
      }
    },
  });

[10:38:23.093] INFO (225266): Request completed
    extra: "i want to log this too"
    res: {
      "status": 200,
      "headers": {}
    }
    req: {
      "url": "/api/v1/inventory",
      "method": "GET",
      "headers": {

[maou-shonen]

Hi, Thank you for liking this package and contributing to it.
I originally planned to provide something similar using transport(but I've busy with work lately),
like this:

pinoLogger({
  pino: {
    transport: 'hono-pino/simple-logger'
  }
})

For me, delegating the task of formatting messages to the pino transport is a better choice.
However, there are indeed some limitations to this approach.
Could you provide some ideas to help me decide how to improve the API for this feature?

[tigawanna]

Anything that exposes a way to format the final output would be nice , personally i wanted minimal logs by default especially on dev to reduce the amount of scrolling necessary to find specific requests since currently a single request can span the entire terminal height

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (31aec6f) to head (f7d62f6).
Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##              main       #68    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files            5         5            
  Lines          169       290   +121     
  Branches        62        73    +11     
==========================================
+ Hits           169       290   +121     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[maou-shonen]

After considering it, I still believe this is the responsibility of pino.transport.
Would you be willing to modify it to the pino.transport version?

[tigawanna]

Will look at it later