chore: use the same noop everywhere (#511)

This minor cleanup uses the same `noop` utility function across the
codebase, rather than defining them as one-offs.

Co-authored-by: Gregor MacLennan <gmaclennan@digital-democracy.org>

src/datatype/index.js

@@ -3,7 +3,7 @@ import { validate } from '@mapeo/schema'
 import { getTableConfig } from 'drizzle-orm/sqlite-core'
 import { eq, inArray, placeholder } from 'drizzle-orm'
 import { randomBytes } from 'node:crypto'
-import { deNullify } from '../utils.js'
+import { noop, deNullify } from '../utils.js'
 import crypto from 'hypercore-crypto'
 import { TypedEmitter } from 'tiny-typed-emitter'
 
@@ -291,5 +291,3 @@ export class DataType extends TypedEmitter {
     this.emit('updated-docs', updatedDocs)
   }
 }
-
-function noop() {}

src/discovery/local-discovery.js

@@ -2,6 +2,7 @@ import { TypedEmitter } from 'tiny-typed-emitter'
 import net from 'node:net'
 import NoiseSecretStream from '@hyperswarm/secret-stream'
 import { once } from 'node:events'
+import { noop } from '../utils.js'
 import { DnsSd } from './dns-sd.js'
 import { isPrivate } from 'bogon'
 import StartStopStateMachine from 'start-stop-state-machine'
@@ -299,5 +300,3 @@ function getAddress(server) {
   }
   return addr
 }
-
-function noop() {}

src/local-peers.js

@@ -1,7 +1,7 @@
 // @ts-check
 import { TypedEmitter } from 'tiny-typed-emitter'
 import Protomux from 'protomux'
-import { assert, ExhaustivenessError, keyToId } from './utils.js'
+import { assert, ExhaustivenessError, keyToId, noop } from './utils.js'
 import { isBlank } from './lib/string.js'
 import cenc from 'compact-encoding'
 import {
@@ -626,8 +626,6 @@ function parseProjectJoinDetails(data) {
   return result
 }
 
-function noop() {}
-
 /**
  * We can temporarily have more than 1 peer for a device while connections are
  * deduplicating. We don't expose these duplicate connections until only one