Fix Netflix by lolamtisch · Pull Request #4223 · MALSync/MALSync · GitHub
Skip to content

Fix Netflix - #4223

Open
lolamtisch wants to merge 2 commits into
masterfrom
netflix_fix
Open

lolamtisch wants to merge 2 commits into
masterfrom
netflix_fix

Conversation

@lolamtisch

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings September 21, 2026 12:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Next-episode behavior and identifier compatibility must be preserved, and Netflix test fixtures are missing.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 Medium severity

Open (1)
What changed in this PR

Migrates Netflix support from the legacy page system to the Chibi implementation.

Changes:

  • Removes the legacy Netflix integration.
  • Adds Chibi Netflix registration, metadata, detection, styling, and synchronization.
  • Updates request interception to read streamed fetch responses.
File Summary
src/​pages/​pages.ts Removes legacy Netflix registration.
src/​pages/​Netflix/​style.less Removes legacy styling.
src/​pages/​Netflix/​proxy.ts Removes legacy proxy.
src/​pages/​Netflix/​meta.json Removes legacy metadata.
src/​pages/​Netflix/​main.ts Removes the legacy implementation.
src/​pages-chibi/​proxies/​requestProxy.ts Adds streamed fetch-response handling.
src/​pages-chibi/​pages.ts Registers the Chibi Netflix implementation.
src/​pages-chibi/​implementations/​Netflix/​style.less Adds Netflix-specific styling.
src/​pages-chibi/​implementations/​Netflix/​main.ts Adds synchronization logic; next-episode behavior, identifier compatibility, and Netflix fixtures remain to be addressed.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +134 to +137
getEpisode($c) {
return $c.getGlobalVariable<NetflixEpisodeNode>('meta.episode').get('seq').run();
},
},
@senkoisbroken

Copy link
Copy Markdown

wouldnt i need to redo this fix after a page reload?

@lolamtisch

Copy link
Copy Markdown
Member Author

This branch has not been deployed

No deployments
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.

3 participants