query params escape; response save to file

Author: heilgar

CHANGELOG.md

@@ -5,6 +5,31 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+
+## [1.2.1] 2025-07-01
+### Added
+
+* Global response state management system
+  * Persistent response caching using `_G._http_client_state`
+  * Response history with configurable size limit
+
+* Enhanced file operations
+  * New `write_file` utility function with error handling
+  * Automatic directory creation for saves
+  * Cross-platform path handling using Plenary
+
+* Improved URL handling
+  * Enhanced URL encoding for complex query parameters
+  * Tested handling for Salesforce SOQL queries
+  * Comprehensive character escaping
+
+* Response saving features
+  * New command `:HttpSaveResponse` for formatted response
+  * Content-type based file extension detection
+  * Keybindings:
+    * `<leader>hs` - Save formatted response
+
+
 ## [1.2.0] 2024-12-30
 ### Added
 - Resopnse window management

doc/tags

@@ -0,0 +1,33 @@
+:HttpCopyCurl	http_client.txt	/*:HttpCopyCurl*
+:HttpDryRun	http_client.txt	/*:HttpDryRun*
+:HttpEnv	http_client.txt	/*:HttpEnv*
+:HttpEnvFile	http_client.txt	/*:HttpEnvFile*
+:HttpRun	http_client.txt	/*:HttpRun*
+:HttpRunAll	http_client.txt	/*:HttpRunAll*
+:HttpStop	http_client.txt	/*:HttpStop*
+:HttpVerbose	http_client.txt	/*:HttpVerbose*
+http_client-commands	http_client.txt	/*http_client-commands*
+http_client-comments	http_client.txt	/*http_client-comments*
+http_client-config-keybindings	http_client.txt	/*http_client-config-keybindings*
+http_client-configuration	http_client.txt	/*http_client-configuration*
+http_client-contents	http_client.txt	/*http_client-contents*
+http_client-copy-curl	http_client.txt	/*http_client-copy-curl*
+http_client-default_env_file	http_client.txt	/*http_client-default_env_file*
+http_client-dry-run	http_client.txt	/*http_client-dry-run*
+http_client-environment-files	http_client.txt	/*http_client-environment-files*
+http_client-global-variables	http_client.txt	/*http_client-global-variables*
+http_client-http-versions	http_client.txt	/*http_client-http-versions*
+http_client-keybindings	http_client.txt	/*http_client-keybindings*
+http_client-no-environment	http_client.txt	/*http_client-no-environment*
+http_client-request_timeout	http_client.txt	/*http_client-request_timeout*
+http_client-response-handlers	http_client.txt	/*http_client-response-handlers*
+http_client-run	http_client.txt	/*http_client-run*
+http_client-select-env-file	http_client.txt	/*http_client-select-env-file*
+http_client-set-env	http_client.txt	/*http_client-set-env*
+http_client-split_direction	http_client.txt	/*http_client-split_direction*
+http_client-ssl-config	http_client.txt	/*http_client-ssl-config*
+http_client-stop	http_client.txt	/*http_client-stop*
+http_client-telescope	http_client.txt	/*http_client-telescope*
+http_client-usage	http_client.txt	/*http_client-usage*
+http_client-verbose	http_client.txt	/*http_client-verbose*
+http_client.txt	http_client.txt	/*http_client.txt*

lua/http_client/commands/init.lua

@@ -3,6 +3,7 @@ local M = {}
 M.request = require('http_client.commands.request')
 M.select_env = require('http_client.commands.select_env')
 M.utils = require('http_client.commands.utils')
+M.response = require('http_client.commands.response')
 
 return M
 

lua/http_client/commands/request.lua

@@ -61,7 +61,7 @@ M.stop_request = function()
 end
 
 M.run_all = function()
- local verbose = vvv.get_verbose_mode()
+    local verbose = vvv.get_verbose_mode()
     vvv.set_verbose_mode(verbose)
 
     local lines = vim.api.nvim_buf_get_lines(0, 0, -1, false)
@@ -72,7 +72,8 @@ M.run_all = function()
     for _, request in ipairs(requests) do
         local env_needed = environment.env_variables_needed(request)
         if env_needed and not next(env) then
-            table.insert(results, string.format("SKIP: %s %s - Environment variables needed but not set", request.method, request.url))
+            table.insert(results,
+                string.format("SKIP: %s %s - Environment variables needed but not set", request.method, request.url))
         else
             request = parser.replace_placeholders(request, env)
             local response = http_client.send_request_sync(request)

lua/http_client/commands/response.lua

@@ -0,0 +1,47 @@
+local M = {}
+local state = require('http_client.state')
+local file_utils = require('http_client.utils.file_utils')
+
+M.save_response = function(opts)
+    local response = state.get_current_response()
+    if not response then
+        vim.notify("No response available to save", vim.log.levels.WARN)
+        return
+    end
+
+    local default_ext = response.content_type == "json" and ".json"
+        or response.content_type == "xml" and ".xml"
+        or response.content_type == "html" and ".html"
+        or ".txt"
+
+    local content = opts and opts.raw and response.raw_body or response.formatted_body
+
+    vim.ui.input({
+        prompt = "Save response as (default extension: " .. default_ext .. "): ",
+        default = "response" .. default_ext,
+        completion = "file"
+    }, function(filename)
+        if not filename then return end
+
+        -- Add default extension if none provided
+        if not filename:match("%.%w+$") then
+            filename = filename .. default_ext
+        end
+
+        -- Write the file using our utility
+        local success, err = file_utils.write_file(filename, content)
+
+        if success then
+            vim.notify(string.format("Response saved to: %s (%s)",
+                    filename,
+                    opts and opts.raw and "raw" or "formatted"),
+                vim.log.levels.INFO)
+        else
+            vim.notify("Failed to save response: " .. tostring(err), vim.log.levels.ERROR)
+        end
+    end)
+end
+
+
+return M
+

lua/http_client/config.lua

@@ -1,30 +1,32 @@
 local M = {}
 
 M.defaults = {
-	default_env_file = ".env.json",
-	request_timeout = 30000, -- 30 seconds
-	split_direction = "right",
-	create_keybindings = true,
-	keybindings = {
-		select_env_file = "<leader>he",
-		set_env = "<leader>hs",
-		run_request = "<leader>hr",
-		stop_request = "<leader>hx",
-		dry_run = "<leader>hd",
-		toggle_verbose = "<leader>hv",
-		copy_curl = "<leader>hc",
-	},
+    default_env_file = ".env.json",
+    request_timeout = 30000, -- 30 seconds
+    split_direction = "right",
+    create_keybindings = true,
+    keybindings = {
+        select_env_file = "<leader>hf",
+        set_env = "<leader>he",
+        run_request = "<leader>hr",
+        stop_request = "<leader>hx",
+        dry_run = "<leader>hd",
+        toggle_verbose = "<leader>hv",
+        copy_curl = "<leader>hc",
+        save_response = "<header>hs",
+    },
 }
 
 M.options = {}
 
 function M.setup(opts)
-	opts = opts or {}
-	M.options = vim.tbl_deep_extend("force", M.defaults, opts) or M.defaults
+    opts = opts or {}
+    M.options = vim.tbl_deep_extend("force", M.defaults, opts) or M.defaults
 end
 
 function M.get(opt)
-	return M.options[opt]
+    return M.options[opt]
 end
 
 return M
+

lua/http_client/core/http_client.lua

@@ -1,6 +1,7 @@
 local M = {}
 local curl = require('plenary.curl')
 local vvv = require('http_client.utils.verbose')
+local state = require('http_client.state')
 
 local current_request = nil
 
@@ -105,7 +106,7 @@ local function prepare_response(request, response)
         formatted_body = format_xml(formatted_body)
     end
 
-    return {
+    local pr = {
         formatted_body = formatted_body,
         headers = response.headers or {},
         status = response.status or "N/A",
@@ -118,6 +119,10 @@ local function prepare_response(request, response)
             test_name = request.test_name or "N/A",
         }
     }
+
+    state.store_response(pr)
+
+    return pr
 end
 
 local function display_response(pr)

lua/http_client/core/parser.lua

@@ -1,5 +1,6 @@
 local M = {}
 local environment = require('http_client.core.environment')
+local url_u = require('http_client.utils.url')
 
 local function trim(s)
     return s:match("^%s*(.-)%s*$")
@@ -198,6 +199,9 @@ M.replace_placeholders = function(request, env)
 
     if request.url then
         request.url = replace(request.url)
+        if (url_u.needs_encoding(request.url)) then
+            request.url = url_u.encode_url(request.url)
+        end
     end
     for k, v in pairs(request.headers) do
         request.headers[k] = replace(v)