feat(web): support Cmd+Click on new-session buttons to open in new tab by YoungY620 · Pull Request #1288 · MoonshotAI/kimi-cli

YoungY620 · 2026-02-28T05:34:41Z

Related Issue

N/A

Description

Support Cmd+Click (Mac) / Ctrl+Click (Windows/Linux) on all "+" new-session buttons to open the create-session flow in a new browser tab.

Changes:

web/src/App.tsx: Detect ?action=create and ?action=create-in-dir&workDir=xxx URL params on page load to auto-open the create dialog or directly create a session in the specified directory. Params are consumed after use to prevent re-triggering on refresh.
web/src/features/sessions/sessions.tsx: Top-level "+" button and grouped-view "New session here" button both support Cmd+Click to open a new tab. Tooltips updated with modifier-key hint.
web/src/features/chat/components/chat-conversation.tsx: Empty-state "Create new session" button supports Cmd+Click to open a new tab. Tooltip updated with modifier-key hint.

Checklist

I have read the CONTRIBUTING document.
I have linked the related issue, if any.
I have added tests that prove my fix is effective or that my feature works.
I have run make gen-changelog to update the changelog.
I have run make gen-docs to update the user documentation.

…ate session Detect `?action=create` to open CreateSessionDialog and `?action=create-in-dir&workDir=xxx` to directly create a session in the specified directory. Params are consumed after use to prevent re-triggering on refresh. Entire-Checkpoint: c0496aeaab22

The top-level "+" button opens a new tab with `?action=create`. The grouped-view "+" button opens a new tab with `?action=create-in-dir&workDir=xxx`. Tooltips updated with modifier-key hint. Entire-Checkpoint: 92c17dd2c99c

Opens a new tab with `?action=create` when modifier key is held. Tooltip updated with modifier-key hint. Entire-Checkpoint: 5cb94db2865a

Entire-Checkpoint: 5b2c427a42e0

Entire-Checkpoint: cb947d4bc3b3

Copilot

Pull request overview

This PR adds support for Cmd+Click (Mac) / Ctrl+Click (Windows/Linux) on all "new session" + buttons to open the create-session flow in a new browser tab. It does so by constructing a URL with ?action=create or ?action=create-in-dir&workDir=xxx params and opening that URL via window.open(..., "_blank"). The receiving tab reads those params on mount in App.tsx, performs the appropriate action, and then cleans the params from the URL.

Changes:

App.tsx — Adds a useEffect that reads ?action=create / ?action=create-in-dir URL params on page load to auto-open the dialog or silently create a session, then removes the params.
sessions.tsx — Top-level + button and per-group "New session here" button both grow Cmd/Ctrl+Click handling and updated tooltips.
chat-conversation.tsx — Empty-state "Create new session" button gains the same Cmd/Ctrl+Click logic and tooltip update.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

File	Description
`web/src/App.tsx`	Adds URL-param–driven effect to auto-open dialog or create a session on page load
`web/src/features/sessions/sessions.tsx`	Adds Cmd/Ctrl+Click handling + updated tooltips on two `+` buttons
`web/src/features/chat/components/chat-conversation.tsx`	Adds Cmd/Ctrl+Click handling + updated tooltip on empty-state button

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-28T05:38:39Z

+                    </KbdGroup>
+                  </div>
+                  <div className="flex items-center gap-2 text-xs text-muted-foreground">
+                    <span>{newSessionShortcutModifier}+Click open in new tab</span>


The tooltip text "{newSessionShortcutModifier}+Click open in new tab" is missing the word "to" — it should read "{newSessionShortcutModifier}+Click to open in new tab".

Suggested change

{newSessionShortcutModifier}+Click open in new tab

{newSessionShortcutModifier}+Click to open in new tab

Copilot · 2026-02-28T05:38:40Z

