Drop currentModId as a concept

[grahamlangford]

What does this PR do?

• Refactors to eliminate the need for currentModId
• No longer set activeModId to null when selecting a modComponentId

Discussion

• See the InvariantMiddleware

For more information on our expectations for the PR process, see the
code review principles doc

[codecov]

Codecov Report

Attention: Patch coverage is 80.00000% with 10 lines in your changes missing coverage. Please review.

Project coverage is 75.79%. Comparing base (8318d74) to head (3b48053).
Report is 511 commits behind head on main.

Files with missing lines	Patch %	Lines
...geEditor/modListingPanel/modals/CreateModModal.tsx	0.00%	5 Missing ⚠️
...src/pageEditor/hooks/useCreateModFromUnsavedMod.ts	0.00%	2 Missing ⚠️
...ditor/modListingPanel/modals/SaveAsNewModModal.tsx	0.00%	1 Missing ⚠️
...geEditor/store/editor/editorInvariantMiddleware.ts	88.88%	1 Missing ⚠️
...ns/browser-extension/src/store/editorMigrations.ts	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9490      +/-   ##
==========================================
+ Coverage   74.24%   75.79%   +1.54%     
==========================================
  Files        1332     1422      +90     
  Lines       40817    42925    +2108     
  Branches     7634     7908     +274     
==========================================
+ Hits        30306    32534    +2228     
+ Misses      10511    10391     -120     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

Playwright test results

 153 passed
 1 flaky
 2 skipped

Details

 Open report ↗︎
 156 tests across 51 suites
 11 minutes, 8 seconds
 3b48053
 For more information on how to debug and view this report, see our readme

Flaky tests

chrome › tests/pageEditor/saveMod.spec.ts › shows error notification when updating a public mod without incrementing the version

Skipped tests

chrome › tests/runtime/googleSheetsIntegration.spec.ts › can activate a google spreadsheet mod with config options
msedge › tests/runtime/googleSheetsIntegration.spec.ts › can activate a google spreadsheet mod with config options

[twschiller]

Looking good overall. I'd prefer a migration: https://github.com/pixiebrix/pixiebrix-extension/pull/9490/files#r1837109301

See failing types job for unused import

[github-actions]

No loom links were found in the first post. Please add one there if you'd like to it to appear on Slack.

Do not edit this comment manually.

[grahamlangford]

Unrelated, just noticed while debugging