Merge pull request #56 from ZhouChaunge/feat/auto-selection-chip

feat: 选中代码自动生成 attachment chip（#55）

Author: ZhouChaunge

README.md

@@ -11,7 +11,7 @@
 
 <p align="center">
   <a href="https://code.visualstudio.com/"><img src="https://img.shields.io/badge/VS%20Code-%E2%89%A51.95.0-blue" alt="VS Code"/></a>
-  <a href="https://github.com/ZhouChaunge/DeepCopilot/releases"><img src="https://img.shields.io/badge/version-0.32.9-success" alt="Version"/></a>
+  <a href="https://github.com/ZhouChaunge/DeepCopilot/releases"><img src="https://img.shields.io/badge/version-0.33.0-success" alt="Version"/></a>
   <a href="./LICENSE"><img src="https://img.shields.io/badge/license-MIT-green" alt="License"/></a>
 </p>
 
@@ -469,6 +469,11 @@ The hook's stdout/stderr is appended to the tool result so the model can react 
 <details>
 <summary>点击展开完整更新日志 · Click to expand full changelog</summary>
 
+### v0.33.0 — Compact Tool UI · Sonar Spinner Redesign · 工具栏精简与进度动画重设计
+
+- 中文：大幅精简工具调用的显示方式，隐藏图标、三角形与分栏，改为单行灰色文本，文件路径更淡化，点击后展开详情；移除所有中间"思考中"气泡的耗时显示；底部进度动画全面重设计——保留蓝色 sonar 光波，去除背景色与边框，加入 20 个英文动词随机轮播（每 3 秒切换，渐显动画）与实时计时器；修复长回复时 spinner 被输入框遮挡的问题（requestAnimationFrame 布局时序修正 + 每秒兜底滚动）。
+- English: Heavily simplified tool-call display — icons, chevrons, and column layout removed in favour of a single-line grey text row with faded file paths and click-to-expand detail. Removed elapsed-time labels from all intermediate "Thinking" chips. Completely redesigned the bottom progress indicator: keeps the blue sonar dot, removes background / border / shimmer, adds a 20-word English verb carousel (randomised every 3 s with a fade-in animation) and a live elapsed timer. Fixed the spinner being obscured by the input box during long streaming responses via `requestAnimationFrame`-deferred scrolling and a per-second scroll safety net.
+
 ### v0.32.9 — Autopilot 静默放行工作区外路径 · Silent pass-through for out-of-workspace paths
 
 - 中文：在 `src/tools/utils.js` 的 `ensurePathAllowed()` 中新增审批模式检测。当 `approvalMode === 'autopilot'` 时，访问工作区外的路径（如 `~/.deepcopilot/memory.md`）**静默放行**，不再弹出“访问工作区之外”对话框，与 autopilot 语义保持一致。其他模式（`manual` / `auto-edit`）行为不变。

media/chat.css

@@ -2,7 +2,7 @@
   "name": "deep-copilot",
   "displayName": "Deep Copilot",
   "description": "Deep Copilot — AI coding agent embedded in VS Code. Powered by DeepSeek V4, with file editing, terminal, search, and plan/todos. Standalone, no backend required.",
-  "version": "0.32.9",
+  "version": "0.33.0",
   "publisher": "ZhouChaunge",
   "icon": "imgs/logo.png",
   "repository": {
@@ -122,6 +122,11 @@
         "command": "deepseekAgent.openDebugLog",
         "title": "Deep Copilot: Open Debug Log",
         "icon": "$(output)"
+      },
+      {
+        "command": "deepseekAgent.attachSelection",
+        "title": "附加到 Deep Copilot",
+        "icon": "$(link)"
       }
     ],
     "configuration": {
@@ -272,6 +277,16 @@
         },
         {
           "command": "deepseekAgent.moveToRight"
+        },
+        {
+          "command": "deepseekAgent.attachSelection"
+        }
+      ],
+      "editor/context": [
+        {
+          "command": "deepseekAgent.attachSelection",
+          "group": "deepcopilot@1",
+          "when": "editorTextFocus"
         }
       ]
     }
@@ -281,7 +296,8 @@
     "watch": "node esbuild.config.js --watch",
     "package": "npm run build && vsce package --out release/",
     "publish": "npm run build && vsce publish",
-    "vscode:prepublish": "npm run build"
+    "vscode:prepublish": "npm run build",
+    "lint": "eslint src/ --max-warnings 0"
   },
   "devDependencies": {
     "@types/node": "^20.0.0",
@@ -291,6 +307,7 @@
     "danger": "^13.0.7",
     "dompurify": "^3.4.2",
     "esbuild": "^0.28.0",
+    "eslint": "^8.57.0",
     "katex": "^0.16.45"
   }
 }

media/chat.js

