Skip to content

fix(taptap-maker): extend init silence timeout - #101

Open
Stella-xixi wants to merge 1 commit into
makecindy:mainfrom
Stella-xixi:codex/fix-taptap-maker-init-timeout
Open

fix(taptap-maker): extend init silence timeout#101
Stella-xixi wants to merge 1 commit into
makecindy:mainfrom
Stella-xixi:codex/fix-taptap-maker-init-timeout

Conversation

@Stella-xixi

@Stella-xixi Stella-xixi commented Sep 2, 2026

Copy link
Copy Markdown

What changed / 改了什么

  • 将 TapTap Maker 账号/初始化类长任务的沉默超时从 60 秒提高到 Cindy Host 允许的最大值 120 秒,保留 15 分钟绝对上限与续期机制。
  • maker_build 与动态 Maker 工具继续保持原有 60 秒沉默窗口,避免扩大无关行为。
  • 按当前官方插件仓要求将该插件从 Manifest v2 等价迁移到 v3,并将版本从 2.1.12 提升到 2.1.13。
  • 增加 maker_init 对可信工作目录、120 秒沉默窗口和 15 分钟总上限的回归断言。

Why / 为什么

关联 makecindy/cindy#3501

Windows 用户调用 maker_init 时会稳定在约 60 秒由 Host 返回 TIMEOUT。初始化在写入工作区前还需完成 Python、认证与项目发现等前置步骤,因此慢环境会表现为超时且工作区零落盘。本修复只扩大初始化链的合法沉默窗口,不修改 Cindy 通用上限。

Checklist

  • Reviewed the complete diff — no credentials, tokens, authorization codes, real user data, unrelated generated files, or node_modules.
  • ghost.json declares only capabilities actually used; the v3 migration preserves the existing card, Node, session context, picker, preview and tool capabilities.
  • Irreversible external-side-effect execution-state behavior is unchanged.
  • Tool descriptions, Manual and settings copy are unchanged and remain consistent.
  • Bumped ghost.json version from 2.1.12 to 2.1.13.
  • Production Cindy verification / 生产版 Cindy 验证 — installed the packaged plugin and exercised maker_init on Windows with stable Cindy v0.1.69 CN.
  • minCindyVersion remains 0.1.48; it was not lowered or removed.
  • Not a new plugin; provisioning audience is unchanged.
  • Ran localization and manifest contract checks.
  • No src/ or bundled dependency changes; rebuild and license updates are not applicable.
  • Every commit is signed off (DCO).

Tool declaration changes / 工具声明改动

无。仅调整内部 Node 请求等待窗口;Agent 可见工具名、参数、描述、副作用及返回语义均未改变。

Verification / 验证

  • node scripts/validate-plugin-manifest.mjs ./taptap-maker — passed
  • node --test .tests/taptap-maker.test.mjs .tests/plugin-contract.test.mjs .tests/localization.test.mjs .tests/provisioning.test.mjs — 48/48 passed
  • node --test .tests/publish-workflows.test.mjs — 6/6 passed(使用 Git for Windows Bash;本机无 jq,测试时使用只实现该用例输入转换的临时 jq shim,未进入提交)
  • git diff --check — passed

Production Windows evidence / Windows 正式版证据

  • Windows 10 x64(10.0.19045),Cindy v0.1.69 CN 正式稳定版。
  • 使用仓库正式打包脚本生成并安装 TapTap Maker 2.1.13 .cindy
  • maker_login 成功;Host 日志记录 totalMs: 105749,证明账号长任务可越过旧 60 秒窗口。
  • 账号最初无已有项目,因此无法执行已有 app_id 绑定路径;改用 create=true 新建脱敏测试项目完成核心初始化。
  • maker_init 成功;Host 日志记录 totalMs: 94756,超过旧 60 秒窗口且低于新 120 秒窗口。
  • 验证 .maker-mcp/config.json 已落盘且 project_id 与新建项目一致;Git checkout 为 main...origin/main
  • 初始化子进程在调用结束后退出,无对应孤儿 PID。
  • Runtime 报告 AI Dev Kit Skill 安装失败为非阻断警告;Git、Python、Lua LSP、Maker 授权与项目初始化均完成。

