From cd0b1be46c0e32633b12e24736095c2c8b8f52ee Mon Sep 17 00:00:00 2001 From: hkfires <10558748+hkfires@users.noreply.github.com> Date: Fri, 26 Sep 2025 21:42:42 +0800 Subject: [PATCH] fix(log): Reduce noise on metadata updates and provider sync --- internal/api/server.go | 6 +++--- internal/registry/model_registry.go | 3 +-- sdk/cliproxy/service.go | 6 +++--- 3 files changed, 7 insertions(+), 8 deletions(-) diff --git a/internal/api/server.go b/internal/api/server.go index 2a35b642..28bda337 100644 --- a/internal/api/server.go +++ b/internal/api/server.go @@ -559,10 +559,10 @@ func (s *Server) applyAccessConfig(oldCfg, newCfg *config.Config) { } s.accessManager.SetProviders(providers) if len(added)+len(updated)+len(removed) > 0 { - log.Infof("server request auth providers reconciled (added=%d updated=%d removed=%d)", len(added), len(updated), len(removed)) - log.Debugf("server request auth provider details - added=%v updated=%v removed=%v", added, updated, removed) + log.Debugf("auth providers reconciled (added=%d updated=%d removed=%d)", len(added), len(updated), len(removed)) + log.Debugf("auth provider changes details - added=%v updated=%v removed=%v", added, updated, removed) } else { - log.Debug("server request auth providers unchanged after config update") + log.Debug("auth providers unchanged after config update") } } diff --git a/internal/registry/model_registry.go b/internal/registry/model_registry.go index 44b6991c..29607a5b 100644 --- a/internal/registry/model_registry.go +++ b/internal/registry/model_registry.go @@ -273,8 +273,7 @@ func (r *ModelRegistry) RegisterClient(clientID, clientProvider string, models [ } if len(added) == 0 && len(removed) == 0 && !providerChanged { - // Only metadata (e.g., display name) changed. - misc.LogCredentialSeparator() + // Only metadata (e.g., display name) changed; skip separator when no log output. return } diff --git a/sdk/cliproxy/service.go b/sdk/cliproxy/service.go index 7c96abe7..725bc4e6 100644 --- a/sdk/cliproxy/service.go +++ b/sdk/cliproxy/service.go @@ -122,10 +122,10 @@ func (s *Service) refreshAccessProviders(cfg *config.Config) { } s.accessManager.SetProviders(providers) if len(added)+len(updated)+len(removed) > 0 { - log.Infof("request auth providers reconciled (added=%d updated=%d removed=%d)", len(added), len(updated), len(removed)) - log.Debugf("provider changes details - added=%v updated=%v removed=%v", added, updated, removed) + log.Debugf("auth providers reconciled (added=%d updated=%d removed=%d)", len(added), len(updated), len(removed)) + log.Debugf("auth provider changes details - added=%v updated=%v removed=%v", added, updated, removed) } else { - log.Debug("request auth providers unchanged after config reload") + log.Debug("auth providers unchanged after config reload") } }