@atomicsmash/coding-standards 18.0.0-beta.5 → 18.0.0-beta.7

package/beta/eslint/index.js

@@ -1,171 +0,0 @@
-const OFF = 0;
-const WARN = 1;
-const ERROR = 2;
-
-import js from "@eslint/js";
-import esLintComments from "@eslint-community/eslint-plugin-eslint-comments/configs";
-import { defineConfig } from "eslint/config";
-import importPlugin from "eslint-plugin-import";
-import { default as playwrightPlugin } from "eslint-plugin-playwright";
-import reactPlugin from "eslint-plugin-react";
-import reactHooksPlugin from "eslint-plugin-react-hooks";
-import globals from "globals";
-import tseslint from "typescript-eslint";
-
-export const config = defineConfig([
-	{
-		files: [
-			"**/*.js",
-			"**/*.cjs",
-			"**/*.mjs",
-			"**/*.ts",
-			"**/*.cts",
-			"**/*.mts",
-		],
-		extends: [
-			js.configs.recommended,
-			esLintComments.recommended,
-			tseslint.configs.strictTypeChecked,
-			tseslint.configs.stylisticTypeChecked,
-			importPlugin.flatConfigs.recommended,
-			importPlugin.flatConfigs.typescript,
-		],
-		plugins: {
-			"@typescript-eslint": tseslint.plugin,
-		},
-		languageOptions: {
-			ecmaVersion: 2022,
-			sourceType: "module",
-			globals: {
-				...globals.browser,
-				...globals.node,
-			},
-			parser: tseslint.parser,
-			parserOptions: {
-				projectService: true,
-			},
-		},
-		rules: {
-			"no-case-declarations": [OFF],
-			"import/order": [
-				ERROR,
-				{
-					alphabetize: {
-						order: "asc",
-					},
-					groups: [
-						"type",
-						"builtin",
-						"external",
-						"internal",
-						"parent",
-						["sibling", "index"],
-					],
-					"newlines-between": "ignore",
-					pathGroups: [],
-					pathGroupsExcludedImportTypes: [],
-				},
-			],
-			"prefer-const": [ERROR],
-			"no-var": [ERROR],
-			"import/no-duplicates": WARN,
-			"@eslint-community/eslint-comments/no-unused-disable": [ERROR],
-			"@eslint-community/eslint-comments/require-description": [
-				ERROR,
-				{ ignore: ["eslint-enable"] },
-			],
-			"@typescript-eslint/naming-convention": [
-				ERROR,
-				{
-					selector: "variableLike",
-					format: ["camelCase", "PascalCase", "UPPER_CASE"],
-					leadingUnderscore: "allow",
-				},
-				{ selector: "function", format: ["camelCase", "PascalCase"] },
-				{ selector: "typeLike", format: ["PascalCase"] },
-				{
-					selector: "variable",
-					types: ["boolean"],
-					format: ["PascalCase"],
-					prefix: ["is", "should", "has", "can", "did", "will", "are"],
-					leadingUnderscore: "allow",
-				},
-			],
-			"@typescript-eslint/consistent-type-definitions": [ERROR, "type"],
-			"@typescript-eslint/non-nullable-type-assertion-style": OFF,
-			"@typescript-eslint/no-unused-vars": [
-				ERROR,
-				{
-					args: "all",
-					argsIgnorePattern: "^_",
-					caughtErrors: "all",
-					caughtErrorsIgnorePattern: "^_",
-					destructuredArrayIgnorePattern: "^_",
-					varsIgnorePattern: "^_",
-					ignoreRestSiblings: true,
-				},
-			],
-		},
-		settings: {
-			"import/resolver": {
-				typescript: true,
-				node: true,
-			},
-		},
-	},
-	{
-		files: ["*.cjs"],
-		languageOptions: {
-			sourceType: "commonjs",
-		},
-	},
-	{
-		files: ["*.mjs"],
-		languageOptions: {
-			sourceType: "module",
-		},
-	},
-	{
-		files: ["**/*.test.*"],
-		rules: {
-			"@typescript-eslint/no-unused-vars": [OFF],
-		},
-	},
-	{
-		files: ["**/*.jsx", "**/*.tsx"],
-		extends: [
-			reactPlugin.configs.flat.recommended,
-			reactPlugin.configs.flat["jsx-runtime"],
-			reactHooksPlugin.configs.flat.recommended,
-		],
-		settings: {
-			react: {
-				version: "detect",
-			},
-		},
-		rules: {
-			"react-hooks/exhaustive-deps": [
-				WARN,
-				{
-					additionalHooks: "(useSelect|useSuspenseSelect)",
-				},
-			],
-		},
-	},
-]);
-
-export const playwrightConfig = defineConfig([
-	{
-		extends: [playwrightPlugin.configs["flat/recommended"]],
-		rules: {
-			"playwright/expect-expect": [
-				ERROR,
-				{
-					assertFunctionNames: [
-						"playAudit", // This is the lighthouse playwright test, which contains threshold assertions.
-					],
-				},
-			],
-		},
-	},
-]);

