endoreg-db 0.8.1__py3-none-any.whl → 0.8.2__py3-none-any.whl

endoreg_db/urls/media.py

@@ -2,11 +2,35 @@ from django.urls import path
 
 from endoreg_db.views.media import (
     VideoMediaView,
-    PDFMediaManagementView as PDFMediaView,  # Alias to avoid conflict with legacy pdf.PDFMediaView
+    PdfMediaView,  # Alias to avoid conflict with legacy pdf.PDFMediaView
+    video_segments_by_pk,
+    video_segments_collection,
+    video_segments_by_video,
+    video_segment_detail,
+    video_segments_stats,
+    video_segment_validate,
+    video_segments_validate_bulk,
+    video_segments_validation_status,
+    video_sensitive_metadata,
+    video_sensitive_metadata_verify,
+    pdf_sensitive_metadata,
+    pdf_sensitive_metadata_verify,
+    sensitive_metadata_list,
+    pdf_sensitive_metadata_list,
 )
 from endoreg_db.views import (
     VideoStreamView,
 )
+from endoreg_db.views.pdf.reimport import PdfReimportView
+from endoreg_db.views.video.reimport import VideoReimportView
+from endoreg_db.views.video.correction import (
+    VideoReprocessView,
+    VideoMetadataView,
+    VideoProcessingHistoryView,
+    VideoAnalyzeView,
+    VideoApplyMaskView,
+    VideoRemoveFramesView,
+)
 # ---------------------------------------------------------------------------------------
 # ANNOTATION API ENDPOINTS
 #
@@ -23,10 +47,183 @@ urlpatterns = [
     path("media/videos/<int:pk>/", VideoStreamView.as_view(), name="video-detail-stream"),  # Support ?type= params
     path("media/videos/<int:pk>/details/", VideoMediaView.as_view(), name="video-detail"),  # JSON metadata
     path("media/videos/<int:pk>/stream/", VideoStreamView.as_view(), name="video-stream"),  # Legacy support
+    
+    # Video Re-import API endpoint (modern media framework)
+    # POST /api/media/videos/<int:pk>/reimport/
+    # Re-imports a video file to regenerate metadata when OCR failed or data is incomplete
+    path("media/videos/<int:pk>/reimport/", VideoReimportView.as_view(), name="video-reimport"),
+    
+    # ---------------------------------------------------------------------------------------
+    # VIDEO CORRECTION API ENDPOINTS (Modern Media Framework - October 14, 2025)
+    #
+    # All video correction endpoints migrated to unified /api/media/videos/<pk>/ pattern
+    # These endpoints enable video correction workflows (Phase 1.1):
+    # - Analysis: Detect sensitive frames using MiniCPM-o 2.6 or OCR+LLM
+    # - Masking: Apply device-specific masks or custom ROI masks
+    # - Frame Removal: Remove sensitive frames from videos
+    # - Reprocessing: Re-run entire anonymization pipeline
+    # - Metadata: View analysis results
+    # - History: Track all correction operations
+    # ---------------------------------------------------------------------------------------
+    
+    # Video Metadata API
+    # GET /api/media/videos/<int:pk>/metadata/
+    # Returns analysis results (sensitive frame count, ratio, frame IDs)
+    path("media/videos/<int:pk>/metadata/", VideoMetadataView.as_view(), name="video-metadata"),
+    
+    # Video Processing History API
+    # GET /api/media/videos/<int:pk>/processing-history/
+    # Returns history of all processing operations (masking, frame removal, analysis)
+    path("media/videos/<int:pk>/processing-history/", VideoProcessingHistoryView.as_view(), name="video-processing-history"),
+    
+    # Video Analysis API
+    # POST /api/media/videos/<int:pk>/analyze/
+    # Analyzes video for sensitive frames using MiniCPM-o 2.6 or OCR+LLM
+    # Body: { detection_method: 'minicpm'|'ocr_llm'|'hybrid', sample_interval: 30 }
+    path("media/videos/<int:pk>/analyze/", VideoAnalyzeView.as_view(), name="video-analyze"),
+    
+    # Video Masking API
+    # POST /api/media/videos/<int:pk>/apply-mask/
+    # Applies device mask or custom ROI mask to video
+    # Body: { mask_type: 'device'|'custom', device_name: 'olympus', roi: {...} }
+    path("media/videos/<int:pk>/apply-mask/", VideoApplyMaskView.as_view(), name="video-apply-mask"),
+    
+    # Video Frame Removal API
+    # POST /api/media/videos/<int:pk>/remove-frames/
+    # Removes specified frames from video
+    # Body: { frame_list: [10,20,30] OR frame_ranges: '10-20,30' OR detection_method: 'automatic' }
+    path("media/videos/<int:pk>/remove-frames/", VideoRemoveFramesView.as_view(), name="video-remove-frames"),
+    
+    # Video Reprocess API endpoint (modern media framework)
+    # POST /api/media/videos/<int:pk>/reprocess/
+    # Re-runs entire anonymization pipeline for a video (correction workflow)
+    path("media/videos/<int:pk>/reprocess/", VideoReprocessView.as_view(), name="video-reprocess"),
+    
+    # ---------------------------------------------------------------------------------------
+    # VIDEO SEGMENT API ENDPOINTS (Modern Media Framework - October 14, 2025)
+    #
+    # Unified segment management endpoints replacing legacy /api/video-segments/
+    # Collection: GET/POST all segments across videos
+    # Video-scoped: GET/POST segments for specific video
+    # Detail: GET/PATCH/DELETE individual segment
+    # ---------------------------------------------------------------------------------------
+    
+    # Video Segments Collection API
+    # GET/POST /api/media/videos/segments/
+    # List all video segments across videos or create new segment
+    path("media/videos/segments/", video_segments_collection, name="video-segments-collection"),
+    
+    # Video Segments Stats API
+    # GET /api/media/videos/segments/stats/
+    # Get statistics about video segments
+    path("media/videos/segments/stats/", video_segments_stats, name="video-segments-stats"),
+    
+    # Video-Specific Segments API
+    # GET/POST /api/media/videos/<int:pk>/segments/
+    # List segments for specific video or create segment for video
+    path("media/videos/<int:pk>/segments/", video_segments_by_video, name="video-segments-by-video"),
+    
+    # Segment Detail API
+    # GET /api/media/videos/<int:pk>/segments/<int:segment_id>/
+    # PATCH /api/media/videos/<int:pk>/segments/<int:segment_id>/
+    # DELETE /api/media/videos/<int:pk>/segments/<int:segment_id>/
+    # Manages individual segment operations
+    path("media/videos/<int:pk>/segments/<int:segment_id>/", video_segment_detail, name="video-segment-detail"),
+    
+    # ---------------------------------------------------------------------------------------
+    # VIDEO SEGMENT VALIDATION API ENDPOINTS (Modern Media Framework - October 14, 2025)
+    #
+    # Unified validation endpoints replacing legacy /api/label-video-segment/*/validate/
+    # Single: POST validate individual segment
+    # Bulk: POST validate multiple segments
+    # Status: GET/POST validation status for all segments
+    # ---------------------------------------------------------------------------------------
+    
+    # Single Segment Validation API
+    # POST /api/media/videos/<int:pk>/segments/<int:segment_id>/validate/
+    # Validates a single video segment
+    # Body: { "is_validated": true, "notes": "..." }
+    path("media/videos/<int:pk>/segments/<int:segment_id>/validate/", video_segment_validate, name="video-segment-validate"),
+    
+    # Bulk Segment Validation API
+    # POST /api/media/videos/<int:pk>/segments/validate-bulk/
+    # Validates multiple segments at once
+    # Body: { "segment_ids": [1,2,3], "is_validated": true, "notes": "..." }
+    path("media/videos/<int:pk>/segments/validate-bulk/", video_segments_validate_bulk, name="video-segments-validate-bulk"),
+    
+    # Segment Validation Status API
+    # GET /api/media/videos/<int:pk>/segments/validation-status/
+    # Returns validation statistics for all segments
+    # POST /api/media/videos/<int:pk>/segments/validation-status/
+    # Marks all segments (or filtered by label) as validated
+    # Body: { "label_name": "polyp", "notes": "..." }
+    path("media/videos/<int:pk>/segments/validation-status/", video_segments_validation_status, name="video-segments-validation-status"),
+
+    # ---------------------------------------------------------------------------------------
+    # SENSITIVE METADATA ENDPOINTS (Modern Media Framework)
+    # ---------------------------------------------------------------------------------------
+    
+    # Video Sensitive Metadata (Resource-Scoped)
+    # GET/PATCH /api/media/videos/<pk>/sensitive-metadata/
+    # Get or update sensitive patient data for a video
+    path(
+        "media/videos/<int:pk>/sensitive-metadata/",
+        video_sensitive_metadata,
+        name="video-sensitive-metadata"
+    ),
+    
+    # POST /api/media/videos/<pk>/sensitive-metadata/verify/
+    # Update verification state (dob_verified, names_verified)
+    path(
+        "media/videos/<int:pk>/sensitive-metadata/verify/",
+        video_sensitive_metadata_verify,
+        name="video-sensitive-metadata-verify"
+    ),
+    
+    # PDF Sensitive Metadata (Resource-Scoped)
+    # GET/PATCH /api/media/pdfs/<pk>/sensitive-metadata/
+    # Get or update sensitive patient data for a PDF
+    path(
+        "media/pdfs/<int:pk>/sensitive-metadata/",
+        pdf_sensitive_metadata,
+        name="pdf-sensitive-metadata"
+    ),
+    
+    # POST /api/media/pdfs/<pk>/sensitive-metadata/verify/
+    # Update verification state (dob_verified, names_verified)
+    path(
+        "media/pdfs/<int:pk>/sensitive-metadata/verify/",
+        pdf_sensitive_metadata_verify,
+        name="pdf-sensitive-metadata-verify"
+    ),
+    
+    # List Endpoints (Collection-Level)
+    # GET /api/media/sensitive-metadata/
+    # List all sensitive metadata (combined PDFs and Videos)
+    # Supports filtering: ?content_type=pdf|video&verified=true&search=name
+    path(
+        "media/sensitive-metadata/",
+        sensitive_metadata_list,
+        name="sensitive-metadata-list"
+    ),
+    
+    # GET /api/media/pdfs/sensitive-metadata/
+    # List sensitive metadata for PDFs only
+    # Replaces legacy /api/pdf/sensitivemeta/list/
+    path(
+        "media/pdfs/sensitive-metadata/",
+        pdf_sensitive_metadata_list,
+        name="pdf-sensitive-metadata-list"
+    ),
 
     # PDF media endpoints
-    path("media/pdfs/", PDFMediaView.as_view(), name="pdf-list"),
-    path("media/pdfs/<int:pk>/", PDFMediaView.as_view(), name="pdf-detail"),
-    path("media/pdfs/<int:pk>/stream/", PDFMediaView.as_view(), name="pdf-stream"),
+    path("media/pdfs/", PdfMediaView.as_view(), name="pdf-list"),
+    path("media/pdfs/<int:pk>/", PdfMediaView.as_view(), name="pdf-detail"),
+    path("media/pdfs/<int:pk>/stream/", PdfMediaView.as_view(), name="pdf-stream"),
+
+    # PDF Re-import API endpoint (modern media framework)
+    # POST /api/media/pdfs/<int:pk>/reimport/
+    # Re-imports a PDF file to regenerate metadata when OCR failed or data is incomplete
+    path("media/pdfs/<int:pk>/reimport/", PdfReimportView.as_view(), name="pdf-reimport"),
 ]
     # ---------------------------------------------------------------------------------------

endoreg_db/urls/report.py

@@ -2,10 +2,7 @@ from django.urls import path
 from endoreg_db.views import (
     ReportListView,
     ReportWithSecureUrlView,
-    SecureFileUrlView,
     ReportFileMetadataView,
-    SecureFileServingView,
-    validate_secure_url,
 )
 
 url_patterns = [        # ---------------------------------------------------------------------------------------
@@ -40,16 +37,6 @@ url_patterns = [        # ------------------------------------------------------
             name='report_with_secure_url'
         ),
 
-        # API-Endpunkt für manuelle sichere URL-Generierung  
-        # POST /api/secure-file-urls/
-        # Body: {"report_id": 123, "file_type": "pdf"}
-        # Generiert eine neue sichere URL für einen bestehenden Report
-        path(
-            'secure-file-urls/', 
-            SecureFileUrlView.as_view(), 
-            name='generate_secure_file_url'
-        ),
-
         # API-Endpunkt für Report-Datei-Metadaten
         # GET /api/reports/{report_id}/file-metadata/
         # Gibt Datei-Metadaten zurück (Größe, Typ, Datum, etc.)
@@ -58,21 +45,4 @@ url_patterns = [        # ------------------------------------------------------
             ReportFileMetadataView.as_view(), 
             name='report_file_metadata'
         ),
-
-        # Sichere Datei-Serving-Endpunkt mit Token-Validierung
-        # GET /api/reports/{report_id}/secure-file/?token={token}
-        # Serviert die tatsächliche Datei über eine sichere, tokenbasierte URL
-        path(
-            'reports/<int:report_id>/secure-file/', 
-            SecureFileServingView.as_view(), 
-            name='secure_file_serving'
-        ),
-        # URL-Validierungs-Endpunkt
-        # GET /api/validate-secure-url/?url={url}
-        # Validiert, ob eine sichere URL noch gültig ist
-        path(
-            'validate-secure-url/', 
-            validate_secure_url, 
-            name='validate_secure_url'
-        ),
 ]

