Skip to content

Chore(deps): Remove unused override from package.json - #4773

Open
VariableVince wants to merge 2 commits into
openfrontio:mainfrom
VariableVince:remove-unused-override
Open

Chore(deps): Remove unused override from package.json#4773
VariableVince wants to merge 2 commits into
openfrontio:mainfrom
VariableVince:remove-unused-override

Conversation

@VariableVince

@VariableVince VariableVince commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Description:

Remove unused override for sanitize-html from package.json. It overrode a sub dependency of lit-markdown, when we still used lit-markdown. We don't anymore and there are no other dependencies using sanitize-html either.

The override was added in: b813792

It is no longer needed since: 94f2293

Please complete the following:

  • I have added screenshots for all UI updates
  • I process any text displayed to the user through translateText() and I've added it to the en.json file
  • I have added relevant tests to the test directory

Please put your Discord username so you can be contacted if a bug or regression is found:

tryout33

…y of lit-markdown, when we still used lit-markdown. We don't anymore and there are no other dependencies using sanitize-html either.

THe override was added in: openfrontio@b813792

It is no longer needed since:
openfrontio@94f2293
@coderabbitai

coderabbitai Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

The npm manifest removes the sanitize-html override that required version >=2.17.4. No other scripts or dependency entries are changed.

Changes

Dependency manifest

Layer / File(s) Summary
Remove sanitize-html override
package.json
Removes the npm override that constrained sanitize-html to version >=2.17.4.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

One override drifts away,
Let npm choose the path today.
A lighter manifest takes flight,
With fewer rules to hold it tight.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly matches the main change: removing an unused package.json override.
Description check ✅ Passed The description is directly related to the change and explains why the override was removed.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Fix failing CI checks

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.

@github-actions github-actions Bot added the small-fix Small fix (≤ 50 lines) — auto-applied by PR gate label Jul 29, 2026
@ryanbarlow97 ryanbarlow97 added this to the v33 milestone Jul 29, 2026
@ryanbarlow97
ryanbarlow97 enabled auto-merge July 29, 2026 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

small-fix Small fix (≤ 50 lines) — auto-applied by PR gate

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

2 participants