package/cspell/index.cjs

@@ -1,142 +0,0 @@
-const fs = require("fs");
-const path = require("path");
-
-/**
- * Search for `package.json`
- * @param {string} from - search `from` directory.
- * @returns {string|void} - path to package.json
- */
-function findNearestPackageJson(from) {
-	from = path.resolve(from);
-	const parent = path.dirname(from);
-	if (!from || parent === from) {
-		return;
-	}
-
-	const pkg = path.join(from, "package.json");
-	if (fs.existsSync(pkg)) {
-		return pkg;
-	}
-	return findNearestPackageJson(parent);
-}
-
-/**
- * Load the nearest package.json
- * @param {string} cwd
- * @returns
- */
-function loadPackageJson(cwd) {
-	const pkgFile = findNearestPackageJson(cwd);
-	if (!pkgFile) return;
-	return JSON.parse(fs.readFileSync(pkgFile, "utf-8"));
-}
-
-function determinePackageNamesAndMethods(cwd = process.cwd()) {
-	const packageImport = loadPackageJson(cwd) || {};
-	const packageNames = Object.keys(packageImport.dependencies || {}).concat(
-		Object.keys(packageImport.devDependencies || {}),
-	);
-	return { packageNames };
-}
-
-/**
- * Search for `composer.json`
- * @param {string} from - search `from` directory.
- * @returns {string|void} - path to composer.json
- */
-function findNearestComposerJson(from) {
-	from = path.resolve(from);
-	const parent = path.dirname(from);
-	if (!from || parent === from) {
-		return;
-	}
-
-	const composerJsonPath = path.join(from, "composer.json");
-	if (fs.existsSync(composerJsonPath)) {
-		return composerJsonPath;
-	}
-	return findNearestComposerJson(parent);
-}
-
-/**
- * Load the nearest composer.json
- * @param {string} cwd
- * @returns
- */
-function loadComposer(cwd) {
-	const composerJsonFile = findNearestComposerJson(cwd);
-	if (!composerJsonFile) return;
-	return JSON.parse(fs.readFileSync(composerJsonFile, "utf-8"));
-}
-
-function determineComposerPackageNamesAndMethods(cwd = process.cwd()) {
-	const packageImport = loadComposer(cwd) || {};
-	const packageNames = Object.keys(packageImport.require || {})
-		.concat(Object.keys(packageImport["require-dev"] || {}))
-		.reduce((allowedWords, packageName) => {
-			return [...allowedWords, ...packageName.split("/")];
-		}, []);
-	return { packageNames };
-}
-
-/** @type { import("@cspell/cspell-types").CSpellUserSettings } */
-const config = {
-	language: "en-GB",
-	words: [
-		...[
-			...determinePackageNamesAndMethods().packageNames,
-			...determineComposerPackageNamesAndMethods().packageNames,
-			"commitlint",
-			"sass",
-			"Userback",
-			"phpcs",
-			"phpcbf",
-		], // package and language names
-		...["tsbuildinfo", "svgs"], // filenames
-		...["combobox", "spinbutton"], // accessible roles
-		...["popover", "dialog"], // custom element names
-		...["atomicsmash"], // company all lowercase
-		...["unstringified", "formattable", "keyof", "nonnullable", "hidable"], // coding actions
-		...[
-			"alignfull",
-			"alignleft",
-			"alignright",
-			"muplugin",
-			"textdomain",
-			"color", // For in theme.json
-			"gfield", // Gravity Forms field class
-			"phpmailer",
-			"WPMDB", // WP Migrate DB
-			"shortcode",
-			"wp_kses",
-			"sprintf", // For @wordpress/i18n
-		], // WordPress stuff
-	],
-	languageSettings: [
-		{
-			languageId: "*",
-			locale: "en-GB",
-		},
-		{
-			languageId: "css, less, scss, sass",
-			locale: "en-US",
-			dictionaries: ["css"],
-		},
-		{
-			languageId: "javascript,typescript,javascriptreact,typescriptreact",
-			locale: "en-GB",
-			dictionaries: ["typescript"],
-		},
-		{
-			languageId: "php",
-			locale: "en-GB",
-			dictionaries: ["php", "html"],
-		},
-		{
-			languageId: "html,vue-html,javascriptreact,typescriptreact",
-			locale: "en-GB",
-			dictionaries: ["html"],
-		},
-	],
-};
-module.exports = config;

