Secret metadata provider

[nukeop]

No description provided.

[Copilot]

Pull Request Overview

Adds a new “Soytify” metadata provider, updates core plugin types and imports, and removes the legacy nuclear identity and remote‐playlists functionality throughout the app.

• Introduce SoytifyMetaProvider and register “Soytify” in the plugin types
• Remove all nuclear identity, remote playlists, and user‐panel code (selectors, reducers, actions, containers, and tests)
• Update import paths for getImageSet and clean up an unused ESLint disable

Reviewed Changes

Copilot reviewed 34 out of 34 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
packages/core/src/plugins/plugins.types.ts	Add Soytify to SearchResultsSource and reposition source field
packages/core/src/plugins/metaProvider.ts	Remove obsolete ESLint disable comment
packages/core/src/plugins/meta/spotify.ts	Update getImageSet import path
packages/core/src/plugins/meta/soytify/soytify.ts	Add new SoytifyMetaProvider
packages/app/test/storeBuilders.ts	Remove remotePlaylists and nuclear identity builders
packages/app/app/selectors/playlists.ts	Drop remotePlaylists selector
packages/app/app/selectors/nuclear.ts	Remove nuclear selectors
packages/app/app/reducers/playlists.ts	Remove remotePlaylists from store class
packages/app/app/reducers/nuclear/index.ts	Delete nuclear reducer
packages/app/app/reducers/nuclear/identity.ts	Delete nuclear identity reducer
packages/app/app/reducers/index.ts	Remove nuclear from rootReducer
packages/app/app/containers/UserPanelContainer/index.tsx	Remove entire UserPanelContainer
packages/app/app/containers/UserPanelContainer/*.test.tsx	Remove all UserPanelContainer tests
packages/app/app/containers/UserPanelContainer/*.tsx	Remove NuclearSignIn/SignUpForm containers
packages/app/app/containers/PlaylistsContainer/hooks.ts	Remove useRemotePlaylists hook
packages/app/app/containers/PlaylistsContainer/*.test.tsx	Remove remote‐playlists tests
packages/app/app/actions/playlists.ts	Remove remote playlists actions and imports
packages/app/app/actions/nuclear/identity.ts	Delete nuclear identity actions
packages/app/app/actions/actionTypes.ts	Remove remote‐playlists action types

Comments suppressed due to low confidence (3)

packages/core/src/plugins/plugins.types.ts:11

• [nitpick] The new enum member Soytify may be confusingly similar to Spotify. Consider renaming it or adding a comment to clarify it’s a distinct source.

Soytify = 'Soytify',

packages/core/src/plugins/metaProvider.ts:38

• Removing this disable may reintroduce lint errors on unused abstract methods. Either implement or re-add the eslint disable if the stub is still needed.

// eslint-disable-next-line @typescript-eslint/no-unused-vars

packages/core/src/plugins/meta/soytify/soytify.ts:1

• New SoytifyMetaProvider is introduced without any accompanying unit tests. Consider adding tests to validate its search and fetch methods.

import { SoytifyClientProvider } from '../../../rest/soytify/soytify-partners-api';

[codecov]

Codecov Report

❌ Patch coverage is 39.60396% with 305 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.26%. Comparing base (962fefd) to head (b80b5fa).
⚠️ Report is 7 commits behind head on master.

Files with missing lines	Patch %	Lines
...ages/core/src/rest/soytify/soytify-partners-api.ts	23.88%	46 Missing and 5 partials ⚠️
packages/core/src/rest/cache/CacheManager.ts	5.71%	27 Missing and 6 partials ⚠️
packages/core/src/plugins/meta/soytify/soytify.ts	11.76%	28 Missing and 2 partials ⚠️
packages/core/src/util/index.ts	15.15%	14 Missing and 14 partials ⚠️
packages/core/src/rest/cache/CacheDecorator.ts	26.47%	18 Missing and 7 partials ⚠️
packages/core/src/logger/nuclear-logger.ts	7.69%	18 Missing and 6 partials ⚠️
...components/SearchResults/PlaylistResults/index.tsx	32.25%	17 Missing and 4 partials ⚠️
packages/core/src/rest/soytify/soytify-mappers.ts	32.25%	13 Missing and 8 partials ⚠️
packages/core/src/rest/soytify/soytify-totp.ts	10.52%	17 Missing ⚠️
...s/app/app/components/SearchResults/ItemResults.tsx	41.17%	7 Missing and 3 partials ⚠️
... and 9 more

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1801      +/-   ##
==========================================
- Coverage   67.81%   66.26%   -1.56%     
==========================================
  Files         379      375       -4     
  Lines        7747     7730      -17     
  Branches      635      676      +41     
==========================================
- Hits         5254     5122     -132     
- Misses       1969     2046      +77     
- Partials      524      562      +38     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.