mkv-episode-matcher 0.9.4__tar.gz → 0.9.5__tar.gz

mkv_episode_matcher-0.9.5/.github/workflows/claude-code-review.yml

@@ -0,0 +1,78 @@
+name: Claude Code Review
+
+on:
+  pull_request:
+    types: [opened, synchronize]
+    # Optional: Only run on specific file changes
+    # paths:
+    #   - "src/**/*.ts"
+    #   - "src/**/*.tsx"
+    #   - "src/**/*.js"
+    #   - "src/**/*.jsx"
+
+jobs:
+  claude-review:
+    # Optional: Filter by PR author
+    # if: |
+    #   github.event.pull_request.user.login == 'external-contributor' ||
+    #   github.event.pull_request.user.login == 'new-developer' ||
+    #   github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR'
+    
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      pull-requests: read
+      issues: read
+      id-token: write
+    
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 1
+
+      - name: Run Claude Code Review
+        id: claude-review
+        uses: anthropics/claude-code-action@beta
+        with:
+          claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
+
+          # Optional: Specify model (defaults to Claude Sonnet 4, uncomment for Claude Opus 4)
+          # model: "claude-opus-4-20250514"
+          
+          # Direct prompt for automated review (no @claude mention needed)
+          direct_prompt: |
+            Please review this pull request and provide feedback on:
+            - Code quality and best practices
+            - Potential bugs or issues
+            - Performance considerations
+            - Security concerns
+            - Test coverage
+            
+            Be constructive and helpful in your feedback.
+
+          # Optional: Use sticky comments to make Claude reuse the same comment on subsequent pushes to the same PR
+          # use_sticky_comment: true
+          
+          # Optional: Customize review based on file types
+          # direct_prompt: |
+          #   Review this PR focusing on:
+          #   - For TypeScript files: Type safety and proper interface usage
+          #   - For API endpoints: Security, input validation, and error handling
+          #   - For React components: Performance, accessibility, and best practices
+          #   - For tests: Coverage, edge cases, and test quality
+          
+          # Optional: Different prompts for different authors
+          # direct_prompt: |
+          #   ${{ github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR' && 
+          #   'Welcome! Please review this PR from a first-time contributor. Be encouraging and provide detailed explanations for any suggestions.' ||
+          #   'Please provide a thorough code review focusing on our coding standards and best practices.' }}
+          
+          # Optional: Add specific tools for running tests or linting
+          # allowed_tools: "Bash(npm run test),Bash(npm run lint),Bash(npm run typecheck)"
+          
+          # Optional: Skip review for certain conditions
+          # if: |
+          #   !contains(github.event.pull_request.title, '[skip-review]') &&
+          #   !contains(github.event.pull_request.title, '[WIP]')
+

mkv_episode_matcher-0.9.5/.github/workflows/claude.yml

@@ -0,0 +1,64 @@
+name: Claude Code
+
+on:
+  issue_comment:
+    types: [created]
+  pull_request_review_comment:
+    types: [created]
+  issues:
+    types: [opened, assigned]
+  pull_request_review:
+    types: [submitted]
+
+jobs:
+  claude:
+    if: |
+      (github.event_name == 'issue_comment' && contains(github.event.comment.body, '@claude')) ||
+      (github.event_name == 'pull_request_review_comment' && contains(github.event.comment.body, '@claude')) ||
+      (github.event_name == 'pull_request_review' && contains(github.event.review.body, '@claude')) ||
+      (github.event_name == 'issues' && (contains(github.event.issue.body, '@claude') || contains(github.event.issue.title, '@claude')))
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      pull-requests: read
+      issues: read
+      id-token: write
+      actions: read # Required for Claude to read CI results on PRs
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 1
+
+      - name: Run Claude Code
+        id: claude
+        uses: anthropics/claude-code-action@beta
+        with:
+          claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
+
+          # This is an optional setting that allows Claude to read CI results on PRs
+          additional_permissions: |
+            actions: read
+          
+          # Optional: Specify model (defaults to Claude Sonnet 4, uncomment for Claude Opus 4)
+          # model: "claude-opus-4-20250514"
+          
+          # Optional: Customize the trigger phrase (default: @claude)
+          # trigger_phrase: "/claude"
+          
+          # Optional: Trigger when specific user is assigned to an issue
+          # assignee_trigger: "claude-bot"
+          
+          # Optional: Allow Claude to run specific commands
+          # allowed_tools: "Bash(npm install),Bash(npm run build),Bash(npm run test:*),Bash(npm run lint:*)"
+          
+          # Optional: Add custom instructions for Claude to customize its behavior for your project
+          # custom_instructions: |
+          #   Follow our coding standards
+          #   Ensure all new code has tests
+          #   Use TypeScript for new files
+          
+          # Optional: Custom environment variables for Claude
+          # claude_env: |
+          #   NODE_ENV: test
+

