Merge pull request #88 from thawn/copilot/add-pre-commit-hooks

Add pre-commit hooks with Black formatter and flake8 linter

Author: thawn

.flake8

@@ -0,0 +1,21 @@
+[flake8]
+max-line-length = 88
+# Black-compatible options
+# E203: Whitespace before ':' (conflicts with Black)
+# E501: Line too long (handled by Black where possible)
+# W503: Line break before binary operator (conflicts with Black)
+extend-ignore = E203,E501,W503
+
+# Enforce all other errors
+select = E,W,F,B,C
+
+exclude =
+    .git,
+    __pycache__,
+    build,
+    dist,
+    .eggs,
+    *.egg,
+    .venv,
+    venv,
+    node_modules,

.github/copilot-instructions.md

@@ -166,7 +166,7 @@ SQLite connection uses `check_same_thread=False` for Flask's multi-threaded envi
 4. Return JSON for AJAX or render template for pages
 
 ### Fixing E2E Test Issues
-1. Before re-running specific test, start the server manually: 
+1. Before re-running specific test, start the server manually:
 ```bash
  ./ttmp32gme > /tmp/server.log 2>&1 & sleep(2)  # Start server in background
 ```

.github/workflows/copilot-setup-steps.yml

@@ -12,30 +12,30 @@ on:
 jobs:
   copilot-setup-steps:
     runs-on: ubuntu-latest
-    
+
     permissions:
       contents: read
-    
+
     steps:
     - name: Checkout repository
       uses: actions/checkout@v4
     # fetch-depth will be overridden
-    
+
     - name: Set up Python
       uses: actions/setup-python@v5
       with:
         python-version: '3.12'
-    
+
     - name: Install uv
       uses: astral-sh/setup-uv@v4
       with:
         version: "latest"
-    
+
     - name: Install system dependencies
       run: |
         sudo apt-get update
         sudo apt-get install -y wget unzip
-    
+
     - name: Install Chrome and ChromeDriver
       run: |
         # Install Chrome
@@ -52,60 +52,60 @@ jobs:
         sudo mv chromedriver-linux64/chromedriver /usr/local/bin/
         sudo chmod +x /usr/local/bin/chromedriver
         rm -rf chromedriver-linux64*
-     
+
     - name: Install tttool (from releases)
       run: |
         # Download a specific working version from GitHub releases
         # Using the correct URL pattern: tttool-{version}.zip
         TTTOOL_VERSION="1.8.1"
         echo "Installing tttool version $TTTOOL_VERSION"
-        
+
         # Use system temp directory for extraction
         TEMP_DIR=$(mktemp -d)
         cd "$TEMP_DIR"
-        
+
         wget -q "https://github.com/entropia/tip-toi-reveng/releases/download/${TTTOOL_VERSION}/tttool-${TTTOOL_VERSION}.zip"
-        
+
         # Extract into temp directory
         unzip -q "tttool-${TTTOOL_VERSION}.zip"
-        
+
         # Move the binary to /usr/local/bin
         chmod +x tttool
         sudo mv tttool /usr/local/bin/
-        
+
         # Cleanup - change permissions recursively then remove
         cd "$GITHUB_WORKSPACE"
         chmod -R u+w "$TEMP_DIR" || true
         rm -rf "$TEMP_DIR"
-        
+
         # Verify installation
         tttool --help || { echo "Error: tttool installation failed"; exit 1; }
-    
+
     - name: Install ffmpeg (static build)
       run: |
         wget -q https://johnvansickle.com/ffmpeg/releases/ffmpeg-release-amd64-static.tar.xz
         tar -xf ffmpeg-release-amd64-static.tar.xz
         sudo mv ffmpeg-*-amd64-static/ffmpeg /usr/local/bin/
         sudo mv ffmpeg-*-amd64-static/ffprobe /usr/local/bin/
         rm -rf ffmpeg-*-amd64-static*
-    
+
     - name: Install Python dependencies
       run: |
         uv pip install --system -e ".[test]"
-    
+
     - name: Prepare test fixtures
       run: |
         # Bundled test audio file already exists in repo
         ls -lh tests/fixtures/test_audio.mp3
-        
+
         # Create test cover image if needed
         if [ ! -f tests/fixtures/test_cover.jpg ]; then
           python3 -c "from PIL import Image; img = Image.new('RGB', (100, 100), color='green'); img.save('tests/fixtures/test_cover.jpg')"
         fi
-        
+
         # Verify files exist
         ls -lh tests/fixtures/
-    
+
     - name: Verify setup
       run: |
         echo "Setup complete. Verifying installations..."

