feat(ess-langsmith-client): replace langsmith-client - #136
Merged
Merged
Conversation
Brings in the renamed package and retires the older, smaller copy it supersedes. The new version is a superset: 42 files against 14, adding the `agent_test` deployment smoke-tester, per-subcommand docs, a README, and tests. Two behavioral differences matter. `merge_secrets` no longer harvests credentials from the environment implicitly -- it reads only the keys a caller names via `auto_detect_keys`, so a deploy cannot ship a secret nobody asked for. And `pyproject.toml` now declares `readme` and `license = "Apache-2.0"`, so the built wheel carries a description and a license instead of omitting both; that needs `hatchling>=1.27`, which is the version understanding the bare SPDX string form. Workspace member swapped into the alphabetical `ess-*` block and the lock regenerated: `langsmith-client` out, `ess-langsmith-client` in.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds
packages/python/ess-langsmith-clientand retires the olderpackages/python/langsmith-clientit supersedes. The new version is a superset — 42 files against 14 — adding theagent_testdeployment smoke-tester, per-subcommand docs underdocs/, a README, and a test suite.Two behavioral differences are worth calling out:
merge_secretsno longer reads the environment implicitly. It previously harvested a hardcoded list of keys (fourAZURE_OPENAI_*plusTAVILY_API_KEY) from whatever shell invoked a deploy, which was documented nowhere and meant a deployment could receive a credential the caller never named. It now reads only the keys passed viaauto_detect_keys, defaulting to none. Callers that want convenience defaults supply their own list.readmenorlicensewas set, so both fields were absent from the builtMETADATA. Now setsreadme = "README.md"andlicense = "Apache-2.0", which requireshatchling>=1.27— the version that understands the bare SPDX string form.The workspace member moves into the alphabetical
ess-*block, anduv.lockis regenerated:langsmith-clientout,ess-langsmith-clientin. Git detected 7 of the moved files as renames, so history follows them.Test plan
uv sync --all-packages— swapslangsmith-client==0.1.0foress-langsmith-client==0.1.0uv run pytest packages/python/ess-langsmith-client— 99 passeduv run ruff checkandruff format --checkcleanuv run langsmith-client --versionreports 0.1.0skills/ess/summarize-change-log/, whose examples quote real historical commits