fix: Sort nested type properties

fixes #21

Author: mskelton

src/__tests__/type-properties.spec.ts

@@ -69,6 +69,15 @@ ruleTester.run("sort/type-properties", rule, {
       errors: [{ messageId: "unsorted" }],
     },
 
+    // Nested
+    {
+      code: "interface A {b:string; a:{b:string; a:number}}",
+      output: "interface A {a:{b:string; a:number}; b:string}",
+      // Rule tester will not run multiple fix iterations, but as long as we
+      // have two errors, we know the nested rule was caught.
+      errors: [{ messageId: "unsorted" }, { messageId: "unsorted" }],
+    },
+
     // Weights
     {
       code: `

src/rules/type-properties.ts

@@ -3,7 +3,7 @@ import {
   ESLintUtils,
   TSESTree,
 } from "@typescript-eslint/experimental-utils"
-import { getName, getNodeRange, getNodeText } from "../ts-utils"
+import { getName, getNodeRange, getNodeText, isDelimiter } from "../ts-utils"
 import { docsURL, enumerate, isUnsorted } from "../utils"
 
 /**
@@ -42,7 +42,7 @@ export default ESLintUtils.RuleCreator.withoutDocs({
     function getRangeWithoutDelimiter(node: TSESTree.Node): TSESTree.Range {
       const range = getNodeRange(source, node)
 
-      return source.getLastToken(node)?.type === "Punctuator"
+      return isDelimiter(source.getLastToken(node))
         ? [range[0], range[1] - 1]
         : range
     }

src/ts-utils.ts

@@ -40,3 +40,9 @@ export function getNodeRange(source: TSESLint.SourceCode, node: TSESTree.Node) {
 export function getNodeText(source: TSESLint.SourceCode, node: TSESTree.Node) {
   return source.getText().slice(...getNodeRange(source, node))
 }
+
+/**
+ * Returns true if the token is a delimiter (comma or semicolon).
+ */
+export const isDelimiter = (token: TSESTree.Token | null) =>
+  token?.type === "Punctuator" && (token.value === "," || token.value === ";")