chore: switch to oxfmt (#1913)

Author: mdjastrzebski

.github/workflows/ci.yml

@@ -27,8 +27,8 @@ jobs:
       - name: Lint
         run: yarn lint
 
-      - name: Prettier
-        run: yarn prettier
+      - name: Format
+        run: yarn format:check
 
   typecheck:
     runs-on: ubuntu-latest

.oxfmtrc.json

@@ -0,0 +1,15 @@
+{
+  "$schema": "./node_modules/oxfmt/configuration_schema.json",
+  "singleQuote": true,
+  "trailingComma": "all",
+  "printWidth": 100,
+  "sortImports": {
+    "groups": [
+      "side_effect",
+      ["builtin", "external", "internal", "subpath", "unknown"],
+      ["parent", "sibling", "index"]
+    ]
+  },
+  "sortPackageJson": false,
+  "ignorePatterns": ["node_modules/", ".yarn", "codemods/**/tests/fixtures/**"]
+}

.prettierignore

@@ -8,7 +8,7 @@
   - `yarn test:ci`
   - `yarn typecheck`
   - `yarn lint`
-  - `yarn prettier`
+  - `yarn format:check`
   - `yarn build`
   - `yarn validate`
 - Task-specific guidance:

AGENTS.md

@@ -32,10 +32,10 @@ Our pre-commit hooks verify that your commit message matches this format when co
 
 ### Linting and tests
 
-We use TypeScript for type checking, `eslint` with `prettier` for linting and formatting the code, and `jest` for testing. Our pre-commit hooks verify that the linter and tests pass when committing. You can also run the following commands manually:
+We use TypeScript for type checking, `eslint` and `oxfmt` for linting and formatting the code, and `jest` for testing. Our pre-commit hooks verify that the linter and tests pass when committing. You can also run the following commands manually:
 
 - `yarn typecheck`: run TypeScript compiler on all files.
-- `yarn lint`: run eslint and prettier.
+- `yarn lint`: run eslint.
 - `yarn test`: run tests.
 
 ### Sending a pull request

CONTRIBUTING.md

@@ -7,14 +7,14 @@
 - Run tests in CI mode: `yarn test:ci`
 - Type check: `yarn typecheck`
 - Lint source files: `yarn lint`
-- Check formatting: `yarn prettier`
+- Check formatting: `yarn format:check`
 - Validate the main package: `yarn validate`
 - Build the package: `yarn build`
 
 ## Command notes
 
 - `yarn lint` runs ESLint on `src`.
-- `yarn validate` runs typecheck, tests, lint, and Prettier checks for the main package.
+- `yarn validate` runs typecheck, tests, lint, and oxfmt checks for the main package.
 - `yarn build` cleans `dist/`, transpiles source with Babel, and emits TypeScript declarations.
 
 ## Repo layout

agents/build-and-validation.md

@@ -4,9 +4,9 @@
 
 - ESLint uses `@callstack/eslint-config` together with `typescript-eslint`.
 - Important enforced rules include `no-console` and consistent type imports.
-- Prettier is the formatter.
+- oxfmt is the formatter.
 - Formatting defaults include single quotes and trailing commas.
 
 ## Imports
 
-- Keep imports sorted with `eslint-plugin-simple-import-sort`.
+- Keep imports sorted with `oxfmt`.

agents/code-style.md

@@ -1,6 +1,6 @@
+import type { Edit, SgNode } from '@codemod.com/jssg-types/main';
 import type { Transform } from 'codemod:ast-grep';
 import type TSX from 'codemod:ast-grep/langs/tsx';
-import type { Edit, SgNode } from '@codemod.com/jssg-types/main';
 
 const FUNCTIONS_TO_MAKE_ASYNC = new Set(['render', 'renderHook', 'act', 'fireEvent']);
 const FIRE_EVENT_METHODS_TO_MAKE_ASYNC = new Set(['press', 'changeText', 'scroll']);
@@ -241,10 +241,16 @@ function extractImportedFunctionNames(
   edits: Edit[],
 ): {
   importedFunctions: Set<string>;
-  specifiersToRemove: Array<{ specifier: SgNode<TSX>; importStmt: SgNode<TSX> }>;
+  specifiersToRemove: Array<{
+    specifier: SgNode<TSX>;
+    importStmt: SgNode<TSX>;
+  }>;
 } {
   const importedFunctions = new Set<string>();
-  const specifiersToRemove: Array<{ specifier: SgNode<TSX>; importStmt: SgNode<TSX> }> = [];
+  const specifiersToRemove: Array<{
+    specifier: SgNode<TSX>;
+    importStmt: SgNode<TSX>;
+  }> = [];
 
   for (const importStmt of rntlImports) {
     const importClause = importStmt.find({
@@ -332,7 +338,10 @@ function extractImportedFunctionNames(
  * @param edits - Array to collect edit operations
  */
 function removeDuplicateImportSpecifiers(
-  specifiersToRemove: Array<{ specifier: SgNode<TSX>; importStmt: SgNode<TSX> }>,
+  specifiersToRemove: Array<{
+    specifier: SgNode<TSX>;
+    importStmt: SgNode<TSX>;
+  }>,
   rootNode: SgNode<TSX>,
   edits: Edit[],
 ): void {
@@ -464,11 +473,17 @@ function findFireEventMethodCalls(
   for (const [asyncName, syncName] of ASYNC_FUNCTIONS_TO_RENAME.entries()) {
     if (syncName === 'fireEvent') {
       const wasImported = rntlImports.some((importStmt) => {
-        const importClause = importStmt.find({ rule: { kind: 'import_clause' } });
+        const importClause = importStmt.find({
+          rule: { kind: 'import_clause' },
+        });
         if (!importClause) return false;
-        const namedImports = importClause.find({ rule: { kind: 'named_imports' } });
+        const namedImports = importClause.find({
+          rule: { kind: 'named_imports' },
+        });
         if (!namedImports) return false;
-        const specifiers = namedImports.findAll({ rule: { kind: 'import_specifier' } });
+        const specifiers = namedImports.findAll({
+          rule: { kind: 'import_specifier' },
+        });
         return specifiers.some((spec) => {
           const identifier = spec.find({ rule: { kind: 'identifier' } });
           return identifier && identifier.text() === asyncName;

codemods/v14-async-functions/scripts/codemod.ts

@@ -400,7 +400,9 @@ expect(
 
 // Include hidden elements
 expect(
-  screen.getByText('Hidden from accessibility', { includeHiddenElements: true }),
+  screen.getByText('Hidden from accessibility', {
+    includeHiddenElements: true,
+  }),
 ).toBeOnTheScreen();
 ```
 

docs/api/queries.md

@@ -121,7 +121,10 @@ You can also transform prop values so that they are more readable (e.g., flatten
 import { StyleSheet } from 'react-native';
 
 screen.debug({
-  mapProps: ({ style, ...props }) => ({ style: StyleSheet.flatten(style), ...props }),
+  mapProps: ({ style, ...props }) => ({
+    style: StyleSheet.flatten(style),
+    ...props,
+  }),
 });
 ```