Refactor to add function names (#6377)

Giving function names (avoiding anonymous) helps debugging.

This refactoring relies on the ESLint `func-names` rule:
https://eslint.org/docs/latest/rules/func-names

lib/__tests__/fixtures/processor-fenced-blocks.js

@@ -1,6 +1,6 @@
 'use strict';
 
-module.exports = function (options = {}) {
+module.exports = function processorFencedBlocks(options = {}) {
 	const specialMessage = options.specialMessage || 'was processed';
 
 	return {

lib/__tests__/fixtures/processor-triple-question-marks.js

@@ -1,6 +1,6 @@
 'use strict';
 
-module.exports = function () {
+module.exports = function processorTripleQuestionMarks() {
 	let found = false;
 
 	return {

lib/createPartialStylelintResult.js

@@ -8,7 +8,7 @@
  * @param {import('stylelint').CssSyntaxError} [cssSyntaxError]
  * @return {StylelintResult}
  */
-module.exports = function (postcssResult, cssSyntaxError) {
+module.exports = function createPartialStylelintResult(postcssResult, cssSyntaxError) {
 	/** @type {StylelintResult} */
 	let stylelintResult;
 	/** @type {string | undefined} */

lib/formatters/__tests__/prepareFormatterOutput.js

@@ -9,7 +9,7 @@ symbolConversions.set('✔', '√');
 symbolConversions.set('⚠', '‼');
 symbolConversions.set('✖', '×');
 
-module.exports = function (results, formatter, returnValue) {
+module.exports = function prepareFormatterOutput(results, formatter, returnValue) {
 	returnValue = returnValue || {
 		ruleMetadata: {},
 	};

lib/formatters/compactFormatter.js

@@ -3,8 +3,8 @@
 /**
  * @type {import('stylelint').Formatter}
  */
-const formatter = (results) =>
-	results
+module.exports = function compactFormatter(results) {
+	return results
 		.flatMap((result) =>
 			result.warnings.map(
 				(warning) =>
@@ -16,5 +16,4 @@ const formatter = (results) =>
 			),
 		)
 		.join('\n');
-
-module.exports = formatter;
+};

lib/formatters/stringFormatter.js

@@ -247,7 +247,7 @@ function formatter(messages, source, cwd) {
 /**
  * @type {import('stylelint').Formatter}
  */
-module.exports = function (results, returnValue) {
+module.exports = function stringFormatter(results, returnValue) {
 	let output = invalidOptionsFormatter(results);
 
 	output += deprecationsFormatter(results);

lib/formatters/tapFormatter.js

@@ -3,7 +3,7 @@
 /**
  * @type {import('stylelint').Formatter}
  */
-const tapFormatter = (results) => {
+module.exports = function tapFormatter(results) {
 	const lines = [`TAP version 13\n1..${results.length}`];
 
 	for (const [index, result] of results.entries()) {
@@ -37,5 +37,3 @@ const tapFormatter = (results) => {
 
 	return lines.join('\n');
 };
-
-module.exports = tapFormatter;

lib/formatters/unixFormatter.js

@@ -3,7 +3,7 @@
 /**
  * @type {import('stylelint').Formatter}
  */
-const unixFormatter = (results) => {
+module.exports = function unixFormatter(results) {
 	const lines = results.flatMap((result) =>
 		result.warnings.map(
 			(warning) =>
@@ -20,5 +20,3 @@ const unixFormatter = (results) => {
 
 	return output;
 };
-
-module.exports = unixFormatter;

lib/reportDisables.js

@@ -11,7 +11,7 @@
  *
  * @param {StylelintResult[]} results
  */
-module.exports = function (results) {
+module.exports = function reportDisables(results) {
 	for (const result of results) {
 		// File with `CssSyntaxError` don't have `_postcssResult`s.
 		if (!result._postcssResult) {

lib/rules/block-closing-brace-space-after/index.js

@@ -27,7 +27,7 @@ const meta = {
 const rule = (primary) => {
 	const checker = whitespaceChecker('space', primary, messages);
 
-	return function (root, result) {
+	return (root, result) => {
 		const validOptions = validateOptions(result, ruleName, {
 			actual: primary,
 			possible: [

lib/rules/declaration-block-semicolon-newline-before/index.js

@@ -24,7 +24,7 @@ const meta = {
 const rule = (primary) => {
 	const checker = whitespaceChecker('newline', primary, messages);
 
-	return function (root, result) {
+	return (root, result) => {
 		const validOptions = validateOptions(result, ruleName, {
 			actual: primary,
 			possible: ['always', 'always-multi-line', 'never-multi-line'],

lib/rules/declaration-block-semicolon-space-after/index.js

@@ -28,7 +28,7 @@ const meta = {
 const rule = (primary, _secondaryOptions, context) => {
 	const checker = whitespaceChecker('space', primary, messages);
 
-	return function (root, result) {
+	return (root, result) => {
 		const validOptions = validateOptions(result, ruleName, {
 			actual: primary,
 			possible: ['always', 'never', 'always-single-line', 'never-single-line'],

lib/utils/atRuleParamIndex.js

@@ -4,7 +4,7 @@
  * @param {import('postcss').AtRule} atRule
  * @returns {number}
  */
-module.exports = function (atRule) {
+module.exports = function atRuleParamIndex(atRule) {
 	// Initial 1 is for the `@`
 	let index = 1 + atRule.name.length;
 

lib/utils/blurComments.js

@@ -5,6 +5,6 @@
  *
  * @returns {string}
  */
-module.exports = function (source, blurChar = '`') {
+module.exports = function blurComments(source, blurChar = '`') {
 	return source.replace(/\/\*.*\*\//g, blurChar);
 };

lib/utils/blurFunctionArguments.js

@@ -17,7 +17,7 @@ const balancedMatch = require('balanced-match');
  * @param {string} functionName
  * @return {string} - The result string, with the function arguments "blurred"
  */
-module.exports = function (source, functionName, blurChar = '`') {
+module.exports = function blurFunctionArguments(source, functionName, blurChar = '`') {
 	const nameWithParen = `${functionName.toLowerCase()}(`;
 	const lowerCaseSource = source.toLowerCase();
 

lib/utils/blurInterpolation.js

@@ -5,6 +5,6 @@
  *
  * @returns {string}
  */
-module.exports = function (source, blurChar = ' ') {
+module.exports = function blurInterpolation(source, blurChar = ' ') {
 	return source.replace(/[#@{}]+/g, blurChar);
 };

lib/utils/configurationError.js

@@ -7,7 +7,7 @@
  * @param {string} text
  * @returns {ConfigurationError}
  */
-module.exports = function (text) {
+module.exports = function configurationError(text) {
 	const err = /** @type {ConfigurationError} */ (new Error(text));
 
 	err.code = 78;

lib/utils/getDimension.js

@@ -12,7 +12,7 @@ const valueParser = require('postcss-value-parser');
  *
  * @returns {{unit: null, number: null} | valueParser.Dimension}
  */
-module.exports = function (node) {
+module.exports = function getDimension(node) {
 	if (!node || !node.value) {
 		return {
 			unit: null,

lib/utils/getSchemeFromUrl.js

@@ -9,7 +9,7 @@ const { URL } = require('url');
  *
  * @param {string} urlString
  */
-module.exports = function (urlString) {
+module.exports = function getSchemeFromUrl(urlString) {
 	let protocol = null;
 
 	try {

lib/utils/hasEmptyBlock.js

@@ -6,7 +6,7 @@
  * @param {import('postcss').Rule | import('postcss').AtRule} statement - postcss rule or at-rule node
  * @return {boolean} True if the statement has a block and it is empty
  */
-module.exports = function (statement) {
+module.exports = function hasEmptyBlock(statement) {
 	return (
 		statement.nodes !== undefined && statement.nodes.length === 0 // has block
 	); // and is empty

lib/utils/hasEmptyLine.js

@@ -6,7 +6,7 @@
  * @param {string | undefined} string
  * @returns {boolean}
  */
-module.exports = function (string) {
+module.exports = function hasEmptyLine(string) {
 	if (string === '' || string === undefined) return false;
 
 	return /\n[\r\t ]*\n/.test(string);

lib/utils/hasInterpolation.js

@@ -11,7 +11,7 @@ const hasTplInterpolation = require('../utils/hasTplInterpolation');
  * @param {string} string
  * @return {boolean} If `true`, a string has interpolation
  */
-module.exports = function (string) {
+module.exports = function hasInterpolation(string) {
 	// SCSS or Less interpolation
 	if (
 		hasLessInterpolation(string) ||

lib/utils/hasLessInterpolation.js

@@ -6,6 +6,6 @@
  * @param {string} string
  * @return {boolean} If `true`, a string has less interpolation
  */
-module.exports = function (string) {
+module.exports = function hasLessInterpolation(string) {
 	return /@\{.+?\}/.test(string);
 };

lib/utils/hasPsvInterpolation.js

@@ -5,6 +5,6 @@
  *
  * @param {string} string
  */
-module.exports = function (string) {
+module.exports = function hasPsvInterpolation(string) {
 	return /\$\(.+?\)/.test(string);
 };

lib/utils/hasScssInterpolation.js

@@ -5,6 +5,6 @@
  *
  * @param {string} string
  */
-module.exports = function (string) {
+module.exports = function hasScssInterpolation(string) {
 	return /#\{.+?\}/.test(string);
 };

lib/utils/hasTplInterpolation.js

@@ -6,6 +6,6 @@
  * @param {string} string
  * @return {boolean} If `true`, a string has template literal interpolation
  */
-module.exports = function (string) {
+module.exports = function hasTplInterpolation(string) {
 	return /\{.+?\}/.test(string);
 };

lib/utils/isAfterComment.js

@@ -5,7 +5,7 @@ const isSharedLineComment = require('./isSharedLineComment');
 /**
  * @param {import('postcss').Node} node
  */
-module.exports = function (node) {
+module.exports = function isAfterComment(node) {
 	const previousNode = node.prev();
 
 	if (!previousNode || previousNode.type !== 'comment') {

lib/utils/isAfterStandardPropertyDeclaration.js

@@ -8,7 +8,7 @@ const { isDeclaration } = require('./typeGuards');
 /**
  * @param {import('postcss').Node} node
  */
-module.exports = function (node) {
+module.exports = function isAfterStandardPropertyDeclaration(node) {
 	const prevNode = getPreviousNonSharedLineCommentNode(node);
 
 	return (

lib/utils/isBlocklessAtRuleAfterBlocklessAtRule.js

@@ -8,7 +8,7 @@ const { isAtRule } = require('./typeGuards');
  * @param {import('postcss').AtRule} atRule
  * @returns {boolean}
  */
-module.exports = function (atRule) {
+module.exports = function isBlocklessAtRuleAfterBlocklessAtRule(atRule) {
 	if (atRule.type !== 'atrule') {
 		return false;
 	}

lib/utils/isBlocklessAtRuleAfterSameNameBlocklessAtRule.js

@@ -8,7 +8,7 @@ const { isAtRule } = require('./typeGuards');
  * @param {import('postcss').AtRule} atRule
  * @returns {boolean}
  */
-module.exports = function (atRule) {
+module.exports = function isBlocklessAtRuleAfterSameNameBlocklessAtRule(atRule) {
 	if (!isBlocklessAtRuleAfterBlocklessAtRule(atRule)) {
 		return false;
 	}

lib/utils/isCounterIncrementCustomIdentValue.js

@@ -7,7 +7,7 @@ const { counterIncrementKeywords } = require('../reference/keywords');
  *
  * @param {string} value
  */
-module.exports = function (value) {
+module.exports = function isCounterIncrementCustomIdentValue(value) {
 	const valueLowerCase = value.toLowerCase();
 
 	if (

lib/utils/isCounterResetCustomIdentValue.js

@@ -7,7 +7,7 @@ const { counterResetKeywords } = require('../reference/keywords');
  *
  * @param {string} value
  */
-module.exports = function (value) {
+module.exports = function isCounterResetCustomIdentValue(value) {
 	const valueLowerCase = value.toLowerCase();
 
 	if (

lib/utils/isCustomElement.js

@@ -10,7 +10,7 @@ const svgTags = require('svg-tags');
  * @param {string} selector
  * @returns {boolean}
  */
-module.exports = function (selector) {
+module.exports = function isCustomElement(selector) {
 	if (!/^[a-z]/.test(selector)) {
 		return false;
 	}

lib/utils/isCustomFunction.js

@@ -6,6 +6,6 @@
  * @param {string} func
  * @returns {boolean}
  */
-module.exports = function (func) {
+module.exports = function isCustomFunction(func) {
 	return func.startsWith('--');
 };

lib/utils/isCustomMediaQuery.js

@@ -5,6 +5,6 @@
  * @param {string} mediaQuery
  * @returns {boolean}
  */
-module.exports = function (mediaQuery) {
+module.exports = function isCustomMediaQuery(mediaQuery) {
 	return mediaQuery.startsWith('--');
 };

lib/utils/isCustomProperty.js

@@ -5,6 +5,6 @@
  * @param {string} property
  * @returns {boolean}
  */
-module.exports = function (property) {
+module.exports = function isCustomProperty(property) {
 	return property.startsWith('--');
 };

lib/utils/isCustomSelector.js

@@ -6,6 +6,6 @@
  * @param {string} selector
  * @returns {boolean}
  */
-module.exports = function (selector) {
+module.exports = function isCustomSelector(selector) {
 	return selector.startsWith(':--');
 };

lib/utils/isFirstNested.js

@@ -6,7 +6,7 @@ const { isComment, hasSource } = require('./typeGuards');
  * @param {import('postcss').Node} statement
  * @returns {boolean}
  */
-module.exports = function (statement) {
+module.exports = function isFirstNested(statement) {
 	const parentNode = statement.parent;
 
 	if (parentNode === undefined || parentNode.type === 'root') {

lib/utils/isFirstNodeOfRoot.js

@@ -6,7 +6,7 @@ const { isRoot } = require('./typeGuards');
  * @param {import('postcss').Node} node
  * @returns {boolean}
  */
-module.exports = function (node) {
+module.exports = function isFirstNodeOfRoot(node) {
 	if (isRoot(node)) return false;
 
 	const parentNode = node.parent;

lib/utils/isKeyframeRule.js

@@ -8,7 +8,7 @@ const { isAtRule } = require('./typeGuards');
  * @param {import('postcss').Rule} rule
  * @returns {boolean}
  */
-module.exports = function (rule) {
+module.exports = function isKeyframeRule(rule) {
 	const parent = rule.parent;
 
 	if (!parent) {

lib/utils/isKeyframeSelector.js

@@ -8,7 +8,7 @@ const { keyframeSelectorKeywords } = require('../reference/keywords');
  * @param {string} selector
  * @returns {boolean}
  */
-module.exports = function (selector) {
+module.exports = function isKeyframeSelector(selector) {
 	if (keyframeSelectorKeywords.has(selector)) {
 		return true;
 	}

lib/utils/isLessVariable.js

@@ -6,7 +6,7 @@ const hasBlock = require('./hasBlock');
  * @param {import('postcss').AtRule | import('postcss-less').AtRule} atRule
  * @returns {boolean}
  */
-module.exports = function (atRule) {
+module.exports = function isLessVariable(atRule) {
 	return (
 		(atRule.type === 'atrule' && 'variable' in atRule && atRule.variable && !hasBlock(atRule)) ||
 		false