feat: defineRequestMidleware, defineResponseMiddleware and rename object synctax hooks

[pi0]

🔗 Linked issue

Context: Find most of DX issues while working on a example for #501

❓ Type of change

• 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality like performance)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This PR introduces two new define utils defineRequestMidleware and defineResponseMiddleware (with RequestMiddleware and ResponseMiddleware) types.

These utils are then used for event handlers with object syntax but can also used independently to define such hooks outside of the event handler in a composable way (imagine compressionMiddleware as util)

Also in order to keep consistency with global hook options and readability, the hooks are renamed from before/after to onRequest/beforeResponse.

Also we support non Array format for most consistency with top level

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[codecov]

Codecov Report

Merging #507 (6bf6992) into main (8079fa9) will decrease coverage by 0.32%.
The diff coverage is 51.85%.

❗ Current head 6bf6992 differs from pull request most recent head f577876. Consider uploading reports for the commit f577876 to get more accurate results

@@            Coverage Diff             @@
##             main     #507      +/-   ##
==========================================
- Coverage   82.43%   82.11%   -0.32%     
==========================================
  Files          31       31              
  Lines        3604     3646      +42     
  Branches      536      536              
==========================================
+ Hits         2971     2994      +23     
- Misses        633      652      +19     

Files Changed	Coverage Δ
src/event/utils.ts	58.38% <31.57%> (-4.30%)	⬇️
src/types.ts	100.00% <100.00%> (ø)

[danielroe]

This looks good to me, from an API perspective.

Note that we will probably need to change the signature of these types before merging any kind of validation support. You could already go ahead and add the appropriate generics (even if unused) in the hook types so we don't break anything when implementing later.

[pi0]

These middleware are same as current before/after. Do you mind to push any required placeholder generic?

[danielroe]

I think prefixing with underscore would be enough to mark them as internal types for now.