Repository navigation
(janitor/dead-code): delete unreferenced ComingSoonPlaceholder component - #8550
Conversation
chore(janitor): delete unreferenced ComingSoonPlaceholder component apps/web/src/app/(app)/wasteland/[owner]/[repo]/_components/ComingSoonPlaceholder.tsx exports ComingSoonPlaceholder, which is not imported anywhere in the repository. The wasteland [owner]/[repo] fork, pulls, and settings routes it scaffolded now render their own clients, so the placeholder is dead. The file is not referenced by any route, barrel, test, config, or string. Verified: no direct, dynamic, configured, or external references.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Executive SummaryPure deletion of the unreferenced Assumptions: Files Reviewed (1 file)
Reviewed by deepseek-flash-latest · Input: 49.9K · Output: 1.4K · Cached: 137K Review guidance: REVIEW.md from base branch |
|
Meta Janitor: Safety review passed; requesting approval. |
There was a problem hiding this comment.
Meta-janitor safety approval for 39468d2. Signal: #8550 (comment). GitHub auto-merge remains responsible for merging.
Summary
Deletes the unreferenced
ComingSoonPlaceholdercomponent atapps/web/src/app/(app)/wasteland/[owner]/[repo]/_components/ComingSoonPlaceholder.tsx.It is a
'use client'empty-state component that scaffolded the per-wasteland[owner]/[repo]routes (fork, pulls, settings) while they were still unimplemented.Those routes now render their own clients, so the placeholder is dead.
Verification
No manual browser verification was performed: this is a pure deletion of an
unreferenced component with no UI change. Automated checks (typecheck and lint)
pass before and after the change; see Reviewer Notes.
Visual Changes
N/A
Reviewer Notes
Evidence of unreachability. A repository-wide search for
ComingSoonPlaceholderacross
apps/,packages/,services/,scripts/, anddev/, plus theKilo-Org/kilocode external consumer, returns only the component's own definition.
There is no import, re-export barrel, test, or string reference to it.
Dynamic and configured entry points checked. The
[owner]/[repo]page.tsxrenders
UpstreamBoardClientandlayout.tsxrendersRepoLayoutShell, not thiscomponent. The
_components/directory has noindex.tsbarrel, and no tRPC router,dynamic
import(), Jest/Playwright setup, or config references the file. Its onlyimport is the external
lucide-reactpackage, so no internal module becomesorphaned by the deletion.
Contract and compatibility considerations. The file is not a public or package
exportsentry, is not referenced by any SDK/CLI/plugin surface, and is not amigration, persisted-data shape, or Cloudflare binding target. No production
behavior changes.
Validation performed.
tsgo --noEmitforapps/webpasses before and after thedeletion (exit 0), and
oxlinton the wasteland_components/directory reports0 warnings and 0 errors.