fix(ci): Add missing pytest markers and fix protocol tests

- Add pytest markers configuration to pyproject.toml
- Add tests/conftest.py for automatic test marking
- Remove redundant __runtime_checkable__ assignment
- Fix protocol tests to check functionality not internals

Author: dugshub

pyproject.toml

@@ -101,6 +101,14 @@ addopts = "-ra -q"
 testpaths = [
     "tests",
 ]
+markers = [
+    "unit: Unit tests for individual components",
+    "integration: Integration tests for component interactions",
+    "parser: Tests for parser system components",
+    "executor: Tests for execution/subprocess components",
+    "design: Tests for design system components",
+    "slow: Tests that take significant time to run",
+]
 
 [dependency-groups]
 dev = [

src/cli_patterns/ui/parser/protocols.py

@@ -54,7 +54,3 @@ def get_suggestions(self, partial: str) -> list[str]:
             List of suggested completions for the partial input
         """
         ...
-
-
-# Explicitly set the runtime checkable attribute for older Python versions
-Parser.__runtime_checkable__ = True

tests/conftest.py

@@ -0,0 +1,21 @@
+"""Pytest configuration for CLI Patterns tests."""
+
+import pytest
+
+
+def pytest_collection_modifyitems(config, items):
+    """Automatically add markers based on test file location."""
+    for item in items:
+        # Add unit/integration markers based on path
+        if "tests/unit" in str(item.fspath):
+            item.add_marker(pytest.mark.unit)
+        elif "tests/integration" in str(item.fspath):
+            item.add_marker(pytest.mark.integration)
+
+        # Add component markers based on path
+        if "parser" in str(item.fspath):
+            item.add_marker(pytest.mark.parser)
+        elif "executor" in str(item.fspath) or "execution" in str(item.fspath):
+            item.add_marker(pytest.mark.executor)
+        elif "design" in str(item.fspath):
+            item.add_marker(pytest.mark.design)

tests/unit/ui/parser/test_protocols.py

@@ -16,7 +16,11 @@ class TestParserProtocol:
 
     def test_parser_is_runtime_checkable(self) -> None:
         """Test that Parser protocol is runtime checkable."""
-        assert hasattr(Parser, "__runtime_checkable__")
+        # Check that we can use isinstance with the protocol
+        # Parser should be decorated with @runtime_checkable
+        # which makes it usable with isinstance
+        mock_parser = Mock(spec=Parser)
+        assert isinstance(mock_parser, Parser)
 
     def test_parser_protocol_methods(self) -> None:
         """Test that Parser protocol has required methods."""
@@ -451,8 +455,10 @@ def test_protocol_typing_information(self) -> None:
         # Should be identifiable as a Protocol
         assert issubclass(Parser, Protocol)
 
-        # Should have runtime checkable decorator
-        assert getattr(Parser, "__runtime_checkable__", False)
+        # Should be runtime checkable (can use isinstance)
+        # The @runtime_checkable decorator enables this
+        mock_obj = Mock(spec=Parser)
+        assert isinstance(mock_obj, Parser), "Parser should be runtime checkable"
 
 
 class TestParserProtocolEdgeCases: