feature/Client-identification · L-260923-fd0e26 — User-Agent client surface in analytics

[thomashebrard]

PipelexAPIClient now sends a User-Agent on every request, authenticated or anonymous, of the form [app_info] pipelex-sdk-python/ mthds-python/ python/<x.y.z> (; ), following the workspace client-identification spec so the platform can attribute SDK traffic. A new app_info constructor argument takes a Stripe-style AppInfo that puts the integrator first and refuses an invalid token with ValueError; the builder lives in pipelex_sdk/user_agent.py because the pinned mthds 0.14.0 does not ship one yet.

Advances L-260923-fd0e26

🤖 Generated with Claude Code

[thomashebrard]

/rev pass, profile 2, round 1 (bar: open), on 81f9663. Reviewers: cubic, Codex (review), and the official code-review (low, clean). The focus was conformance to the client-identification spec and consistency of app_info validation across the SDKs.

Fixed in 13ae4ad: an empty version or url now counts as absent, as the spec requires. The url is restricted to visible ASCII, so a non-ASCII URL is refused at construction instead of making httpx raise on every request. The runtime comment now keeps whichever platform part is a readable token, which matches mthds-python.

Deferred to wip/client-identification/plan.md under "Deferred from review": the AppInfo shape does not match mthds (strict, with a list rather than a tuple for details, unverified); it is settled when this SDK switches to the mthds builder. The same empty-value and URL defects exist in mthds-python and mthds-js; those were verified and are tracked there.

Verdict: round 2 — bar defects — profile 3.

[thomashebrard]

/rev pass — profile 3, round 2, bar defects, on 13ae4ad. Reviewers that produced a review: cubic and code-review (Codex failed: workspace out of credits). No defects found, nothing fixed.