refactor(storage)!: relocate ReadObjectResponse

[dbolduc]

Fixes #2889

I am open to a shorter module name. Just read_object?

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.71%. Comparing base (e836fe5) to head (09829b7).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2942   +/-   ##
=======================================
  Coverage   96.71%   96.71%           
=======================================
  Files         105      105           
  Lines        4353     4353           
=======================================
  Hits         4210     4210           
  Misses        143      143           

☔ 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.

[suzmue]

I see the issue says to move it to its own module, but I'm wondering if this would be appropriate to include in model_ext instead to have less places for looking for types.

I am open to a shorter module name. Just read_object?
+1 for just read_object

[coryan]

I see the issue says to move it to its own module, but I'm wondering if this would be appropriate to include in model_ext instead to have less places for looking for types.

We have kept traits outside model and model_ext. Maybe we shouldn't....

I am open to a shorter module name. Just read_object?
+1 for just read_object

Meeee toooo. I am stuck in meetings, please make a decision and let me know.

[suzmue]

We have kept traits outside model and model_ext. Maybe we shouldn't....

Ok, sticking with our norms SGTM. Rename to read_object and it LGTM

[dbolduc]

Rename to read_object

Done.

I think model_ext also made sense as it's kind of a return type.