Skip to content
This repository was archived by the owner on Nov 4, 2026. It is now read-only.
This repository was archived by the owner on Nov 4, 2026. It is now read-only.

packages/runner/README.md §Features 的「Hot Reload - Automatic reload on schema changes」未按 loader 分路,?api= 那条不成立 #3620

Closed

Description

@yinlianghui

越界发现,记录于 #3604(改写同文件 §Development Workflow 第 1、3 步)期间通读全文时。未在该 PR 中顺手修改 —— #3604 的派发范围被明确锁死在 §Development Workflow 这一节,这是 §Features,属另一处改动面。

事实(对 origin/main 复核)

packages/runner/README.md 第 8 行,§Features 列表:

- **Hot Reload** - Automatic reload on schema changes

这与 #3604 处理掉的 §Development Workflow 第 3 步「Edit the schema - changes reload automatically」是同一句无条件宣称,只是换了个位置。它对两条元数据加载路线中的一条不成立:

  • src/app-data/ 路(LocalBundleLoader) —— JSON 经 import.meta.glob 进入 Vite 的 module graph(packages/runner/src/lib/MetadataLoader.ts:26-28),改动不需要重启 dev server,这一侧大体成立。
  • ?api= 路(NetworkLoader) —— 元数据由外部后端提供,fetch 发生在运行时(同文件 89、101 行),Vite 对该后端没有任何可见性,不存在任何 watch 或 reload 机制。改了后端返回的 JSON,页面不会自己更新。

?api= 是本文件 §Metadata Loading 一节(#3537 / PR #3581 补)正式记录的、与本地路并列的加载方式,不是边角情况;§Features 站在全文最前面,读者先读到它。

建议

按 #3604 对第 3 步的处理口径同样处理:限定而非删除。例如把这一条改成只承诺本地那条路,或者干脆让它指向 §Metadata Loading,由那一节把两条路的差别讲清楚。

顺带一并核:同节还有一处措辞值得看 —— 第 8 行说的是 schema「changes」,而真正被 watch 的是 src/app-data/ 下的 JSON 文件;该目录 git-ignored 且新检出下不存在(见 #3604 的正文),所以对一个刚 clone 的读者,这条 Feature 在两条路上都暂时无从触发。

同源:#3533(文档不许描述实现里不存在的能力)、#3604、#3576 / PR #3602。已搜开放 issue 与 PR(关键字 runner README / Hot Reload / Features,三仓),无重复;#3619 是 content/docs/utilities/runner.mdx 的 Features 一节,不同文件、不同断言。

Activity

  1. self-assigned this
    on Aug 7, 2026
  2. yinlianghui commented on Aug 7, 2026

    @yinlianghui
    ContributorAuthor

    认领(PM 派发)。


    Generated by Claude Code

  3. yinlianghui commented on Aug 7, 2026

    @yinlianghui
    ContributorAuthor

    验收 ACCEPT → PR #3634(session_01GTRjn8xBqp75dk7kFupVRt)。

    已核:单 bullet +3/−1,与 #3621 落地措辞逐字对齐(Vite's module graph 同页两处命中同句);门禁覆盖面如实标注(packages/** 不在 check-doc-links 扫描面,绿灯不构成背书 — 且这条结论是对新基线重读 SCAN_ROOTS 得出,不是照抄 #3621,预测纪律到位)。

    裁决记录:

    1. 标题一并改掉接受:「Hot Reload」一词即「更新自动出现在屏幕上」的断言本体,正是 docs(runner): README §Development Workflow 按两个 loader 的真相改写第 1、3 步 (#3604) #3621 拒绝无实测写下的那条;只改从句会把矛盾从跨节搬进 bullet 内部。围栏内扩大,与预测的出入已如实记。
    2. 不 install、不跑测试的判断成立(markdown 无代码路径,两个门禁零依赖)。
    3. 新发现 packages/runner/README.md 的插件清单两处写成开放集合(§Features 的「etc.」、§Pre-installed Plugins 的非插件条目),实际只预装 kanban + charts 两个 #3632(插件清单两处开放集合措辞,实际仅 kanban+charts)入池,观察级判断正确;⚠️ 同文件串行等本 PR 落 main。

    Generated by Claude Code

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

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions