Skip to content

task: OBS — 解散 observability 过渡包并归位记录与展示 #413

Closed

Description

@Dong1017

已确认实施方案:用户于 2026-11-12 11:35–11:36(北京时间)确认综合推荐,并要求快速、干净切换。

合入后 overall isolation_complete 必须仍为 false。

关联架构总题 #387、Apps 设计 #371;#389 是当前清单与 observability 来源,#398 为已合入 H01,#397 保持纯 Apps 标准输出且只读不扩范围。

Activity

  1. added
    kind:taskBounded delivery task with a delegated slice
    area:usabilityUsability and collaboration: issue navigation, templates, readable reports
    on Nov 12, 2026
  2. amy-why-3459 commented on Nov 12, 2026

    @amy-why-3459
    Collaborator

    谢谢,这个包确实不在 #387 的目标图里,拆回最终归属是必要的。我核对了当前 main(e2f4cd1e),补充几点:

    1. 要删除的 K 边已确认存在。 src/inferrsi/observability/knowledge_capture.py 第 20 行是 from .._skill_selection import TASK_RELEVANCE,第 74 行使用。按正文"所需取值由 Apps 注入"的做法,留在 H 的记录部分不能再拿到 TASK_RELEVANCE,但目前写入 knowledge capture 的 relevance_method 取决于它。请在 PR 里说明注入点(Apps 在调用时传入),并用测试锁定迁移前后 capture 记录的这个字段不变。
    2. 前置 Publish standardized Apps run output #397 仍是 Draft,task: Apps 应用层详细设计与迁移计划(CLI / Python / Plugin) #371 仍是 status:proposed,main 上还没有 apps/。 Dashboard、时间线和模板要进 apps/views,因此依赖这两项。请保持为显式前置。
    3. "固定记录下 CLI / 离线 Dashboard / live / trace 一致",建议用一组固定的 campaign 夹具,生成迁移前后的离线 HTML 和 snapshot JSON,比对字节(或规范化后的 JSON)。PR Compare stage timings and inspect knowledge searches for each profile #401 刚加了不少 dashboard 渲染测试(tests/check_profile_review.cjs 等),可以直接复用作回归基线。
    4. dashboard_workflow.py / profile_timeline.py 需要先补 D008 owner,这一步建议在 PR 描述里单列,先过 owner 评审再搬,避免和搬家混在一起审。
  3. Dong1017 commented on Nov 15, 2026

    @Dong1017
    CollaboratorAuthor

    @amy-why-3459 @zengchuang-hw — under the delegated architecture-coordination role, I adopt the joint five-entry D008 record for these two governing issues:

    1. Benchmark owns sampling order, timing and artifact hashes; transport stays caller-injected.
    2. Harness owns the required snapshot port; Apps binds the infra implementation, with no Harness concrete default.
    3. Apps owns workflow display projections.
    4. Harness owns factual profile records, bounded single-trace overview and integrity/digest reads; Apps owns UI and live-route decoration.
    5. Apps owns pre-profile search display projections.

    B01 delivers its implementation plus this shared decision metadata; OBS retains its implementation. This records architecture adoption, not independent implementation acceptance. Both issues retain their evidence requirements; isolation_complete remains false.

  4. Dong1017 commented on Nov 15, 2026

    @Dong1017
    CollaboratorAuthor

    Closing the bounded OBS migration slice delivered by PR #425, merged as 36440109.

    Harness owns factual records, Apps owns display/live decoration, and Knowledge owns pure projection under the D008 adoption. The joint architecture decision preserves the historical six-key layout without retrieval for readers supplying neither observation nor provenance, and retains the extended path otherwise. Format retirement requires a separate contract upgrade.

    Evidence includes final-head hosted checks and 362 passing exact-head Linux tests. Windows main-control evidence remains inconclusive. Overall isolation_complete remains false.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    area:usabilityUsability and collaboration: issue navigation, templates, readable reportskind:taskBounded delivery task with a delegated slice

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions