Found unused variables in TypeScript code JS-0356
Performance
Major
a month ago
—
2 months old
'signature' is assigned a value but never used
229 params: { data: fields.fields },
230 }
231
232 const signature = (await provider.request(webMessage)) as {233 data: string[]
234 publicKey: string
235 signature: string'currentNetworkInfo' is assigned a value but never used
88 // check if chainId has been set and not '...'
89 const chainId = result.current.getChainId()
90 const networkInfos = result.current.getNetworkInfo(defaultNetwork)
91 const currentNetworkInfo = result.current.getCurrentNetworkInfo() 92 expect(chainId).not.toEqual("...")
93
94 // check if first credential is in the store'networkInfos' is assigned a value but never used
87
88 // check if chainId has been set and not '...'
89 const chainId = result.current.getChainId()
90 const networkInfos = result.current.getNetworkInfo(defaultNetwork) 91 const currentNetworkInfo = result.current.getCurrentNetworkInfo()
92 expect(chainId).not.toEqual("...")
93'signedMessage' is assigned a value but never used
208 networkType: "testnet",
209 }
210 // sign payload
211 const signedMessage = await result.current.request(212 keyAgentName,
213 groupedCredential as GroupedCredentials,
214 message,'ImportMeta' is defined but never used
9 readonly VITE_APP_DEFAULT_NETWORK: string
10}
11
12interface ImportMeta {13 readonly env: ImportMetaEnv
14}
Description
Unused variables are generally considered a code smell and should be avoided.
Removing unused references - It prevents unused modules from being loaded at runtime, improving performance, and preventing the compiler from loading metadata that will never be used. - It prevents conflicts that may occur when trying to reference another variable.
NOTE: If you have intentionally left a variable unused, we suggest you to prefix the variable name with a _ to prevent them from being flagged by DeepSource.
Bad Practice
import fs from 'fs' // <- unused
import { readFileSync } from 'fs'
const text = readFileSync('declaration_of_independence.txt', 'utf-8')
console.log(text)
Recommended
import { readFileSync } from 'fs'
const text = readFileSync('declaration_of_independence.txt', 'utf-8')
console.log(text)