@markuplint/rules 3.15.0 → 4.0.0-alpha.10

package/lib/wai-aria/types.d.ts

@@ -1,13 +1,13 @@
 import type { ARIAVersion } from '@markuplint/ml-spec';
 export type Options = {
-	checkingValue: boolean;
-	checkingDeprecatedProps: boolean;
-	permittedAriaRoles: boolean;
-	checkingRequiredOwnedElements: boolean;
-	checkingPresentationalChildren: boolean;
-	checkingInteractionInHidden: boolean;
-	disallowSetImplicitRole: boolean;
-	disallowSetImplicitProps: boolean;
-	disallowDefaultValue: boolean;
-	version: ARIAVersion;
+    checkingValue: boolean;
+    checkingDeprecatedProps: boolean;
+    permittedAriaRoles: boolean;
+    checkingRequiredOwnedElements: boolean;
+    checkingPresentationalChildren: boolean;
+    checkingInteractionInHidden: boolean;
+    disallowSetImplicitRole: boolean;
+    disallowSetImplicitProps: boolean;
+    disallowDefaultValue: boolean;
+    version: ARIAVersion;
 };

package/lib/wai-aria/types.js

@@ -1,2 +1 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
+export {};

package/package.json

@@ -1,13 +1,18 @@
 {
 	"name": "@markuplint/rules",
-	"version": "3.15.0",
+	"version": "4.0.0-alpha.10",
 	"description": "Built-in rules of markuplint",
 	"repository": "git@github.com:markuplint/markuplint.git",
 	"author": "Yusuke Hirao <yusukehirao@me.com>",
 	"license": "MIT",
 	"private": false,
-	"main": "lib/index.js",
-	"types": "lib/index.d.ts",
+	"type": "module",
+	"exports": {
+		".": {
+			"import": "./lib/index.js"
+		}
+	},
+	"types": "./lib/index.d.ts",
 	"publishConfig": {
 		"access": "public"
 	},
@@ -20,19 +25,18 @@
 		"./lib/permitted-contents/debug.js": "./lib/permitted-contents/debug.browser.js"
 	},
 	"dependencies": {
-		"@markuplint/html-spec": "3.14.0",
-		"@markuplint/ml-core": "3.15.0",
-		"@markuplint/ml-spec": "3.14.0",
-		"@markuplint/selector": "3.14.0",
-		"@markuplint/shared": "3.8.0",
-		"@markuplint/types": "3.12.0",
+		"@markuplint/html-spec": "4.0.0-alpha.10",
+		"@markuplint/ml-core": "4.0.0-alpha.10",
+		"@markuplint/ml-spec": "4.0.0-alpha.10",
+		"@markuplint/selector": "4.0.0-alpha.10",
+		"@markuplint/shared": "4.0.0-alpha.10",
+		"@markuplint/types": "4.0.0-alpha.10",
 		"@types/debug": "^4.1.12",
 		"@ungap/structured-clone": "^1.2.0",
 		"ansi-colors": "^4.1.3",
-		"chrono-node": "^2.7.3",
+		"chrono-node": "^2.7.5",
 		"debug": "^4.3.4",
-		"tslib": "^2.6.2",
-		"type-fest": "^4.8.2"
+		"type-fest": "^4.10.1"
 	},
-	"gitHead": "b37b749d7ac0f9e6cbd022ee7031bc020c6677d3"
+	"gitHead": "b41153ea665aa8f091daf6114a06047f4ccb8350"
 }

package/test/attr-check.spec.js

@@ -1,77 +0,0 @@
-const { translator } = require('@markuplint/i18n');
-const { i18n } = require('markuplint');
-
-const { valueCheck } = require('../lib/attr-check');
-
-let t;
-
-beforeAll(async () => {
-	const locale = await i18n('en');
-	t = translator(locale);
-});
-
-test('OneCodePointChar List', () => {
-	expect(
-		valueCheck(t, 'accesskey', '@ 12', {
-			token: 'OneCodePointChar',
-			separator: 'space',
-			unique: true,
-		}),
-	).toStrictEqual([
-		'the "accesskey" attribute expects space-separated list. the the content of the list part includes unexpected characters. It expects one code point character (https://html.spec.whatwg.org/multipage/interaction.html#the-accesskey-attribute)',
-		{
-			col: 2,
-			line: 0,
-			raw: '12',
-		},
-	]);
-});
-
-test('BrowsingContextNameOrKeyword', () => {
-	expect(valueCheck(t, 'target', '', 'BrowsingContextNameOrKeyword')).toStrictEqual([
-		'the "target" attribute must not be empty. It expects either "_blank", "_self", "_parent", "_top", "browsing context name" (https://html.spec.whatwg.org/multipage/browsers.html#valid-browsing-context-name-or-keyword)',
-		{
-			col: 0,
-			line: 0,
-			raw: '',
-		},
-	]);
-});
-
-test('BCP47', () => {
-	expect(valueCheck(t, 'lang', '', 'BCP47')).toStrictEqual([
-		'the "lang" attribute must not be empty. It expects the BCP47 format (https://tools.ietf.org/rfc/bcp/bcp47.html)',
-		{
-			col: 0,
-			line: 0,
-			raw: '',
-		},
-	]);
-});
-
-test('DateTime', () => {
-	expect(valueCheck(t, 'datetime', '200-1-1', 'DateTime')).toStrictEqual([
-		'the year part of the "datetime" attribute expects four or more digits (https://html.spec.whatwg.org/multipage/text-level-semantics.html#datetime-value)',
-		{
-			col: 0,
-			line: 0,
-			raw: '200',
-		},
-	]);
-	expect(valueCheck(t, 'datetime', '2000-1-1', 'DateTime')).toStrictEqual([
-		'the month part of the "datetime" attribute expects two digits (https://html.spec.whatwg.org/multipage/text-level-semantics.html#datetime-value)',
-		{
-			col: 5,
-			line: 0,
-			raw: '1',
-		},
-	]);
-	expect(valueCheck(t, 'datetime', '00:00:00.0000', 'DateTime')).toStrictEqual([
-		'the fractional part part of the "datetime" attribute expects one to three digits (https://html.spec.whatwg.org/multipage/text-level-semantics.html#datetime-value)',
-		{
-			col: 9,
-			line: 0,
-			raw: '0000',
-		},
-	]);
-});

package/test/attr-duplication/index.spec.js

@@ -1,159 +0,0 @@
-const { mlRuleTest } = require('markuplint');
-
-const rule = require('../../lib/attr-duplication').default;
-
-test('is test 1', async () => {
-	const { violations } = await mlRuleTest(
-		rule,
-		`
-		<div data-attr="value" data-Attr='db' data-attR=tr>
-			lorem
-			<p>ipsam</p>
-		</div>
-		`,
-	);
-
-	expect(violations).toStrictEqual([
-		{
-			severity: 'error',
-			message: 'The attribute name is duplicated',
-			line: 2,
-			col: 26,
-			raw: 'data-Attr',
-		},
-		{
-			severity: 'error',
-			message: 'The attribute name is duplicated',
-			line: 2,
-			col: 41,
-			raw: 'data-attR',
-		},
-	]);
-});
-
-test('is test 2', async () => {
-	const { violations } = await mlRuleTest(
-		rule,
-		`
-		<div
-			data-attr="value"
-			data-Attr='db'
-			data-attR=tr>
-			lorem
-			<p>ipsam</p>
-		</div>
-		`,
-	);
-
-	expect(violations).toStrictEqual([
-		{
-			severity: 'error',
-			message: 'The attribute name is duplicated',
-			line: 4,
-			col: 4,
-			raw: 'data-Attr',
-		},
-		{
-			severity: 'error',
-			message: 'The attribute name is duplicated',
-			line: 5,
-			col: 4,
-			raw: 'data-attR',
-		},
-	]);
-});
-
-test('is test 3', async () => {
-	const { violations } = await mlRuleTest(rule, '<img src="/" SRC="/" >', undefined, false, 'ja');
-
-	expect(violations.map(_ => _.message)).toStrictEqual(['属性名が重複しています']);
-});
-
-test('nodeRules disable', async () => {
-	const { violations } = await mlRuleTest(rule, '<div><span attr attr></span></div>', {
-		nodeRule: [
-			{
-				selector: 'span',
-				rule: false,
-			},
-		],
-	});
-
-	expect(violations.length).toStrictEqual(0);
-});
-
-test('Vue', async () => {
-	const { violations } = await mlRuleTest(rule, '<template><div attr v-bind:attr /></template>', {
-		parser: {
-			'.*': '@markuplint/vue-parser',
-		},
-	});
-
-	expect(violations).toStrictEqual([
-		{
-			severity: 'error',
-			line: 1,
-			col: 21,
-			message: 'The attribute name is duplicated',
-			raw: 'v-bind:attr',
-		},
-	]);
-});
-
-test('Vue (exception)', async () => {
-	const { violations } = await mlRuleTest(
-		rule,
-		'<template><div class="foo" v-bind:class="bar" style="a: b;" :style="{c: d}" /></template>',
-		{
-			parser: {
-				'.*': '@markuplint/vue-parser',
-			},
-		},
-	);
-
-	expect(violations.length).toBe(0);
-});
-
-test('React', async () => {
-	const { violations } = await mlRuleTest(rule, '<div tabindex tabIndex />', {
-		parser: {
-			'.*': '@markuplint/vue-parser',
-		},
-	});
-
-	expect(violations).toStrictEqual([]);
-});
-
-test('Pug', async () => {
-	const { violations } = await mlRuleTest(rule, '.hoge(class="hoge2")&attributes({class: "hoge3"})', {
-		parser: {
-			'.*': '@markuplint/pug-parser',
-		},
-	});
-
-	expect(violations.length).toBe(0);
-});
-
-test('Svelte', async () => {
-	const { violations } = await mlRuleTest(
-		rule,
-		'<div class:selected="{isSelected}" class:focused="{isFocused}"></div>',
-		{
-			parser: {
-				'.*': '@markuplint/svelte-parser',
-			},
-		},
-	);
-
-	expect(violations.length).toBe(0);
-});
-
-test('Astro', async () => {
-	const { violations } = await mlRuleTest(rule, '<div class:list="a" class="b"></div>', {
-		parser: {
-			'.*': '@markuplint/astro-parser',
-		},
-	});
-
-	expect(violations.length).toBe(1);
-});

