refactor: remove unused auto_save method and all its calls (#7)

- Remove UserSession.auto_save() method from user_session.py
- Remove all auto_save() calls from app.py (10 locations)
- Update documentation to remove auto_save references
- Simplify session state management by removing meaningless save operations

The auto_save method was a no-op placeholder that provided no actual functionality.
All session state is now managed through BrowserState integration.

.github/copilot-instructions.md

@@ -21,12 +21,6 @@ from yomitalk.common import APIType
 from yomitalk.components.audio_generator import initialize_global_voicevox_manager
 ```
 
-### Session Management Pattern
-```python
-# Always work within UserSession context
-user_session = UserSession(session_id)
-user_session.auto_save()  # After state changes
-```
 
 ## Code Quality Standards
 

CLAUDE.md

@@ -157,7 +157,6 @@ Located in `tests/e2e/features/`, written in Gherkin syntax:
 3. **Use Components**: Leverage existing TextProcessor, AudioGenerator, ContentExtractor
 4. **Follow Templates**: Use PromptManager for any LLM interactions
 5. **Handle Both APIs**: Ensure new features work with both OpenAI and Gemini
-6. **Add Auto-Save**: If your feature modifies session state, add `user_session.auto_save()` calls
 
 ### When Debugging
 1. **Check Session State**: User issues often relate to session-specific state

yomitalk/app.py

@@ -680,7 +680,6 @@ class PaperPodcastApp:
 
         success = user_session.text_processor.set_openai_api_key(api_key)
         logger.debug(f"OpenAI API key set for session {user_session.session_id}: {success}")
-        user_session.auto_save()  # Save session state after API key change
         return user_session
 
     def set_gemini_api_key(self, api_key: str, user_session: UserSession):
@@ -691,7 +690,6 @@ class PaperPodcastApp:
 
         success = user_session.text_processor.set_gemini_api_key(api_key)
         logger.debug(f"Gemini API key set for session {user_session.session_id}: {success}")
-        user_session.auto_save()  # Save session state after API key change
         return user_session
 
     def switch_llm_type(self, api_type: APIType, user_session: UserSession) -> UserSession:
@@ -701,7 +699,6 @@ class PaperPodcastApp:
             logger.debug(f"LLM type switched to {api_type.display_name} for session {user_session.session_id}")
         else:
             logger.debug(f"{api_type.display_name} API key not set for session {user_session.session_id}")
-        user_session.auto_save()  # Save session state after API type change
         return user_session
 
     def extract_url_text(
@@ -1758,7 +1755,6 @@ class PaperPodcastApp:
         """
         success = user_session.text_processor.openai_model.set_model_name(model_name)
         logger.debug(f"OpenAI model set to {model_name}: {success}")
-        user_session.auto_save()  # Save session state after model name change
         return user_session
 
     def set_gemini_model_name(self, model_name: str, user_session: UserSession) -> UserSession:
@@ -1770,7 +1766,6 @@ class PaperPodcastApp:
         """
         success = user_session.text_processor.gemini_model.set_model_name(model_name)
         logger.debug(f"Gemini model set to {model_name}: {success}")
-        user_session.auto_save()  # Save session state after model name change
         return user_session
 
     def set_openai_max_tokens(self, max_tokens: int, user_session: UserSession) -> UserSession:
@@ -1782,7 +1777,6 @@ class PaperPodcastApp:
         """
         success = user_session.text_processor.openai_model.set_max_tokens(max_tokens)
         logger.debug(f"OpenAI max tokens set to {max_tokens}: {success}")
-        user_session.auto_save()  # Save session state after max tokens change
         return user_session
 
     def set_gemini_max_tokens(self, max_tokens: int, user_session: UserSession) -> UserSession:
@@ -1794,7 +1788,6 @@ class PaperPodcastApp:
         """
         success = user_session.text_processor.gemini_model.set_max_tokens(max_tokens)
         logger.debug(f"Gemini max tokens set to {max_tokens}: {success}")
-        user_session.auto_save()  # Save session state after max tokens change
         return user_session
 
     def set_character_mapping(self, character1: str, character2: str, user_session: UserSession) -> UserSession:
@@ -1806,7 +1799,6 @@ class PaperPodcastApp:
         """
         success = user_session.text_processor.set_character_mapping(character1, character2)
         logger.debug(f"Character mapping set: {character1}, {character2}: {success}")
-        user_session.auto_save()  # Save session state after character mapping change
         return user_session
 
     def set_podcast_mode(self, mode: str, user_session: UserSession, browser_state: Dict[str, Any]) -> Tuple[UserSession, Dict[str, Any]]:
@@ -1835,8 +1827,6 @@ class PaperPodcastApp:
             else:
                 logger.warning(f"Failed to set podcast mode to {mode}")
 
-            user_session.auto_save()  # Save session state after podcast mode change
-
         except ValueError as e:
             logger.error(f"Error setting podcast mode: {str(e)}")
 
@@ -2002,8 +1992,6 @@ class PaperPodcastApp:
             else:
                 logger.warning(f"Failed to set document type to {doc_type}")
 
-            user_session.auto_save()  # Save session state after document type change
-
         except ValueError as e:
             logger.error(f"Error setting document type: {str(e)}")
 

yomitalk/user_session.py

@@ -299,7 +299,3 @@ class UserSession:
             "last_update": None,
             "estimated_total_parts": 1,
         }
-
-    def auto_save(self) -> None:
-        """Temporary compatibility method for tests."""
-        pass