Merge pull request #2 from TryQuiet/feat/error-handling-improvements

feat(2883): Error handling improvements

Author: islathehut

.github/workflows/api-publish.yml

@@ -24,6 +24,7 @@ jobs:
           node-version: 'lts/*'
           registry-url: https://registry.npmjs.org/
       - run: npm ci
+      - run: npm run lint
       - run: npm run build:docs
       - uses: actions/upload-pages-artifact@v3
         with:

.github/workflows/npm-publish.yml

@@ -16,6 +16,7 @@ jobs:
           node-version: 'lts/*'
           registry-url: https://registry.npmjs.org/
       - run: npm ci
+      - run: npm run lint
       - run: npm test
       - run: npm publish
         env:

.github/workflows/run-linter.yml

@@ -0,0 +1,18 @@
+---
+name: Run Linter
+
+on: [push, pull_request]
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 'lts/*'
+          registry-url: https://registry.npmjs.org/
+      - name: Install dependencies
+        run: npm ci
+      - name: Run linter
+        run: npm run lint

.github/workflows/run-test-node-min-ver.yml

@@ -14,8 +14,6 @@ jobs:
           registry-url: https://registry.npmjs.org/
       - name: Install dependencies
         run: npm ci
-      - name: Run linter
-        run: npm run lint
       - name: Run tests
         run: npm run test:ci
   test-browser:
@@ -28,8 +26,6 @@ jobs:
           registry-url: https://registry.npmjs.org/
       - name: Install dependencies
         run: npm ci
-      - name: Run linter
-        run: npm run lint
       - name: Run a webrtc relay in the background
         run: npm run webrtc:background
       - name: Run browser tests

.github/workflows/run-test.yml

@@ -14,8 +14,6 @@ jobs:
           registry-url: https://registry.npmjs.org/
       - name: Install dependencies
         run: npm ci
-      - name: Run linter
-        run: npm run lint
       - name: Run tests
         run: npm run test:ci
   test-browser:
@@ -28,8 +26,6 @@ jobs:
           registry-url: https://registry.npmjs.org/
       - name: Install dependencies
         run: npm ci
-      - name: Run linter
-        run: npm run lint
       - name: Run a webrtc relay in the background
         run: npm run webrtc:background
       - name: Run browser tests

.gitignore

@@ -6,6 +6,10 @@ test/ipfs/
 test/browser/ipfs/
 test/browser/bundle.js*
 
+# Don't track test databases
+test-data/
+benchmark-data/
+
 # Don't track builds in git
 # Distribution builds are available via npm
 dist/

benchmarks/log-append.js

@@ -28,7 +28,7 @@ const queryLoop = async () => {
 
   console.log('Benchmark duration is ' + benchmarkDuration + ' seconds')
 
-  await rmrf('./orbitdb')
+  await rmrf('./benchmarks/test-data/orbitdb')
 
   const identities = await Identities()
   const testIdentity = await identities.createIdentity({ id: 'userA' })
@@ -59,7 +59,7 @@ const queryLoop = async () => {
     }
     if (seconds >= benchmarkDuration) {
       clearInterval(interval)
-      await rmrf('./orbitdb')
+      await rmrf('./benchmarks/test-data/orbitdb')
       process.exit(0)
     }
     console.log(`${queriesPerSecond} queries per second, ${totalQueries} queries in ${seconds} seconds`)

benchmarks/log-iterator.js

@@ -6,7 +6,7 @@ import { rimraf as rmrf } from 'rimraf'
 ;(async () => {
   console.log('Starting benchmark...')
 
-  await rmrf('./orbitdb')
+  await rmrf('./benchmarks/test-data/orbitdb')
 
   const identities = await Identities()
   const testIdentity = await identities.createIdentity({ id: 'userA' })
@@ -55,7 +55,7 @@ import { rimraf as rmrf } from 'rimraf'
 
   console.log(`Iterating ${all.length} entries took ${duration2} ms, ${operationsPerSecond2} ops/s, ${millisecondsPerOp2} ms/op`)
 
-  await rmrf('./orbitdb')
+  await rmrf('./benchmarks/test-data/orbitdb')
 
   process.exit(0)
 })()

benchmarks/orbitdb-documents.js

@@ -11,8 +11,8 @@ EventEmitter.defaultMaxListeners = 10000
 
   const entryCount = 1000
 
-  await rmrf('./ipfs')
-  await rmrf('./orbitdb')
+  await rmrf('./benchmarks/test-data/ipfs')
+  await rmrf('./benchmarks/test-data/orbitdb')
 
   const ipfs = await createHelia()
   const orbitdb = await createOrbitDB({ ipfs })
@@ -61,8 +61,8 @@ EventEmitter.defaultMaxListeners = 10000
   await orbitdb.stop()
   await ipfs.stop()
 
-  await rmrf('./ipfs')
-  await rmrf('./orbitdb')
+  await rmrf('./benchmarks/test-data/ipfs')
+  await rmrf('./benchmarks/test-data/orbitdb')
 
   process.exit(0)
 })()

benchmarks/orbitdb-events.js

@@ -10,8 +10,8 @@ EventEmitter.defaultMaxListeners = 10000
 
   const entryCount = 1000
 
-  await rmrf('./ipfs')
-  await rmrf('./orbitdb')
+  await rmrf('./benchmarks/test-data/ipfs')
+  await rmrf('./benchmarks/test-data/orbitdb')
 
   const ipfs = await createHelia()
   const orbitdb = await createOrbitDB({ ipfs })
@@ -53,8 +53,8 @@ EventEmitter.defaultMaxListeners = 10000
   await orbitdb.stop()
   await ipfs.stop()
 
-  await rmrf('./ipfs')
-  await rmrf('./orbitdb')
+  await rmrf('./benchmarks/test-data/ipfs')
+  await rmrf('./benchmarks/test-data/orbitdb')
 
   process.exit(0)
 })()