Make Settings immutable (frozen=True) and fix all test mutations

Fork: 0

root / navi-1

Browse code Make Settings immutable (frozen=True) and fix all test mutations - Add frozen=True to SettingsConfigDict in navi/config.py - Convert model_validator to mode="before" since mode="after" cannot mutate frozen instances - Replace all field-level monkeypatches in tests with whole-Settings object replacement - Ensure cross-module settings consistency (content_store, session_files, share_file, content_publish, filesystem) 392 passed, 1 skipped Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> feature/navi-code master vmkdemo
1 parent d2695c7 commit a97c203c3197247697223158af18f4d047e9381c Eugene Sukhodolskiy authored on 18 May

Browse code

- Add frozen=True to SettingsConfigDict in navi/config.py
- Convert model_validator to mode="before" since mode="after" cannot mutate frozen instances
- Replace all field-level monkeypatches in tests with whole-Settings object replacement
- Ensure cross-module settings consistency (content_store, session_files, share_file, content_publish, filesystem)

392 passed, 1 skipped

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

feature/navi-code master vmkdemo

1 parent d2695c7 commit a97c203c3197247697223158af18f4d047e9381c

Eugene Sukhodolskiy authored on 18 May

Patch

Unified Split

Showing 10 changed files

Ignore Space Show notes View navi/config.py

Ignore Space Show notes View tests/integration/conftest.py

Ignore Space Show notes View tests/unit/api/test_session_files.py

Ignore Space Show notes View tests/unit/core/test_context_builder.py

Ignore Space Show notes View tests/unit/core/test_registry.py

Ignore Space Show notes View tests/unit/llm/test_ollama.py

Ignore Space Show notes View tests/unit/test_content_store.py

Ignore Space Show notes View tests/unit/tools/test_content_publish.py

Ignore Space Show notes View tests/unit/tools/test_filesystem.py

Ignore Space Show notes View tests/unit/tools/test_share_file.py

Show line notes below