refactor(hammerspoon): 에러 가능 지점과 미사용 설정 정리

configs/hammerspoon/Spoons/hotkey-window.lua

@@ -11,18 +11,14 @@ local function create_hot_key_window(app_name, command_key, key)
 		else
 			local win = app:mainWindow()
 
-			local activeSpaces = spaces.activeSpaces()
-
 			spaces.moveWindowToSpace(win:id(), spaces.activeSpaceOnScreen())
 			win:focus()
 		end
 	end)
 end
 
--- create_hot_key_window("net.ankiweb.dtop", { "cmd", "shift" }, "m")
 create_hot_key_window("md.obsidian", { "cmd", "shift" }, "z")
 create_hot_key_window("net.kovidgoyal.kitty", { "cmd", "shift" }, "s")
 create_hot_key_window("com.hnc.Discord", { "cmd", "shift" }, "d")
--- create_hot_key_window("com.mitchellh.ghostty", { "cmd", "shift" }, "g")
 create_hot_key_window("com.google.Chrome", { "cmd", "shift" }, "a")
 create_hot_key_window("abnerworks.Typora", { "cmd", "shift" }, "m")

configs/hammerspoon/Spoons/tiling.lua

@@ -12,10 +12,13 @@ local windowChooser = function(active_win_id, callback)
 	end)
 	-- format for chooser
 	win_list = hs.fnutils.map(win_list, function(win)
+		local app = win:application()
+		local app_name = app and app:name() or ""
+		local bundle_id = app and app:bundleID() or ""
 		return {
-			text = win:title() .. " - " .. win:application():name() .. " - " .. win:application():bundleID(),
-			subText = win:application():name(),
-			image = hs.image.imageFromAppBundle(win:application():bundleID()),
+			text = (win:title() or "") .. " - " .. app_name .. " - " .. bundle_id,
+			subText = app_name,
+			image = bundle_id ~= "" and hs.image.imageFromAppBundle(bundle_id) or nil,
 			value = win:id(),
 		}
 	end)
@@ -45,6 +48,10 @@ hs.hotkey.bind({ "cmd", "shift" }, "h", function()
 	active_win:setFrame(active_frame)
 
 	windowChooser(active_win:id(), function(choice)
+		if choice == nil then
+			return
+		end
+
 		local other_frame = {}
 		other_frame.x = active_frame.x + active_frame.w
 		other_frame.y = active_frame.y
@@ -97,6 +104,10 @@ hs.hotkey.bind({ "cmd", "shift" }, "l", function()
 	active_win:setFrame(active_frame)
 
 	windowChooser(active_win:id(), function(choice)
+		if choice == nil then
+			return
+		end
+
 		local other_frame = {}
 		other_frame.x = active_frame.x - active_frame.w
 		other_frame.y = active_frame.y
@@ -116,7 +127,16 @@ end)
 -- find & focus
 hs.hotkey.bind({ "cmd", "shift" }, "f", function()
 	local activeWindow = hs.window.focusedWindow()
-	windowChooser(activeWindow:id(), function(choice)
-		hs.window(choice.value):focus()
+	local active_win_id = activeWindow and activeWindow:id()
+	windowChooser(active_win_id, function(choice)
+		if choice == nil then
+			return
+		end
+
+		local win = hs.window(choice.value)
+		if win == nil then
+			return
+		end
+		win:focus()
 	end)
 end)

configs/hammerspoon/init.lua

@@ -2,7 +2,6 @@ require("hs.ipc")
 
 -- reload config
 hs.hotkey.bind({ "cmd", "alt", "ctrl" }, "R", function()
-	require("Spoons.Annotation").init()
 	hs.reload()
 end)
 

configs/hammerspoon/sketchybar/init.lua

@@ -1,7 +1,4 @@
 local manager = require("sketchybar.manager")
 
 manager.add_widget(require("sketchybar.widgets.active-app"), { only_init = true })
--- manager.add_widget(require("sketchybar.widgets.battery"), { time = 120 })
--- manager.add_widget(require("sketchybar.widgets.memory"), { time = 120 })
--- manager.add_widget(require("sketchybar.widgets.storage"), { time = 120 })
 manager.add_widget(require("sketchybar.widgets.wifi"), { only_init = true })

configs/hammerspoon/sketchybar/manager/init.lua

@@ -29,7 +29,7 @@ M.add_widget = function(widget, opts)
 end
 
 M.remove_widget = function(widget_id)
-	if not _widgets[widget_id].timer then
+	if _widgets[widget_id].timer then
 		_widgets[widget_id].timer:stop()
 	end
 	_widgets[widget_id] = nil