{mkv_episode_matcher-0.9.4 → mkv_episode_matcher-0.9.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mkv-episode-matcher
-Version: 0.9.4
+Version: 0.9.5
 Summary: The MKV Episode Matcher is a tool for identifying TV series episodes from MKV files and renaming the files accordingly.
 Home-page: https://github.com/Jsakkos/mkv-episode-matcher
 Author: Jonathan Sakkos

{mkv_episode_matcher-0.9.4 → mkv_episode_matcher-0.9.5}/mkv_episode_matcher/config.py

@@ -43,7 +43,7 @@ def set_config(
     Returns:
         None
     """
-    config = configparser.ConfigParser()
+    config = configparser.ConfigParser(interpolation=None)
     config["Config"] = {
         "tmdb_api_key": str(tmdb_api_key),
         "show_dir": show_dir,
@@ -56,7 +56,7 @@ def set_config(
     logger.info(
         f"Setting config with API:{tmdb_api_key}, show_dir: {show_dir}, and max_threads: {MAX_THREADS}"
     )
-    with open(file, "w") as configfile:
+    with open(file, "w", encoding="utf-8") as configfile:
         config.write(configfile)
 
 
@@ -72,8 +72,8 @@ def get_config(file):
 
     """
     logger.info(f"Loading config from {file}")
-    config = configparser.ConfigParser()
+    config = configparser.ConfigParser(interpolation=None)
     if Path(file).exists():
-        config.read(file)
+        config.read(file, encoding="utf-8")
         return config["Config"] if "Config" in config else None
     return {}

{mkv_episode_matcher-0.9.4 → mkv_episode_matcher-0.9.5}/mkv_episode_matcher.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mkv-episode-matcher
-Version: 0.9.4
+Version: 0.9.5
 Summary: The MKV Episode Matcher is a tool for identifying TV series episodes from MKV files and renaming the files accordingly.
 Home-page: https://github.com/Jsakkos/mkv-episode-matcher
 Author: Jonathan Sakkos

{mkv_episode_matcher-0.9.4 → mkv_episode_matcher-0.9.5}/mkv_episode_matcher.egg-info/SOURCES.txt

@@ -12,6 +12,8 @@ setup.cfg
 setup.py
 uv.lock
 .github/funding.yml
+.github/workflows/claude-code-review.yml
+.github/workflows/claude.yml
 .github/workflows/documentation.yml
 .github/workflows/python-publish.yml
 .github/workflows/tests.yml
@@ -39,6 +41,7 @@ mkv_episode_matcher.egg-info/entry_points.txt
 mkv_episode_matcher.egg-info/requires.txt
 mkv_episode_matcher.egg-info/top_level.txt
 tests/__init__.py
+tests/test_config_special_characters.py
 tests/test_main.py
 tests/test_path_handling.py
 tests/test_trailing_slash.py

{mkv_episode_matcher-0.9.4 → mkv_episode_matcher-0.9.5}/setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = mkv_episode_matcher
-version = 0.9.4
+version = 0.9.5
 author = Jonathan Sakkos
 author_email = jonathansakkos@gmail.com
 description = The MKV Episode Matcher is a tool for identifying TV series episodes from MKV files and renaming the files accordingly.

mkv_episode_matcher-0.9.5/tests/test_config_special_characters.py

@@ -0,0 +1,174 @@
+"""Test cases for config.py handling of special characters in passwords."""
+
+import tempfile
+from pathlib import Path
+
+import pytest
+
+from mkv_episode_matcher.config import get_config, set_config
+
+
+class TestConfigSpecialCharacters:
+    """Test that config handling works correctly with special characters in passwords."""
+
+    @pytest.fixture
+    def temp_config_file(self):
+        """Create a temporary config file for testing."""
+        with tempfile.NamedTemporaryFile(mode='w', suffix='.ini', delete=False) as f:
+            yield f.name
+        Path(f.name).unlink(missing_ok=True)
+
+    @pytest.fixture
+    def mock_config_data(self):
+        """Mock configuration data for testing."""
+        return {
+            "tmdb_api_key": "test_tmdb_api_key",
+            "open_subtitles_api_key": "test_os_api_key",
+            "open_subtitles_user_agent": "test_user_agent",
+            "open_subtitles_username": "test_username",
+            "show_dir": "/test/path"
+        }
+
+    def test_password_with_percent_symbol(self, temp_config_file, mock_config_data):
+        """Test that passwords containing % symbol don't cause interpolation errors."""
+        password_with_percent = "H7z*X$X29JdJ^#%Q"  # gitguardian:ignore
+        
+        # This should not raise a ValueError
+        set_config(
+            mock_config_data["tmdb_api_key"],
+            mock_config_data["open_subtitles_api_key"],
+            mock_config_data["open_subtitles_user_agent"],
+            mock_config_data["open_subtitles_username"],
+            password_with_percent,
+            mock_config_data["show_dir"],
+            temp_config_file,
+        )
+        
+        # Verify the config was written successfully
+        config = get_config(temp_config_file)
+        assert config is not None
+        assert config["open_subtitles_password"] == password_with_percent
+
+    def test_password_with_multiple_percent_symbols(self, temp_config_file, mock_config_data):
+        """Test that passwords with multiple % symbols work correctly."""
+        password_with_percents = "password%with%multiple%percent%signs"
+        
+        set_config(
+            mock_config_data["tmdb_api_key"],
+            mock_config_data["open_subtitles_api_key"],
+            mock_config_data["open_subtitles_user_agent"],
+            mock_config_data["open_subtitles_username"],
+            password_with_percents,
+            mock_config_data["show_dir"],
+            temp_config_file,
+        )
+        
+        config = get_config(temp_config_file)
+        assert config["open_subtitles_password"] == password_with_percents
+
+    def test_password_with_interpolation_like_syntax(self, temp_config_file, mock_config_data):
+        """Test that passwords resembling interpolation syntax are handled correctly."""
+        # This resembles ConfigParser interpolation syntax but should be treated literally
+        password_with_interpolation = "%(section)s_password_%(option)s"
+        
+        set_config(
+            mock_config_data["tmdb_api_key"],
+            mock_config_data["open_subtitles_api_key"],
+            mock_config_data["open_subtitles_user_agent"],
+            mock_config_data["open_subtitles_username"],
+            password_with_interpolation,
+            mock_config_data["show_dir"],
+            temp_config_file,
+        )
+        
+        config = get_config(temp_config_file)
+        assert config["open_subtitles_password"] == password_with_interpolation
+
+    def test_password_with_various_special_characters(self, temp_config_file, mock_config_data):
+        """Test that passwords with various special characters work correctly."""
+        special_passwords = [
+            "pass@word!123",
+            "my$ecret#key*",
+            "complex&password^with()brackets[]",
+            "unicode_测试_password",
+            "spaces in password",
+            "tabs\tand\nnewlines",
+        ]
+        
+        for password in special_passwords:
+            set_config(
+                mock_config_data["tmdb_api_key"],
+                mock_config_data["open_subtitles_api_key"],
+                mock_config_data["open_subtitles_user_agent"],
+                mock_config_data["open_subtitles_username"],
+                password,
+                mock_config_data["show_dir"],
+                temp_config_file,
+            )
+            
+            config = get_config(temp_config_file)
+            assert config["open_subtitles_password"] == password, f"Failed for password: {password}"
+
+    def test_original_bug_case(self, temp_config_file, mock_config_data):
+        """Test the specific password from the original bug report."""
+        # This is the exact password that caused the original issue
+        problematic_password = "H7z*X$X29JdJ^#%Q"  # gitguardian:ignore
+        
+        # Before the fix, this would raise:
+        # ValueError: invalid interpolation syntax in 'H7z*X$X29JdJ^#%Q' at position 14
+        set_config(
+            mock_config_data["tmdb_api_key"],
+            mock_config_data["open_subtitles_api_key"],
+            mock_config_data["open_subtitles_user_agent"],
+            mock_config_data["open_subtitles_username"],
+            problematic_password,
+            mock_config_data["show_dir"],
+            temp_config_file,
+        )
+        
+        # Verify we can read it back correctly
+        config = get_config(temp_config_file)
+        assert config is not None
+        assert config["open_subtitles_password"] == problematic_password
+        
+        # Verify all other fields are preserved
+        assert config["tmdb_api_key"] == mock_config_data["tmdb_api_key"]
+        assert config["open_subtitles_username"] == mock_config_data["open_subtitles_username"]
+        assert config["show_dir"] == mock_config_data["show_dir"]
+
+    def test_empty_password(self, temp_config_file, mock_config_data):
+        """Test that empty passwords are handled correctly."""
+        empty_password = ""
+        
+        set_config(
+            mock_config_data["tmdb_api_key"],
+            mock_config_data["open_subtitles_api_key"],
+            mock_config_data["open_subtitles_user_agent"],
+            mock_config_data["open_subtitles_username"],
+            empty_password,
+            mock_config_data["show_dir"],
+            temp_config_file,
+        )
+        
+        config = get_config(temp_config_file)
+        assert config["open_subtitles_password"] == empty_password
+
+    def test_config_persistence(self, temp_config_file, mock_config_data):
+        """Test that config values persist correctly across multiple operations."""
+        password = "persistent%password#123"
+        
+        # Set config
+        set_config(
+            mock_config_data["tmdb_api_key"],
+            mock_config_data["open_subtitles_api_key"],
+            mock_config_data["open_subtitles_user_agent"],
+            mock_config_data["open_subtitles_username"],
+            password,
+            mock_config_data["show_dir"],
+            temp_config_file,
+        )
+        
+        # Read config multiple times to ensure consistency
+        for _ in range(3):
+            config = get_config(temp_config_file)
+            assert config["open_subtitles_password"] == password