Skip to content

Conversation

@wangl-cc
Copy link
Member

@wangl-cc wangl-cc commented Jan 4, 2026

Summary by Sourcery

Documentation:

  • 添加有关在 Windows 上通过 winget 安装、升级和卸载 maa-cli 的中文文档,包括可选的配置和数据文件清理(完全移除)。
Original summary in English

Summary by Sourcery

Documentation:

  • Add Chinese documentation for installing, upgrading, and uninstalling maa-cli on Windows via winget, including optional purge of config and data files.
Original summary in English

Summary by Sourcery

Documentation:

  • 添加有关在 Windows 上通过 winget 安装、升级和卸载 maa-cli 的中文文档,包括可选的配置和数据文件清理(完全移除)。
Original summary in English

Summary by Sourcery

Documentation:

  • Add Chinese documentation for installing, upgrading, and uninstalling maa-cli on Windows via winget, including optional purge of config and data files.

wangl-cc and others added 3 commits January 4, 2026 21:12
Co-Authored-By: JasonHuang79 <63983700+JasonHuang79@users.noreply.github.com>
Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey - 我在这里给出了一些整体性的反馈:

  • 对于 Windows 示例,建议使用更明确的包标识符(例如:winget install MaaAssistantArknights.maa-cli),以避免在 maa-cli 存在多个匹配项时产生歧义。
  • 请在文档中澄清 --purge 选项是否是与 winget uninstall 搭配使用的(例如:winget uninstall maa-cli --purge),这样用户就能明确知道如何执行完全卸载。
  • 由于这些命令面向的是 Windows Shell,建议将代码块的语言标记从 bash 改为 powershell(或省略不写),以更好地匹配 Windows 上的典型环境和工具链。
给 AI Agent 的提示词
Please address the comments from this code review:

## Overall Comments
- For the Windows examples, consider using the explicit package identifier (e.g., `winget install MaaAssistantArknights.maa-cli`) to avoid ambiguity if multiple matches exist for `maa-cli`.
- Clarify in the text whether the `--purge` option is intended to be used with `winget uninstall` (e.g., `winget uninstall maa-cli --purge`) so users know exactly how to perform a full removal.
- Since these commands target Windows shells, you might want to change the fenced code block language from `bash` to `powershell` (or omit it) to better match the typical environment and tooling on Windows.

Sourcery 对开源项目是免费的——如果你觉得我们的 Review 有帮助,欢迎分享 ✨
帮我变得更有用!请在每条评论上点击 👍 或 👎,我会根据你的反馈来改进后续的 Review。
Original comment in English

Hey - I've left some high level feedback:

  • For the Windows examples, consider using the explicit package identifier (e.g., winget install MaaAssistantArknights.maa-cli) to avoid ambiguity if multiple matches exist for maa-cli.
  • Clarify in the text whether the --purge option is intended to be used with winget uninstall (e.g., winget uninstall maa-cli --purge) so users know exactly how to perform a full removal.
  • Since these commands target Windows shells, you might want to change the fenced code block language from bash to powershell (or omit it) to better match the typical environment and tooling on Windows.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- For the Windows examples, consider using the explicit package identifier (e.g., `winget install MaaAssistantArknights.maa-cli`) to avoid ambiguity if multiple matches exist for `maa-cli`.
- Clarify in the text whether the `--purge` option is intended to be used with `winget uninstall` (e.g., `winget uninstall maa-cli --purge`) so users know exactly how to perform a full removal.
- Since these commands target Windows shells, you might want to change the fenced code block language from `bash` to `powershell` (or omit it) to better match the typical environment and tooling on Windows.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@wangl-cc wangl-cc merged commit d8b48cb into main Jan 4, 2026
4 checks passed
@wangl-cc wangl-cc deleted the docs/winget branch January 4, 2026 14:35
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