Skip to content

chore(frontend): remove dead boilerplate and unused design assets - #83

Merged
OthmaneZ05 merged 1 commit into
mainfrom
fix/d4-dead-assets-cleanup
Jul 22, 2026
Merged

chore(frontend): remove dead boilerplate and unused design assets#83
OthmaneZ05 merged 1 commit into
mainfrom
fix/d4-dead-assets-cleanup

Conversation

@OthmaneZ05

Copy link
Copy Markdown
Collaborator

Summary of Changes

Removes dead frontend assets: the default Vite scaffold files (react.svg, vite.svg), the orphaned hero.png, and two additional unreferenced leftovers found during the sweep — public/icons.svg (a generic social-icon sprite: bluesky/discord/x/github/documentation, never wired into the UI) and public/favicon.svg (an old logo concept that no longer matches the current logo/favicon.png and isn't linked from index.html).

No source vector file for the current logo (src/assets/logo.png) exists anywhere in the repo (no .ai/.fig/.eps/design folder), so no SVG variant was added — flagging this per the task's acceptance criteria instead of guessing at a re-trace.

Types of Changes

  • New feature / node type addition
  • Bug fix (non-breaking change resolving an issue)
  • Refactoring / structural cleanup
  • Documentation update

Verification & Testing

Automated Checks

  • Run npm run lint successfully with no errors
  • Run npm run build successfully with no compilation errors
  • Run npm test successfully (all tests pass)

Manual Verification

Confirmed each removed file has zero references across the repo (grep for filename in *.ts/*.tsx/*.js/*.html/*.json/*.css/*.md, excluding node_modules/dist) before deleting. npm run build produces a clean dist/ with no missing-asset warnings; npx vitest run --maxWorkers=1 → 231/231 tests pass across 21 files.

Checklist

  • My code follows the repository's code style and lint standards
  • I have updated the documentation or instructions if necessary
  • All unit and integration tests are passing

@OthmaneZ05
OthmaneZ05 requested a review from Derssa as a code owner July 22, 2026 18:59
@OthmaneZ05
OthmaneZ05 merged commit 40827e8 into main Jul 22, 2026
2 checks passed
@OthmaneZ05
OthmaneZ05 deleted the fix/d4-dead-assets-cleanup branch July 22, 2026 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant