Skip to content

feat: add Obsidian frontmost note support - #263

Draft
johnyoonh wants to merge 1 commit into
Ji4n1ng:masterfrom
johnyoonh:feature/obsidian-cli-official-pr
Draft

feat: add Obsidian frontmost note support#263
johnyoonh wants to merge 1 commit into
Ji4n1ng:masterfrom
johnyoonh:feature/obsidian-cli-official-pr

Conversation

@johnyoonh

Copy link
Copy Markdown

Summary

  • add Obsidian as a supported editor option
  • use Obsidian's official CLI to resolve the frontmost active note when Obsidian is focused
  • open terminal actions in the active note's directory and editor actions with the active note file
  • add an iTerm-specific Obsidian path launch path so iTerm receives the resolved directory reliably
  • document the CLI lookup paths and Finder fallback behavior

Details

OpenInTerminal checks Obsidian's bundled obsidian-cli first, then common Homebrew CLI locations. If the CLI is unavailable, cannot resolve the active file, or the resolved file does not exist under the active vault, OpenInTerminal falls back to the existing Finder-based behavior.

Verification

  • xcrun swiftc -typecheck OpenInTerminalCore/*.swift OpenInTerminalCore/Extensions/*.swift OpenInTerminalCore/ScriptingBridge/*.swift
  • git diff --check
  • xcodebuild -workspace OpenInTerminal.xcworkspace -scheme OpenInTerminal -configuration Debug -destination 'platform=macOS,arch=arm64' -derivedDataPath /tmp/OpenInTerminalDerivedDataCli CODE_SIGNING_ALLOWED=NO COMPILER_INDEX_STORE_ENABLE=NO build\n- Manual: with Obsidian frontmost and an active note in a nested directory, Ctrl+Shift+Cmd+O opened Terminal in the active note parent directory\n- Manual: direct iTerm launch created an iTerm window in the active note parent directory\n- Manual: editor launch generated an open command targeting the active note file\n

feat: add Obsidian CLI support
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