+    if (action === "create") {
+      setShowCreateDialog(true);
+    } else if (action === "create-in-dir") {
+      const workDir = params.get("workDir");
+      if (workDir) {
+        createSession(workDir);
+      }
+    } else {
+      return;
+    }
+    params.delete("action");
+    params.delete("workDir");
+    const url = new URL(window.location.href);
+    url.search = params.toString();
+    window.history.replaceState({}, "", url.toString());


There are two related issues with the create-in-dir branch:

Unhandled promise rejection: createSession(workDir) is called without await and without error handling. Since useEffect callbacks cannot be async, the promise rejection is silently swallowed. If the API call fails (e.g. network error, invalid directory), the user gets no feedback. The effect should use an inner async IIFE and wrap the call in try/catch to surface errors (e.g. via toast.error).

Params consumed when no action is taken: When action === "create-in-dir" but workDir is null/empty (e.g. a manually crafted URL), createSession is never called, but the URL params are still removed (lines 129–133 run). The user receives no feedback and no retry is possible. A return should guard line 129 when workDir is missing, or an error should be displayed.

Copilot · 2026-02-28T05:38:40Z

+                    </KbdGroup>
+                  </div>
+                  <div className="flex items-center gap-2 text-xs text-muted-foreground">
+                    <span>{newSessionShortcutModifier}+Click open in new tab</span>


The tooltip text "{newSessionShortcutModifier}+Click open in new tab" is missing a verb — it should read something like "{newSessionShortcutModifier}+Click to open in new tab". The same phrasing issue appears in all three tooltip locations in this PR: sessions.tsx lines 676 and 884, and chat-conversation.tsx line 167.

Suggested change

{newSessionShortcutModifier}+Click open in new tab

{newSessionShortcutModifier}+Click to open in new tab

Copilot · 2026-02-28T05:38:40Z

-                                <TooltipContent side="right">New session here</TooltipContent>
+                                <TooltipContent className="flex flex-col items-center gap-1" side="right">
+                                  <span>New session here</span>
+                                  <span className="text-xs text-muted-foreground">{newSessionShortcutModifier}+Click open in new tab</span>


The tooltip text "{newSessionShortcutModifier}+Click open in new tab" is missing the word "to" — it should read "{newSessionShortcutModifier}+Click to open in new tab".

Suggested change

{newSessionShortcutModifier}+Click open in new tab

{newSessionShortcutModifier}+Click to open in new tab

Entire-Checkpoint: 3dcde4c73a04

devin-ai-integration

Devin Review found 1 new potential issue.

View 8 additional findings in Devin Review.

devin-ai-integration · 2026-02-28T06:52:13Z