.github/workflows/e2e-tests.yml

@@ -13,37 +13,37 @@ on:
 
 permissions:
   contents: read
-    
+
 jobs:
   e2e-tests:
     name: End-to-End Tests with Selenium
     runs-on: ubuntu-latest
     # Only run when 'run-e2e-tests' label is added or workflow is manually dispatched
     # if: github.event_name == 'workflow_dispatch' || github.event.label.name == 'run-e2e-tests'
-    
+
     permissions:
       contents: read
-    
+
     steps:
     - uses: actions/checkout@v4
       with:
         fetch-depth: 0  # Needed for setuptools-scm
-    
+
     - name: Set up Python
       uses: actions/setup-python@v5
       with:
         python-version: '3.12'
-    
+
     - name: Install uv
       uses: astral-sh/setup-uv@v4
       with:
         version: "latest"
-    
+
     - name: Install system dependencies
       run: |
         sudo apt-get update
         sudo apt-get install -y wget unzip
-    
+
     - name: Install Chrome and ChromeDriver
       run: |
         # Install Chrome
@@ -60,64 +60,64 @@ jobs:
         sudo mv chromedriver-linux64/chromedriver /usr/local/bin/
         sudo chmod +x /usr/local/bin/chromedriver
         rm -rf chromedriver-linux64*
-     
+
     - name: Install tttool (from releases)
       run: |
         # Download a specific working version from GitHub releases
         # Using the correct URL pattern: tttool-{version}.zip
         TTTOOL_VERSION="1.8.1"
         echo "Installing tttool version $TTTOOL_VERSION"
-        
+
         # Use system temp directory for extraction
         TEMP_DIR=$(mktemp -d)
         cd "$TEMP_DIR"
-        
+
         wget -q "https://github.com/entropia/tip-toi-reveng/releases/download/${TTTOOL_VERSION}/tttool-${TTTOOL_VERSION}.zip"
-        
+
         # Extract into temp directory
         unzip -q "tttool-${TTTOOL_VERSION}.zip"
-        
+
         # Move the binary to /usr/local/bin
         chmod +x tttool
         sudo mv tttool /usr/local/bin/
-        
+
         # Cleanup - change permissions recursively then remove
         cd "$GITHUB_WORKSPACE"
         chmod -R u+w "$TEMP_DIR" || true
         rm -rf "$TEMP_DIR"
-        
+
         # Verify installation
         tttool --help || { echo "Error: tttool installation failed"; exit 1; }
-    
+
     - name: Install ffmpeg (static build)
       run: |
         wget -q https://johnvansickle.com/ffmpeg/releases/ffmpeg-release-amd64-static.tar.xz
         tar -xf ffmpeg-release-amd64-static.tar.xz
         sudo mv ffmpeg-*-amd64-static/ffmpeg /usr/local/bin/
         sudo mv ffmpeg-*-amd64-static/ffprobe /usr/local/bin/
         rm -rf ffmpeg-*-amd64-static*
-    
+
     - name: Install Python dependencies
       run: |
         uv pip install --system -e ".[test]"
-    
+
     - name: Prepare test fixtures
       run: |
         # Bundled test audio file already exists in repo
         ls -lh tests/fixtures/test_audio.mp3
-        
+
         # Create test cover image if needed
         if [ ! -f tests/fixtures/test_cover.jpg ]; then
           python3 -c "from PIL import Image; img = Image.new('RGB', (100, 100), color='green'); img.save('tests/fixtures/test_cover.jpg')"
         fi
-        
+
         # Verify files exist
         ls -lh tests/fixtures/
-        
+
     - name: Run E2E tests with pytest (includes tttool tests)
       run: |
         pytest tests/e2e/ -v --tb=short --html=e2e-report.html --self-contained-html
-    
+
     - name: Upload test report
       uses: actions/upload-artifact@v4
       with:

.github/workflows/javascript-tests.yml

@@ -9,29 +9,29 @@ on:
 jobs:
   test:
     runs-on: ubuntu-latest
-    
+
     strategy:
       matrix:
         node-version: [18.x, 20.x]
-    
+
     steps:
     - uses: actions/checkout@v4
-    
+
     - name: Setup Node.js ${{ matrix.node-version }}
       uses: actions/setup-node@v4
       with:
         node-version: ${{ matrix.node-version }}
         cache: 'npm'
-    
+
     - name: Install dependencies
       run: npm ci
-    
+
     - name: Run tests
       run: npm test
-    
+
     - name: Generate coverage report
       run: npm run test:coverage
-    
+
     - name: Upload coverage reports
       uses: codecov/codecov-action@v3
       with: