[PR #247] [MERGED] Add custom logger support #188

New issue

Closed

opened 2026-05-23 08:32:55 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-23 08:32:55 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/appy-one/acebase/pull/247
Author: @appy-one
Created: 10/9/2023
Status: ✅ Merged
Merged: 9/16/2025
Merged by: @appy-one

Base: master ← Head: feature/custom-logger-support

📝 Commits (5)

dd1a646 Add custom logger support
098adb4 Add no-invalid-this eslint rule
eb53d25 test with pino logger
a9e8a04 Merge branch 'master' into feature/custom-logger-support
116f869 Remove Pino and custom logger from tests

📊 Changes

29 files changed (+3714 additions, -3654 deletions)

View changed files

📝 .eslintrc.json (+47 -46)
📝 package-lock.json (+17 -33)
📝 package.json (+1 -1)
📝 src/api-local.ts (+4 -2)
📝 src/btree/binary-tree.spec.ts (+365 -365)
📝 src/btree/binary-tree.ts (+69 -69)
📝 src/data-index/array-index.ts (+274 -274)
📝 src/data-index/data-index.ts (+32 -27)
📝 src/data-index/fulltext-index.ts (+1041 -1041)
📝 src/data-index/geo-index.ts (+284 -284)
📝 src/index.ts (+3 -0)
📝 src/ipc/browser.ts (+6 -6)
📝 src/ipc/index.ts (+137 -137)
📝 src/ipc/ipc.ts (+15 -12)
📝 src/ipc/remote.ts (+326 -326)
📝 src/ipc/service/index.ts (+113 -99)
📝 src/ipc/service/start.ts (+29 -27)
📝 src/ipc/socket.ts (+269 -262)
📝 src/node-lock.ts (+364 -366)
📝 src/query.ts (+10 -18)

...and 9 more files

📄 Description

This enables replacing AceBase's default logger with a logging library such as Bunyan, Winston, Pino and any other logger supporting trace, debug, info, warn, error and fatal methods.

This also adds the methods trace, debug, info and fatal to the default built-in logger.

Also see https://github.com/appy-one/acebase-core/pull/50

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/appy-one/acebase/pull/247 **Author:** [@appy-one](https://github.com/appy-one) **Created:** 10/9/2023 **Status:** ✅ Merged **Merged:** 9/16/2025 **Merged by:** [@appy-one](https://github.com/appy-one) **Base:** `master` ← **Head:** `feature/custom-logger-support` --- ### 📝 Commits (5) - [`dd1a646`](https://github.com/appy-one/acebase/commit/dd1a6462e57a7525d8c29279bfe9e2b5f64cf487) Add custom logger support - [`098adb4`](https://github.com/appy-one/acebase/commit/098adb4c831db28d1e1eea30fa6a23313cdf87fd) Add `no-invalid-this` eslint rule - [`eb53d25`](https://github.com/appy-one/acebase/commit/eb53d25ffb7624ff872c1917f6596a608e28b73c) test with pino logger - [`a9e8a04`](https://github.com/appy-one/acebase/commit/a9e8a04297e4e54f46f6019c1b48e9caaf3048eb) Merge branch 'master' into feature/custom-logger-support - [`116f869`](https://github.com/appy-one/acebase/commit/116f869aeacc6e75197a825d1d2232ab7069c93d) Remove Pino and custom logger from tests ### 📊 Changes **29 files changed** (+3714 additions, -3654 deletions) <details> <summary>View changed files</summary> 📝 `.eslintrc.json` (+47 -46) 📝 `package-lock.json` (+17 -33) 📝 `package.json` (+1 -1) 📝 `src/api-local.ts` (+4 -2) 📝 `src/btree/binary-tree.spec.ts` (+365 -365) 📝 `src/btree/binary-tree.ts` (+69 -69) 📝 `src/data-index/array-index.ts` (+274 -274) 📝 `src/data-index/data-index.ts` (+32 -27) 📝 `src/data-index/fulltext-index.ts` (+1041 -1041) 📝 `src/data-index/geo-index.ts` (+284 -284) 📝 `src/index.ts` (+3 -0) 📝 `src/ipc/browser.ts` (+6 -6) 📝 `src/ipc/index.ts` (+137 -137) 📝 `src/ipc/ipc.ts` (+15 -12) 📝 `src/ipc/remote.ts` (+326 -326) 📝 `src/ipc/service/index.ts` (+113 -99) 📝 `src/ipc/service/start.ts` (+29 -27) 📝 `src/ipc/socket.ts` (+269 -262) 📝 `src/node-lock.ts` (+364 -366) 📝 `src/query.ts` (+10 -18) _...and 9 more files_ </details> ### 📄 Description This enables replacing AceBase's default logger with a logging library such as Bunyan, Winston, Pino and any other logger supporting `trace`, `debug`, `info`, `warn`, `error` and `fatal` methods. This also adds the methods `trace`, `debug`, `info` and `fatal` to the default built-in logger. Also see https://github.com/appy-one/acebase-core/pull/50 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>