Nam Pham Dinh ThanhandClaude Opus 5
ab0d26761f
R02-T03 Typed Settings Facade
Khắp nơi đang viết ctx.config.routing.get("switch_mode", "off"). Gõ sai một
chữ thì lặng lẽ nhận mặc định, không ai biết cho tới lúc tính năng "không
hiểu sao không chạy". ProviderSettings / RoutingSettings / SecuritySettings
làm sai tên là lỗi ngay, và kiểu ghi rõ nên đọc là biết confirm_timeout_sec
tính bằng giây.
Là KHUNG NHÌN lên dict sống, không phải dataclass sao chép — sửa qua đây là
sửa vào cấu hình, save() là xuống đĩa, khỏi sinh chuyện đồng bộ hai chiều.
Có raw() để ai thiếu thuộc tính thì dùng tạm, đừng vòng lại config.data.
Bắt cả trường hợp giá trị là null: file cũ hay để null, đọc ra None rồi đem
so sánh số là vỡ.
R02-T06 Schema versioning + phục hồi
config.json hôm nay không có số phiên bản, nên mọi thay đổi hình dạng phải
đoán — _migrate_connectors() đoán "có khoá office nghĩa là file cũ". Giờ:
thiếu schema_version thì coi là v1, mỗi bước là một hàm chạy tuần tự, sao
lưu trước khi nâng, và file mới hơn app thì dùng nguyên trạng chứ không đoán
ngược.
R02-T05 Chuyển API key sang kho bí mật
Là bước v1→v2. Người dùng cập nhật app, mở lên, khoá cũ tự vào keyring và
biến khỏi đĩa — có test cho đúng cảnh đó.
Hai chỗ cố tình không làm:
- Máy chưa có keyring: KHÔNG chuyển, giữ nguyên v1. Thà để khoá trong file
còn hơn xoá đi rồi người dùng mất khoá mà không hiểu vì sao.
- Giá trị "ollama" là bù nhìn (Ollama đòi có api_key nhưng bỏ qua nội
dung), đẩy vào keyring chỉ tổ rác.
Hai chuỗi test trông giống khoá thật bị CASAN Check 1 bắt — đánh dấu
"# casan: allow" kèm lý do, đúng lối thoát đã thiết kế cho cả đội.
150 test xanh (129 + 21 mới). CASAN Check 1 sạch. File mới đều dưới 200 dòng.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
93 lines
3.5 KiB
Python
93 lines
3.5 KiB
Python
"""Typed Settings Facade — R02-T03."""
|
|
from __future__ import annotations
|
|
|
|
from cowork_local.infrastructure.config.settings_facade import (
|
|
ProviderSettings, RoutingSettings, SecuritySettings, Settings,
|
|
)
|
|
from cowork_local.tests.fakes.fake_config import FakeConfigRepository
|
|
|
|
|
|
def test_provider_doc_duoc_ba_truong():
|
|
p = ProviderSettings({"base_url": "http://x/v1", "model": "llama3",
|
|
"api_key": "sk-abc"})
|
|
assert p.base_url == "http://x/v1"
|
|
assert p.model == "llama3"
|
|
assert p.api_key == "sk-abc"
|
|
assert p.configured is True
|
|
|
|
|
|
def test_ollama_khong_can_khoa_van_tinh_la_da_cau_hinh():
|
|
"""Điều kiện là có base_url và model, không phải có api_key — Ollama chạy
|
|
cục bộ nên không cần khoá."""
|
|
p = ProviderSettings({"base_url": "http://localhost:11434/v1", "model": "llama3"})
|
|
assert p.api_key == ""
|
|
assert p.configured is True
|
|
|
|
|
|
def test_thieu_model_thi_chua_cau_hinh():
|
|
assert ProviderSettings({"base_url": "http://x/v1"}).configured is False
|
|
assert ProviderSettings({}).configured is False
|
|
|
|
|
|
def test_gia_tri_None_tra_ve_mac_dinh_chu_khong_None():
|
|
"""File cấu hình cũ hay có khoá để null. Đọc ra None rồi đem so sánh số là
|
|
vỡ — nên khung nhìn phải nuốt luôn trường hợp này."""
|
|
r = RoutingSettings({"switch_mode": None, "min_score_gain": None,
|
|
"confirm_timeout_sec": None})
|
|
assert r.switch_mode == "off"
|
|
assert r.min_score_gain == 0.05
|
|
assert r.confirm_timeout_sec == 60
|
|
|
|
|
|
def test_routing_kieu_du_lieu_dung():
|
|
r = RoutingSettings({"switch_mode": "auto", "min_score_gain": "0.2",
|
|
"confirm_timeout_sec": "90"})
|
|
assert r.enabled is True
|
|
assert isinstance(r.min_score_gain, float) and r.min_score_gain == 0.2
|
|
assert isinstance(r.confirm_timeout_sec, int) and r.confirm_timeout_sec == 90
|
|
|
|
|
|
def test_tat_dinh_tuyen():
|
|
assert RoutingSettings({"switch_mode": "off"}).enabled is False
|
|
assert RoutingSettings({}).enabled is False
|
|
|
|
|
|
def test_sua_qua_khung_nhin_la_sua_vao_dict_that():
|
|
"""Khung nhìn, không phải bản sao — sửa xong gọi save() là xuống đĩa."""
|
|
d = {"switch_mode": "off"}
|
|
RoutingSettings(d).switch_mode = "auto"
|
|
assert d["switch_mode"] == "auto"
|
|
|
|
|
|
def test_raw_de_khong_ai_bi_ket():
|
|
d = {"switch_mode": "auto", "khoa_chua_dua_vao_khung_nhin": 1}
|
|
assert RoutingSettings(d).raw()["khoa_chua_dua_vao_khung_nhin"] == 1
|
|
|
|
|
|
def test_security_mac_dinh_la_bat():
|
|
"""Mặc định an toàn: thiếu cấu hình thì bật kiểm tra, không phải tắt."""
|
|
s = SecuritySettings({})
|
|
assert s.enabled is True
|
|
assert s.validate_prompt is True
|
|
assert s.validate_commands is True
|
|
assert s.cowork_confirm_commands is True
|
|
assert s.command_ai_check is False # trừ cái này: gọi AI, tốn tiền
|
|
|
|
|
|
def test_settings_noi_vao_repo():
|
|
repo = FakeConfigRepository(active_provider="openai",
|
|
routing={"switch_mode": "auto"},
|
|
agent_security={"cowork_confirm_commands": False})
|
|
s = Settings(repo)
|
|
assert s.provider().model == "gpt-4o-mini"
|
|
assert s.routing.enabled is True
|
|
assert s.security.cowork_confirm_commands is False
|
|
|
|
|
|
def test_doi_provider_thi_khung_nhin_theo_ngay():
|
|
repo = FakeConfigRepository(active_provider="ollama")
|
|
s = Settings(repo)
|
|
assert s.provider().model == "llama3"
|
|
repo.set_active_provider("openai")
|
|
assert s.provider().model == "gpt-4o-mini"
|