eslint-soml 0.0.5 → 0.0.6

package/index.d.ts

@@ -18,6 +18,7 @@ import type {
 	FloatNode,
 	BooleanNode,
 	NullNode,
+	DateNode,
 	InstantNode,
 	DurationNode,
 	Token,
@@ -37,6 +38,7 @@ export type {
 	FloatNode,
 	BooleanNode,
 	NullNode,
+	DateNode,
 	InstantNode,
 	DurationNode,
 	DurationPart,
@@ -178,6 +180,17 @@ export interface SomlRuleVisitor extends RuleVisitor, CustomRuleVisitorWithExit<
 	Float?(node: FloatNode, parent: ValueParent): void;
 	Boolean?(node: BooleanNode, parent: ValueParent): void;
 	Null?(node: NullNode, parent: ValueParent): void;
+	/**
+	Visits a calendar date without needing Temporal to read its components.
+
+	@example
+	```
+	Date(node) {
+		const year: number = node.year;
+	}
+	```
+	*/
+	Date?(node: DateNode, parent: ValueParent): void;
 	Instant?(node: InstantNode, parent: ValueParent): void;
 	Duration?(node: DurationNode, parent: ValueParent): void;
 }> {}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "eslint-soml",
-	"version": "0.0.5",
+	"version": "0.0.6",
 	"description": "ESLint plugin for SOML, a config format for humans",
 	"license": "MIT",
 	"repository": "github:soml-lang/eslint-soml",
@@ -53,23 +53,23 @@
 	"dependencies": {
 		"@eslint/core": "^1.2.1",
 		"@eslint/plugin-kit": "^0.7.3",
-		"soml-lang": "^0.0.7"
+		"soml-lang": "^0.0.8"
 	},
 	"devDependencies": {
 		"@babel/code-frame": "^8.0.6",
 		"@eslint-community/eslint-utils": "^4.10.1",
-		"@typescript-eslint/types": "^8.71.2",
+		"@typescript-eslint/types": "^8.72.1",
 		"enquirer": "^2.4.1",
-		"eslint": "^10.13.0",
+		"eslint": "^10.14.0",
 		"eslint-config-xo": "^7.0.0",
 		"eslint-doc-generator": "^3.7.1",
 		"eslint-plugin-eslint-plugin": "^7.6.2",
-		"eslint-plugin-jsdoc": "^65.3.11",
+		"eslint-plugin-jsdoc": "^65.4.2",
 		"indent-string": "^5.0.0",
 		"markdownlint-cli": "^0.50.0",
 		"nano-spawn": "^2.1.0",
 		"node-style-text": "^2.1.2",
-		"npm-run-all2": "^9.0.3",
+		"npm-run-all2": "^9.0.4",
 		"open-editor": "^6.0.0",
 		"outdent": "^0.8.0",
 		"typescript": "^6.0.3",

package/readme.md

@@ -16,7 +16,7 @@ npm install --save-dev eslint eslint-soml
 
 **Requires Node.js 22.13 or later, ESLint `>=10.4`, [flat config](https://eslint.org/docs/latest/use/configure/configuration-files), and [ESM](https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c#how-can-i-make-my-typescript-project-output-esm).**
 
-The rules of this plugin do not need `Temporal`, so they work on every supported Node.js version, also for files with instants and durations. Only reading the `value` of an `Instant` or `Duration` node needs it, as in your own rule or in a `no-restricted-syntax` selector that matches on `value`, such as `[value=1]`. Node.js 26 and later have `Temporal` built in. On older versions, load a [`Temporal` polyfill](https://github.com/fullcalendar/temporal-polyfill) first: make `import 'temporal-polyfill/global';` the first import in `eslint.config.js`, before `eslint-soml`.
+The rules of this plugin do not need `Temporal`, so they work on every supported Node.js version, also for files with dates, instants, and durations. Only reading the `value` of a `Date`, `Instant`, or `Duration` node needs it, as in your own rule or in a `no-restricted-syntax` selector that matches on `value`, such as `[value=1]`. Node.js 26 and later have `Temporal` built in. On older versions, load a [`Temporal` polyfill](https://github.com/fullcalendar/temporal-polyfill) first: make `import 'temporal-polyfill/global';` the first import in `eslint.config.js`, before `eslint-soml`.
 
 ## Usage
 
@@ -137,8 +137,8 @@ A selector can also say what the program that reads a file does not support. SOM
 		message: 'This program does not support the key `__proto__`.',
 	},
 	{
-		selector: ':matches(Instant, Duration)',
-		message: 'This program does not support instants and durations.',
+		selector: ':matches(Date, Instant, Duration)',
+		message: 'This program does not support dates, instants, and durations.',
 	},
 ],
 ```
@@ -187,7 +187,7 @@ A block that shows a mistake on purpose can be skipped with `<!-- eslint-skip --
 | [no-decimal-file-modes](docs/rules/no-decimal-file-modes.md)                                         | Disallow file modes written as decimal numbers.                                       | ✅ ☑️ |    | 💡 |
 | [no-dotted-key-collisions](docs/rules/no-dotted-key-collisions.md)                                   | Disallow quoted keys with a `.` that read as a path into an object next to them.      | ✅    |    |    |
 | [no-doubled-backslashes-in-literal-strings](docs/rules/no-doubled-backslashes-in-literal-strings.md) | Disallow doubled backslashes in literal strings, such as `'^\\d+$'`.                  | ✅    |    | 💡 |
-| [no-duplicate-array-items](docs/rules/no-duplicate-array-items.md)                                   | Disallow the same string twice in an array.                                           |      |    | 💡 |
+| [no-duplicate-array-items](docs/rules/no-duplicate-array-items.md)                                   | Disallow duplicate strings and dates in an array.                                     |      |    | 💡 |
 | [no-empty-keys](docs/rules/no-empty-keys.md)                                                         | Disallow empty and whitespace-only keys.                                              | ✅ ☑️ |    |    |
 | [no-escapes-in-literal-strings](docs/rules/no-escapes-in-literal-strings.md)                         | Disallow escapes in literal strings, such as `'Line one\nLine two'`.                  | ✅    |    | 💡 |
 | [no-escapes-in-paths](docs/rules/no-escapes-in-paths.md)                                             | Disallow `\n` and `\t` escapes in Windows paths.                                      | ✅ ☑️ |    | 💡 |

package/rules/no-duplicate-array-items.js

@@ -4,7 +4,6 @@ import {escapeInvisibleCharacters, hasAdjacentComment, isPunctuator} from './uti
 /**
 @import {SomlContext} from './rule/soml-context.js';
 @import {SomlRule} from './rule/to-eslint-rule.js';
-@import {StringNode} from 'soml-lang';
 */
 
 const MESSAGE_ID_ERROR = 'no-duplicate-array-items/error';
@@ -48,20 +47,22 @@ const create = context => {
 
 	context.on('Array', function * (node) {
 		/**
-		@type {Map<string, StringNode>}
+		@type {Map<string, number>}
 		*/
 		const seen = new Map();
 
 		for (const item of node.elements) {
-			// Only strings, because numbers and the like repeat in data, as in `[0, 0, 1]`.
-			if (item.type !== 'String') {
+			// Only strings and dates, because numbers and the like repeat in data, as in `[0, 0, 1]`.
+			if (item.type !== 'String' && item.type !== 'Date') {
 				continue;
 			}
 
-			const first = seen.get(item.value);
+			const value = item.type === 'Date' ? `${item.year}-${item.month}-${item.day}` : item.value;
+			const key = `${item.type}:${value}`;
+			const firstLine = seen.get(key);
 
-			if (first === undefined) {
-				seen.set(item.value, item);
+			if (firstLine === undefined) {
+				seen.set(key, sourceCode.getLoc(item).start.line);
 				continue;
 			}
 
@@ -73,8 +74,8 @@ const create = context => {
 				messageId: MESSAGE_ID_ERROR,
 				data: {
 					// A block string spans lines, so it is shown as a one-line string with the same value. A bidirectional control is escaped, or it could reorder the message, whichever spelling the string uses.
-					item: escapeInvisibleCharacters(item.block ? stringifyValue(item.value) : sourceCode.getText(item)),
-					line: sourceCode.getLoc(first).start.line,
+					item: escapeInvisibleCharacters(item.type === 'String' && item.block ? stringifyValue(item.value) : sourceCode.getText(item)),
+					line: firstLine,
 				},
 				// Removing an item changes the value, so it is a suggestion. It is left out when it would remove a comment.
 				suggest: hasComment
@@ -98,7 +99,7 @@ const config = {
 	meta: {
 		type: 'problem',
 		docs: {
-			description: 'Disallow the same string twice in an array.',
+			description: 'Disallow duplicate strings and dates in an array.',
 			recommended: false,
 		},
 		hasSuggestions: true,

package/rules/no-quoted-values-in-arrays.js

@@ -17,6 +17,7 @@ The types a string can spell. A boolean or `null` in quotes is for `no-quoted-ke
 */
 const TYPES = new Map([
 	['Integer', 'ints'],
+	['Date', 'dates'],
 	['Instant', 'instants'],
 	['Duration', 'durations'],
 ]);