Refactor: Modular architecture, migrate javscript code and test alignment - #47
Merged
Merged
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
Contributor
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
…es in environment variable values
…re growth - Restructured project to follow a modular layered architecture (CLI, application, domain, infrastructure) for clarity and maintainability. - Aligned test folder structure and naming conventions with production code for consistency. - Added basic unit tests for core modules to ensure testability and CI compatibility. - This refactor prepares the codebase for future features and complexity as outlined in the project roadmap, enabling easier scaling and extension.
…nd EnvFileManager
macalbert
added a commit
that referenced
this pull request
Jun 10, 2025
5 tasks
5 tasks
This was referenced Apr 27, 2026
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
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.
Description
This PR restructures the codebase to follow a modular layered architecture, migrates all source files from JavaScript to TypeScript, and aligns the test structure and naming conventions with the production code. These changes are made to support future growth and maintainability as outlined in the project roadmap.
Key Changes
Adopted a modular layered structure with clear separation between CLI, application, domain, and infrastructure layers. This improves code clarity, testability, and scalability.
All source files have been moved from JavaScript (.js) to TypeScript (.ts), enabling better type safety and developer experience.
Test folders and filenames now mirror the production code structure, using PascalCase for consistency. This makes it easier to locate and maintain tests as the project evolves.
Added simple unit tests for core modules (e.g., EnvilderFactory, AwsSsmStoreSecrets) to ensure the test suite passes and to provide a foundation for future test coverage.
Motivation
These changes lay the groundwork for new features and increased complexity as described in the project roadmap. The new structure will make it easier to extend the CLI, add support for new secret stores, and maintain high code quality as the project grows.
Summary by CodeRabbit
New Features
.envfiles.Bug Fixes
.envfiles to ensure correct formatting.Chores
.gitignoreand code formatting settings.Documentation
Tests