Comfy-Org / Comfy-Org/ComfyUI_frontend

Dead test fixtures and unused computed properties

Open
#11,057 2 comments 0 reactions 0 assignees View on GitHub
area:testing audit:dead-code code-audit
Dominant language
TypeScript
Stars
2k
Forks
702
Avg merge
1d 8h
Merged PRs (30d)
512

Description

## Recommendation

### 🗑️ SAFE TO REMOVE (after verification)

Dead test utilities and unused computed properties.

## What Was Found

### Dead Test Fixtures
- `src/lib/litegraph/src/__fixtures__/assets/testGraphs.ts` — not imported by any file (including test files). Sibling `testExtensions.ts` is used but `testGraphs.ts` is not.
- `src/utils/__tests__/litegraphTestUtils.ts` — never imported by any test file. Orphaned test helper from a refactor.

### Unused Computed Property
- `src/stores/aboutPanelStore.ts` — `coreVersion` computed property is defined but never accessed externally. (`coreBadges` is consumed internally via `allBadges`→`badges`, so it's fine.)

## If Removing (⚠️ proceed with caution)

Before removing, verify:
- [ ] No dynamic imports reference these files
- [ ] Check git blame — confirm they're not recently added WIP
- [ ] Search for string-based references

---

**Category:** Dead Code | **Confidence:** HIGH | **Tools:** Semantic analysis
_Filed by audit-code skill_

Part of #11022

┆Issue is synchronized with this [Notion page](https://www.notion.so/Issue-11057-Dead-test-fixtures-and-unused-computed-properties-33e6d73d36508144b9deebf7263fe73b) by [Unito](https://www.unito.io)

Contributor guide

Open the contributing guide

Research direction

Start with the three paths named in the issue: src/lib/litegraph/src/__fixtures__/assets/testGraphs.ts, src/utils/__tests__/litegraphTestUtils.ts, and src/stores/aboutPanelStore.ts. Search for dynamic and string-based references, then check git blame before removing the two unused files and coreVersion. Done means the verification finds no references and the remaining test utilities and store behavior are unaffected.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend, testing, tooling
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.