Skip to content

fix(agents): detect macOS DeepSeek Harness desktop application and bundled cli - #485

Merged
lzt404 merged 4 commits into
router-for-me:devfrom
Roomen:fix/deepseek-harness-macos-detection
Oct 19, 2026
Merged

lzt404 merged 4 commits into
router-for-me:devfrom
Roomen:fix/deepseek-harness-macos-detection

Conversation

@Roomen

@Roomen Roomen commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Probe /Applications and ~/Applications for DeepSeek Harness.app on macOS instead of returning None
  • Extract the macOS desktop application version from Contents/Info.plist via read_macos_app_version
  • Fall back to managed and bundled CLI paths (.dsh/bin/dsh, DeepSeek Harness.app/Contents/Resources/runtime/cli/bin/dsh) when dsh is not in PATH
  • Add isolated path priority and bundled CLI regression tests in src-tauri/src/tests/deepseek_desktop.rs

Validation

  • TMPDIR=/private/tmp cargo test --locked --manifest-path src-tauri/Cargo.toml deepseek_desktop (4 passed)
  • bun run check (0 errors)
  • Verified detection of host /Applications/DeepSeek Harness.app (version 0.2.1-rc.2) and bundled CLI binary

lzt404 and others added 4 commits October 18, 2026 12:58
…ndled cli

- Probe /Applications and ~/Applications for DeepSeek Harness.app on macOS
- Extract macOS desktop bundle version via Info.plist CFBundleShortVersionString
- Fall back to managed and bundled CLI paths when dsh is not on PATH
- Add regression tests covering macOS root priority and bundled CLI discovery

Validation: cargo test deepseek_desktop (4 passed)
@lzt404
lzt404 merged commit cc32f1e into router-for-me:dev Oct 19, 2026
8 checks passed
@Roomen
Roomen deleted the fix/deepseek-harness-macos-detection branch October 20, 2026 06:02
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.

2 participants