Merge pull request #1 from taigrr/cd/bugfixes-and-cleanup

fix(health,checker): fix plugin display bug and remove redundant version checks

LICENSE

@@ -1,6 +1,6 @@
 BSD Zero Clause License
 
-Copyright (c) 2025 Tai Groot
+Copyright (c) 2026 Tai Groot
 
 Permission to use, copy, modify, and/or distribute this software for any
 purpose with or without fee is hereby granted.

README.md

@@ -6,16 +6,16 @@
 
 <p>
   <a href="https://github.com/taigrr/glaze.nvim/releases/latest">
-    <img alt="Latest release" src="https://img.shields.io/github/v/release/taigrr/glaze.nvim?style=for-the-badge&logo=starship&color=FF6AD5&logoColor=D9E0EE&labelColor=302D41&include_prerelease&sort=semver">
+    <img alt="Latest release" src="https://img.shields.io/github/v/release/taigrr/glaze.nvim?style=for-the-badge&logo=starship&color=FF6AD5&logoColor=D9E0EE&labelColor=302D41&sort=semver&include_prerelease">
   </a>
   <a href="https://github.com/taigrr/glaze.nvim/pulse">
-    <img alt="Last commit" src="https://img.shields.io/github/last-commit/taigrr/glaze.nvim?style=for-the-badge&logo=starship&color=8bd5ca&logoColor=D9E0EE&labelColor=302D41">
+    <img alt="Last commit" src="https://img.shields.io/github/last-commit/taigrr/glaze.nvim?style=for-the-badge&logo=starship&color=8bd5ca&labelColor=302D41&logoColor=D9E0EE">
   </a>
   <a href="https://github.com/taigrr/glaze.nvim/blob/master/LICENSE">
-    <img alt="License" src="https://img.shields.io/github/license/taigrr/glaze.nvim?style=for-the-badge&logo=starship&color=ee999f&logoColor=D9E0EE&labelColor=302D41">
+    <img alt="License" src="https://img.shields.io/github/license/taigrr/glaze.nvim?style=for-the-badge&logo=starship&color=ee999f&labelColor=302D41&logoColor=D9E0EE">
   </a>
   <a href="https://github.com/taigrr/glaze.nvim/stargazers">
-    <img alt="Stars" src="https://img.shields.io/github/stars/taigrr/glaze.nvim?style=for-the-badge&logo=starship&color=c69ff5&logoColor=D9E0EE&labelColor=302D41">
+    <img alt="Stars" src="https://img.shields.io/github/stars/taigrr/glaze.nvim?style=for-the-badge&logo=starship&color=c69ff5&labelColor=302D41&logoColor=D9E0EE">
   </a>
 </p>
 

lua/glaze/checker.lua

@@ -127,6 +127,60 @@ function M.get_update_info()
   return M._update_info
 end
 
+---Refresh version info for a single binary (called after install/update).
+---@param name string Binary name
+---@param callback? fun() Optional callback when done
+function M.refresh_version(name, callback)
+  local glaze = require("glaze")
+  local binary = glaze._binaries[name]
+  if not binary then
+    if callback then
+      callback()
+    end
+    return
+  end
+
+  get_installed_version(name, function(installed)
+    local info = M._update_info[name] or {
+      name = name,
+      installed_version = nil,
+      latest_version = nil,
+      has_update = false,
+    }
+    info.installed_version = installed
+
+    -- If we have latest version cached, check if still needs update
+    if info.latest_version and installed then
+      info.has_update = installed ~= info.latest_version
+    else
+      info.has_update = false
+    end
+
+    M._update_info[name] = info
+
+    -- Persist to state
+    local state = read_state()
+    state.update_info = state.update_info or {}
+    state.update_info[name] = {
+      installed_version = info.installed_version,
+      latest_version = info.latest_version,
+      has_update = info.has_update,
+    }
+    write_state(state)
+
+    -- Refresh UI if open
+    vim.schedule(function()
+      local ok, view = pcall(require, "glaze.view")
+      if ok and view._float and view._float:valid() then
+        view.render()
+      end
+      if callback then
+        callback()
+      end
+    end)
+  end)
+end
+
 ---Check for updates on all registered binaries.
 ---@param opts? { silent?: boolean }
 function M.check(opts)
@@ -149,12 +203,9 @@ function M.check(opts)
   end
 
   M._checking = true
-  local remaining = 0
   local updates_found = 0
-
+  local remaining = vim.tbl_count(binaries)
-  for name, _ in pairs(binaries) do
+  for name, binary in pairs(binaries) do
-    remaining = remaining + 2 -- installed version + latest version
-
     local info = {
       name = name,
       installed_version = nil,
@@ -163,16 +214,6 @@ function M.check(opts)
     }
     M._update_info[name] = info
 
