Skip to content

Reorganize repository directory structure and general clean up#1924

Merged
hjiangsu merged 5 commits intodevelopfrom
chore/repo-reorganization
Aug 12, 2025
Merged

Reorganize repository directory structure and general clean up#1924
hjiangsu merged 5 commits intodevelopfrom
chore/repo-reorganization

Conversation

@hjiangsu
Copy link
Member

Pull Request Description

This PR cleans up and reorganizes the repository to make it easier to navigate and maintain! I've reorganized it such that it now loosely follows a feature-first, clean architecture approach.

Here's a high-level summary of the changes that were made:

  • The lib directory now only contains l10n and src. All localization files are kept in l10n and l10n/generated.
  • The src directory has been reorganized into app, core, features, and shared directories:
    • app: contains the main entry point for Thunder.
    • core: contains core logic such as the database.
    • shared: contains shared widgets and functions that are used across features.
    • features: contains domain-specific widgets and logic.

Aside from the reorganization, there was also some cleanup done:

  • Cleaned up the settings pages and reduced unnecessary usages of SliverToBoxAdapter
  • Removed macos, linux, and windows directories since these platforms are not officially supported. They can be re-added anytime using: flutter create --platforms=windows,macos,linux .
    • Note: I've decided to keep the web platform even though its not officially supported as there was previous work done on it.

Issue Being Fixed

Issue Number: N/A

Screenshots / Recordings

Checklist

  • If a new package was added, did you ensure it uses an appropriate license and is actively maintained?
  • Did you use localized strings (and added appropriate descriptions) where applicable?
  • Did you add semanticLabels where applicable for accessibility?

@hjiangsu hjiangsu merged commit 533c6fc into develop Aug 12, 2025
1 check passed
@hjiangsu hjiangsu deleted the chore/repo-reorganization branch August 12, 2025 18:50
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