chore: upgrade blueprint packages by tjuanitas · Pull Request #4490 · box/box-ui-elements

tjuanitas · 2026-03-26T22:01:13Z

Summary by CodeRabbit

Chores
- Updated internal package dependencies to newer compatible versions for improved stability.
Bug Fixes
- Fixed invitee role ordering so sharing roles appear in a consistent, expected order.
- Defaulted collaboration messaging to off in the sharing modal while preserving existing shared-link email behavior.

coderabbitai · 2026-03-26T22:01:29Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7a967ae3-9dd3-49cf-adf0-18a5a31e0fd4

📥 Commits

Reviewing files that changed from the base of the PR and between d4aa75f and 5f6a832.

⛔ Files ignored due to path filters (1)

yarn.lock is excluded by !**/yarn.lock, !**/*.lock

📒 Files selected for processing (1)

package.json

✅ Files skipped from review due to trivial changes (1)

package.json

Walkthrough

Updated several @box/* dependency version ranges in package.json; ContentSharingV2 now forces collaborationMessage: false in the memoized UnifiedShareModal config; added SORTED_INVITEE_ROLES and changed collaborationRoles derivation to follow that canonical ordering before mapping.

Changes

Cohort / File(s)	Summary
Dependency changes `package.json`	Bumped version ranges in `devDependencies` and `peerDependencies` for multiple `@box/*` packages (e.g., `@box/blueprint-web`, `@box/blueprint-web-assets`, `@box/combobox-with-api`, `@box/copy-input`, `@box/item-icon`, `@box/types`, `@box/unified-share-modal`, `@box/user-selector`).
USM config update `src/elements/content-sharing/ContentSharingV2.tsx`	The memoized UnifiedShareModal config now explicitly sets `collaborationMessage: false` (keeps existing `sharedLinkEmail` behavior and forwards consumer `usmConfig` overrides).
Role ordering & mapping `src/elements/content-sharing/constants.js`, `src/elements/content-sharing/utils/convertItemResponse.ts`	Added exported `SORTED_INVITEE_ROLES` and changed `collaborationRoles` derivation to filter that canonical list by `allowed_invitee_roles` before mapping to `{ id, isDefault }` (mapping logic otherwise unchanged).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

fix(content-sharing): Improve content sharing performance and add consumer ability to override config #4413 — overlaps with how ContentSharingV2 builds/forwards USM config to UnifiedShareModal.
feat(content-sharing): add email shared link #4488 — touches content-sharing role mapping and API↔USM role mappings related to ordering logic.
fix(content-sharing): Collaborator current user #4328 — modifies content-sharing role handling and convertItemResponse, related to the new sorted roles constant.

Suggested reviewers

reneshen0328
jfox-box
jpan-box

Poem

🐰 I hopped through lines and set roles in place,
I told the modal: "no collab message, embrace,"
Packages nudged forward, versions in a row,
Roles sorted tidy, mapped with a glow,
A little rabbit clap — code ready to go!

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	No pull request description was provided by the author; the PR relies solely on the template comments.	Add a descriptive pull request description explaining the purpose of the version upgrades, impact on the codebase, and any testing performed to validate compatibility.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'chore: upgrade blueprint packages' accurately describes the main change: updating dependency versions for multiple `@box` blueprint and related packages in package.json.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch chore-upgrade-blueprint-packages

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.}

src/elements/content-sharing/ContentSharingV2.tsx

package.json

coderabbitai

🧹 Nitpick comments (1)

package.json (1)
301-302: Metadata package versions can remain independent from their peer declarations. While @box/metadata-filter and @box/metadata-view have newer peer minimums than dev versions, the team's precedent shows comfort with such mismatches during upgrades. This alignment is optional and not necessary for functionality.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@package.json` around lines 301 - 302, The package version mismatch is
acceptable; leave the `@box/metadata-filter` and `@box/metadata-view` entries as-is
— no change required to align their versions with peer declarations; if you
choose to adjust later, update the package.json entries for
"@box/metadata-filter" and "@box/metadata-view" to the desired versions and run
yarn/npm install, but for now keep the current versions unchanged.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@package.json`:
- Around line 301-302: The package version mismatch is acceptable; leave the
`@box/metadata-filter` and `@box/metadata-view` entries as-is — no change required
to align their versions with peer declarations; if you choose to adjust later,
update the package.json entries for "@box/metadata-filter" and
"@box/metadata-view" to the desired versions and run yarn/npm install, but for
now keep the current versions unchanged.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 31ba43fe-ae15-449e-a13f-c71a55663e59

📥 Commits

Reviewing files that changed from the base of the PR and between aabfda0 and ce69c11.

⛔ Files ignored due to path filters (2)

src/elements/content-sidebar/__tests__/__snapshots__/SidebarFileProperties.test.js.snap is excluded by !**/*.snap
yarn.lock is excluded by !**/yarn.lock, !**/*.lock

📒 Files selected for processing (3)

package.json
src/elements/content-sharing/ContentSharingV2.tsx
src/elements/content-sharing/utils/convertItemResponse.ts

src/elements/content-sharing/utils/convertItemResponse.ts

reneshen0328

LGTM!

jfox-box

Love it

mergify · 2026-03-26T23:45:14Z

chore: upgrade blueprint packages

ce69c11

tjuanitas requested review from a team as code owners March 26, 2026 22:01

tjuanitas commented Mar 26, 2026

View reviewed changes

src/elements/content-sharing/ContentSharingV2.tsx Show resolved Hide resolved

tjuanitas commented Mar 26, 2026

View reviewed changes

package.json Outdated Show resolved Hide resolved

coderabbitai bot reviewed Mar 26, 2026

View reviewed changes

tjuanitas commented Mar 26, 2026

View reviewed changes

src/elements/content-sharing/utils/convertItemResponse.ts Outdated Show resolved Hide resolved

tjuanitas added 2 commits March 26, 2026 16:14

fix: invitee roles

3b236aa

fix: upgrade

d4aa75f

reneshen0328 approved these changes Mar 26, 2026

View reviewed changes

jfox-box approved these changes Mar 26, 2026

View reviewed changes

tjuanitas added the ready-to-merge label Mar 26, 2026

mergify bot added the queued label Mar 26, 2026

Merge branch 'master' into chore-upgrade-blueprint-packages

5f6a832

mergify bot merged commit 5368ed3 into master Mar 26, 2026
12 checks passed

mergify bot deleted the chore-upgrade-blueprint-packages branch March 26, 2026 23:56

mergify bot removed the queued label Mar 26, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: upgrade blueprint packages#4490

chore: upgrade blueprint packages#4490
mergify[bot] merged 4 commits intomasterfrom
chore-upgrade-blueprint-packages

tjuanitas commented Mar 26, 2026 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Mar 26, 2026 •

edited

Loading

❌ Failed checks (2 warnings)

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

reneshen0328 left a comment

Uh oh!

jfox-box left a comment

Uh oh!

mergify bot commented Mar 26, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

tjuanitas commented Mar 26, 2026 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

coderabbitai bot commented Mar 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Poem

❌ Failed checks (2 warnings)

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

reneshen0328 left a comment

Choose a reason for hiding this comment

Uh oh!

jfox-box left a comment

Choose a reason for hiding this comment

Uh oh!

mergify bot commented Mar 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge Queue Status

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

tjuanitas commented Mar 26, 2026 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Mar 26, 2026 •

edited

Loading

mergify bot commented Mar 26, 2026 •

edited

Loading