Repository navigation
docs(dsh-plugin): refresh catalog install recovery guidance - #6000
huangruiteng merged 1 commit into
Conversation
5583da6 to
66cf6a9
Compare
loopx-agent
left a comment
There was a problem hiding this comment.
Reviewer: actor_kind=model_agent; model=gpt-6.1-sol; provider=OpenAI; declaration_source=runtime_reported; reasoning_effort=xhigh.
精确 head 66cf6a95362820041d60dc1c8a634c521f268dc8,不可变PR差异基线 647e216bacc0a69368dccc18fcb52dcba6002553;全量差异已核验。
动机
从 DSH Plugin Hub 安装或升级 LoopX 的用户,需要能按恢复指南拿到文档承诺的发行包。 原恢复段落说在线 catalog 仍选 beta.5;新段落说链接的在线安装页已经指向 beta.6,并让用户升级 Hub 后从那里安装。我实际打开该页,复制命令仍固定到 beta.5,所以这条新指引目前不能提供承诺的 beta.6。 已核实 beta.6 GitHub发行包存在、npm包查询404、Hub1.6.0发布说明包含相关修复;但同一真实catalog页面与新段落的版本承诺相矛盾,需改正文案后才能完成这次恢复指导更新。 本次只修改包README恢复段落,不修改安装器或运行权限,不关闭#5947的未诊断环境故障,不声称已完成Windows、真实Bot或安装/更新/卸载验收。
改动思路
保留已核实的发行包、npm状态与profile升级指引,把“源catalog已更新”和“在线入口实际已更新”分开;在在线页正确前将直接beta.6固定URL作为明确操作路径,不要求新增版本路由或控制面机制。 安装恢复信息应留在现有包README,并复用既有固定release URL与profile语义;这个小范围无需新开关、provider或状态owner。 当前PR边界是已有安装恢复指导的事实更新;在线入口仍beta.5这个事实必须诚实写出,真实环境故障继续由#5947的原诊断入口处理。
具体改动
关键代码讲解
这里的三个节点都是现有README段落,未增加运行函数;完整差异仅 packages/dsh-loopx-plugin/README.md,+18/-12。
- 第397–399行明确已发布beta.6与本地同版本构建的区别。我独立核实beta.6发行包及其固定asset。
- 第407–416行改为在线catalog beta.6及Hub1.6.0/profile升级指引。Hub1.6.0说明支持profile指引与相关修复;但catalog当前事实不成立,见下列阻塞。
- 第418–425行明确npm包未发布,提供直接beta.6 URL与原有source installer。实际公开npm精确查询E404;web/desktop替换仍是原有profile作用域,未授权修改账户或自动安装。
判断依据为修改前 packages/dsh-loopx-plugin/README.md,修订 647e216bacc0a69368dccc18fcb52dcba6002553,第 6 项 marketplace closeout:catalog合并或listing不等于实际消费固定发行包。这里只要求当前事实和操作路径诚实,不把完整运行安装验收变成文档修正的前置。
[P2,阻塞] 第407–412行:不要把catalog源更新等同链接页面已更新。 我在真实浏览器打开该精确链接,How to Install仍显示:
dsh plugin --profile web add https://github.com/loopx-project/loopx/releases/download/dsh-loopx-plugin-v0.1.1-beta.5/dsh-loopx-plugin-0.1.1-beta.5.tgzsource catalog PR6633已合并,当前YAML确实beta.6;两者是不同消费者。用户按新段落从链接安装页复制命令会选择beta.5,和“current entry points to beta.6”相矛盾。最小修复是在部署读回前明确该lag,并直接引导使用已核实的固定beta.6 URL;或等外部页真正刷新后读取该命令再作声明。只检查字符串存在/删除会漏掉这个反例。
非阻塞的同文一致性建议:第154–156行仍说Hub PR98需要合并/发布;官方已合并并由Hub1.5.0发布。更新恢复段落时可一并退掉这条旧结论。
对主干的风险
同一真实页面读回下,base明确beta.5的版本句符合实际,head beta.6句失败;独立版本比较保留该差异,没有把全文当等价。官方release、Hub98/1.5.0/1.6.0、source catalog与npm分别检查,避免用一处source成功替代用户入口。HTTP直读先遇403,后来通过真实Ego页面恢复并取得命令;没有把403记作成功。
diff check通过;首屏和recovery之前完整前缀字节不变,既有default-off、authority和版本兼容条款保留。包README属于分发文档,未改变自动加载skill、安装器或运行导出,因此无须为此docs改动补全产品frontend/机器状态测试。没有执行安装/更新/卸载、Windows或真实Bot路径;#5947的内部失败日志仍不可得,这些均未认证。没有查询或等待CI。
我的整体评价
REQUEST_CHANGES。正确的发行包/npm/profile信息有价值,范围和既有owner合适,但当前catalog版本承诺是可复现的文档回归,尚未完成用户可正确选择artifact的恢复结果。长程运行维度不适用:此diff不改持续执行、调度或历史状态;用户路径维度有回归:指引的真实入口仍选旧包。维护性检查建议直接修正文案及邻近旧Hub98结论,暂不增加抽象;不要求此PR修复#5947未知故障或认证整个安装系统。
English verdict: REQUEST_CHANGES - head 66cf6a9. The single README update correctly distinguishes published beta.6 from local builds, documents unpublished npm status and aligns Hub profile guidance with official releases. However, the exact linked live catalog still offers beta.5 in its How to Install command, contradicting lines 407-412 and the recommended route; merged source catalog beta.6 is not deployed-consumer evidence. Disclose lag/use the verified direct beta.6 URL or re-read the genuinely updated page. Existing activation/authority text and first viewport are unchanged; actual installation and #5947 diagnosis remain unverified. CI was not consulted.
Signed-off-by: mikamikasuki <211269698+mikamikasuki@users.noreply.github.com>
66cf6a9 to
dfc3c56
Compare
loopx-agent
left a comment
There was a problem hiding this comment.
Reviewer: model_agent · gpt-6.1-sol · OpenAI · runtime_reported · reasoning_effort=max
APPROVE。此前在线目录与新版本承诺不一致的问题已核实解除;当前完整改动能给出明确、可验证的 beta.6 发行包路径,未发现阻塞问题。
动机
从 DSH Plugin Hub 安装或升级 LoopX 插件、需要按恢复指南选择发行包的用户。
旧指南仍把已发布的 beta.6 当作本地构建,称目录仍是 beta.5,并推荐一个未发布的 npm 包名;用户照做可能拿错包或再次安装失败。
当前指南明确使用已核实的 beta.6 固定发行包,区分源码与发布产物,并让升级和安装命令使用用户选择的 profile。
本次只完成已有安装恢复文档的事实修正,不宣称已解决 #5947 的未知环境故障,也不替代真实安装、更新、卸载或 Windows 验收。
改动思路
恢复信息应继续放在现有包 README,复用已发布的固定 tarball 和 DSH 自己的 profile 选择;增加新的版本路由、同步状态或安装控制面都不能比直接正确指引更好地解决本问题。
当前 PR 边界是让用户按已有恢复指南选择正确的已发布产物,并保持来源更新与真实 marketplace 消费的区分;完整 marketplace 采用和 #5947 诊断仍由原发布/报告入口负责。
普通用户需要的结果是选到正确包,而不是先完成源码构建或等所有外部目录同步。指南给出固定发行 URL,并保留检查宿主版本、profile、原生结果读回及原有恢复步骤;没有增加一轮确认或让安装本身授予执行权限。网页部署、Hub 数据源和源码目录仍分别观察,不能用其中一处成功替代其余消费者。
具体改动
关键代码讲解(本次为可定位的文档段落)
完整复审 head dfc3c561e2a5a3ae80df8fa6c88c905327c2147b,实际差异基线 98709770d80e9c332f3c07d2b545226a6dd17521。全部改动只有 README 一文件 +25/-12;安装器、包配置、Driver、自动初始化及此前全部正文逐字节不变。旧 66cf6a9 评审到当前头的修复和主干带入内容分别核对,没有把旧判断直接沿用。
- Installation and upgrade recovery 区分已发布的 beta.6 资产与同版本本地产物。我重新下载了官方66354字节包,SHA-256 与 GitHub asset digest 一致,包内版本和编译入口正确;这证明资产身份,不证明已在用户宿主安装。
- source catalog / marketplace guidance 保留外部页面可能滞后的边界。我在真实 Ego 页面读回英文及中文安装命令,本轮两者都已指向 beta.6;目录源 PR6633也确已合并。旧评审时真实页面仍为 beta.5 的观察保持历史事实,不改写为当时通过。
- profile-specific Hub upgrade and pinned install 与 Hub1.6.0发布说明一致,并保留 desktop 替换规则;Hub1.5.0已包含 release 读回修复。本轮 npm 包查询仍404,因此直接固定包比未发布包名可操作。
规范依据:spec_ref packages/dsh-loopx-plugin/README.md,spec_revision 98709770d80e9c332f3c07d2b545226a6dd17521。criterion_id Installation and upgrade recovery:implemented,保留兼容宿主、选定 profile 和直接包恢复步骤。criterion_id Maintainer release and marketplace handoff:implemented,保留独立 catalog 和发行资产/源码的区分。criterion_id 6. Open the upstream directory pull request:deferred,仅指完整 marketplace 安装/更新/卸载资格,由已有发布 owner 继续;本 PR 没有声称完成它。两个 Hub API feed 的直接读取403,浏览器英文 feed 两次超时,中文 feed 成功但无 LoopX 行;这些限制保持明确,没有把网页命令读回升级成 Hub 端完整采用。
对主干的风险
这次不改自动加载 skill 或运行行为;首屏和恢复段落以前全部正文、现有 GoalBar opt-in、Driver 激活、权限及移除规则都保持原字节,因此无需为文档更新另加 frontend 或原生安装机制。最强反例是“源码目录已合并,但用户实际入口仍选旧包”;当前真实中英文页面已选 beta.6,而且新版固定 URL 也不依赖页面缓存,下载字节与公开 digest 匹配。仍没有在真实账户执行安装、升级、卸载,也没有复现 #5947 的内部日志故障。
差异、DCO与公共边界检查通过。原生 premerge 初次调用被本机 pnpm11 的非交互模块迁移错误阻断;保留原失败后,仅在评审环境使用仓库指定 pnpm10.33.0,最终4项直接检查与16项选中检查全部通过,包含插件质量、打包、profile 和本地 runtime smoke。没有修改源码或 lockfile来绕过失败,也未查询或等待CI。非阻塞建议:网页本轮已刷新,可日后把持续 redeploy 的措辞收为一般缓存提示;现有直接包路径仍正确,不构成交付阻塞。
我的整体评价
APPROVE。user_experience 改善:恢复任务有正确的固定资产路径,明确 profile,减少走到不存在 npm 包的准备和失败成本;long_horizon 在此文档切片不适用,调度和续跑状态未变。改动没有新增机制或兼容层,复用当前发布 owner,规模与实际旧指引问题相称。它完成文档修正,不关闭完整 marketplace 采用、#5947 或原生 Windows 资格,也不授予实际安装或合并权限。
English verdict: APPROVE — head dfc3c56. The complete README-only change now distinguishes published beta.6 from local builds, supplies the verified immutable package URL and preserves profile/authority boundaries. Current English and Chinese website commands select beta.6; source catalog, release bytes/digest, Hub notes and npm404 were checked separately. Pinned-pnpm native validation passes; actual installation, Hub-feed adoption and issue5947 diagnosis remain unqualified.
Goal and Delivered Outcome
The DSH plugin recovery guide still said the catalog selected beta.5, described release readback as needing another repair, and sent users to install
dsh-loopx-pluginfrom npm. The current public catalog points to the beta.6 GitHub release asset, DSH Plugin Hub v1.6.0 includes the relevant readback and desktop-install fixes, and the npm package is not published.This change updates the guide with the current catalog route and profile-specific Hub command, keeps the beta.6 tarball as a direct-install fallback, and distinguishes the published artifact from a local source build. It is related to #5947; the issue's specific wrapper failure remains undiagnosed because its inner install log is unavailable.
Author Declaration
Implemented against
Scope And Continuation
Validation
Frontend / Visual Evidence
Type of Change
LoopX Area
Technical Direction
Shared-authority RFC fixture impact
Boundary Checklist