fix incompatible fast-xml-parser

[gugu]

Summary by CodeRabbit

• Chores
  • Removed version constraint override for fast-xml-parser to enable more flexible dependency resolution.

[coderabbitai]

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: 0baa7733-c7b6-4085-abd7-d0598816702a

📥 Commits

Reviewing files that changed from the base of the PR and between f42cea1 and 7383dcd.

⛔ Files ignored due to path filters (1)

• pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (1)

• package.json

💤 Files with no reviewable changes (1)

• package.json

---

📝 Walkthrough

Walkthrough

The pnpm.overrides constraint in package.json that forced fast-xml-parser versions below 5.7.0 to resolve to >=5.7.0 has been removed, allowing standard dependency resolution without this override.

Changes

Cohort / File(s)	Summary
Dependency Configuration package.json	Removed pnpm.overrides constraint for fast-xml-parser, eliminating forced version resolution above 5.7.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A rabbit hops with glee so bright,
One line removed—a config's flight!
Fast XML flows at its own pace,
No override holds back this race! 🐰✨

🚥 Pre-merge checks | ✅ 5 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Security Check	⚠️ Warning	The PR removes a pnpm override for fast-xml-parser that was added to address known security advisories, allowing potentially vulnerable versions to remain in the dependency tree.	Retain the fast-xml-parser override enforcing versions >=5.7.0, or update it to a newer safe version if incompatibility exists.

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: removing a pnpm override constraint on fast-xml-parser to resolve incompatibility.
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.

_{✏️ 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 rollback-pnpm-audit

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Removes an overrides entry related to fast-xml-parser, presumably to address a compatibility issue caused by the previously forced version range.

Changes:

• Removed the fast-xml-parser@<5.7.0 override mapping to >=5.7.0 from package.json.

Files not reviewed (1)

• pnpm-lock.yaml: Language not supported

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Given the PR title ('fix incompatible fast-xml-parser'), removing the override entirely may make installs drift over time (depending on transitive dependency resolution) and can allow reintroducing the previously-problematic version range. Consider pinning fast-xml-parser to a known-compatible range (either via overrides or an explicit direct dependency where it's used) and/or document in the PR description which versions are incompatible and why.

[Copilot]

If this repository uses a lockfile (e.g., package-lock.json, pnpm-lock.yaml, or yarn.lock), this change should be accompanied by a lockfile update so CI and local installs are reproducible and the override removal actually takes effect consistently.