rework to_string / serialization

Author: simon-massa

lua/CopilotChat/config/providers.lua

@@ -242,11 +242,12 @@ M.copilot = {
 
     if response.body and response.body.endpoints and response.body.endpoints.api then
       log.info('get_headers ok, authenticated. Use api endpoint: ' .. response.body.endpoints.api)
+      log.info(utils.to_string(response)) --TODO: remove me, testing only
       M.endpoints_api = response.body.endpoints.api
     else
       log.error(
         'get_headers authenticated, but missing key "endpoints.api" in server response. response: '
-          .. utils.serializeTable(response)
+          .. utils.to_string(response)
       )
       error(
         'get_headers authenticated, but missing key "endpoints.api" in server response. Check log for details: '

lua/CopilotChat/utils.lua

@@ -450,38 +450,44 @@ M.curl_post = async.wrap(function(url, opts, callback)
   curl.post(url, args)
 end, 3)
 
-function M.serializeTable(val, name, skipnewlines, depth)
-  skipnewlines = skipnewlines or false
-  depth = depth or 0
-
-  local tmp = string.rep(' ', depth)
-
-  if name then
-    tmp = tmp .. name .. ' = '
-  end
-
-  if type(val) == 'table' then
-    tmp = tmp .. '{' .. (not skipnewlines and '\n' or '')
-
-    for k, v in pairs(val) do
-      tmp = tmp .. M.serializeTable(v, k, skipnewlines, depth + 1) .. ',' .. (not skipnewlines and '\n' or '')
+function M.to_string(tbl)
+  -- credit: http://lua-users.org/wiki/TableSerialization (universal tostring)
+  local function table_print(tt, indent, done)
+    done = done or {}
+    indent = indent or 0
+    if type(tt) == 'table' then
+      local sb = {}
+      for key, value in pairs(tt) do
+        table.insert(sb, string.rep(' ', indent)) -- indent it
+        if type(value) == 'table' and not done[value] then
+          done[value] = true
+          table.insert(sb, key .. ' = {\n')
+          table.insert(sb, table_print(value, indent + 2, done))
+          table.insert(sb, string.rep(' ', indent)) -- indent it
+          table.insert(sb, '}\n')
+        elseif 'number' == type(key) then
+          table.insert(sb, string.format('"%s"\n', tostring(value)))
+        else
+          table.insert(sb, string.format('%s = "%s"\n', tostring(key), tostring(value)))
+        end
+      end
+      return table.concat(sb)
+    else
+      return tt .. '\n'
     end
+  end
 
-    tmp = tmp .. string.rep(' ', depth) .. '}'
-  elseif type(val) == 'number' then
-    tmp = tmp .. tostring(val)
-  elseif type(val) == 'string' then
-    tmp = tmp .. string.format('%q', val)
-  elseif type(val) == 'boolean' then
-    tmp = tmp .. (val and 'true' or 'false')
+  if 'nil' == type(tbl) then
+    return tostring(nil)
+  elseif 'table' == type(tbl) then
+    return table_print(tbl)
+  elseif 'string' == type(tbl) then
+    return tbl
   else
-    tmp = tmp .. '"[inserializeable datatype:' .. type(val) .. ']"'
+    return tostring(tbl)
   end
-
-  return tmp
 end
 
-
 local function filter_files(files, max_count)
   local filetype = require('plenary.filetype')