+      setShowCreateDialog(true);
+    } else if (action === "create-in-dir") {
+      const workDir = params.get("workDir");
+      if (!workDir) return; // invalid params, ignore silently


🟡 URL action params not cleaned up when create-in-dir is missing workDir

When the URL contains ?action=create-in-dir but no workDir parameter, the early return on line 123 skips the URL cleanup code (lines 130–134). The action parameter remains in the URL permanently.

Root Cause and Impact

The cleanup logic at web/src/App.tsx:130-134 only runs if the code doesn't hit an early return. The return on line 123 exits the effect before reaching the params.delete("action") call.

This means if a user navigates to ?action=create-in-dir (without workDir), the action param stays in the URL. On every page refresh, the effect re-runs, hits the same early return, and the stale param persists. While this is an edge case (the app always constructs URLs with workDir present at web/src/features/sessions/sessions.tsx:871-872), it violates the stated design goal of "Params are consumed after use to prevent re-triggering on refresh."

Suggested change

if (!workDir) return; // invalid params, ignore silently

if (!workDir) {

// Fall through to cleanup below

} else {

createSession(workDir).catch(() => {

// Errors are already handled globally via sessionsError → toast

});

}

Was this helpful? React with 👍 or 👎 to provide feedback.

Signed-off-by: Young E <49367723+YoungY620@users.noreply.github.com>

YoungY620 added 4 commits February 28, 2026 13:27

feat(web): add Cmd+Click support to sidebar new-session buttons

550f23f

The top-level "+" button opens a new tab with `?action=create`. The grouped-view "+" button opens a new tab with `?action=create-in-dir&workDir=xxx`. Tooltips updated with modifier-key hint. Entire-Checkpoint: 92c17dd2c99c

feat(web): add Cmd+Click support to empty-state create-session button

a130cbc

Opens a new tab with `?action=create` when modifier key is held. Tooltip updated with modifier-key hint. Entire-Checkpoint: 5cb94db2865a

feat(web): add Cmd+Click tooltip to grouped-view new-session button

c8c8932

Entire-Checkpoint: 5b2c427a42e0

Copilot AI review requested due to automatic review settings February 28, 2026 05:34

Copilot started reviewing on behalf of YoungY620 February 28, 2026 05:35 View session

chore: update changelog and docs for Cmd+Click new-tab feature

5151a4f

Entire-Checkpoint: cb947d4bc3b3

Copilot AI reviewed Feb 28, 2026

View reviewed changes

This comment was marked as resolved.

Sign in to view

YoungY620 added 2 commits February 28, 2026 14:47

fix: add missing 'to' in Cmd+Click tooltip text

bec5991

Entire-Checkpoint: 3dcde4c73a04

fix: handle createSession Promise rejection in URL action handler

07d035e

devin-ai-integration Bot reviewed Feb 28, 2026

View reviewed changes

This was referenced Feb 28, 2026

📊 AI CLI 工具社区动态日报 2026-02-28 rollysys/agents-radar#3

Open

📊 AI CLI 工具社区动态日报 2026-03-01 duanyytop/agents-radar#31

Closed

Merge branch 'main' into feat/cmd-click-new-tab-create-session

c2bce52

Signed-off-by: Young E <49367723+YoungY620@users.noreply.github.com>

YoungY620 merged commit ce49cf6 into main Mar 2, 2026
14 checks passed

YoungY620 deleted the feat/cmd-click-new-tab-create-session branch March 2, 2026 05:14

github-actions Bot mentioned this pull request Mar 3, 2026

📊 AI CLI 工具社区动态日报 2026-03-03 duanyytop/agents-radar#45

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(web): support Cmd+Click on new-session buttons to open in new tab#1288

feat(web): support Cmd+Click on new-session buttons to open in new tab#1288
YoungY620 merged 8 commits intomainfrom
feat/cmd-click-new-tab-create-session

YoungY620 commented Feb 28, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 28, 2026

Uh oh!

Copilot AI Feb 28, 2026

Uh oh!

Copilot AI Feb 28, 2026

Uh oh!

Copilot AI Feb 28, 2026

Uh oh!

This comment was marked as resolved.

Uh oh!

devin-ai-integration Bot left a comment

Uh oh!

devin-ai-integration Bot Feb 28, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	<span>{newSessionShortcutModifier}+Click open in new tab</span>
	<span>{newSessionShortcutModifier}+Click to open in new tab</span>

	<span className="text-xs text-muted-foreground">{newSessionShortcutModifier}+Click open in new tab</span>
	<span className="text-xs text-muted-foreground">{newSessionShortcutModifier}+Click to open in new tab</span>

-      if (!workDir) return; // invalid params, ignore silently
+      if (!workDir) {
+        // Fall through to cleanup below
+      } else {
+        createSession(workDir).catch(() => {
+          // Errors are already handled globally via sessionsError → toast
+        });
+      }

Conversation

YoungY620 commented Feb 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Related Issue

Description

Checklist

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

Uh oh!

This comment was marked as resolved.

Uh oh!

devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

Uh oh!

devin-ai-integration Bot Feb 28, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

YoungY620 commented Feb 28, 2026 •

edited

Loading