bun lockfile generation

[Forceres]

Current Behavior

Lockfile creation is skipped

Expected Behavior

Bun now supports text lockfile support, but this plugin anyway skip lockfile generation.
File should be generated.

GitHub Repo

No response

Steps to Reproduce

1. Project with rspack and bun
2. Run build
3. Get warning that it skipped

Nx Report

NX   Report complete - copy this into the issue template

Node           : 22.18.0
OS             : linux-x64
Native Target  : x86_64-linux
bun            : 1.2.22

nx                     : 21.5.3
@nx/js                 : 21.5.3
@nx/jest               : 21.5.3
@nx/linter             : 16.0.0-beta.1
@nx/eslint             : 21.5.3
@nx/workspace          : 21.5.3
@nx/devkit             : 21.5.3
@nx/eslint-plugin      : 21.5.3
@nx/module-federation  : 21.5.3
@nx/nest               : 21.5.3
@nx/next               : 21.5.3
@nx/node               : 21.5.3
@nx/playwright         : 21.5.3
@nx/react              : 21.5.3
@nx/rollup             : 21.5.3
@nx/rspack             : 21.5.3
@nx/rsbuild            : 21.5.3
@nx/vite               : 21.5.3
@nx/vue                : 21.5.3
@nx/web                : 21.5.3
@nx/webpack            : 21.5.3
@nx/docker             : 21.5.3
typescript             : 5.9.2
---------------------------------------
Registered Plugins:
@nx/eslint/plugin
@nx/jest/plugin
@nx/next/plugin
@nx/vite/plugin
@nx/rsbuild
---------------------------------------
Community plugins:
@nx/nx-plugin : 16.0.0-beta.1
---------------------------------------
Cache Usage: 14.39 KB / 95.19 GB
---------------------------------------
The following packages should match the installed version of nx
  - @nx/[email protected]

To fix this, run `nx migrate [email protected]`

Failure Logs

●  ━━━━━━━━━━━━━━━━━━━━━━━━━ (93%) sealing asset processing                                                                                                             Error:   × caused by plugins in Compilation.hooks.processAssets
  ╰─▶   × TypeError: Cannot read properties of undefined (reading 'data')
        │     at node_modules/nx/src/plugins/js/package-json/create-package-json.js:165:55
        │     at Array.forEach (<anonymous>)
        │     at findProjectsNpmDependencies (node_modules/nx/src/plugins/js/package-json/create-package-json.js:163:33)
        │     at createPackageJson (node_modules/nx/src/plugins/js/package-json/create-package-json.js:25:21)
        │     at node_modules/@nx/rspack/src/plugins/utils/plugins/generate-package-json-plugin.js:30:64
        │     at Object.fn (node_modules/@rspack/core/dist/index.js:10260:29)
        │     at next (node_modules/@rspack/lite-tapable/dist/index.js:523:25)
        │     at AsyncSeriesHook.callAsyncStageRange (node_modules/@rspack/lite-tapable/dist/index.js:543:9)
        │     at node_modules/@rspack/lite-tapable/dist/index.js:86:18
        │     at new Promise (<anonymous>)

Package Manager Version

No response

Operating System

• macOS
• Linux
• Windows
• Other (Please specify)

Additional Information

No response

[Coly010]

Can you retry this with latest? There was work done on parsing bun lockfiles recently that should allow this to not fail.
A lockfile still wont be generated but the package.json will be.

Also, if you could provide a reproduction, that would help a lot.

[Forceres]

Can you retry this with latest? There was work done on parsing bun lockfiles recently that should allow this to not fail. A lockfile still wont be generated but the package.json will be.

Also, if you could provide a reproduction, that would help a lot.

Ok, I'll try to test it as soon as possible, thx.

[Forceres]

Can you retry this with latest? There was work done on parsing bun lockfiles recently that should allow this to not fail. A lockfile still wont be generated but the package.json will be.

Also, if you could provide a reproduction, that would help a lot.

Ok, package.json is now generated, but lockfile still not supported, but bun already supports it. Can we integrate bun lockfile generation to rspack?

[Coly010]

@Forceres It’s in our plans to support. As the log message states, running bun install is quite fast. We know it doesn’t ensure immutability.

I will change this issue to a feature request discussion