fastify / fastify-autoload

Require all plugins in a directory
MIT License
333 stars 67 forks source link

fix: autohooks prefix concatenation #378

Closed climba03003 closed 6 months ago

climba03003 commented 6 months ago

Since no core member add value to https://github.com/fastify/fastify-autoload/issues/376#issuecomment-2129429693 I want to move things forward to users desire behavior.

Fixes #376

As mention in last submitted PR, I believe the long term solution is refactor the code to create a single tree that contain both hook and plugin information.

Checklist

climba03003 commented 6 months ago

Changing of test seems to indicate this is breaking, but actually not.

The new behavior is introduced in 5.8.1 and it was never tested before. And that is the one introducing new breaking behavior (or called non-clarified behavior).

Here is just reverting to behavior to the user most expected one.