Small fixes to make global types and dugite work with dev and build scripts

Author: Nils-Kolvenbach

src/main/index.ts

@@ -18,6 +18,17 @@ Sentry.init({
 
 class Main {
   constructor() {
+    // Overwrite dugites resolved path to the embedded git directory
+    // @see https://github.com/desktop/dugite/blob/0f5a4f11300fbfa8d2dd272b8ee9b771e5b34cd4/lib/git-environment.ts#L25
+    // This seems to be necessary, since it resolves to `elek.io Client.app/Contents/Resources/app/git` instead of dugites git inside node_modules `elek.io Client.app/Contents/Resources/app/node_modules/dugite/git`
+    process.env.LOCAL_GIT_DIRECTORY = Path.resolve(
+      __dirname,
+      '../../',
+      'node_modules',
+      'dugite',
+      'git'
+    );
+
     // Handle creating/removing shortcuts on Windows when installing/uninstalling.
     if (require('electron-squirrel-startup')) {
       app.quit();

src/renderer/react/app.tsx

@@ -5,10 +5,17 @@ import {
 } from '@tanstack/react-router';
 import { StrictMode } from 'react';
 import ReactDOM from 'react-dom/client';
+import { type ContextBridgeApi } from '../preload';
 
 // Import the generated route tree
 import { routeTree } from './routeTree.gen';
 
+declare global {
+  interface Window {
+    ipc: ContextBridgeApi;
+  }
+}
+
 // Create a new router instance
 const hashHistory = createHashHistory(); // Use hash based routing since in production electron just loads the index.html via the file protocol
 const router = createRouter({

src/types.d.ts

@@ -3,13 +3,3 @@
 // whether you're running in development or production).
 declare const MAIN_WINDOW_VITE_DEV_SERVER_URL: string;
 declare const MAIN_WINDOW_VITE_NAME: string;
-
-// This file should augment the properties of the `Window` with the type of the
-// `ContextBridgeApi` from `Electron.contextBridge` declared in `./preload.ts`.
-import type { ContextBridgeApi } from './renderer/preload';
-
-declare global {
-  interface Window {
-    ipc: ContextBridgeApi;
-  }
-}

vite.main.config.ts

@@ -7,6 +7,7 @@ export default defineConfig({
     // Some libs that can run in both Web and Node.js, such as `axios`, we need to tell Vite to build them in Node.js.
     browserField: false,
     mainFields: ['module', 'jsnext:main', 'jsnext'],
+    conditions: ['node'], // To get the resolution behavior of Node.js. Otherwise we get "getRandomValues() not supported" while creating UUIDs @see https://github.com/uuidjs/uuid/issues/544#issuecomment-740394448
   },
 
   build: {