chore: eslint sort import (#10)

Author: mdjastrzebski

.editorconfig

@@ -0,0 +1,17 @@
+# EditorConfig helps developers define and maintain consistent
+# coding styles between different editors and IDEs
+# editorconfig.org
+
+root = true
+
+[*]
+
+indent_style = space
+indent_size = 2
+
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true
+
+max_line_length = 100

.github/ISSUE_TEMPLATE/bug.md

@@ -1,22 +1,28 @@
 ## Bug Description
+
 [Concise description of the bug]
 
 ## Steps to Reproduce
-1. 
-2. 
-3. 
+
+1.
+2.
+3.
 
 ## Expected Behavior
+
 [What you expected to happen]
 
 ## Actual Behavior
+
 [What actually happened]
 
 ## Environment
+
 - Library version:
 - Node.js version:
 - Browser (if applicable):
 - OS:
 
 ## Additional Context
+
 [Any other relevant information, screenshots, etc.]

.github/ISSUE_TEMPLATE/feature_request.md

@@ -1,14 +1,19 @@
 ## Feature Description
+
 [Concise description of the proposed feature]
 
 ## Problem It Solves
+
 [Explain the problem this feature would solve]
 
 ## Proposed Solution
+
 [Your ideas on how to implement the feature]
 
 ## Alternatives Considered
+
 [Any alternative solutions or features you've considered]
 
 ## Additional Context
-[Any other relevant information, mockups, etc.]
+
+[Any other relevant information, mockups, etc.]

.github/pull_request_template.md

@@ -1,17 +1,21 @@
 ## Summary
+
 [Provide a brief description of the changes in this PR]
 
 ## Type of change
+
 - [ ] Bug fix
 - [ ] New feature
 - [ ] Breaking change
 - [ ] Documentation update
 - [ ] Other (specify below)
 
 ## How Has This Been Tested?
+
 [Describe the tests you ran to verify your changes]
 
 ## Checklist:
+
 - [ ] Code follows style guidelines
 - [ ] Self-review completed
 - [ ] Code commented where necessary
@@ -22,4 +26,5 @@
 - [ ] Dependent changes merged
 
 ## Additional Notes:
+
 [Add any additional information or context about the PR here]

.github/workflows/ci.yaml

@@ -12,7 +12,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      
+
       - name: Setup deps
         uses: ./.github/actions/setup
 

eslint.config.js

@@ -1,29 +1,50 @@
-import eslint from '@eslint/js';
-import tseslint from 'typescript-eslint';
+import eslint from "@eslint/js";
+import tseslint from "typescript-eslint";
+import simpleImportSort from "eslint-plugin-simple-import-sort";
 
 export default tseslint.config(
-    eslint.configs.recommended,
-    ...tseslint.configs.strictTypeChecked,
-    {
-        files: ['src/**/*.{ts,tsx}'],
-        languageOptions: {
-            parserOptions: {
-                projectService: true,
-                tsconfigRootDir: import.meta.dirname,
-            },
-        },
+  eslint.configs.recommended,
+  ...tseslint.configs.strictTypeChecked,
+  {
+    files: ["src/**/*.{ts,tsx}"],
+    languageOptions: {
+      parserOptions: {
+        projectService: true,
+        tsconfigRootDir: import.meta.dirname,
+      },
     },
-    {
-        rules: {
-            '@typescript-eslint/no-unused-vars': ['error', {
-                'argsIgnorePattern': '^_',
-                'varsIgnorePattern': '^_',
-                'caughtErrorsIgnorePattern': '^_'
-            }],
-        },
+  },
+  {
+    plugins: {
+      "simple-import-sort": simpleImportSort,
     },
-    {
-
-        ignores: ['node_modules/**', 'dist/**', "*.config.{js,ts}", "react-native.js", "react-native.d.ts"],
+    rules: {
+      "@typescript-eslint/no-unused-vars": [
+        "error",
+        {
+          argsIgnorePattern: "^_",
+          varsIgnorePattern: "^_",
+          caughtErrorsIgnorePattern: "^_",
+        },
+      ],
+      "@typescript-eslint/consistent-type-imports": ["error", {
+        fixStyle: "separate-type-imports",
+      }],
+      "simple-import-sort/imports": [
+        "error",
+        {
+          groups: [["^\\u0000", "^node:", "^@?\\w", "^"], ["^\\."]],
+        },
+      ],
     },
-);
+  },
+  {
+    ignores: [
+      "node_modules/**",
+      "dist/**",
+      "*.config.{js,ts}",
+      "react-native.js",
+      "react-native.d.ts",
+    ],
+  },
+);

jest.config.js

@@ -1,6 +1,6 @@
 export default {
-    preset: 'ts-jest',
-    testEnvironment: 'node',
-    testMatch: ['**/__tests__/**/*.ts?(x)', '**/?(*.)+(spec|test).ts?(x)'],
-    moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx', 'json', 'node'],
-};
+  preset: "ts-jest",
+  testEnvironment: "node",
+  testMatch: ["**/__tests__/**/*.ts?(x)", "**/?(*.)+(spec|test).ts?(x)"],
+  moduleFileExtensions: ["ts", "tsx", "js", "jsx", "json", "node"],
+};

package.json

@@ -51,7 +51,9 @@
     "@types/react": "^19.0.10",
     "@types/react-reconciler": "^0.28.9",
     "eslint": "^9.21.0",
+    "eslint-plugin-simple-import-sort": "^12.1.1",
     "jest": "^29.7.0",
+    "prettier": "^3.5.2",
     "react": "^19.0.0",
     "release-it": "^18.1.2",
     "ts-jest": "^29.2.6",

pnpm-lock.yaml

@@ -1,6 +1,9 @@
-import { act, createElement, ReactElement } from "react";
 import { expect, test } from "@jest/globals";
-import { createRoot, RootOptions } from "../renderer";
+import type { ReactElement } from "react";
+import { act, createElement } from "react";
+
+import type { RootOptions } from "../renderer";
+import { createRoot } from "../renderer";
 
 const originalConsoleError = console.error;
 
@@ -43,7 +46,7 @@ test("render with single allowed text component", () => {
     <div>
       <hr />
     </div>,
-    { textComponents: ["Text"] }
+    { textComponents: ["Text"] },
   );
   expect(renderer.root?.toJSON()).toMatchInlineSnapshot(`
 <div>
@@ -56,7 +59,7 @@ test("render with single allowed text component", () => {
       renderer.render(<div>Hello!</div>);
     });
   }).toThrowErrorMatchingInlineSnapshot(
-    `"Invariant Violation: Text strings must be rendered within a <Text> component. Detected attempt to render "Hello!" string within a <div> component."`
+    `"Invariant Violation: Text strings must be rendered within a <Text> component. Detected attempt to render "Hello!" string within a <div> component."`,
   );
 });
 
@@ -66,7 +69,7 @@ test("render with two allowed text components", () => {
       {createElement("A", null, "Hello!")}
       {createElement("B", null, "Hi!")}
     </div>,
-    { textComponents: ["A", "B"] }
+    { textComponents: ["A", "B"] },
   );
   expect(renderer.root?.toJSON()).toMatchInlineSnapshot(`
 <div>
@@ -84,7 +87,7 @@ test("render with two allowed text components", () => {
       renderer.render(createElement("X", null, "Hello!"));
     });
   }).toThrowErrorMatchingInlineSnapshot(
-    `"Invariant Violation: Text strings must be rendered within a <A> or <B> component. Detected attempt to render "Hello!" string within a <X> component."`
+    `"Invariant Violation: Text strings must be rendered within a <A> or <B> component. Detected attempt to render "Hello!" string within a <X> component."`,
   );
 });
 
@@ -95,14 +98,14 @@ test("render with multiple allowed text components", () => {
       {createElement("B", null, "Hi!")}
       {createElement("C", null, "Hola!")}
     </div>,
-    { textComponents: ["A", "B", "C"] }
+    { textComponents: ["A", "B", "C"] },
   );
 
   expect(() => {
     act(() => {
       renderer.render(createElement("X", null, "Hello!"));
     });
   }).toThrowErrorMatchingInlineSnapshot(
-    `"Invariant Violation: Text strings must be rendered within a <A>, <B>, or <C> component. Detected attempt to render "Hello!" string within a <X> component."`
+    `"Invariant Violation: Text strings must be rendered within a <A>, <B>, or <C> component. Detected attempt to render "Hello!" string within a <X> component."`,
   );
 });

