Merged
Conversation
fix2:因为win32前台的DXGI_DesktopDup_Window方式有bug,将screencap参数改成PrintWindow
…feat/AutoEcoFarm
Contributor
There was a problem hiding this comment.
Hey - 我在这里给出了一些整体性的反馈:
- 请确认新的
AutoEcoFarmPrepare.json流水线已经正确接入到AutoEcoFarm.json以及所有相关的任务入口点中,这样准备步骤(清空库存)就能始终在主农场逻辑之前按预期顺序执行。 - 由于你在多个本地化文件中添加或修改了任务描述,请再次检查各语言之间的 key 和占位符是否一致,避免在某些语言环境中出现回退字符串或不匹配的文案。
- 对于
AutoEcoFarmSwipeToTarget.json中修复的滑动 / 视角旋转问题,建议显式记录或约束预期的起始屏幕中心参考(例如使用归一化坐标),以避免在未来相机或分辨率处理发生变化时再次引入回归问题。
提供给 AI 代理的提示
Please address the comments from this code review:
## Overall Comments
- Verify that the new `AutoEcoFarmPrepare.json` pipeline is properly wired into `AutoEcoFarm.json` and any relevant task entry points so the preparation step (clearing inventory) always runs in the intended order before the main farm logic.
- Since you added new/changed task descriptions across multiple locale files, double‑check that the keys and placeholders are consistent between languages so that the UI does not show fallback or mismatched strings in some locales.
- For the swipe/turn‑view bug fix in `AutoEcoFarmSwipeToTarget.json`, consider explicitly documenting or constraining the expected starting screen center reference (e.g., normalized coordinates) to avoid future regressions when camera or resolution handling changes.帮我变得更有用!请在每条评论上点 👍 或 👎,我会根据你的反馈持续改进评审质量。
Original comment in English
Hey - I've left some high level feedback:
- Verify that the new
AutoEcoFarmPrepare.jsonpipeline is properly wired intoAutoEcoFarm.jsonand any relevant task entry points so the preparation step (clearing inventory) always runs in the intended order before the main farm logic. - Since you added new/changed task descriptions across multiple locale files, double‑check that the keys and placeholders are consistent between languages so that the UI does not show fallback or mismatched strings in some locales.
- For the swipe/turn‑view bug fix in
AutoEcoFarmSwipeToTarget.json, consider explicitly documenting or constraining the expected starting screen center reference (e.g., normalized coordinates) to avoid future regressions when camera or resolution handling changes.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Verify that the new `AutoEcoFarmPrepare.json` pipeline is properly wired into `AutoEcoFarm.json` and any relevant task entry points so the preparation step (clearing inventory) always runs in the intended order before the main farm logic.
- Since you added new/changed task descriptions across multiple locale files, double‑check that the keys and placeholders are consistent between languages so that the UI does not show fallback or mismatched strings in some locales.
- For the swipe/turn‑view bug fix in `AutoEcoFarmSwipeToTarget.json`, consider explicitly documenting or constraining the expected starting screen center reference (e.g., normalized coordinates) to avoid future regressions when camera or resolution handling changes.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
MistEO
reviewed
Mar 7, 2026
| "_AutoEcoFarmPrepareStart": { | ||
| "doc": "开始任务前的准备工作", | ||
| "next": [ | ||
| "_AutoEcoFarmPrepareClearBagStart", |
Contributor
There was a problem hiding this comment.
格式化一下,可以 pnpm format 或者用 vscode。还有其他几个文件也是
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
新功能:
1、增加开始任务前清理背包的功能,进入帝江号打开仓库然后点击一键存放,将包里的杂物放入仓库
修改内容:
1、优化任务描述,能够更清晰地提示用户配置队伍、布置农场,增加任务成功率
bug修复:
1、修复了大幅度转视角节点的起点不在屏幕中间的问题
Summary by Sourcery
在修复大幅度相机旋转导致的视口问题的同时,优化 AutoEcoFarm 任务流程和引导体验。
新功能:
错误修复:
增强改进:
Original summary in English
Summary by Sourcery
Optimize the AutoEcoFarm task flow and guidance while fixing viewport issues for large camera rotations.
New Features:
Bug Fixes:
Enhancements: