[PR2/5] feat(upstream): ホットキー型拡張 + Prev/Next Tab・Workspace の unbound 化 (#3422)

[MocA-Love]

Upstream Merge PR#2 - ホットキー型拡張

upstream superset-sh#3422 を cherry-pick し、フォーク固有ホットキーと整合させるための手動修正を加えます。

改善内容の詳細

背景：PR#3403 で何が起きたか（このPRではまだ取り込んでいないが、関連する文脈）

upstream は PR#3403 で「Cmd+Alt+Arrow で v2 ペイン間を空間認識的に移動」する機能を入れるため、以下を削除しました：

• `PREV_TAB` / `NEXT_TAB` （Cmd+Alt+Left/Right）
• `PREV_WORKSPACE` / `NEXT_WORKSPACE` （Cmd+Alt+Up/Down）
• `PREV_PANE` / `NEXT_PANE` （線形循環）

これらのキーを FOCUS_PANE_LEFT/RIGHT/UP/DOWN に付け替えるためでした。

このPRで取り込む superset-sh#3422 の内容

問題： superset-sh#3403 でタブ切替・ワークスペース切替のホットキーが完全に削除されてしまい、これらの機能に依存していたユーザーが困った。

解決： ホットキー ID としては復活させるが、デフォルトキーは `null` とし、ユーザーが Settings から自由に rebind できる形にする。

このアプローチのために、ホットキーの型システムを以下のように拡張する必要があります：

型定義の拡張（事前修正として手動コミット）

```typescript
// Before
export type PlatformKey = { mac: string; windows: string; linux: string };
export interface HotkeyDefinition {
key: string;
label: string;
...
}

// After
export type PlatformKey = {
mac: string | null;
windows: string | null;
linux: string | null;
};
export interface HotkeyDefinition {
key: string | null;
label: string;
...
}
```

これで `key: { mac: null, windows: null, linux: null }` というデフォルト未割当エントリがレジストリに登録できるようになります。

cherry-pick: c925f4d (superset-sh#3422)

1. registry.ts

   • `PREV_TAB`, `NEXT_TAB` を null-bound エントリとして復活
   • `PREV_WORKSPACE`, `NEXT_WORKSPACE` を null-bound エントリとして復活
   • 各エントリに `description` を追加
   • 旧キーバインド（`meta+alt+left/right` 等）は削除

2. useRecordHotkeys.ts

   • `canonicalizeChord(defaultKey)` を null-guard
   • これで unbound ホットキーに対する新規 chord 記録時に例外が出なくなる

3. 各 handler ファイル

   • `useDashboardSidebarShortcuts.ts`: PREV/NEXT_WORKSPACE ハンドラ復活、`index === -1` ガード追加
   • `v2 useWorkspaceHotkeys.ts`: PREV/NEXT_TAB ハンドラ復活
   • `v1 workspace/page.tsx`: PREV/NEXT_TAB ハンドラ復活

コンフリクト解決

ファイル	解決方針
`registry.ts`	フォーク独自の `BROWSER_RELOAD` / `BROWSER_HARD_RELOAD` / `SEARCH_IN_FILES` / `HotkeyCategory "Browser"` を保持。`PREV_PANE` / `NEXT_PANE` も PR#3 の取り込み時まで一時的に保持。重複した `PREV_TAB`/`NEXT_TAB` エントリを削除
`useDashboardSidebarShortcuts.ts`	フォークのコメントを除去、upstream の `index === -1` ガードを採用
`workspace/$workspaceId/page.tsx`	フォーク独自の deep-link navigation（`useSearch` / `WorkspaceSearchParams`）を保持。フォーク独自の `CLOSE_TERMINAL` / `CLOSE_TAB` ハンドラを保持。cherry-pick が挿入した重複 workspace クエリとハンドラを削除

フォーク固有機能の保持

• ブラウザホットキー: `BROWSER_RELOAD`, `BROWSER_HARD_RELOAD`, `SEARCH_IN_FILES` すべて維持
• HotkeyCategory "Browser": 維持
• v1 deep-link navigation: `useSearch` 経由のファイル/行/列パラメータ処理を維持
• v1 tRPC-based PREV/NEXT_WORKSPACE: 維持（upstream の `flattenedWorkspaces` 方式ではなくフォーク独自の tRPC クエリ方式）

ユーザーへのメリット

1. 既存のカスタムバインディングが保持される

   • 以前 PREV_TAB/NEXT_TAB に独自のキーを割り当てていたユーザーは、localStorage の override がそのまま有効になり、自動的にバインディングが復活する

2. 設定画面で自由に rebind できる

   • デフォルトは unbound なので、ユーザーが Settings → Keyboard で好きなキーを設定できる

テスト計画

• `bun run typecheck` パス
• Settings → Keyboard に PREV_TAB / NEXT_TAB が「unbound」として表示される
• Settings → Keyboard に PREV_WORKSPACE / NEXT_WORKSPACE が「unbound」として表示される
• フォーク独自ホットキーが引き続き動作：ブラウザリロード、ハードリロード、ファイル内検索
• PR#3403 以前の localStorage override を持つユーザーは、PREV_TAB/NEXT_TAB のバインディングが自動復活する

[coderabbitai]

Warning

Rate limit exceeded

@MocA-Love has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 54 minutes and 37 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 54 minutes and 37 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 08772e7b-7e41-4c82-94a1-b1ecaee0458b

📥 Commits

Reviewing files that changed from the base of the PR and between 80ab6a0 and 5649462.

📒 Files selected for processing (5)

• apps/desktop/src/renderer/hotkeys/hooks/useRecordHotkeys/useRecordHotkeys.ts
• apps/desktop/src/renderer/hotkeys/registry.ts
• apps/desktop/src/renderer/hotkeys/types.ts
• apps/desktop/src/renderer/hotkeys/utils/resolveHotkeyFromEvent.test.ts
• apps/desktop/src/renderer/routes/_authenticated/_dashboard/components/DashboardSidebar/hooks/useDashboardSidebarShortcuts/useDashboardSidebarShortcuts.ts

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch upstream-merge/pr2-hotkey-types

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}