AOSSIE-Org / AOSSIE-Org/OrgExplorer

[FEATURE]: Implement Code Splitting and Dynamic Routing to optimize Vite bundle size

Aberta Para iniciantes
#140 2 comentários 0 reações 0 responsáveis Ver no GitHub
enhancement
Linguagem predominante
JavaScript
Estrelas
34
Forks
92
Merge médio
8d 7h
PRs com merge (30d)
12

Descrição

### Feature and its Use Cases

### Description
Currently, `OrgExplorer` eagerly imports all page components inside `App.jsx` (e.g., `HomePage`, `OverviewPage`, `ContributorProfilePage`, etc.). Because of this, Vite bundles the entire application into a single massive JavaScript chunk.

If you run `npm run build`, Vite actually throws a warning about this:
> `(!) Some chunks are larger than 500 kB after minification. Consider using dynamic import() to code-split the application.`

By implementing React's `lazy()` and `Suspense` for our route components, we can split the application into smaller chunks. This will drastically reduce the initial load time, as users will only download the JavaScript for the specific page they are visiting (e.g., the Home Page) rather than downloading the code for the analytics dashboards and charts before they even need them.

### Expected Behavior
- Replace static imports in `App.jsx` with `React.lazy(() => import('./pages/...'))`.
- Wrap the `` (or individual routes) in a `` boundary with a fallback loading indicator (we can reuse the existing `Spinner` component).
- Running `npm run build` should no longer throw the >500kB chunk size warning, and the build output should show multiple smaller `.js` chunks instead of one giant one.

### Impact
High - Directly improves the initial page load speed and core web vitals for all users.

### Additional Context

_No response_

### Code of Conduct

- [x] I have joined the [Discord server](https://discord.gg/hjUhu33uAn) and will post updates there
- [x] I have searched existing issues to avoid duplicates
- [x] I am ready to open a pr on this issue

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Start in App.jsx, where HomePage, OverviewPage, ContributorProfilePage and other route components are currently imported eagerly. Replace those route imports with React.lazy imports and add a Suspense boundary using the existing Spinner component as the fallback. Run npm run build before and after; done means the >500kB chunk warning is gone and the output shows multiple smaller JavaScript chunks.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
javascript, react, vite
Domínio
frontend, performance
Tipo de issue
Funcionalidade
Dificuldade
2/5
Tempo estimado
1-3 horas
Status de atividade
Ativa
Clareza
Claramente especificada
Facilidade para iniciantes
78/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.