@@ -86,7 +86,9 @@ class AgentLoop {
             const MAX_TOTAL = 256 * 1024;
             let totalSize = 0;
             for (const a of textAttachments) {
-                const block = `<attachment path="${a.path}">\n${a.content || '(empty)'}\n</attachment>`;
+                // Include line range attribute when the attachment is a selection
+                const lineAttr = (a.startLine && a.endLine) ? ` lines="${a.startLine}-${a.endLine}"` : '';
+                const block = `<attachment path="${a.path}"${lineAttr}>\n${a.content || '(empty)'}\n</attachment>`;
                 if (totalSize + block.length > MAX_TOTAL) {
                     attachmentBlocks += `<attachment path="${a.path}">(truncated — total attachment budget exceeded)</attachment>\n\n`;
                     break;
@@ -146,6 +148,21 @@ class AgentLoop {
 
         run.abortCtrl = new AbortController();
         const signal  = run.abortCtrl.signal;
+        const runT0   = Date.now();
+
+        // Helper: throw immediately if user pressed Stop.  Used at every
+        // await-boundary so the loop can unwind on the first stop click
+        // (issue #58 P0-2).
+        const checkAbort = () => { if (signal.aborted) throw new Error('aborted'); };
+        const postProgress = (phase, extra = {}) => {
+            this._postToRun(run, {
+                type: 'progress',
+                phase,
+                iter: run._iter || 0,
+                elapsedMs: Date.now() - runT0,
+                ...extra,
+            });
+        };
 
         const sysPrompt = buildSystemPrompt({ includeWorkspaceInstructions: true });
         const _itersRaw = Number(cfg.get('maxIterations'));
@@ -179,14 +196,17 @@ class AgentLoop {
         try {
             while (iter++ < MAX_ITERS) {
                 run._iter = iter;
+                checkAbort();
 
                 // Auto-compact
                 const compactRes = autoCompactIfNeeded(run.messages, COMPACT_BUDGET);
                 if (compactRes.compacted) {
                     run.messages = compactRes.messages;
                     Logger.info('AUTOCOMPACT', { sid, iter, dropped: compactRes.dropped });
                     this._postToRun(run, { type: 'status', text: isZh() ? '🗜 压缩历史…' : 'Compacting history…' });
+                    postProgress('compacting');
                 }
+                checkAbort();
 
                 // Plan nudge
                 if (run.plan && Array.isArray(run.plan.steps) && run.plan.steps.length) {
@@ -254,6 +274,7 @@ class AgentLoop {
                     }
                 }
                 if (ctxLimitHit) break; // break while loop — do not call the API
+                checkAbort();
 
                 // Rebuild msgs in case pre-flight compaction modified run.messages
                 const finalMsgs = [{ role: 'system', content: effectiveSysPrompt }, ...run.messages];
@@ -268,10 +289,14 @@ class AgentLoop {
                 const STREAMABLE_TOOLS = new Set(['write_file', 'str_replace_in_file', 'apply_patch']);
                 const allTools = getToolDefs(mcpManager.getToolDefs());
 
+                postProgress('waiting_first_token');
+
+                let _gotFirstToken = false;
                 const { toolCalls, usage } = await streamDeepSeek(
                     { apiKey, baseUrl, messages: finalMsgs, model, noTools: askMode, tools: allTools },
                     {
                         onDelta: (delta) => {
+                            if (!_gotFirstToken) { _gotFirstToken = true; postProgress('streaming'); }
                             assistantText += delta; run.reply.asst += delta;
                             pendingDelta += delta;
                             const now = Date.now();
@@ -281,6 +306,7 @@ class AgentLoop {
                             }
                         },
                         onThinking: (delta) => {
+                            if (!_gotFirstToken) { _gotFirstToken = true; postProgress('thinking'); }
                             reasoningText += delta; run.reply.thoughts += delta;
                             Logger.thinking(delta);
                             this._postToRun(run, { type: 'thinkingDelta', text: delta });
@@ -339,6 +365,7 @@ class AgentLoop {
 
                 // ── Parallel read / serial mutating dispatch ──────────────────
                 const results = new Array(toolCalls.length);
+                checkAbort();
 
                 // Phase 1: read-only tools in parallel
                 const parallelTasks = [];
@@ -347,6 +374,7 @@ class AgentLoop {
                     if (!READ_ONLY.has(tc.name)) continue;
                     const args = this._exec.logToolStart(run, tc);
                     const tT0  = Date.now();
+                    postProgress('tool_running', { activeTool: tc.name });
                     parallelTasks.push(
                         this._exec.execute(tc.name, args, mode, run, signal, tc.id)
                             .catch(e => `Error: ${e.message}`)
@@ -356,18 +384,22 @@ class AgentLoop {
                     );
                 }
                 if (parallelTasks.length) await Promise.all(parallelTasks);
+                checkAbort();
 
                 // Phase 2: mutating tools serially
                 for (let i = 0; i < toolCalls.length; i++) {
                     const tc = toolCalls[i];
                     if (READ_ONLY.has(tc.name)) continue;
+                    checkAbort();
                     const args = this._exec.logToolStart(run, tc);
                     const tT0  = Date.now();
+                    postProgress('tool_running', { activeTool: tc.name });
                     let rawResult = '';
                     try { rawResult = await this._exec.execute(tc.name, args, mode, run, signal, tc.id); }
                     catch (e) { rawResult = `Error: ${e.message}`; }
                     results[i] = { tc, args, result: this._exec.logToolResult(run, tc, rawResult, Date.now() - tT0) };
                 }
+                checkAbort();
 
                 // Phase 3: push tool messages + loop-guard checks
                 // IMPORTANT: The API requires all tool result messages to form a contiguous
@@ -498,8 +530,10 @@ class AgentLoop {
         Logger.flush();
 
         try { flushDelta(); } catch {}
-        this._postToRun(run, { type: 'replyEnd', empty: false });
+        const wasAborted = signal.aborted;
+        this._postToRun(run, { type: 'replyEnd', empty: false, aborted: wasAborted });
         this._postToRun(run, { type: 'status', text: '' });
+        if (wasAborted) this._postToRun(run, { type: 'stopped' });
         run.abortCtrl = null;
         run.busy = false;