Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Investigate adding fallow for dead code and codebase health analysis

Ouverte
  • Quelque chose ne va pas ?
#883 1 commentaire 0 réactions 1 personne assignée Voir sur GitHub

Les mainteneurs répondent en général sous 2 jours

@EhabY y travaille déjà.

Depuis le 15/4/2026.

Évaluation

Cette issue n'a pas encore été évaluée.

Description

Summary

fallow is a Rust-based codebase analysis tool that detects dead code, circular dependencies, code duplication, and complexity hotspots. It is much faster than Knip (5-41x in benchmarks) and works with zero configuration. It auto-detected our Electron + Vite + Vitest + TypeScript setup out of the box.

Trial run on this repo

I ran npx fallow against the current codebase. It analyzed 213 files in 0.34 seconds and found:

Dead code
  • 2 unused files -- .vscode-test.mjs and test/mocks/vscode.runtime.ts
  • 5 unused exports -- getEventName, AgentMetadataEventSchema, REMOTE_SSH_EXTENSION_IDS, RemoteSSHLogPortRegex, MockOAuthInterceptor
  • 15 unused type exports -- across exec.ts, eventStreamConnection.ts, sshConfig.ts, remote.ts, and others
  • 119 unused class members -- mostly in test mock helpers (46 in testHelpers.ts alone), but also 8 in coderApi.ts and 8 in pathResolver.ts
Circular dependencies (2 cycles)
  • cliCredentialManager.ts -> cliExec.ts -> cli.ts -> cliCredentialManager.ts (11 files affected)
  • A shorter cycle through the same area
Code duplication
  • 73 clone groups detected, including repeated patterns in test setup and SSH config handling
Complexity hotspots
  • extension.ts:activate -- cognitive complexity 30
  • workspacesProvider.ts:getChildren -- cognitive complexity 37
  • remote.ts:setup -- cognitive complexity 32
Maintainability
  • Overall maintainability index: 91.0 (rated "good")

What would be useful

  1. Dead code cleanup -- the unused exports and types are low-hanging fruit. Fallow can auto-fix these with fallow fix --dry-run.
  2. CI integration -- fallow has a GitHub Action that can run on PRs with --since main to catch new dead code before it lands.
  3. Circular dependency tracking -- the two cycles through cliCredentialManager/cliExec/cli are flagged as the top refactoring targets.

Caveats

  • Fallow does syntactic analysis only (no TypeScript type resolution), so some "unused" class members may actually be consumed through interfaces or dynamic access. The test mock findings in particular will need filtering.
  • VS Code extension APIs use a lot of convention-based exports (e.g. activate/deactivate) that fallow already handles via its Electron plugin, but edge cases are possible.

Suggested next steps

  • Review the unused exports/types and clean up what is genuinely dead
  • Decide if fallow should be added as a dev dependency or just run ad-hoc
  • Consider adding a pnpm fallow script and/or CI check for new dead code on PRs
Langage dominant
TypeScript
Étoiles
132
Forks
48
Merge moyen
2 j 5 h
PR mergées (30 j)
19

Préparer son environnement

Ouvrir dans Codespaces

Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de coder/vscode-coder

Toutes les issues de coder/vscode-coder

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.