👌 Wrote down some notes during Deep Dive

phpmon/Common/Core/Shell.swift

@@ -108,6 +108,7 @@ public class Shell {
      Checks if a file exists at a certain path.
      Used to be done with a shell command, now uses the native FileManager class instead.
      */
+    // TODO: To be moved
     public static func fileExists(_ path: String) -> Bool {
         let fullPath = path.replacingOccurrences(of: "~", with: "/Users/\(Paths.whoami)")
         return FileManager.default.fileExists(atPath: fullPath)

phpmon/Domain/App/App.swift

@@ -23,6 +23,7 @@ class App {
     }
     
     /** Whether the app is busy doing something. Used to determine what UI to display. */
+    // TODO: Remove this, and always use PhpEnv.shared.isBusy
     static var busy: Bool {
         return PhpEnv.shared.isBusy
     }

phpmon/Domain/Integrations/Valet/Valet.swift

@@ -24,7 +24,7 @@ class Valet {
     /// Whether we're busy with some blocking operation.
     var isBusy: Bool = false
     
-    /// When initialising the Valet singleton, extract the Valet version and assume no sites loaded.
+    /// When initialising the Valet singleton assume no sites loaded. We will load the version later.
     init() {
         self.version = nil
         self.sites = []