Skip to content

refactor: standardize logger field name in config REST classes #540

Description

@ginccc

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:

  1. Rename the field: private static final Logger log to private static final Logger LOGGER
  2. 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 (7 files)

log -> LOGGER (5 files):

  • src/main/java/ai/labs/eddi/configs/admin/rest/RestOrphanAdmin.java
  • src/main/java/ai/labs/eddi/configs/agents/rest/RestAgentStore.java
  • src/main/java/ai/labs/eddi/configs/descriptors/rest/RestDocumentDescriptorStore.java
  • src/main/java/ai/labs/eddi/configs/properties/rest/RestPropertiesStore.java
  • src/main/java/ai/labs/eddi/configs/workflows/rest/RestWorkflowStore.java

LOG -> LOGGER (2 files):

  • src/main/java/ai/labs/eddi/configs/channels/rest/RestChannelIntegrationStore.java
  • src/main/java/ai/labs/eddi/configs/groups/rest/RestAgentGroupStore.java

Acceptance criteria

  • All 7 files use LOGGER
  • ./mvnw compile && ./mvnw test pass

Metadata

Metadata

Assignees

No one assigned

    Labels

    good first issueClear scope, low risk, pattern to follow - ideal for newcomershelp wantedrefactoringCode quality improvement with no behavior changesize: SSmall: about 1 hour, 2-8 files, mechanical pattern-following

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions