Skip to content

feat: add daily Spotify sync scheduling - #61

Merged
rianjs merged 4 commits into
mainfrom
codex/60-daily-spotify-sync
Sep 3, 2026
Merged

feat: add daily Spotify sync scheduling#61
rianjs merged 4 commits into
mainfrom
codex/60-daily-spotify-sync

Conversation

@rianjs

@rianjs rianjs commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • schedule automatic Spotify syncs every 24 hours with private persisted next-sync state and legacy fallback
  • centralize durable quota/transient cooldown precedence, observation, expiration, and network-only quota clearing
  • expose compositional Spotify sync status and compose the clickable Spotify/Last.fm status bar surfaces
  • propagate cache-vs-network metadata through Last.fm matching and update architecture documentation

Verification

  • cargo fmt --all --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace
  • npm run test --prefix apps/desktop -- --run
  • npm run lint --prefix apps/desktop
  • npm run build --prefix apps/desktop
  • node scripts/check-docs.mjs
  • node scripts/check-release.mjs
  • node scripts/check-tauri-acl.mjs
  • npm exec --prefix apps/desktop tauri permission list
  • git diff --check

Closes #60

@rianjs

rianjs commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator Author

CI follow-up: the initial frontend failure was unrelated to the new permission. In a pristine checkout, and on current main run 33507025070, fails before any Rust build with . The existing already validates the checked-in command/permission manifest, and the native matrix builds Tauri on each platform, so commit 4cccf0a removes this artifact-dependent duplicate step instead of adding a Rust build to the frontend job.

@rianjs

rianjs commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator Author

Correction to the previous comment: in a pristine checkout, and on current main run 33507025070, the command npx tauri permission list fails before any Rust build with the message permission file not found, please build your application once first. The existing script scripts/check-tauri-acl.mjs already validates the checked-in command and permission manifest. Commit 4cccf0a removes this artifact-dependent duplicate step; it does not add a Rust build to the frontend job.

@rianjs
rianjs force-pushed the codex/60-daily-spotify-sync branch from 04f4f2e to a4d51d1 Compare September 3, 2026 14:13
@rianjs
rianjs merged commit 1103d98 into main Sep 3, 2026
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Daily Spotify sync, durable cooldown, and compositional status bar

1 participant