Skip to content

Conversation

@jfox-box
Copy link
Contributor

@jfox-box jfox-box commented Nov 18, 2025

Summary by CodeRabbit

  • Chores
    • Updated UI framework and related asset packages to newer compatible minor versions to improve stability, compatibility, and visual assets.

@jfox-box jfox-box requested a review from a team as a code owner November 18, 2025 19:54
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 18, 2025

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • src/elements/content-sidebar/__tests__/__snapshots__/SidebarFileProperties.test.js.snap is excluded by !**/*.snap

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

Updates package.json to bump @box/blueprint-web from ^12.93.5 to ^12.98.0 and @box/blueprint-web-assets from ^4.79.1 to ^4.80.4 in both devDependencies and peerDependencies.

Changes

Cohort / File(s) Change Summary
Blueprint package bumps
package.json
Upgrades @box/blueprint-web -> ^12.98.0 and @box/blueprint-web-assets -> ^4.80.4 in both devDependencies and peerDependencies

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Files/areas to spot-check:

  • package.json dependency and peerDependency entries for exact version strings.

Possibly related PRs

Suggested reviewers

  • jpan-box
  • tjuanitas
  • greg-in-a-box

Poem

🐰 I hopped through package.json today,
Pushed versions forward, then away.
Blueprint raised its flag so tall,
Assets tuned to answer the call.
A tiny rabbit clap — upgrade, hooray! 🎉

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description contains only the template with merge process guidance and lacks substantive information about the changes, rationale, or what was updated. Add details about why blueprint-web was upgraded, what changes or improvements it includes, and whether blueprint-web-assets was also upgraded.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: upgrading blueprint-web to version 12.98.0.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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.

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 35080aa and 27bee84.

⛔ 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 (3)
📓 Common learnings
Learnt from: tjuanitas
Repo: box/box-ui-elements PR: 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
Repo: box/box-ui-elements PR: 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
📚 Learning: 2025-08-21T19:19:44.584Z
Learnt from: jpan-box
Repo: box/box-ui-elements PR: 4237
File: src/api/ZipDownload.ts:6-8
Timestamp: 2025-08-21T19:19:44.584Z
Learning: For Box TypeScript SDK integrations, jpan-box prefers deep imports (e.g., from box-typescript-sdk-gen/lib/schemas/...generated.d.ts.js) over defining local types, citing that Box v2 APIs don't typically have breaking changes, making the deep import approach more viable despite potential brittleness.

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). (3)
  • GitHub Check: lint_test_build
  • GitHub Check: Analyze (javascript-typescript)
  • GitHub Check: Summary
🔇 Additional comments (1)
package.json (1)

127-128: Approve: Consistent version bumps across dev and peer dependencies.

The versions are updated consistently in both devDependencies and peerDependencies sections, and the changes follow semantic versioning with caret ranges (^). Based on learnings, the team is comfortable with raising peerDependency minimum versions for blueprint-web packages.

Also applies to: 299-300

@jfox-box jfox-box requested a review from a team as a code owner November 18, 2025 20:21
@mergify mergify bot added the queued label Nov 18, 2025
@mergify mergify bot merged commit 7f91bd4 into master Nov 18, 2025
10 checks passed
@mergify mergify bot deleted the bump-blueprint-web-12.98.0 branch November 18, 2025 21:04
@mergify mergify bot removed the queued label Nov 18, 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