Return ES2015 Proxy object

[schnittstabil]

ECMAScript6 Proxies are currently not available on node, therefore this issue is a reminder for future versions.

pify(obj) should return an ECMAScript6 Proxy object, forwarding almost all operations to obj, and promisifying only some functions, thus we almost surely get rid of most non-enumerable and inheritance issues.

---

IssueHunt Summary

frangio has been rewarded.

Backers (Total: $70.00)

• issuehunt ($70.00)

Submitted pull Requests

• #73 Implement using Proxy

---

Tips

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on IssueHunt to raise funds.

---

IssueHunt has been backed by the following sponsors. Become a sponsor

[sindresorhus]

@schnittstabil Interested in exploring this?

I'm requiring Node.js 4 in the next version, so we can safely use Proxy then.

[schnittstabil]

At least v4.5.0 (LTS) and v5.12.0 don't support Proxies 😢

[sindresorhus]

Oh no. I was sure Proxy was in Node.js 4 for some reason...

[schnittstabil]

@sindresorhus If you'd like to give me access rights, I'll create a proxy-branch and push some v6 code for further usage…

[sindresorhus]

Done :)

[sindresorhus]

Yes, good idea. Can at least explore it now. There might be some performance issues using Proxies.

[schnittstabil]

There might be some performance issues using Proxies.

I've experienced the same issue - solved it by introducing a cache: https://github.com/sindresorhus/pify/blob/proxy/index.js#L44

[sindresorhus]

@schnittstabil Nice! Already looks very good. I commented some quick feedback on the commit, but haven't tried it out yet. When that's resolved, can you open a PR? Would make it easier to review.

[fregante]

Yes but why? The code doesn't seem much/any shorter/simpler/faster than the regular version.

[sindresorhus]

@bfred-it Some things will just work compared to now, like ES6 classes, this binding, etc.

[sindresorhus]

Anyone wants to step in and finish #32? Including handle my review comments. I'll put a bounty on this shortly. I think it's better now to just have the proxy interface instead of both. Proxy is pretty optimized in newer Node.js versions. This task includes adding tests to handle all the things mentioned in the open issues too.

[IssueHuntBot]

@IssueHunt has funded $70.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

[SamVerschueren]

I'll take it.

[issuehunt-oss]

@sindresorhus has rewarded $63.00 to @frangio. See it on IssueHunt

• 💰 Total deposit: $70.00
• 🎉 Repository reward(0%): $0.00
• 🔧 Service fee(10%): $7.00