endoreg_db/urls/video.py

@@ -1,17 +1,10 @@
 from django.urls import path
 
 from endoreg_db.views import (
-    VideoReimportView,
     SensitiveMetaDetailView,
     VideoLabelView,
-    VideoStreamView,
-    # Video Correction Views (Phase 1.1)
-    VideoMetadataView,
-    VideoProcessingHistoryView,
-    VideoAnalyzeView,
-    VideoApplyMaskView,
-    VideoRemoveFramesView,
-    VideoReprocessView,
+    # Note: VideoStreamView moved to modern media framework. See: endoreg_db/urls/media.py
+    # Note: All Video Correction Views moved to modern media framework. See: endoreg_db/urls/media.py
 )
 
 url_patterns = [
@@ -24,23 +17,22 @@ url_patterns = [
         name='video_label_segments'
     ),
     
-    # Video streaming API endpoint
-    # GET /api/videostream/<int:pk>/
-    # Streams the raw video file with HTTP range and CORS support
-    path(
-        'videostream/<int:pk>/', 
-        VideoStreamView.as_view(), 
-        name='video_stream'
-    ),
+    # ---------------------------------------------------------------------------------------
+    # VIDEO STREAMING - MOVED TO MODERN MEDIA FRAMEWORK
+    #
+    # Video streaming endpoint has been migrated to the media framework
+    # as of October 14, 2025. Please use the new endpoints:
+    #
+    # OLD → NEW:
+    # GET /api/videostream/<pk>/              → GET /api/media/videos/<pk>/
+    # GET /api/videostream/<pk>/              → GET /api/media/videos/<pk>/stream/
+    #
+    # See: endoreg_db/urls/media.py for new URL registrations
+    # ---------------------------------------------------------------------------------------
+    
+    # Note: Video Re-import moved to modern media framework
+    # See: endoreg_db/urls/media.py - POST /api/media/videos/<int:pk>/reimport/
     
-    # Video Re-import API endpoint
-    # POST /api/video/<int:video_id>/reimport/
-    # Re-imports a video file to regenerate metadata when OCR failed or data is incomplete
-    path(
-        'video/<int:video_id>/reimport/', 
-        VideoReimportView.as_view(), 
-        name='video_reimport'
-    ),
     # Video Sensitive Meta endpoints (for video anonymization)
     # GET /api/video/sensitivemeta/<int:sensitive_meta_id>/
     # PATCH /api/video/sensitivemeta/<int:sensitive_meta_id>/
@@ -51,69 +43,19 @@ url_patterns = [
     ),
     
     # ---------------------------------------------------------------------------------------
-    # VIDEO CORRECTION API ENDPOINTS (Phase 1.1)
+    # VIDEO CORRECTION API ENDPOINTS - MOVED TO MODERN MEDIA FRAMEWORK
+    #
+    # All video correction endpoints have been migrated to the modern media framework
+    # as of October 14, 2025. Please use the new endpoints:
+    #
+    # OLD → NEW:
+    # GET  /api/video-metadata/<id>/              → GET  /api/media/videos/<pk>/metadata/
+    # GET  /api/video-processing-history/<id>/    → GET  /api/media/videos/<pk>/processing-history/
+    # POST /api/video-analyze/<id>/               → POST /api/media/videos/<pk>/analyze/
+    # POST /api/video-apply-mask/<id>/            → POST /api/media/videos/<pk>/apply-mask/
+    # POST /api/video-remove-frames/<id>/         → POST /api/media/videos/<pk>/remove-frames/
+    # POST /api/video-reprocess/<id>/             → POST /api/media/videos/<pk>/reprocess/
     #
-    # These endpoints enable video correction workflows:
-    # - Analysis: Detect sensitive frames using MiniCPM-o 2.6 or OCR+LLM
-    # - Masking: Apply device-specific masks or custom ROI masks
-    # - Frame Removal: Remove sensitive frames from videos
-    # - Processing History: Track all correction operations
+    # See: endoreg_db/urls/media.py for new URL registrations
     # ---------------------------------------------------------------------------------------
-    
-    # Video Metadata API
-    # GET /api/video-metadata/<int:id>/
-    # Returns analysis results (sensitive frame count, ratio, frame IDs)
-    path(
-        'video-metadata/<int:id>/',
-        VideoMetadataView.as_view(),
-        name='video_metadata'
-    ),
-    
-    # Video Processing History API
-    # GET /api/video-processing-history/<int:id>/
-    # Returns history of all processing operations (masking, frame removal, analysis)
-    path(
-        'video-processing-history/<int:id>/',
-        VideoProcessingHistoryView.as_view(),
-        name='video_processing_history'
-    ),
-    
-    # Video Analysis API
-    # POST /api/video-analyze/<int:id>/
-    # Analyzes video for sensitive frames using MiniCPM-o 2.6 or OCR+LLM
-    # Body: { detection_method: 'minicpm'|'ocr_llm'|'hybrid', sample_interval: 30 }
-    path(
-        'video-analyze/<int:id>/',
-        VideoAnalyzeView.as_view(),
-        name='video_analyze'
-    ),
-    
-    # Video Masking API
-    # POST /api/video-apply-mask/<int:id>/
-    # Applies device mask or custom ROI mask to video
-    # Body: { mask_type: 'device'|'custom', device_name: 'olympus', roi: {...} }
-    path(
-        'video-apply-mask/<int:id>/',
-        VideoApplyMaskView.as_view(),
-        name='video_apply_mask'
-    ),
-    
-    # Frame Removal API
-    # POST /api/video-remove-frames/<int:id>/
-    # Removes specified frames from video
-    # Body: { frame_list: [10,20,30] OR frame_ranges: '10-20,30' OR detection_method: 'automatic' }
-    path(
-        'video-remove-frames/<int:id>/',
-        VideoRemoveFramesView.as_view(),
-        name='video_remove_frames'
-    ),
-    
-    # Video Reprocessing API
-    # POST /api/video-reprocess/<int:id>/
-    # Re-runs entire anonymization pipeline for a video
-    path(
-        'video-reprocess/<int:id>/',
-        VideoReprocessView.as_view(),
-        name='video_reprocess'
-    ),
 ]

endoreg_db/views/anonymization/validate.py

@@ -21,6 +21,7 @@ class AnonymizationValidateView(APIView):
       "casenumber":         "12345",
       "anonymized_text":    "...",             // nur für PDFs; Videos ignorieren
       "is_verified":        true               // optional; default true
+      "file_type":        "video"            // optional; "video" oder "pdf"; wenn nicht angegeben, wird zuerst Video, dann PDF versucht
     }
     
     Rückwärtskompatibilität: ISO-Format (YYYY-MM-DD) wird ebenfalls akzeptiert.
@@ -35,7 +36,8 @@ class AnonymizationValidateView(APIView):
         payload.setdefault("is_verified", True)
 
         # Try Video first
-        video = VideoFile.objects.filter(pk=file_id).first()
+        if payload.get("file_type") == "video" or not payload.get("file_type"):
+            video = VideoFile.objects.filter(pk=file_id).first()
         if video:
             # Ensure center_name is in payload for hash calculation
             if video.center and not payload.get("center_name"):
@@ -49,7 +51,8 @@ class AnonymizationValidateView(APIView):
             return Response({"message": "Video validated."}, status=status.HTTP_200_OK)
 
         # Then PDF
-        pdf = RawPdfFile.objects.filter(pk=file_id).first()
+        if payload.get("file_type") == "pdf" or not payload.get("file_type"):
+            pdf = RawPdfFile.objects.filter(pk=file_id).first()
         if pdf:
             # Ensure center_name is in payload for hash calculation
             if pdf.center and not payload.get("center_name"):

endoreg_db/views/media/__init__.py

@@ -1,9 +1,45 @@
 # Media Management Views (Phase 1.2)
 
 from .video_media import VideoMediaView
