mirror of https://github.com/Nezreka/SoulSync.git
You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
164 lines
6.6 KiB
164 lines
6.6 KiB
"""Pure planners for sync-editor playlist mutations (#768 Bug C).
|
|
|
|
The sync editor's "Find & add" and remove actions rewrite the whole server
|
|
playlist from a flat list of track IDs (Subsonic/Navidrome + Jellyfin have no
|
|
position-level ops). Two bugs lived in the inline endpoint logic:
|
|
|
|
* **Duplicate on manual match.** "Find & add" always *inserted* the chosen
|
|
track — but when the user is matching an UNMATCHED source to a server track
|
|
that's already in the playlist (an orphan "extra"), the intent is to LINK
|
|
them, not add a second copy. Each attempt appended another duplicate
|
|
(positions 72, 73, 74…). ``plan_playlist_add`` skips the insert when it's a
|
|
link to an already-present track (the caller still persists the override).
|
|
|
|
* **Delete removes ALL copies.** The inline remove filtered out *every* entry
|
|
with the target ID. With duplicates present, deleting one removed them all.
|
|
``remove_one_occurrence`` drops a single entry (duplicates are the same
|
|
track, so removing any one is correct).
|
|
|
|
Pure, no I/O — the caller fetches the current track-id list and applies the
|
|
returned plan to the media-server client.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
from typing import List, Optional, Tuple
|
|
|
|
|
|
def plan_playlist_add(
|
|
current_ids: List[str],
|
|
track_id: str,
|
|
*,
|
|
is_link: bool,
|
|
position: Optional[int] = None,
|
|
) -> dict:
|
|
"""Plan a "Find & add" against a flat track-id playlist.
|
|
|
|
``is_link`` is True when the add carries a ``source_track_id`` (i.e. the
|
|
user is matching an unmatched source to this server track). In that case,
|
|
if the track is ALREADY in the playlist, return ``should_insert=False`` so
|
|
the caller only records the override and never duplicates it.
|
|
|
|
Returns ``{'should_insert': bool, 'new_ids': [...]}``. ``new_ids`` equals
|
|
the input (stringified) when no insert is needed."""
|
|
tid = str(track_id)
|
|
current = [str(t) for t in current_ids]
|
|
if is_link and tid in current:
|
|
return {"should_insert": False, "new_ids": current}
|
|
pos = len(current) if position is None else max(0, min(int(position), len(current)))
|
|
new_ids = current[:pos] + [tid] + current[pos:]
|
|
return {"should_insert": True, "new_ids": new_ids}
|
|
|
|
|
|
def remove_one_occurrence(
|
|
track_ids: List[str],
|
|
target_id: str,
|
|
position: Optional[int] = None,
|
|
) -> Tuple[List[str], bool]:
|
|
"""Remove a SINGLE occurrence of ``target_id`` from a flat id list.
|
|
|
|
If ``position`` is given and the id there matches, that exact entry is
|
|
removed (so the user removes the row they clicked); otherwise the first
|
|
matching id is removed. Returns ``(new_ids, removed)``. ``removed`` is
|
|
False when the id isn't present (caller should 404)."""
|
|
target = str(target_id)
|
|
ids = [str(t) for t in track_ids]
|
|
if position is not None and 0 <= position < len(ids) and ids[position] == target:
|
|
return ids[:position] + ids[position + 1:], True
|
|
for idx, tid in enumerate(ids):
|
|
if tid == target:
|
|
return ids[:idx] + ids[idx + 1:], True
|
|
return ids, False
|
|
|
|
|
|
def plan_playlist_reconcile(
|
|
current_ids: List[str],
|
|
desired_ids: List[str],
|
|
) -> dict:
|
|
"""Plan an in-place reconcile of a server playlist toward a desired tracklist.
|
|
|
|
Used by ``sync_mode='reconcile'`` (#792): instead of deleting + recreating
|
|
the playlist (which destroys its custom image, description, and identity),
|
|
the caller keeps the existing playlist object and applies only the delta —
|
|
adding the tracks that are missing and removing the ones no longer in the
|
|
source. Pure, no I/O.
|
|
|
|
Returns ``{'add': [...], 'remove': [...]}`` (both lists of string ids):
|
|
- ``add`` — desired ids not currently present, in desired order.
|
|
- ``remove`` — current ids no longer desired (each occurrence kept once;
|
|
duplicates of a still-desired id are left for the caller's
|
|
dedupe to handle, never mass-removed).
|
|
|
|
Order-preserving and duplicate-safe: a desired id already present is not
|
|
re-added; a current id that's still desired is not removed even if it
|
|
appears more than once.
|
|
"""
|
|
desired = [str(t) for t in desired_ids]
|
|
current = [str(t) for t in current_ids]
|
|
current_set = set(current)
|
|
desired_set = set(desired)
|
|
add = [d for d in desired if d not in current_set]
|
|
# Preserve order of removal as it appears in the current list; one entry per
|
|
# id (the caller maps ids back to concrete playlist entries to delete).
|
|
seen_remove = set()
|
|
remove = []
|
|
for c in current:
|
|
if c not in desired_set and c not in seen_remove:
|
|
seen_remove.add(c)
|
|
remove.append(c)
|
|
return {"add": add, "remove": remove}
|
|
|
|
|
|
def plan_playlist_append(
|
|
current_ids: List[str],
|
|
desired_ids: List[str],
|
|
) -> List[str]:
|
|
"""Plan an append: which desired ids are NOT already in the playlist.
|
|
|
|
Used by ``sync_mode='append'`` (#823 round 2): the Jellyfin/Emby and
|
|
Navidrome appends deduped with ``{t.id for t in existing}`` — but their
|
|
track wrappers only define ``ratingKey``, never ``id``, so the existing-ids
|
|
set was ALWAYS empty and every sync re-appended the full matched list
|
|
(every track N times, "skipped 0 already present"). Pure planner so the
|
|
dedupe logic is testable; the caller fetches current ids however its
|
|
server API works and applies the returned adds.
|
|
|
|
Order-preserving and duplicate-safe: desired order is kept, ids already
|
|
present are dropped, and duplicates WITHIN desired are emitted once.
|
|
"""
|
|
current_set = {str(t) for t in current_ids}
|
|
out: List[str] = []
|
|
seen = set()
|
|
for d in desired_ids:
|
|
tid = str(d)
|
|
if tid and tid not in current_set and tid not in seen:
|
|
seen.add(tid)
|
|
out.append(tid)
|
|
return out
|
|
|
|
|
|
VALID_SYNC_MODES = ("replace", "append", "reconcile")
|
|
|
|
|
|
def normalize_sync_mode(requested, configured, default: str = "replace") -> str:
|
|
"""Resolve the effective playlist sync mode.
|
|
|
|
An explicit per-request value wins; otherwise the configured default
|
|
(Settings > Playlist sync mode); anything unrecognized falls back to
|
|
``default``. Keeping ``reconcile`` in ``VALID_SYNC_MODES`` is load-bearing —
|
|
a validation list that omits it silently downgrades reconcile to replace,
|
|
which is exactly the #792 regression this helper exists to prevent.
|
|
"""
|
|
mode = (requested or "") or (configured or "") or default
|
|
return mode if mode in VALID_SYNC_MODES else default
|
|
|
|
|
|
__all__ = [
|
|
"plan_playlist_add",
|
|
"remove_one_occurrence",
|
|
"plan_playlist_reconcile",
|
|
"plan_playlist_append",
|
|
"normalize_sync_mode",
|
|
"VALID_SYNC_MODES",
|
|
]
|