aboutsummaryrefslogtreecommitdiffstats
path: root/packages/main
Commit message (Collapse)AuthorAge
...
* feat: Save selected service to fileLibravatar Kristóf Marussy2022-02-08
| | | | | | | Sophie start off from the service that was selected when it was last open. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* test: Add tests for main window hardeningLibravatar Kristóf Marussy2022-02-08
| | | | | | | We try to stub/mock the Electron API to make sure the test environment is as close to the runtime environment for this security critical code. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* build: Add test-utils packageLibravatar Kristóf Marussy2022-02-08
| | | | | | | | | | Added as a common devDependency, this lets us handle test utility code from one place. For now, the main reason for its existence is the workaround code for importing jest-each from ESM. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* refactor: Extract main window hardeningLibravatar Kristóf Marussy2022-02-08
| | | | | | | This code is security critical, so it should be properly extracted to enable testing. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* refactor: Extract resource path managementLibravatar Kristóf Marussy2022-02-08
| | | | | | | Lets us access absolute paths and URLs without directly calling node APIs. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* refactor: Extract config handlingLibravatar Kristóf Marussy2022-02-08
| | | | | | | Move the handling of the contents of the config file out of the stores and into dedicated files to simplify the code of the stores. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* refactor: Coding conventionsLibravatar Kristóf Marussy2022-02-08
| | | | | | | | | Make sure that files have a default import with the same name as the file whenever possible to reduce surprise. Also shuffles around some file names for better legibility. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* refactor: Apply shared store patches in batchesLibravatar Kristóf Marussy2022-02-08
| | | | | | Makes sure that the renderer always sees a consistent state. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* refactor: Store services in a mapLibravatar Kristóf Marussy2022-02-08
| | | | | | | Makes the synchronization of references across the main/renderer process boundary more robust. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* refactor: Move runtime state into shared modelsLibravatar Kristóf Marussy2022-02-08
| | | | | | | | | | | Now the runtime state lives inside the model (instead of being associated to the static settings via a map), which simplifies state management. Static settings are now located inside the runtime models, so we must create tests to make sure that the settings are being persisted correctly. The contents of the config file are now generated as a view of store (instead of a snapshot), which adds flexibility. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* feat: Add selected service field to SharedStoreLibravatar Kristóf Marussy2022-02-08
| | | | | | Lets the main process see which service is currently selected. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* refactor: Rename main services to infrastructureLibravatar Kristóf Marussy2022-02-08
| | | | | | | | Makes it clear that we are talking about wrappers for nodejs and electron functionality, and not the services being displayed by the application. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* feat: Add RuntimeService storeLibravatar Kristóf Marussy2022-02-08
| | | | | | | Stores transient state for services shared between the main and renderer processes. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* feat: Source mapping for stacktraces in dev modeLibravatar Kristóf Marussy2022-02-08
| | | | | | We have to cheat again and use require() to lazy load a dev dependency. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* feat: Rewrite the config if new details are addedLibravatar Kristóf Marussy2022-02-08
| | | | | | | If we generate a new ID or a new profile, it should be added to the config file immediately. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* feat: Add Profile and Service storesLibravatar Kristóf Marussy2022-02-08
| | | | | | | | | | In the main process, it is optional to specify the ID of a Profile or a Service. The missing ID will be filled in with a randomly generated one. Moreover, services without a profile will get a profile generated with the same name. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* chore(deps): update electron to version 17.0.0Libravatar Depfu Bot2022-02-08
|
* chore: Bump dependenciesLibravatar Kristóf Marussy2022-01-27
|
* Upgrade 'electron' to '16.0.7'Libravatar Vijay A2022-01-12
|
* build: Add eslint-plugin-securityLibravatar Kristóf Marussy2022-01-10
| | | | | | This would run as part of gitlab SAST (if it didn't time out). Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* build: Add eslint-plugin-promiseLibravatar Kristóf Marussy2022-01-09
| | | | Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* build: Add eslint-plugin-unicornLibravatar Kristóf Marussy2022-01-09
| | | | Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* build: Add eslint-plugin-jestLibravatar Kristóf Marussy2022-01-09
| | | | Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* build: Add prettierLibravatar Kristóf Marussy2022-01-09
| | | | | | | eslint will also enforce prettier rules, so there is no need to call prettier separately in CI. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* build: Disable single-run eslint-typescriptLibravatar Kristóf Marussy2022-01-09
| | | | | | | | | See https://github.com/typescript-eslint/typescript-eslint/issues/3851 Also upgrades dependencies and simplifies eslint config (used during debugging this issue to eliminate other possible sources of errors.) Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* build: Eslint fixes for multi-module projectLibravatar Kristóf Marussy2022-01-09
| | | | Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* New configurations based on review comments (WIP)Libravatar Vijay A2022-01-09
| | | | Signed-off-by: Vijay A <vraravam@users.noreply.github.com>
* Incorporate 'eslint' (fixes #3)Libravatar Vijay A2022-01-09
| | | | Signed-off-by: Vijay A <vraravam@users.noreply.github.com>
* Upgrade npm modulesLibravatar Vijay A2022-01-09
| | | | Signed-off-by: Vijay A <vraravam@users.noreply.github.com>
* Upgrade 'electron' to '16.0.6'Libravatar Vijay A2022-01-04
|
* Enhance 'About' dialog with more infoLibravatar Vijay A2022-01-03
| | | | Signed-off-by: Vijay A <vraravam@users.noreply.github.com>
* chore: Add @avijayr as a copyright holderLibravatar Kristóf Marussy2022-01-02
| | | | Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* Ensure that the 'userData' directory is present when starting up the appLibravatar Vijay A2022-01-02
| | | | Signed-off-by: Vijay A <vraravam@users.noreply.github.com>
* chore: Fix and bump dependenciesLibravatar Kristóf Marussy2022-01-02
| | | | | | Test devDependencies in the main package were incorrect. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* fix: Allow devtools extensions to be installedLibravatar Kristóf Marussy2022-01-02
| | | | | | | | | | | | | On the first startup in dev mode, Electron must be able to download the devtools extensions and wait for them to be installed. Loosens the UI process request filter a bit, but the behavior should match production mode in all cases except chrome webstore URLs. Nevertheless, only production mode should be considered secure. Fixes #6 Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* fix: Ensure dev user directory existsLibravatar Kristóf Marussy2022-01-02
| | | | | | Fixes #5 Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* feat: Load browserview contents asynchronouslyLibravatar Kristóf Marussy2022-01-01
| | | | | | Also changes the default browserview contents to the gitlab repo. Signed-off-by: Kristóf Marussy <kristof@marussy.com>
* build: Use nodenext module resolutionLibravatar Kristóf Marussy2021-12-31
| | | | | The renderer package still has to use node module resolution to be able to find MUI packages.
* chore: Add import extensions preparing for ts 4.6 nodenextLibravatar Kristóf Marussy2021-12-31
|
* build: Upgrade to chalk 5.0.0 anywaysLibravatar Kristóf Marussy2021-12-31
| | | | Requires some workarounds for ts-jest to find the vendored dependencies.
* test: Add tests for main packageLibravatar Kristóf Marussy2021-12-31
| | | | | | | | | | | | | - Changed jest to run from the root package and reference the packages as projects. This required moving the base jest config file away from the project root. - Module isolation seems to prevent ts-jest from loading the shared package, so we disabled it for now. - To better facilitate mocking, services should be split into interfaces and implementation - Had to downgrade to chald 4.1.2 as per https://github.com/chalk/chalk/releases/tag/v5.0.0 at least until https://github.com/microsoft/TypeScript/issues/46452 is resolved.
* refactor: Install devtools extensions earlierLibravatar Kristóf Marussy2021-12-30
| | | | | | This doesn't seem to cause any race conditions (except for the warnings that appear even if we install the extensions before initializing the main window and are ignored by yarn watch).
* build: Fully switch over to ESMLibravatar Kristóf Marussy2021-12-30
| | | | | | | | | | | | | | | | | | | | | | | | | | Now we can run with ESM at build and test time and transpile into commonjs for electron. This greatly simplifies testing, since we treat everything as ESM during build with esbuild anyways. Now the test environment and the build scripts match the apparent (but not the actual for the main, preload, and inject modules) runtime environment. Caveats: - We may use top-level async expressions in tests and script, but not in code that gets transpiled into commonjs or scripts that get imported by vite. The limitation w.r.t. commonjs seems fundamental. - Jest only experimentally supports ESM and there are some limitations with mocking. Most limitations (except the lack of automatic mocks) can be worked around by async importing code that uses mocks. - There are packages marked as modules (so that node reads any scripts in them as ESM) that nevertheless get transpiled into commonjs modules. However, these should be clearly marked by using a .cjs extension as their bundle. The worst offender is the root package, which has a .cjs as its main entry point that gets read by electron, but is in fact marked as a module. This doesn't seem to bother electron at all. The service-inject package is an IIFE with a .js extension, but it outputs a fully self-contained bundle, so the choice of module format should be irrelevant.
* build: Only use vite typings in rendererLibravatar Kristóf Marussy2021-12-30
| | | | | The rest of the packages aren't buld with vite, so we shouldn't use its typings.
* build: Simplify clean scriptLibravatar Kristóf Marussy2021-12-30
|
* refactor: Improve loggingLibravatar Kristóf Marussy2021-12-30
|
* build: Switch to esbuildLibravatar Kristóf Marussy2021-12-30
| | | | | | | | | | We will build all packages except the frontend (where vite remains in use) with esbuild. For some reason, the @yarnpkg/esbuild-plugin-pnp doesn't allow esbuild to load esm modules and we fall back to commonjs for dependencies. Hence we had to switch back to node_modules (but still rely on yarn hardlinking for a more efficient install).
* Revert "refactor: Switch back to consola for prettyness"Libravatar Kristóf Marussy2021-12-29
| | | | This reverts commit 5c38af061348ec604337280009775832edc66270.
* refactor: Switch back to consola for prettynessLibravatar Kristóf Marussy2021-12-28
|
* refactor: Simpler logging with loglevelLibravatar Kristóf Marussy2021-12-28
|