Skip to content

refactor: make frontend icons canonical - #343

Merged
willchen96 merged 1 commit into
mainfrom
codex/move-icons-to-frontend
Aug 15, 2026
Merged

refactor: make frontend icons canonical#343
willchen96 merged 1 commit into
mainfrom
codex/move-icons-to-frontend

Conversation

@willchen96

Copy link
Copy Markdown
Collaborator

Summary

  • move the canonical SVG set into frontend/public/icons
  • remove the shared icon package and frontend copy script
  • point the Word add-in Webpack alias at the frontend public icon directory
  • remove icon sync hooks from frontend lifecycle commands

Why

Next.js already requires these assets under public to serve the existing /icons URLs. Keeping that directory canonical removes generated-file synchronization while preserving all frontend URLs and Word add-in imports.

Verification

  • Frontend icon-focused tests: 4 files, 18 tests passed
  • Word add-in application and E2E TypeScript checks passed
  • Word add-in Webpack development build passed and emitted 16 content-hashed SVG assets
  • git diff --check passed

The production Word wrapper was also invoked, but correctly stopped before Webpack because deployment environment variables were not supplied.

@willchen96
willchen96 merged commit a6ec113 into main Aug 15, 2026
13 checks passed
@willchen96
willchen96 deleted the codex/move-icons-to-frontend branch August 15, 2026 19:03
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