Skip to content

Route pastes into the active history search query - #45262

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/2ec13fdddcb7503a72a21fc03b2a27ed36313a8a
Sep 13, 2026
Merged

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/2ec13fdddcb7503a72a21fc03b2a27ed36313a8a

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 13, 2026 •

Copy link
Copy Markdown

Route pastes into the active history search query

Why

Pasting during Ctrl+R history search previously went through normal composer paste handling instead of updating the search query.

What changed

  • Append sanitized pasted text to the active query and restart matching from the newest history entry, including for large pastes and image paths.
  • Ignore empty pastes so they preserve the selected match.
  • Display newlines and tabs as ↵ and ⇥ in the footer while matching the original query, and clamp the cursor safely for very large queries.

Testing

Add regression tests and snapshots covering pasted query acceptance, empty pastes, draft restoration on misses and cancellation, sanitization, large pastes, image paths, separator rendering, and cursor placement.

## Why

Pasting during `Ctrl+R` history search previously went through normal composer paste handling instead of updating the search query.

## What changed

- Append sanitized pasted text to the active query and restart matching from the newest history entry, including for large pastes and image paths.
- Ignore empty pastes so they preserve the selected match.
- Display newlines and tabs as `↵` and `⇥` in the footer while matching the original query, and clamp the cursor safely for very large queries.

## Testing

Add regression tests and snapshots covering pasted query acceptance, empty pastes, draft restoration on misses and cancellation, sanitization, large pastes, image paths, separator rendering, and cursor placement.

GitOrigin-RevId: 2ec13fdddcb7503a72a21fc03b2a27ed36313a8a
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/2ec13fdddcb7503a72a21fc03b2a27ed36313a8a branch from 3928a64 to a505c71 Compare September 13, 2026 17:16
@copyberry
copyberry Bot merged commit a505c71 into main Sep 13, 2026
1 check passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/2ec13fdddcb7503a72a21fc03b2a27ed36313a8a branch September 13, 2026 17:16
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 13, 2026
@neomij9
neomij9 deployed to issue-triage September 13, 2026 17:57 — with GitHub Actions Active
@neomij9
neomij9 deployed to issue-triage September 13, 2026 17:57 — with GitHub Actions Active
@neomij9
neomij9 deployed to issue-triage September 13, 2026 17:57 — with GitHub Actions Active
@neomij9
neomij9 deployed to issue-triage September 13, 2026 17:58 — with GitHub Actions Active
@pholgy
pholgy deployed to issue-triage September 13, 2026 17:59 — with GitHub Actions Active
@pholgy
pholgy deployed to issue-triage September 13, 2026 17:59 — with GitHub Actions Active
@pholgy
pholgy deployed to issue-triage September 13, 2026 17:59 — with GitHub Actions Active
@pholgy
pholgy deployed to issue-triage September 13, 2026 17:59 — with GitHub Actions Active
@pholgy
pholgy deployed to issue-triage September 13, 2026 17:59 — with GitHub Actions Active
@pholgy
pholgy deployed to issue-triage September 13, 2026 17:59 — with GitHub Actions Active
@pholgy
pholgy deployed to issue-triage September 13, 2026 18:00 — with GitHub Actions Active
@pholgy
pholgy deployed to issue-triage September 13, 2026 18:00 — with GitHub Actions Active
@ChristyZ
ChristyZ deployed to issue-triage September 13, 2026 18:12 — with GitHub Actions Active
@ChristyZ
ChristyZ deployed to issue-triage September 13, 2026 18:12 — with GitHub Actions Active
@ChristyZ
ChristyZ deployed to issue-triage September 13, 2026 18:12 — with GitHub Actions Active

This branch was successfully deployed

1 active deployment
issue-triage — a505c714 Deployed Sep 13, 2026 by ChristyZ via Generate label suggestions #45258
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants