fix: copy share link to clipboard command display issue

[pengx17]

fix AF-1027

[pengx17]

• fix: copy share link to clipboard command display issue #7411 👈
• canary

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

Join @pengx17 and the rest of your teammates on Graphite

[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.

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit ef7ba27. 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 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 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 57.24%. Comparing base (b8b30e7) to head (ef7ba27).

Files	Patch %	Lines
...end/core/src/modules/quicksearch/impls/commands.ts	0.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           canary    #7411      +/-   ##
==========================================
+ Coverage   57.21%   57.24%   +0.03%     
==========================================
  Files         902      902              
  Lines       39421    39421              
  Branches     4425     4425              
==========================================
+ Hits        22553    22567      +14     
+ Misses      16516    16499      -17     
- Partials      352      355       +3     

Flag	Coverage Δ
server-test	77.91% <ø> (ø)
unittest	28.99% <0.00%> (+0.08%)	⬆️

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.

[graphite-app]

Merge activity

• Jul 2, 11:18 AM EDT: pengx17 added this pull request to the Graphite merge queue.
• Jul 2, 11:42 AM EDT: pengx17 merged this pull request with the Graphite merge queue.

[CatsJuice]

It's a little strange to bind shortcut via invisible CMDK command. And ⌘ F has the same issue.

[pengx17]

It's a little strange to bind shortcut via invisible CMDK command. And ⌘ F has the same issue.

registerAffineCommand is designed to register command to affine. CMDK panel is one way to show the registered commands but may not be the only way.

I think registering keybinding here make things more flexible and maintainable. Keybinding should always be related to individual commands. Having a standalone keybinding registery seems not to be too beneficial.