Switch our defaults around so that Hammerspoon runs with its dock ico…

…n hidden by default. I believe this is more in line with what people expect from a tool like this, and it won't affect anyone who has already run the app at least once

Hammerspoon/Hammerspoon-Info.plist

@@ -167,8 +167,6 @@
 			</array>
 		</dict>
 		<dict>
-			<key>CFBundleTypeRole</key>
-			<string>Editor</string>
 			<key>CFBundleURLName</key>
 			<string>hammerspoon.Hammerspoon.http</string>
 			<key>CFBundleURLSchemes</key>
@@ -177,8 +175,6 @@
 			</array>
 		</dict>
 		<dict>
-			<key>CFBundleTypeRole</key>
-			<string>Editor</string>
 			<key>CFBundleURLName</key>
 			<string>hammerspoon.Hammerspoon.https</string>
 			<key>CFBundleURLSchemes</key>
@@ -207,6 +203,8 @@
 	<string>public.app-category.developer-tools</string>
 	<key>LSMinimumSystemVersion</key>
 	<string>${MACOSX_DEPLOYMENT_TARGET}</string>
+	<key>LSUIElement</key>
+	<true/>
 	<key>NSAppSleepDisabled</key>
 	<true/>
 	<key>NSAppTransportSecurity</key>

Hammerspoon/MJAppDelegate.m

@@ -280,7 +280,7 @@ - (NSApplicationTerminateReply)applicationShouldTerminate:(NSApplication *)sende
 - (void) registerDefaultDefaults {
     [[NSUserDefaults standardUserDefaults]
      registerDefaults: @{@"NSApplicationCrashOnExceptions": @YES,
-                         MJShowDockIconKey: @YES,
+                         MJShowDockIconKey: @NO,
                          MJShowMenuIconKey: @YES,
                          HSAutoLoadExtensions: @YES,
                          HSUploadCrashDataKey: @YES,

Hammerspoon/MJDockIcon.m

@@ -18,12 +18,21 @@ void MJDockIconSetVisible(BOOL visible) {
 }
 
 static void reflect_defaults(void) {
-    NSApplication* app = [NSApplication sharedApplication]; // NSApp is typed to 'id'; lame
+    NSApplication* app = [NSApplication sharedApplication];
+    NSApplicationActivationPolicy currentPolicy = app.activationPolicy;
+    NSApplicationActivationPolicy targetPolicy = MJDockIconVisible() ? NSApplicationActivationPolicyRegular : NSApplicationActivationPolicyAccessory;
+
+    if (currentPolicy == targetPolicy) {
+        // No need to do anything, we already have the policy we want
+        return;
+    }
+
 #pragma clang diagnostic push
 #pragma clang diagnostic ignored "-Wdeprecated-declarations"
     NSDisableScreenUpdates();
 #pragma clang diagnostic pop
-    [app setActivationPolicy: MJDockIconVisible() ? NSApplicationActivationPolicyRegular : NSApplicationActivationPolicyAccessory];
+
+    [app setActivationPolicy:targetPolicy];
     dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(0.1 * NSEC_PER_SEC)), dispatch_get_main_queue(), ^{
         [app unhide: nil];
         [app activateIgnoringOtherApps:YES];