Repository navigation
shadcn-sync 的 registry 请求没有超时:连接被黑洞时 pnpm shadcn:check 会一直挂着(46 次串行请求) #5968
Description
观察类发现,来自 objectstack#5803 的实现过程(objectui PR #3496),不在该 issue 范围内,未在本 PR 中修。
现象
objectui/scripts/shadcn-sync.js 的 fetchUrl 用 https.get 发起请求,只监听 error 与 end,没有设置任何超时(既没有 req.setTimeout,也没有 socket/连接超时)。
- 连接被丢包黑洞掉(SYN 无响应),或 TCP 已建立但服务端永不返回,请求就一直挂着。
--check是 46 次串行请求,任何一次挂住,整条命令就停在那里。.github/workflows/shadcn-check.yml里跑的正是这条命令,挂住会一路耗到 job 超时才失败,日志上看不出原因。
PR #3496 之后,基于响应的失败(非 2xx、非 JSON)都会立刻 reject 并带上原因;没有响应是这条取数路径上剩下的唯一一种静默失败。
现状影响
今天没人踩到:沙箱/CI 的 egress 拦截是立刻返回 403,不是黑洞。因此打 finding 标签、不进 pm:queue,由 PM 定级。
修的话需要决定的事
- 超时值(单次请求 10s?整体预算?)。
- 超时后是重试一次还是直接算 fetch 失败(现在的错误路径已经能优雅承接:
--check标 error 且退出码不变,--update拒绝写文件)。
req.setTimeout(ms, () => req.destroy(new Error(...))) 一行即可,但值和重试策略是产品决定,所以单独开。
备注
按 objectstack#4949 的纪律先搜了开放 issue(shadcn-sync timeout、shadcn registry fetch timeout hang)均 0 命中;更宽的 shadcn-sync 查询因共享账号触发 GitHub API 限流没能跑完,如与既有 issue 重复请直接关掉本条。
Activity
claude commentedon Aug 6, 2026
发现分诊轮(objectstack
finding存量 43 > 15,插队执行):晋级(摘finding换pm:queue)+ 路由repo:objectui。origin/main实测):scripts/shadcn-sync.js:139的fetchUrl用https.get,全文件git grep -n "setTimeout" -- scripts/shadcn-sync.js只命中:862的退避sleep,请求路径上确无任何超时。前提成立。objectui 仓内落点 ⇒ 只打repo:objectui,不打domain:*。req.setTimeout(ms, () => req.destroy(...)))、错误路径已由 PR objectui#3496 铺好(--check标 error、--update拒写),验收可一句话说清。正文的两个「待决定」是技术默认值而非产品语义,按分诊规则属技术判断,不升维护者:建议 单请求 10s、不重试(与既有失败语义同向,46 次串行的整体预算因此有上界),dev 有实测反对可在单上改。pm:dispatched,PR objectui#3496)、objectui: shadcn-check.yml 用 continue-on-error 吞掉 shadcn:check 的退出码,该周期任务无法对任何失败告警 #5805(continue-on-error吞退出码,pm:dispatched,PR objectui#3497)均在飞;本条是同一取数路径上剩下的最后一种静默失败(无响应),既有单未覆盖,不重复。正文作者因限流未跑完查重、请求「若重复直接关」—— 本轮以三仓全文比对复核过:不重复。scripts/shadcn-sync.js与在飞 PR objectui#3496 同文件。objectui 座位派发前须确认 feat(spec)!: prune dead aria/performance from ReportSchema + dogfood the live report chart (report-liveness close-out) #3496 已合,否则跨轮串行,⛔ 不要与 objectui: shadcn-sync 的 registry 缓存会把 403/HTML 错误响应当成正常数据缓存 1 小时,一次网络抖动毒化后续所有 --check #5803 同批。本评论来自分诊座位 Routine(#5474 试点),不构成认领。
Generated by Claude Code
os-zhuang commentedon Aug 10, 2026
Migrated to objectstack-ai/objectui#4031 under #7167 (file-at-destination ruling, maintainer 2026-08-10). Native GitHub issue transfer is not available to this session's credential, so the card was recreated at the destination; this thread stays as the authoritative history and is linked from the new card. Closing as not planned here — moved, not rejected.
Generated by Claude Code