[Bug]: Cmd+C copies the entire conversation instead of the selection
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 72/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- javascript
調査の方向性
Start with the feed's copy listener and the Cmd+A and Mod+C handlers in the bundled renderer, then reproduce the issue with a tall expanded row and a selection inside it. Done means Cmd+C copies only the selection, explicit select-all still produces a transcript, and selections from the diff viewer's open shadow root do not trigger the transcript override.
索引モデルが issue の本文から書いたものです。
説明
GUI version
0.1.40
Operating system
macOS (Apple silicon)
Reproduction steps
- Open a conversation in which a single chat row can fill the mounted window — e.g. a turn whose work trace is expanded, a turn containing a large inline diff, or a very long answer.
- Press
Ctrl+O("Expand or collapse all traces") to expand every trace, or click a turn's "Worked for … · N steps" line to expand just that turn. - Select a few characters inside that turn. A small selection is enough; it does not need to span the conversation.
- Press
Cmd+Cand paste into any text editor.
Expected behavior
Cmd+C copies exactly the current selection. A whole-conversation transcript should only be produced by an explicit select-all action.
Actual behavior
The clipboard receives a plain-text transcript of the entire session — every message, each prefixed with User: / Assistant: — including messages that were never rendered on screen and were never part of the selection. The result is indistinguishable from the app's own select-all. Once it happens it repeats on every subsequent Cmd+C until the selection is cleared, because nothing clears the feed selection after the override.
Contributing factors, from reading the bundled renderer:
- The feed installs a
copylistener that builds a transcript of the whole message list and replaces the clipboard when a geometric predicate passes: the current DOM selection must partially contain (containsNode(row, true)) both the first and the last currently mounted[data-feed-row]element. - Because the feed is virtualized (
increaseViewportBy: {top: 800, bottom: 800}), the mounted rows are only the viewport ± 800 px window. When one tall row fills that window, the first and last row are the same element, so any selection inside that row satisfies the predicate — even three characters. Tall rows are easy to produce: expanding a trace renders the full reasoning text, tool calls and output cards inside a single row. - The predicate is evaluated against the rendered window, but the payload is always the entire session. So the clipboard can contain messages that were never rendered, let alone selected.
Related, same root cause: Mod+C is bound to run.interrupt ("Stops the agent when nothing is selected; copies otherwise"), and the Cmd+A handler only skips INPUT / TEXTAREA / contentEditable targets. Key events originating inside a shadow root are retargeted to the host element, so pressing Cmd+A while focus is inside the diff viewer (diffs-container, an open shadow root) is not excluded — it selects the entire feed, which then trips the transcript override on the next Cmd+C.
Screenshots, recordings, or sanitized logs
Sanitized reconstruction of the predicate (minified identifiers omitted), verified in a stock Chromium 153 harness using the same DOM shape ([data-feed-row] rows inside a scroll container):
const sel = feedEl.ownerDocument.getSelection();
if (!sel || sel.rangeCount === 0 || sel.isCollapsed) return false;
const rows = feedEl.querySelectorAll("[data-feed-row]");
const first = rows[0], last = rows[rows.length - 1];
return sel.containsNode(first, true) && sel.containsNode(last, true);
| setup | predicate result |
|---|---|
| 6 rows mounted, a few characters selected in a middle row | false |
| 6 rows mounted, selection spanning the first to the last mounted row | true |
| 1 row mounted (tall row), 3 characters selected inside it | true |
| selection made inside an open shadow root | false |
Also verified: when the row containing the selection is unmounted, Chromium collapses the selection — so detached/stale selections are not the cause; the geometry alone is.
Fix direction: gate the transcript override on an explicit flag set by the app's own select-all handler instead of inferring intent from selection geometry; if geometry is kept, require full containment (containsNode(row, false)) and emit only the rows actually covered by the selection; clear the feed selection after an override; skip the override when the selection's range root is not the document.
Safety check
- I removed credentials and private project information from this report.
- 主要言語
- Shell
- スター
- 85
- フォーク
- 5
- 平均マージ
- 2時間 46分
- マージ済み PR(30日)
- 1
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
CommandCodeAI/desktop のほかの issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
CommandCodeAI/desktop#125 · コメント 1 件 · 担当者 1 名 ·
-
[Feedback]: add copy block under a code block対応中かも @aliarain が 6 日前に担当しました。 オープンenhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
CommandCodeAI/desktop#94 · 担当者 1 名 ·
-
[Feedback]: Add Expand & Collapse for changes対応中かも @aliarain が 6 日前に担当しました。 オープンenhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
CommandCodeAI/desktop#66 · コメント 1 件 · リアクション 2 件 · 担当者 1 名 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
CommandCodeAI/desktop#128 ·
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
CommandCodeAI/desktop#127 ·
CommandCodeAI/desktop の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
pearu/agent-sandbox#156 ·
メンテナーはふだん 1 日以内に返信
-
area/documentation squad/marvin
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
rancher/stackstate-product-docs#443 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
getumbrel/umbrel-apps#6142 ·
メンテナーはふだん 2 日以内に返信
-
triage/confirmed
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
agentscope-ai/agentscope#3030 ·
メンテナーはふだん 1 日以内に返信