Merge pull request #16 from jsonjoy-com/biome

Biome

Author: streamich

.github/workflows/pr.yml

@@ -38,7 +38,7 @@ jobs:
       - run: yarn install --frozen-lockfile
       - run: yarn build
       - run: yarn lint
-      - run: yarn prettier:check
+      - run: yarn format
       - run: yarn typedoc
   e2e-sample-api-http1:
     runs-on: ubuntu-latest

.github/workflows/release.yml

@@ -21,7 +21,7 @@ jobs:
           cache: yarn
       - run: yarn install --frozen-lockfile
       - run: yarn prettier:check
-      - run: yarn lint
+      - run: yarn format
       - run: yarn test:ci --ci
       - run: PORT=10000 yarn test:e2e --server http1 --suite sample-api
       - run: PORT=10001 yarn test:e2e --server uws --suite sample-api

biome.json

@@ -0,0 +1,42 @@
+{
+	"$schema": "https://biomejs.dev/schemas/1.9.3/schema.json",
+	"organizeImports": {
+		"enabled": true
+	},
+	"formatter": {
+		"indentStyle": "space",
+		"lineWidth": 120
+	},
+	"javascript": {
+		"formatter": {
+			"quoteStyle": "single",
+			"trailingCommas": "all",
+			"bracketSpacing": false
+		}
+	},
+	"linter": {
+		"enabled": true,
+		"rules": {
+			"recommended": true,
+			"style": {
+				"noNonNullAssertion": "off",
+				"useNodejsImportProtocol": "off",
+				"useTemplate": "off",
+				"noInferrableTypes": "off",
+				"noUselessElse": "off",
+				"noParameterAssign": "off",
+				"noCommaOperator": "off",
+				"useSingleVarDeclarator": "off"
+			},
+			"suspicious": {
+				"noExplicitAny": "off",
+				"useIsArray": "off",
+				"noAssignInExpressions": "off"
+			},
+			"complexity": {
+				"noStaticOnlyClass": "off",
+				"useOptionalChain": "off"
+			}
+		}
+	}
+}

package.json

@@ -39,10 +39,10 @@
   ],
   "license": "Apache-2.0",
   "scripts": {
-    "prettier": "prettier --ignore-path .gitignore --write \"src/**/*.{ts,tsx,js,jsx}\"",
-    "prettier:check": "prettier --ignore-path .gitignore --list-different 'src/**/*.{ts,tsx,js,jsx}'",
-    "lint": "oxlint ./src/",
-    "lint:fix": "oxlint ./src/ --fix --fix-suggestions",
+    "format": "biome format ./src",
+    "format:write": "biome format --write ./src",
+    "lint": "biome lint ./src",
+    "lint:fix": "biome lint --apply ./src",
     "clean": "rimraf lib typedocs coverage gh-pages yarn-error.log db dist",
     "build": "tsc --project tsconfig.build.json --module commonjs --target es2020 --outDir lib",
     "jest": "node -r ts-node/register ./node_modules/.bin/jest",
@@ -92,6 +92,7 @@
     "thingies": "^2.1.0"
   },
   "devDependencies": {
+    "@biomejs/biome": "^1.9.3",
     "@types/benchmark": "^2.1.5",
     "@types/jest": "^29.5.12",
     "@types/react": "^18.3.8",
@@ -107,8 +108,6 @@
     "jest": "^29.7.0",
     "nano-theme": "^1.4.2",
     "nice-ui": "^1.9.2",
-    "oxlint": "^0.9.9",
-    "prettier": "^3.2.5",
     "react": "^18.3.1",
     "react-dom": "^18.3.1",
     "rimraf": "^5.0.5",
@@ -117,8 +116,6 @@
     "ts-loader": "^9.5.1",
     "ts-node": "^10.9.2",
     "tslib": "^2.6.2",
-    "tslint": "^6.1.3",
-    "tslint-config-common": "^1.6.2",
     "typedoc": "^0.25.13",
     "typescript": "^5.4.5",
     "uWebSockets.js": "uNetworking/uWebSockets.js#v20.44.0",
@@ -143,15 +140,5 @@
     },
     "transformIgnorePatterns": [],
     "testRegex": ".*/(__tests__|__jest__|demo)/.*\\.(test|spec)\\.ts$"
-  },
-  "prettier": {
-    "arrowParens": "always",
-    "printWidth": 120,
-    "tabWidth": 2,
-    "useTabs": false,
-    "semi": true,
-    "singleQuote": true,
-    "trailingComma": "all",
-    "bracketSpacing": false
   }
 }

src/__demos__/json-crdt-server/index.ts

@@ -1,4 +1,4 @@
-import {createCaller} from './routes';
+import type {createCaller} from './routes';
 
 export {createCaller} from './routes';
 export type JsonJoyDemoRpcCaller = ReturnType<typeof createCaller>['caller'];

src/__demos__/json-crdt-server/routes/block/index.ts

@@ -48,7 +48,7 @@ export const block =
 
     system.alias('BlockEvent', BlockEvent);
 
-    // prettier-ignore
+    // biome-ignore format: each on its own line
     return (
     ( new_(d)
     ( get(d)

src/__demos__/json-crdt-server/routes/block/methods/get.ts

@@ -1,4 +1,4 @@
-import {ResolveType} from 'json-joy/lib/json-type';
+import type {ResolveType} from 'json-joy/lib/json-type';
 import {BlockIdRef, BlockRef} from '../schema';
 import type {RouteDeps, Router, RouterBase} from '../../types';
 
@@ -12,10 +12,7 @@ export const get =
       }),
     );
 
-    // prettier-ignore
-    const Response = t.Object(
-      t.prop('block', BlockRef),
-    );
+    const Response = t.Object(t.prop('block', BlockRef));
 
     const Func = t.Function(Request, Response).options({
       title: 'Read Block',

src/__demos__/json-crdt-server/routes/block/methods/listen.ts

@@ -16,7 +16,7 @@ export const listen =
 
     const Func = t.Function$(Request, Response).options({
       title: 'Listen for block changes',
-      description: `Subscribe to a block to receive updates when it changes.`,
+      description: 'Subscribe to a block to receive updates when it changes.',
     });
 
     return r.prop('block.listen', Func, (req$) => {

src/__demos__/json-crdt-server/routes/block/methods/new.ts

@@ -16,10 +16,7 @@ export const new_ =
       }),
     );
 
-    // prettier-ignore
-    const Response = t.Object(
-      t.prop('snapshot', BlockSnapshotReturnRef),
-    ).options({
+    const Response = t.Object(t.prop('snapshot', BlockSnapshotReturnRef)).options({
       title: 'New block creation response',
       description:
         'The response object for the new block creation, contains server generated metadata without blobs supplied by the client.',

src/__demos__/json-crdt-server/routes/block/methods/pull.ts

@@ -4,7 +4,7 @@ import type {RouteDeps, Router, RouterBase} from '../../types';
 export const pull =
   ({t, services}: RouteDeps) =>
   <R extends RouterBase>(r: Router<R>) => {
-    // prettier-ignore
+    // biome-ignore format: props
     const Request = t.Object(
       t.prop('id', BlockIdRef).options({
         title: 'Block ID',
@@ -26,7 +26,7 @@ export const pull =
       }),
     );
 
-    // prettier-ignore
+    // biome-ignore format: props
     const Response = t.Object(
       t.prop('batches', t.Array(BlockBatchRef)).options({
         title: 'Batches',