fix(ui): regenerate bun.lock for PR #61 - #110
Merged
nazarli-shabnam merged 2 commits intoJul 11, 2026
Merged
Conversation
package.json swapped lucide-react for @phosphor-icons/react but bun.lock was never updated, causing CI and Docker builds to fail at `bun install --frozen-lockfile`. Co-authored-by: Cursor <cursoragent@cursor.com>
The register page came from main with a lucide-react import, but PR nazarli-shabnam#61 removed that dependency. Use CircleNotch to match login/setup. Co-authored-by: Cursor <cursoragent@cursor.com>
nazarli-shabnam
approved these changes
Jul 11, 2026
nazarli-shabnam
merged commit Jul 11, 2026
4653ddf
into
nazarli-shabnam:feat/tactical-telemetry-redesign
4 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
apps/ui/bun.lockafter the Lucide → Phosphor icon migration in Feat/tactical telemetry redesign #61package.jsonalready lists@phosphor-icons/reactand removedlucide-react, but the lockfile still referencedlucide-react, so CI and Docker both failed atbun install --frozen-lockfileContext
Fixes the blocker on #61 (tactical telemetry redesign). CodeRabbit review items were already addressed in
d2c042a; this is the remaining CI failure after the Jul 9 merge frommain.Test plan
bun install --frozen-lockfile(apps/ui)bun run typecheckbun run lintbun run test(28 tests passed)bun run build(not run locally — Google Fonts fetch blocked by local SSL; should pass in CI)Made with Cursor