feat(electron): shared storage

[EYHN]

No description provided.

[graphite-app]

Your org has enabled the Graphite merge queue for merging into canary

Add the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[EYHN]

• feat(electron): multi tabs support #7440 : 2 dependent PRs (#7509 , #7635 )
• feat(electron): shared storage #7492 👈
• refactor(infra): memento use undefined #7491
• canary

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @EYHN and the rest of your teammates on Graphite

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit dca88e2. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

---

✅ Successfully ran 6 targets

• nx test:coverage @affine/monorepo
• nx build @affine/server-native -- --target x86_64-unknown-linux-gnu --use-napi-cross
• nx build @affine/native -- --target x86_64-pc-windows-msvc --use-napi-cross
• nx build @affine/native -- --target aarch64-apple-darwin --use-napi-cross
• nx build @affine/native -- --target x86_64-apple-darwin --use-napi-cross
• nx build @affine/native -- --target x86_64-unknown-linux-gnu --use-napi-cross

---

Sent with 💌 from NxCloud.

[codecov]

Codecov Report

Attention: Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 29.18%. Comparing base (0f14097) to head (dca88e2).

Files	Patch %	Lines
packages/common/infra/src/storage/memento.ts	60.00%	2 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           canary    #7492       +/-   ##
===========================================
- Coverage   57.74%   29.18%   -28.57%     
===========================================
  Files         910      708      -202     
  Lines       40090    16847    -23243     
  Branches     4536     3014     -1522     
===========================================
- Hits        23150     4916    -18234     
+ Misses      16587    11629     -4958     
+ Partials      353      302       -51     

Flag	Coverage Δ
server-test	?
unittest	29.18% <66.66%> (+0.06%)	⬆️

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.

[pengx17]

LGTM

[EYHN]

Merge activity

• Jul 14, 10:49 PM EDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.
• Jul 14, 11:04 PM EDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.
• Jul 14, 11:20 PM EDT: EYHN added this pull request to the Graphite merge queue.
• Jul 14, 11:36 PM EDT: EYHN merged this pull request with the Graphite merge queue.

[sentry-io]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ Error: The livedata is poisoned, original error: TypeError: Cannot read properties of undefined (reading... / View Issue

_{Did you find this useful? React with a 👍 or 👎}