Background
The project convention for logger fields is LOGGER (used in 96+ files), but some older files use log or LOG. Standardizing improves grep-ability and consistency across the codebase.
How
In each file:
- Rename the field:
private static final Logger log to private static final Logger LOGGER
- Replace all usages:
log.info( to LOGGER.info(, log.warn( to LOGGER.warn(, etc.
Tip: IntelliJ's Rename refactoring (Shift+F6 on the field name) handles this safely per file.
Why this is safe
The field is private static final -- file-scoped with zero external references. This is a pure find-and-replace within each file.
Pattern to follow
Any file in src/main/java/ai/labs/eddi/engine/mcp/ uses the LOGGER convention.
Files to update (log -> LOGGER, 8 files)
src/main/java/ai/labs/eddi/engine/exception/ResourceStoreExceptionMapper.java
src/main/java/ai/labs/eddi/engine/httpclient/impl/HttpClientWrapper.java
src/main/java/ai/labs/eddi/engine/internal/RestAgentAdministration.java
src/main/java/ai/labs/eddi/engine/internal/RestAgentManagement.java
src/main/java/ai/labs/eddi/engine/internal/RestCoordinatorAdmin.java
src/main/java/ai/labs/eddi/engine/internal/RestLogAdmin.java
src/main/java/ai/labs/eddi/engine/memory/rest/RestConversationStore.java
src/main/java/ai/labs/eddi/engine/runtime/rest/interceptors/DocumentDescriptorFilter.java
Acceptance criteria
Background
The project convention for logger fields is
LOGGER(used in 96+ files), but some older files uselogorLOG. Standardizing improves grep-ability and consistency across the codebase.How
In each file:
private static final Logger logtoprivate static final Logger LOGGERlog.info(toLOGGER.info(,log.warn(toLOGGER.warn(, etc.Why this is safe
The field is
private static final-- file-scoped with zero external references. This is a pure find-and-replace within each file.Pattern to follow
Any file in
src/main/java/ai/labs/eddi/engine/mcp/uses theLOGGERconvention.Files to update (
log->LOGGER, 8 files)src/main/java/ai/labs/eddi/engine/exception/ResourceStoreExceptionMapper.javasrc/main/java/ai/labs/eddi/engine/httpclient/impl/HttpClientWrapper.javasrc/main/java/ai/labs/eddi/engine/internal/RestAgentAdministration.javasrc/main/java/ai/labs/eddi/engine/internal/RestAgentManagement.javasrc/main/java/ai/labs/eddi/engine/internal/RestCoordinatorAdmin.javasrc/main/java/ai/labs/eddi/engine/internal/RestLogAdmin.javasrc/main/java/ai/labs/eddi/engine/memory/rest/RestConversationStore.javasrc/main/java/ai/labs/eddi/engine/runtime/rest/interceptors/DocumentDescriptorFilter.javaAcceptance criteria
LOGGER./mvnw compile&&./mvnw testpass