fix(mobile-mcp): align CD workflow npm auth with release.yml - #6257
Conversation
…pe, drop provenance)
qwen-code-ci-bot
left a comment
There was a problem hiding this comment.
Thanks for the PR @LaZzyMan! 👋
The PR body is missing the required headings from our PR template. The current body uses ## Summary and ## Test plan, but the template expects:
## What this PR does## Why it's needed## Reviewer Test Plan(with### How to verify,### Evidence (Before & After),### Tested on)## Risk & Scope## Linked Issues
Could you update the description to follow the template? It helps reviewers (and future-you) understand the motivation, scope, and how to verify the change. The actual content you've written is good — just needs to be mapped into the right sections.
The code change itself looks straightforward and well-motivated — happy to dig into it once the template is sorted. 🙏
中文说明
感谢 @LaZzyMan 的贡献!👋
PR 描述缺少 PR 模板 中要求的标题。目前使用了 ## Summary 和 ## Test plan,但模板要求的是:
## What this PR does## Why it's needed## Reviewer Test Plan(含### How to verify、### Evidence (Before & After)、### Tested on)## Risk & Scope## Linked Issues
请按模板格式重新组织描述内容。你写的实际内容很好,只需要映射到对应的章节即可。
代码改动本身很清晰,动机也合理——模板整理好后我会继续审查。🙏
— Qwen Code · qwen3.7-max
Code Coverage Summary
CLI Package - Full Text ReportCore Package - Full Text ReportFor detailed HTML reports, please see the 'coverage-reports-22.x-ubuntu-latest' artifact from the main CI run. |
Summary
Fix
npm publishENEEDAUTH failure in cd-mobile-mcp.yml by aligning with the working release.yml configuration:scope: '@qwen-code'to setup-node (was missing → npm couldn't auth for scoped package)--provenancefrom publish (release.yml doesn't use it, and it requiresid-token: writepermissions)contents: 'read'only (matching release.yml).nvmrcfor node version + npm cache (matching release.yml)Test plan