[Bug] 修改DeepCode Studio页面

[CJBshuosi]

What happened?

DeepCode Studio页面有一些用户交互不合理，需要修改相关代码问题

What did you expect?

完善UI页面，完善页面逻辑

Steps to reproduce

1.本地修改代码
2.本地测试代码功能正常
3.合入自己仓库后提pr到主仓dev分支

Logs / stack trace

Commit / branch

No response

Environment

No response

[jerry609]

DeepCode Studio 全面审查报告

基于 dev 分支的完整代码审查，涵盖前端 22 个文件 + 后端 34 个文件。
按严重程度排序：🔴 高 / 🟡 中 / 🔵 低

---

一、安全漏洞（必须修复）

🔴 1. 命令注入 — verification_agent.py L219 / verification_node.py L540

使用 f-string 拼接 output_dir 进 shell 命令，攻击者可通过构造路径执行任意命令：

# 当前代码 — 危险
f"cd '{output_dir}' && ..."
# 修复方案：使用 subprocess.run(cwd=output_dir) 参数

🔴 2. --dangerously-skip-permissions — studio_chat.py L75

Code 模式绕过所有 Claude Code 工具审批，任何用户请求都能执行任意文件/命令操作。应改为 --allowedTools 白名单机制。

🔴 3. 全局环境变量竞争 — e2b_executor.py L99/193/314

os.environ["E2B_API_KEY"] = self.api_key 直接写全局环境变量，多线程/并发请求时会互相覆盖。应通过 SDK 参数传递。

---

二、后端 Bug

🔴 4. paper2code_job 未注册 — arq_worker.py

job_manager.py L263 提交 "paper2code_job" 到 Redis 队列，但 arq_worker.py 的 WorkerSettings.functions 中未注册此函数。提交的任务会被静默丢弃。

🔴 5. E2B Sandbox 资源泄漏 — e2b_executor.py

run() 方法每次新建 sandbox 但从不存入 self._sandbox，导致 cleanup() 无法清理。_get_sandbox() 方法存在但从未被 run() 调用 — 死代码。

🔴 6. asyncio.create_task 无引用 — repro_context.py L201

asyncio.create_task(_run()) 没有保存返回的 task 引用。CPython 可能在任务完成前将其垃圾回收，导致任务静默消失。

🟡 7. 异步中调用同步阻塞 — 多处

• repro_agent.py L462: self.executor.run() 是同步方法，阻塞事件循环
• studio_chat.py L581: subprocess.run() 阻塞 async endpoint
• host.py L107: self.llm.invoke() 同步调用

🟡 8. 临时目录泄漏 — gen_code.py L81

tempfile.mkdtemp() 创建的目录永远不会清理。每次调用都会在 /tmp 中留下残留目录。

🟡 9. Anthropic API 调用错误 — studio_chat.py L495

stream_anthropic_api 将 role: "system" 放入 messages 数组。Anthropic Messages API 不接受此角色，system prompt 应放在顶层 system 参数中。调用必定返回 400 错误。

🟡 10. 模块级副作用 — runbook.py L30-31

_provider = SessionProvider()
Base.metadata.create_all(_provider.engine)

import 时即创建数据库表，影响测试隔离性和启动性能。

🟡 11. Redis 连接泄漏 — sandbox.py

每个请求创建新的 JobManager() + connect()，没有连接池或单例模式。高并发下大量 Redis 连接。

🟡 12. Docker container.wait(timeout) 语义错误 — docker_executor.py L80

Docker SDK 的 timeout 参数是 HTTP 请求超时，不是容器执行超时。长时间运行的容器不会在 timeout_sec 后被杀死。

🟡 13. ParallelOrchestrator 是空壳 — orchestrator.py L341-368

run_parallel() 直接调用 self.run() 顺序执行。"并行"功能完全缺失。

🟡 14. 幽灵依赖 claude_agent_sdk — 7 个文件

planning_node.py, analysis_node.py, blueprint_node.py, environment_node.py, generation_node.py, verification_node.py, debugging_agent.py 都 import 了不存在的 claude_agent_sdk。所有 LLM 驱动的代码路径永远不会执行。

🟡 15. SymbolIndex 符号覆盖 — symbol_index.py L105

同名符号（如不同文件中的 class Model）会静默覆盖，丢失多文件项目中的符号信息。

🔵 16. fcntl 仅限 Linux/macOS — runbook.py L2

Windows 上 import fcntl 会抛出 ImportError。

🔵 17. Orchestrator 超时配置无效 — orchestrator.py L54

timeout_seconds 字段被定义但从未使用，没有 asyncio.wait_for 包装。

---

三、前端 Bug

🔴 18. Paper ID 不匹配 — page.tsx L98-141

URL 参数 paper_id 指向一篇已有论文时正常。但当论文不存在时，addPaper() 生成新 ID，而后续 generate({ paperId }) 仍使用 URL 中的旧 ID — 指向错误的论文。

