feat: add svelte/html-closing-bracket-spacing

.gitignore

@@ -108,3 +108,6 @@ dist
 /svelte.config-dist.js
 /docs-svelte-kit/shim/eslint.mjs
 /docs-svelte-kit/shim/assert.mjs
+
+# Intellij IDES
+.idea

README.md

@@ -292,10 +292,13 @@ These rules relate to style guidelines, and are therefore quite subjective:
 | Rule ID | Description |    |
 |:--------|:------------|:---|
 | [svelte/first-attribute-linebreak](https://ota-meshi.github.io/eslint-plugin-svelte/rules/first-attribute-linebreak/) | enforce the location of first attribute | :wrench: |
+| [svelte/html-closing-bracket-spacing](https://ota-meshi.github.io/eslint-plugin-svelte/rules/html-closing-bracket-spacing/) | require or disallow a space before tag's closing brackets | :wrench: |
 | [svelte/html-quotes](https://ota-meshi.github.io/eslint-plugin-svelte/rules/html-quotes/) | enforce quotes style of HTML attributes | :wrench: |
+| [svelte/html-self-closing](https://ota-meshi.github.io/eslint-plugin-svelte/rules/html-self-closing/) | enforce self-closing style | :wrench: |
 | [svelte/indent](https://ota-meshi.github.io/eslint-plugin-svelte/rules/indent/) | enforce consistent indentation | :wrench: |
 | [svelte/max-attributes-per-line](https://ota-meshi.github.io/eslint-plugin-svelte/rules/max-attributes-per-line/) | enforce the maximum number of attributes per line | :wrench: |
 | [svelte/mustache-spacing](https://ota-meshi.github.io/eslint-plugin-svelte/rules/mustache-spacing/) | enforce unified spacing in mustache | :wrench: |
+| [svelte/no-spaces-around-equal-signs-in-attribute](https://ota-meshi.github.io/eslint-plugin-svelte/rules/no-spaces-around-equal-signs-in-attribute/) | disallow spaces around equal signs in attribute | :wrench: |
 | [svelte/prefer-class-directive](https://ota-meshi.github.io/eslint-plugin-svelte/rules/prefer-class-directive/) | require class directives instead of ternary expressions | :wrench: |
 | [svelte/prefer-style-directive](https://ota-meshi.github.io/eslint-plugin-svelte/rules/prefer-style-directive/) | require style directives instead of style attribute | :wrench: |
 | [svelte/shorthand-attribute](https://ota-meshi.github.io/eslint-plugin-svelte/rules/shorthand-attribute/) | enforce use of shorthand syntax in attribute | :wrench: |

docs/rules.md

@@ -52,10 +52,13 @@ These rules relate to style guidelines, and are therefore quite subjective:
 | Rule ID | Description |    |
 |:--------|:------------|:---|
 | [svelte/first-attribute-linebreak](./rules/first-attribute-linebreak.md) | enforce the location of first attribute | :wrench: |
+| [svelte/html-closing-bracket-spacing](./rules/html-closing-bracket-spacing.md) | require or disallow a space before tag's closing brackets | :wrench: |
 | [svelte/html-quotes](./rules/html-quotes.md) | enforce quotes style of HTML attributes | :wrench: |
+| [svelte/html-self-closing](./rules/html-self-closing.md) | enforce self-closing style | :wrench: |
 | [svelte/indent](./rules/indent.md) | enforce consistent indentation | :wrench: |
 | [svelte/max-attributes-per-line](./rules/max-attributes-per-line.md) | enforce the maximum number of attributes per line | :wrench: |
 | [svelte/mustache-spacing](./rules/mustache-spacing.md) | enforce unified spacing in mustache | :wrench: |
+| [svelte/no-spaces-around-equal-signs-in-attribute](./rules/no-spaces-around-equal-signs-in-attribute.md) | disallow spaces around equal signs in attribute | :wrench: |
 | [svelte/prefer-class-directive](./rules/prefer-class-directive.md) | require class directives instead of ternary expressions | :wrench: |
 | [svelte/prefer-style-directive](./rules/prefer-style-directive.md) | require style directives instead of style attribute | :wrench: |
 | [svelte/shorthand-attribute](./rules/shorthand-attribute.md) | enforce use of shorthand syntax in attribute | :wrench: |

docs/rules/html-closing-bracket-spacing.md

@@ -0,0 +1,77 @@
+---
+pageClass: "rule-details"
+sidebarDepth: 0
+title: "svelte/html-closing-bracket-spacing"
+description: "require or disallow a space before tag's closing brackets"
+---
+
+# svelte/html-closing-bracket-spacing
+
+> require or disallow a space before tag's closing brackets
+
+- :exclamation: <badge text="This rule has not been released yet." vertical="middle" type="error"> **_This rule has not been released yet._** </badge>
+- :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
+
+## :book: Rule Details
+
+You can choose either two styles for spacing before closing bracket
+
+- always: `<div />`
+- never: `<div/>`
+
+<ESLintCodeBlock fix>
+
+<!-- prettier-ignore-start -->
+<!--eslint-skip-->
+
+```svelte
+<script>
+  /* eslint svelte/html-closing-bracket-spacing: "error" */
+</script>
+
+<!-- ✓ GOOD -->
+<div />
+<p>Hello</p>
+<div
+ >
+</div>
+
+<!-- ✗ BAD -->
+<div/>
+<p >Hello</p >
+<div  >
+</div >
+```
+
+<!-- prettier-ignore-end -->
+
+</ESLintCodeBlock>
+
+## :wrench: Options
+
+```json
+{
+  "svelte/html-closing-bracket-spacing": [
+    "error",
+    {
+      "startTag": "never", // or "always" or "ignore"
+      "endTag": "never", // or "always" or "ignore"
+      "selfClosingTag": "always" // or "never" or "ignore"
+    }
+  ]
+}
+```
+
+- `startTag` (`"never"` by default)... Spacing in start tags
+- `endTag` (`"never"` by default)... Spacing in end tags
+- `selfClosingTag` (`"always"` by default)... Spacing in self closing tags
+
+Every option can be set to
+- "always" (`<div />`)
+- "never" (`<div/>`)
+- "ignore" (either `<div />` or `<div/>`)
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/ota-meshi/eslint-plugin-svelte/blob/main/src/rules/html-closing-bracket-spacing.ts)
+- [Test source](https://github.com/ota-meshi/eslint-plugin-svelte/blob/main/tests/src/rules/html-closing-bracket-spacing.ts)

docs/rules/html-self-closing.md

@@ -0,0 +1,78 @@
+---
+pageClass: "rule-details"
+sidebarDepth: 0
+title: "svelte/html-self-closing"
+description: "enforce self-closing style"
+---
+
+# svelte/html-self-closing
+
+> enforce self-closing style
+
+- :exclamation: <badge text="This rule has not been released yet." vertical="middle" type="error"> **_This rule has not been released yet._** </badge>
+- :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
+
+## :book: Rule Details
+
+You can choose either two styles for elements without content
+
+- always: `<div />`
+- never: `<div></div>`
+
+<ESLintCodeBlock fix>
+
+<!-- prettier-ignore-start -->
+<!--eslint-skip-->
+
+```svelte
+<script>
+  /* eslint svelte/html-self-closing: "error" */
+</script>
+
+<!-- ✓ GOOD -->
+<div />
+<p>Hello</p>
+<div><div /></div>
+<img>
+
+<!-- ✗ BAD -->
+<div></div>
+<p> </p>
+<div><div></div></div>
+<img />
+```
+
+<!-- prettier-ignore-end -->
+
+</ESLintCodeBlock>
+
+## :wrench: Options
+
+```json
+{
+  "svelte/html-self-closing": [
+    "error",
+    {
+      "html": {
+        "void": "always", // or "always" or "ignore"
+        "normal": "always", // or "never" or "ignore"
+        "component": "always" // or "never" or "ignore"
+      }
+    }
+  ]
+}
+```
+
+- `html.void` (`"always"` by default)... Style of HTML void elements
+- `html.component` (`"always"` by default)... Style of svelte components
+- `html.normal` (`"always"` by default)... Style of other elements
+
+Every option can be set to
+- "always" (`<div />`)
+- "never" (`<div></div>`)
+- "ignore" (either `<div />` or `<div></div>`)
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/ota-meshi/eslint-plugin-svelte/blob/main/src/rules/html-self-closing.ts)
+- [Test source](https://github.com/ota-meshi/eslint-plugin-svelte/blob/main/tests/src/rules/html-self-closing.ts)

docs/rules/no-spaces-around-equal-signs-in-attribute.md

@@ -0,0 +1,57 @@
+---
+pageClass: "rule-details"
+sidebarDepth: 0
+title: "svelte/no-spaces-around-equal-signs-in-attribute"
+description: "disallow spaces around equal signs in attribute"
+---
+
+# svelte/no-spaces-around-equal-signs-in-attribute
+
+> disallow spaces around equal signs in attribute
+
+- :exclamation: <badge text="This rule has not been released yet." vertical="middle" type="error"> **_This rule has not been released yet._** </badge>
+- :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
+
+## :book: Rule Details
+
+This rule disallows spaces around equal signs in attributes
+
+<ESLintCodeBlock fix>
+
+<!-- prettier-ignore-start -->
+<!--eslint-skip-->
+
+```svelte
+<script>
+  /* eslint svelte/no-spaces-around-equal-signs-in-attribute: "error" */
+</script>
+
+<!-- ✓ GOOD -->
+<div class=""/>
+<p style="color: red;">hi</p>
+<img src="img.png" alt="A photo of a very cute {animal}">
+
+<!-- ✗ BAD -->
+<div class = ""/>
+<p style ="color: red;">hi</p>
+<img src
+    =
+    "img.png" alt   = "A photo of a very cute {animal}">
+```
+
+<!-- prettier-ignore-end -->
+
+</ESLintCodeBlock>
+
+## :wrench: Options
+
+```json
+{
+  "svelte/no-spaces-around-equal-signs-in-attribute": ["error"]
+}
+```
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/ota-meshi/eslint-plugin-svelte/blob/main/src/rules/no-spaces-around-equal-signs-in-attribute.ts)
+- [Test source](https://github.com/ota-meshi/eslint-plugin-svelte/blob/main/tests/src/rules/no-spaces-around-equal-signs-in-attribute.ts)

src/configs/prettier.ts

@@ -7,10 +7,13 @@ export = {
   rules: {
     // eslint-plugin-svelte rules
     "svelte/first-attribute-linebreak": "off",
+    "svelte/html-closing-bracket-spacing": "off",
     "svelte/html-quotes": "off",
+    "svelte/html-self-closing": "off",
     "svelte/indent": "off",
     "svelte/max-attributes-per-line": "off",
     "svelte/mustache-spacing": "off",
+    "svelte/no-spaces-around-equal-signs-in-attribute": "off",
     "svelte/shorthand-attribute": "off",
     "svelte/shorthand-directive": "off",
   },

src/rules/html-closing-bracket-spacing.ts

@@ -0,0 +1,117 @@
+import { createRule } from "../utils"
+import type { AST } from "svelte-eslint-parser"
+
+export default createRule("html-closing-bracket-spacing", {
+  meta: {
+    docs: {
+      description: "require or disallow a space before tag's closing brackets",
+      category: "Stylistic Issues",
+      conflictWithPrettier: true,
+      recommended: false,
+    },
+    schema: [
+      {
+        type: "object",
+        properties: {
+          startTag: {
+            enum: ["always", "never", "any"],
+          },
+          endTag: {
+            enum: ["always", "never", "any"],
+          },
+          selfClosingTag: {
+            enum: ["always", "never", "any"],
+          },
+        },
+        additionalProperties: false,
+      },
+    ],
+    messages: {
+      expectedSpace: "Expected space before '>', but not found.",
+      unexpectedSpace: "Expected no space before '>', but found.",
+    },
+    fixable: "code",
+    type: "layout",
+  },
+  create(ctx) {
+    const options = {
+      startTag: "never",
+      endTag: "never",
+      selfClosingTag: "always",
+      ...ctx.options[0],
+    }
+    const src = ctx.getSourceCode()
+
+    /**
+     * Returns true if string contains newline characters
+     */
+    function containsNewline(string: string): boolean {
+      return string.includes("\n")
+    }
+
+    /**
+     * Report
+     */
+    function report(
+      node: AST.SvelteStartTag | AST.SvelteEndTag,
+      shouldHave: boolean,
+    ) {
+      const tagSrc = src.getText(node)
+      const match = /(\s*)\/?>$/.exec(tagSrc)
+
+      const end = node.range[1]
+      const start = node.range[1] - match![0].length
+      const loc = {
+        start: src.getLocFromIndex(start),
+        end: src.getLocFromIndex(end),
+      }
+
+      ctx.report({
+        loc,
+        messageId: shouldHave ? "expectedSpace" : "unexpectedSpace",
+        *fix(fixer) {
+          if (shouldHave) {
+            yield fixer.insertTextBeforeRange([start, end], " ")
+          } else {
+            const spaces = match![1]
+
+            yield fixer.removeRange([start, start + spaces.length])
+          }
+        },
+      })
+    }
+
+    return {
+      "SvelteStartTag, SvelteEndTag"(
+        node: AST.SvelteStartTag | AST.SvelteEndTag,
+      ) {
+        if (node.type === "SvelteEndTag" && options.endTag === "any") return
+
+        const tagSrc = src.getText(node)
+        const selfClosing =
+          tagSrc.slice(
+            Math.max(tagSrc.length - 2, 0),
+            Math.max(tagSrc.length - 1, 0),
+          ) === "/"
+
+        const tagType =
+          node.type === "SvelteEndTag"
+            ? "endTag"
+            : selfClosing
+            ? "selfClosingTag"
+            : "startTag"
+
+        if (node.type === "SvelteStartTag" && options[tagType] === "any") return
+
+        const match = /(\s*)\/?>$/.exec(tagSrc)
+        if (containsNewline(match![1])) return
+
+        if (options[tagType] === "always" && !match![1]) {
+          report(node, true)
+        } else if (options[tagType] === "never" && match![1]) {
+          report(node, false)
+        }
+      },
+    }
+  },
+})