chore: improve prettier config (#231)

* fix: `typeorm` missing in `bun.lockb`

* chore: add npm `prepare` script

* Update bun.lockb

* Update bun.lockb

* chore: build package on `prepublishOnly`

* chore: add `typeorm` dependency top-level

* fix: eslint jest error in CI

* chore: use prettier in VS Code

* fix: typescript-eslint setup

* chore: extract prettier config

* chore: update prettier config

* chore: move prettier config to top level

* chore: run prettier

* chore: remove prettier from eslitn

* fix: eslint errors

Author: chrispader

.prettierignore

@@ -0,0 +1,20 @@
+*.html
+*.yml
+*.yaml
+*.css
+*.scss
+*.md
+*.markdown
+
+package/lib
+
+.well-known
+android
+ios
+vendor
+package.json
+package-lock.json
+
+# The GH actions don't seem to compile and verify themselves well when Prettier is applied to them
+.github/actions/javascript/**/index.js
+

.prettierrc.js

@@ -0,0 +1,9 @@
+module.exports = {
+  quoteProps: 'consistent',
+  singleQuote: true,
+  tabWidth: 2,
+  trailingComma: 'all',
+  useTabs: false,
+  singleAttributePerLine: true,
+  semi: false,
+}

.watchmanconfig

@@ -1 +1 @@
-{}
+{}

config/.eslintrc.js

@@ -1,18 +1,22 @@
 module.exports = {
   root: true,
   ignorePatterns: [
-    "**/node_modules",
-    "**/lib",
-    "**/.eslintrc.js",
-    "**/.prettierrc.js",
-    "**/jest.config.js",
-    "**/babel.config.js",
-    "**/metro.config.js",
-    "**/react-native.config.js",
-    "**/tsconfig.json"
+    '**/node_modules',
+    '**/lib',
+    '**/.eslintrc.js',
+    '**/.prettierrc.js',
+    '**/jest.config.js',
+    '**/babel.config.js',
+    '**/metro.config.js',
+    '**/react-native.config.js',
+    '**/tsconfig.json',
   ],
-  extends: ['@react-native','plugin:prettier/recommended'],
-  plugins: ['@typescript-eslint', 'prettier'],
+  extends: [
+    '@react-native',
+    'plugin:prettier/recommended',
+    'plugin:@typescript-eslint/recommended',
+  ],
+  plugins: ['@typescript-eslint', 'jest'],
   parser: '@typescript-eslint/parser',
   parserOptions: {
     project: true,
@@ -23,17 +27,4 @@ module.exports = {
     ecmaVersion: 2018,
     sourceType: 'module',
   },
-  rules: {
-   'prettier/prettier': [
-      'warn',
-      {
-        quoteProps: 'consistent',
-        singleQuote: true,
-        tabWidth: 2,
-        trailingComma: 'es5',
-        useTabs: false,
-        semi: false,
-      },
-    ],
-  },
 }

example/.eslintrc.js

@@ -5,4 +5,4 @@ module.exports = {
     project: true,
     tsconfigRootDir: __dirname,
   },
-};
+}

example/.watchmanconfig

@@ -1 +1 @@
-{}
+{}

example/babel.config.js

@@ -1,5 +1,5 @@
-const path = require('path');
-const pak = require('../package/package.json');
+const path = require('path')
+const pak = require('../package/package.json')
 
 module.exports = {
   presets: ['module:@react-native/babel-preset'],
@@ -10,12 +10,12 @@ module.exports = {
         extensions: ['.tsx', '.ts', '.js', '.json'],
         alias: {
           [pak.name]: path.join(__dirname, '../package', pak.source),
-          stream: 'stream-browserify',
+          'stream': 'stream-browserify',
           'react-native-sqlite-storage': 'react-native-nitro-sqlite',
         },
       },
     ],
     'babel-plugin-transform-typescript-metadata',
-    ['@babel/plugin-proposal-decorators', {legacy: true}],
+    ['@babel/plugin-proposal-decorators', { legacy: true }],
   ],
-};
+}

example/metro.config.js

@@ -1,42 +1,42 @@
-const fs = require('fs');
-const path = require('path');
-const {getDefaultConfig, mergeConfig} = require('@react-native/metro-config');
-const exclusionList = require('metro-config/src/defaults/exclusionList');
-const escape = require('escape-string-regexp');
+const fs = require('fs')
+const path = require('path')
+const { getDefaultConfig, mergeConfig } = require('@react-native/metro-config')
+const exclusionList = require('metro-config/src/defaults/exclusionList')
+const escape = require('escape-string-regexp')
 
