refactor(core): patch pdf viewer

[forehalo]

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.

[forehalo]

• refactor: pdf viewer components #8784
• refactor(core): patch pdf viewer #8782 👈 (View in Graphite)
• feat(core): pdf preview #8569 : 5 other dependent PRs (#8615 , #8671 , #8720 and 2 others)
• feat: allow peek view to be closed by the caller && bump bs #8542
• canary

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

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit cd74d55. 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 1 target

• nx test:coverage @affine/monorepo

---

Sent with 💌 from NxCloud.

[codecov]

Codecov Report

Attention: Patch coverage is 71.42857% with 4 lines in your changes missing coverage. Please review.

Project coverage is 70.19%. Comparing base (032f22a) to head (cd74d55).
Report is 2 commits behind head on fundon/10_21_feat_pdf_preview.

Files with missing lines	Patch %	Lines
packages/common/infra/src/op/message.ts	42.85%	3 Missing and 1 partial ⚠️

Additional details and impacted files

@@                        Coverage Diff                        @@
##           fundon/10_21_feat_pdf_preview    #8782      +/-   ##
=================================================================
- Coverage                          70.25%   70.19%   -0.06%     
=================================================================
  Files                                549      549              
  Lines                              33892    33898       +6     
  Branches                            3042     3043       +1     
=================================================================
- Hits                               23812    23796      -16     
- Misses                              9710     9731      +21     
- Partials                             370      371       +1     

Flag	Coverage Δ
server-test	77.12% <ø> (-0.07%)	⬇️
unittest	45.87% <71.42%> (-0.02%)	⬇️

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.

[EYHN]

i think fromPromise(renderer.call('render')) is more readable