tools,lib: enable jsdoc/require-returns-check lint rule

.eslintrc.js

@@ -353,7 +353,6 @@ module.exports = {
     'jsdoc/tag-lines': 'off',
     'jsdoc/require-returns-type': 'off',
     'jsdoc/check-alignment': 'off',
-    'jsdoc/require-returns-check': 'off',
 
     // Custom rules from eslint-plugin-node-core
     'node-core/no-unescaped-regexp-dot': 'error',

lib/assert.js

@@ -129,7 +129,6 @@ function innerFail(obj) {
  * @param {string | Error} [message]
  * @param {string} [operator]
  * @param {Function} [stackStartFn]
- * @returns {never}
  */
 function fail(actual, expected, message, operator, stackStartFn) {
   const argsLen = arguments.length;

lib/events.js

@@ -79,7 +79,7 @@ const kMaxEventTargetListenersWarned =
 /**
  * Creates a new `EventEmitter` instance.
  * @param {{ captureRejections?: boolean; }} [opts]
- * @returns {EventEmitter}
+ * @constructs {EventEmitter}
  */
 function EventEmitter(opts) {
   EventEmitter.init.call(this, opts);

lib/fs.js

@@ -239,7 +239,7 @@ function access(path, mode, callback) {
  * directory specified by `path`.
  * @param {string | Buffer | URL} path
  * @param {number} [mode]
- * @returns {void | never}
+ * @returns {void}
  */
 function accessSync(path, mode) {
   path = getValidatedPath(path);

lib/internal/modules/esm/resolve.js

@@ -104,7 +104,7 @@ function emitTrailingSlashPatternDeprecation(match, pjsonUrl, base) {
  * @param {URL} packageJSONUrl
  * @param {string | URL | undefined} base
  * @param {string} main
- * @returns
+ * @returns {void}
  */
 function emitLegacyIndexDeprecation(url, packageJSONUrl, base, main) {
   const format = defaultGetFormat(url);