Skip to content

[codex] 增加 Ctrl+R 输入历史搜索 - #102

Merged
Cyberforker merged 1 commit into
Yapie0:mainfrom
Star-Star66:codex/prompt-history-search
Jun 18, 2026
Merged

Cyberforker merged 1 commit into
Yapie0:mainfrom
Star-Star66:codex/prompt-history-search

Conversation

@Star-Star66

Copy link
Copy Markdown
Contributor

变更内容

  • 为交互式输入增加 Ctrl+R 历史搜索。
  • 支持输入过滤、上下选择、连续 Ctrl+R 循环、Enter 回填与 Esc 取消。
  • 增加中英文提示、快捷键提示和聚焦测试。
  • Windows 无符号链接权限时,仅跳过确实依赖该权限的版本测试;支持符号链接的环境仍完整执行。

原因

方向键同时承担聊天滚动与输入历史切换会产生交互冲突。独立的历史搜索入口可以保留终端滚动行为,同时更快找回旧输入。Windows 测试修复则避免开发者模式未开启时出现与产品逻辑无关的 EPERM 失败。

验证

  • npm run verify
  • 339 个测试文件通过
  • 3914 个测试通过
  • 4 个平台互斥或权限相关测试按环境跳过

@Star-Star66
Star-Star66 force-pushed the codex/prompt-history-search branch 2 times, most recently from 512dec2 to 76373b4 Compare June 15, 2026 04:20
@Star-Star66
Star-Star66 force-pushed the codex/prompt-history-search branch from 76373b4 to 7fe6508 Compare June 17, 2026 01:48
@Star-Star66
Star-Star66 marked this pull request as ready for review June 17, 2026 01:49
@Cyberforker
Cyberforker merged commit ecd6a60 into Yapie0:main Jun 18, 2026
4 checks passed
@Cyberforker

Copy link
Copy Markdown
Collaborator

✅ Merged — Ctrl+R 输入历史搜索

新增 PromptHistoryPicker 组件,支持实时过滤、上下选择、Enter 回填与 Esc 取消。

Thanks @Star-Star66 for the clean isolated feature with thorough keyboard interaction tests and comprehensive i18n. The change is now on main at `ecd6a60`.

— Carbon Code

@Cyberforker

Copy link
Copy Markdown
Collaborator

Review Summary

Reviewed & merged by Carbon Code

Tests: npm run verify — 339 files, 3914/3914 passed, 4 skipped
Feat: Ctrl+R 输入历史搜索,支持实时过滤、上下选择、Enter 回填与 Esc 取消

新增 PromptHistoryPicker 组件,集成到 ComposerArea 和 App.tsx 快捷键处理。提示条从 ↑/↓ 改为 ^R。中英文 i18n 完整覆盖。附带 prompt-history-picker 聚焦测试。

Thanks @Star-Star66 for the clean isolated feature with thorough keyboard interaction tests and comprehensive i18n coverage.

Co-Authored-By: Carbon Code carboncode@code.ai6666.com

-- Carbon Code

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.

2 participants