fix(logger): replace winston with bunyan

Winston does not play nice in web, and requires additional polyfills. bunyan is feature-par and is more suited for web environments

package.json

@@ -117,10 +117,9 @@
     "arbundles": "0.11.0",
     "arweave": "1.14.4",
     "axios": "1.4.0",
-    "setimmediate": "^1.0.5",
+    "bunyan": "^1.8.15",
     "warp-arbundles": "^1.0.4",
-    "warp-contracts": "1.4.39",
-    "winston": "^3.11.0"
+    "warp-contracts": "1.4.39"
   },
   "lint-staged": {
     "**/*.{ts,js,mjs,cjs,md,json}": [

src/common/arweave.ts

@@ -14,7 +14,7 @@
  * You should have received a copy of the GNU Affero General Public License
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
-import Arweave from 'arweave';
+import Arweave from 'arweave/node/index.js';
 
 export const defaultArweave = Arweave.init({
   host: 'ar-io.dev',

src/common/logger.ts

@@ -14,68 +14,47 @@
  * You should have received a copy of the GNU Affero General Public License
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
-// small polyfill for winston
-import 'setimmediate';
-import winston, { createLogger, format, transports } from 'winston';
+import bunyan from 'bunyan';
 
 import { Logger } from '../types.js';
 import { version } from '../version.js';
 
 export class DefaultLogger implements Logger {
-  private logger: winston.Logger;
+  private logger: bunyan.Logger;
   constructor({
     level = 'info',
-    logFormat = 'simple',
+    // logFormat = 'simple',
   }: {
     level?: 'info' | 'debug' | 'error' | 'none' | undefined;
-    logFormat?: 'simple' | 'json' | undefined;
+    // logFormat?: 'simple' | 'json' | undefined;
   } = {}) {
-    this.logger = createLogger({
+    this.logger = bunyan.createLogger({
       level,
-      defaultMeta: { client: 'ar-io-sdk', version },
-      silent: level === 'none',
-      format: getLogFormat(logFormat),
-      transports: [new transports.Console()],
+      name: 'ar-io-sdk',
+      version,
+      serializers: bunyan.stdSerializers,
     });
   }
 
   /* eslint-disable @typescript-eslint/no-explicit-any */
   info(message: string, ...args: any[]) {
-    this.logger.info(message, ...args);
+    this.logger.info(...args, message);
   }
 
   warn(message: string, ...args: any[]) {
-    this.logger.warn(message, ...args);
+    this.logger.warn(...args, message);
   }
 
   error(message: string, ...args: any[]) {
-    this.logger.error(message, ...args);
+    this.logger.error(...args, message);
   }
 
   debug(message: string, ...args: any[]) {
-    this.logger.debug(message, ...args);
+    this.logger.debug(...args, message);
   }
 
   setLogLevel(level: string) {
     this.logger.level = level;
   }
-
-  setLogFormat(logFormat: string) {
-    this.logger.format = getLogFormat(logFormat);
-  }
   /* eslint-enable @typescript-eslint/no-explicit-any */
 }
-
-function getLogFormat(logFormat: string) {
-  return format.combine(
-    format((info) => {
-      if (info.stack && info.level !== 'error') {
-        delete info.stack;
-      }
-      return info;
-    })(),
-    format.errors({ stack: true }), // Ensure errors show a stack trace
-    format.timestamp(),
-    logFormat === 'json' ? format.json() : format.simple(),
-  );
-}

src/node/index.ts

@@ -19,4 +19,3 @@ export { ArweaveSigner, ArconnectSigner } from 'arbundles';
 export * from '../types.js';
 export * from '../common/index.js';
 export * from '../constants.js';
-export * from '../utils/index.js';

src/web/index.ts

@@ -19,4 +19,3 @@ export { ArweaveSigner, ArconnectSigner } from 'arbundles';
 export * from '../types.js';
 export * from '../common/index.js';
 export * from '../constants.js';
-export * from '../utils/index.js';