Skip to content

docs(observability): 明确持久 Job 事实与诊断查询窗口 - #57

Merged
xiaoland merged 2 commits into
mainfrom
hotfix/job-observation-history
Nov 5, 2026
Merged

xiaoland merged 2 commits into
mainfrom
hotfix/job-observation-history

Conversation

@xiaoland

@xiaoland xiaoland commented Nov 5, 2026 •

Copy link
Copy Markdown
Member

原因与范围

Dogfood 发现两起独立故障。一条已保存的提交诊断比 Job 创建时间早 82ms,被详情页查询下界裁掉。另一次提交已持久化,但成功关闭弹窗时出现放弃参数确认,阻塞期间异步提交诊断超时。相关事实与复现已记录在可观测性跟踪项。关联 #52。

这次补充产品行为和跨 Peer 合同,明确实现与验收所依赖的语义。

  • 已保存的 Job 提供提交事实、当前状态及创建、开始和结束时间,提交方退出后其他 Peer 仍可读取。
  • 诊断列表和摘要使用后端实际返回的记录。采集失败时,业务事实继续由 Job 提供。
  • 默认时间窗是可调整的搜索起点,需要考虑生产者时钟差,并支持过去的绝对窗口。
  • 31 天是单次查询宽度。长任务的当前窗口、返回数量上限及存储保留期分别表达,缺项原因需要证据支持。

验证与影响

检查两份文档的 7 个相对链接、代码围栏和 git diff --check,全部通过。语义复用了既有 Job 持久化及有界观测合同。此次本地验证限于文档,真实旅程由指定 Dogfood 聊天负责。

仅修改共享 PRD 与 Product TDD。Web 默认窗口余量、交互与回归由 impl-ts 拥有。回滚可撤销本提交,并协调消费方恢复共享引用。

Stack 与交付

直接父 PR 为 #56,父层合并后已重排到 main 并完成云端门禁。当前已知同仓子 PR 为零。配套实现为 InKCre/impl-ts#189,直接父 PR 为 InKCre/impl-ts#188。实现以独立提交引用已推送的 Hub 5c5c254,正式候选已交回指定 Dogfood 聊天复验。

本 PR 已按用户授权完成主干重排、云端门禁和普通 squash 合并。正式 Web f584867 搭配本合同 5c5c254 的四组 Dogfood 复验已通过,覆盖提交端硬停后的跨 Peer 事实读取、真实历史与窗口、弹窗交互。正式复验回执。

2026-11-05 合并交付已完成。Hub 四层按 #51 → #53 → #56 → #57 合入 main,最终 17fc19e 与已通过 Dogfood 的 5c5c254 整树一致,最终文档站正式发布成功。对应运行 42005096027。

- 明确提交已保存的事实由持久 Job 提供,诊断摘要使用实际返回记录
- 说明生产者时钟偏差、可调整历史窗口及单次查询宽度语义
- 规定长任务窗口和诊断缺项的展示要求
@xiaoland
xiaoland force-pushed the hotfix/job-observation-history branch from 5c5c254 to e3e3e9d Compare November 5, 2026 09:44
@xiaoland
xiaoland changed the base branch from feat/app-peer-naming to main November 5, 2026 09:44
@xiaoland
xiaoland marked this pull request as draft November 5, 2026 09:44
@xiaoland
xiaoland marked this pull request as ready for review November 5, 2026 09:44
@xiaoland
xiaoland merged commit 17fc19e into main Nov 5, 2026
1 of 2 checks passed
@github-project-automation github-project-automation Bot moved this from Review / acceptance to Done in InKCre Current Work Nov 5, 2026

This branch was successfully deployed

1 active deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant