Comfy-Org / Comfy-Org/ComfyUI_frontend
Dead test fixtures and unused computed properties
- 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
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