Why v13-dev requires some files in runtime

[Drylozu]

I think the title says everything but there's an example.
There're some requires inside the send function.

Isn't it bad to require files within functions? Shouldn't it be at the root of the file?

[iShibi]

The code you're referring to was last touched three years. It's not that it was added for v13, it's more of something that hasn't been changed.

[monbrey]

This is intentional - scoping the require avoids creating circular imports.

[kyranet]

It could be moved to the bottom, which also fixes the circular:

discord.js/src/structures/interfaces/TextBasedChannel.js

Lines 428 to 431 in 755c180

	module.exports = TextBasedChannel;
	// Fixes Circular
	const MessageManager = require('../../managers/MessageManager');

Circulars only happen because A requires B, B requires A, and A's exports weren't available, which is why that require is below the export.