fix: get rid of redundant dependencies (#69)

Author: JackHamer09

.github/workflows/release.yml

@@ -22,10 +22,6 @@ jobs:
           registry-url: "https://registry.npmjs.org"
       - name: Install dependencies and build
         run: npm ci && npm run build
-      - name: Create .env-public-analytics file
-        run: |
-          echo "RUDDER_STACK_KEY=${{ secrets.RUDDER_STACK_KEY }}" > .env-public-analytics
-          echo "RUDDER_STACK_DATAPLANE_URL=${{ secrets.RUDDER_STACK_DATAPLANE_URL }}" >> .env-public-analytics
       - name: Release
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.npmignore

@@ -3,7 +3,6 @@
 
 # Include specific files
 !bin/**/*
-!.env-public-analytics
 !package.json
 !package-lock.json
 !LICENSE-MIT

README.md

@@ -81,10 +81,6 @@ If you're using [local setup (dockerized testing node)](https://github.com/matte
 At the moment, we don't have any tests, but we are working on it.
 In the meantime, you can test the code manually by running the code in [development mode](#run-in-development-mode).
 
-### 📊 Tracking
-
-zkSync-cli tracks its usage for the single purpose of providing data so it can be improved. Data is fully anonymized. If you want to disable the tracking, set the environment variable `NO_TRACKING` to `true`.
-
 ## 🌍 Official Links
 
 - [Website](https://zksync.io/)

package-lock.json

@@ -34,17 +34,13 @@
     "**/*.ts": "tsc-files --noEmit"
   },
   "dependencies": {
-    "@rudderstack/rudder-sdk-node": "^2.0.5",
     "chalk": "4.1.2",
     "commander": "^11.0.0",
     "compare-versions": "^6.1.0",
     "cross-env": "^7.0.3",
     "dotenv": "^16.3.1",
-    "eslint-import-resolver-typescript": "^3.6.1",
     "ethers": "5.7.2",
     "inquirer": "^8.1.4",
-    "node-machine-id": "^1.1.12",
-    "ofetch": "^1.3.3",
     "winston": "^3.10.0",
     "zkcli-dockerized-node": "^1.0.2",
     "zkcli-in-memory-node": "^1.0.3",

package.json

@@ -11,7 +11,6 @@ import {
   zeekOption,
 } from "../../common/options.js";
 import { l2Chains } from "../../data/chains.js";
-import { track } from "../../utils/analytics.js";
 import { ETH_TOKEN } from "../../utils/constants.js";
 import { bigNumberToDecimal, decimalToBigNumber } from "../../utils/formatters.js";
 import {
@@ -119,8 +118,6 @@ export const handler = async (options: DepositOptions) => {
       Logger.info(` Transaction link: ${fromChain.explorerUrl}/tx/${depositHandle.hash}`);
     }
 
-    track("deposit", { network: toChain?.network ?? "Unknown chain", zeek: options.zeek });
-
     const senderBalance = await l1Provider.getBalance(senderWallet.address);
     Logger.info(`\nSender L1 balance after transaction: ${bigNumberToDecimal(senderBalance)} ETH`);
 
@@ -130,7 +127,6 @@ export const handler = async (options: DepositOptions) => {
   } catch (error) {
     Logger.error("There was an error while depositing funds:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/bridge/deposit.ts

@@ -4,7 +4,6 @@ import inquirer from "inquirer";
 import Program from "./command.js";
 import { chainOption, l1RpcUrlOption, l2RpcUrlOption, privateKeyOption, zeekOption } from "../../common/options.js";
 import { l2Chains } from "../../data/chains.js";
-import { track } from "../../utils/analytics.js";
 import { bigNumberToDecimal } from "../../utils/formatters.js";
 import {
   getAddressFromPrivateKey,
@@ -117,8 +116,6 @@ export const handler = async (options: WithdrawFinalizeOptions) => {
     const receipt = await finalizationHandle.wait();
     Logger.info(` Finalization transaction was mined in block ${receipt.blockNumber}`);
 
-    track("confirm-withdraw", { network: toChain?.network ?? "Unknown chain", zeek: options.zeek });
-
     const senderBalance = await l1Provider.getBalance(senderWallet.address);
     Logger.info(`\nSender L1 balance after transaction: ${bigNumberToDecimal(senderBalance)} ETH`);
 
@@ -128,7 +125,6 @@ export const handler = async (options: WithdrawFinalizeOptions) => {
   } catch (error) {
     Logger.error("There was an error while finalizing withdrawal:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/bridge/withdraw-finalize.ts

@@ -11,7 +11,6 @@ import {
   zeekOption,
 } from "../../common/options.js";
 import { l2Chains } from "../../data/chains.js";
-import { track } from "../../utils/analytics.js";
 import { ETH_TOKEN } from "../../utils/constants.js";
 import { bigNumberToDecimal, decimalToBigNumber } from "../../utils/formatters.js";
 import {
@@ -119,8 +118,6 @@ export const handler = async (options: WithdrawOptions) => {
       Logger.info(` Transaction link: ${fromChain.explorerUrl}/tx/${withdrawHandle.hash}`);
     }
 
-    track("withdraw", { network: toChain?.network ?? "Unknown chain", zeek: options.zeek });
-
     const senderBalance = await l2Provider.getBalance(senderWallet.address);
     Logger.info(`\nSender L2 balance after transaction: ${bigNumberToDecimal(senderBalance)} ETH`);
 
@@ -130,7 +127,6 @@ export const handler = async (options: WithdrawOptions) => {
   } catch (error) {
     Logger.error("There was an error while withdrawing funds:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/bridge/withdraw.ts

@@ -6,7 +6,6 @@ import path from "path";
 
 import Program from "./command.js";
 import { zeekOption } from "../../common/options.js";
-import { track } from "../../utils/analytics.js";
 import { fileOrDirExists } from "../../utils/files.js";
 import { cloneRepo } from "../../utils/git.js";
 import { optionNameToParam, executeCommand } from "../../utils/helpers.js";
@@ -111,15 +110,12 @@ ${chalk.magentaBright("Further Reading:")}
 Check out the README file for more details: ${path.join(folderLocation, "README.md")}
 `);
 
-    track("create", { template: options.template, zeek: options.zeek });
-
     if (options.zeek) {
       zeek();
     }
   } catch (error) {
     Logger.error("There was an error while creating new project:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/create/create.ts

@@ -1,6 +1,5 @@
 import Program from "./command.js";
 import configHandler from "./ConfigHandler.js";
-import { track } from "../../utils/analytics.js";
 import Logger from "../../utils/logger.js";
 
 import type Module from "./modules/Module.js";
@@ -27,7 +26,6 @@ export const handler = async () => {
   } catch (error) {
     Logger.error("There was an error while cleaning the testing environment:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/dev/clean.ts

@@ -4,7 +4,6 @@ import inquirer from "inquirer";
 import Program from "./command.js";
 import configHandler from "./ConfigHandler.js";
 import { ModuleCategory } from "./modules/Module.js";
-import { track } from "../../utils/analytics.js";
 import Logger from "../../utils/logger.js";
 
 import type { ModuleNode } from "./modules/Module.js";
@@ -125,7 +124,6 @@ export const handler = async (options: LocalConfigOptions = {}) => {
   } catch (error) {
     Logger.error("There was an error while configuring the testing environment:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/dev/config.ts

@@ -3,7 +3,6 @@ import { Option } from "commander";
 
 import Program from "./command.js";
 import { modulesPath } from "./modules/Module.js";
-import { track } from "../../utils/analytics.js";
 import { executeCommand } from "../../utils/helpers.js";
 import Logger from "../../utils/logger.js";
 
@@ -25,7 +24,6 @@ export const handler = async (moduleNames: string[], options: { link: boolean })
   } catch (error) {
     Logger.error("There was an error while installing module:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/dev/install.ts

@@ -2,7 +2,6 @@ import chalk from "chalk";
 
 import Program from "./command.js";
 import configHandler from "./ConfigHandler.js";
-import { track } from "../../utils/analytics.js";
 import Logger from "../../utils/logger.js";
 
 export const handler = async () => {
@@ -34,7 +33,6 @@ export const handler = async () => {
   } catch (error) {
     Logger.error("There was an error displaying logs:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/dev/logs.ts

@@ -1,6 +1,5 @@
 import chalk from "chalk";
 
-import { track } from "../../../utils/analytics.js";
 import Logger from "../../../utils/logger.js";
 import Program from "../command.js";
 import configHandler from "../ConfigHandler.js";
@@ -30,7 +29,6 @@ export const handler = async () => {
   } catch (error) {
     Logger.error("There was an error displaying installed modules:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/dev/modules/index.ts

@@ -1,7 +1,6 @@
 import Program from "./command.js";
 import { handler as start } from "./start.js";
 import { handler as stop } from "./stop.js";
-import { track } from "../../utils/analytics.js";
 import Logger from "../../utils/logger.js";
 
 export const handler = async () => {
@@ -11,7 +10,6 @@ export const handler = async () => {
   } catch (error) {
     Logger.error("There was an error while restarting the testing environment:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/dev/restart.ts

@@ -5,7 +5,6 @@ import { setupConfig } from "./config.js";
 import configHandler from "./ConfigHandler.js";
 import { ModuleCategory } from "./modules/Module.js";
 import { getModulesRequiringUpdates } from "./modules/utils/updates.js";
-import { track } from "../../utils/analytics.js";
 import { formatLogs } from "../../utils/formatters.js";
 import Logger from "../../utils/logger.js";
 
@@ -111,7 +110,6 @@ export const handler = async () => {
   } catch (error) {
     Logger.error("There was an error while starting the testing environment:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/dev/start.ts

@@ -1,6 +1,5 @@
 import Program from "./command.js";
 import configHandler from "./ConfigHandler.js";
-import { track } from "../../utils/analytics.js";
 import Logger from "../../utils/logger.js";
 
 export const handler = async () => {
@@ -11,7 +10,6 @@ export const handler = async () => {
   } catch (error) {
     Logger.error("There was an error while stopping the testing environment:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/dev/stop.ts

@@ -5,7 +5,6 @@ import Program from "./command.js";
 import configHandler from "./ConfigHandler.js";
 import { modulesPath } from "./modules/Module.js";
 import { findDefaultModules } from "./modules/utils/packages.js";
-import { track } from "../../utils/analytics.js";
 import { executeCommand } from "../../utils/helpers.js";
 import Logger from "../../utils/logger.js";
 
@@ -34,7 +33,6 @@ export const handler = async (moduleNames: string[], options: { unlink: boolean
   } catch (error) {
     Logger.error("There was an error while uninstalling module:");
     Logger.error(error);
-    track("error", { error });
   }
 };
 

src/commands/dev/uninstall.ts

@@ -4,7 +4,6 @@ import { Option } from "commander";
 import Program from "./command.js";
 import configHandler from "./ConfigHandler.js";
 import { modulesPath } from "./modules/Module.js";
-import { track } from "../../utils/analytics.js";
 import { executeCommand } from "../../utils/helpers.js";
 import Logger from "../../utils/logger.js";
 
@@ -67,7 +66,6 @@ export const handler = async (moduleNames: string[], options: ModuleUpdateOption
   } catch (error) {
     Logger.error("There was an error while updating module:");
     Logger.error(error);
-    track("error", { error });
   }
 };