Project Structure Refactoring #2
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Move coreLogic.js to core/providers/ - Move optimizely provider files to core/providers/ - Move event listeners to core/providers/events/ Part of the project restructure to improve code organization and maintainability.
- Move all CDN adapters to src/adapters/ - Move shared adapter helpers to src/adapters/base/ - Remove old cdn-adapters directory Part of the project restructure to improve code organization and maintainability.
- Move config files to utils/config/ - Move logger to utils/logging/ - Move abstraction helpers and classes to utils/ - Remove old _helpers_ and _config_ directories Part of the project restructure to improve code organization and maintainability.
- Move API router and handlers to src/core/api/ - Remove old _api_ directory Part of the project restructure to improve code organization and maintainability.
- Update import paths in abstraction classes - Update import paths in config files - Update import paths in logging - Update import paths in helpers Adjust imports to match new directory structure.
- Update import paths in all CDN adapters (Akamai, Cloudflare, CloudFront, Fastly, Vercel) - Update import paths in KV interfaces - Update import paths in adapter entry files Adjust imports to match new directory structure.
- Update import paths in API handlers and router - Update import paths in providers (core logic, optimizely provider) - Update import paths in event listeners - Update import paths in user profile service Adjust imports to match new directory structure.
- Update import paths in index.js - Update import paths in cloudfront/index.entry.js - Update import paths in vercel/index.entry.js Finalize import path updates to match new directory structure.
- Move abstraction classes to core/interfaces/ - Elevate config/ to root level - Group helpers in utils/helpers/ - Remove empty types/ directory - Update import paths
- Rename project to 'Optimizely Edge Agent' - Add comprehensive project structure documentation - Remove 'Hybrid Edge Serverless' references - Update architecture descriptions
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
This PR reorganizes the project structure to improve code organization, maintainability, and discoverability. The changes focus on logical grouping of related functionality and clearer separation of concerns.
Changes
Directory Structure Updates
Moved abstraction classes to
core/interfaces/Elevated
config/to root levelCreated
utils/helpers/directoryRemoved empty
types/directoryImport Path Updates
_helpers_directoryImpact
Testing
The changes are purely structural - all functionality remains the same. However, thorough testing of all CDN adapters is recommended to ensure no import paths were missed.