Flight Log / Deepractice/PromptX #119
🔧 ToolSandbox统一重构:修复process API缺失和代码重复问题 — a Claude Code (Anthropic) flight by Sean
Sean (@deepracticexs) opened this Claude Code (Anthropic) pull request in Deepractice/PromptX on Jul 5, 2025; merged clean in 1 min, merged by @deepracticexs: +87 −51, 1 file, 0 reviews.
- Lines
- +87 −51
- Files
- 1
- Commits
- 2
- Reviews
- 0
- To merge
- 1 min
- Task
- fix
Line and file counts are summed per commit from the AIDev dataset; GitHub's net diff can be smaller.
The agent, Claude Code, opened a fix on Deepractice/PromptX to unify the ToolSandbox refactor, replacing two sandbox-creation methods with createSandbox(options) and adding a createProcessMock() helper for the process API. The change touched 1 file across 2 commits, with 87 additions and 51 deletions. It merged clean with no changes requested, in 0.01 hours.
From the pull request description
## 🎯 **核心改进** ### **统一沙箱架构** - **合并重复方法**:删除`createBasicSandbox()`和`createSmartSandbox()`,统一为`createSandbox(options)` - **参数化控制**:通过`supportDependencies`参数控制分析/执行阶段行为 - **消除代码重复**:净减少~70行重复代码,提升可维护性 ### **完整process API支持** - **新增`createProcessMock()`**:提供完整的process对象mock - **关键API支持**:`process.cwd…
Excerpt (first 300 characters) — the full description is on GitHub.
View the pull request on GitHub →
This is your flight? Claim it — sign in with GitHub. Ownership is proven by the sign-in: the account that opened the PR is the only one that can claim it.
Flight timeline
- —committedsean
- —committedsean
- Jul 5, 2025mergeddeepracticexs
- Jul 5, 2025closeddeepracticexs
Read next
Cite this flight: AgentRiders Flight Log, Deepractice/PromptX#119, record updated Nov 3, 2026, https://agentriders.com/logs/Deepractice/PromptX/119/
Data from public GitHub activity; this page is not affiliated with GitHub or the repository.