fix: Use Arrow functions to preserve 'this' from parent scope when passing functions as args

Author: thebongy

src/connection/socket-connection.ts

@@ -13,7 +13,9 @@ export default class SocketConnection extends BaseConnection {
 	setupConnection(): Promise<void> {
 		return new Promise(resolve => {
 			this.client = net.createConnection(this.sockPath);
-			this.client.on('data', this.onData);
+			this.client.on('data', (data) => {
+				this.onData(data);
+			});
 			this.client.on('connect', () => {
 				resolve();
 			});

src/gotham-node.ts

@@ -22,7 +22,7 @@ export default class GothamModule {
 	constructor(connection: BaseConnection, protocol: BaseProtocol) {
 		this.protocol = protocol;
 		this.connection = connection;
-		this.connection.setOnDataListener(this.onDataHandler);
+		// this.connection.setOnDataListener(this.onDataHandler);
 	}
 
 	public static default(socketPath: string): GothamModule {
@@ -36,7 +36,9 @@ export default class GothamModule {
 	) {
 		// Setup Connection only when initialize called?
 		await this.connection.setupConnection();
-		this.connection.setOnDataListener(this.onDataHandler);
+		this.connection.setOnDataListener((data) => {
+			this.onDataHandler(data);
+		});
 		return this.sendRequest(
 			this.protocol.initialize(
 				moduleId,