package/test/attr-value-quotes/index.spec.js

@@ -1,133 +0,0 @@
-const { mlRuleTest } = require('markuplint');
-
-const rule = require('../../lib/attr-value-quotes').default;
-
-describe('verify', () => {
-	test('default', async () => {
-		const { violations } = await mlRuleTest(
-			rule,
-			`
-		<div data-attr="value" data-Attr='db' data-attR=tr>
-			lorem
-			<p>ipsam</p>
-		</div>
-		`,
-		);
-		expect(violations).toStrictEqual([
-			{
-				severity: 'warning',
-				message: 'Attribute value is must quote on double quotation mark',
-				line: 2,
-				col: 26,
-				raw: "data-Attr='db'",
-			},
-			{
-				severity: 'warning',
-				message: 'Attribute value is must quote on double quotation mark',
-				line: 2,
-				col: 41,
-				raw: 'data-attR=tr',
-			},
-		]);
-	});
-
-	test('double', async () => {
-		const { violations } = await mlRuleTest(
-			rule,
-			`
-		<div data-attr="value" data-Attr='db' data-attR=tr>
-			lorem
-			<p>ipsam</p>
-		</div>
-		`,
-			{
-				rule: {
-					severity: 'error',
-					value: 'double',
-					options: null,
-				},
-			},
-		);
-		expect(violations).toStrictEqual([
-			{
-				severity: 'error',
-				message: 'Attribute value is must quote on double quotation mark',
-				line: 2,
-				col: 26,
-				raw: "data-Attr='db'",
-			},
-			{
-				severity: 'error',
-				message: 'Attribute value is must quote on double quotation mark',
-				line: 2,
-				col: 41,
-				raw: 'data-attR=tr',
-			},
-		]);
-	});
-
-	test('single', async () => {
-		const { violations } = await mlRuleTest(
-			rule,
-			`
-		<div data-attr="value" data-Attr='db' data-attR=tr>
-			lorem
-			<p>ipsam</p>
-		</div>
-		`,
-			{
-				rule: {
-					severity: 'error',
-					value: 'single',
-					options: null,
-				},
-			},
-		);
-		expect(violations).toStrictEqual([
-			{
-				severity: 'error',
-				message: 'Attribute value is must quote on single quotation mark',
-				line: 2,
-				col: 8,
-				raw: 'data-attr="value"',
-			},
-			{
-				severity: 'error',
-				message: 'Attribute value is must quote on single quotation mark',
-				line: 2,
-				col: 41,
-				raw: 'data-attR=tr',
-			},
-		]);
-	});
-
-	test('empty', async () => {
-		const { violations } = await mlRuleTest(
-			rule,
-			`
-		<div data-attr>
-			lorem
-			<p>ipsam</p>
-		</div>
-		`,
-		);
-		expect(violations.length).toBe(0);
-	});
-});
-
-// describe('fix', () => {
-// 	test('empty', async () => {
-// 		const { fixedCode } = await mlRuleTest(rule, '<div attr noop=noop foo="bar" hoge=\'fuga\'>', undefined, true);
-// 		expect(fixedCode).toEqual('<div attr noop="noop" foo="bar" hoge="fuga">');
-// 	});
-
-// 	test('empty', async () => {
-// 		const { fixedCode } = await mlRuleTest(
-// 			rule,
-// 			'<div attr noop=noop foo="bar" hoge=\'fuga\'>',
-// 			{ rule: 'single' },
-// 			true,
-// 		);
-// 		expect(fixedCode).toEqual("<div attr noop='noop' foo='bar' hoge='fuga'>");
-// 	});
-// });

