feat: validate setup() options and report clear errors

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: SpollaL

lua/datasight/config.lua

@@ -14,8 +14,54 @@ M.defaults = {
 -- Start with a copy of defaults. setup() will merge user options into this.
 M.options = vim.deepcopy(M.defaults)
 
+local VALID_BORDERS = { rounded=true, single=true, double=true, solid=true, shadow=true, none=true }
+
+local function validate(opts)
+  local errors = {}
+
+  if opts.binary_path ~= nil and type(opts.binary_path) ~= "string" then
+    table.insert(errors, "binary_path must be a string")
+  end
+
+  if opts.width ~= nil then
+    if type(opts.width) ~= "number" or opts.width <= 0 or opts.width > 1 then
+      table.insert(errors, "width must be a number between 0 and 1 (e.g. 0.85)")
+    end
+  end
+
+  if opts.height ~= nil then
+    if type(opts.height) ~= "number" or opts.height <= 0 or opts.height > 1 then
+      table.insert(errors, "height must be a number between 0 and 1 (e.g. 0.85)")
+    end
+  end
+
+  if opts.border ~= nil and not VALID_BORDERS[opts.border] then
+    table.insert(errors, "border must be one of: rounded, single, double, solid, shadow, none")
+  end
+
+  if opts.auto_open ~= nil and type(opts.auto_open) ~= "boolean" then
+    table.insert(errors, "auto_open must be a boolean")
+  end
+
+  if opts.keymaps ~= nil then
+    if type(opts.keymaps) ~= "table" then
+      table.insert(errors, "keymaps must be a table")
+    elseif opts.keymaps.open ~= nil
+      and opts.keymaps.open ~= false
+      and type(opts.keymaps.open) ~= "string" then
+      table.insert(errors, "keymaps.open must be a string or false")
+    end
+  end
+
+  if #errors > 0 then
+    error("datasight.nvim: invalid config:\n  - " .. table.concat(errors, "\n  - "), 2)
+  end
+end
+
 function M.setup(opts)
-  M.options = vim.tbl_deep_extend("force", M.defaults, opts or {})
+  opts = opts or {}
+  validate(opts)
+  M.options = vim.tbl_deep_extend("force", M.defaults, opts)
 end
 
 return M