Skip to content

fix(agents,skills): show sync interval in tooltip and preserve skill updated_at on download - #98

Closed
liuxy0551 wants to merge 1 commit into
masterfrom
fix_ued_92_93
Closed

liuxy0551 wants to merge 1 commit into
masterfrom
fix_ued_92_93

Conversation

@liuxy0551

@liuxy0551 liuxy0551 commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator

提交的变更类型是

  • 添加新功能
  • 修复 bug
  • 增加单元测试

需求描述和解决方案

描述

  1. UED-92【Doraemon】Agent Market 详情页自动更新时机:
    • 查看 Agent 默认从 GitLab 同步的定时间隔(默认配置为 1h,即 1 小时)。
    • “最近同步代码时间”的 hover 提示目前是“最近一次成功拉取新代码变动并完成同步入库的时间”,需在结尾加上“,定时间隔:1小时”。
  2. UED-93【Doraemon】Skills 详情页面点击下载 .zip 会更新 Skill 的最近更新时间:
    • 期望点击下载只进行下载数量的更新,不应触发“最近更新”的时间变化。

解决方案

  1. Agent Market 同步提示与文档更新 (UED-92):
    • 在 AgentDetailContent.tsx 中,“最近同步代码时间”的 Tooltip 提示更新为:最近一次成功拉取新代码变动并完成同步入库的时间,定时间隔:1小时。
    • 同步修正 docs/docsify/zh-cn/guide/agent-market.md 帮助文档中过时的 5 分钟描述,与当前代码及环境变量默认值 1h 保持一致。
  2. 下载 Skill 保持最近更新时间不变 (UED-93):
    • 排查发现 MySQL skills_items 表定义了 updated_at datetime NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP。
    • 原代码使用 Sequelize SkillsItem.increment('downloads', { silent: true }),silent: true 仅从 SET 子句省略了 updated_at,但在 MySQL 中执行 UPDATE 语句时,若未显式指定 updated_at = updated_at,会触发 MySQL 原生 ON UPDATE CURRENT_TIMESTAMP 从而被更新为当前时间。
    • 在 incrementDownloads 调用中显式传入 attributes: { updated_at: this.app.Sequelize.col('updated_at') },执行 UPDATE ... SET downloads = downloads + 1, updated_at = updated_at,保持原有更新时间不被重置。
  3. 补充单元测试:
    • 补充 test/agent-detail-layout.test.js 断言 hover 提示包含定时间隔 1 小时。
    • 补充 test/skills-registry-contract.test.js 断言 incrementDownloads 在自增下载量时传递了 silent: true 和 attributes.updated_at 保护字段。

自查清单

  • 代码遵循这个项目的风格指南
  • 对代码进行了自我审查
  • 已经在难以理解的代码处做了相关注释
  • 更改不会产生新的警告
  • 任何依赖更改都已合并并发布在下游模块中

…updated_at on download

- UED-92: Agent Market 详情页提示补充定时间隔(1小时),同步更新帮助文档
- UED-93: 下载 Skill ZIP 时显式保留 updated_at,防止 MySQL ON UPDATE CURRENT_TIMESTAMP 导致更新时间变动
@liuxy0551 liuxy0551 closed this Oct 28, 2026
@liuxy0551
liuxy0551 deleted the fix_ued_92_93 branch October 28, 2026 03:08
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.

1 participant