Skip to content

Conversation

@tjiang-box
Copy link
Contributor

@tjiang-box tjiang-box commented Sep 25, 2025

Summary by CodeRabbit

  • Chores
    • Updated UI and metadata-related dependencies to newer releases.
    • Pulls in upstream fixes and compatibility improvements for metadata handling and item visuals.
    • No new features or user-facing UI changes expected; users should see more stable and consistent metadata and item displays.

@tjiang-box tjiang-box requested a review from a team as a code owner September 25, 2025 18:25
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 25, 2025

Walkthrough

I pity the fool — this PR bumps version ranges for several @box/* packages in package.json (devDependencies and peerDependencies); no other files or exported API changes were made. (≤50 words)

Changes

Cohort / File(s) Summary
Dependency version bumps
package.json
Updated @box/* package versions in devDependencies and peerDependencies: @box/box-item-type-selector ^0.63.12 → ^0.73.1, @box/item-icon ^0.17.15 → ^0.27.1, @box/metadata-filter ^1.19.2 → ^1.30.1, @box/metadata-view ^0.48.14 → ^0.54.0. No other files changed.

Sequence Diagram(s)

(omitted — change is dependency version updates only; no control-flow changes)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

ready-to-merge

Suggested reviewers

  • tjuanitas
  • jsenar
  • jpan-box

Poem

I pity the fool who stalls the merge,
Versions climbed up, now they converge,
No code was changed — just numbers surge,
Ship it fast, and let no conflict emerge.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning I pity the fool who doesn’t realize that there is no description provided and the template sections for ready-to-merge guidance and change details are completely missing from the pull request. I pity the fool who doesn’t add a pull request description following the repository’s template to include the ready-to-merge instructions, change summary, and any relevant details required for reviewers.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed I pity the fool who doesn’t notice that the title refers to bumping the metadata-view version which is indeed part of the changes, but it omits the other dependency updates so it only partially reflects the full scope of the pull request.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c9c54a6 and c1928b6.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (1)
  • package.json (2 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • package.json
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: lint_test_build
  • GitHub Check: Summary

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

jpan-box
jpan-box previously approved these changes Sep 25, 2025
greg-in-a-box
greg-in-a-box previously approved these changes Sep 25, 2025
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 33dc075 and 608c3ed.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (1)
  • package.json (2 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: tjuanitas
PR: box/box-ui-elements#4224
File: package.json:296-297
Timestamp: 2025-08-12T18:04:17.698Z
Learning: In the box-ui-elements project, the team is comfortable with raising peerDependency minimum versions when upgrading blueprint-web packages, even if it's a breaking change for consumers.
📚 Learning: 2025-08-12T18:04:17.698Z
Learnt from: tjuanitas
PR: box/box-ui-elements#4224
File: package.json:296-297
Timestamp: 2025-08-12T18:04:17.698Z
Learning: In the box-ui-elements project, the team is comfortable with raising peerDependency minimum versions when upgrading blueprint-web packages, even if it's a breaking change for consumers.

Applied to files:

  • package.json
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: lint_test_build
  • GitHub Check: Summary

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Issue Resolved

My previous comment about missing yarn.lock update was incorrect. The yarn.lock file was properly updated alongside the package.json changes in this PR. I apologize for the confusion!

@tjiang-box tjiang-box dismissed stale reviews from jpan-box and greg-in-a-box via 76a7676 September 25, 2025 21:04
greg-in-a-box
greg-in-a-box previously approved these changes Sep 25, 2025
jpan-box
jpan-box previously approved these changes Sep 25, 2025
@tjiang-box tjiang-box dismissed stale reviews from jpan-box and greg-in-a-box via c9c54a6 September 26, 2025 01:00
@mergify mergify bot added the queued label Sep 26, 2025
@mergify mergify bot merged commit f5887d7 into box:master Sep 26, 2025
8 checks passed
@mergify mergify bot removed the queued label Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants