Greptile Summary

This PR exposes HTTP client configuration on pklr's evaluator so callers (e.g. hk) can inject proxy settings, custom CA certificates, and timeouts without having to add reqwest as a direct dependency.

Key changes:

• Evaluator::set_http_client(client) — new setter that replaces the internal reqwest::Client.
• eval_to_json_with_client(path, Option<reqwest::Client>) — new top-level convenience function; eval_to_json is cleanly refactored to delegate to it with None.
• pub use reqwest — re-exports reqwest at the crate root so consumers can construct a Client without pinning a separate reqwest version. Note that the re-exported crate has default-features = false, features = ["rustls-tls"] only; consumers using pklr::reqwest for anything beyond client construction (e.g. the json feature) will find those features absent.
• One issue found: set_http_client replaces the client but does not clear http_cache. For long-lived Evaluator instances that swap clients mid-session, previously fetched URLs will still be served from cache, bypassing the new client's configuration (proxy, cert pinning, etc.).

Confidence Score: 4/5

• Safe to merge with minor concern about stale HTTP cache after client replacement.
• The changes are small, well-scoped, and correct for the primary use case (fresh Evaluator per call via the convenience functions). The one issue — http_cache not being cleared in set_http_client — only affects long-lived Evaluator instances that swap clients mid-session, which is not the advertised usage pattern. The re-export of reqwest is idiomatic and the refactoring of eval_to_json is clean with no behavioral change.
• src/eval.rs — review the set_http_client implementation regarding cache invalidation.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Caller
    participant lib as pklr::lib
    participant Eval as Evaluator
    participant HTTP as reqwest::Client
    participant Cache as http_cache

    Note over Caller,Cache: eval_to_json_with_client(path, Some(client))
    Caller->>lib: eval_to_json_with_client(path, Some(client))
    lib->>Eval: Evaluator::new()
    lib->>Eval: set_base_path(path.parent())
    lib->>Eval: set_http_client(client)
    Eval->>HTTP: replace http_client field
    lib->>Eval: eval_source(&source, path)
    alt remote import encountered
        Eval->>Cache: check http_cache[url]
        alt cache miss
            Eval->>HTTP: client.get(url).send()
            HTTP-->>Eval: response body
            Eval->>Cache: store in http_cache[url]
        else cache hit
            Cache-->>Eval: cached body
        end
    end
    Eval-->>lib: Value
    lib-->>Caller: serde_json::Value

    Note over Caller,Cache: eval_to_json(path) — convenience wrapper
    Caller->>lib: eval_to_json(path)
    lib->>lib: eval_to_json_with_client(path, None)
    Note right of Eval: default reqwest::Client used

_{Reviews (1): Last reviewed commit: "feat: expose set_http_client and eval_to..." | Re-trigger Greptile}