Fix - address declarative config warning

[cleverchuk]

TLDR

Reorders config parsing so registered ConfigParser implementations are checked before the type-based fallback, eliminating a dead intermediate assignment that produced an unused-result warning.

Technical Details

DeclarativeConfigParser.parse() iterates over YAML config keys and resolves each to a value. Two parsing strategies exist:

1. Registered parsers — ConfigParser implementations loaded via ServiceLoader, keyed by config name (e.g., agent.logging uses a structured parser).
2. Type-based fallback — dispatches on ConfigProperty.getTypeClass() to call getString, getBoolean, getInt, or getLong.

Previously, the type-based parse ran unconditionally for every key, and then a separate containsKey/get check on the parser registry would overwrite the result. This meant the type-based conversion could trigger a warning for some keys when there's a type-mismatch. This change avoids it by using the config parser first which handles structured types.

The fix restructures the logic into a single if/else: look up the registered parser first via register.get(key) with a null-check, and only fall back to type-based parsing when no registered parser exists. This also collapses the prior double map lookup (containsKey + get) into a single get.

Behavior is unchanged — registered parsers always took precedence, and they still do. The difference is that the type-based branch no longer executes when it would be immediately overwritten.

Test services data

1. e-1712644058766987264
2. e-1712643928659124224
3. e-1742334541200846848
4. e-1777406072376840192

[Copilot]

Pull request overview

Restructures DeclarativeConfigParser.parse() to prioritize registered ConfigParser implementations before falling back to type-based parsing, eliminating a dead/overwritten intermediate parse result (and the associated unused-result warning).

Changes:

• Looks up a registered ConfigParser first (single register.get(key) + null-check) and only executes type-based parsing when absent.
• Removes the prior unconditional type-based parse and the redundant containsKey + get double lookup.

[tammy-baylis-swi]

👍

[jerrytfleung]

LGTM