Major: discordjs/rest crashing on Node v16.9.1 about discord.js HOT 14 CLOSED

I removed node_modules and ran npm install targetting Discord.JS v14.12.1 (Also upgraded to Node v16.13.0). The issue still occured when the project was ran, however this may have been caused by NPM caching packages - I didn't think of that at the time but thats likely what it was. I have reverted to v14.11 for now and will wait for further updates.

from discord.js.

Updated to Node v18 as specified in the rest 2.0.0 milestone changelog.

from discord.js.

16.11 is fine too, and shouldn't be too problematic for now.

from discord.js.

Reopened because this affects Discord.JS directly itself when updating from 14.11.* to 14.12.1.

from discord.js.

16.11 is fine too, and shouldn't be too problematic for now.

Ah cheers, I'll downgrade to 16.11. Might wanna update docs if possible, thanks for the quick reply.

from discord.js.

FYI, the Node 18 requirement was reverted as to avoid a major bump in discord.js, and as crawl said, this feature was added in Node 16.11 which I think is a reasonable enough expectation to update to when updating dependencies. Will work on a solution to either fix the way tsc compiles this or document it.

from discord.js.

FYI, the Node 18 requirement was reverted as to avoid a major bump in discord.js, and as crawl said, this feature was added in Node 16.11 which I think is a reasonable enough expectation to update to when updating dependencies. Will work on a solution to either fix the way tsc compiles this or document it.

In this case, docs should be updated to reflect this, right?

from discord.js.

The 2.0.0 release says 18+ is required because of the use of global fetch so a bit confused, is it 16.11+ or 18+?

from discord.js.

I'm not too sure but this workaround/polyfill should work for 16.11. So until v15, it should be 16.11+.

https://github.com/discordjs/discord.js/blob/main/packages/rest/src/index.ts#L7

from discord.js.

That breaking change comment was left in because that commit had 4 other breaking changes, the revert of the engine requirement bump happened later and the changelog generator did not pick it up for some reason.

Also, the reason it was reverted in the first place is that the bump was not actually necessary since we don't use global fetch in node, only in web / web-like environments.

from discord.js.

Just downgraded to 16.11 and the issue is still occuring?

from discord.js.

To be clear, the same issue is occurring?

The parsing of this syntax was added in v8 9.4.146, which Node updated to in 16.11.0. I also tested this with a minimum reproducible sample in 16.10 and 16.11 to confirm this, and the sample ran fine on 16.11 as expected.

from discord.js.

I made a fresh package, installed discord.js and created a file that only imported discord.js, it crashed on 16.10 as expected, and ran fine on 16.11 in both cjs and esm.

If you are using some type of process manager, make sure it is updating when you update your global version of node, especially if you have multiple versions of node installed at the same time.

from discord.js.

Inactive, issue resolved with Node v16.11+. Docs may need to be updated to include the version bump (assuming they are not already). Thanks for the help. 👍

from discord.js.