mkv-episode-matcher 0.9.3__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.3 → mkv_episode_matcher-0.9.5}/CHANGELOG.md

@@ -5,6 +5,18 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.9.3] - 2025-07-07
+
+### Added
+- Onboarding flag (`--onboard`) and interactive onboarding sequence for first-time setup and configuration updates
+- Onboarding prompts for TMDb API key, OpenSubtitles API key, Consumer Name, Username, Password, and Show Directory
+- Existing config values are shown as defaults during onboarding and can be accepted or overwritten
+- Documentation updated to reflect onboarding requirements and workflow in README and docs
+
+### Changed
+- Improved configuration experience for new and returning users
+- Quick Start and Configuration documentation now reference onboarding and required credentials
+
 ## [0.9.0] - 2025-06-01
 
 ### Changed

mkv_episode_matcher-0.9.5/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2025 Jonathan Sakkos
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

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

@@ -1,11 +1,31 @@
 Metadata-Version: 2.4
 Name: mkv-episode-matcher
-Version: 0.9.3
+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
 Author-email: Jsakkos <jonathansakkos@gmail.com>
-License: MIT
+License: MIT License
+        
+        Copyright (c) 2025 Jonathan Sakkos
+        
+        Permission is hereby granted, free of charge, to any person obtaining a copy
+        of this software and associated documentation files (the "Software"), to deal
+        in the Software without restriction, including without limitation the rights
+        to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+        copies of the Software, and to permit persons to whom the Software is
+        furnished to do so, subject to the following conditions:
+        
+        The above copyright notice and this permission notice shall be included in all
+        copies or substantial portions of the Software.
+        
+        THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+        IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+        FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+        AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+        LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+        OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+        SOFTWARE.
 Project-URL: Documentation, https://github.com/Jsakkos/mkv-episode-matcher#readme
 Project-URL: Issues, https://github.com/Jsakkos/mkv-episode-matcher/issues
 Project-URL: Source, https://github.com/Jsakkos/mkv-episode-matcher
@@ -16,6 +36,8 @@ Classifier: Programming Language :: Python :: Implementation :: CPython
 Classifier: Programming Language :: Python :: Implementation :: PyPy
 Requires-Python: <3.13,>=3.9
 Description-Content-Type: text/markdown
+License-File: LICENSE
+Requires-Dist: chardet>=5.2.0
 Requires-Dist: configparser>=7.1.0
 Requires-Dist: ffmpeg>=1.4
 Requires-Dist: loguru>=0.7.2
@@ -28,6 +50,7 @@ Requires-Dist: tmdb-client>=0.0.1
 Requires-Dist: torch>=2.5.1
 Requires-Dist: torchaudio>=2.5.1
 Requires-Dist: torchvision>=0.20.1
+Dynamic: license-file
 
 # MKV Episode Matcher
 

{mkv_episode_matcher-0.9.3 → 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.3 → mkv_episode_matcher-0.9.5}/mkv_episode_matcher.egg-info/PKG-INFO

@@ -1,11 +1,31 @@
 Metadata-Version: 2.4
 Name: mkv-episode-matcher
-Version: 0.9.3
+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
 Author-email: Jsakkos <jonathansakkos@gmail.com>
-License: MIT
+License: MIT License
+        
+        Copyright (c) 2025 Jonathan Sakkos
+        
+        Permission is hereby granted, free of charge, to any person obtaining a copy
+        of this software and associated documentation files (the "Software"), to deal
+        in the Software without restriction, including without limitation the rights
+        to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+        copies of the Software, and to permit persons to whom the Software is
+        furnished to do so, subject to the following conditions:
+        
+        The above copyright notice and this permission notice shall be included in all
+        copies or substantial portions of the Software.
+        
+        THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+        IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+        FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+        AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+        LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+        OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+        SOFTWARE.
 Project-URL: Documentation, https://github.com/Jsakkos/mkv-episode-matcher#readme
 Project-URL: Issues, https://github.com/Jsakkos/mkv-episode-matcher/issues
 Project-URL: Source, https://github.com/Jsakkos/mkv-episode-matcher
@@ -16,6 +36,8 @@ Classifier: Programming Language :: Python :: Implementation :: CPython
 Classifier: Programming Language :: Python :: Implementation :: PyPy
 Requires-Python: <3.13,>=3.9
 Description-Content-Type: text/markdown
+License-File: LICENSE
+Requires-Dist: chardet>=5.2.0
 Requires-Dist: configparser>=7.1.0
 Requires-Dist: ffmpeg>=1.4
 Requires-Dist: loguru>=0.7.2
@@ -28,6 +50,7 @@ Requires-Dist: tmdb-client>=0.0.1
 Requires-Dist: torch>=2.5.1
 Requires-Dist: torchaudio>=2.5.1
 Requires-Dist: torchvision>=0.20.1
+Dynamic: license-file
 
 # MKV Episode Matcher
 

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

@@ -4,6 +4,7 @@
 .gitmodules
 .python-version
 CHANGELOG.md
+LICENSE
 README.md
 mkdocs.yml
 pyproject.toml
@@ -11,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
@@ -38,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.3 → mkv_episode_matcher-0.9.5}/mkv_episode_matcher.egg-info/requires.txt

@@ -1,3 +1,4 @@
+chardet>=5.2.0
 configparser>=7.1.0
 ffmpeg>=1.4
 loguru>=0.7.2

{mkv_episode_matcher-0.9.3 → mkv_episode_matcher-0.9.5}/pyproject.toml

@@ -10,7 +10,7 @@ name = "mkv-episode-matcher"
 description = "The MKV Episode Matcher is a tool for identifying TV series episodes from MKV files and renaming the files accordingly."
 dynamic = ["version"]
 readme = "README.md"
-license = { text = "MIT" }
+license = { file = "LICENSE" }
 classifiers = [
   "Development Status :: 4 - Beta",
   "Programming Language :: Python",
@@ -23,6 +23,7 @@ authors = [
 ]
 requires-python = ">=3.9,<3.13"
 dependencies = [
+    "chardet>=5.2.0",
     "configparser>=7.1.0",
     "ffmpeg>=1.4",
     "loguru>=0.7.2",
@@ -49,7 +50,6 @@ dev = [
     "pytest-cov>=6.0.0",
     "pytest>=8.3.3",
     "ruff>=0.8.0",
-    "chardet>=5.2.0",
     "matplotlib>=3.9.4",
     "memory-profiler>=0.61.0",
     "line-profiler>=4.2.0",

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

@@ -1,6 +1,6 @@
 [metadata]
 name = mkv_episode_matcher
-version = 0.9.3
+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