Skip to content

Conversation

@jfox-box
Copy link
Contributor

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

Summary by CodeRabbit

  • Chores
    • Bumped blueprint-related packages to newer compatible versions to pick up recent fixes and improvements.
    • Updated both development and peer dependency constraints so consumers and dev environments align with the new releases.
    • Confirmed no other dependencies or build scripts were modified.

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

coderabbitai bot commented Nov 6, 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 Blueprint package versions in package.json: @box/blueprint-web bumped from 12.92.3 to 12.93.5 and @box/blueprint-web-assets bumped from 4.76.5 to 4.79.0 in both devDependencies and peerDependencies.

Changes

Cohort / File(s) Summary
Dependency version updates
package.json
Bumps @box/blueprint-web from 12.92.3 to 12.93.5 in devDependencies and peerDependencies; bumps @box/blueprint-web-assets from 4.76.5 to 4.79.0 (devDependencies exact, peerDependencies ^4.79.0).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

ready-to-merge

Suggested reviewers

  • jpan-box
  • tjuanitas

Poem

🐰 I hopped through package.json with cheer,
Pushed blueprint versions, now up a tier,
Web and assets aligned in tune,
A tiny bump beneath the moon,
Boxes dance — dependency cheer!

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description contains only the repository's merge guidance template without any project-specific information about the changes, objectives, or rationale for the upgrade. Add a description explaining what changed, why the upgrade is important, and any testing or validation performed. The template content alone is procedural guidance, not substantive change documentation.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The PR title clearly and specifically summarizes the main change: upgrading blueprint-web to version 12.93.5.
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.

@jfox-box jfox-box force-pushed the upgrade-blueprint-web-12.93.5 branch from 499e871 to bee2d0d Compare November 6, 2025 19:36
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 4851c37 and bee2d0d.

⛔ 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). (2)
  • GitHub Check: lint_test_build
  • GitHub Check: Summary

@jfox-box jfox-box force-pushed the upgrade-blueprint-web-12.93.5 branch from bee2d0d to 8be3099 Compare November 6, 2025 19:46
jpan-box
jpan-box previously approved these changes Nov 6, 2025
Copy link
Contributor

@tjiang-box tjiang-box left a comment

Choose a reason for hiding this comment

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

Need to regenerate SidebarFileProperties snapshot

tjuanitas
tjuanitas previously approved these changes Nov 6, 2025
@jfox-box jfox-box dismissed stale reviews from tjuanitas and jpan-box via d0732a9 November 6, 2025 21:56
@jfox-box jfox-box requested a review from a team as a code owner November 6, 2025 21:56
@mergify mergify bot added the queued label Nov 6, 2025
@mergify mergify bot merged commit 84aaf2f into master Nov 6, 2025
10 checks passed
@mergify mergify bot deleted the upgrade-blueprint-web-12.93.5 branch November 6, 2025 22:13
@mergify mergify bot removed the queued label Nov 6, 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.

5 participants