fix(connect): log mapStateToProps errors to console.error in dev (#1942)

[webdevelopersrinu]

Summary

Logs errors thrown in mapStateToProps (or selectors) to console.error
in non-production environments, so they surface during development
instead of being silently swallowed.

Fixes #1942

Background

Since v8.0.0, errors thrown inside mapStateToProps are caught in the
subscription callback (connect.tsx#L133-L136) and stored in
lastThrownError. In v7, those errors surfaced because the component
unmounted on selector errors (the unmount path re-throws
lastThrownError). In v8+, the component no longer unmounts — it just
re-renders in its pre-error state — so the error is silently swallowed.

This was diagnosed by @alexanderchr in
#1942 (Dec 12, 2024), and @markerikson approved the dev-only
logging approach in the same thread:

"logging this in dev seems like a passable fallback"

Changes

• src/components/connect.tsx — log the caught error to console.error
  when process.env.NODE_ENV !== 'production', with a brief message and
  a link back to this issue
• test/components/connect.spec.tsx — add a unit test that dispatches
  an action which causes mapStateToProps to throw, then asserts the
  error was logged via console.error

Behavior

• ✅ Production: unchanged (the new branch is dead-code-eliminated)
• ✅ Development: error is now logged to the console with the
  original error object, making it discoverable in DevTools

Test plan

• yarn test — all connect tests pass (65/65), new test passes,
  no regressions in code I touched
• yarn lint runs as part of pretest and passes
• Type-check passes (vitest --typecheck reports no errors)

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

[aryaemami59]

The current CI failure is related to arethetypeswrong/arethetypeswrong.github.io#258 which is caused by 101arrowz/fflate#258.

As a temporary workaround, we can pin fflate to 0.8.2 via resolutions in package.json:

  "resolutions": {
    "fflate": "0.8.2"
  },

This should unblock CI until the upstream issue is resolved.

[webdevelopersrinu]

Thanks @aryaemami59 — totally missed your earlier comment, sorry about
that. Happy to add the fflate resolutions pin to this PR if you'd
prefer, or wait if you're patching master directly. Just let me know.

[aryaemami59]

@webdevelopersrinu

Thanks @aryaemami59 — totally missed your earlier comment, sorry about that. Happy to add the fflate resolutions pin to this PR if you'd prefer, or wait if you're patching master directly. Just let me know.

Thanks. I'd prefer to wait and see whether this gets resolved upstream first. That also gives the rest of the team more time to review the PR.

Update: This has since been fixed upstream in @arethetypeswrong/core@0.18.3, so the fflate resolutions pin isn't needed here. I've opened #2307 to bump @arethetypeswrong/cli to 0.18.3 and unblock CI.