Skip to content

fix(deps): Upgrade draft-js immutable to 3.8.3#4474

Open
jfox-box wants to merge 3 commits intomasterfrom
draft-js-immutable-resolution-3.8.3
Open

fix(deps): Upgrade draft-js immutable to 3.8.3#4474
jfox-box wants to merge 3 commits intomasterfrom
draft-js-immutable-resolution-3.8.3

Conversation

@jfox-box
Copy link
Contributor

@jfox-box jfox-box commented Mar 9, 2026

Sets resolution for draft-js to 3.8.3 and bumps immutable ^4.0.0 to 4.3.8.

Summary by CodeRabbit

  • Chores
    • Pinned an internal library to version ^3.8.3 via dependency resolution to ensure consistent builds.
    • Applied the resolution at project configuration level; no user-facing behavior, public APIs, or exported interfaces were changed.
    • No other dependency resolutions or visible features were modified.

@jfox-box jfox-box requested a review from a team as a code owner March 9, 2026 23:22
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 9, 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: 46c0eb9e-9766-4648-9bd2-8b5937ea33ff

📥 Commits

Reviewing files that changed from the base of the PR and between d1ceeff and 6e7a8ac.

⛔ 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 as they are similar to previous changes (1)
  • package.json

Walkthrough

The PR updates dependency versions and adds a resolution: immutable bumps from ^3.7.4 to ^3.8.3 and draft-js/immutable is pinned to ^3.8.3. No public/exported declarations were changed.

Changes

Cohort / File(s) Summary
Dependency updates & resolutions
package.json, manifest_file
Bumped immutable from ^3.7.4 to ^3.8.3 and added a resolutions entry for draft-js/immutable set to ^3.8.3. No public API changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

ready-to-merge

Suggested reviewers

  • tjuanitas
  • jpan-box
  • reneshen0328

Poem

🐰 I hopped through package.json, light and spry,
Bumped immutable up and set a fix-up tie,
Pinned draft-js/immutable with a careful paw,
One small change — dependencies purr and thaw. 🥕✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description only contains the repository's merge guidance template without any project-specific change documentation beyond the single-line summary. Replace the boilerplate merge guidance with a meaningful description of why the immutable version is being upgraded and any potential impact on draft-js compatibility.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: upgrading the immutable dependency for draft-js to version 3.8.3.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch draft-js-immutable-resolution-3.8.3

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

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@package.json`:
- Around line 359-362: Update package.json to align the peerDependency for
immutable with the resolutions entry: change the peerDependency "immutable"
version range (currently "^3.7.4") to at least "^3.8.3" so consumers are
protected the same way as the local "resolutions" for "draft-js/immutable";
modify the "immutable" entry under peerDependencies accordingly and run a quick
install/test to ensure nothing else breaks.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 022ab8cd-aef0-46e6-b531-77a87730f84e

📥 Commits

Reviewing files that changed from the base of the PR and between c471350 and 0ead18b.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (1)
  • package.json

tjiang-box
tjiang-box previously approved these changes Mar 9, 2026
tjuanitas
tjuanitas previously approved these changes Mar 9, 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.

3 participants