Enforce consistent type definitions

Author: Trinovantes

.eslintrc.cjs

@@ -182,5 +182,7 @@ module.exports = {
             allowNullableBoolean: true,
             allowNullableString: true,
         }],
+
+        '@typescript-eslint/consistent-type-definitions': ['error', 'type'],
     },
 }

src/generator/transforms/Transform.ts

@@ -1,6 +1,6 @@
 import type { TagNode } from '../../parser/AstNode.js'
 
-export interface Transform {
+export type Transform = {
     name: string
 
     skipChildren?: boolean // Do not recursively render children nodes (e.g. [img]url[/url] should not render url)

src/lexer/Token.ts

@@ -1,6 +1,6 @@
 import { TokenType } from './TokenType.js'
 
-export interface Token {
+export type Token = {
     type: TokenType
     offset: number
     length: number

src/parser/nodeIsType.ts

@@ -1,6 +1,6 @@
 import { AstNodeType, AstNode, AttrNode, RootNode, TagNode, TextNode, LinebreakNode, EndTagNode, StartTagNode } from './AstNode.js'
 
-interface AstMap {
+type AstMap = {
     [AstNodeType.RootNode]: RootNode
     [AstNodeType.LinebreakNode]: LinebreakNode
     [AstNodeType.TextNode]: TextNode

tests/data/TestCase.ts

@@ -1,10 +1,10 @@
-export interface ValidationTestCase {
+export type ValidationTestCase = {
     name: string
     input: string
     expectedOutputs: Array<string>
 }
 
-export interface XssTestCase {
+export type XssTestCase = {
     name: string
     input: string
     unexpectedSubstrings: Array<string>