feat(infra): livedata effect

[EYHN]

No description provided.

[graphite-app]

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

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

You can enable merging using labels in your Graphite merge queue settings.

[EYHN]

• feat(infra): livedata effect #6281 👈
• style: enable import-x/no-duplicates #6279
• style: no import infra submodule #6278
• feat(electron): define runtimeConfig in esbuild #6287
• chore(electron): config vitest swc #6282
• style: enable rxjs/finnish #6276
• 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 a8cd157. 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/storage -- --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 x86_64-apple-darwin --use-napi-cross
• nx build @affine/native -- --target aarch64-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 81.70732% with 30 lines in your changes are missing coverage. Please review.

Project coverage is 60.04%. Comparing base (f2adbda) to head (a8cd157).

Files	Patch %	Lines
packages/common/infra/src/livedata/livedata.ts	83.82%	16 Missing and 6 partials ⚠️
packages/common/infra/src/livedata/effect/index.ts	45.45%	6 Missing ⚠️
packages/common/infra/src/livedata/ops.ts	88.23%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           canary    #6281      +/-   ##
==========================================
- Coverage   60.10%   60.04%   -0.07%     
==========================================
  Files         507      509       +2     
  Lines       23481    23512      +31     
  Branches     2151     2150       -1     
==========================================
+ Hits        14113    14117       +4     
- Misses       9107     9134      +27     
  Partials      261      261              

Flag	Coverage Δ
server-test	70.43% <ø> (-0.12%)	⬇️
unittest	38.69% <81.70%> (+0.14%)	⬆️

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]

Can you share some thoughts on effect design?

[pengx17]

Can you please share some thoughts on effect design?

[EYHN]

Can you please share some thoughts on effect design?

Main reference https://sigi.how/en/basic/effects
I guess we can see if it's practical later

[graphite-app]

Merge activity

• Mar 25, 2:08 AM EDT: Brooooooklyn added this pull request to the Graphite merge queue.
• Mar 25, 2:19 AM EDT: Brooooooklyn merged this pull request with the Graphite merge queue.