reject toPrompt promise on exit (eg with ESC key)

TakshPSingh · TakshPSingh · commit 256040109b3f · 2024-02-05T16:02:47.000-05:00
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
diff --git a/lib/prompts.js b/lib/prompts.js
@@ -11,7 +11,7 @@ function toPrompt(type, args, opts={}) {
     const onExit = opts.onExit || noop;
     p.on('state', args.onState || noop);
     p.on('submit', x => res(onSubmit(x)));
-    p.on('exit', x => res(onExit(x)));
+    p.on('exit', x => rej(onExit(x)));
     p.on('abort', x => rej(onAbort(x)));
   });
 }
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@withgraphite/prompts",
-  "version": "0.0.1",
+  "version": "0.0.2",
   "description": "Lightweight, beautiful and user-friendly prompts",
   "license": "MIT",
   "repository": "withgraphite/prompts",

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ function toPrompt(type, args, opts={}) {`
`11`	`11`	`const onExit = opts.onExit \|\| noop;`
`12`	`12`	`p.on('state', args.onState \|\| noop);`
`13`	`13`	`p.on('submit', x => res(onSubmit(x)));`
`14`		`- p.on('exit', x => res(onExit(x)));`
	`14`	`+ p.on('exit', x => rej(onExit(x)));`
`15`	`15`	`p.on('abort', x => rej(onAbort(x)));`
`16`	`16`	`});`
`17`	`17`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@withgraphite/prompts",`
`3`		`- "version": "0.0.1",`
	`3`	`+ "version": "0.0.2",`
`4`	`4`	`"description": "Lightweight, beautiful and user-friendly prompts",`
`5`	`5`	`"license": "MIT",`
`6`	`6`	`"repository": "withgraphite/prompts",`