Refine preferences

[syuilo]

What

Why

• defaultStoreとは別の簡易pizzax的なもので管理する
  • 今まではごちゃ混ぜだった
• アプリケーションの「状態」と「設定」が明確に分離されることで設定のバックアップで抜けが発生したりすることが原理的になくなる
• アカウント、デバイスに応じた設定のプロファイル切り替えとかも実装できる
• プロファイルを持ち出して別のクライアントで使用ができる
• レジストリAPIの叩きを削減しパフォーマンス向上
• 特定のアカウントでだけ設定値を上書きするなどの芸当も可能に
• 設定の自動バックアップ

TODO

• プロファイルのバックアップと復元 (preferences-backup)
• プロファイルの自動バックアップ
• 旧設定値のマイグレート
• テーマ関連(マイグレート)
• accountDependent考慮
• オーバーライドを設定するためのUI
• タブ間での状態同期
• その他のdefaultStoreプロパティについてpreferencesへの移行要否判断

nice2have

• 設定が吹き飛んだことを検知してバックアップからの復元を提案する
• バックアップはドライブではなくレジストリに保存する(ドライブだと内容をjsから読み込むことが難しい)
• 「アカウントで同期」
• plugin関連
• defaultStore -> storeにする？
  • 変数名衝突回避
• defaultStoreにdeckStoreを統合？
• ColdDeviceStorage消す？

Additional info (optional)

Checklist

• Read the contribution guide
• Test working in a local environment
• (If needed) Add story of storybook
• (If needed) Update CHANGELOG.md
• (If possible) Add tests

[syuilo]

設定が吹き飛んだことを検知してバックアップからの復元を提案するようにしたいけど、設定が吹き飛んでいるので設定が吹き飛んだことを原理的に検知できない

[syuilo]

設定が存在しなかった場合、新しいデバイス/ブラウザからログインしたから存在しなかったのか、設定が吹き飛んだから存在しなかったのか判別できない

[syuilo]

デバイス固有のIDみたいなのがJSで取得できればレジストリAPIで書き込んでおくことで、初めてのデバイスなのかそうでないのか判別することはできるけど、そのようなデバイスを特定できる情報は昨今プライバシーの関係で取得できなくなっている

[syuilo]

まあ新しいデバイス/ブラウザからログインしたから設定が存在しなかった場合でもバックアップから設定を復元したいことは変わりがないことが多そうだから区別する必要はそんなにないか

[codecov]

Codecov Report

Attention: Patch coverage is 21.20429% with 2133 lines in your changes missing coverage. Please review.

Project coverage is 15.28%. Comparing base (a3d236c) to head (f727eaf).
Report is 728 commits behind head on develop.

Files with missing lines	Patch %	Lines
packages/frontend/src/deck.ts	0.41%	241 Missing ⚠️
packages/frontend/src/preferences/utility.ts	0.53%	185 Missing ⚠️
packages/frontend/src/preferences/profile.ts	0.59%	166 Missing ⚠️
...ckages/frontend/src/pages/settings/preferences.vue	0.00%	161 Missing ⚠️
packages/frontend/src/plugin.ts	0.00%	139 Missing ⚠️
...ackages/frontend/src/pages/settings/appearance.vue	0.00%	131 Missing ⚠️
packages/frontend/src/boot/main-boot.ts	0.00%	105 Missing ⚠️
...ntend/src/scripts/autogen/settings-search-index.ts	0.00%	74 Missing ⚠️
packages/frontend/src/pages/settings/plugin.vue	0.00%	72 Missing ⚠️
packages/frontend/src/preferences.ts	1.72%	57 Missing ⚠️
... and 124 more

Additional details and impacted files

@@             Coverage Diff              @@
##           develop   #15597       +/-   ##
============================================
- Coverage    41.74%   15.28%   -26.47%     
============================================
  Files         1549      784      -765     
  Lines       196555    72025   -124530     
  Branches      2767     1268     -1499     
============================================
- Hits         82055    11008    -71047     
+ Misses      113939    60593    -53346     
+ Partials       561      424      -137     

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

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.