Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

npm-shrinkwrap.json renovation failing #6811

Closed
dominykas opened this issue Jul 22, 2020 · 7 comments · Fixed by #6817
Closed

npm-shrinkwrap.json renovation failing #6811

dominykas opened this issue Jul 22, 2020 · 7 comments · Fixed by #6817
Assignees
Labels
auto:reproduction A minimal reproduction is necessary to proceed priority-1-critical A bad bug or work that is holding up a lot of other important features or fixes type:bug Bug fix of existing functionality

Comments

@dominykas
Copy link
Contributor

What Renovate type are you using?

Self hosted

Describe the bug

Lock file maintenance for npm-shrinkwrap.json does not get applied

Relevant debug logs

There is a lock file error log with the following stack trace:

TypeError: Cannot read property 'overwrite' of undefined
    at Object.move (/Users/dominykas/devel/projects/test/node_modules/fs-extra/lib/move/move.js:17:26)
    at /Users/dominykas/devel/projects/test/node_modules/universalify/index.js:8:12
    at new Promise (<anonymous>)
    at Object.move (/Users/dominykas/devel/projects/test/node_modules/universalify/index.js:7:14)
    at Object.move (/Users/dominykas/devel/projects/test/node_modules/renovate/dist/util/fs/proxies.js:69:15)
    at Object.generateLockFile (/Users/dominykas/devel/projects/test/node_modules/renovate/dist/manager/npm/post-update/npm.js:87:24)
    at async Object.getAdditionalFiles (/Users/dominykas/devel/projects/test/node_modules/renovate/dist/manager/npm/post-update/index.js:349:21)
    at async Object.processBranch (/Users/dominykas/devel/projects/test/node_modules/renovate/dist/workers/branch/index.js:231:33)
    at async Object.writeUpdates (/Users/dominykas/devel/projects/test/node_modules/renovate/dist/workers/repository/process/write.js:25:21)
    at async Object.update (/Users/dominykas/devel/projects/test/node_modules/renovate/dist/workers/repository/process/extract-update.js:95:15)
    at async Object.renovateRepository (/Users/dominykas/devel/projects/test/node_modules/renovate/dist/workers/repository/index.js:41:21)
    at async Object.start (/Users/dominykas/devel/projects/test/node_modules/renovate/dist/workers/global/index.js:76:13)
    at async /Users/dominykas/devel/projects/test/node_modules/renovate/dist/renovate.js:28:24

To Reproduce

  1. Have a repo with an npm-shrinkwrap.json
  2. Perform a renovation there

Additional context

I suspect #6617 / #6683 might have broken it?

@viceice viceice self-assigned this Jul 22, 2020
@viceice viceice added #internal type:bug Bug fix of existing functionality priority-1-critical A bad bug or work that is holding up a lot of other important features or fixes labels Jul 22, 2020
@rarkins rarkins added the auto:reproduction A minimal reproduction is necessary to proceed label Jul 22, 2020
@rarkins
Copy link
Collaborator

rarkins commented Jul 22, 2020

This likely needs a reproduction repo to be sure it's fixed. It would also be useful if you can pinpoint which Renovate version exactly it broke in

@dominykas
Copy link
Contributor Author

dominykas commented Jul 22, 2020

It works with 21.31.1 and does not work with 21.31.2.

Will see if I can set up a public repo.

@dominykas
Copy link
Contributor Author

Sorry, I think I left some typos in the versions there: it's 21.23.1 and 21.23.2.

@dominykas
Copy link
Contributor Author

And here's a repo for testing: https://github.com/dominykas/test-renovate-6811

@dominykas
Copy link
Contributor Author

@renovate-release
Copy link
Collaborator

🎉 This issue has been resolved in version 21.33.13 🎉

The release is available on:

Your semantic-release bot 📦🚀

@dominykas
Copy link
Contributor Author

Thanks

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto:reproduction A minimal reproduction is necessary to proceed priority-1-critical A bad bug or work that is holding up a lot of other important features or fixes type:bug Bug fix of existing functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants