Summary of Changes

Hello @deluan, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request introduces new API functionality to allow users to update specific aspects of their play queue (like current track or position) or clear it entirely, in addition to the existing ability to replace the whole queue. This involved extending the data repository interface and implementation to support partial updates and clearing, and adding new API handlers and tests.

Highlights

• New API Endpoints: I've added two new endpoints to the native API for managing the user's play queue: PUT /queue for updating specific fields and DELETE /queue for clearing the entire queue.
• Partial Queue Updates: The PlayQueueRepository.Store method has been modified to support partial updates. It now accepts optional column names, allowing callers (like the new PUT endpoint) to specify which fields (items, current, position) should be updated without replacing the entire queue. If no columns are specified, it behaves like the previous full replacement.
• Repository Interface & Implementation: The model.PlayQueueRepository interface and its persistence implementation have been extended to include a Clear method, used by the new DELETE endpoint, and the Store method signature was updated to support partial updates.
• Payload Struct Reuse: The queuePayload struct was renamed to updateQueuePayload and its fields made optional (pointers with omitempty) so it can be reused for both the POST /queue (full save) and PUT /queue (partial update) endpoints.
• New Tests: Comprehensive tests have been added for the new PUT /queue and DELETE /queue endpoints, covering successful operations and error cases, and existing POST /queue tests were updated to use the new payload struct.

Code Review

The pull request introduces update and clear play queue endpoints. The changes include extending the PlayQueueRepository with a Clear method and supporting partial updates in playQueueRepository.Store. The review suggests adding more test cases for the new endpoints and considering the validation of the current index when updating the queue.

Pull Request Overview

Adds support for partial updates and clearing of the play queue via new repository methods, HTTP endpoints, and tests.

• Extend PlayQueueRepository with Clear, RetrieveWithMediaFiles, and optional column updates in Store
• Implement PUT /queue and DELETE /queue handlers and register routes
• Add comprehensive tests for update and clear actions

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

copyItems := make(model.MediaFiles, len(q.Items))
copy(copyItems, q.Items)
qCopy.Items = copyItems

qCopy.Items = copyItems

pos := V(payload.Position)
if pos < 0 { pos = 0 }
pq.Position = pos

Position:  max(V(payload.Position), 0),

pos := *payload.Position
if pos < 0 { pos = 0 }
pq.Position = pos

pq.Position = max(*payload.Position, 0) // Ensure position is non-negative