feat: Migrate to bun.lock

[pyranota]

(In backwards compatible way)

Read more: https://bun.sh/blog/bun-lock-text-lockfile

---

Important

Migrate from bun.lockb to bun.lock in a backwards-compatible way, updating lock file handling in bun_executor.rs and run.bun.config.proto.

• Behavior:
  • Migrate from bun.lockb to bun.lock in run.bun.config.proto and bun_executor.rs.
  • Update split_lockfile() to handle both bun.lock and bun.lockb formats.
  • Modify install_bun_lockfile() to use --save-text-lockfile.
• Functions:
  • Update split_lockfile() to return an additional boolean indicating if the lock file is binary.
  • Rename write_lockb() to write_lock() and update its logic to handle both text and binary lock files.
• Configuration:
  • Add mount for bun.lock in run.bun.config.proto.

^{This description was created by for 3e24c13. It will automatically update as commits are pushed.}

[cloudflare-workers-and-pages]

Deploying windmill with    Cloudflare Pages

Latest commit:	3e24c13
Status:	✅  Deploy successful!
Preview URL:	https://11295b5f.windmill.pages.dev
Branch Preview URL:	https://bun-lock-migration.windmill.pages.dev

View logs

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 3e24c13 in 2 minutes and 9 seconds

More details

• Looked at 223 lines of code in 2 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. backend/windmill-worker/src/bun_executor.rs:924

• Draft comment:
  The error message should be updated to reflect the new split pattern for bun.lock files.
• Reason this comment was not posted:
  Comment looked like it was already resolved.

2. backend/windmill-worker/src/bun_executor.rs:1615

• Draft comment:
  The error message should be updated to reflect the new split pattern for bun.lock files.
• Reason this comment was not posted:
  Marked as duplicate.

Workflow ID: wflow_zNlU76Hj6lxYekAe

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.