Skip to content

Conversation

@Junyi-99
Copy link
Member

@Junyi-99 Junyi-99 commented Dec 9, 2025

Refactor project structure by merging actor logic, renaming error handling, introducing a repository layer, reorganizing libraries, and renaming the webapp directory to improve clarity and maintainability.

This PR implements the specified project structure optimization plan, addressing several areas:

  • Authentication Logic: Consolidated api/actor into accesscontrol to centralize user actor parsing and resolve potential circular dependencies with a UserExistenceChecker function type.
  • Error Handling: Renamed the shared package to errors and updated all imports to use an apperrors alias, preventing conflicts with the standard library and improving semantic clarity.
  • Data Access Layer: Introduced a dedicated repository layer to abstract data access logic from services, enhancing modularity and testability.
  • Configuration Naming: Renamed the cfg library to config for clearer naming conventions.
  • Webapp Directory: Renamed webapp/_webapp to webapp/extension to better reflect its purpose as a browser extension.

Open in Cursor Open in Web

This commit centralizes error handling into a new `errors` package and renames `cfg` to `config` for better clarity. It also removes unused webapp files.

Co-authored-by: e0945797 <e0945797@u.nus.edu>
@cursor
Copy link

cursor bot commented Dec 9, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@Junyi-99 Junyi-99 closed this Dec 10, 2025
@Junyi-99 Junyi-99 deleted the cursor/optimize-project-structure-80ae branch January 7, 2026 14:57
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.

3 participants