[bug] string_decoder/ is not builtin

[loynoir]

CJS & ESM

CJS and ESM both think string_decoder/ is external

> require('string_decoder/')
Uncaught Error: Cannot find module 'string_decoder/'

> await import('string_decoder/')
Uncaught:
Error [ERR_MODULE_NOT_FOUND]:

Actual

> isBuiltinModule = require('is-builtin-module');
> isBuiltinModule('string_decoder/')
true

Expected

> isBuiltinModule = require('is-builtin-module');
> isBuiltinModule('string_decoder/')
false

Related

> isBuiltinModule('string_decoder')
true
> isBuiltinModule('fs/promises')
true

[sindresorhus]

When sindresorhus/builtin-modules#12 is fixed, we can remove

is-builtin-module/index.js

Lines 16 to 19 in 819869f

	const slashIndex = moduleName.indexOf('/');
	if (slashIndex !== -1) {
	moduleName = moduleName.slice(0, slashIndex);
	}

and this issue will be fixed.

[papb]

sindresorhus/builtin-modules#17