Fix workspace packages not being found when they are moved

zackradisic commented 1 week ago

What does this PR do?

Fixes #10833

It looks like when we were diffing the lockfile from disk and parsed lockfile we did not update with the new workspace_paths from the newly parsed one

github-actions[bot] commented 1 week ago

❌ @zackradisic, your commit has failing tests :(

💪 3 failing tests Darwin AARCH64

test/js/bun/http/serve.test.ts 2 failing
test/js/bun/shell/commands/rm.test.ts 1 failing
test/js/node/http2/node-http2.test.js 1 failing

💻 3 failing tests Darwin x64 baseline

test/cli/install/bun-create.test.ts 2 failing
test/js/bun/http/serve.test.ts 1 failing
test/js/bun/test/test-test.test.ts 1 failing

💻 2 failing tests Darwin x64

test/cli/install/bun-create.test.ts 2 failing
test/js/bun/http/serve.test.ts 1 failing

🐧💪 1 failing tests Linux AARCH64

test/js/bun/shell/leak.test.ts 1 failing

🪟💻 7 failing tests Windows x64 baseline

test/cli/install/bunx.test.ts 1 failing
test/integration/next-pages/test/dev-server-ssr-100.test.ts 1 failing
test/integration/next-pages/test/dev-server.test.ts 1 failing
test/integration/next-pages/test/next-build.test.ts 1 failing
test/js/bun/shell/bunshell.test.ts 1 failing
test/js/bun/spawn/spawn.test.ts 1 failing
test/js/node/dns/node-dns.test.js 2 failing

🪟💻 9 failing tests Windows x64

test/cli/install/bunx.test.ts 1 failing
test/integration/next-pages/test/dev-server-ssr-100.test.ts 1 failing
test/integration/next-pages/test/dev-server.test.ts 1 failing
test/integration/next-pages/test/next-build.test.ts 1 failing
test/js/bun/http/serve-body-leak.test.ts 1 failing
test/js/bun/http/serve.test.ts 10 failing
test/js/bun/shell/bunshell.test.ts 1 failing
test/js/bun/spawn/spawn.test.ts 1 failing
test/js/node/dns/node-dns.test.js 2 failing

View logs

Jarred-Sumner commented 1 week ago

Tested that #5413 is not fixed by this

Reproduction: mono.zip