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.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

This pull request has had no activity for 60 days and is being marked as stale. It will be closed in another 30 days if no further activity occurs. To keep it open, push a new commit or leave a comment. Maintainers may apply pinned, status/blocked, status/on-hold, or status/ready-for-merge to exempt it from auto-close.

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 status/stale

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