Remove instanceof Response checks in favor of isResponse

[brophdawg11]

When using @remix-run/router for server-side data fetching, we seem to get different Response instances from a return fetch(...) in a loader/action. I'm not entirely sure why yet since we polyfill all fetch/Request/Response globals, but the instanceof Response checks were failing. Copied the same isResponse method used in Remix and replaced all checks with that.

I think the underlying cause here is that our loader is built via ESM and uses the ESM version of the fetch polyfills, but @remix-run/server-runtime + @remix-run/router are runningin node and therefore using the CJS versions of the fetch polyfills. Hence the instanceof failing.

Closes remix-run/remix#4741

[changeset-bot]

🦋 Changeset detected

Latest commit: 9fbf646

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

This PR includes changesets to release 5 packages

Name	Type
@remix-run/router	Patch
react-router	Patch
react-router-dom	Patch
react-router-dom-v5-compat	Patch
react-router-native	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

[machour]

That made me nuts a few days ago, fetch Response is being seen as web-fetch/Response