Fix declaration-block-no-redundant-longhand-properties with basic keywords

.changeset/lovely-ligers-sip.md

@@ -0,0 +1,5 @@
+---
+"stylelint": patch
+---
+
+Fixed: `declaration-block-no-redundant-longhand-properties` with basic keywords

lib/rules/declaration-block-no-redundant-longhand-properties/README.md

@@ -22,7 +22,7 @@ a {
 }
 ```
 
-This rule will only complain if you've used the longhand equivalent of _all_ the properties that the shorthand will set and if their values are not `inherit`.
+This rule will only complain if you've used the longhand equivalent of _all_ the properties that the shorthand will set and if their values are not [CSS-wide keywords](https://www.w3.org/TR/css-values/#common-keywords) like `initial`, `inherit` etc.
 
 This rule complains when the following shorthand properties can be used:
 

lib/rules/declaration-block-no-redundant-longhand-properties/__tests__/index.js

@@ -46,6 +46,14 @@ testRule({
 		{
 			code: 'a { top: 0; right: 0; bottom: 0; }',
 		},
+		{
+			code: 'a { top: 0; right: 0; bottom: initial; left: 0; }',
+			description: 'contains basic keyword (initial)',
+		},
+		{
+			code: 'a { margin-top: 0; margin-right: 0; margin-bottom: unset; margin-left: 0; }',
+			description: 'contains basic keyword (unset)',
+		},
 	],
 
 	reject: [

lib/rules/declaration-block-no-redundant-longhand-properties/index.js

@@ -1,6 +1,7 @@
 'use strict';
 
 const arrayEqual = require('../../utils/arrayEqual');
+const { basicKeywords } = require('../../reference/keywords');
 const eachDeclarationBlock = require('../../utils/eachDeclarationBlock');
 const optionsMatches = require('../../utils/optionsMatches');
 const report = require('../../utils/report');
@@ -21,8 +22,6 @@ const meta = {
 	fixable: true,
 };
 
-const IGNORED_VALUES = new Set(['inherit']);
-
 /** @typedef {import('postcss').Declaration} Declaration */
 
 /** @type {import('stylelint').Rule} */
@@ -68,7 +67,8 @@ const rule = (primary, secondaryOptions, context) => {
 			const longhandDeclarationNodes = new Map();
 
 			eachDecl((decl) => {
-				if (IGNORED_VALUES.has(decl.value)) {
+				// basic keywords are not allowed in shorthand properties
+				if (basicKeywords.has(decl.value)) {
 					return;
 				}