🟡 19. SSE 流无终结状态 — ReproductionLog.tsx L371-393

如果 SSE 流结束时既没有 result 也没有 error 事件，状态永远停在 "running"。

🟡 20. Hunk 预览 revert 路径错误 — WorkspacePanel.tsx L472

点击 hunk 预览的 "Revert" 按钮时，使用组合字符串（如 "file.py - hunk-1"）作为文件路径调用 revert API — 路径无效。

🟡 21. 空文件内容误判 — ReproductionLog.tsx L632

newContent || diff || '// Modified file content'：空字符串（删除文件的合法内容）因 || 被跳过，显示 diff 或占位符。应改用 ??。

🟡 22. Task ID 碰撞 — studio-store.ts L316

addTask 用 Date.now() 生成 ID，毫秒级并发时会碰撞。addPaper 已用 Date.now() + random 方案，应统一。

🔵 23. localStorage JSON.parse 无验证 — studio-store.ts L87

JSON.parse(stored) as StudioPaper[] 无运行时校验。旧版本数据结构或损坏数据会导致后续运行时崩溃。

🔵 24. 无 SSE 取消机制 — ReproductionLog.tsx

用户切换论文或导航离开时，SSE fetch 在后台继续执行，可能更新过时状态。应使用 AbortController。

🔵 25. Chat 不自动滚动 — ReproductionLog.tsx

新消息到达时 ScrollArea 不会自动滚动到底部。

---

四、死代码（11/22 前端文件未使用）

以下 Studio 组件未被任何文件 import，属于死代码：

文件	状态
RunbookPanel.tsx	@deprecated 标记
BlueprintPanel.tsx	@deprecated 标记
DeepCodeChat.tsx	未 import
MCPSettings.tsx	@deprecated 标记
TasksPanel.tsx	@deprecated 标记
PaperCard.tsx (studio)	未被 studio 组件 import
PapersPanel.tsx	未 import
PromptInput.tsx	未 import
ExecutionLog.tsx	未 import
WorkspacePanel.tsx	未 import
DeepCodeEditor.tsx	仅被死代码 WorkspacePanel import

建议：统一清理或归档这些文件，避免维护负担。

---

五、代码重复

重复代码	出现位置
formatRelativeTime()	PaperCard, PapersPanel, PaperGallery, TasksPanel
statusConfig 对象	page.tsx, PapersPanel, PaperGallery, PaperCard
languageForPath()	FilesPanel, WorkspacePanel
ActionItem + actionIcons	ReproductionLog, ExecutionLog
验证逻辑	verification_agent.py ↔ verification_node.py（大段重复）

---

六、AgentSwarm 架构差距

以下为设计文档 docs/proposals/agentswarm-design.md 中规划但完全缺失的组件：

设计组件	当前状态	影响
AgentRouter	❌ 缺失	无任务路由，固定顺序执行
AgentAdapter (统一接口)	❌ 缺失	Claude CLI 直接 subprocess 调用
MessageBus (类型化事件)	❌ 缺失	Agent 共享可变 Dict[str, Any]
SessionManager	❌ 缺失	每次请求无状态
IntentGate / CategoryRouter	❌ 缺失	无意图分类
SharedWorkspace	❌ 缺失	原始文件系统访问
/api/swarm/* 端点	❌ 缺失	无 swarm 会话管理 API
Event Sourcing	❌ 缺失	无事件重放
并行执行	⚠️ 空壳	ParallelOrchestrator 仍调用顺序 run()
ToolApproval 工作流	❌ 缺失	使用危险的 --dangerously-skip-permissions

---

建议修复优先级

P0（安全 + 数据丢失）: #1 命令注入, #2 权限绕过, #3 环境变量竞争, #4 任务静默丢弃, #6 task GC
P1（功能 Bug）: #5 sandbox 泄漏, #7 阻塞事件循环, #8 临时目录泄漏, #9 API 错误, #18 ID 不匹配, #19 无终结状态
P2（代码质量）: 死代码清理, 代码重复消除, #14 幽灵依赖
P3（架构）: AgentSwarm 组件实现（按 Epic #197 规划推进）

[jerry609]

🔗 Related Issue

• [Epic] P2C Pipeline 接入 Claude Code Agent SDK — 从裸 API 调用升级为 Agent 驱动 #227 — P2C Pipeline 接入 Claude Code Agent SDK：详细的 CC 集成方案，解决当前 P2C 管线使用 phantom claude_agent_sdk 依赖和直接调用 raw LLM API 的问题。

该 issue 直接关联 #222 中「AgentSwarm 架构缺失」部分的核心问题：P2C 管线缺乏真正的 Agent 能力（工具调用、会话管理、安全沙箱）。

[jerry609]

Fixed in DeepCode Studio page cleanup commit.