package/cspell/index.js

@@ -1 +0,0 @@
-export { default } from "./index.mjs";

package/cspell/index.mjs

@@ -1,142 +0,0 @@
-import { existsSync, readFileSync } from "fs";
-import { resolve, dirname, join } from "path";
-
-/**
- * Search for `package.json`
- * @param {string} from - search `from` directory.
- * @returns {string|void} - path to package.json
- */
-function findNearestPackageJson(from) {
-	from = resolve(from);
-	const parent = dirname(from);
-	if (!from || parent === from) {
-		return;
-	}
-
-	const pkg = join(from, "package.json");
-	if (existsSync(pkg)) {
-		return pkg;
-	}
-	return findNearestPackageJson(parent);
-}
-
-/**
- * Load the nearest package.json
- * @param {string} cwd
- * @returns
- */
-function loadPackageJson(cwd) {
-	const pkgFile = findNearestPackageJson(cwd);
-	if (!pkgFile) return;
-	return JSON.parse(readFileSync(pkgFile, "utf-8"));
-}
-
-function determinePackageNamesAndMethods(cwd = process.cwd()) {
-	const packageImport = loadPackageJson(cwd) || {};
-	const packageNames = Object.keys(packageImport.dependencies || {}).concat(
-		Object.keys(packageImport.devDependencies || {}),
-	);
-	return { packageNames };
-}
-
-/**
- * Search for `composer.json`
- * @param {string} from - search `from` directory.
- * @returns {string|void} - path to composer.json
- */
-function findNearestComposerJson(from) {
-	from = resolve(from);
-	const parent = dirname(from);
-	if (!from || parent === from) {
-		return;
-	}
-
-	const composerJsonPath = join(from, "composer.json");
-	if (existsSync(composerJsonPath)) {
-		return composerJsonPath;
-	}
-	return findNearestComposerJson(parent);
-}
-
-/**
- * Load the nearest composer.json
- * @param {string} cwd
- * @returns
- */
-function loadComposer(cwd) {
-	const composerJsonFile = findNearestComposerJson(cwd);
-	if (!composerJsonFile) return;
-	return JSON.parse(readFileSync(composerJsonFile, "utf-8"));
-}
-
-function determineComposerPackageNamesAndMethods(cwd = process.cwd()) {
-	const packageImport = loadComposer(cwd) || {};
-	const packageNames = Object.keys(packageImport.require || {})
-		.concat(Object.keys(packageImport["require-dev"] || {}))
-		.reduce((allowedWords, packageName) => {
-			return [...allowedWords, ...packageName.split("/")];
-		}, []);
-	return { packageNames };
-}
-
-/** @type { import("@cspell/cspell-types").CSpellUserSettings } */
-const config = {
-	language: "en-GB",
-	words: [
-		...[
-			...determinePackageNamesAndMethods().packageNames,
-			...determineComposerPackageNamesAndMethods().packageNames,
-			"commitlint",
-			"sass",
-			"Userback",
-			"phpcs",
-			"phpcbf",
-		], // package and language names
-		...["tsbuildinfo", "svgs"], // filenames
-		...["combobox", "spinbutton"], // accessible roles
-		...["popover", "dialog"], // custom element names
-		...["atomicsmash"], // company all lowercase
-		...["unstringified", "formattable", "keyof", "nonnullable", "hidable"], // coding actions
-		...[
-			"alignfull",
-			"alignleft",
-			"alignright",
-			"muplugin",
-			"textdomain",
-			"color", // For in theme.json
-			"gfield", // Gravity Forms field class
-			"phpmailer",
-			"WPMDB", // WP Migrate DB
-			"shortcode",
-			"wp_kses",
-			"sprintf", // For @wordpress/i18n
-		], // WordPress stuff
-	],
-	languageSettings: [
-		{
-			languageId: "*",
-			locale: "en-GB",
-		},
-		{
-			languageId: "css, less, scss, sass",
-			locale: "en-US",
-			dictionaries: ["css"],
-		},
-		{
-			languageId: "javascript,typescript,javascriptreact,typescriptreact",
-			locale: "en-GB",
-			dictionaries: ["typescript"],
-		},
-		{
-			languageId: "php",
-			locale: "en-GB",
-			dictionaries: ["php", "html"],
-		},
-		{
-			languageId: "html,vue-html,javascriptreact,typescriptreact",
-			locale: "en-GB",
-			dictionaries: ["html"],
-		},
-	],
-};
-export default config;

