Repository navigation
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
rix1337
changed the title
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
rix1337
changed the title
3 tasks done
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
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.
Fixes #470
DT layout change
DT redesigned its post and listing pages, which broke DT in two ways:
div.full-story-body) out of<article>, so every DT grab failed with "Could not find download section". The handler now looks up that body anywhere on the page. Pages that still nest it inside<article>keep working.div.h1.dt-flexible-title, and the calendar date block was replaced by add.mm.yyyy hh:mmline. The feed found no titles and returned nothing. Titles now match by class, and dates fall back to the inline timestamp when the calendar block is missing. Search result pages still use the old markup and parse the same as before.HE publish dates (from #471)
HE search results only show a relative age ("Posted 2 days ago"), so each search reported a different
publishedDatefor the same release. Sonarr's blocklist compares publish dates, so after a failed HE grab it grabbed the same release again on the next search. Search already fetches each HE post page, and that page carries an absolutedatePublishedtimestamp. Results now use it and fall back to the relative age when it is missing.No config or hostname changes.
Verification:
datePublishedpresent on every sampled post and parsed to UTCtrail check🤖 Generated with Claude Code