Skip to content

refactor(tui): add core command status and workspace helpers - #4

Closed
creatang wants to merge 70 commits into
codex/tui-split-02-adapters-infra-servicesfrom
codex/tui-split-03-core-support
Closed

refactor(tui): add core command status and workspace helpers#4
creatang wants to merge 70 commits into
codex/tui-split-02-adapters-infra-servicesfrom
codex/tui-split-03-core-support

Conversation

@creatang

@creatang creatang commented Apr 7, 2026

Copy link
Copy Markdown
Owner

背景:为 core/app 提供稳定可复用的支撑模块,减少重复逻辑。
本次改动:
新增 internal/tui/core/commands/*:命令解析与 workspace 命令逻辑。
新增 internal/tui/core/status/snapshot.go:状态快照收敛。
新增 internal/tui/core/utils/view_helpers.go:Fallback/TrimMiddle/TrimRunes/Clamp 统一工具函数。
新增 internal/tui/core/workspace/resolver.go:工作区路径解析。
影响范围:仅 internal/tui。
验证:
go test ./internal/tui/... ./internal/app
风险与回滚:
风险低;如异常可独立回滚该 PR。

phantom5099 and others added 30 commits April 4, 2026 23:31
feat(provider):增加透明重传以及解决缓冲区溢出问题
feat(security): 权限审批闭环 + ask策略引擎完整实现
feat(context): 增加旧工具结果的读时 Micro Compact
Cai-Tang-www and others added 25 commits April 7, 2026 20:04
…-adapters-infra-services

refactor(tui): extract components infra and services layers
refactor(session): 从 runtime 完全拆分会话模块并统一收口到 internal/session
…est robustness

- Add NUL check in services/file_service.go (Linux filepath.Abs does not error on NUL)
- Relax shell menu newline assertion on Windows for CJK path wrapping
- Add empty base path test case in workspace resolver (79% -> 92%)
@creatang
creatang force-pushed the codex/tui-split-03-core-support branch from 5bdc898 to c2fa334 Compare April 8, 2026 00:51
@creatang creatang closed this Apr 10, 2026
creatang pushed a commit that referenced this pull request Apr 14, 2026
fix: address memo review findings and simplify memo flow
creatang pushed a commit that referenced this pull request Apr 14, 2026
fix(runtime): recycle session locks and harden nil maps
creatang pushed a commit that referenced this pull request Apr 22, 2026
test: raise patch coverage for gateway runtime migration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants