@markuplint/rules 4.9.3 → 4.10.0

package/CHANGELOG.md

@@ -3,17 +3,37 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
-## [4.9.3](https://github.com/markuplint/markuplint/compare/@markuplint/rules@4.9.2...@markuplint/rules@4.9.3) (2024-06-25)
+# [4.10.0](https://github.com/markuplint/markuplint/compare/@markuplint/rules@4.9.4...@markuplint/rules@4.10.0) (2024-09-23)
 
 
 ### Bug Fixes
 
-* **ml-core:** `localName` returns lowercase when using case-sensitive parser for tag names ([b1acadd](https://github.com/markuplint/markuplint/commit/b1acaddfd6bf939ee809f6419ce85a701033ca4f))
+* **rules:** excluded cases where the element is palpable from the `no-empty-palpable-content` rule ([6071c51](https://github.com/markuplint/markuplint/commit/6071c5133b7c5d52d8e052ac9f39fb5b10c38b8e))
+* **rules:** fix the message of the `heading-levels` rule and add to translate in Japanese ([ec57e1e](https://github.com/markuplint/markuplint/commit/ec57e1e5ff4549ee5574928ad20fd461d87974a5))
+
+
+### Features
+
+* **types:** avoid parsing and simply accept any value when the CSS includes `var()` ([5817693](https://github.com/markuplint/markuplint/commit/5817693cfcd1a253c627db323505e4b515f69395))
 
 
 
 
 
+## [4.9.4](https://github.com/markuplint/markuplint/compare/@markuplint/rules@4.9.3...@markuplint/rules@4.9.4) (2024-09-02)
+
+**Note:** Version bump only for package @markuplint/rules
+
+
+
+
+
+## [4.9.3](https://github.com/markuplint/markuplint/compare/@markuplint/rules@4.9.2...@markuplint/rules@4.9.3) (2024-06-25)
+
+### Bug Fixes
+
+- **ml-core:** `localName` returns lowercase when using case-sensitive parser for tag names ([b1acadd](https://github.com/markuplint/markuplint/commit/b1acaddfd6bf939ee809f6419ce85a701033ca4f))
+
 ## [4.9.2](https://github.com/markuplint/markuplint/compare/@markuplint/rules@4.9.1...@markuplint/rules@4.9.2) (2024-06-09)
 
 ### Bug Fixes

package/lib/create-message.js

@@ -224,7 +224,7 @@ function createExpectedObject(type, matches, t) {
         ? null
         : 1 < expectedObject.length
             ? t('either {0}', t(expectedObject))
-            : expectedObject[0] ?? null;
+            : (expectedObject[0] ?? null);
     return expects;
 }
 function expectValueToWord(t, expect, type) {

package/lib/heading-levels/index.js

@@ -12,7 +12,7 @@ export default createRule({
             if (prevLevel + 1 < level) {
                 report({
                     scope: heading,
-                    message: t('Heading levels should not be skipped'),
+                    message: t('{0} must not be skipped', t('Heading levels')),
                 });
             }
             prevLevel = level;

package/lib/helpers.js

@@ -67,7 +67,7 @@ node, attrSpecs,
 log) {
     let invalid = false;
     const spec = attrSpecs.find(s => s.name.toLowerCase() === name.toLowerCase());
-    log && log('Spec of the %s attr: %o', name, spec);
+    log?.('Spec of the %s attr: %o', name, spec);
     invalid = attrCheck(t, name, value, false, spec);
     if (invalid === false &&
         spec &&

package/lib/no-empty-palpable-content/index.js

@@ -1,6 +1,16 @@
 import { createRule } from '@markuplint/ml-core';
 import { isNothingContentModel, isPalpableElement } from '@markuplint/ml-spec';
 import meta from './meta.js';
+const allowedElements = new Set([
+    // These elements are possibly empty because it to be filled by user interaction.
+    'textarea',
+    'output',
+    // Since the element itself is palpable, there is no need to determine whether its content is empty.
+    'audio',
+    'canvas',
+    'video',
+    'img',
+]);
 export default createRule({
     meta: meta,
     defaultSeverity: 'warning',
@@ -10,12 +20,7 @@ export default createRule({
     },
     async verify({ document, report, t }) {
         await document.walkOn('Element', el => {
-            /**
-             * Exception
-             *
-             * - The `textarea` element is possibly empty because a user inputs it.
-             */
-            if (el.localName === 'textarea') {
+            if (allowedElements.has(el.localName)) {
                 return;
             }
             if (!isPalpableElement(el, el.ownerMLDocument.specs, {

package/lib/permitted-contents/count-pattern.js

@@ -22,7 +22,6 @@ childNodes, specs, options, depth) {
     let prevResult = null;
     let barelyResult = null;
     let loopCount = 0;
-    // eslint-disable-next-line no-constant-condition
     while (true) {
         loopCount++;
         ptLog('Check#%s: %s', loopCount, collection);
@@ -88,7 +87,7 @@ childNodes, specs, options, depth) {
                 result.type === 'MISSING_NODE_ONE_OR_MORE' ||
                 result.type === 'TRANSPARENT_MODEL_DISALLOWS'
                 ? result.type
-                : missingType ?? 'MISSING_NODE_REQUIRED';
+                : (missingType ?? 'MISSING_NODE_REQUIRED');
             ptLog('%s(in %s); Needs %s', resultType, missingType, result.query);
             return compereResult({
                 type: resultType,

package/lib/permitted-contents/debug.js

@@ -1,7 +1,6 @@
 import color from 'ansi-colors';
 import { log } from '../debug.js';
 export const cmLog = log.extend('content-model');
-/* eslint-disable import/no-named-as-default-member */
 export const bgGreen = color.bgGreen;
 export const green = color.green;
 export const bgRed = color.bgRed;
@@ -9,4 +8,3 @@ export const bgBlue = color.bgBlue;
 export const blue = color.blue;
 export const bgMagenta = color.bgMagenta;
 export const cyan = color.cyan;
-/* eslint-enable import/no-named-as-default-member */

package/lib/permitted-contents/start.js

@@ -55,7 +55,7 @@ el, specs, options) {
                     type: result.type,
                     scope: result.type === 'MISSING_NODE_REQUIRED' || result.type === 'MISSING_NODE_ONE_OR_MORE'
                         ? el
-                        : result.unmatched[0] ?? el,
+                        : (result.unmatched[0] ?? el),
                     query: result.query,
                     hint: result.hint,
                 },

package/lib/permitted-contents/utils.d.ts

@@ -31,7 +31,7 @@ export declare function mergeHints(a: Readonly<Hints>, b: Readonly<Hints>): Part
     not?: ChildNode;
     transparent?: import("./types.js").Element;
 }>;
-export declare function cleanObject<T extends Object>(object: T): Partial<T>;
+export declare function cleanObject<T extends object>(object: T): Partial<T>;
 export declare class Collection {
     #private;
     constructor(origin: readonly ChildNode[]);

package/lib/permitted-contents/utils.js

@@ -40,7 +40,7 @@ node, specs) {
         };
     }
     const not = selectorResult
-        .flatMap(r => (r.matched ? [] : r.not ?? []))
+        .flatMap(r => (r.matched ? [] : (r.not ?? [])))
         .flatMap(descendants)
         .shift();
     return {

package/lib/wai-aria/checkings/required-prop.js

@@ -17,7 +17,7 @@ export const checkingRequiredProp = ({ el, role, propSpecs }) => t => {
             const elAriaSpec = getARIA(el.ownerMLDocument.specs, el.localName, el.namespaceURI, el.rule.options.version, el.matches.bind(el));
             const alt = elAriaSpec?.properties === false
                 ? null
-                : elAriaSpec?.properties?.without?.find(p => p.name === requiredProp)?.alt ?? null;
+                : (elAriaSpec?.properties?.without?.find(p => p.name === requiredProp)?.alt ?? null);
             if (alt?.method === 'set-attr' && el.hasAttribute(alt.target)) {
                 return;
             }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@markuplint/rules",
-	"version": "4.9.3",
+	"version": "4.10.0",
 	"description": "Built-in rules of markuplint",
 	"repository": "git@github.com:markuplint/markuplint.git",
 	"author": "Yusuke Hirao <yusukehirao@me.com>",
@@ -25,18 +25,18 @@
 		"./lib/permitted-contents/debug.js": "./lib/permitted-contents/debug.browser.js"
 	},
 	"dependencies": {
-		"@markuplint/html-spec": "4.8.2",
-		"@markuplint/ml-core": "4.8.2",
-		"@markuplint/ml-spec": "4.6.3",
-		"@markuplint/selector": "4.6.5",
-		"@markuplint/shared": "4.4.3",
-		"@markuplint/types": "4.5.3",
+		"@markuplint/html-spec": "4.9.1",
+		"@markuplint/ml-core": "4.9.0",
+		"@markuplint/ml-spec": "4.7.0",
+		"@markuplint/selector": "4.6.7",
+		"@markuplint/shared": "4.4.5",
+		"@markuplint/types": "4.6.0",
 		"@types/debug": "4.1.12",
 		"@ungap/structured-clone": "1.2.0",
 		"ansi-colors": "4.1.3",
-		"chrono-node": "2.7.6",
-		"debug": "4.3.5",
-		"type-fest": "4.20.1"
+		"chrono-node": "2.7.7",
+		"debug": "4.3.7",
+		"type-fest": "4.26.1"
 	},
-	"gitHead": "05fdca254661ec335ff0cae4c6a11db164b032b9"
+	"gitHead": "05d2eabfcc41b67847c24049f12dd2b9f5ca6485"
 }

package/.eslintrc

@@ -1,5 +0,0 @@
-{
-	"rules": {
-		"import/no-default-export": 0
-	}
-}