Skip to content

chore: migrate remaining config singleton uses in transform.py, db.py, status.py #175

Description

@JesuFemi-O

Context

#172 migrated the four ingestion command modules (sources.py, run_all.py, sync_cmd.py, explore.py) away from the module-level config singleton to load_config() constructed at invocation time.

Three modules still import the singleton directly:

  • src/tycoon/commands/transform.py
  • src/tycoon/commands/db.py
  • src/tycoon/commands/status.py

The tell is test_templates_e2e.py::_rebind_config, which still needs to patch both transform_mod.config and cfg_mod.config to make e2e tests work.

Goal

Migrate these three modules to load_config() following the same pattern as #172, and drop the remaining monkeypatch.setattr calls from _rebind_config.

Depends on

#172

Metadata

Metadata

Assignees

No one assigned

    Labels

    area: configtycoon.yml schema + config singletontype: taskIndividual implementation tasks

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions