Skip to content

fix: initialize readline before capture - #969

Merged
tobiasraabe merged 4 commits into
mainfrom
codex/add-libedit-readline-workaround
Aug 28, 2026
Merged

tobiasraabe merged 4 commits into
mainfrom
codex/add-libedit-readline-workaround

Conversation

@tobiasraabe

Copy link
Copy Markdown
Member

Import readline before FD capture starts so libedit attaches to the original terminal handles. Adds unit coverage for missing readline and initialization order, plus an interactive PDB regression test, and documents the fix in the changelog. Ported from pytest-dev/pytest@b4009b3. Validation: capture and debugging tests, just lint, and just typing pass.

@tobiasraabe
tobiasraabe force-pushed the codex/add-libedit-readline-workaround branch from ee74248 to 9fe1533 Compare August 28, 2026 20:32
@tobiasraabe
tobiasraabe added this pull request to the merge queue Aug 28, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Aug 28, 2026
@tobiasraabe
tobiasraabe enabled auto-merge August 28, 2026 21:05
@tobiasraabe
tobiasraabe added this pull request to the merge queue Aug 28, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Aug 28, 2026
@tobiasraabe
tobiasraabe enabled auto-merge August 28, 2026 21:25
@tobiasraabe
tobiasraabe added this pull request to the merge queue Aug 28, 2026
Merged via the queue into main with commit 2aab7d4 Aug 28, 2026
23 checks passed
@tobiasraabe
tobiasraabe deleted the codex/add-libedit-readline-workaround branch August 28, 2026 21:36
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.

1 participant