Tech debt: Config.load() scaling — consider splitting config by service #184

New Issue

shahondin1624 · 2026-03-11T07:43:13+01:00

shahondin1624 commented

2026-03-11 07:43:13 +01:00

Background

Found during periodic refactoring review after 5 completed stories.

Problem

As more services are added, Config.load() patterns may accumulate more fields and env vars. Currently manageable but worth monitoring.

Suggested Fix

Consider splitting config into per-service config modules if the number of config fields grows beyond ~15-20, or use a config framework that supports hierarchical config.

Labels

Type: refactor, Priority: low

## Background Found during periodic refactoring review after 5 completed stories. ## Problem As more services are added, `Config.load()` patterns may accumulate more fields and env vars. Currently manageable but worth monitoring. ## Suggested Fix Consider splitting config into per-service config modules if the number of config fields grows beyond ~15-20, or use a config framework that supports hierarchical config. ## Labels Type: refactor, Priority: low

shahondin1624 referenced this issue

2026-03-11 10:48:47 +01:00

Refactor: Config.load() boilerplate — replace manual field-by-field parsing #199

shahondin1624 commented

2026-03-11 10:49:39 +01:00

This has progressed: Config.load() is now 180 lines of boilerplate with 9 sub-config sections. Created #199 with a concrete fix proposal (generic _load_dataclass helper). This issue can be closed once #199 is addressed.

This has progressed: `Config.load()` is now 180 lines of boilerplate with 9 sub-config sections. Created #199 with a concrete fix proposal (generic `_load_dataclass` helper). This issue can be closed once #199 is addressed.

shahondin1624 commented

2026-03-11 11:32:24 +01:00

Resolved by #199 (PR #215). Config.load() boilerplate has been replaced with a generic _load_dataclass() helper, reducing the method from ~180 to ~30 lines. The scaling concern is no longer relevant.

Resolved by #199 (PR #215). Config.load() boilerplate has been replaced with a generic `_load_dataclass()` helper, reducing the method from ~180 to ~30 lines. The scaling concern is no longer relevant.

shahondin1624 closed this issue

2026-03-11 11:32:24 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: llm-multiverse/llm-multiverse#184