Skip to content

ci(publish): try to fix npm publish error - #416

Merged
inomdzhon merged 1 commit into
mainfrom
imirdzhamolov/ci/fix-oidc-publish
Nov 25, 2025
Merged

ci(publish): try to fix npm publish error#416
inomdzhon merged 1 commit into
mainfrom
imirdzhamolov/ci/fix-oidc-publish

Conversation

@inomdzhon

@inomdzhon inomdzhon commented Nov 25, 2025

Copy link
Copy Markdown

После #407 публикация падает с ошибкой:

npm error code E404
npm error 404 Not Found - PUT https://registry.npmjs.org/swc-plugin-pre-paths - Not found
npm error 404
npm error 404  'swc-plugin-pre-paths@7.1.0' is not in this registry.

В npm/cli#8730 (comment) указано, что возможно причина в версии NodeJS да и @actions/setup-node@v6 требует NodeJS >= 24.

Поэтому создал файл .nvmrc с нужной версией и заменил параметр node-version на node-version-file у @actions/setup-node. В будущем можно будет править версию через этот файл, а не через изменение воркфлоу.

Поправил по ходу дела:

  • @actions/setup-node рекомендует использовать contents: read
  • в package.json поправил repository.url, чтобы поправить предупреждение npm auto-corrected some errors in your package.json when publishing;
  • удалил provenance из package.json, т.к. больше не нужен, также оно автоматически включено при использовании Trusted Publishing;
  • заменил yarn.lock на package-lock.json – ни на что не влияет, но в вокрфлоу команды вызываются через NPM, а не Yarn.

После #407 публикация падает с ошибкой:

```
npm error code E404
npm error 404 Not Found - PUT https://registry.npmjs.org/swc-plugin-pre-paths - Not found
npm error 404
npm error 404  'swc-plugin-pre-paths@7.1.0' is not in this registry.
```

В npm/cli#8730 (comment) указано, что возможно причина в версии NodeJS да и `@actions/setup-node@v6` требует NodeJS >= 24.

Поэтому создал файл `.nvmrc` с нужной версией и заменил параметр `node-version` на `node-version-file` у `@actions/setup-node`. В будущем можно будет править версию через этот файл, а не через изменение воркфлоу.

Поправил по ходу дела:
- `@actions/setup-node` рекомендует использовать `contents: read`
- в `package.json` поправил `repository.url`, что поправить предупреждение ` npm auto-corrected some errors in your package.json when publishing`;
- удалил `provenance` из `package.json`, т.к. больше не нужен, также оно автоматически включено при использовании [Trusted Publishing](https://docs.npmjs.com/trusted-publishers#automatic-provenance-generation);
- заменил `yarn.lock` на `package-lock.json` – ни на что не влияет, но в вокрфлоу команды вызываются через NPM, а не Yarn.
@inomdzhon
inomdzhon requested review from a team as code owners November 25, 2025 13:12
@inomdzhon
inomdzhon merged commit 887c46e into main Nov 25, 2025
2 checks passed
@inomdzhon
inomdzhon deleted the imirdzhamolov/ci/fix-oidc-publish branch November 25, 2025 13:22
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.

4 participants