refactor: rename class, clean noqa comments, remove xfail test (#82)

Author: VioletsOleander

.github/workflows/docs.yml

@@ -25,12 +25,10 @@ jobs:
       - uses: astral-sh/setup-uv@v7
         with:
           enable-cache: true
-      - name: Install dependencies
-        run: uv add --no-sync zensical mkdocstrings-python
       - name: Generate API reference markdown files
         run: python scripts/generate_api_reference.py
       - name: Build documentation site
-        run: uv run --no-sync zensical build --clean
+        run: uv run --no-install-project --with zensical --with mkdocstrings-python zensical build --clean
       - name: Upload artifact
         uses: actions/upload-pages-artifact@v4
         with:

src/naive_speculate/dependency/maker.py

@@ -1,5 +1,7 @@
 """Provide factory functions to conduct creation of various components."""
 
+# ruff: noqa: PLC0415
+
 from typing import TYPE_CHECKING
 
 from naive_speculate.config.registry import InferencerType, KVCacheType, LanguageModelType
@@ -16,7 +18,7 @@ def make_lm(model_name: str) -> LanguageModel:
     family_name = model_name.split("/")[0].lower()
     match LanguageModelType[family_name]:
         case LanguageModelType.QWEN3:
-            import naive_speculate.infer.lm.qwen3 as impl_module  # noqa: PLC0415
+            import naive_speculate.infer.lm.qwen3 as impl_module
 
     lm_class = impl_module.LanguageModelImpl
     return lm_class(model_name=model_name)
@@ -25,9 +27,9 @@ def make_lm(model_name: str) -> LanguageModel:
 def make_inferencer(language_model: LanguageModel, inferencer_type: InferencerType) -> Inferencer:
     match inferencer_type:
         case InferencerType.BASIC:
-            import naive_speculate.infer.inferencer.basic as impl_module  # noqa: PLC0415
+            import naive_speculate.infer.inferencer.basic as impl_module
         case InferencerType.CHUNKWISE:
-            import naive_speculate.infer.inferencer.chunkwise as impl_module  # noqa: PLC0415
+            import naive_speculate.infer.inferencer.chunkwise as impl_module
 
     inferencer_class = impl_module.InferencerImpl
     return inferencer_class(language_model=language_model)
@@ -44,9 +46,9 @@ def make_scorer(language_model: LanguageModel) -> Scorer:
 def make_kvcache(kvcache_type: KVCacheType) -> KVCache:
     match kvcache_type:
         case KVCacheType.DYNAMIC:
-            import naive_speculate.infer.kvcache.dynamic as impl_module  # noqa: PLC0415
+            import naive_speculate.infer.kvcache.dynamic as impl_module
         case KVCacheType.DYNAMIC_NO_UPDATE:
-            import naive_speculate.infer.kvcache.dynamic_no_update as impl_module  # noqa: PLC0415
+            import naive_speculate.infer.kvcache.dynamic_no_update as impl_module
 
     kvcache_class = impl_module.KVCacheImpl
     return kvcache_class()

src/naive_speculate/infer/lm/qwen3.py

@@ -13,11 +13,11 @@
 
     from naive_speculate.infer import KVCache
 
-__all__ = ["LanguageModelImpl"]
+__all__ = ["LanguageModelImpl", "Qwen3LanguageModel"]
 
 
-class Qwen3Model(LanguageModel):
-    """Qwen3Model wraps huggingface Qwen3 models, implementing `LanguageModel`.
+class Qwen3LanguageModel(LanguageModel):
+    """Qwen3LanguageModel wraps huggingface Qwen3 models, implementing `LanguageModel`.
 
     Attributes:
         hf_model (Qwen3ForCausalLM): The underlying huggingface Qwen3 model.
@@ -70,4 +70,4 @@ def forward(self, query_token_ids: torch.Tensor, kv_cache: KVCache) -> torch.Ten
         return forward_out.logits
 
 
-LanguageModelImpl = Qwen3Model
+LanguageModelImpl = Qwen3LanguageModel

tests/unit/infer/kvcache/test_dynamic_no_update.py

@@ -17,11 +17,11 @@ class TestDynamicNoUpdateCacheContract(KVCacheContractTests):
     def dynamic_no_update_cache(self) -> DynamicNoUpdateCache:
         return DynamicNoUpdateCache()
 
-    @pytest.mark.xfail(reason="DynamicNoUpdateCache does not support updates", strict=True)
     def test_dynamic_cache_update_xfail(
         self, dynamic_no_update_cache: DynamicNoUpdateCache, kv_states: list[KVState]
     ) -> None:
-        super().update_test(dynamic_no_update_cache, kv_states)
+        with pytest.raises(AssertionError):
+            super().update_test(dynamic_no_update_cache, kv_states)
 
     def test_dynamic_cache_update(
         self, dynamic_no_update_cache: DynamicNoUpdateCache, kv_states: list[KVState]