fix errors

Author: yconst

avlos/templates/docs.rst.jinja

@@ -1,6 +1,3 @@
-
-.. _api-reference:
-
 API REFERENCE
 =============
 

avlos/validation.py

@@ -3,9 +3,12 @@
 Validates C identifiers, detects conflicts, and ensures consistency.
 """
 
+import logging
 import re
 from typing import List
 
+_logger = logging.getLogger("avlos")
+
 # C reserved words (C11 standard)
 C_RESERVED_WORDS = {
     "auto",
@@ -86,9 +89,11 @@ def validate_c_identifier(name: str, context: str = "") -> None:
     if len(name) > 63:
         # C99 requires at least 63 significant characters for identifiers
         ctx = f" ({context})" if context else ""
-        print(
-            f"Warning: Identifier '{name}'{ctx} is very long ({len(name)} chars). "
-            f"Some compilers may truncate after 63 characters."
+        _logger.warning(
+            "Identifier '%s'%s is very long (%d chars). " "Some compilers may truncate after 63 characters.",
+            name,
+            ctx,
+            len(name),
         )
 
 

docs/config.rst

@@ -75,7 +75,7 @@ The output config defines the output modules that will be used and their options
 There are three main generated files that are configured above: A header containing enums (`output_enums`), a header containing function declarations (`output_header`), and an implementation containing function definitions (`output_impl`).
 
 Optional: Endpoint metadata (for type-aware UART/ASCII parsing)
-^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
 If you want the C generator to emit an endpoint type metadata table (so firmware can parse and format ASCII without a hand-maintained table), add both of these optional paths under ``paths``:
 

tests/test_validation.py

@@ -82,8 +82,11 @@ def test_long_identifier_warning(self):
         """Test that very long identifiers generate warnings."""
         # 64+ character identifier (C99 requires at least 63 significant chars)
         long_name = "a" * 70
-        # Should not raise, but might print warning
-        validate_c_identifier(long_name)
+        # Should not raise; warning is logged (captured here so it does not print)
+        with self.assertLogs("avlos", level="WARNING") as cm:
+            validate_c_identifier(long_name)
+        self.assertIn("very long", cm.output[0])
+        self.assertIn("63 characters", cm.output[0])
 
     def test_valid_device_passes_all_validation(self):
         """Test that good_device.yaml passes all validations."""