tools: add support for mjs and cjs JS snippet linting

Refs: #37162 Refs: nodejs/remark-preset-lint-node#176 PR-URL: #37311 Refs: eslint/markdown#172 Reviewed-By: Rich Trott <rtrott@gmail.com>

Author: aduh95

.eslintrc.js

@@ -53,11 +53,6 @@ module.exports = {
  overrides: [
  {
  files: [
- 'doc/api/esm.md/*.js',
- 'doc/api/module.md/*.js',
- 'doc/api/modules.md/*.js',
- 'doc/api/packages.md/*.js',
- 'doc/api/wasi.md/*.js',
  'test/es-module/test-esm-type-flag.js',
  'test/es-module/test-esm-type-flag-alias.js',
  '*.mjs',
@@ -70,10 +65,21 @@ module.exports = {
  processor: 'markdown/markdown',
  },
  {
- files: ['**/*.md/*.js'],
- parserOptions: { ecmaFeatures: { impliedStrict: true } },
+ files: ['**/*.md/*.cjs', '**/*.md/*.js'],
+ parserOptions: {
+ sourceType: 'script',
+ ecmaFeatures: { impliedStrict: true }
+ },
  rules: { strict: 'off' },
  },
+ {
+ files: [
+ '**/*.md/*.mjs',
+ 'doc/api/esm.md/*.js',
+ 'doc/api/packages.md/*.js',
+ ],
+ parserOptions: { sourceType: 'module' },
+ },
  ],
  rules: {
  // ESLint built-in rules

Makefile

@@ -1233,6 +1233,7 @@ lint-js-fix:
 .PHONY: lint-js-doc
 # Note that on the CI `lint-js-ci` is run instead.
 # Lints the JavaScript code with eslint.
+lint-js-doc: LINT_JS_TARGETS=doc
 lint-js lint-js-doc:
 @if [ "$(shell $(node_use_openssl))" != "true" ]; then \
 echo "Skipping $@ (no crypto)"; \

doc/api/buffer.md

@@ -2492,7 +2492,6 @@ buf.writeFloatBE(0xcafebabe, 0);
 
 console.log(buf);
 // Prints: <Buffer 4f 4a fe bb>
-
 ```
 
 ### `buf.writeFloatLE(value[, offset])`
@@ -2701,7 +2700,6 @@ buf.writeIntBE(0x1234567890ab, 0, 6);
 
 console.log(buf);
 // Prints: <Buffer 12 34 56 78 90 ab>
-
 ```
 
 ### `buf.writeIntLE(value, offset, byteLength)`

doc/api/module.md

@@ -29,13 +29,13 @@ if a module is maintained by a third party or not.
 `module` in this context isn't the same object that's provided
 by the [module wrapper][]. To access it, require the `Module` module:
 
-```js
+```mjs
 // module.mjs
 // In an ECMAScript module
 import { builtinModules as builtin } from 'module';
 ```
 
-```js
+```cjs
 // module.cjs
 // In a CommonJS module
 const builtin = require('module').builtinModules;
@@ -51,7 +51,7 @@ added: v12.2.0
  string.
 * Returns: {require} Require function
 
-```js
+```mjs
 import { createRequire } from 'module';
 const require = createRequire(import.meta.url);
 
@@ -134,13 +134,13 @@ To enable source map parsing, Node.js must be run with the flag
 [`--enable-source-maps`][], or with code coverage enabled by setting
 [`NODE_V8_COVERAGE=dir`][].
 
-```js
+```mjs
 // module.mjs
 // In an ECMAScript module
 import { findSourceMap, SourceMap } from 'module';
 ```
 
-```js
+```cjs
 // module.cjs
 // In a CommonJS module
 const { findSourceMap, SourceMap } = require('module');

doc/api/wasi.md

@@ -10,7 +10,7 @@ The WASI API provides an implementation of the [WebAssembly System Interface][]
 specification. WASI gives sandboxed WebAssembly applications access to the
 underlying operating system via a collection of POSIX-like functions.
 
-```js
+```mjs
 import fs from 'fs';
 import { WASI } from 'wasi';
 

doc/guides/writing-tests.md

@@ -221,7 +221,6 @@ const server = http.createServer(common.mustCall((req, res) => {
  server.close();
  }));
 }));
-
 ```
 
 **Note:** Many functions invoke their callback with an `err` value as the first