Conversation
Walkthrough本次变更大幅清理了各视频聚合源(如独播影视、荐片、天天影视、快车资源、华为吧、MP4电影网等)相关的所有路由、请求、命名空间和工具模块,仅保留了部分核心配置与常量。配置文件优化为从外部常量导入敏感词数组。部分命名空间字符串做了简体化。 Changes
Sequence Diagram(s)sequenceDiagram
participant config as 配置模块
participant word as 敏感词常量模块
Note over config,word: 初始化流程
config->>word: 导入 BANNED_KEYWORDS
config-->>config: bannedKeywords 赋值为 BANNED_KEYWORDS(如无环境变量)
Poem
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
src/constant/word.ts (1)
1-1: 建议将关键字数组声明为只读以防止被意外篡改当前导出的数组仍可在运行时被修改。出于配置安全性与类型推导友好性,建议使用
as const或Object.freeze让其只读化。-export const BANNED_KEYWORDS = ['福利', '伦理', '成人', '三级', '激情', '情色', '里番', '无码', '未分类', '两性', '热舞', '女频']; +export const BANNED_KEYWORDS = [ + '福利', + '伦理', + '成人', + '三级', + '激情', + '情色', + '里番', + '无码', + '未分类', + '两性', + '热舞', + '女频', +] as const;这样可避免外部模块对关键列表进行 push/splice 等操作。
src/config/index.ts (1)
6-13:looger拼写错误可能导致日志配置被忽略属性名应为
logger才符合常见约定,也便于 IDE/类型检查捕获。建议修正拼写并同步影响到的调用方。
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (49)
src/config/index.ts(2 hunks)src/constant/word.ts(1 hunks)src/routes/dubo/category.ts(0 hunks)src/routes/dubo/detail.ts(0 hunks)src/routes/dubo/home.ts(0 hunks)src/routes/dubo/homeVod.ts(0 hunks)src/routes/dubo/namespace.ts(0 hunks)src/routes/dubo/play.ts(0 hunks)src/routes/dubo/request.ts(0 hunks)src/routes/dubo/search.ts(0 hunks)src/routes/hw8/category.ts(0 hunks)src/routes/hw8/detail.ts(0 hunks)src/routes/hw8/home.ts(0 hunks)src/routes/hw8/homeVod.ts(0 hunks)src/routes/hw8/namespace.ts(0 hunks)src/routes/hw8/play.ts(0 hunks)src/routes/hw8/search.ts(0 hunks)src/routes/jianpian/category.ts(0 hunks)src/routes/jianpian/detail.ts(0 hunks)src/routes/jianpian/home.ts(0 hunks)src/routes/jianpian/homeVod.ts(0 hunks)src/routes/jianpian/namespace.ts(0 hunks)src/routes/jianpian/play.ts(0 hunks)src/routes/jianpian/request.ts(0 hunks)src/routes/jianpian/search.ts(0 hunks)src/routes/kczy/category.ts(0 hunks)src/routes/kczy/detail.ts(0 hunks)src/routes/kczy/home.ts(0 hunks)src/routes/kczy/homeVod.ts(0 hunks)src/routes/kczy/namespace.ts(0 hunks)src/routes/kczy/play.ts(0 hunks)src/routes/kczy/search.ts(0 hunks)src/routes/lzzy/namespace.ts(1 hunks)src/routes/mp4movie/category.ts(0 hunks)src/routes/mp4movie/detail.ts(0 hunks)src/routes/mp4movie/home.ts(0 hunks)src/routes/mp4movie/homeVod.ts(0 hunks)src/routes/mp4movie/namespace.ts(0 hunks)src/routes/mp4movie/play.ts(0 hunks)src/routes/mp4movie/request.ts(0 hunks)src/routes/mp4movie/search.ts(0 hunks)src/routes/tiantian/category.ts(0 hunks)src/routes/tiantian/detail.ts(0 hunks)src/routes/tiantian/home.ts(0 hunks)src/routes/tiantian/homeVod.ts(0 hunks)src/routes/tiantian/namespace.ts(0 hunks)src/routes/tiantian/play.ts(0 hunks)src/routes/tiantian/request.ts(0 hunks)src/routes/tiantian/search.ts(0 hunks)
💤 Files with no reviewable changes (46)
- src/routes/kczy/play.ts
- src/routes/hw8/homeVod.ts
- src/routes/hw8/category.ts
- src/routes/kczy/home.ts
- src/routes/jianpian/play.ts
- src/routes/kczy/search.ts
- src/routes/hw8/home.ts
- src/routes/mp4movie/namespace.ts
- src/routes/tiantian/request.ts
- src/routes/jianpian/namespace.ts
- src/routes/dubo/namespace.ts
- src/routes/kczy/namespace.ts
- src/routes/tiantian/namespace.ts
- src/routes/kczy/homeVod.ts
- src/routes/hw8/search.ts
- src/routes/hw8/detail.ts
- src/routes/kczy/category.ts
- src/routes/kczy/detail.ts
- src/routes/mp4movie/request.ts
- src/routes/hw8/namespace.ts
- src/routes/jianpian/detail.ts
- src/routes/mp4movie/homeVod.ts
- src/routes/dubo/homeVod.ts
- src/routes/hw8/play.ts
- src/routes/mp4movie/home.ts
- src/routes/jianpian/request.ts
- src/routes/mp4movie/play.ts
- src/routes/dubo/play.ts
- src/routes/jianpian/search.ts
- src/routes/tiantian/detail.ts
- src/routes/tiantian/play.ts
- src/routes/dubo/request.ts
- src/routes/mp4movie/search.ts
- src/routes/jianpian/category.ts
- src/routes/jianpian/home.ts
- src/routes/dubo/category.ts
- src/routes/dubo/home.ts
- src/routes/tiantian/home.ts
- src/routes/tiantian/homeVod.ts
- src/routes/dubo/search.ts
- src/routes/tiantian/search.ts
- src/routes/mp4movie/category.ts
- src/routes/tiantian/category.ts
- src/routes/mp4movie/detail.ts
- src/routes/jianpian/homeVod.ts
- src/routes/dubo/detail.ts
🧰 Additional context used
🧬 Code Graph Analysis (1)
src/config/index.ts (1)
src/constant/word.ts (1)
BANNED_KEYWORDS(1-1)
🔇 Additional comments (1)
src/routes/lzzy/namespace.ts (1)
4-7: 简体化命名无功能性风险,变更可接受
| }, | ||
| // 禁用关键词 | ||
| bannedKeywords: process.env.BANNED_KEYWORDS?.split(',').filter((keyword) => keyword.trim()) || ['福利', '伦理', '成人', '三级', '激情', '情色', '里番', '无码'] | ||
| bannedKeywords: process.env.BANNED_KEYWORDS?.split(',').filter((keyword) => keyword.trim()) || BANNED_KEYWORDS |
There was a problem hiding this comment.
空字符串会令禁用词列表退化为空数组
process.env.BANNED_KEYWORDS 若存在但内容为空 (''),split 结果为 [''],filter 后得到 [],从而覆盖默认值导致过滤逻辑失效。
- bannedKeywords: process.env.BANNED_KEYWORDS?.split(',').filter((keyword) => keyword.trim()) || BANNED_KEYWORDS
+ bannedKeywords: (() => {
+ const envList = process.env.BANNED_KEYWORDS
+ ?.split(',')
+ .map((kw) => kw.trim())
+ .filter(Boolean);
+ return envList && envList.length ? envList : BANNED_KEYWORDS;
+ })()这样既保留了运行时配置能力,又能在未设置或空值时安全回退。
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| bannedKeywords: process.env.BANNED_KEYWORDS?.split(',').filter((keyword) => keyword.trim()) || BANNED_KEYWORDS | |
| bannedKeywords: (() => { | |
| const envList = process.env.BANNED_KEYWORDS | |
| ?.split(',') | |
| .map((kw) => kw.trim()) | |
| .filter(Boolean); | |
| return envList && envList.length ? envList : BANNED_KEYWORDS; | |
| })() |
🤖 Prompt for AI Agents
In src/config/index.ts at line 24, the current code splits
process.env.BANNED_KEYWORDS and filters empty strings, but if BANNED_KEYWORDS is
an empty string, it results in an empty array overriding the default. To fix
this, check if process.env.BANNED_KEYWORDS is a non-empty string before
splitting and filtering; if it is empty or undefined, fallback to the default
BANNED_KEYWORDS array to ensure the filter logic remains effective.
Summary by CodeRabbit
新功能
重构
样式