Simplify bodyWriter write method using io.Copy

[samber]

Refactor write handling in bodyWriter to use io.Copy for simplicity.

poke @schnoddelbotz

[codecov]

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (d098e39) to head (67f8f4e).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
dump.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff          @@
##            main     #76   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          4       4           
  Lines        404     372   -32     
=====================================
+ Misses       404     372   -32     

Flag	Coverage Δ
unittests	0.00% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

Refactors bodyWriter.ReadFrom in dump.go to rely on io.Copy instead of maintaining a custom copy loop, simplifying response body capture while preserving the ability to delegate to the underlying gin.ResponseWriter when possible.

Changes:

• Removed the custom buffered copy loop and replaced it with io.Copy (with a wrapper to avoid io.ReaderFrom recursion).
• Deleted the now-unneeded errInvalidWrite error value.

Comments suppressed due to low confidence (1)

dump.go:19

• After removing errInvalidWrite, the errors import in this file becomes unused, which will cause the package to fail to compile. Please remove the errors import (or reintroduce a use if needed).

	gin.ResponseWriter
	body    *bytes.Buffer
	maxSize int

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

io.Copy(struct{ io.Writer }{w}, r) intentionally wraps w to hide the io.ReaderFrom method (otherwise io.Copy would dispatch back to ReadFrom and recurse). Please add a short comment or use a named wrapper type to make this non-obvious requirement clearer and prevent accidental simplification to io.Copy(w, r) later.

Suggested change

	// Wrap w to hide its ReadFrom method and avoid infinite recursion via io.Copy.