chore(deps): jsdom 30, knip 6, Kotlin 2.4.10 / Gradle 9.7.1 / commonmark 0.30 — with the code changes the bumps need

[itsdestin]

Lands three dependabot bumps that were red because each needs a small accompanying change. Supersedes #271 (jsdom), #338 (gradle group), and #270 (knip) — those can be closed once this merges. Also unblocks the knip prerequisite for the TypeScript 7 bump (#242).

One commit per bump:

jsdom 29.1.1 → 30.0.1 (supersedes #271)

tests/dialog-shell.test.tsx failed because jsdom 30's CSSOM re-serializes math functions on the way in: min(476px, calc(100vh - 6rem)) reads back as min(476px, -6rem + 100vh). The two assertions compared the panel's inline style to the raw DIALOG_MAX_HEIGHTS constant, so they were pinning jsdom's spelling rather than the Dialog's behaviour. They now round-trip the constant through the same CSSOM and compare against that, with a non-empty guard so a jsdom that silently drops the value can't pass as '' === ''. jsdom 30 also parses height: min(...) (29 dropped it), so the fill test reads .style.height directly.

Gradle group: Kotlin 2.4.10, Gradle 9.7.1, commonmark 0.30.0 (supersedes #338)

Same four versions dependabot proposed (wrapper jar taken from its branch).

• app/build.gradle.kts: Kotlin 2.4 removed the android { kotlinOptions {} } DSL, so the build script itself stopped compiling → migrated to top-level kotlin { compilerOptions { jvmTarget.set(JvmTarget.JVM_17) } }.
• Three JVM unit tests used kotlin.io.createTempDir(), which Kotlin 2.4 makes a compile error → migrated to kotlin.io.path.createTempDirectory (the replacement PluginInstallerUpgradeTest already used).

knip 5.88.1 → 6.32.2 (supersedes #270)

What actually made npm run knip exit 1 is one new category: knip 6 reads command names out of execFile/spawn calls and reports "unlisted binaries" (ss, tasklist, rclone), which defaults to error. Verified: knip 6 with --exclude binaries exits 0 on the same tree. The 77 unused exports / 172 unused types are not new — they were already warn (knip 5 reports 81 / 379 on the same tree and exits 0), so no code was deleted.

knip.jsonc changes, each per a knip 6 configuration hint (all WHY-commented in the file):

• $schema → knip@6; classMembers rule dropped (issue type removed in v6).
• entry: main.ts / renderer/index.tsx / vite.config.ts removed — the vite and electron plugins derive them; listing them is flagged as redundant.
• ignore: ["dist/**"] removed (redundant).
• project now includes .css so CSS @imports are followed; that lets knip see @import "tailwindcss" in globals.css, so the tailwindcss ignoreDependencies entry is gone too.
• ignoreBinaries: ss (Linux, engine-supervisor port-owner lookup), tasklist (Windows, sync PID liveness), rclone (Drive sync transport, user-installed). binaries stays at error so a new unlisted binary still fails CI.

Result: npm run knip exits 0 with zero configuration hints.

Verification

• bash scripts/verify.sh worktrees/deps-desktop → OK — all checks passed (tsc, full vitest suite 606 files / 7790 tests, knip, eslint, ast-grep).
• JAVA_HOME=java-21 ANDROID_HOME=~/.android-sdk ./gradlew test -x bundleWebUi → BUILD SUCCESSFUL, 636 tests / 0 failures across debug, release, releaseTest.

For the reviewer

• The jsdom test helper (cssSerialized) is the one judgement call: it asserts "the panel carries exactly this cap as the CSSOM serializes it" rather than a literal string. The non-empty guard is what keeps it from being vacuous.
• No live code was deleted for knip; the exports/types warnings are the same backlog as before.

🤖 Generated with Claude Code

https://claude.ai/code/session_017RfiCWxEQuKBW4vM7GWkhY