Skip to content

fix: avoid false failures after npm tag deletion - #257

Open
XXPermanentXX wants to merge 1 commit into
mainfrom
fix/npm-tag-propagation
Open

XXPermanentXX wants to merge 1 commit into
mainfrom
fix/npm-tag-propagation

Conversation

@XXPermanentXX

Copy link
Copy Markdown
Collaborator

问题与修改

npm dist-tag rm 成功后,registry 的紧接着一次查询可能仍返回旧标签。本次首次试跑已成功删除 bailian-cli / release-test,但后置核验将旧数据误报为删除失败。

删除后最多查询 5 次,查询间隔为 1、2、4、8 秒。只重试读取,删除命令只执行一次;标签持续存在或查询失败仍报错。补充旧数据最终收敛和重试上限的回归测试,并记录 npm 页面已确认清理连接为 Valid、无需发布版本。

验证

  • 新回归用例在旧实现失败,修复后通过。
  • 发布工具测试:5 个文件,25/25 通过。
  • vp check:0 错误,3 个已有 warning。
  • 独立代码 review:无阻断问题。

本 PR 没有执行其他线上标签删除,也没有发布 npm 版本。

This branch has not been deployed

No deployments
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