refactor: streamline log worksheet naming by removing `GSHEET_WORKSHEET_LOGS` configuration and adjusting name generation.

.github/workflows/process_csv.yml

@@ -29,9 +29,6 @@ on:
       AUDIO_LIBRARY_GSHEET_WORKSHEET:
         description: 'Audio Library Worksheet'
         required: false
-      GSHEET_WORKSHEET_LOGS:
-        description: 'Logs Worksheet'
-        required: false
       CONTENT_STRATEGY_FILE:
         description: 'Content Strategy File'
         required: false
@@ -174,7 +171,6 @@ jobs:
           GSHEET_ID: ${{ github.event.inputs.GSHEET_ID || vars.GSHEET_ID }}
           VIDEO_LIBRARY_GSHEET_WORKSHEET: ${{ github.event.inputs.VIDEO_LIBRARY_GSHEET_WORKSHEET || vars.VIDEO_LIBRARY_GSHEET_WORKSHEET }}
           AUDIO_LIBRARY_GSHEET_WORKSHEET: ${{ github.event.inputs.AUDIO_LIBRARY_GSHEET_WORKSHEET || vars.AUDIO_LIBRARY_GSHEET_WORKSHEET }}
-          GSHEET_WORKSHEET_LOGS: ${{ github.event.inputs.GSHEET_WORKSHEET_LOGS || vars.GSHEET_WORKSHEET_LOGS }}
           DRIVE_UPLOAD_FOLDER_ID: ${{ github.event.inputs.DRIVE_UPLOAD_FOLDER_ID || vars.DRIVE_UPLOAD_FOLDER_ID }}
           
         run: |

github_workflow_app/index.html

@@ -521,7 +521,6 @@
             'GSHEET_ID',
             'VIDEO_LIBRARY_GSHEET_WORKSHEET',
             'AUDIO_LIBRARY_GSHEET_WORKSHEET',
-            'GSHEET_WORKSHEET_LOGS',
             'CONTENT_STRATEGY_FILE',
             'IS_A2E_LIP_SYNC',
             'USE_VEO',
@@ -540,7 +539,6 @@
             'GSHEET_ID',
             'VIDEO_LIBRARY_GSHEET_WORKSHEET',
             'AUDIO_LIBRARY_GSHEET_WORKSHEET',
-            'GSHEET_WORKSHEET_LOGS',
             'GENERATION_COUNT',
             'DRIVE_UPLOAD_FOLDER_ID'
         ];

src/cleanup_manager.py

@@ -32,9 +32,6 @@ async def process_delete_entries():
     Delete corresponding video files from GCS or Drive and then remove the row from the sheet.
     """
     gsheet_name = get_config_value("logs_worksheet")
-    if not gsheet_name:
-        logger.warning("GSHEET_WORKSHEET_LOGS not set, skipping delete entry check")
-        return
 
     try:
         reader = GoogleSheetReader(worksheet_name=gsheet_name)

src/config.py

@@ -83,7 +83,7 @@ def load_configuration(force_reload: bool = False) -> Dict[str, Any]:
         "gsheet_id": os.getenv("GSHEET_ID"),
         "video_library_worksheet": os.getenv("VIDEO_LIBRARY_GSHEET_WORKSHEET"),
         "audio_library_worksheet": os.getenv("AUDIO_LIBRARY_GSHEET_WORKSHEET"),
-        "logs_worksheet": f'{os.getenv("GSHEET_NAME")} {os.getenv("SETUP_TYPE")} LOGS',
+        "logs_worksheet": f'{os.getenv("SETUP_TYPE")} LOGS',
         "content_strategy_worksheet": os.getenv("CONTENT_STRATEGY_GSHEET_WORKSHEET"),
         
         # Pipeline settings
@@ -104,7 +104,7 @@ def load_configuration(force_reload: bool = False) -> Dict[str, Any]:
         "delete_all_a2e_videos": _env_bool("DELETE_ALL_A2E_VIDEOS"),
 
         # publisher settings
-        "publisher_logs_worksheet": 'Publisher LOGS',
+        "publisher_logs_worksheet": f'{os.getenv("GCS_BUCKET_NAME")} Publisher LOGS',
 
         # TikTok
         "tiktok_access_token": os.getenv("TIKTOK_ACCESS_TOKEN"),

src/execution_tracker.py

@@ -12,11 +12,7 @@ def load_executed_from_gsheet(setup_type=None, job_index=None):
         job_index: Optional. If provided, only count rows for this specific job.
     """
     gsheet_name = get_config_value("logs_worksheet")
-    
-    if not gsheet_name:
-        logger.warning("GSHEET_WORKSHEET_LOGS not set")
-        return []
-    
+
     try:
         reader = GoogleSheetReader(worksheet_name=gsheet_name)
         df = reader.get_dataframe()
@@ -49,9 +45,6 @@ def log_progress_to_gsheet(result: dict, job_index: int, commit=False):
         return
 
     gsheet_name = get_config_value("logs_worksheet")
-    if not gsheet_name:
-        logger.warning("GSHEET_WORKSHEET_LOGS not set, skipping gsheet logging")
-        return
 
     try:
         reader = GoogleSheetReader(worksheet_name=gsheet_name)