fmt changes (#856)

swap prettier with oxfmt/oxlint, update CI, and fix the formatting/lint
issues needed for the new checks to pass.

Author: simonklee

.github/workflows/format-lint.yml

@@ -0,0 +1,66 @@
+name: Format and Lint
+
+on:
+  pull_request:
+    branches: [main]
+
+jobs:
+  fmt:
+    name: Format Check
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup Bun
+        uses: oven-sh/setup-bun@v2
+        with:
+          bun-version: latest
+
+      - name: Install dependencies
+        run: bun install
+
+      - name: Check formatting
+        run: bun run fmt:check
+
+      - name: Show formatting differences
+        if: failure()
+        run: |
+          echo "=== Showing differences for each file ==="
+          bun run fmt
+          git diff --color=always
+
+  lint:
+    name: Lint Check
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup Bun
+        uses: oven-sh/setup-bun@v2
+        with:
+          bun-version: latest
+
+      - name: Install dependencies
+        run: bun install
+
+      - name: Run linter
+        run: bun run lint:ci
+
+  checks-complete:
+    name: Format and Lint
+    needs: [fmt, lint]
+    runs-on: ubuntu-latest
+    if: always()
+
+    steps:
+      - name: Check results
+        run: |
+          if [ "${{ needs.fmt.result }}" != "success" ] || [ "${{ needs.lint.result }}" != "success" ]; then
+            echo "Format or lint checks failed"
+            exit 1
+          fi
+          echo "Format and lint checks passed"

.github/workflows/prettier.yml

@@ -0,0 +1,7 @@
+{
+  "$schema": "./node_modules/oxfmt/configuration_schema.json",
+  "semi": false,
+  "printWidth": 120,
+  "sortPackageJson": false,
+  "ignorePatterns": ["**/.zig-cache/**", "packages/core/src/lib/tree-sitter/default-parsers.ts"]
+}

.oxfmtrc.json

@@ -0,0 +1,18 @@
+{
+  "$schema": "./node_modules/oxlint/configuration_schema.json",
+  "plugins": ["react", "typescript", "import"],
+  "categories": {
+    "correctness": "off",
+    "suspicious": "off",
+    "pedantic": "off",
+    "perf": "off",
+    "style": "off",
+    "restriction": "off",
+    "nursery": "off"
+  },
+  "rules": {
+    "no-async-promise-executor": "error",
+    "react/react-in-jsx-scope": "off",
+    "react/no-unknown-property": "off"
+  }
+}

.oxlintrc.json

@@ -46,7 +46,7 @@ FROM THE `packages/core` DIRECTORY.
 ## Typescript Code Style
 
 - **Runtime**: Bun with TypeScript
-- **Formatting**: Prettier (semi: false, printWidth: 120)
+- **Formatting**: oxfmt (semi: false, printWidth: 120)
 - **Imports**: Use explicit imports, group by: built-ins, external deps, internal modules
 - **Types**: Strict TypeScript, use interfaces for options/configs, explicit return types for public APIs
 - **Naming**: camelCase for variables/functions, PascalCase for classes/interfaces, UPPER_CASE for constants

.prettierignore

@@ -7,20 +7,21 @@
   ],
   "scripts": {
     "build": "cd packages/core && bun run build && cd ../solid && bun run build && cd ../react && bun run build",
+    "fmt": "oxfmt --write .",
+    "fmt:check": "oxfmt --check .",
+    "lint": "oxlint .",
+    "lint:ci": "oxlint . --deny-warnings",
+    "lint:fix": "oxlint --fix .",
     "pre-publish": "bun scripts/pre-publish.ts",
     "publish": "bun run pre-publish && bun run publish:core && bun run publish:react && bun run publish:solid",
     "publish:core": "cd packages/core && bun run publish",
     "publish:react": "cd packages/react && bun run publish",
     "publish:solid": "cd packages/solid && bun run publish",
     "prepare-release": "bun scripts/prepare-release.ts",
-    "prettier:write": "prettier --write .",
     "test": "bun run --filter '@opentui/core' --filter '@opentui/solid' --filter '@opentui/react' --if-present test"
   },
   "devDependencies": {
-    "prettier": "3.6.2"
-  },
-  "prettier": {
-    "semi": false,
-    "printWidth": 120
+    "oxfmt": "0.41.0",
+    "oxlint": "1.56.0"
   }
 }

AGENTS.md

@@ -81,34 +81,29 @@ class ParserWorker {
     if (this.initializePromise) {
       return this.initializePromise
     }
-    this.initializePromise = new Promise(async (resolve, reject) => {
+    this.initializePromise = (async () => {
       this.dataPath = dataPath
       this.tsDataPath = path.join(dataPath, "tree-sitter")
 
-      try {
-        await mkdir(path.join(this.tsDataPath, "languages"), { recursive: true })
-        await mkdir(path.join(this.tsDataPath, "queries"), { recursive: true })
+      await mkdir(path.join(this.tsDataPath, "languages"), { recursive: true })
+      await mkdir(path.join(this.tsDataPath, "queries"), { recursive: true })
 
-        let { default: treeWasm } = await import("web-tree-sitter/tree-sitter.wasm" as string, {
-          with: { type: "wasm" },
-        })
+      let { default: treeWasm } = await import("web-tree-sitter/tree-sitter.wasm" as string, {
+        with: { type: "wasm" },
+      })
 
-        if (isBunfsPath(treeWasm)) {
-          treeWasm = normalizeBunfsPath(path.parse(treeWasm).base)
-        }
+      if (isBunfsPath(treeWasm)) {
+        treeWasm = normalizeBunfsPath(path.parse(treeWasm).base)
+      }
 
-        await Parser.init({
-          locateFile() {
-            return treeWasm
-          },
-        })
+      await Parser.init({
+        locateFile() {
+          return treeWasm
+        },
+      })
 
-        this.initialized = true
-        resolve()
-      } catch (error) {
-        reject(error)
-      }
-    })
+      this.initialized = true
+    })()
     return this.initializePromise
   }