Skip to content

Update bun lockfile detection and use proper bun nerdfont icon - #590

Merged
IlanCosman merged 4 commits into
IlanCosman:mainfrom
saintwinkle:bun-lockfile
Aug 3, 2025
Merged

Update bun lockfile detection and use proper bun nerdfont icon#590
IlanCosman merged 4 commits into
IlanCosman:mainfrom
saintwinkle:bun-lockfile

Conversation

@saintwinkle

Copy link
Copy Markdown
Contributor

Description

Bun v1.1.39 introduced a new lockfile format bun.lock.
See: https://bun.com/blog/bun-lock-text-lockfile

Motivation and Context

Support for both bun.lockb and bun.lock format.

Screenshots (if appropriate)

None.

How Has This Been Tested

  • I have tested using Linux.
  • I have tested using MacOS.

Checklist

  • I am ready to update the wiki accordingly. (no need to update)
  • I have updated the tests accordingly.

Copilot AI review requested due to automatic review settings July 27, 2025 03:39
@saintwinkle
saintwinkle requested a review from IlanCosman as a code owner July 27, 2025 03:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the Bun lockfile detection to support both the legacy bun.lockb format and the new bun.lock text format introduced in Bun v1.1.39.

  • Added support for detecting bun.lock files in addition to bun.lockb
  • Updated tests to verify both lockfile formats work correctly

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
functions/_tide_item_bun.fish Extended lockfile detection to include both bun.lockb and bun.lock formats
tests/_tide_item_bun.test.fish Added test case to verify bun.lock detection works with Bun v1.1.39

@IlanCosman IlanCosman changed the title Update bun lockfile detection Update bun lockfile detection and use proper bun nerdfont icon Aug 3, 2025
@IlanCosman
IlanCosman merged commit bbcd806 into IlanCosman:main Aug 3, 2025
10 checks passed
efaver pushed a commit to efaver/tide that referenced this pull request Aug 21, 2025
…osman#590)

* Update bun lockfile detection

See: https://bun.com/blog/bun-lock-text-lockfile

* Update _tide_item_bun.test.fish

* only look for bun.lock, update icon

---------

Co-authored-by: Ilan Cosman <ilancosman@gmail.com>
@jetpks jetpks mentioned this pull request Sep 29, 2025
4 tasks
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.

3 participants