fix(registry): prefer configured registry volume over legacy session defaults

[bcastelino]

Summary

Registry configuration resolution now treats environment/settings values (registry_catalog, registry_schema, registry_volume) as authoritative in the last-resort fallback of RegistryCfg.from_domain, using legacy per-session domain.settings["registry"] values only when the configured values are unset. Previously the order was inverted (reg.get(...) or settings...), so a stale session value could shadow the configured environment — local Registry init kept selecting the legacy OntoBricksRegistry default even with REGISTRY_VOLUME=ontology loaded, causing binary-archive paths and Delta views to resolve against the wrong Volume when no bound Volume or Lakebase registry row was present.

Linked Issue / milestone

N/A

Plan

N/A, single-file precedence fix with regression coverage.

Type of change

• feat — new feature
• fix — bug fix
• docs — documentation only
• refactor — no behaviour change
• test — tests only
• perf — perf improvement
• ci / build / chore — tooling

Author checklist

• Conventional Commit PR title (<type>(<scope>): <subject>).
• changelogs/<today>.log updated with title + context + numbered changes + files + test result.
• Tests added or modified for every behaviour change.
• uv run pytest tests/<scope>/ green locally.
• pre-commit run --all-files clean (or skipped hooks documented).
• If src/agents/** or any MLflow-traced LLM path changed: N/A — no agent/LLM paths touched.
• If src/mcp-server/** changed: N/A — no MCP changes.
• No gsd-* references re-introduced.

MLflow eval run

N/A, not an agent PR.

Test plan

• python -m pytest tests/units/registry/test_registry.py -q --no-cov → 55 passed
• uv run pytest tests/ -m "not e2e and not property and not eval" --cov-fail-under=90
• Per-package coverage thresholds (scripts/check_coverage.py) green for touched packages.

Reviewer hint

Focus is the fallback return cls(...) block in src/back/objects/registry/RegistryService.py (from_domain): the three fields now read settings.* or reg.get(...) instead of reg.get(...) or settings.*. Behaviour is unchanged when env/settings are empty. Regression coverage: tests/units/registry/test_registry.py::TestRegistryCfgFromDomain::test_settings_override_legacy_session_registry.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.