Skip to content

Commit cdf449f

Browse files
committed
docs(cli): clarify SyncClient log level control and OpenTelemetry isolation
1 parent 4965c7b commit cdf449f

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

src/humanloop/cli/__main__.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,11 @@ def pull(
219219
220220
Currently only supports syncing Prompt and Agent files. Other file types will be skipped."""
221221
client = get_client(api_key, env_file, base_url)
222-
# Although pull() is available on the Humanloop client, we instantiate SyncClient separately as we need to control its log level
222+
# Although pull() is available on the Humanloop client, we instantiate SyncClient separately to control its log level.
223+
# This allows CLI users to toggle between detailed logging (--verbose) and minimal output without affecting the
224+
# main Humanloop client logger. The SyncClient uses its own logger namespace (humanloop.sdk.sync), making this
225+
# modification isolated from the client's OpenTelemetry setup. This client instance is short-lived and only
226+
# exists for the duration of the CLI command execution.
223227
sync_client = SyncClient(
224228
client, base_dir=local_files_directory, log_level=logging.DEBUG if verbose else logging.WARNING
225229
)

0 commit comments

Comments
 (0)