package/test/case-sensitive-attr-name/index.spec.js

@@ -1,65 +0,0 @@
-const { mlRuleTest } = require('markuplint');
-
-const rule = require('../../lib/case-sensitive-attr-name').default;
-
-describe('verify', () => {
-	test('lower case', async () => {
-		const { violations } = await mlRuleTest(rule, '<div data-lowercase></div>');
-		expect(violations).toStrictEqual([]);
-	});
-
-	test('upper case', async () => {
-		const { violations } = await mlRuleTest(rule, '<div data-UPPERCASE="value"></div>');
-		expect(violations[0].severity).toBe('warning');
-		expect(violations[0].message).toBe('Attribute names of HTML elements should be lowercase');
-		expect(violations[0].raw).toBe('data-UPPERCASE');
-	});
-
-	test('upper case', async () => {
-		const { violations } = await mlRuleTest(rule, '<div data-UPPERCASE="value"></div>', {
-			rule: {
-				severity: 'error',
-				value: 'upper',
-				options: null,
-			},
-		});
-		expect(violations[0].severity).toBe('error');
-		expect(violations[0].message).toBe('Attribute names of HTML elements must be uppercase');
-	});
-
-	test('upper case', async () => {
-		const { violations } = await mlRuleTest(rule, '<div data-uppercase="value"></div>', {
-			rule: {
-				severity: 'error',
-				value: 'upper',
-				options: null,
-			},
-		});
-		expect(violations[0].severity).toBe('error');
-		expect(violations[0].message).toBe('Attribute names of HTML elements must be uppercase');
-	});
-
-	test('upper case', async () => {
-		const { violations } = await mlRuleTest(rule, '<div DATA-UPPERCASE="value"></div>', {
-			rule: { severity: 'error', value: 'upper' },
-		});
-		expect(violations.length).toBe(0);
-	});
-
-	test('svg', async () => {
-		const { violations } = await mlRuleTest(rule, '<svg viewBox="0 0 100 100"></svg>');
-		expect(violations.length).toBe(0);
-	});
-});
-
-// describe('fix', () => {
-// 	test('upper case', async () => {
-// 		const { fixedCode } = await mlRuleTest(rule, '<DIV DATA-LOWERCASE></DIV>', undefined, true);
-// 		expect(fixedCode).toBe('<DIV data-lowercase></DIV>');
-// 	});
-
-// 	test('upper case', async () => {
-// 		const { fixedCode } = await mlRuleTest(rule, '<DIV data-lowercase></DIV>', { rule: 'upper' }, true);
-// 		expect(fixedCode).toBe('<DIV DATA-LOWERCASE></DIV>');
-// 	});
-// });

package/test/case-sensitive-tag-name/index.spec.js

