Skip to content

fix(weekly-flow): distinguish queued upgrades and Lidarr tracks - #577

Merged
lklynet merged 1 commit into
mainfrom
t3code/fix-upgrade-track-labels
Aug 10, 2026
Merged

fix(weekly-flow): distinguish queued upgrades and Lidarr tracks#577
lklynet merged 1 commit into
mainfrom
t3code/fix-upgrade-track-labels

Conversation

@lklynet

@lklynet lklynet commented Aug 10, 2026

Copy link
Copy Markdown
Owner

Summary

Distinguish queued quality upgrades from ineligible tracks and label files reused from Lidarr. Add regression coverage for both behaviors.

Linked issues

None

Validation

  • CI passes
  • Tested using the ghcr.io/lklynet/aurral:pr-<number> preview image, or not required
  • Upgrade, migration, and rollback notes are updated where applicable

Test plan

  • Affected area(s): weekly flow, quality upgrades, playlists, UI, documentation
  • Automated coverage: Added tests for duplicate queued upgrades and reused Lidarr files.
  • Manual steps and expected result: Start an upgrade search twice and confirm the second attempt reports that it is already queued. Verify Lidarr-reused tracks display the Lidarr label and are not eligible for upgrades.

Release impact

  • Major: incompatible change
  • Minor: backward-compatible feature
  • Patch: backward-compatible fix
  • None: documentation, CI, tests, or internal-only change

Summary by CodeRabbit

  • New Features
    • Re-search requests now clearly indicate whether an upgrade was newly queued or already in progress.
    • Tracks reused from Lidarr are labeled “Lidarr” in playlist quality information.
  • Bug Fixes
    • Duplicate upgrade requests no longer return an error when an upgrade is already queued.
    • External Lidarr files are correctly classified and excluded from upgrade eligibility checks.
  • Documentation
    • Updated playlist guidance to explain Lidarr labels and queued upgrade behavior.

- Report already queued upgrade searches
- Label reused Lidarr files and exclude them from upgrades
@github-actions github-actions Bot added the bug Something isn't working label Aug 10, 2026
@github-actions

Copy link
Copy Markdown

Aurral preview image ready

This image was rebuilt from the latest push to this pull request. It will be replaced when you push another change.

docker pull ghcr.io/lklynet/aurral:pr-577

To test it with your existing Docker Compose setup:

  1. Back up your Aurral config.
  2. Temporarily change the Aurral service image to ghcr.io/lklynet/aurral:pr-577.
  3. Run docker compose pull aurral && docker compose up -d aurral.
  4. Exercise the behavior changed by this pull request.
  5. Restore the image reference that was configured before testing.

View the preview workflow run · Report a problem

@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 1f64e7a3-d89f-4ff4-9417-768b5addd00c

📥 Commits

Reviewing files that changed from the base of the PR and between 1f6cec7 and 713106e.

📒 Files selected for processing (8)
  • .tests/weekly-flow/approved-import-path.test.js
  • .tests/weekly-flow/download-tracker.test.js
  • backend/routes/weeklyFlow/handlers/jobs.js
  • backend/services/qualityProfileService.js
  • backend/services/weeklyFlow/weeklyFlowDownloadTracker.js
  • docs/src/content/docs/using/playlists.mdx
  • frontend/src/pages/FlowPage.jsx
  • frontend/src/pages/flows/flowComponents/flowTrackComponents.jsx

📝 Walkthrough

Walkthrough

The change updates quality classification for completed files stored outside the library, including reused Lidarr files. Upgrade queueing now returns distinct statuses and detects active duplicate jobs. The route returns a successful response for already-queued requests. The frontend displays separate messages for new and duplicate searches. Tests cover repeated requests and external-file classification. Documentation describes the Lidarr label and duplicate-search behavior.


Comment @coderabbitai help to get the list of available commands.

@lklynet
lklynet merged commit 60b3c0d into main Aug 10, 2026
7 checks passed
@lklynet
lklynet deleted the t3code/fix-upgrade-track-labels branch August 10, 2026 01:46
@github-actions github-actions Bot mentioned this pull request Aug 10, 2026
5 tasks
@github-actions github-actions Bot added the nightly Available in the nightly image but not yet in a stable release. label Aug 10, 2026
@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown

Included in stable release 2.3.0

This change is included in the Aurral 2.3.0 release.

docker pull ghcr.io/lklynet/aurral:2.3.0

View the release

@github-actions github-actions Bot added released Included in a stable release. and removed nightly Available in the nightly image but not yet in a stable release. labels Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working released Included in a stable release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant