fix: upgrade brace-expansion to 5.0.8 (CVE-2026-14257)

[anupamme]

Summary

Upgrade brace-expansion from 5.0.6 to 5.0.8 to fix CVE-2026-14257.

Vulnerability

Field	Value
ID	CVE-2026-14257
Severity	HIGH
Scanner	trivy
Rule	CVE-2026-14257
File	package-lock.json
Assessment	Likely exploitable

Description: brace-expansion through 5.0.7 is vulnerable to denial of service via m ...

Evidence

Scanner confirmation: trivy rule CVE-2026-14257 flagged this pattern.

Production code: This file is in the production codebase, not test-only code.

Threat Model Context

This is a Node.js library - vulnerabilities affect downstream consumers who use this package.

Changes

• package.json
• package-lock.json

Behavior Preservation

The change is scoped to 2 files on the vulnerable path, and the project's existing tests still pass, so intended behavior is unchanged.

Verification

• Build passes
• Scanner re-scan confirms fix
• LLM code review passed

---

This change addresses a pattern flagged by static analysis. The code path handles user-influenced input and the fix reduces the attack surface against both manual and automated exploitation.

---

Automated security fix by OrbisAI Security

Summary by CodeRabbit

• Chores
  • Added a required package dependency to support application functionality.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 31b198c3-a2f3-4b56-b4c8-64bf94952c15

📥 Commits

Reviewing files that changed from the base of the PR and between 22f5617 and 21bd1ea.

⛔ Files ignored due to path filters (1)

• package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (1)

• package.json

---

📝 Walkthrough

Walkthrough

Added brace-expansion version ^5.0.8 as a direct dependency in package.json.

Changes

Dependency update

Layer / File(s)	Summary
Add brace-expansion dependency package.json	Adds brace-expansion version ^5.0.8 under dependencies.

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

Possibly related PRs

• smixs/iva#50: Updates the same brace-expansion dependency to version ^5.0.7.

Suggested reviewers: smixs

🚥 Pre-merge checks | ✅ 5

✅ 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 clearly matches the main change: upgrading brace-expansion to 5.0.8 to fix CVE-2026-14257.
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.

---

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

[smixs]

Thanks for the report — the brace-expansion advisory is real and confirmed by npm audit here. We fixed it in #72 with a slightly different shape, so I'm closing this PR in its favor:

• the bump is lockfile-only: Iva never imports brace-expansion directly, and minimatch accepts ^5.0.5, so adding it to dependencies in package.json isn't needed for the fix to hold;
• this PR's lockfile was regenerated with an older npm and dropped the libc fields on optional platform binaries, which current npm would immediately churn back;
• npm audit reported a second high advisory (fast-xml-parser, GHSA-8r6m-32jq-jx6q) — Patch DoS advisories in brace-expansion and fast-xml-parser #72 closes both at once.

Appreciate the scan and the nudge — it's credited in the changelog.

[anupamme]

Thanks for the quick turnaround and the detailed explanation; makes sense that a lockfile-only bump is the cleaner fix here, and good catch on the fast-xml-parser advisory too. Glad #72 covers both. Appreciate the credit in the changelog!