Chore: Refactor for di#125
Merged
Merged
Conversation
added 17 commits
May 20, 2026 20:55
…iables - Added `@mariodebono/di-config` dependency for configuration management. - Implemented CLI argument parsing using `yargs` in `app-config.cli.ts`. - Created environment variable parsing in `app-config.env.ts` with Zod validation. - Defined application configuration schema in `app-config.schema.ts`. - Introduced `current-app-config.ts` to manage the current application configuration state. - Updated `app.module.ts` to utilize the new configuration module. - Refactored `electron-app.ts` to access configuration settings for window behavior and features. - Modified `main.ts` to initialize application configuration and set it globally. - Updated various utility functions to use the new configuration management system. - Added tests for configuration parsing and validation.
- Updated package.json to include tsc-alias for better path resolution. - Introduced new TypeScript declaration files in the shared directory for app, ipc, preferences, projects, releases, renderer, and tools. - Modified tsconfig.json files to support path aliases for shared types. - Updated Vite and Vitest configurations to resolve shared types using aliases. - Improved the structure and organization of type definitions for better maintainability and clarity.
…g and improve app initialization logic
Move Electron code into main/src, renderer code into renderer/src, shared types remain under shared/src, and relocate runtime resources to their own ownership roots. Also move renderer HTML/public assets under renderer, keep Vite config at the repo root, and package main runtime assets from main/assets while preserving the existing dist-electron and dist-react outputs.
…s in translation guide
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
1 participant
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.
This is a bit of a large PR, mainly targeting refactor and prep work to move into DI for future work.
Summary
Refactor the application into clearer runtime boundaries:
main/renderer/locales/shared/