Skip to content

fix: harden desktop dependency graph - #96

Merged
finalflash159 merged 2 commits into
mainfrom
fix/desktop-runtime-dependency-audit
Sep 16, 2026
Merged

finalflash159 merged 2 commits into
mainfrom
fix/desktop-runtime-dependency-audit

Conversation

@finalflash159

Copy link
Copy Markdown
Owner

Summary

  • remove the unused shadcn CLI from desktop runtime dependencies
  • vendor the exact static Tailwind variant stylesheet with its MIT license
  • declare the supported Node.js floor and update Vitest past the current mocker advisory
  • add regression coverage for the dependency boundary

Verification

  • npm ci (Node 22.22.0)
  • npm test --prefix desktop -- --run (263 passed)
  • npm run build --prefix desktop
  • npm audit --prefix desktop --audit-level=high --registry=https://registry.npmjs.org (0 vulnerabilities)
  • uv run pytest -q tests/test_desktop_dependency_manifest.py tests/test_desktop_macos_microphone_plist.py (5 passed)
  • uv run ruff check tests/test_desktop_dependency_manifest.py

@qodo-code-review

Copy link
Copy Markdown

ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing

@finalflash159
finalflash159 force-pushed the fix/desktop-runtime-dependency-audit branch from 9746079 to eeeb6e9 Compare September 16, 2026 21:04
@finalflash159
finalflash159 merged commit 5c97de1 into main Sep 16, 2026
7 checks passed
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