src/__tests__/renderer.test.tsx

@@ -1,16 +1,14 @@
 import { CONTAINER_TYPE } from "./constants";
-import { Container, Instance, TextInstance } from "./reconciler";
-import { JsonNode, renderToJson } from "./render-to-json";
+import type { Container, Instance, TextInstance } from "./reconciler";
+import type { JsonNode} from "./render-to-json";
+import { renderToJson } from "./render-to-json";
 
 export type HostNode = HostElement | string;
 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 export type HostElementProps = Record<string, any>;
 
-const instanceToHostElementMap = new WeakMap<
-  Container | Instance,
-  HostElement
->();
+const instanceToHostElementMap = new WeakMap<Container | Instance, HostElement>();
 
 export class HostElement {
   private instance: Instance | Container;
@@ -20,9 +18,7 @@ export class HostElement {
   }
 
   get type(): string {
-    return this.instance.tag === "INSTANCE"
-      ? this.instance.type
-      : CONTAINER_TYPE;
+    return this.instance.tag === "INSTANCE" ? this.instance.type : CONTAINER_TYPE;
   }
 
   get props(): HostElementProps {

src/host-element.ts

@@ -1,5 +1,7 @@
-import { ReactElement } from "react";
-import { createRoot, Root } from "../renderer";
+import type { ReactElement } from "react";
+
+import type { Root } from "../renderer";
+import { createRoot } from "../renderer";
 
 export type ReactNativeRootOptions = {
   createNodeMock?: (element: ReactElement) => object;