Skip to content

fix(cli): treat CRLF as paste in passthrough mode#37

Open
BingqingLyu wants to merge 1 commit into
mainfrom
fork-pr-1456-fix-299-windows-paste-crlf
Open

fix(cli): treat CRLF as paste in passthrough mode#37
BingqingLyu wants to merge 1 commit into
mainfrom
fork-pr-1456-fix-299-windows-paste-crlf

Conversation

@BingqingLyu

@BingqingLyu BingqingLyu commented Apr 27, 2026

Copy link
Copy Markdown
Owner

Fixes QwenLM#299.

Windows paste workaround (stdin passthrough) can receive a standalone CRLF chunk during multiline paste. Treating it as a normal return can trigger submit and break pasting.

Changes:

  • Treat standalone as a paste event in passthrough mode.
  • Add a regression test for the CRLF case.

Tests:

  • npx vitest packages/cli/src/ui/contexts/KeypressContext.test.tsx --run

@BingqingLyu

BingqingLyu commented May 7, 2026

Copy link
Copy Markdown
Owner Author

Conflict Group 1

This PR shares modified functions with 2 other PR(s): #87, #94.

These PRs should be reviewed as a batch — merging one may affect the others.

Function File Also modified by
KeypressProvider KeypressContext.tsx #87, #94
graph LR
    PR37["PR #37"]
    FKeypressProvider_396["KeypressProvider<br>KeypressContext.tsx"]
    PR37 -->|modifies| FKeypressProvider_396
    PR87["PR #87"]
    PR87 -->|modifies| FKeypressProvider_396
    PR94["PR #94"]
    PR94 -->|modifies| FKeypressProvider_396
Loading

Posted by codegraph-ai conflict detection.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflicting-group-1 Conflicting PR group 1 — review as a batch conflicting-pr Shares at least one cross-PR dependency with other PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

the paste is not working properly, I am not able to paste multiline errors. Also new lines should be shift+enter instead of alt+enter

2 participants