Skip to content

feat: Add new source: iqiyi-hot#283

Merged
ourongxing merged 3 commits intoourongxing:devfrom
bruceblink:iqiyi-hot
Dec 19, 2025
Merged

feat: Add new source: iqiyi-hot#283
ourongxing merged 3 commits intoourongxing:devfrom
bruceblink:iqiyi-hot

Conversation

@bruceblink
Copy link
Copy Markdown
Contributor

@bruceblink bruceblink commented Dec 17, 2025

新增"爱奇艺-热播榜"和"爱奇艺-动漫-追番表"的数据源


Note

Introduce new iQiyi sources for hot ranklist and today’s cartoon schedule, and wire them into typings, registry, and pinyin mappings.

  • Sources:
    • Add server/sources/iqiyi.ts providing iqiyi-hot-ranklist and iqiyi-cartoon-today.
  • Wiring/Registry:
    • Export iqiyi in server/glob.d.ts.
    • Register iqiyi with subs in shared/pre-sources.ts (metadata, intervals, homes).
    • Map pinyin for iqiyi-hot-ranklist and iqiyi-cartoon-today in shared/pinyin.json.
    • Update shared/sources.json by removing legacy youku entries.

Written by Cursor Bugbot for commit 611f622. This will update automatically on new commits. Configure here.

@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 17, 2025

@bruceblink is attempting to deploy a commit to the ourongxing's projects Team on Vercel.

A member of the Team first needs to authorize it.

@ourongxing ourongxing changed the base branch from main to dev December 19, 2025 10:05
@ourongxing ourongxing merged commit 5df05c7 into ourongxing:dev Dec 19, 2025
1 of 2 checks passed
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Comment thread shared/sources.json
"color": "green",
"interval": 600000
},
"youku": {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: Missing iqiyi and youku entries in generated sources.json

The sources.json file is auto-generated by running npm run presource, but in this PR the youku entries are removed while no iqiyi entries are added. Since both iqiyi and youku are defined in pre-sources.ts, the genSources() function should generate entries for both. This means the new iQiyi source won't work (entries not present) and the existing Youku source is broken (entries accidentally deleted). The file appears to have been manually edited instead of being properly regenerated.

Fix in Cursor Fix in Web

Comment thread shared/pinyin.json

"iqiyi-hot-ranklist": "aiqiyi-rebobang",
"iqiyi-cartoon-today": "aiqiyi-dongman-zhuifanbiao"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: Invalid JSON syntax with missing comma

The pinyin.json file has invalid JSON syntax - there's a missing comma after the iqiyi-cartoon-today entry (line 53) before the youku-webcomic-today entry (line 55). This file is supposed to be auto-generated by npm run presource using JSON.stringify, which would produce valid JSON. The manual edit introduced a parse error that will cause the application to crash when loading this configuration file.

Fix in Cursor Fix in Web

ourongxing added a commit that referenced this pull request Dec 19, 2025
* feat: Add new source: iqiyi-hot

* feat: 数据源"iqiyi-hot-rankList"更名为"iqiyi-hot-ranklist",并统一调整相关的数据接口

---------

Co-authored-by: Ou <orongxing@gmail.com>
Darkstar326 pushed a commit to iflow-mcp/ourongxing-newsnow that referenced this pull request Feb 11, 2026
* feat: Add new source: iqiyi-hot

* feat: 数据源"iqiyi-hot-rankList"更名为"iqiyi-hot-ranklist",并统一调整相关的数据接口

---------

Co-authored-by: Ou <orongxing@gmail.com>
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.

2 participants