@@ -1,80 +0,0 @@
-const { mlRuleTest } = require('markuplint');
-
-const rule = require('../../lib/case-sensitive-tag-name').default;
-
-describe('verify', () => {
-	test('lower case', async () => {
-		const { violations } = await mlRuleTest(rule, '<div data-lowercase></div>');
-		expect(violations).toStrictEqual([]);
-	});
-
-	test('upper case', async () => {
-		const { violations } = await mlRuleTest(rule, '<DIV data-lowercase></DIV>');
-		expect(violations[0].severity).toBe('warning');
-		expect(violations[0].message).toBe('Tag names of HTML elements should be lowercase');
-		expect(violations[0].raw).toBe('DIV');
-	});
-
-	test('upper case', async () => {
-		const { violations } = await mlRuleTest(rule, '<div data-UPPERCASE="value"></div>', {
-			rule: {
-				severity: 'error',
-				value: 'upper',
-			},
-		});
-		expect(violations[0].severity).toBe('error');
-		expect(violations[0].message).toBe('Tag names of HTML elements must be uppercase');
-	});
-
-	test('upper case', async () => {
-		const { violations } = await mlRuleTest(rule, '<DIV data-uppercase="value"></DIV>', {
-			rule: {
-				severity: 'error',
-				value: 'upper',
-			},
-		});
-		expect(violations.length).toBe(0);
-	});
-
-	test('upper case', async () => {
-		const { violations } = await mlRuleTest(rule, '<DIV DATA-UPPERCASE="value"></div>', {
-			rule: {
-				severity: 'error',
-				value: 'upper',
-			},
-		});
-		expect(violations.length).toBe(1);
-	});
-
-	test('upper case', async () => {
-		const { violations } = await mlRuleTest(rule, '<div DATA-UPPERCASE="value"></DIV>', {
-			rule: {
-				severity: 'error',
-				value: 'upper',
-			},
-		});
-		expect(violations.length).toBe(1);
-	});
-
-	test('svg', async () => {
-		const { violations } = await mlRuleTest(rule, '<svg viewBox="0 0 100 100"><textPath></textPath></svg>');
-		expect(violations.length).toBe(0);
-	});
-
-	test('custom elements', async () => {
-		const { violations } = await mlRuleTest(rule, '<xxx-hoge>lorem</xxx-hoge>');
-		expect(violations.length).toBe(0);
-	});
-
-	test('custom elements', async () => {
-		const { violations } = await mlRuleTest(rule, '<XXX-hoge>lorem</XXX-hoge>');
-		expect(violations.length).toBe(0);
-	});
-});
-
-// describe('fix', () => {
-// 	test('upper case', async () => {
-// 		const { fixedCode } = await mlRuleTest(rule, '<DIV data-lowercase></DIV>', undefined, true);
-// 		expect(fixedCode).toBe('<div data-lowercase></div>');
-// 	});
-// });

package/test/character-reference/index.spec.js

@@ -1,72 +0,0 @@
-const { mlRuleTest } = require('markuplint');
-
-const rule = require('../../lib/character-reference').default;
-
-test('character-reference', async () => {
-	const { violations } = await mlRuleTest(rule, '<div id="a"> > < & " \' &amp;</div>');
-	expect(violations.length).toBe(4);
-	expect(violations[0]).toStrictEqual({
-		severity: 'error',
-		message: 'Illegal characters must escape in character reference',
-		line: 1,
-		col: 14,
-		raw: '>',
-	});
-	expect(violations[1].col).toBe(16);
-	expect(violations[1].raw).toBe('<');
-	expect(violations[2].col).toBe(18);
-	expect(violations[2].raw).toBe('&');
-	expect(violations[3].col).toBe(20);
-	expect(violations[3].raw).toBe('"');
-});
-
-test('character-reference', async () => {
-	const { violations } = await mlRuleTest(rule, '<img src="path/to?a=b&c=d">');
-	expect(violations).toStrictEqual([
-		{
-			severity: 'error',
-			message: 'Illegal characters must escape in character reference',
-			line: 1,
-			col: 22,
-			raw: '&',
-		},
-	]);
-});
-
-test('character-reference', async () => {
-	const { violations } = await mlRuleTest(rule, '<script>if (i < 0) console.log("<markuplint>");</script>');
-	expect(violations.length).toBe(0);
-});
-
-test('in Vue', async () => {
-	const { violations } = await mlRuleTest(rule, '<template><div v-if="a < b"></div></template>', {
-		parser: {
-			'.*': '@markuplint/vue-parser',
-		},
-	});
-	expect(violations.length).toBe(0);
-});
-
-test('in EJS', async () => {
-	const { violations } = await mlRuleTest(rule, '<title><%- "title" _%></title>', {
-		parser: {
-			'.*': '@markuplint/ejs-parser',
-		},
-	});
-	expect(violations.length).toBe(0);
-});
-
-describe('Issues', () => {
-	test('#1074', async () => {
-		const { violations } = await mlRuleTest(rule, '<span>&#9660;</span><span>&#x25BC;</span><span>&x25BC;</span>');
-		expect(violations).toStrictEqual([
-			{
-				severity: 'error',
-				message: 'Illegal characters must escape in character reference',
-				line: 1,
-				col: 48,
-				raw: '&',
-			},
-		]);
-	});
-});