Refactor proxy code to pass strict-null checks

[petebacondarwin]

No description provided.

[changeset-bot]

🦋 Changeset detected

Latest commit: 15d7a40

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
wrangler	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[threepointone]

I prefer inlined abstractions to prevent jumping around to understand the code, but ok.

[threepointone]

Please add a changeset!

[petebacondarwin]

I prefer inlined abstractions to prevent jumping around to understand the code, but ok.

Yeah, I know what you mean.

There is also the converse problem where the inlined function is so big that it is difficult to understand the calling code: i.e. it is difficult to see and understand the calling code in one go, since the rest of the code is off the screen.

In this case, it felt that the benefits to the calling code outweighed the cost of not having the function inline. (Cmd-Click is my friend in these cases).