-const root = path.resolve(__dirname, '..');
-const rootNodeModulesPath = path.join(root, 'node_modules');
-const exampleNodeModulesPath = path.join(__dirname, 'node_modules');
+const root = path.resolve(__dirname, '..')
+const rootNodeModulesPath = path.join(root, 'node_modules')
+const exampleNodeModulesPath = path.join(__dirname, 'node_modules')
 
 function getPackageNames(nodeModulesPath) {
   if (!fs.existsSync(nodeModulesPath)) {
-    return [];
+    return []
   }
 
-  const allFiles = fs.readdirSync(nodeModulesPath);
+  const allFiles = fs.readdirSync(nodeModulesPath)
 
   // Filter out only directories (package names)
-  const packageNames = allFiles.filter(file => {
-    const filePath = path.join(nodeModulesPath, file);
-    return fs.statSync(filePath).isDirectory();
-  });
+  const packageNames = allFiles.filter((file) => {
+    const filePath = path.join(nodeModulesPath, file)
+    return fs.statSync(filePath).isDirectory()
+  })
 
   // Handle scoped packages (e.g., @scope/package)
   const scopedPackages = packageNames
-    .filter(pkg => pkg.startsWith('@'))
-    .flatMap(scope => {
-      const scopePath = path.join(nodeModulesPath, scope);
-      const scopedFiles = fs.readdirSync(scopePath);
-      return scopedFiles.map(scopedFile => `${scope}/${scopedFile}`);
-    });
+    .filter((pkg) => pkg.startsWith('@'))
+    .flatMap((scope) => {
+      const scopePath = path.join(nodeModulesPath, scope)
+      const scopedFiles = fs.readdirSync(scopePath)
+      return scopedFiles.map((scopedFile) => `${scope}/${scopedFile}`)
+    })
 
   // Return both regular and scoped package names
   return packageNames
-    .filter(pkg => !pkg.startsWith('@'))
-    .concat(scopedPackages);
+    .filter((pkg) => !pkg.startsWith('@'))
+    .concat(scopedPackages)
 }
 
-const exampleNodeModules = getPackageNames(exampleNodeModulesPath);
+const exampleNodeModules = getPackageNames(exampleNodeModulesPath)
 
 const config = {
   projectRoot: __dirname,
@@ -48,7 +48,8 @@ const config = {
     unstable_enableSymlinks: true,
     blockList: exclusionList(
       exampleNodeModules.map(
-        m => new RegExp(`^${escape(path.join(rootNodeModulesPath, m))}\\/.*$`),
+        (m) =>
+          new RegExp(`^${escape(path.join(rootNodeModulesPath, m))}\\/.*$`),
       ),
     ),
 
@@ -70,6 +71,6 @@ const config = {
       },
     }),
   },
-};
+}
 
-module.exports = mergeConfig(getDefaultConfig(__dirname), config);
+module.exports = mergeConfig(getDefaultConfig(__dirname), config)

example/react-native.config.js

@@ -1,10 +1,10 @@
-const path = require('path');
-const pak = require('../package/package.json');
+const path = require('path')
+const pak = require('../package/package.json')
 
 module.exports = {
   dependencies: {
     [pak.name]: {
       root: path.join(__dirname, '../package'),
     },
   },
-};
+}

example/src/App.tsx

@@ -11,9 +11,18 @@ export default function App() {
   return (
     <NavigationContainer>
       <RootStack.Navigator initialRouteName="NitroSQLite Example">
-        <RootStack.Screen name="NitroSQLite Example" component={HomeScreen} />
-        <RootStack.Screen name="Unit Tests" component={UnitTestScreen} />
-        <RootStack.Screen name="Benchmarks" component={BenchmarkScreen} />
+        <RootStack.Screen
+          name="NitroSQLite Example"
+          component={HomeScreen}
+        />
+        <RootStack.Screen
+          name="Unit Tests"
+          component={UnitTestScreen}
+        />
+        <RootStack.Screen
+          name="Benchmarks"
+          component={BenchmarkScreen}
+        />
       </RootStack.Navigator>
 
       <StatusBar style="auto" />