fix(node): make sure onBeforeResponse and onAfterResponse are called with error code

[imslepov]

🔗 Linked issue

Closes #754

❓ 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)

📝 Checklist

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

[imslepov]

/cc @pi0 can you review my PR? 🙏 This fix is very necessary for my app.

[codecov]

Codecov Report

Attention: Patch coverage is 94.73684% with 1 line in your changes missing coverage. Please review.

Project coverage is 79.67%. Comparing base (a58d7c9) to head (ed3f579).
Report is 81 commits behind head on main.

Files	Patch %	Lines
src/app.ts	75.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #756      +/-   ##
==========================================
+ Coverage   77.83%   79.67%   +1.84%     
==========================================
  Files          47       54       +7     
  Lines        4286     5363    +1077     
  Branches      611      692      +81     
==========================================
+ Hits         3336     4273     +937     
- Misses        933     1071     +138     
- Partials       17       19       +2     

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

[pi0]

Thanks for PR and nice tests @imslepov ❤️

I have updated the implementation to minimize the required changes. Expected tests pass still but we call hooks only before sending response body.

In future, It should be cleaner but requires more additional core refactors later.