Skip to content

Comments

Reject toPrompt Promise when 'exit' is emitted.#369

Open
zArubaru wants to merge 1 commit intoterkelg:masterfrom
zArubaru:fix/reject-on-exit
Open

Reject toPrompt Promise when 'exit' is emitted.#369
zArubaru wants to merge 1 commit intoterkelg:masterfrom
zArubaru:fix/reject-on-exit

Conversation

@zArubaru
Copy link

@zArubaru zArubaru commented Sep 9, 2022

Resolves #236, #362

This fixes the 'autocomplete' prompt incorrectly firing onSubmit when it is exitted by pressing ESC. Now correctly fires onCancel.

This fixes the 'autocomplete' prompt incorrectly firing onSubmit when it
is exitted by pressing ESC. Now correctly fires onCancel.
@frenchbread
Copy link

Any ideas when this PR will be merged?

@crazyxhz
Copy link

crazyxhz commented Jun 4, 2023

is this project not maintained? such a big bug persist so long without merged

@TMaize
Copy link

TMaize commented Sep 13, 2023

Here is a fork to resolve it,and publish to npm

TakshPSingh added a commit to withgraphite/prompts that referenced this pull request Feb 5, 2024
this is an open issue in the library: terkelg#362

here's a fix for this that someone made a while ago (which has not been merged for some reason): terkelg#369

This PR makes the same fix in our internal fork
TakshPSingh added a commit to withgraphite/prompts that referenced this pull request Feb 5, 2024
this is an open issue in the library: terkelg#362

here's a fix for this that someone made a while ago (which has not been merged for some reason): terkelg#369

This PR makes the same fix in our internal fork
TakshPSingh added a commit to withgraphite/prompts that referenced this pull request Feb 5, 2024
this is an open issue in the library: terkelg#362

here's a fix for this that someone made a while ago (which has not been merged for some reason): terkelg#369

This PR makes the same fix to our fork
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.

Prompt not waited for when running as pipe target

4 participants