Sensitive changes / 敏感变更

  • Manifest v2 → v3 为等价表示迁移:保留既有能力,不新增权限、网络目标、凭证流、Node 入口或工具。
  • 无依赖、vendor、二进制、OAuth/API scope、provisioning audience、发布策略或 minCindyVersion 变更。

Signed-off-by: Stella-xixi <2559689615@qq.com>
@Stella-xixi
Stella-xixi marked this pull request as ready for review September 2, 2026 02:31
@greptile-apps

greptile-apps Bot commented Sep 2, 2026

Copy link
Copy Markdown

Greptile Summary

敏感变更,需要维护者人工 review;该发布候选尚未在 Windows 正式稳定版 Cindy 中安装最终 .cindy 并实测 maker_init。PR 将 TapTap Maker 账号及初始化类长任务的沉默超时提高到 120 秒,同时保留 15 分钟总上限,并完成等价的 Manifest v3 迁移与版本提升。

  • 将账号请求的长任务沉默窗口从 60 秒延长至 120 秒
  • 保留 maxTotalMs: 900000 的绝对上限
  • 将清单从 v2 迁移至 v3,并保留原有工具、Card、Node、会话上下文、目录选择和预览能力
  • 增加初始化请求超时、可信工作目录及总时限断言

Confidence Score: 4/5

该 PR 未发现阻断性代码缺陷,但合并发布前应完成并记录正式稳定版 Cindy 上的最终插件包实机验证。

超时范围和 Manifest v3 能力映射与变更目标一致,但当前发布候选尚未满足仓库要求的生产客户端验证流程。

Files Needing Attention: taptap-maker/ghost.json

Important Files Changed

Filename Overview
taptap-maker/main.js 将账号路径中所有标记为长任务的 Node 请求沉默超时提高至 120 秒,并维持原有 15 分钟总上限。
taptap-maker/ghost.json 版本提升至 2.1.13 并等价迁移到 Manifest v3;发布前仍缺少规则要求的正式稳定版 Cindy 实机验证。
.tests/taptap-maker.test.mjs 更新版本和 v3 能力断言,并新增 maker_init 的 120 秒沉默窗口与 15 分钟总上限回归检查。
Prompt To Fix All With AI
### Issue 1
taptap-maker/ghost.json:6
**缺少正式版实机验证**

本次发布修改了 `maker_init` 等账号操作的宿主请求超时并将插件版本提升至 2.1.13,但生产版 Cindy 验证项仍未完成,因此尚未按仓库要求确认最终 `.cindy` 包在 Windows 正式稳定版中可安装且修复后的核心流程可正常执行。

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Reviews (1): Last reviewed commit: "fix(taptap-maker): extend init silence t..." | Re-trigger Greptile

@nanaco666

Copy link
Copy Markdown
Collaborator

当前合并门禁未通过(head bf9d9d2

新 PR 已进入巡检,当前状态:

  • Greptile 已对当前 head 复审,Confidence Score 为 4/5(未达 5/5),并指出发布前需在正式稳定版 Cindy(Windows)完成最终插件包实机验证并记录证据;当前无 approving review,无维护者人工 review。
  • 未解决 review thread 为 0;DCO success。
  • Verify pull request 为 action_required:Repository-wide gates 与 changed-plugin packaging 尚未产生证据。
  • 本 PR 涉及 TapTap Maker 的 Manifest v2→v3 等价迁移与长任务超时调整(2.1.12→2.1.13),需维护者人工 review 确认迁移等价性与实机验证记录。

请补充正式版实机验证证据、批准并跑绿 Verify workflow,并完成维护者人工 review 后正式 Approve;完成前请勿合并。

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