REPL broken for async function about node-chakracore HOT 10 CLOSED

@aruneshchandra What's expected behavior?
@boingoing Do you happen to know about this?
Thanks!

from node-chakracore.

Async functions return a Promise when called. Do you get the same error with something like Promise.resolve() ?

from node-chakracore.

@targos - Yes

from node-chakracore.

I'm not familiar with REPL but looks like util.js is using the mirror debugger to inspect the promise instance. It may be probing for internal promise state. I don't have experience with it. Looks like the API is Debug.MakeMirror. Perhaps this is not supported yet.

@kunalspathak Do you have context on this?

from node-chakracore.

Yes, MakeMirror is a v8 specific debugging API that node uses to inspect the properties of an object and Chakracore doesn't have the implementation of it yet.

Sent from Outlook Mobilehttps://aka.ms/blhgte

On Mon, Mar 21, 2016 at 3:18 PM -0700, "Taylor Woll" <[email protected]mailto:[email protected]> wrote:

I'm not familiar with REPL but looks like util.js is using the mirror debugger to inspect the promise instance. It may be probing for internal promise state. I don't have experience with it. Looks like the API is Debug.MakeMirror. Perhaps this is not supported yet.

@kunalspathakhttps://github.com/kunalspathak Do you have context on this?

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHubhttps://github.com//issues/41#issuecomment-199513901

from node-chakracore.

Thanks for these info! I prototyped a fake Debug.MakeMirror support to make this look nicer:

> async function foo() { }
undefined
> foo()
Promise { undefined }
> foo().then(function() { console.log("done") })
Promise { undefined }
> done

>

Currently I can only get "Promise { undefined }" disregarding real status (pending, resolved, ...). If we have APIs to inspect Promise internal states, we could improve this. Do we have any way to inspect? @boingoing

@aruneshchandra Would you like this to get in?

from node-chakracore.

@jianchun The engine tracks internal promise states but they aren't exposed via any Jsrt API. It shouldn't be too hard to add one. The fields you're interested in are JavascriptPromise::status and JavascriptPromise::result, I think.

from node-chakracore.

Having the real fix backed by the new JsRT api will be ideal, however I think we should take the current fix. Also it would be nice to show Promise { unknown } which seems more accurate in this case.

from node-chakracore.

@aruneshchandra Changed to

> async function foo() {}
undefined
> foo()
Promise { '<unknown>' }
> foo().then(()=>console.log('done'))
Promise { '<unknown>' }
> done

>

@agarwal-sandeep @liminzhu How will we inspect a Promise object from a debugger? Specifically need to access its "status" and "result" as @boingoing said.

from node-chakracore.

Sweet - thanx @jianchun !

from node-chakracore.