Add release notes/docs, minor text adjustments

Author: ikappaki

.github/workflows/test.yml

@@ -86,7 +86,7 @@ jobs:
           # while downloading dependencies. Workaround: offload some
           # of the deps download to the clojure CLI tool.
           #
-          # See https://clojure.atlassian.net/jira/software/c/projects/TBUILD/issues?jql=project%20%3D%20TBUILD%20ORDER%20BY%20created%20DESC&selectedIssue=TBUILD-47
+          # See https://clojure.atlassian.net/browse/TBUILD-47
           clojure -P
           bb prod-cli
 

.gitignore

@@ -36,5 +36,5 @@ docs/README.md
 docs/CHANGELOG.md
 docs/images/
 
-integration-test/out
+/integration-test/out/
 reachability-metadata.json

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - Add dynamic model discovery via `fetchModels` provider config for OpenAI-compatible `/models` endpoints
 - Improve error handling for incompatible models messages in chat. #209
+- Add basic username/password proxy authentication support and recognize lowercase http[s]_proxy env var alongside HTTP[S]_PROXY. #248
 
 ## 0.87.2
 

docs/configuration.md

@@ -57,6 +57,19 @@ It's possible to retrieve content of any configs with a string value using the `
 - `classpath`: `${classpath:path/to/eca/file}` to get a file content from [ECA's classpath](https://github.com/editor-code-assistant/eca/tree/master/resources)
 - `netrc`: Support Unix RC [credential files](./models.md#credential-file-authentication)
 
+## Proxy Configuration
+
+ECA supports proxies with basic cleartext authentication via the de-facto env vars:
+
+```bash
+HTTP_PROXY="http://user:pass@host:port"
+HTTPS_PROXY="http://user:pass@host:port"
+http_proxy="http://user:pass@host:port"
+https_proxy="http://user:pass@host:port"
+```
+
+Lowercase var wins if both are set. Credentials (if used) must match for HTTP and HTTPS.
+
 ## Providers / Models
 
 For providers and models configuration check the [dedicated models section](./models.md#custom-providers).

scripts/make.clj

@@ -102,7 +102,7 @@
     :list-ns {:desc "List available test namespaces."
               :alias :l}
 
-    :ns {:desc "Comma-separated list of test namespaces to run.."
+    :ns {:desc "Comma-separated list of test namespaces to run."
          :type :string
          :validate (fn [s]
                      (every? #(some #{%} entrypoint/namespaces)

src/eca/proxy.clj

@@ -43,7 +43,7 @@
 
 (defn proxy-urls-system-env-get
   "Returns a map of the HTTP and HTTPS proxy environment variables,
-  preferring lowercase variable names over uppercase:
+  preferring lowercase variable names over uppercase.
 
   The map includes:
 

test/eca/client_test_helpers.clj

@@ -29,11 +29,6 @@
   32 KB and does not buffer responses."
   [handler-fn]
   (proxy [HttpFiltersSource] []
-    (clientToProxyRequest [http-obj]
-      ;; always return nil to accept the request
-      ;; or you can normalize the URI here if you want
-      nil)
-
     (filterRequest [original-request ctx]
       (proxy [HttpFiltersAdapter] [original-request]
         (proxyToServerRequest [http-obj]
@@ -179,13 +174,12 @@
   nil)
 
 (defmacro with-client-proxied
-  "Runs BODY with a temporary LittleProxy server active on a random local port,
-  while setting `eca.client-http/*hato-http-client*` to route traffic through the
-  proxy for any Hato requests made in BODY.
-
-  Starts a proxy using the provided request HANDLER-FN and optional OPTS. Ensures
-  the proxy is shut down after BODY executes, and `eca.client-http/*hato-http-client*`
-  is reset to nil.
+  "Runs BODY with a temporary LittleProxy server on a random local port,
+  configuring `eca.client-http/*hato-http-client*` so all Hato
+  requests in BODY are routed through the proxy. The proxy is started
+  using the provided HANDLER-FN and optional OPTS, and is always shut
+  down afterward, with `eca.client-http/*hato-http-client*` reset to
+  nil.
 
   During execution, any calls to `eca.client-http/merge-with-global-http-client`
   are recorded in `*http-client-captures*` as a sequence of merged options maps.

test/eca/llm_providers/openai_chat_test.clj

@@ -22,7 +22,7 @@
                   :model "ibm/granite-4-h-tiny",
                   :choices [{:index 0,
                              :message {:role "assistant",
-                                       :content "Hello from proxy!"
+                                       :content "Hello there!"
                                        :tool_calls []},
                              :logprobs nil,
                              :finish_reason "stop"}]
@@ -47,7 +47,7 @@
                   :uri "/v1/chat/completions"
                   :body body}
                  (select-keys @req* [:method :uri :body])))
-          (is (= {:output-text "Hello from proxy!"}
+          (is (= {:output-text "Hello there!"}
                  (select-keys response [:output-text]))))))))
 
 (deftest normalize-messages-test

test/eca/llm_providers/openai_test.clj

@@ -13,7 +13,7 @@
           (reset! req* req)
           ;; fake a successful non-stream JSON response
           {:status 200
-           :body {:output [{:content [{:text "Hello from responses proxy!"}]}]}})
+           :body {:output [{:content [{:text "Hello from responses!"}]}]}})
 
         (let [body {:model "mymodel"
                     :input "hi"
@@ -31,7 +31,7 @@
                  (select-keys @req* [:method :uri :body])))
 
           ;; parsed response
-          (is (= {:output-text "Hello from responses proxy!"}
+          (is (= {:output-text "Hello from responses!"}
                  (select-keys response [:output-text]))))))))
 
 (deftest oauth-authorize-test