Dev tool

[LGG686]

@AzurTian 开发的非常好用的工具
部署后浏览器输入url:port/tool/annotator访问

[sourcery-ai]

Sorry @LGG686, your pull request is larger than the review limit of 150000 diff characters

[github-actions]

Findings

• C1 [风险]
  位置:module/server/tool_router.py;module/server/web/annotator/**;module/server/script_router.py;module/config/utils.py
  原因:同一 PR 同时引入新标注工具、WebSocket 行为调整和配置日志改造,变更目标不够收敛,且超出标题“Dev tool”可直接表达的范围。
  修改:将“标注工具新增”与“现有脚本通信/配置日志改动”拆分为独立 PR,并分别补充行为变化与回归验证说明。

Warning

⚠️ Firewall blocked 4 domains

The following domains were blocked by the firewall during workflow execution:

• ab.chatgpt.com
• api.github.com
• chatgpt.com
• github.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "ab.chatgpt.com"
    - "api.github.com"
    - "chatgpt.com"
    - "github.com"

See Network Configuration for more information.

Generated by PR Review Checklist for issue #1606 · ◷

[github-actions]

Change Summary

• 这次 PR 实际新增了一整套 /tool/annotator Web 标注工具链:后端新增 tool_router + AnnotatorManager,前端新增独立页面、组件化 HTML/CSS/JS 与静态资源挂载。
• 标注工具能力覆盖会话管理、图片上传/删除、模拟器取帧、规则加载/保存、规则测试、裁剪模板图、以及保存后触发 AssetsExtractor 生成资产代码。
• 规则编辑从“前端硬编码字段”转向“后端 schema 驱动”(annotator_rule_schema.py),并统一了 image/ocr/click/swipe/long_click/list 的默认值与字段约束。
• 同时对原有脚本 websocket 管理做了稳健性改动(单连接发送、断连容错、异常捕获),以及配置读写日志从 print 切为 logger。

Reconstructed Intent

点击此处展开

- 这次改动最可能是为了把原先分散/手工的规则标注流程收敛成一个可视化开发工具,并直接接入现有 `tasks/*` 规则文件与 `assets.py` 生成链路。 - 次级 intent 看起来是提升调试闭环效率:在同一 UI 内完成“取图 -> 画 ROI -> 测试匹配 -> 保存规则 -> 生成资产”。 - 对 websocket 与日志的改动更像是为该工具运行过程提供更稳定的连接行为和更可控的日志输出,而非业务功能变更本身。

Observed Constraints

点击此处展开

- 路径安全强依赖:所有写入都要求在 `tasks/` 与会话目录内,且 `json_relpath` 必须相对路径、后缀必须 `.json`。 - 数据模型约束强依赖:`rule_type`/`mode`/`direction`/`type` 必须命中 schema 选项;ROI 必须是 `x,y,w,h` 且宽高>0。 - 会话模型有隐含边界:`create_session` 会替换并清理其他 active session 与目录,工具语义更接近“单活会话”。 - 资产生成是“保存后的副作用”:规则保存成功不代表资产生成一定成功(返回 `partial_success`),发布侧需接受“规则已写入但资产未更新”的中间态。 - 前端运行依赖新增静态挂载 `/tool/annotator/static` 与按顺序加载 `layout-ui.js -> app.js`。

Intent Alignment

• 结论:实现与推断 intent 基本一致,并且覆盖了从 UI 到后端再到资产生成的主要闭环。
• 但同时引入了额外行为变化:标注工具采用单活会话替换策略,且规则保存与资产生成解耦为“可部分成功”,这两点是显式行为语义变化。

Release Risk

• 风险等级:中
• 单活会话策略可能导致多人/多标签页并发使用时互相清理会话目录,出现“会话失效/图片丢失”的体感回归。
• save_rules_and_generate 的部分成功路径如果未被前端/操作者正确处理,可能把“规则文件与 assets.py 不一致”带入后续运行。
• 该 PR 体量大(新增前后端大量代码),但未在变更中看到对应链路级验证证据;一旦某个 API/前端状态机分支异常,定位成本较高。

Validation Gaps

点击此处展开

- 未看到针对 `/tool/annotator` 关键 API 的系统化验证证据(尤其是 session 生命周期、异常码、路径校验失败分支)。 - 未看到“保存成功但资产生成失败(partial_success)”在前端交互层的明确回归验证(是否能阻断发布或给出可执行修复路径)。 - 未看到多会话/多页面并发场景验证:创建新会话是否会意外清理正在工作的旧会话。 - 未看到对 `list` 类型规则(`list_meta` + item 列表)完整 round-trip(加载->编辑->测试->保存->再加载)验证记录。

Warning

⚠️ Firewall blocked 4 domains

The following domains were blocked by the firewall during workflow execution:

• ab.chatgpt.com
• api.github.com
• chatgpt.com
• github.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "ab.chatgpt.com"
    - "api.github.com"
    - "chatgpt.com"
    - "github.com"

See Network Configuration for more information.

Generated by PR Review Intent for issue #1606 · ◷

[LGG686]

还有这个，非常好用👍🏻
https://github.com/AzurTian/OASX/releases/tag/v0.3.9

[runhey]

可以，明天有空看看