narcello/findead

:mag: Dead react components finder

findead's Issues

Add tests

https://github.com/sstephenson/bats

Add license and changelog

False Positives

Problem

After successfully installing using yarn global add [email protected] (per #74), and running findead path/to/dir, I get an output that contains many files that I've verified are used in the project.

---

Possible Cause

Many of the exports in my project use the structure

- ComponentDirectory
    - Component.tsx
    - index.tsx

Where index.tsx looks like

export { default } from './Component'

Not working with npx

$ npx findead src
sh: 1: findead: not found

Enhance readme

enhance readme:

• gif
• emoji
• organize | refactor
• explain dead components
• badges
• yarn and npx samples

https://medium.com/@meakaakka/a-beginners-guide-to-writing-a-kickass-readme-7ac01da88ab3

Predicates when run comand

• Add argument for ignore folders to search components and to find usages
  • findead path/to/search/ -i path/to/ignore
• Add helper
  • findead --help | -h
• Use .gitignore option
  • -gi

Command not found after installing

Does this package have any dependency?
It doesn't seem to work in my setup 🤷‍♂️

➜  npm install -g findead
added 1 package, and audited 2 packages in 661ms
found 0 vulnerabilities
➜  findead
zsh: command not found: findead

Thanks!

Incompatible with OS X

The version of grep installed by default on OS X is BSD grep, not gnu grep, and does not support the -P flag used in the bash script. Mac users can install gnu grep using homebrew, but this installs it as ggrep and not just grep. If the script could detect the current OS and either use OS X-compatible commands (some examples at https://stackoverflow.com/questions/16658333/grep-p-no-longer-works-how-can-i-rewrite-my-searches) or rely on ggrep, that would make it work on OS X.

Doesn't work on Windows

findead.cmd invokes bash.exe

@IF EXIST "%~dp0\/bin/bash.exe" (
  "%~dp0\/bin/bash.exe"  "%~dp0\pnpm-global\3\node_modules\findead\findead.sh" %*
) ELSE (
  @SETLOCAL
  @SET PATHEXT=%PATHEXT:;.JS;=;%
  /bin/bash  "%~dp0\pnpm-global\3\node_modules\findead\findead.sh" %*
)

Add more datails

Log more details like:

• How much time are taking
• Spent on how many files
• Which step of whole process are now
• Show file path of dead component

Doesn't do anything

Hi!

I tried this tool with my project, but it didn't bring anything useful:

(develop *+$% u=)$ findead src/
==================================================================================================================== Findead is looking for components... =====================================================================================================================

=================================================================================================================================== Results ===================================================================================================================================
========================================================================================================================= No unused components found ==========================================================================================================================
================================================================== 1 browsed files in find: paths must precede expression: `jira.js'
find: possible unquoted pattern after predicate `-name'?
0,003 seconds ===================================================================

Remove sudo

Remove intern sudo and let final user type your own sudo

Enhance script time

Enhance script time maybe with parallel call functions

Check comments too

• Check imported commented for unused components too

Create site gatsby ghpages

Create site with gatsby and publish on ghpages

Wording

Findead are looking for components

should be "Findead is looking for components"

FindDead consider Lazy Loaded components as Dead/Unused components

Find dead see Error404.js file as dead component even if the file is imported inside route.js
{ path: '/errors/error-404', exact: true, component: lazy(() => import('./views/Errors/Error404')) },

Refactor code

• Refactor code
• Enhance var names