-    get_installed_version(name, function()
-      -- callback receives version from the jobstart; re-check via closure
-    end)
-  end
-
-  -- Simplified: check each binary sequentially-ish
-  remaining = vim.tbl_count(binaries)
-  for name, binary in pairs(binaries) do
-    local info = M._update_info[name]
-
     get_installed_version(name, function(installed)
       info.installed_version = installed
 

lua/glaze/health.lua

@@ -72,10 +72,13 @@ function M.check()
   else
     vim.health.ok(count .. " binary(ies) registered")
     for name, binary in pairs(binaries) do
+      local pi = (binary.plugins and #binary.plugins > 0)
+          and (" (" .. table.concat(binary.plugins, ", ") .. ")")
+        or ""
       if glaze.is_installed(name) then
-        vim.health.ok("  " .. name .. " — installed" .. (binary.plugin and (" (" .. binary.plugin .. ")") or ""))
+        vim.health.ok("  " .. name .. " — installed" .. pi)
       else
-        vim.health.warn("  " .. name .. " — missing" .. (binary.plugin and (" (" .. binary.plugin .. ")") or ""), {
+        vim.health.warn("  " .. name .. " — missing" .. pi, {
           "Run :GlazeInstall " .. name,
           "Or: go install " .. binary.url .. "@latest",
         })

lua/glaze/init.lua

@@ -15,8 +15,8 @@ local M = {}
 ---@class GlazeBinary
 ---@field name string Binary name (executable name)
 ---@field url string Go module URL (without @version)
----@field plugin? string Plugin that registered this binary
+---@field plugins string[] Plugins that registered this binary
----@field callback? fun(success: boolean) Optional callback after install/update
+---@field callbacks table<string, fun(success: boolean)> Callbacks keyed by plugin name
 
 ---@class GlazeAutoCheckConfig
 ---@field enabled? boolean Whether to auto-check for updates
@@ -147,11 +147,41 @@ end
 ---@param opts? { plugin?: string, callback?: fun(success: boolean) }
 function M.register(name, url, opts)
   opts = opts or {}
+  local plugin = opts.plugin or "unknown"
+
+  -- Check if this URL is already registered under a different name
+  for existing_name, binary in pairs(M._binaries) do
+    if binary.url == url and existing_name ~= name then
+      -- Same URL, different name - merge into existing entry
+      if not vim.tbl_contains(binary.plugins, plugin) then
+        table.insert(binary.plugins, plugin)
+      end
+      if opts.callback then
+        binary.callbacks[plugin] = opts.callback
+      end
+      return
+    end
+  end
+
+  -- Check if this name is already registered
+  local existing = M._binaries[name]
+  if existing then
+    -- Merge plugin into existing entry
+    if not vim.tbl_contains(existing.plugins, plugin) then
+      table.insert(existing.plugins, plugin)
+    end
+    if opts.callback then
+      existing.callbacks[plugin] = opts.callback
+    end
+    return
+  end
+
+  -- New binary registration
   M._binaries[name] = {
     name = name,
     url = url,
-    plugin = opts.plugin,
+    plugins = opts.plugin and { opts.plugin } or {},
-    callback = opts.callback,
+    callbacks = opts.callback and { [plugin] = opts.callback } or {},
   }
 
   -- Auto-install if enabled and binary is missing

lua/glaze/runner.lua

@@ -93,10 +93,17 @@ local function run_task(task)
       task.status = code == 0 and "done" or "error"
       task.job_id = nil
 
-      -- Call binary callback if set
+      -- Refresh version info on success
-      if task.binary.callback then
+      if code == 0 then
+        require("glaze.checker").refresh_version(task.binary.name)
+      end
+
+      -- Call all registered callbacks
+      if task.binary.callbacks then
         vim.schedule(function()
-          task.binary.callback(code == 0)
+          for _, cb in pairs(task.binary.callbacks) do
+            cb(code == 0)
+          end
         end)
       end
 

lua/glaze/view.lua

@@ -164,16 +164,16 @@ function M.render()
   text:append(" ", nil, { indent = 2 })
   text:append(" U ", "GlazeButtonActive"):append(" Update All ", "GlazeButton")
   text:append("  ")
-  text:append(" u ", "GlazeButtonActive"):append(" Update ", "GlazeButton")
+  text:append(" u ", "GlazeButtonActive"):append(" Update Selected ", "GlazeButton")
   text:append("  ")
   text:append(" I ", "GlazeButtonActive"):append(" Install All ", "GlazeButton")
   text:append("  ")
-  text:append(" i ", "GlazeButtonActive"):append(" Install ", "GlazeButton")
+  text:append(" i ", "GlazeButtonActive"):append(" Install Selected ", "GlazeButton")
   text:nl()
   text:append(" ", nil, { indent = 2 })
   text:append(" x ", "GlazeButtonActive"):append(" Abort ", "GlazeButton")
   text:append("  ")
-  text:append(" ⏎ ", "GlazeButtonActive"):append(" Details ", "GlazeButton")
+  text:append(" <enter> ", "GlazeButtonActive"):append(" Details ", "GlazeButton")
   text:append("  ")
   text:append(" q ", "GlazeButtonActive"):append(" Close ", "GlazeButton")
   text:nl():nl()
@@ -323,8 +323,8 @@ function M._render_binary(text, binary, icons, update_info)
   text:append("  " .. icon .. " ", icon_hl)
   text:append(binary.name, "GlazeBinary")
 
-  if binary.plugin then
+  if binary.plugins and #binary.plugins > 0 then
-    text:append(" (" .. binary.plugin .. ")", "GlazePlugin")
+    text:append(" (" .. table.concat(binary.plugins, ", ") .. ")", "GlazePlugin")
   end
 
   -- Show update available indicator
@@ -353,9 +353,9 @@ function M._render_binary(text, binary, icons, update_info)
       text:append(bin_path, "GlazeUrl"):nl()
     end
 
-    if binary.plugin then
+    if binary.plugins and #binary.plugins > 0 then
-      text:append("Plugin: ", "GlazeComment", { indent = 6 })
+      text:append("Plugins: ", "GlazeComment", { indent = 6 })
-      text:append(binary.plugin, "GlazePlugin"):nl()
+      text:append(table.concat(binary.plugins, ", "), "GlazePlugin"):nl()
     end
 
     -- Show last error output from tasks