feat(keybindings): start the first terminal from the keyboard

cmd-t is bound under the TerminalPane context, so a project with no
terminals — which has no pane to dispatch through — could only be started
by clicking the empty state's "Start Terminal" button. Reaching such a
project by keyboard therefore led nowhere.

Bind AddTab globally as well and handle it at the window level, but only
when the focused project has no layout: a focused pane always takes the
action first, and the guard means nothing changes for a project that
already has terminals. Modal context bails out too, so an overlay that
owns the keyboard while leaving a project focused cannot spawn terminals
behind itself.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Ud5F7z9cXJtzSjaJVRhzeN

Author: 2 people

crates/okena-app/src/keybindings/config.rs

@@ -171,11 +171,17 @@ impl KeybindingConfig {
                 KeybindingEntry::new("ctrl-d", Some("TerminalPane")),
             ],
         );
+        // Also bound globally: a project with no terminals has no TerminalPane
+        // to dispatch through, so the pane-scoped binding alone leaves "Start
+        // Terminal" reachable only by mouse. The window-level handler only acts
+        // when there is genuinely no pane to take it (see `render.rs`).
         bindings.insert(
             "AddTab".to_string(),
             vec![
                 KeybindingEntry::new("cmd-t", Some("TerminalPane")),
                 KeybindingEntry::new("ctrl-shift-t", Some("TerminalPane")),
+                KeybindingEntry::new("cmd-t", None),
+                KeybindingEntry::new("ctrl-shift-t", None),
             ],
         );
         bindings.insert(

crates/okena-app/src/views/window/render.rs

@@ -1,10 +1,10 @@
 use crate::keybindings::{
-    CheckForUpdates, ClearFocus, CloseWindow, CreateWorktree, EqualizeLayout, FocusActiveProject,
-    FocusSidebar, InstallUpdate, NewProject, NewWindow, OpenSettingsFile, RestartDaemon,
-    ReviewChanges, ShowBranchSwitcher, ShowCommandPalette, ShowContentSearch, ShowDiffViewer,
-    ShowFileSearch, ShowHookLog, ShowKeybindings, ShowLogConsole, ShowPairingDialog,
-    ShowProfileManager, ShowProjectSwitcher, ShowSessionManager, ShowSettings, ShowThemeSelector,
-    StartAllServices, StopAllServices, TogglePaneSwitcher, ToggleProjectLayout,
+    AddTab, CheckForUpdates, ClearFocus, CloseWindow, CreateWorktree, EqualizeLayout,
+    FocusActiveProject, FocusSidebar, InstallUpdate, NewProject, NewWindow, OpenSettingsFile,
+    RestartDaemon, ReviewChanges, ShowBranchSwitcher, ShowCommandPalette, ShowContentSearch,
+    ShowDiffViewer, ShowFileSearch, ShowHookLog, ShowKeybindings, ShowLogConsole,
+    ShowPairingDialog, ShowProfileManager, ShowProjectSwitcher, ShowSessionManager, ShowSettings,
+    ShowThemeSelector, StartAllServices, StopAllServices, TogglePaneSwitcher, ToggleProjectLayout,
     ToggleProjectVisibility, ToggleSidebar, ToggleSidebarAutoHide,
 };
 use crate::settings::{open_settings_file, settings_entity};
@@ -888,6 +888,41 @@ impl Render for WindowView {
                     ws.toggle_project_layout_mode(window_id, cx);
                 });
             }))
+            // Start the first terminal in a project that has none — the
+            // keyboard equivalent of the empty state's "Start Terminal" button.
+            //
+            // `AddTab` is normally handled by the focused TerminalPane; this
+            // only runs when the focused project has no pane to handle it,
+            // which is exactly the case the pane-scoped binding cannot reach. A
+            // project WITH a layout falls through untouched, so an open modal
+            // (which owns the keyboard while leaving a project focused) can't
+            // spawn terminals behind itself either.
+            .on_action(cx.listener(|this, _: &AddTab, _window, cx| {
+                if this.focus_manager.read(cx).is_modal() {
+                    return;
+                }
+                let project_id = {
+                    let fm = this.focus_manager.read(cx);
+                    fm.focused_terminal_state()
+                        .map(|state| state.project_id)
+                        .or_else(|| fm.focused_project_id().map(String::from))
+                };
+                let Some(project_id) = project_id else { return };
+                if this
+                    .workspace
+                    .read(cx)
+                    .project(&project_id)
+                    .is_none_or(|p| p.layout.is_some())
+                {
+                    return;
+                }
+                if let Some(dispatcher) = this.dispatcher_for_project(&project_id, cx) {
+                    dispatcher.dispatch(
+                        okena_core::api::ActionRequest::CreateTerminal { project_id },
+                        cx,
+                    );
+                }
+            }))
             // Hide the active project from this window's overview — the
             // keyboard route to the eye toggle in the project header. Scoped to
             // this window, like every other visibility toggle. Resolving the