Skip to content

(janitor/dead-code): remove orphaned GrafanaLabs and RedHat logo components - #6790

Merged
kilo-code-bot[bot] merged 1 commit into
mainfrom
janitor/dead-code-orphaned-logo-assets
Sep 28, 2026
Merged

kilo-code-bot[bot] merged 1 commit into
mainfrom
janitor/dead-code-orphaned-logo-assets

Conversation

@kilo-code-bot

@kilo-code-bot kilo-code-bot Bot commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Removes two orphaned SVG logo components from apps/web/src/components/assets/:

  • GrafanaLabsLogo.tsx
  • RedHatLogo.tsx

What was deleted

Both are presentational React components (default-exported inline <svg> brand logos) with no imports, no props, no module-level side effects. Neither is rendered by LogosSection.tsx, which renders the other five assets/* logos (Meta, Amazon, Airbnb, PayPal, Square) from a hardcoded companies array.

Evidence of unreachability

  • GrafanaLogo (the default export of GrafanaLabsLogo.tsx) appears nowhere in the repository outside its own file.
  • RedHatLogo appears nowhere outside its own file.
  • Whole-repo, case-insensitive searches for grafana and redhat / red hat match only the files themselves plus unrelated observability content (KiloClaw Grafana dashboards, KILO_GRAFANA_PORT, etc.), which has no relationship to these components.
  • components/assets/ has no index.ts barrel, so no re-export path exists.
  • No .stories.tsx / .storybook reference, and no snapshot test references either symbol.
  • The Kilo-Org/kilocode external consumer repo contains no reference to either symbol.

Entry points and configuration checked

  • Next.js file conventions: neither file is a page / layout / route / loading / error / middleware / instrumentation convention file, so deletion does not change routing or file-based discovery.
  • No dynamic import(...) / require(...) reference.
  • No string reference in next.config, tsconfig paths, jest/vitest, tailwind, or any other config.
  • No Cloudflare bindings, Durable Objects, queues, cron triggers, alarms, RPC, workflows, or wrangler configuration reference (these are web UI components, not worker code).
  • No package exports, SDK API, CLI, plugin, or published-package contract surface (all packages/* are private: true; these files are internal to apps/web).
  • No test, script, CI, deployment, development-tooling, or example references.

Contract and compatibility considerations

  • Pure presentation components with zero consumers, no module-initialization side effects, no persisted data, and no migration / rollback / incomplete-rollout concerns.
  • No change to any public or published contract.

Validation

  • pnpm --filter web lint — 0 warnings, 0 errors.
  • pnpm --filter web typecheck — passes (exit 0).

Both SVG components are unreferenced anywhere in the repository.
LogosSection only renders Meta, Amazon, Airbnb, PayPal, and Square.
@kilo-code-bot kilo-code-bot Bot added the janitor Automated dead-code/duplication cleanup label Sep 28, 2026
@kilo-code-bot
kilo-code-bot Bot enabled auto-merge (squash) September 28, 2026 08:16
@kilo-code-bot

kilo-code-bot Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor Author

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Deletion-only cleanup of two unreferenced SVG logo components; no consumers, side effects, or contract changes were found, and the remaining assets/* logos are untouched.

Files Reviewed (2 files)
  • apps/web/src/components/assets/GrafanaLabsLogo.tsx (deleted)
  • apps/web/src/components/assets/RedHatLogo.tsx (deleted)
Verification Notes
  • Repo-wide search found no references to GrafanaLabsLogo/GrafanaLogo/RedHatLogo; remaining grafana matches are unrelated KiloClaw/o11y infra code and no redhat references exist.
  • apps/web/src/components/assets/ has no barrel index.ts; the directory now contains only the five logos rendered by LogosSection.tsx.
  • Pure presentational components with no module side effects or persisted data; no routing, worker, or config surface affected.

Reviewed by deepseek-v4.1-flash · Input: 0 · Output: 0 · Cached: 0

Review guidance: REVIEW.md from base branch main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

janitor Automated dead-code/duplication cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant