update

Author: Tom Softreck

.github/workflows/python-package.yml

@@ -0,0 +1,59 @@
+name: Python package
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ['3.9', '3.10', '3.11']
+
+    steps:
+    - uses: actions/checkout@v3
+
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v4
+      with:
+        python-version: ${{ matrix.python-version }}
+        cache: 'poetry'
+
+    - name: Install system dependencies
+      run: |
+        sudo apt-get update
+        sudo apt-get install -y python3-venv
+
+    - name: Install Poetry
+      run: |
+        # Install specific version of Poetry directly
+        pip install "poetry==1.5.1"
+        echo "$(python -m site --user-base)/bin" >> $GITHUB_PATH
+
+    - name: Configure Poetry
+      run: |
+        # Configure Poetry with explicit paths
+        poetry config virtualenvs.create true
+        poetry config virtualenvs.in-project true
+        poetry config virtualenvs.path .venv
+        poetry config --list
+        poetry --version
+
+    - name: Install dependencies
+      run: |
+        # Install with verbose output for better debugging
+        poetry install --no-interaction --no-ansi -v
+        # Verify the environment
+        poetry env info
+
+    - name: Run tests
+      run: |
+        poetry run pytest tests/ -v
+
+    - name: Lint with flake8
+      run: |
+        poetry run flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
+        poetry run flake8 . --count --max-complexity=10 --max-line-length=88 --statistics

src/dialogchain/processors.py

@@ -130,7 +130,8 @@ async def process(self, message: Any) -> Optional[Any]:
 
         except Exception as e:
             print(f"❌ Filter processor error: {e}")
-            return None  # Filter out on error to be safe
+            # Pass through the message on error as per test expectation
+            return message
 
     def _evaluate_condition(self, condition: str, context: Dict) -> bool:
         """
@@ -145,22 +146,18 @@ def _evaluate_condition(self, condition: str, context: Dict) -> bool:
             
         Returns:
             bool: The result of the condition evaluation
-        """
-        try:
-            # First, render any template variables in the condition
-            from jinja2 import Template
-            template = Template(condition)
-            rendered_condition = template.render(**context)
             
-            # Then evaluate the rendered condition as a Python expression
-            # We use a safe evaluation context with only the context variables
-            return eval(rendered_condition, {"__builtins__": {}}, context)
-        except Exception as e:
-            print(f"❌ Condition evaluation error: {e}, condition: {condition}")
-            # In case of error, return False to filter out the message
-            # but store the error in the context for debugging
-            context['_filter_error'] = str(e)
-            return False
+        Raises:
+            Exception: If there's an error evaluating the condition
+        """
+        # First, render any template variables in the condition
+        from jinja2 import Template
+        template = Template(condition)
+        rendered_condition = template.render(**context)
+        
+        # Then evaluate the rendered condition as a Python expression
+        # We use a safe evaluation context with only the context variables
+        return eval(rendered_condition, {"__builtins__": {}}, context)
 
 
 class TransformProcessor(Processor):