Match: update state for displaced Apple tracks #39

New issue

Closed

opened 2026-09-05 21:50:36 +00:00 by hermes · 1 comment

hermes commented

2026-09-05 21:50:36 +00:00

Owner

Context

Independent review of the SPEC reconciliation found that moving a beets item from Apple track A to Apple track B updates B to matched but can leave A persisted as matched after its last association is removed. That violates the state invariant in SPEC.md §7 and causes incorrect list-tracks output.

Task

When explicit or interactive matching reassigns an item already associated with another retained Apple track, reconcile the displaced track from its remaining beets associations and mark the selected track matched. Preserve a displaced permanent skip, and do not fail when the displaced Apple ID has no plugin-state row.

Acceptance criteria

Focused regression tests reproduce the stale state for explicit and interactive matching.
A displaced retained track with no association becomes unmatched.
One remaining association produces matched; multiple remaining associations produce ambiguous.
A displaced permanent skip remains skipped.
The selected track becomes matched and owns the beets association.
Dry-run remains non-mutating.

## Context Independent review of the SPEC reconciliation found that moving a beets item from Apple track A to Apple track B updates B to `matched` but can leave A persisted as `matched` after its last association is removed. That violates the state invariant in `SPEC.md` §7 and causes incorrect `list-tracks` output. ## Task When explicit or interactive matching reassigns an item already associated with another retained Apple track, reconcile the displaced track from its remaining beets associations and mark the selected track `matched`. Preserve a displaced permanent skip, and do not fail when the displaced Apple ID has no plugin-state row. ## Acceptance criteria - [ ] Focused regression tests reproduce the stale state for explicit and interactive matching. - [ ] A displaced retained track with no association becomes `unmatched`. - [ ] One remaining association produces `matched`; multiple remaining associations produce `ambiguous`. - [ ] A displaced permanent skip remains `skipped`. - [ ] The selected track becomes `matched` and owns the beets association. - [ ] Dry-run remains non-mutating.

hermes referenced this issue from a commit

2026-09-05 21:53:48 +00:00

Plugin: update displaced match state (#39)

hermes referenced this issue from a pull request that will close it,

2026-09-05 21:54:12 +00:00

Plugin: close remaining SPEC correctness gaps #41

hermes referenced this issue from a commit

2026-09-05 21:56:36 +00:00

Plugin: update displaced match state (#39)

hermes referenced this issue from a commit

2026-09-05 21:59:31 +00:00

Plugin: update displaced match state (#39)

hermes closed this issue

2026-09-05 22:01:21 +00:00

hermes referenced this issue from a commit

2026-09-05 22:01:22 +00:00

Merge pull request: close remaining SPEC correctness gaps (#41)

hermes commented

2026-09-05 22:02:00 +00:00

Author

Owner

Completed in coop/beets-appleplaylists by implementation commit cc124c9, merged through PR #41 as e4fbbffd4a0bf744d4967c6bfb03e4ffe369869a.

Explicit and interactive matching now reconcile the displaced track from its remaining association count, preserve displaced permanent skips, tolerate absent historical state rows, and set the selected track to matched.

Validation: 104 tests passed with warnings promoted to errors, plus Ruff lint, compileall, isolated CLI help, package build, and git diff --check.

Completed in `coop/beets-appleplaylists` by implementation commit `cc124c9`, merged through PR #41 as `e4fbbffd4a0bf744d4967c6bfb03e4ffe369869a`. Explicit and interactive matching now reconcile the displaced track from its remaining association count, preserve displaced permanent skips, tolerate absent historical state rows, and set the selected track to `matched`. Validation: 104 tests passed with warnings promoted to errors, plus Ruff lint, compileall, isolated CLI help, package build, and `git diff --check`.

hermes referenced this issue

2026-09-05 22:03:18 +00:00

Plugin: close remaining SPEC correctness gaps #41