Repository navigation
feat(webfetch): support reusable seven-family native comparisons - #42
Conversation
1a7f2e3 to
da3f01b
Compare
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
lanyue-llk
left a comment
There was a problem hiding this comment.
审查结论:建议修正下列问题后再合入。
本 PR 将 Web Fetch 扩展为五浏览器、七类任务、每浏览器 430 题,并统一原生接口、正文评分和发布校验。共享任务投影与统一路由选择器是合适的抽象;保留动态任务分类也合理,但调用校验没有同步迁移。资源投影还把单个 URL 当成 srcset 列表解析,导致合法资源地址损坏。
确认两处功能回退,均做了基线对照:目标分支通过,PR 失败;在隔离副本中验证最小修复后,相关测试 63 项通过。现有相关测试分两组共 478 项通过、13 项跳过;启用四种真实本地浏览器后,资源、取消和文件测试 24 项通过、1 项失败,失败原因见第三条。进程清理失败后的隔离行为符合现有安全合同,不列为缺陷。
这次审查覆盖代码、调用合同及相关测试,不改变报告中独立逐结果复核 0/2,150 的范围。
|
Claude encountered an error after 3s —— View job I'll analyze this and get back to you. |
7607b18 to
2166c94
Compare
cd0b669 to
0598c6a
Compare
85f8524 to
caa269e
Compare
This reverts commit 8dd0a1a.
06b86f4 to
b61c880
Compare
The Web Fetch harness could not fairly run and explain a browser-native comparison across seven task families. Native Markdown, rendered HTML, embedded resources, direct files, retries, scoring, and failure review were handled by separate paths, and the first implementation also stored one report's profile, rerun scope, release prose, and participant identities in Git.
This change makes those boundaries reusable:
report-assetswhen supplied. The root release registry is a legacy fallback, and the shared pricing catalog contains product facts rather than one run's wording.The published report remains on Artifact Site, with only a compact version/profile-hash archive binding in Git: https://artifact-site.lexmount.cn/v/IqGpMUYHWat-dw4n8usQttTptoPzboAo
Reproduce
The external profile declares participants, case inventory, run policy, executable identities, and report output. The assembly rejects a missing profile, stale source hash, mismatched case ownership, unregistered correction control, or report-local input embedded in the reusable authority.
Validation
git diff --checkpass.