🐛 修复私聊无法使用道具

[HibiKier]

Summary by Sourcery

错误修复：

• 修复在私聊中无法使用props的问题，通过将user_id和group_id的类型从int更改为str或None。

Original summary in English

Summary by Sourcery

Bug Fixes:

• Fix issue where props could not be used in private chats by changing user_id and group_id types from int to str or None.

[sourcery-ai]

Sourcery 的审查指南

此拉取请求解决了无法在私人聊天中使用道具的相关错误。更改涉及修改两个文件中的类型注解，以便更灵活地处理用户和群组 ID，可能在私人聊天场景中启用道具的使用。

ShopParam 和相关函数的更新类图

classDiagram
    class ShopParam {
        +str|None goods_name
        +str|None user_id
        +str|None group_id
        +Any bot
    }
    class Goods {
        ...
    }
    class EventSession {
        ...
    }
    class UserConsole {
        +static get_user(id1, platform)
    }
    class SignUser {
        +static get_or_create()
    }
    ShopParam --> Goods
    EventSession --> UserConsole
    EventSession --> SignUser
    note for ShopParam "user_id 和 group_id 类型从 int 改为 str|None"

Loading

文件级更改

更改	详情	文件
修改 ShopParam 类中 user_id 和 group_id 的类型注解	将 user_id 类型从 int 改为 str	None 将 group_id 类型从 int 改为 str
更新签到相关函数中 user_id 的函数参数类型	在异步函数中将 user_id 参数类型从 int 改为 str	zhenxun/builtin_plugins/sign_in/goods_register.py

---

提示和命令

与 Sourcery 互动

• 触发新审查： 在拉取请求中评论 @sourcery-ai review。
• 继续讨论： 直接回复 Sourcery 的审查评论。
• 从审查评论生成 GitHub 问题： 通过回复审查评论请求 Sourcery 创建一个问题。
• 生成拉取请求标题： 在拉取请求标题的任何地方写 @sourcery-ai 以随时生成标题。
• 生成拉取请求摘要： 在拉取请求正文的任何地方写 @sourcery-ai summary 以随时生成 PR 摘要。您也可以使用此命令指定摘要应插入的位置。

自定义您的体验

访问您的仪表板以：

• 启用或禁用审查功能，例如 Sourcery 生成的拉取请求摘要、审查指南等。
• 更改审查语言。
• 添加、删除或编辑自定义审查说明。
• 调整其他审查设置。

获取帮助

• 联系我们的支持团队以获取问题或反馈。
• 访问我们的文档以获取详细指南和信息。
• 通过关注我们在 X/Twitter、LinkedIn 或 GitHub 上的 Sourcery 团队保持联系。

Original review guide in English

Reviewer's Guide by Sourcery

This pull request addresses a bug related to the inability to use props in private chats. The changes involve modifying type annotations in two files to allow for more flexible handling of user and group IDs, potentially enabling the use of props in private chat scenarios.

Updated class diagram for ShopParam and related functions

classDiagram
    class ShopParam {
        +str|None goods_name
        +str|None user_id
        +str|None group_id
        +Any bot
    }
    class Goods {
        ...
    }
    class EventSession {
        ...
    }
    class UserConsole {
        +static get_user(id1, platform)
    }
    class SignUser {
        +static get_or_create()
    }
    ShopParam --> Goods
    EventSession --> UserConsole
    EventSession --> SignUser
    note for ShopParam "user_id and group_id types changed from int to str|None"

Loading

File-Level Changes

Change	Details	Files
Modified type annotations for user_id and group_id in ShopParam class	Changed user_id type from int to str	None Changed group_id type from int to str
Updated function parameter type for user_id in sign-in related function	Changed user_id parameter type from int to str in the async function	zhenxun/builtin_plugins/sign_in/goods_register.py

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time. You can also use
  this command to specify where the summary should be inserted.

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[sourcery-ai]

嘿 @HibiKier - 我已经审查了你的更改 - 这里有一些反馈：

总体评论：

• 考虑检查整个代码库中关于 user_id 和 group_id 是整数的任何假设，并确保在整个项目中一致地将这些处理为 str | None 类型。添加注释以解释此更改背后的理由可能对未来的维护有帮助。

这是我在审查期间查看的内容

• 🟢 一般问题：一切看起来都很好
• 🟢 安全性：一切看起来都很好
• 🟢 测试：一切看起来都很好
• 🟢 复杂性：一切看起来都很好
• 🟢 文档：一切看起来都很好

---

Sourcery 对开源项目免费 - 如果你喜欢我们的评论，请考虑分享它们 ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{帮助我变得更有用！请点击每条评论上的 👍 或 👎，我将使用反馈来改进你的评论。}

Original comment in English

Hey @HibiKier - I've reviewed your changes - here's some feedback:

Overall Comments:

• Consider reviewing the entire codebase for any assumptions about user_id and group_id being integers, and ensure consistent handling of these as str | None types throughout the project. Adding comments to explain the rationale behind this change could be helpful for future maintenance.

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}