Skip to content

fix(review): 允许宿主支持的本地预览地址 - #132

Merged
GaoWeiLiuXD merged 1 commit into
makecindy:mainfrom
GaoWeiLiuXD:fix/preview-loopback-review-rule
Sep 22, 2026
Merged

GaoWeiLiuXD merged 1 commit into
makecindy:mainfrom
GaoWeiLiuXD:fix/preview-loopback-review-rule

Conversation

@GaoWeiLiuXD

Copy link
Copy Markdown
Collaborator

What changed / 改了什么

仅修正 .greptile/config.json 的 network-allowlist 规则:允许 preview.hosts 声明宿主支持的 localhost、127.0.0.1、[::1],并通过宿主 preview 能力打开匹配的本地 URL;不再仅因回环地址或 HTTP 判为 P1。

Why / 为什么

TapTap Maker 等本地控制台依赖回环地址。原规则把这类宿主已有能力与插件自行访问内网混为一谈,造成误报。相关场景见 #131。

已对照宿主源码 GHOST_PREVIEW_LOOPBACK_HOSTS、GhostPreviewSlot.handleRequest 与 ghostPreviewUrlAllowed 调用链:清单需要声明预览范围,打开 URL 时仍由宿主校验。例外不扩展到 network.hosts、其他内网地址或插件自行直连。

Checklist

  • 已审查完整 diff,无凭证、真实用户数据或无关文件。
  • 每个 commit 均带 DCO sign-off。
  • 插件能力、工具声明、副作用处理、版本、实机验证、最低版本、提案、worker 构建与许可证项:不适用。本次没有修改任何插件打包内容。

Tool declaration changes / 工具声明改动

无。

Verification / 验证

  • JSON 解析与 git diff --check 通过。
  • 仓库提交门禁通过:plugin-contract、localization、provisioning、publish-workflows、binary-dependencies 五组检查(24 项 Node 测试)。
  • check-source-size.mjs 通过。
  • 没有改动插件包,不涉及实机安装验证或版本升级。

Sensitive changes / 敏感变更

本次修改 .greptile/ 审查策略,需要维护者人工 review;现有敏感变更路由和自动 approve 排除配置保持有效。例外仅对齐宿主已有本地 preview 能力,不修改运行时鉴权实现。

Signed-off-by: liugaowei <liugaowei@xd.com>
@greptile-apps

greptile-apps Bot commented Sep 22, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

未发现代码或规则边界缺陷;实现层面可以合并,但作为审查策略敏感变更仍须维护者人工 review。

Summary

敏感变更,需要维护者人工 review。

本 PR 仅调整 .greptile/config.json 中的网络审查策略:

  • 允许插件在 preview.hosts 中声明宿主支持的 localhost、127.0.0.1 和 [::1]。
  • 仅当匹配 URL 通过宿主 preview 能力打开时,避免因回环地址或 HTTP 本身误判为阻断问题。
  • 明确保留 network.hosts、其他内网地址及插件自行直连的原有限制。

网络边界核查覆盖了 .greptile/config.json、生成的 .db-rules.md、清单契约中的 GHOST_PREVIEW_LOOPBACK_HOSTS 与 preview.hosts/network.hosts 校验,以及现有预览声明和相关测试。未发现规则将例外扩展至普通联网、私有网段或直接请求的情况。该变更命中审查策略敏感路由,因此仍需维护者人工确认,但未发现需要行级评论的实现问题。

Reviews (1) · Last reviewed commit: "fix(review): 允许宿主支持的本地预览地址"

@GaoWeiLiuXD
GaoWeiLiuXD merged commit 9cb3bba into makecindy:main Sep 22, 2026
4 checks passed
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