Refactoring - #50
Merged
Merged
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Design for the Layer-2 Core foundation: apps/core package with a runtime-agnostic importFolder pipeline (config → metadata → recognize → episodes → persist), DI ports (fs/network/logger), and a queryable Job model. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
… search) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…hases Wrap NFO/tmdbid-name/tvdbid-name metadata fetches in try/catch so a network failure in one phase falls through to the next, matching the UI orchestrator which try/catches every phase. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…epo root Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…m catch, structural deps, invalid-path jobs, integration test) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Replace the removed multi-step rename flow across e2e helpers, docs, and localization so branch coverage exercises the supported one-shot plan API. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Capture architecture for Core-owned metadata CRUD, CLI RPC bridges with ProblemDetails, and UI/e2e clients that no longer know cache file paths. Co-authored-by: Cursor <cursoragent@cursor.com>
Task-by-task plan for Core CRUD, CLI ProblemDetails RPCs, UI hooks, and e2e HTTP helpers. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Retire /api/readMediaMetadata and /api/writeMediaMetadata now that UI and e2e callers use Core metadata RPCs or smm metadata --set. Co-authored-by: Cursor <cursoragent@cursor.com>
Add shared Radix context-menu helpers and scope episode-table interactions so Movie-RenameVideoFile e2e passes after metadata CRUD refactor. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Strip UI-only status/files before create-metadata, and point CLI e2e plan lookups at appDataDir so Linux CI matches Core persistence. Co-authored-by: Cursor <cursoragent@cursor.com>
BrowserNetworkPort imports NetworkPort from apps/core; ui.Dockerfile only copied packages/core, so Docker UI builds failed tsc while host CI passed. Co-authored-by: Cursor <cursoragent@cursor.com>
Metadata caches and plans live under getAppDataDir(); Linux XDG splits that from userDataDir, so MCP/AI/TV e2e looked for files Core never wrote. Co-authored-by: Cursor <cursoragent@cursor.com>
Persist mutations were writing API metadata into the query cache without UI-only files, and TV rows hid videoFile when that listing was empty. Preserve files across setQueryData, show video from mediaFiles, and pass userDataDir into chat tools for the XDG config path split. Co-authored-by: Cursor <cursoragent@cursor.com>
common/httpproxy was removed; Docker job matched zero specs. Metadata API no longer returns UI-only files, so InitializeTvShowByTvdb must not expect files: []. Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.