fix(parser): allow GRAPHQL method (#552)

* fix(parser): allow GRAPHQL method
* fix(lsp): skip requests with no name

Author: YaroSpace

lua/kulala/cmd/lsp.lua

@@ -286,6 +286,7 @@ local header_values = {
   { "application/octet-stream" },
   { "application/pdf" },
   { "application/zip" },
+  { "application/graphql-response+json" },
   { "text/plain" },
   { "text/html" },
   { "text/css" },
@@ -836,7 +837,9 @@ local function get_symbols()
   vim.iter(cache.requests):each(function(request)
     local cnum = 0
     local line = request.show_icon_line_number - 2
-    symbol = get_symbol(request.name, kind.Function, line) or {}
+
+    symbol = get_symbol(request.name, kind.Function, line)
+    if not symbol then return end
 
     if #request.scripts.pre_request.inline + #request.scripts.pre_request.files > 0 then
       table.insert(symbol.children, get_symbol("|< Pre-request script", kind.Module, line - 2))

lua/kulala/parser/graphql.lua

@@ -55,7 +55,7 @@ end
 
 M.get_json = function(body)
   local query, variables = parse(body)
-  local json = { query = "", variables = "" }
+  local json = { query = "" }
 
   if not (query and #query > 0) then return end
 

lua/kulala/parser/request.lua

@@ -276,6 +276,7 @@ local function process_graphql(request)
     or PARSER_UTILS.contains_header(request.headers, "x-request-type", "graphql")
 
   if request.body and #request.body > 0 and is_graphql then
+    request.method = "POST"
     local gql_json = GRAPHQL_PARSER.get_json(request.body)
 
     if gql_json then
@@ -285,6 +286,7 @@ local function process_graphql(request)
       request.body_computed = gql_json
     end
   end
+
   return request
 end
 

lua/kulala/utils/json.lua

@@ -27,6 +27,10 @@ M.format = function(json_string)
   return result.stdout
 end
 
+---Parse a JSON string into a Lua table
+---@param str string
+---@param opts table<{object: boolean, array: boolean, verbose: boolean}> -- verbose: log errors
+---@return table|nil, string|nil
 M.parse = function(str, opts)
   opts = opts or { object = true, array = true }
   local verbose = opts.verbose or false

tests/functional/requests_spec.lua

@@ -504,10 +504,15 @@ describe("requests", function()
       kulala.run()
       wait_for_requests(1)
 
-      local request_body_computed = DB.data.current_request.body_computed
+      local request = DB.data.current_request
 
-      assert.has_string(request_body_computed, '"query":"query Person($id: ID) { person(personID: $id) { name } }')
-      assert.has_string(request_body_computed, '"variables":{"id":1}')
+      assert.is_same("POST", request.method)
+      assert.has_properties(request.headers, {
+        ["Content-Type"] = "application/json",
+      })
+
+      assert.has_string(request.body_computed, '"query":"query Person($id: ID) { person(personID: $id) { name } }')
+      assert.has_string(request.body_computed, '"variables":{"id":1}')
     end)
 
     it("runs API callbacks", function()