feat(routes): add bfzy route - #96
Conversation
新增暴风资源路由
总览走查这个拉取请求引入了一系列针对暴风资源（BFZY）的新路由和实用工具。新增了多个路由文件，包括类别、详情、主页、主页视频、播放和搜索等。同时，对类别和搜索的实用工具进行了微调，将HTTP请求方法从POST更改为GET。这些变更旨在优化API交互和资源处理。 变更
序列图sequenceDiagram
participant Client
participant BFZYRoute
participant CMSHandler
participant APIServer
Client->>BFZYRoute: 发送请求（例如/category）
BFZYRoute->>CMSHandler: 调用处理器
CMSHandler->>APIServer: 发送GET请求
APIServer-->>CMSHandler: 返回数据
CMSHandler-->>BFZYRoute: 处理并转换数据
BFZYRoute-->>Client: 返回响应
诗歌
Tip CodeRabbit's docstrings feature is now available as part of our Early Access Program! Simply use the command Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
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 (4)
src/routes/bfzy/home.ts (1)
8-14: 路由定义规范且功能描述完善！
- 路径、名称、示例与描述等信息都能让其他开发者一目了然。
- 建议考虑在描述中补充数据返回格式或示例结果，便于前端或调用方理解。
src/routes/bfzy/homeVod.ts (1)
8-14: 路由功能与命名符合业务需求！
/homeVod用于显示最近更新列表，命名直观；- 建议在说明中添加更多使用场景或参数示例，帮助其他人使用该接口时能更快上手。
src/routes/bfzy/play.ts (2)
5-7: 播放逻辑与类型说明定义合理！在后续扩展时可考虑增加错误处理或对返回值的封装，以提高可维护性。
8-15: 路由信息完善，支持 POST 请求！
- 在描述上标注“获取播放地址”，交互方式明确。
- 建议在提交播放地址时，验证输入数据的格式与安全性，防止不必要的安全风险。
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (9)
src/routes/bfzy/category.ts(1 hunks)src/routes/bfzy/detail.ts(1 hunks)src/routes/bfzy/home.ts(1 hunks)src/routes/bfzy/homeVod.ts(1 hunks)src/routes/bfzy/namespace.ts(1 hunks)src/routes/bfzy/play.ts(1 hunks)src/routes/bfzy/search.ts(1 hunks)src/utils/cms/category/index.ts(1 hunks)src/utils/cms/search/index.ts(1 hunks)
🔇 Additional comments (11)
src/routes/bfzy/namespace.ts (2)
1-2: 导入类型定义的方式很清晰！
没有发现明显问题，保留现有写法即可。
3-7: 命名空间配置符合预期！
namespace 对象的属性命名简洁清晰，同时提供了必要的描述信息，可用于后续路由或其他逻辑的扩展。若需要支持多语言或描述更详细的字段，可以考虑在此处进行扩展。
src/routes/bfzy/home.ts (2)
1-4: 依赖导入正确且结构合理！
Context 来自 hono，namespace 来自同目录下的文件，符合项目常规引入方式。
5-7: 业务类型与处理函数的导入清晰！
引入 HomeRoute 与 handler 明确分工，保持模块化设计。
src/routes/bfzy/homeVod.ts (2)
1-4: 依赖导入正常！
同样无明显问题。引入 namespace 文件使路由信息更加易于维护。
5-7: 类型定义与处理函数引用正确！
HomeVodRoute 搭配 handler 使用，不存在类型冲突或不一致的情况。
src/routes/bfzy/play.ts (1)
1-4: 引入命名空间的方式简洁清晰！
继续保持这种分层与抽象，方便管理和复用。
src/routes/bfzy/detail.ts (1)
1-15: 建议确认路由方法与后端请求方式保持一致
当前路由的 method 为 POST，但需确认 handler 中对应的后端接口是否也使用 POST 接收参数。如果后端期望通过 GET 获取数据，建议进行统一，以避免数据接收方式不匹配的问题。
src/routes/bfzy/search.ts (1)
1-15: 注意 SearchRoute 的请求方法可能与后端不一致
从提交信息看，搜索功能在后端已切换为 GET，本地路由依然配置成 POST。请确保与后端通信方式一致，以免请求参数无法被正确解析。
src/utils/cms/category/index.ts (1)
19-19: 已将后端请求方式切换至 GET，需要同步路由配置
从此处可见对后端接口使用了 request.get，若路由仍定义为 POST，前端调用时必须额外处理 query 参数。请与路由保持一致，减少意外错误。
src/utils/cms/search/index.ts (1)
20-20: 请确认后端接口是否支持 GET 方式请求
切换为 GET 请求后，参数将通过查询字符串传递，应当验证后端服务是否正确解析该参数格式。同时建议关注潜在的 URL 长度限制或敏感信息泄露风险。
| import { Context } from 'hono'; | ||
|
|
||
| import { namespace } from './namespace'; | ||
|
|
||
| import { CategoryRoute } from '@/types'; | ||
| import { handler } from '@/utils/cms/category'; | ||
|
|
||
| export const route: CategoryRoute = { | ||
| path: '/category', | ||
| name: 'category', | ||
| example: '/bfzy/category', | ||
| description: `获取分类列表`, | ||
| handler: (ctx: Context) => handler(ctx, namespace), | ||
| method: 'POST' | ||
| }; |
There was a problem hiding this comment.
🛠️ Refactor suggestion
当前路由设置为 POST，但请求可能需要改为 GET
从修改记录来看，handler 调整为使用 GET 方式获取分类数据，而路由的 method 依然为 POST。这可能导致请求参数无法正常传递，建议统一为 GET 或者在后端保留 POST 兼容。
新增暴风资源路由
Summary by CodeRabbit
新功能
改进
基础设施