@bamboocss/eslint-plugin 1.44.1 → 1.45.0

package/dist/index.cjs

@@ -8,7 +8,7 @@ require("@typescript-eslint/utils/ts-eslint");
 let _bamboocss_shared = require("@bamboocss/shared");
 //#region package.json
 var name = "@bamboocss/eslint-plugin";
-var version = "1.44.1";
+var version = "1.45.0";
 //#endregion
 //#region src/utils/index.ts
 const createRule = _typescript_eslint_utils.ESLintUtils.RuleCreator((name) => `https://github.com/gajus/bamboocss/blob/main/packages/eslint-plugin/docs/rules/${name}.md`);
@@ -318,8 +318,8 @@ function isRecipeVariant(node, context) {
 }
 //#endregion
 //#region src/rules/file-not-included.ts
-const RULE_NAME$22 = "file-not-included";
-const rule$22 = createRule({
+const RULE_NAME$23 = "file-not-included";
+const rule$23 = createRule({
 	create(context) {
 		if (isValidFile(context)) return {};
 		let hasReported = false;
@@ -340,11 +340,11 @@ const rule$22 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$22
+	name: RULE_NAME$23
 });
 //#endregion
 //#region src/rules/no-config-function-in-source.ts
-const RULE_NAME$21 = "no-config-function-in-source";
+const RULE_NAME$22 = "no-config-function-in-source";
 const CONFIG_FUNCTIONS = new Set([
 	"defineConfig",
 	"defineGlobalStyles",
@@ -359,7 +359,7 @@ const CONFIG_FUNCTIONS = new Set([
 	"defineTokens",
 	"defineUtility"
 ]);
-const rule$21 = createRule({
+const rule$22 = createRule({
 	create(context) {
 		if (!hasPkgImport(context)) return {};
 		if (!isValidFile(context)) return {};
@@ -399,12 +399,12 @@ const rule$21 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$21
+	name: RULE_NAME$22
 });
 //#endregion
 //#region src/rules/no-debug.ts
-const RULE_NAME$20 = "no-debug";
-const rule$20 = createRule({
+const RULE_NAME$21 = "no-debug";
+const rule$21 = createRule({
 	create(context) {
 		const processedNodes = /* @__PURE__ */ new WeakSet();
 		const checkObjectForDebug = (object) => {
@@ -473,12 +473,12 @@ const rule$20 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$20
+	name: RULE_NAME$21
 });
 //#endregion
 //#region src/rules/no-deprecated-tokens.ts
-const RULE_NAME$19 = "no-deprecated-tokens";
-const rule$19 = createRule({
+const RULE_NAME$20 = "no-deprecated-tokens";
+const rule$20 = createRule({
 	create(context) {
 		const deprecatedTokensCache = /* @__PURE__ */ new Map();
 		const sendReport = (property, node, value) => {
@@ -532,11 +532,11 @@ const rule$19 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$19
+	name: RULE_NAME$20
 });
 //#endregion
 //#region src/rules/no-descendant-selectors.ts
-const RULE_NAME$18 = "no-descendant-selectors";
+const RULE_NAME$19 = "no-descendant-selectors";
 /**
 * Split a selector on combinators that are not inside brackets, parentheses or quotes.
 *
@@ -602,7 +602,7 @@ const selectorOf = (node) => {
 	if (isLiteral(node) && typeof node.value === "string") return node.value;
 	if (isTemplateLiteral(node) && node.expressions.length === 0) return node.quasis[0]?.value.raw;
 };
-const rule$18 = createRule({
+const rule$19 = createRule({
 	create(context) {
 		return { "Property[key.type!=/Identifier/][value.type=\"ObjectExpression\"]"(node) {
 			const selector = selectorOf(node.key);
@@ -626,12 +626,12 @@ const rule$18 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$18
+	name: RULE_NAME$19
 });
 //#endregion
 //#region src/rules/no-dynamic-styling.ts
-const RULE_NAME$17 = "no-dynamic-styling";
-const rule$17 = createRule({
+const RULE_NAME$18 = "no-dynamic-styling";
+const rule$18 = createRule({
 	create(context) {
 		function isStaticValue(node) {
 			if (!node) return false;
@@ -702,12 +702,12 @@ const rule$17 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$17
+	name: RULE_NAME$18
 });
 //#endregion
 //#region src/rules/no-escape-hatch.ts
-const RULE_NAME$16 = "no-escape-hatch";
-const rule$16 = createRule({
+const RULE_NAME$17 = "no-escape-hatch";
+const rule$17 = createRule({
 	create(context) {
 		const removeBrackets = (range) => {
 			const [start, end] = range;
@@ -780,12 +780,12 @@ const rule$16 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$16
+	name: RULE_NAME$17
 });
 //#endregion
 //#region src/rules/no-hardcoded-color.ts
-const RULE_NAME$15 = "no-hardcoded-color";
-const rule$15 = createRule({
+const RULE_NAME$16 = "no-hardcoded-color";
+const rule$16 = createRule({
 	create(context) {
 		const noOpacity = context.options[0]?.noOpacity;
 		const whitelist = context.options[0]?.whitelist ?? [];
@@ -881,14 +881,14 @@ const rule$15 = createRule({
 		}],
 		type: "problem"
 	},
-	name: RULE_NAME$15
+	name: RULE_NAME$16
 });
 //#endregion
 //#region src/rules/no-important.ts
 const exclamationRegex = /\s*!$/;
 const importantRegex = /\s*!important\s*$/;
-const RULE_NAME$14 = "no-important";
-const rule$14 = createRule({
+const RULE_NAME$15 = "no-important";
+const rule$15 = createRule({
 	create(context) {
 		const removeQuotes = (range) => {
 			const [start, end] = range;
@@ -992,12 +992,12 @@ const rule$14 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$14
+	name: RULE_NAME$15
 });
 //#endregion
 //#region src/rules/no-invalid-nesting.ts
-const RULE_NAME$13 = "no-invalid-nesting";
-const rule$13 = createRule({
+const RULE_NAME$14 = "no-invalid-nesting";
+const rule$14 = createRule({
 	create(context) {
 		const bambooFunctionCache = /* @__PURE__ */ new WeakMap();
 		const jsxPropertyCache = /* @__PURE__ */ new WeakMap();
@@ -1052,12 +1052,12 @@ const rule$13 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$13
+	name: RULE_NAME$14
 });
 //#endregion
 //#region src/rules/no-invalid-token-paths.ts
-const RULE_NAME$12 = "no-invalid-token-paths";
-const rule$12 = createRule({
+const RULE_NAME$13 = "no-invalid-token-paths";
+const rule$13 = createRule({
 	create(context) {
 		const invalidTokensCache = /* @__PURE__ */ new Map();
 		const sendReport = (node, value) => {
@@ -1157,12 +1157,12 @@ const rule$12 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$12
+	name: RULE_NAME$13
 });
 //#endregion
 //#region src/rules/no-margin-properties.ts
-const RULE_NAME$11 = "no-margin-properties";
-const rule$11 = createRule({
+const RULE_NAME$12 = "no-margin-properties";
+const rule$12 = createRule({
 	create(context) {
 		const whitelist = context.options[0]?.whitelist ?? [];
 		const longhandCache = /* @__PURE__ */ new Map();
@@ -1238,7 +1238,7 @@ const rule$11 = createRule({
 		}],
 		type: "suggestion"
 	},
-	name: RULE_NAME$11
+	name: RULE_NAME$12
 });
 //#endregion
 //#region src/utils/physical-properties.ts
@@ -1282,7 +1282,7 @@ const physicalPropertyValues = { textAlign: {
 } };
 //#endregion
 //#region src/rules/no-physical-properties.ts
-const RULE_NAME$10 = "no-physical-properties";
+const RULE_NAME$11 = "no-physical-properties";
 const MESSAGES = {
 	physical: "Use logical property instead of {{physical}}. Prefer `{{logical}}`.",
 	physicalValue: "Use logical value instead of {{physical}}. Prefer `{{logical}}`.",
@@ -1364,7 +1364,7 @@ const createValueReport = (valueNode, valueText, logical, context) => {
 		}]
 	});
 };
-const rule$10 = createRule({
+const rule$11 = createRule({
 	create(context) {
 		const whitelist = context.options[0]?.whitelist ?? [];
 		const cache = new PropertyCache();
@@ -1420,12 +1420,12 @@ const rule$10 = createRule({
 		}],
 		type: "suggestion"
 	},
-	name: RULE_NAME$10
+	name: RULE_NAME$11
 });
 //#endregion
 //#region src/rules/no-property-renaming.ts
-const RULE_NAME$9 = "no-property-renaming";
-const rule$9 = createRule({
+const RULE_NAME$10 = "no-property-renaming";
+const rule$10 = createRule({
 	create(context) {
 		const bambooPropertyCache = /* @__PURE__ */ new WeakMap();
 		const bambooAttributeCache = /* @__PURE__ */ new WeakMap();
@@ -1489,6 +1489,174 @@ const rule$9 = createRule({
 		schema: [],
 		type: "problem"
 	},
+	name: RULE_NAME$10
+});
+//#endregion
+//#region src/rules/no-redundant-value.ts
+const RULE_NAME$9 = "no-redundant-value";
+/**
+* Properties whose value is 1-4 edges in the order top, right, bottom, left, and which therefore
+* collapse by the same rule: four equal edges are one value, a matching pair is two, a matching
+* left and right is three.
+*
+* An allowlist rather than a shape test, because "several lengths separated by spaces" is not
+* enough to know the collapse is sound. `background-position: 0 0` is left-top and `0` is
+* left-centre — same shape, different meaning. Every entry here is a box-edge property where the
+* CSS specification defines the omitted values as copies of the ones given.
+*
+* Bamboo's own shorthands are listed beside the css names they map to, since a project may write
+* either.
+*/
+const EDGE_PROPERTIES = new Set([
+	"borderColor",
+	"borderStyle",
+	"borderWidth",
+	"inset",
+	"m",
+	"margin",
+	"p",
+	"padding",
+	"scrollMargin",
+	"scrollPadding"
+]);
+/**
+* Properties taking one or two values, where a repeated pair collapses to a single value.
+*
+* `gap: 8px 8px` is row then column, and one value sets both — the two-value case of the same
+* idea, kept separate because the four-edge collapse does not apply.
+*/
+const PAIR_PROPERTIES = new Set([
+	"gap",
+	"gridGap",
+	"overflow",
+	"overscrollBehavior"
+]);
+/** Splits on whitespace, but never inside `var(...)`, `calc(...)` or any other function. */
+const parts = (value) => {
+	const out = [];
+	let depth = 0;
+	let current = "";
+	for (const char of value) {
+		if (char === "(") depth++;
+		if (char === ")") depth--;
+		if (depth === 0 && /\s/.test(char)) {
+			if (current) out.push(current);
+			current = "";
+			continue;
+		}
+		current += char;
+	}
+	if (current) out.push(current);
+	return depth === 0 ? out : void 0;
+};
+/**
+* Zero with a unit, written as plain zero.
+*
+* Every property this rule touches takes lengths, and a zero length is the same zero whatever
+* unit it carries — including `0%`, which resolves against the containing block to the same
+* place. Worth doing before the collapse rather than instead of it: `16px 0` and `16px 0px` are
+* one atom apart on their own, and normalizing first is what lets `0px 16px 0px 16px` reach
+* `0 16px`.
+*/
+const ZERO = /^0(?:px|rem|em|%|vh|vw|vmin|vmax|pt|pc|in|cm|mm|ex|ch)$/i;
+/** The shortest spelling of an edge list that sets the same four edges. */
+const collapseEdges = (list) => {
+	const [top, right = top, bottom = top, left = right] = list;
+	if (top === bottom && right === left) return top === right ? [top] : [top, right];
+	if (right === left) return [
+		top,
+		right,
+		bottom
+	];
+	return [
+		top,
+		right,
+		bottom,
+		left
+	];
+};
+const rule$9 = createRule({
+	create(context) {
+		const whitelist = context.options[0]?.whitelist ?? [];
+		const shortest = (property, value) => {
+			const isEdge = EDGE_PROPERTIES.has(property);
+			if (!isEdge && !PAIR_PROPERTIES.has(property)) return;
+			const raw = parts(value);
+			if (!raw) return;
+			const list = raw.map((part) => ZERO.test(part) ? "0" : part);
+			if (isEdge) {
+				if (list.length > 4) return;
+				const shorter = (list.length > 1 ? collapseEdges(list) : list).join(" ");
+				return shorter === value ? void 0 : shorter;
+			}
+			const shorter = list.length === 2 && list[0] === list[1] ? list[0] : list.join(" ");
+			return shorter === value ? void 0 : shorter;
+		};
+		const check = (node, property, value) => {
+			if (whitelist.includes(property)) return;
+			const replacement = shortest(property, value);
+			if (!replacement || replacement === value) return;
+			const data = {
+				property,
+				replacement,
+				value
+			};
+			context.report({
+				data,
+				fix: (fixer) => {
+					const text = context.sourceCode.getText(node);
+					const quote = text.at(0) === "`" || text.at(0) === "\"" || text.at(0) === "'" ? text.at(0) : "'";
+					return fixer.replaceText(node, `${quote}${replacement}${quote}`);
+				},
+				messageId: "redundant",
+				node
+			});
+		};
+		return {
+			JSXAttribute(node) {
+				if (!isJSXIdentifier(node.name)) return;
+				if (!isBambooProp(node, context) || !node.value) return;
+				const property = node.name.name;
+				const valueNode = node.value;
+				if (isLiteral(valueNode)) {
+					check(valueNode, property, valueNode.value?.toString() ?? "");
+					return;
+				}
+				if (!isJSXExpressionContainer(valueNode)) return;
+				const expression = valueNode.expression;
+				if (isLiteral(expression)) check(expression, property, expression.value?.toString() ?? "");
+				else if (isTemplateLiteral(expression) && expression.expressions.length === 0) check(expression, property, expression.quasis[0].value.raw);
+			},
+			Property(node) {
+				if (!isIdentifier(node.key)) return;
+				if (!isBambooAttribute(node, context)) return;
+				if (isRecipeVariant(node, context)) return;
+				const property = node.key.name;
+				const valueNode = node.value;
+				if (isLiteral(valueNode)) check(valueNode, property, valueNode.value?.toString() ?? "");
+				else if (isTemplateLiteral(valueNode) && valueNode.expressions.length === 0) check(valueNode, property, valueNode.quasis[0].value.raw);
+			}
+		};
+	},
+	defaultOptions: [{ whitelist: [] }],
+	meta: {
+		docs: { description: "Report an edge or pair value written longer than it needs to be, where a shorter spelling sets exactly the same properties." },
+		fixable: "code",
+		messages: { redundant: "`{{value}}` sets the same edges as `{{replacement}}`." },
+		schema: [{
+			additionalProperties: false,
+			properties: { whitelist: {
+				items: {
+					minLength: 0,
+					type: "string"
+				},
+				type: "array",
+				uniqueItems: true
+			} },
+			type: "object"
+		}],
+		type: "suggestion"
+	},
 	name: RULE_NAME$9
 });
 //#endregion
@@ -2625,6 +2793,7 @@ const rule = createRule({
 //#endregion
 //#region src/rules/index.ts
 const rules = {
+	[RULE_NAME$23]: rule$23,
 	[RULE_NAME$22]: rule$22,
 	[RULE_NAME$21]: rule$21,
 	[RULE_NAME$20]: rule$20,
@@ -2652,9 +2821,9 @@ const rules = {
 //#endregion
 //#region src/configs/all.ts
 const errorRules = [
+	RULE_NAME$23,
 	RULE_NAME$22,
-	RULE_NAME$21,
-	RULE_NAME$12
+	RULE_NAME$13
 ];
 //#endregion
 //#region src/index.ts

package/dist/index.mjs

@@ -8,7 +8,7 @@ import "@typescript-eslint/utils/ts-eslint";
 import { getArbitraryValue, parseFallbackValue } from "@bamboocss/shared";
 //#region package.json
 var name = "@bamboocss/eslint-plugin";
-var version = "1.44.1";
+var version = "1.45.0";
 //#endregion
 //#region src/utils/index.ts
 const createRule = ESLintUtils.RuleCreator((name) => `https://github.com/gajus/bamboocss/blob/main/packages/eslint-plugin/docs/rules/${name}.md`);
@@ -318,8 +318,8 @@ function isRecipeVariant(node, context) {
 }
 //#endregion
 //#region src/rules/file-not-included.ts
-const RULE_NAME$22 = "file-not-included";
-const rule$22 = createRule({
+const RULE_NAME$23 = "file-not-included";
+const rule$23 = createRule({
 	create(context) {
 		if (isValidFile(context)) return {};
 		let hasReported = false;
@@ -340,11 +340,11 @@ const rule$22 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$22
+	name: RULE_NAME$23
 });
 //#endregion
 //#region src/rules/no-config-function-in-source.ts
-const RULE_NAME$21 = "no-config-function-in-source";
+const RULE_NAME$22 = "no-config-function-in-source";
 const CONFIG_FUNCTIONS = new Set([
 	"defineConfig",
 	"defineGlobalStyles",
@@ -359,7 +359,7 @@ const CONFIG_FUNCTIONS = new Set([
 	"defineTokens",
 	"defineUtility"
 ]);
-const rule$21 = createRule({
+const rule$22 = createRule({
 	create(context) {
 		if (!hasPkgImport(context)) return {};
 		if (!isValidFile(context)) return {};
@@ -399,12 +399,12 @@ const rule$21 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$21
+	name: RULE_NAME$22
 });
 //#endregion
 //#region src/rules/no-debug.ts
-const RULE_NAME$20 = "no-debug";
-const rule$20 = createRule({
+const RULE_NAME$21 = "no-debug";
+const rule$21 = createRule({
 	create(context) {
 		const processedNodes = /* @__PURE__ */ new WeakSet();
 		const checkObjectForDebug = (object) => {
@@ -473,12 +473,12 @@ const rule$20 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$20
+	name: RULE_NAME$21
 });
 //#endregion
 //#region src/rules/no-deprecated-tokens.ts
-const RULE_NAME$19 = "no-deprecated-tokens";
-const rule$19 = createRule({
+const RULE_NAME$20 = "no-deprecated-tokens";
+const rule$20 = createRule({
 	create(context) {
 		const deprecatedTokensCache = /* @__PURE__ */ new Map();
 		const sendReport = (property, node, value) => {
@@ -532,11 +532,11 @@ const rule$19 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$19
+	name: RULE_NAME$20
 });
 //#endregion
 //#region src/rules/no-descendant-selectors.ts
-const RULE_NAME$18 = "no-descendant-selectors";
+const RULE_NAME$19 = "no-descendant-selectors";
 /**
 * Split a selector on combinators that are not inside brackets, parentheses or quotes.
 *
@@ -602,7 +602,7 @@ const selectorOf = (node) => {
 	if (isLiteral(node) && typeof node.value === "string") return node.value;
 	if (isTemplateLiteral(node) && node.expressions.length === 0) return node.quasis[0]?.value.raw;
 };
-const rule$18 = createRule({
+const rule$19 = createRule({
 	create(context) {
 		return { "Property[key.type!=/Identifier/][value.type=\"ObjectExpression\"]"(node) {
 			const selector = selectorOf(node.key);
@@ -626,12 +626,12 @@ const rule$18 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$18
+	name: RULE_NAME$19
 });
 //#endregion
 //#region src/rules/no-dynamic-styling.ts
-const RULE_NAME$17 = "no-dynamic-styling";
-const rule$17 = createRule({
+const RULE_NAME$18 = "no-dynamic-styling";
+const rule$18 = createRule({
 	create(context) {
 		function isStaticValue(node) {
 			if (!node) return false;
@@ -702,12 +702,12 @@ const rule$17 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$17
+	name: RULE_NAME$18
 });
 //#endregion
 //#region src/rules/no-escape-hatch.ts
-const RULE_NAME$16 = "no-escape-hatch";
-const rule$16 = createRule({
+const RULE_NAME$17 = "no-escape-hatch";
+const rule$17 = createRule({
 	create(context) {
 		const removeBrackets = (range) => {
 			const [start, end] = range;
@@ -780,12 +780,12 @@ const rule$16 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$16
+	name: RULE_NAME$17
 });
 //#endregion
 //#region src/rules/no-hardcoded-color.ts
-const RULE_NAME$15 = "no-hardcoded-color";
-const rule$15 = createRule({
+const RULE_NAME$16 = "no-hardcoded-color";
+const rule$16 = createRule({
 	create(context) {
 		const noOpacity = context.options[0]?.noOpacity;
 		const whitelist = context.options[0]?.whitelist ?? [];
@@ -881,14 +881,14 @@ const rule$15 = createRule({
 		}],
 		type: "problem"
 	},
-	name: RULE_NAME$15
+	name: RULE_NAME$16
 });
 //#endregion
 //#region src/rules/no-important.ts
 const exclamationRegex = /\s*!$/;
 const importantRegex = /\s*!important\s*$/;
-const RULE_NAME$14 = "no-important";
-const rule$14 = createRule({
+const RULE_NAME$15 = "no-important";
+const rule$15 = createRule({
 	create(context) {
 		const removeQuotes = (range) => {
 			const [start, end] = range;
@@ -992,12 +992,12 @@ const rule$14 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$14
+	name: RULE_NAME$15
 });
 //#endregion
 //#region src/rules/no-invalid-nesting.ts
-const RULE_NAME$13 = "no-invalid-nesting";
-const rule$13 = createRule({
+const RULE_NAME$14 = "no-invalid-nesting";
+const rule$14 = createRule({
 	create(context) {
 		const bambooFunctionCache = /* @__PURE__ */ new WeakMap();
 		const jsxPropertyCache = /* @__PURE__ */ new WeakMap();
@@ -1052,12 +1052,12 @@ const rule$13 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$13
+	name: RULE_NAME$14
 });
 //#endregion
 //#region src/rules/no-invalid-token-paths.ts
-const RULE_NAME$12 = "no-invalid-token-paths";
-const rule$12 = createRule({
+const RULE_NAME$13 = "no-invalid-token-paths";
+const rule$13 = createRule({
 	create(context) {
 		const invalidTokensCache = /* @__PURE__ */ new Map();
 		const sendReport = (node, value) => {
@@ -1157,12 +1157,12 @@ const rule$12 = createRule({
 		schema: [],
 		type: "problem"
 	},
-	name: RULE_NAME$12
+	name: RULE_NAME$13
 });
 //#endregion
 //#region src/rules/no-margin-properties.ts
-const RULE_NAME$11 = "no-margin-properties";
-const rule$11 = createRule({
+const RULE_NAME$12 = "no-margin-properties";
+const rule$12 = createRule({
 	create(context) {
 		const whitelist = context.options[0]?.whitelist ?? [];
 		const longhandCache = /* @__PURE__ */ new Map();
@@ -1238,7 +1238,7 @@ const rule$11 = createRule({
 		}],
 		type: "suggestion"
 	},
-	name: RULE_NAME$11
+	name: RULE_NAME$12
 });
 //#endregion
 //#region src/utils/physical-properties.ts
@@ -1282,7 +1282,7 @@ const physicalPropertyValues = { textAlign: {
 } };
 //#endregion
 //#region src/rules/no-physical-properties.ts
-const RULE_NAME$10 = "no-physical-properties";
+const RULE_NAME$11 = "no-physical-properties";
 const MESSAGES = {
 	physical: "Use logical property instead of {{physical}}. Prefer `{{logical}}`.",
 	physicalValue: "Use logical value instead of {{physical}}. Prefer `{{logical}}`.",
@@ -1364,7 +1364,7 @@ const createValueReport = (valueNode, valueText, logical, context) => {
 		}]
 	});
 };
-const rule$10 = createRule({
+const rule$11 = createRule({
 	create(context) {
 		const whitelist = context.options[0]?.whitelist ?? [];
 		const cache = new PropertyCache();
@@ -1420,12 +1420,12 @@ const rule$10 = createRule({
 		}],
 		type: "suggestion"
 	},
-	name: RULE_NAME$10
+	name: RULE_NAME$11
 });
 //#endregion
 //#region src/rules/no-property-renaming.ts
-const RULE_NAME$9 = "no-property-renaming";
-const rule$9 = createRule({
+const RULE_NAME$10 = "no-property-renaming";
+const rule$10 = createRule({
 	create(context) {
 		const bambooPropertyCache = /* @__PURE__ */ new WeakMap();
 		const bambooAttributeCache = /* @__PURE__ */ new WeakMap();
@@ -1489,6 +1489,174 @@ const rule$9 = createRule({
 		schema: [],
 		type: "problem"
 	},
+	name: RULE_NAME$10
+});
+//#endregion
+//#region src/rules/no-redundant-value.ts
+const RULE_NAME$9 = "no-redundant-value";
+/**
+* Properties whose value is 1-4 edges in the order top, right, bottom, left, and which therefore
+* collapse by the same rule: four equal edges are one value, a matching pair is two, a matching
+* left and right is three.
+*
+* An allowlist rather than a shape test, because "several lengths separated by spaces" is not
+* enough to know the collapse is sound. `background-position: 0 0` is left-top and `0` is
+* left-centre — same shape, different meaning. Every entry here is a box-edge property where the
+* CSS specification defines the omitted values as copies of the ones given.
+*
+* Bamboo's own shorthands are listed beside the css names they map to, since a project may write
+* either.
+*/
+const EDGE_PROPERTIES = new Set([
+	"borderColor",
+	"borderStyle",
+	"borderWidth",
+	"inset",
+	"m",
+	"margin",
+	"p",
+	"padding",
+	"scrollMargin",
+	"scrollPadding"
+]);
+/**
+* Properties taking one or two values, where a repeated pair collapses to a single value.
+*
+* `gap: 8px 8px` is row then column, and one value sets both — the two-value case of the same
+* idea, kept separate because the four-edge collapse does not apply.
+*/
+const PAIR_PROPERTIES = new Set([
+	"gap",
+	"gridGap",
+	"overflow",
+	"overscrollBehavior"
+]);
+/** Splits on whitespace, but never inside `var(...)`, `calc(...)` or any other function. */
+const parts = (value) => {
+	const out = [];
+	let depth = 0;
+	let current = "";
+	for (const char of value) {
+		if (char === "(") depth++;
+		if (char === ")") depth--;
+		if (depth === 0 && /\s/.test(char)) {
+			if (current) out.push(current);
+			current = "";
+			continue;
+		}
+		current += char;
+	}
+	if (current) out.push(current);
+	return depth === 0 ? out : void 0;
+};
+/**
+* Zero with a unit, written as plain zero.
+*
+* Every property this rule touches takes lengths, and a zero length is the same zero whatever
+* unit it carries — including `0%`, which resolves against the containing block to the same
+* place. Worth doing before the collapse rather than instead of it: `16px 0` and `16px 0px` are
+* one atom apart on their own, and normalizing first is what lets `0px 16px 0px 16px` reach
+* `0 16px`.
+*/
+const ZERO = /^0(?:px|rem|em|%|vh|vw|vmin|vmax|pt|pc|in|cm|mm|ex|ch)$/i;
+/** The shortest spelling of an edge list that sets the same four edges. */
+const collapseEdges = (list) => {
+	const [top, right = top, bottom = top, left = right] = list;
+	if (top === bottom && right === left) return top === right ? [top] : [top, right];
+	if (right === left) return [
+		top,
+		right,
+		bottom
+	];
+	return [
+		top,
+		right,
+		bottom,
+		left
+	];
+};
+const rule$9 = createRule({
+	create(context) {
+		const whitelist = context.options[0]?.whitelist ?? [];
+		const shortest = (property, value) => {
+			const isEdge = EDGE_PROPERTIES.has(property);
+			if (!isEdge && !PAIR_PROPERTIES.has(property)) return;
+			const raw = parts(value);
+			if (!raw) return;
+			const list = raw.map((part) => ZERO.test(part) ? "0" : part);
+			if (isEdge) {
+				if (list.length > 4) return;
+				const shorter = (list.length > 1 ? collapseEdges(list) : list).join(" ");
+				return shorter === value ? void 0 : shorter;
+			}
+			const shorter = list.length === 2 && list[0] === list[1] ? list[0] : list.join(" ");
+			return shorter === value ? void 0 : shorter;
+		};
+		const check = (node, property, value) => {
+			if (whitelist.includes(property)) return;
+			const replacement = shortest(property, value);
+			if (!replacement || replacement === value) return;
+			const data = {
+				property,
+				replacement,
+				value
+			};
+			context.report({
+				data,
+				fix: (fixer) => {
+					const text = context.sourceCode.getText(node);
+					const quote = text.at(0) === "`" || text.at(0) === "\"" || text.at(0) === "'" ? text.at(0) : "'";
+					return fixer.replaceText(node, `${quote}${replacement}${quote}`);
+				},
+				messageId: "redundant",
+				node
+			});
+		};
+		return {
+			JSXAttribute(node) {
+				if (!isJSXIdentifier(node.name)) return;
+				if (!isBambooProp(node, context) || !node.value) return;
+				const property = node.name.name;
+				const valueNode = node.value;
+				if (isLiteral(valueNode)) {
+					check(valueNode, property, valueNode.value?.toString() ?? "");
+					return;
+				}
+				if (!isJSXExpressionContainer(valueNode)) return;
+				const expression = valueNode.expression;
+				if (isLiteral(expression)) check(expression, property, expression.value?.toString() ?? "");
+				else if (isTemplateLiteral(expression) && expression.expressions.length === 0) check(expression, property, expression.quasis[0].value.raw);
+			},
+			Property(node) {
+				if (!isIdentifier(node.key)) return;
+				if (!isBambooAttribute(node, context)) return;
+				if (isRecipeVariant(node, context)) return;
+				const property = node.key.name;
+				const valueNode = node.value;
+				if (isLiteral(valueNode)) check(valueNode, property, valueNode.value?.toString() ?? "");
+				else if (isTemplateLiteral(valueNode) && valueNode.expressions.length === 0) check(valueNode, property, valueNode.quasis[0].value.raw);
+			}
+		};
+	},
+	defaultOptions: [{ whitelist: [] }],
+	meta: {
+		docs: { description: "Report an edge or pair value written longer than it needs to be, where a shorter spelling sets exactly the same properties." },
+		fixable: "code",
+		messages: { redundant: "`{{value}}` sets the same edges as `{{replacement}}`." },
+		schema: [{
+			additionalProperties: false,
+			properties: { whitelist: {
+				items: {
+					minLength: 0,
+					type: "string"
+				},
+				type: "array",
+				uniqueItems: true
+			} },
+			type: "object"
+		}],
+		type: "suggestion"
+	},
 	name: RULE_NAME$9
 });
 //#endregion
@@ -2625,6 +2793,7 @@ const rule = createRule({
 //#endregion
 //#region src/rules/index.ts
 const rules = {
+	[RULE_NAME$23]: rule$23,
 	[RULE_NAME$22]: rule$22,
 	[RULE_NAME$21]: rule$21,
 	[RULE_NAME$20]: rule$20,
@@ -2652,9 +2821,9 @@ const rules = {
 //#endregion
 //#region src/configs/all.ts
 const errorRules = [
+	RULE_NAME$23,
 	RULE_NAME$22,
-	RULE_NAME$21,
-	RULE_NAME$12
+	RULE_NAME$13
 ];
 //#endregion
 //#region src/index.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bamboocss/eslint-plugin",
-  "version": "1.44.1",
+  "version": "1.45.0",
   "description": "ESLint plugin for Bamboo CSS",
   "homepage": "https://bamboocss.com",
   "license": "MIT",
@@ -36,9 +36,9 @@
     "@typescript-eslint/utils": "^8.21.0",
     "micromatch": "^4.0.8",
     "synckit": "^0.9.0",
-    "@bamboocss/config": "1.44.1",
-    "@bamboocss/generator": "1.44.1",
-    "@bamboocss/shared": "1.44.1"
+    "@bamboocss/config": "1.45.0",
+    "@bamboocss/generator": "1.45.0",
+    "@bamboocss/shared": "1.45.0"
   },
   "devDependencies": {
     "@types/micromatch": "^4.0.10",
@@ -46,10 +46,10 @@
     "eslint": "^9.35.0",
     "multiline-ts": "^4.0.1",
     "typescript": "^5.7.2",
-    "@bamboocss/dev": "1.44.1",
-    "@bamboocss/preset-bamboo": "1.44.1",
-    "@bamboocss/preset-base": "1.44.1",
-    "@bamboocss/types": "1.44.1"
+    "@bamboocss/dev": "1.45.0",
+    "@bamboocss/preset-bamboo": "1.45.0",
+    "@bamboocss/preset-base": "1.45.0",
+    "@bamboocss/types": "1.45.0"
   },
   "peerDependencies": {
     "eslint": "*"