Revamp

Author: coffandro

.github/workflows/build-binary.yml

@@ -33,7 +33,7 @@ jobs:
         with:
           target: ${{ matrix.target }}
           manylinux: 2014
-          args: --release --out dist --interpreter 3.8 3.9 3.10 3.11 3.12 3.13
+          args: --release --out dist --interpreter 3.10 3.11 3.12 3.13 3.14
 
       - name: Upload wheels
         uses: actions/upload-artifact@v4
@@ -55,7 +55,7 @@ jobs:
         uses: PyO3/maturin-action@v1
         with:
           target: ${{ matrix.target }}
-          args: --release --out dist --interpreter 3.8 3.9 3.10 3.11 3.12 3.13
+          args: --release --out dist --interpreter 3.10 3.11 3.12 3.13 3.14
 
       - name: Upload wheels
         uses: actions/upload-artifact@v4
@@ -77,7 +77,7 @@ jobs:
         uses: PyO3/maturin-action@v1
         with:
           target: ${{ matrix.target }}
-          args: --release --out dist --interpreter 3.8 3.9 3.10 3.11 3.12 3.13
+          args: --release --out dist --interpreter 3.10 3.11 3.12 3.13 3.14
 
       - name: Upload wheels
         uses: actions/upload-artifact@v4
@@ -113,13 +113,14 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
-        python-version: ['3.8', '3.9', '3.10', '3.11', '3.12', '3.13']
+        python-version: ['3.10', '3.11', '3.12', '3.13', '3.14']
     steps:
       - uses: actions/checkout@v4
 
       - uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python-version }}
+          allow-prereleases: true
 
       - name: Download wheels
         uses: actions/download-artifact@v4
@@ -137,17 +138,26 @@ jobs:
         shell: bash
         run: |
           python -c "
-          from ofd_validator import PyValidationOrchestrator
-          print('✓ Import successful')
-
-          # Test instantiation (will fail gracefully if dirs don't exist)
-          try:
-              orch = PyValidationOrchestrator('.', '.')
-              print('✓ PyValidationOrchestrator created')
-          except Exception as e:
-              print(f'✓ Orchestrator creation handled: {e}')
-
-          print('✓ All tests passed')
+          import ofd_validator
+          print('Available:', [x for x in dir(ofd_validator) if not x.startswith('_')])
+
+          # Test types
+          r = ofd_validator.ValidationResult()
+          assert r.is_valid == True
+          assert r.error_count == 0
+          print('Types work correctly')
+
+          # Test functions exist
+          assert callable(ofd_validator.validate_all)
+          assert callable(ofd_validator.validate_json_files)
+          assert callable(ofd_validator.validate_logo_files)
+          assert callable(ofd_validator.validate_folder_names)
+          assert callable(ofd_validator.validate_store_ids)
+          assert callable(ofd_validator.validate_gtin_ean)
+          assert callable(ofd_validator.validate_required_files)
+          print('All functions available')
+
+          print('All tests passed')
           "
 
   publish:
@@ -156,7 +166,7 @@ jobs:
     runs-on: ubuntu-latest
     if: startsWith(github.ref, 'refs/tags/v') || (github.event_name == 'workflow_dispatch' && github.event.inputs.publish-to-pypi == 'true')
     permissions:
-      id-token: write  # IMPORTANT: this permission is mandatory for trusted publishing
+      id-token: write
     steps:
       - name: Download all artifacts
         uses: actions/download-artifact@v4

.github/workflows/build-wheels.yml

@@ -14,7 +14,7 @@ jobs:
         uses: PyO3/maturin-action@v1
         with:
           command: build
-          args: --release --out dist --interpreter 3.8 3.9 3.10 3.11 3.12 3.13
+          args: --release --out dist --interpreter 3.10 3.11 3.12 3.13 3.14
 
       - name: Build sdist
         uses: PyO3/maturin-action@v1

.github/workflows/test-publish.yml

@@ -22,40 +22,57 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
-        python-version: ['3.8', '3.13']  # Test oldest and newest
+        python-version: ['3.10', '3.14']
     steps:
       - uses: actions/checkout@v4
 
       - uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python-version }}
+          allow-prereleases: true
 
       - name: Install Rust toolchain
         uses: actions-rust-lang/setup-rust-toolchain@v1
 
-      - name: Install maturin
-        run: pip install maturin
-
-      - name: Build Python package
-        run: maturin develop --release --features python
-
-      - name: Test import
+      - name: Install maturin and build
+        shell: bash
         run: |
-          python -c "from ofd_validator import PyValidationOrchestrator; print('✓ Import successful')"
+          pip install maturin
+          maturin develop --release
 
-      - name: Test basic functionality
+      - name: Test import and functionality
+        shell: bash
         run: |
           python -c "
-          from ofd_validator import PyValidationOrchestrator
-          import json
-
-          # Create orchestrator
-          orch = PyValidationOrchestrator('.', '.')
-          print('✓ Orchestrator created')
-
-          # Test methods exist
-          assert hasattr(orch, 'validate_all')
-          assert hasattr(orch, 'validate_json_files')
-          assert hasattr(orch, 'validate_logo_files')
-          print('✓ All methods exist')
+          import ofd_validator
+
+          # Test types
+          r = ofd_validator.ValidationResult()
+          assert r.is_valid == True
+          assert r.error_count == 0
+          assert r.warning_count == 0
+
+          # Test error creation
+          e = ofd_validator.ValidationError(ofd_validator.ValidationLevel.Error, 'Test', 'test msg', '/path')
+          r.add_error(e)
+          assert r.error_count == 1
+          assert r.is_valid == False
+
+          # Test to_dict
+          d = r.to_dict()
+          assert d['error_count'] == 1
+          assert d['is_valid'] == False
+
+          # Test all functions exist
+          assert callable(ofd_validator.validate_all)
+          assert callable(ofd_validator.validate_json_files)
+          assert callable(ofd_validator.validate_logo_files)
+          assert callable(ofd_validator.validate_folder_names)
+          assert callable(ofd_validator.validate_store_ids)
+          assert callable(ofd_validator.validate_gtin_ean)
+          assert callable(ofd_validator.validate_required_files)
+          assert callable(ofd_validator.validate_logo_file)
+          assert callable(ofd_validator.validate_folder_name)
+
+          print('All tests passed')
           "