package/index.d.ts

@@ -1,7 +0,0 @@
-export const cspellConfig: import("@cspell/cspell-types").CSpellUserSettings;
-export const prettierConfig: import("prettier").Config;
-export const stylelintConfig: {
-	classic: import("stylelint").Config;
-	themeJson: import("stylelint").Config;
-	tailwind: import("stylelint").Config;
-};

package/index.js

@@ -1,5 +0,0 @@
-module.exports = {
-	cspellConfig: require("./cspell/index.cjs"),
-	prettierConfig: require("./prettier/index.cjs"),
-	stylelintConfig: require("./stylelint/index.cjs"),
-};

package/index.mjs

@@ -1,4 +0,0 @@
-import { default as cspellConfig } from "./cspell/index.mjs";
-import { default as prettierConfig } from "./prettier/index.mjs";
-import { default as stylelintConfig } from "./stylelint/index.mjs";
-export { cspellConfig, prettierConfig, stylelintConfig };

package/prettier/index.cjs

@@ -1,35 +0,0 @@
-module.exports = {
-	/* Expected defaults:
-	Prettier recommendation
-	printWidth: 80,
-
-	Single quotes are more often found in text strings (used for apostrophes) so using double quotes prevents lots of escaping characters.
-	singleQuote: false,
-
-	Keep very nested indentation near the left to prevent lots of horizontal scrolling.
-	tabWidth: 2,
-
-	Semicolons prevent AST errors and are more explicit where a line ends
-	semi: true,
-	
-	Consistent line endings for MacOS and Windows
-	endOfLine: "lf",
-	*/
-
-	// Tabs are better for visually impaired coders https://www.reddit.com/r/javascript/comments/c8drjo/nobody_talks_about_the_real_reason_to_use_tabs/
-	useTabs: true,
-
-	// Helps clean up git diffs by removing changes of a single comma on a line if adding a new line to the bottom
-	trailingComma: "all",
-
-	overrides: [
-		{
-			// yml files require specific indentation rules
-			files: ["*.yml", "*.yaml"],
-			options: {
-				useTabs: false,
-				tabWidth: 2,
-			},
-		},
-	],
-};

package/prettier/index.js

