Skip to content

feat(blueprint-web): Bump blueprint web and other shared features version#4505

Merged
mergify[bot] merged 5 commits into
box:masterfrom
tjiang-box:bump-blueprint-web
Apr 17, 2026
Merged

feat(blueprint-web): Bump blueprint web and other shared features version#4505
mergify[bot] merged 5 commits into
box:masterfrom
tjiang-box:bump-blueprint-web

Conversation

@tjiang-box
Copy link
Copy Markdown
Collaborator

@tjiang-box tjiang-box commented Apr 15, 2026

Summary by CodeRabbit

  • Bug Fixes

    • Corrected button icon positioning in action menus and sidebar headers for improved UI consistency.
  • Improvements

    • Sidebar AI actions now reflect request-in-progress status.
    • Updated Box-related dependencies and added a new content-field package to dependency lists.
  • Tests

    • Added mocks for markdown-related libraries and adjusted unit test expectations for filter handling and float option mapping.
  • Chores

    • Included the new package in translation and test transform configurations.

@tjiang-box tjiang-box requested review from a team as code owners April 15, 2026 20:41
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 15, 2026

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: e1dc1c7d-0277-46e8-b5cd-4d44ef23fcd1

📥 Commits

Reviewing files that changed from the base of the PR and between 25bef99 and ecaf221.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (1)
  • scripts/i18n.config.js
✅ Files skipped from review due to trivial changes (1)
  • scripts/i18n.config.js

Walkthrough

This PR updates many @box/* dependency versions and adds @box/content-field; adds Jest mocks for react-markdown and remark-gfm and updates Jest transform allowlist; changes two Button props from icon to endIcon; updates metadata filter tests; and passes hasRequestInProgress to the AI sidebar clear button.

Changes

Cohort / File(s) Summary
Dependencies & Jest config
package.json, scripts/jest/jest.config.js
Bumped multiple @box/* devDependencies and peerDependencies; added @box/content-field@^1.40.23; added @box/content-field to Jest transform allowlist; mapped react-markdown and remark-gfm to local mocks.
Jest Mocks
scripts/jest/mocks/reactMarkdownMock.js, scripts/jest/mocks/remarkGfmMock.js
Added CommonJS-compatible mocks: ReactMarkdown renders a wrapper div with children; remarkGfm returns a no-op transformer.
Button icon prop changes
src/elements/common/sub-header/BulkItemActionMenu.tsx, src/elements/common/sub-header/SubHeaderRight.tsx
Replaced icon={...} with endIcon={...} on two buttons (adjusts icon placement).
Tests
src/elements/content-explorer/__tests__/MetadataViewContainer.test.tsx
Updated expected "All Filters" label from All Filters 2 to All Filters 4; changed float filter mapping expectations from 'range' to 'between' and added type assertions in test inputs.
AI sidebar prop
src/elements/content-sidebar/BoxAISidebarContent.tsx
Passed hasRequestInProgress={hasRequestInProgress} into ClearConversationButton.
i18n
scripts/i18n.config.js
Added @box/content-field to translationDependencies.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested labels

ready-to-merge

Suggested reviewers

  • jpan-box
  • tjuanitas
  • reneshen0328

Poem

🐰 I hopped through package lanes so wide,
Bumped boxes, added mocks to hide,
Swapped icons to the button’s rear,
Tests now hum a clearer cheer,
Chat button knows when work’s inside.

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description contains only boilerplate template text about merge queue procedures with no actual description of the changes being made. Provide a meaningful description of the changes: what dependencies were updated, why they were updated, and any migration notes or testing guidance for reviewers.
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 PR title clearly summarizes the main change: bumping Blueprint Web and related shared feature versions, which aligns with the dependency updates in package.json and related configuration changes.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

Comment thread src/elements/common/sub-header/BulkItemActionMenu.tsx
Comment thread package.json
Comment thread scripts/jest/jest.config.js
Comment thread scripts/jest/jest.config.js
Comment thread package.json
Copy link
Copy Markdown
Contributor

@jfox-box jfox-box left a comment

Choose a reason for hiding this comment

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

Nice work!

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 17, 2026

Merge Queue Status

  • Entered queue2026-04-17 17:30 UTC · Rule: Automatic strict merge
  • Checks skipped · PR is already up-to-date
  • Merged2026-04-17 17:31 UTC · at 50428f90bb3be7db92fb861aae2d7fcd5d3846ef · squash

This pull request spent 14 seconds in the queue, including 3 seconds running CI.

Required conditions to merge

@mergify mergify Bot merged commit f98eaae into box:master Apr 17, 2026
8 of 9 checks passed
@mergify mergify Bot removed the queued label Apr 17, 2026
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