From fe5b051226d1806295ba45fc36ed1cf373669943 Mon Sep 17 00:00:00 2001
From: Joshua Tye <21010072+catgoose@users.noreply.github.com>
Date: Sat, 21 Mar 2026 12:49:53 -0500
Subject: [PATCH] fix: correctly detect nightly api

---
 lua/colorizer.lua | 25 +++++++++++++++++++------
 1 file changed, 19 insertions(+), 6 deletions(-)

diff --git a/lua/colorizer.lua b/lua/colorizer.lua
index a66134c..e8ab638 100644
--- a/lua/colorizer.lua
+++ b/lua/colorizer.lua
@@ -90,17 +90,30 @@ local matcher_mod = require("colorizer.matcher")
 local utils = require("colorizer.utils")
 
 --- State and configuration dynamic holding information table tracking
+--- Detect whether vim.lsp.document_color.enable accepts a filter table
+--- as its second argument instead of a plain bufnr number.
+local dc_uses_filter_api = (function()
+  if not vim.lsp.document_color then
+    return false
+  end
+  local info = debug.getinfo(vim.lsp.document_color.enable, "u")
+  if info.nparams == 2 then
+    -- 2-param version is the new (enable, filter) API
+    return true
+  end
+  return false
+end)()
+
 --- Disable vim.lsp.document_color for a buffer.
---- Handles both old (enable, bufnr) and new (enable, filter) Neovim APIs.
+--- Handles both old `enable(enable, bufnr)` and new `enable(enable, filter)` Neovim APIs.
 local function disable_document_color(bufnr)
   if not vim.lsp.document_color then
     return
   end
-  -- Neovim nightly changed the signature to enable(enable, filter_table).
-  -- Detect by trying the new API first; fall back to the old one.
-  local ok = pcall(vim.lsp.document_color.enable, false, { bufnr = bufnr })
-  if not ok then
-    disable_document_color(bufnr)
+  if dc_uses_filter_api then
+    vim.lsp.document_color.enable(false, { bufnr = bufnr })
+  else
+    vim.lsp.document_color.enable(false, bufnr)
   end
 end