refactor: DH-17778: Move exportLogs to the log package

[vbabich]

• Move exportLogs and logInit to the log package so they could be reused in Enterprise
• Remove @deephaven/redux and @deephaven/jsapi-shim dependencies from LogExport.ts
• Serialize Maps in redux data
• Unit tests for getReduxDataString

[codecov]

Codecov Report

Attention: Patch coverage is 15.38462% with 22 lines in your changes missing coverage. Please review.

Project coverage is 46.62%. Comparing base (229adf0) to head (bdd069a).
Report is 16 commits behind head on main.

Files with missing lines	Patch %	Lines
packages/log/src/LogExport.ts	20.00%	16 Missing ⚠️
packages/log/src/LogInit.ts	0.00%	3 Missing ⚠️
packages/code-studio/src/index.tsx	0.00%	2 Missing ⚠️
packages/code-studio/src/settings/SettingsMenu.tsx	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2279      +/-   ##
==========================================
+ Coverage   46.57%   46.62%   +0.05%     
==========================================
  Files         698      702       +4     
  Lines       38767    38937     +170     
  Branches     9866     9902      +36     
==========================================
+ Hits        18054    18153      +99     
- Misses      20702    20730      +28     
- Partials       11       54      +43     

Flag	Coverage Δ
unit	46.62% <15.38%> (+0.05%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[bmingles]

Is it ok that '' parses to NaN?

[vbabich]

This is existing code moved from LogInit - https://github.com/deephaven/web-client-ui/pull/2279/files#diff-8f738bb62eda99bdb9b608feb9a011431c9daef91788b6ad56166ff4d950ca9eL15
I don't know if this was the original intent, but NaN here results in a warning -

if (!Number.isFinite(level)) {
      console.warn(
        `Expected a number for log level. Received: ${level}. Ignoring`
      );
      return;
    }

[bmingles]

Should this be marked as a breaking change in the PR?

[vbabich]

The exportLogs wasn't previously exposed. The @deephaven/code-studio package didn't export anything from LogExport.ts.

[bmingles]

Had 1 question + I think this needs to be marked as a breaking change due to exportLogs contract change. Otherwise looks good.

[bmingles]

Looks good