@@ -1 +0,0 @@
-export { default } from "./index.mjs";

package/prettier/index.mjs

@@ -1,35 +0,0 @@
-export default {
-	/* Expected defaults:
-	Prettier recommendation
-	printWidth: 80,
-
-	Single quotes are more often found in text strings (used for apostrophes) so using double quotes prevents lots of escaping characters.
-	singleQuote: false,
-
-	Keep very nested indentation near the left to prevent lots of horizontal scrolling.
-	tabWidth: 2,
-
-	Semicolons prevent AST errors and are more explicit where a line ends
-	semi: true,
-	
-	Consistent line endings for MacOS and Windows
-	endOfLine: "lf",
-	*/
-
-	// Tabs are better for visually impaired coders https://www.reddit.com/r/javascript/comments/c8drjo/nobody_talks_about_the_real_reason_to_use_tabs/
-	useTabs: true,
-
-	// Helps clean up git diffs by removing changes of a single comma on a line if adding a new line to the bottom
-	trailingComma: "all",
-
-	overrides: [
-		{
-			// yml files require specific indentation rules
-			files: ["*.yml", "*.yaml"],
-			options: {
-				useTabs: false,
-				tabWidth: 2,
-			},
-		},
-	],
-};

package/stylelint/base.cjs

@@ -1,91 +0,0 @@
-const commonRules = {
-	"custom-property-pattern": null,
-	"function-url-quotes": ["always"],
-	"rule-empty-line-before": [
-		"always",
-		{
-			except: ["first-nested"],
-			ignore: ["after-comment"],
-		},
-	],
-	"comment-empty-line-before": [
-		"always",
-		{
-			except: ["first-nested"],
-			ignore: ["stylelint-commands"],
-		},
-	],
-	"order/order": [
-		{
-			type: "at-rule",
-			hasBlock: false,
-		},
-		"custom-properties",
-		"dollar-variables",
-		"declarations",
-		{
-			type: "at-rule",
-			hasBlock: true,
-		},
-		"rules",
-	],
-	"plugin/no-restricted-syntax": [
-		[
-			{
-				selector:
-					"rule :matches(atrule[name=media], atrule[name=container]) rule",
-				message:
-					"Don't nest rules inside of media/container blocks. Instead, nest the rule and use the at rule within that.",
-			},
-		],
-	],
-	"selector-class-pattern": [
-		"^([a-z][a-z0-9]*)([_-]+[a-z0-9]+)*$",
-		{
-			message:
-				"Selector should use lowercase and separate words with hyphens or underscores (selector-class-pattern)",
-		},
-	],
-	"at-rule-empty-line-before": [
-		"always",
-		{
-			ignore: ["blockless-after-blockless", "first-nested", "after-comment"],
-		},
-	],
-};
-
-/** @type {import('stylelint').Config} */
-module.exports = {
-	reportDescriptionlessDisables: true,
-	reportInvalidScopeDisables: true,
-	reportNeedlessDisables: true,
-	plugins: ["stylelint-order", "stylelint-no-restricted-syntax"],
-	rules: commonRules,
-	overrides: [
-		{
-			files: ["*.css", "**/*.css"],
-			extends: ["stylelint-config-standard", "@wordpress/stylelint-config"],
-			rules: {
-				"at-rule-no-unknown": [true, { ignoreAtRules: "context" }],
-			},
-		},
-		{
-			files: ["*.scss", "**/*.scss"],
-			extends: [
-				"stylelint-config-standard-scss",
-				"@wordpress/stylelint-config/scss",
-			],
-			rules: {
-				"scss/operator-no-newline-after": null,
-				"no-descending-specificity": null,
-				"scss/at-rule-no-unknown": [true, { ignoreAtRules: "context" }],
-			},
-		},
-		{
-			files: ["**/editor-style?(s).?(s)css"],
-			rules: {
-				"declaration-property-value-allowed-list": null,
-			},
-		},
-	],
-};

package/stylelint/base.js

@@ -1 +0,0 @@
-export { default } from "./base.mjs";