midterm-submission

podcraft/src/pages/Podcasts.tsx

@@ -41,7 +41,8 @@ const Podcasts: React.FC = () => {
         );
       }
       
-      setPodcasts(prev => prev.filter(p => p.filename !== podcast.filename));
+      // Refresh the podcast list after successful deletion
+      await fetchPodcasts();
       
     } catch (err) {
       console.error('Delete error:', err);
@@ -60,16 +61,33 @@ const Podcasts: React.FC = () => {
       
       const files = await response.json();
       
+      if (!Array.isArray(files) || files.length === 0) {
+        setPodcasts([]);
+        setLoading(false);
+        return;
+      }
+      
       const podcastList: Podcast[] = files.map((file: any, index: number) => {
         const filename = file.filename;
-        const [queryPart, descriptionPart, categoryWithExt] = filename.split('-');
+        const parts = filename.split('-');
+        let queryPart = '', descriptionPart = '', categoryWithExt = '';
+        
+        // Handle filenames that might not have all parts
+        if (parts.length >= 3) {
+          [queryPart, descriptionPart, categoryWithExt] = parts;
+        } else {
+          queryPart = parts[0] || '';
+          descriptionPart = parts[1] || queryPart;
+          categoryWithExt = parts[2] || 'general.mp3';
+        }
+        
         const category = categoryWithExt.replace('.mp3', '');
         
         return {
           id: index + 1,
           title: `${descriptionPart.replace(/_/g, ' ').replace(/^\w/, c => c.toUpperCase())}`,
           description: `A debate exploring ${queryPart.replace(/_/g, ' ')}`,
-          audio_file: file.path, // Use relative path returned from server
+          audio_file: `${API_URL}${file.path}`, // Add API_URL for audio files
           filename: filename,
           category: category.replace(/_/g, ' ')
         };
@@ -77,6 +95,7 @@ const Podcasts: React.FC = () => {
 
       setPodcasts(podcastList);
     } catch (err) {
+      console.error('Fetch error:', err);
       setError(err instanceof Error ? err.message : 'An error occurred');
     } finally {
       setLoading(false);
@@ -91,14 +110,6 @@ const Podcasts: React.FC = () => {
     );
   }
 
-  if (error !== "") {
-    return (
-      <div className="podcasts-container">
-        <div className="error-message">Error: {error}</div>
-      </div>
-    );
-  }
-
   return (
     <div className="podcasts-container">
       <header className="podcasts-header">
@@ -106,42 +117,47 @@ const Podcasts: React.FC = () => {
         <p>Listen to AI-generated debate podcasts on various topics</p>
       </header>
 
+      {error && (
+        <div className="error-message">Error: {error}</div>
+      )}
+
       <div className="podcasts-grid">
-        {podcasts.map(podcast => (
-          <div 
-            key={podcast.id} 
-            className="podcast-card"
-            onClick={() => navigate(`/podcast/${podcast.id}`)}
-            style={{ cursor: 'pointer' }}
-          >
-            <div className="podcast-content">
-              <div className="podcast-header">
-                <h2 className="podcast-title">{podcast.title}</h2>
-                <button 
-                  className="delete-button"
-                  onClick={(e) => {
-                    e.stopPropagation();
-                    handleDelete(podcast);
-                  }}
-                  aria-label="Delete podcast"
-                >
-                  ×
-                </button>
-              </div>
-              <div className="category-pill">{podcast.category}</div>
-              <p className="description">{podcast.description}</p>
-              <div className="audio-player" onClick={e => e.stopPropagation()}>
-                <audio 
-                  controls
-                  src={podcast.audio_file}
-                >
-                  Your browser does not support the audio element.
-                </audio>
+        {podcasts.length > 0 ? (
+          podcasts.map(podcast => (
+            <div 
+              key={podcast.id} 
+              className="podcast-card"
+              onClick={() => navigate(`/podcast/${podcast.id}`)}
+              style={{ cursor: 'pointer' }}
+            >
+              <div className="podcast-content">
+                <div className="podcast-header">
+                  <h2 className="podcast-title">{podcast.title}</h2>
+                  <button 
+                    className="delete-button"
+                    onClick={(e) => {
+                      e.stopPropagation();
+                      handleDelete(podcast);
+                    }}
+                    aria-label="Delete podcast"
+                  >
+                    ×
+                  </button>
+                </div>
+                <div className="category-pill">{podcast.category}</div>
+                <p className="description">{podcast.description}</p>
+                <div className="audio-player" onClick={e => e.stopPropagation()}>
+                  <audio 
+                    controls
+                    src={podcast.audio_file}
+                  >
+                    Your browser does not support the audio element.
+                  </audio>
+                </div>
               </div>
             </div>
-          </div>
-        ))}
-        {podcasts.length === 0 && (
+          ))
+        ) : (
           <div className="no-podcasts-message">
             No podcasts found. Generate your first podcast from the home page!
           </div>

server/main.py

@@ -133,9 +133,10 @@ async def list_audio_files():
                     "path": f"/audio-files/{file}",
                     "size": os.path.getsize(file_path)
                 })
-        return audio_files
+        return audio_files if audio_files else []
     except Exception as e:
-        raise HTTPException(status_code=500, detail=str(e))
+        logger.error(f"Error listing audio files: {str(e)}")
+        return []
 
 @api_router.get("/audio/{filename}")
 async def get_audio_file(filename: str):
@@ -143,56 +144,56 @@ async def get_audio_file(filename: str):
     try:
         file_path = os.path.join(audio_dir, filename)
         if not os.path.exists(file_path):
+            logger.error(f"Audio file not found: {filename}")
             raise HTTPException(status_code=404, detail="File not found")
-        return FileResponse(file_path)
+        return FileResponse(file_path, media_type="audio/mpeg")
     except Exception as e:
+        logger.error(f"Error serving audio file: {str(e)}")
         raise HTTPException(status_code=500, detail=str(e))
 
 @api_router.delete("/audio/{filename}")
 async def delete_audio_file(filename: str):
     """Delete an audio file and its corresponding transcript."""
     try:
-        # Delete audio file
+        # Check if file exists before attempting deletion
         file_path = os.path.join(audio_dir, filename)
         if not os.path.exists(file_path):
+            logger.error(f"File not found for deletion: {filename}")
             raise HTTPException(status_code=404, detail="File not found")
         
-        # Get all audio files to determine the podcast ID
-        audio_files = [f for f in os.listdir(audio_dir) if f.endswith(('.mp3', '.wav'))]
         try:
-            # Find the index (0-based) of the file being deleted
-            podcast_id = audio_files.index(filename) + 1  # Convert to 1-based ID
-            logger.info(f"Deleting podcast with ID: {podcast_id}")
+            # Delete the audio file first
+            os.remove(file_path)
+            logger.info(f"Deleted audio file: {filename}")
             
-            # Path to transcripts file
-            transcripts_file = os.path.join(os.path.dirname(__file__), "transcripts", "podcasts.json")
+            # Get all remaining audio files
+            audio_files = [f for f in os.listdir(audio_dir) if f.endswith(('.mp3', '.wav'))]
             
-            # Update transcripts if file exists
+            # Try to update transcripts if they exist
+            transcripts_file = os.path.join(os.path.dirname(__file__), "transcripts", "podcasts.json")
             if os.path.exists(transcripts_file):
                 with open(transcripts_file, 'r') as f:
                     transcripts = json.load(f)
                 
-                # Remove the transcript at the corresponding index
-                if len(transcripts) >= podcast_id:
-                    transcripts.pop(podcast_id - 1)  # Convert back to 0-based index
-                    
-                    # Save updated transcripts
-                    with open(transcripts_file, 'w') as f:
-                        json.dump(transcripts, f, indent=2)
-                    logger.info(f"Removed transcript for podcast ID {podcast_id}")
+                # Find the index of the deleted file in the original list
+                try:
+                    podcast_id = audio_files.index(filename) + 1
+                    if len(transcripts) >= podcast_id:
+                        transcripts.pop(podcast_id - 1)
+                        with open(transcripts_file, 'w') as f:
+                            json.dump(transcripts, f, indent=2)
+                        logger.info(f"Updated transcripts after deletion")
+                except ValueError:
+                    logger.warning(f"Could not find podcast ID for {filename} in transcripts")
             
-            # Delete the audio file
-            os.remove(file_path)
-            logger.info(f"Deleted audio file: {filename}")
-            
-            return {"message": "File and transcript deleted successfully"}
+            return {"message": "File deleted successfully"}
             
-        except ValueError:
-            logger.error(f"Could not determine podcast ID for file: {filename}")
-            # Still delete the audio file even if transcript removal fails
-            os.remove(file_path)
-            return {"message": "Audio file deleted, but transcript could not be removed"}
+        except Exception as e:
+            logger.error(f"Error during file deletion process: {str(e)}")
+            raise HTTPException(status_code=500, detail=str(e))
             
+    except HTTPException as he:
+        raise he
     except Exception as e:
         logger.error(f"Error in delete_audio_file: {str(e)}")
         raise HTTPException(status_code=500, detail=str(e))