Memoize scope provider context value

[jhoch-nmnl]

Description of the change

I was testing whether jotai-molecules is a good fit for some of our state management needs, and ran into an issue during prototyping where a ScopeProvider would trigger downstream component renders more often than seemed necessary. I think I pinned down the problem, to an array instantiation that happens without memoization on every render of the ScopeProvider.

This change memoizes this array that is passed to the Context.Provider.

Note: I want to add a test or two to cover this but wanted to get feedback on whether this is a welcome change or not, first.

Type of change

• [x] Bug fix (non-breaking change that fixes an issue)
• [ ] New feature (non-breaking change that adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
• [ ] Documentation or Development tools (readme, specs, tests, code formatting)

Links

• Jira issue number: (PUT IT HERE)
• Process.st launch checklist: (PUT IT HERE)

Checklists

Development

• [ ] Prettier was run (if applicable)
• [ ] The behaviour changes in the pull request are covered by specs
• [x] All tests related to the changed code pass in development

Paperwork

• [ ] This pull request has a descriptive title and information useful to a reviewer
• [ ] This pull request has a Jira number
• [ ] This pull request has a Process.st launch checklist

Code review

• [ ] Changes have been reviewed by at least one other engineer
• [ ] Security impacts of this change have been considered

[loganvolkers]

@jhoch-nmnl the new version has a new format for doing context, so I think that this might no longer be the case.

If this seems like it's still a problem, can you please review and add a some test case? The testing was redone using vitest as part of V2

Finally, I'm very sorry that this has sat around so long without a comment or review. We'll try to do better going forward.