Skip to content

chore(desktop/demo): remediate non-breaking Dependabot vulns (#7327)#7368

Open
mdmohsin7 wants to merge 1 commit into
mainfrom
rex/desktop-demo-deps
Open

chore(desktop/demo): remediate non-breaking Dependabot vulns (#7327)#7368
mdmohsin7 wants to merge 1 commit into
mainfrom
rex/desktop-demo-deps

Conversation

@mdmohsin7
Copy link
Copy Markdown
Member

Part of #7327 — desktop Tier-3 (demo app). Branched from fresh main; single commit, lockfile-only.

Result

npm audit: 12 → 4 (0C/6H/2M/4L → 0C/0H/2M/2L). All 6 HIGH cleared via non-breaking npm audit fix. No package.json change.

Residual (4 — document-and-defer per #7327 Tier-3 judgment)

All dev/build-tooling, breaking-major-gated, demo-only:

pkg sev blocked by
eslint, @eslint/plugin-kit LOW eslint 8 → 9 major
@remotion/tailwind-v4, postcss MOD @remotion/tailwind-v4 4.x major

Not worth forcing breaking majors on a demo for low/moderate dev-tooling advisories — this matches the agreed Tier-3 rule (fix cheap non-breaking; defer breaking-major-gated with rationale).

Validation

npm run build (remotion bundle) → exit 0. Pre-existing npm run lint src errors are unrelated (a lockfile-only bump cannot introduce ESLint src-rule violations) and out of scope.

🤖 Generated with Claude Code

Tier-3 (demo app) — took the cheap non-breaking wins:
npm audit 12 -> 4 (0C/6H/2M/4L -> 0C/0H/2M/2L). ALL 6 HIGH cleared
via non-breaking `npm audit fix`. Lockfile-only, no package.json change.

RESIDUAL (4, document-and-defer per #7327 Tier-3 judgment — all
dev/build-tooling, breaking-major-gated, demo-only):
- eslint / @eslint/plugin-kit (LOW) — fix needs eslint 8 -> 9 major
- @remotion/tailwind-v4 / postcss (MOD) — fix needs
  @remotion/tailwind-v4 4.x major
Not worth forcing breaking majors on a demo for low/moderate
dev-tooling advisories.

Validated: `npm run build` (remotion bundle) exit 0. Pre-existing
`npm run lint` src errors are unrelated (a lockfile-only bump cannot
introduce ESLint src-rule violations) and out of scope.
@mdmohsin7 mdmohsin7 marked this pull request as ready for review May 18, 2026 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant