Merge pull request #202 from pavlovcik/fix/knip

ubiquity · Mar 20, 2024 · d589f16 · d589f16
2 parents 9504caa + 3ce0d43
commit d589f16
Show file tree

Hide file tree

Showing 12 changed files with 47 additions and 352 deletions.
diff --git a/.knip.jsonc b/.knip.jsonc
@@ -0,0 +1,5 @@
+{
+  "$schema": "https://unpkg.com/knip@5/schema.json",
+  "entry": ["static/scripts/rewards/init.ts"],
+  "ignore": ["lib/**"]
+}
diff --git a/build/plugins/invert-colors.ts b/build/plugins/invert-colors.ts
diff --git a/package.json b/package.json
@@ -33,22 +33,13 @@
   ],
   "dependencies": {
     "@ethersproject/providers": "^5.7.2",
-    "@octokit/core": "^5.1.0",
-    "@octokit/plugin-create-or-update-text-file": "^4.0.1",
-    "@octokit/plugin-throttling": "^8.1.3",
-    "@octokit/rest": "^20.0.2",
-    "@sinclair/typebox": "^0.32.14",
-    "@supabase/supabase-js": "2.39.7",
-    "@types/libsodium-wrappers": "^0.7.13",
+    "@sinclair/typebox": "^0.32.15",
+    "@supabase/supabase-js": "^2.39.8",
     "@uniswap/permit2-sdk": "^1.2.0",
     "axios": "^1.6.7",
     "dotenv": "^16.4.4",
     "ethers": "^5.7.2",
-    "godb": "^0.6.2",
-    "libsodium-wrappers": "^0.7.13",
-    "npm-run-all": "^4.1.5",
-    "retry-axios": "^3.1.3",
-    "yaml": "^2.3.4"
+    "npm-run-all": "^4.1.5"
   },
   "devDependencies": {
     "@commitlint/cli": "^18.6.1",
@@ -61,11 +52,8 @@
     "@typescript-eslint/eslint-plugin": "^7.0.1",
     "@typescript-eslint/parser": "^7.0.1",
     "cspell": "^8.3.2",
-    "cspell-dict-html": "^1.2.5",
     "esbuild": "^0.20.0",
     "eslint": "^8.56.0",
-    "eslint-config-prettier": "^9.1.0",
-    "eslint-plugin-prettier": "^5.1.3",
     "eslint-plugin-sonarjs": "^0.24.0",
     "husky": "^9.0.11",
     "knip": "^5.0.1",
@@ -74,8 +62,7 @@
     "npm-run-all": "^4.1.5",
     "prettier": "^3.2.5",
     "tsx": "^4.7.1",
-    "typescript": "^5.3.3",
-    "yarn-upgrade-all": "^0.7.2"
+    "typescript": "^5.3.3"
   },
   "lint-staged": {
     "*.ts": [

diff --git a/scripts/typescript/utils.ts b/scripts/typescript/utils.ts
@@ -40,7 +40,7 @@ const colors = {
   bgCyan: "\x1b[46m",
   bgWhite: "\x1b[47m",
 };
-export function colorizeText(text: string, color: keyof typeof colors): string {
+function colorizeText(text: string, color: keyof typeof colors): string {
   return colors[color].concat(text).concat(colors.reset);
 }
 

diff --git a/static/scripts/rewards/constants.ts b/static/scripts/rewards/constants.ts
@@ -4,19 +4,19 @@
 
 declare const extraRpcs: Record<string, string[]>; // @DEV: passed in at build time check build/esbuild-build.ts
 
-export enum NetworkIds {
+enum NetworkIds {
   Mainnet = 1,
   Goerli = 5,
   Gnosis = 100,
   Anvil = 31337,
 }
 
-export enum Tokens {
+enum Tokens {
   DAI = "0x6b175474e89094c44da98b954eedeac495271d0f",
   WXDAI = "0xe91d153e0b41518a2ce8dd3d7944fa863463a97d",
 }
 
-export const networkNames = {
+const networkNames = {
   [NetworkIds.Mainnet]: "Ethereum Mainnet",
   [NetworkIds.Goerli]: "Goerli Testnet",
   [NetworkIds.Gnosis]: "Gnosis Chain",
@@ -53,4 +53,4 @@ export const networkRpcs: Record<number, string[]> = {
 };
 
 export const permit2Address = "0x000000000022D473030F116dDEE9F6B43aC78BA3";
-export const nftAddress = "0xAa1bfC0e51969415d64d6dE74f27CDa0587e645b";
+const nftAddress = "0xAa1bfC0e51969415d64d6dE74f27CDa0587e645b";
diff --git a/static/scripts/rewards/render-transaction/claim-rewards-pagination.ts b/static/scripts/rewards/render-transaction/claim-rewards-pagination.ts
@@ -13,7 +13,7 @@ export function claimRewardsPagination(rewardsCount: HTMLElement) {
   if (prevTxButton) prevTxButton.addEventListener("click", () => transactionHandler("previous"));
 }
 
-export function transactionHandler(direction: "next" | "previous") {
+function transactionHandler(direction: "next" | "previous") {
   removeAllEventListeners(getMakeClaimButton()) as HTMLButtonElement;
   direction === "next" ? app.nextPermit() : app.previousPermit();
   table.setAttribute(`data-make-claim`, "error");

diff --git a/static/scripts/rewards/render-transaction/insert-table-data.ts b/static/scripts/rewards/render-transaction/insert-table-data.ts
@@ -2,7 +2,7 @@ import { BigNumber, ethers } from "ethers";
 import { AppState, app } from "../app-state";
 import { Erc20Permit, Erc721Permit } from "./tx-type";
 
-export function shortenAddress(address: string): string {
+function shortenAddress(address: string): string {
   return `${address.slice(0, 10)}...${address.slice(-8)}`;
 }
 

diff --git a/static/scripts/rewards/render-transaction/read-claim-data-from-url.ts b/static/scripts/rewards/render-transaction/read-claim-data-from-url.ts
@@ -66,17 +66,10 @@ function decodeClaimData(base64encodedTxData: string): RewardPermit[] {
 
   try {
     permit = JSON.parse(atob(base64encodedTxData));
-  } catch (error) {
-    console.error(error);
-    setClaimMessage({ type: "Error", message: `1. Invalid claim data passed in URL` });
-    table.setAttribute(`data-make-claim`, "error");
-    throw error;
-  }
-  try {
     return [Value.Decode(Type.Array(claimTxT), permit)];
   } catch (error) {
     console.error(error);
-    setClaimMessage({ type: "Error", message: `2. Invalid claim data passed in URL` });
+    setClaimMessage({ type: "Error", message: `Invalid claim data passed in URL` });
     table.setAttribute(`data-make-claim`, "error");
     throw error;
   }

diff --git a/static/scripts/rewards/toaster.ts b/static/scripts/rewards/toaster.ts
@@ -11,15 +11,15 @@ export const toaster = {
   },
 };
 
-export const controls = document.getElementById("controls") as HTMLDivElement;
+const controls = document.getElementById("controls") as HTMLDivElement;
 export function getMakeClaimButton() {
   return document.getElementById("make-claim") as HTMLButtonElement;
 }
 export const viewClaimButton = document.getElementById("view-claim") as HTMLButtonElement;
-export const notifications = document.querySelector(".notifications") as HTMLUListElement;
+const notifications = document.querySelector(".notifications") as HTMLUListElement;
 export const buttonController = new ButtonController(controls);
 
-export function createToast(meaning: keyof typeof toaster.icons, text: string) {
+function createToast(meaning: keyof typeof toaster.icons, text: string) {
   if (meaning != "info") buttonController.hideLoader();
   const toastDetails = {
     timer: 5000,

diff --git a/static/scripts/rewards/web3/erc20-permit.ts b/static/scripts/rewards/web3/erc20-permit.ts
@@ -128,7 +128,7 @@ export function claimErc20PermitHandlerWrapper(app: AppState) {
   };
 }
 
-export async function checkPermitClaimable(app: AppState): Promise<boolean> {
+async function checkPermitClaimable(app: AppState): Promise<boolean> {
   let isClaimed: boolean;
   try {
     isClaimed = await isNonceClaimed(app);
@@ -248,7 +248,7 @@ invalidateButton.addEventListener("click", async function invalidateButtonClickH
 });
 
 //mimics https://github.com/Uniswap/permit2/blob/a7cd186948b44f9096a35035226d7d70b9e24eaf/src/SignatureTransfer.sol#L150
-export async function isNonceClaimed(app: AppState): Promise<boolean> {
+async function isNonceClaimed(app: AppState): Promise<boolean> {
   const provider = app.provider;
 
   const permit2Contract = new ethers.Contract(permit2Address, permit2Abi, provider);
@@ -266,7 +266,7 @@ export async function isNonceClaimed(app: AppState): Promise<boolean> {
   return bit.and(flipped).eq(0);
 }
 
-export async function invalidateNonce(signer: JsonRpcSigner, nonce: BigNumberish): Promise<void> {
+async function invalidateNonce(signer: JsonRpcSigner, nonce: BigNumberish): Promise<void> {
   const permit2Contract = new ethers.Contract(permit2Address, permit2Abi, signer);
   const { wordPos, bitPos } = nonceBitmap(nonce);
   // mimics https://github.com/ubiquity/pay.ubq.fi/blob/c9e7ed90718fe977fd9f348db27adf31d91d07fb/scripts/solidity/test/Permit2.t.sol#L428
@@ -277,15 +277,15 @@ export async function invalidateNonce(signer: JsonRpcSigner, nonce: BigNumberish
 }
 
 // mimics https://github.com/Uniswap/permit2/blob/db96e06278b78123970183d28f502217bef156f4/src/SignatureTransfer.sol#L142
-export function nonceBitmap(nonce: BigNumberish): { wordPos: BigNumber; bitPos: number } {
+function nonceBitmap(nonce: BigNumberish): { wordPos: BigNumber; bitPos: number } {
   // wordPos is the first 248 bits of the nonce
   const wordPos = BigNumber.from(nonce).shr(8);
   // bitPos is the last 8 bits of the nonce
   const bitPos = BigNumber.from(nonce).and(255).toNumber();
   return { wordPos, bitPos };
 }
 
-export async function updatePermitTxHash(app: AppState, hash: string): Promise<boolean> {
+async function updatePermitTxHash(app: AppState, hash: string): Promise<boolean> {
   const { error } = await supabase
     .from("permits")
     .update({ transaction: hash })

diff --git a/static/scripts/rewards/web3/erc721-permit.ts b/static/scripts/rewards/web3/erc721-permit.ts
@@ -62,7 +62,7 @@ export function claimErc721PermitHandler(reward: Erc721Permit) {
   };
 }
 
-export async function isNonceRedeemed(reward: Erc721Permit, provider: JsonRpcProvider): Promise<boolean> {
+async function isNonceRedeemed(reward: Erc721Permit, provider: JsonRpcProvider): Promise<boolean> {
   const nftContract = new ethers.Contract(reward.permit.permitted.token, nftRewardAbi, provider);
   return nftContract.nonceRedeemed(reward.request.nonce);
 }