-from .pdf_media import PDFMediaManagementView
+from .pdf_media import PdfMediaView
+from ..video.reimport import VideoReimportView
+from ..pdf.reimport import PdfReimportView
+from .segments import video_segments_by_pk
+from .video_segments import (
+    video_segments_collection,
+    video_segments_by_video,
+    video_segment_detail,
+    video_segments_stats,
+    video_segment_validate,
+    video_segments_validate_bulk,
+    video_segments_validation_status,
+)
+from .sensitive_metadata import (
+    video_sensitive_metadata,
+    video_sensitive_metadata_verify,
+    pdf_sensitive_metadata,
+    pdf_sensitive_metadata_verify,
+    sensitive_metadata_list,
+    pdf_sensitive_metadata_list,
+)
 
 __all__ = [
     'VideoMediaView',
-    'PDFMediaManagementView',
+    'PdfMediaView',
+    'VideoReimportView',
+    'PdfReimportView',
+    'video_segments_by_pk',
+    'video_segments_collection',
+    'video_segments_by_video',
+    'video_segment_detail',
+    'video_segments_stats',
+    'video_segment_validate',
+    'video_segments_validate_bulk',
+    'video_segments_validation_status',
+    'video_sensitive_metadata',
+    'video_sensitive_metadata_verify',
+    'pdf_sensitive_metadata',
+    'pdf_sensitive_metadata_verify',
+    'sensitive_metadata_list',
+    'pdf_sensitive_metadata_list',
 ]

endoreg_db/views/media/pdf_media.py

@@ -21,7 +21,7 @@ from endoreg_db.utils.permissions import EnvironmentAwarePermission
 logger = logging.getLogger(__name__)
 
 
-class PDFMediaManagementView(APIView):
+class PdfMediaView(APIView):
     """
     PDF Media Management API for CRUD operations on PDF files.
     

endoreg_db/views/media/segments.py

@@ -0,0 +1,71 @@
+"""
+Modern Media Framework - Video Segment API Views
+October 14, 2025 - Migration to unified /api/media/videos/<pk>/segments/ pattern
+
+This module provides modern framework views for video segment management,
+wrapping legacy segment views with pk-based parameter handling.
+"""
+from endoreg_db.models import Label, LabelVideoSegment, VideoFile
+from endoreg_db.serializers.label_video_segment.label_video_segment import LabelVideoSegmentSerializer
+
+from django.db import transaction
+from rest_framework import status
+from rest_framework.decorators import api_view, permission_classes
+from rest_framework.response import Response
+
+from endoreg_db.utils.permissions import EnvironmentAwarePermission
+import logging
+
+logger = logging.getLogger(__name__)
+
+
+@api_view(['GET'])
+@permission_classes([EnvironmentAwarePermission])
+def video_segments_by_pk(request, pk):
+    """
+    Modern media framework endpoint for retrieving video segments.
+    
+    GET /api/media/videos/<int:pk>/segments/?label=<label_name>
+    
+    Returns all segments for a video, optionally filtered by label name.
+    This is the modern replacement for /api/video/<id>/segments/
+    
+    Query Parameters:
+        label (str, optional): Filter segments by label name (e.g., 'outside')
+    
+    Returns:
+        200: List of video segments
+        404: Video not found
+    """
+    try:
+        video = VideoFile.objects.get(id=pk)
+    except VideoFile.DoesNotExist:
+        logger.warning(f"Video with pk {pk} not found")
+        return Response(
+            {'error': f'Video with id {pk} not found'},
+            status=status.HTTP_404_NOT_FOUND
+        )
+    
+    # Start with all segments for this video
+    queryset = LabelVideoSegment.objects.filter(video_file=video)
+    
+    # Optional filtering by label name
+    label_name = request.GET.get('label')
+    if label_name:
+        try:
+            label = Label.objects.get(name=label_name)
+            queryset = queryset.filter(label=label)
+            logger.info(f"Filtering segments for video {pk} by label '{label_name}'")
+        except Label.DoesNotExist:
+            logger.warning(f"Label '{label_name}' not found, returning empty result")
+            return Response(
+                {'error': f"Label '{label_name}' not found"},
+                status=status.HTTP_404_NOT_FOUND
+            )
+    
+    # Order by start time for consistent results
+    segments = queryset.order_by('start_frame_number')
+    serializer = LabelVideoSegmentSerializer(segments, many=True)
+    
+    logger.info(f"Returning {len(segments)} segments for video {pk}")
+    return Response(serializer.data)