Skip to content

fix: clarify active profile auth display#121

Draft
Ramon Nogueira (ramon-langchain) wants to merge 1 commit into
mainfrom
ramonn/profile-active-auth-display
Draft

fix: clarify active profile auth display#121
Ramon Nogueira (ramon-langchain) wants to merge 1 commit into
mainfrom
ramonn/profile-active-auth-display

Conversation

@ramon-langchain
Copy link
Copy Markdown
Member

Summary

  • stop marking a profile active when flag/env API-key auth takes precedence
  • print the env API-key precedence note below profile list and profile show tables in pretty mode
  • centralize LANGSMITH_API_KEY lookup in config for the command resolver paths

Test Plan

  • go test ./internal/config ./internal/cmd ./internal/cmdutil -run 'TestProfile|TestAuth|TestGetAPIKey|TestResolve'
  • deslop internal/cmd/profile.go internal/cmd/profile_test.go internal/cmd/root.go internal/cmd/auth.go internal/cmdutil/resolve.go internal/config/config.go

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant