Release v0.2.0 - #4
Conversation
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014V513wLcoqKEPYtbsCApBU
Greptile SummaryThis PR releases
Confidence Score: 4/5Safe to merge after fixing the missing constructor parameter. The rename and dependency updates are consistent, but the documented timeout API currently raises
What T-Rex did
Important Files Changed
Sequence Diagram%%{init: {'theme': 'neutral'}}%%
sequenceDiagram
participant Caller
participant PipelexAPIClient
participant Env as PIPELEX env
participant Mthds as mthds.load_config
participant API as Pipelex/MTHDS API
Caller->>PipelexAPIClient: __init__(api_key, base_url)
PipelexAPIClient->>Mthds: load_config()
Mthds-->>PipelexAPIClient: api_key, base_url
PipelexAPIClient->>Env: read PIPELEX_API_KEY / PIPELEX_BASE_URL
PipelexAPIClient->>PipelexAPIClient: resolve api_key/base_url and validate host-only URL
Caller->>PipelexAPIClient: start_client()
PipelexAPIClient->>API: "requests to {base_url}/v1/* with optional Authorization"
API-->>PipelexAPIClient: protocol/product responses
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
sequenceDiagram
participant Caller
participant PipelexAPIClient
participant Env as PIPELEX env
participant Mthds as mthds.load_config
participant API as Pipelex/MTHDS API
Caller->>PipelexAPIClient: __init__(api_key, base_url)
PipelexAPIClient->>Mthds: load_config()
Mthds-->>PipelexAPIClient: api_key, base_url
PipelexAPIClient->>Env: read PIPELEX_API_KEY / PIPELEX_BASE_URL
PipelexAPIClient->>PipelexAPIClient: resolve api_key/base_url and validate host-only URL
Caller->>PipelexAPIClient: start_client()
PipelexAPIClient->>API: "requests to {base_url}/v1/* with optional Authorization"
API-->>PipelexAPIClient: protocol/product responses
Prompt To Fix All With AIFix the following 1 code review issue. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 1
pipelex_sdk/client.py:158
**Missing timeout parameter**
The release notes and inline docs say callers can pass `request_timeout_seconds`, but the constructor still only accepts `api_key` and `base_url`. `PipelexAPIClient(request_timeout_seconds=...)` raises `TypeError`, so users cannot configure the inherited protocol timeout despite the new documented API.
Reviews (1): Last reviewed commit: "Release v0.2.0" | Re-trigger Greptile |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d3ee5bfa27
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Release v0.2.0
Bumps version from
0.1.1to0.2.0.Changelog
Added
request_timeout_secondsconstructor parameter toPipelexAPIClient, setting a per-instance blocking-execute ceiling for the inherited protocol routes (execute,start,validate,models,version).Changed
PipelexAPIClientconstructor parameters and attributes to match themthdsbase client and the@pipelex/sdkJavaScript counterpart:api_token→api_keyandapi_base_url→base_url. (Migration: update all instantiations and property reads to the new names.)PIPELEX_API_URL→PIPELEX_BASE_URLand themthdsfallbackMTHDS_API_URL→MTHDS_BASE_URL. No read aliases are kept for the old names.mthdsdependency from>=0.6.1to>=0.7.0, and updated config loading to importload_configfrommthds.config(previouslyload_credentialsfrommthds.config.credentials).README.md,CLAUDE.md,docs/architecture.md) and unit tests to reflect the new client signature, environment variables, andmthdsconfiguration.🤖 Generated with Claude Code
https://claude.ai/code/session_014V513wLcoqKEPYtbsCApBU