Skip to content

[0.1.5] Use project Ruby SDK for all bundle exec docscribe commands - #32

Merged
unurgunite merged 1 commit into
v0.1.5from
feature/ruby-sdk-launch
Jul 5, 2026
Merged

[0.1.5] Use project Ruby SDK for all bundle exec docscribe commands#32
unurgunite merged 1 commit into
v0.1.5from
feature/ruby-sdk-launch

Conversation

@unurgunite

@unurgunite unurgunite commented Jul 5, 2026

Copy link
Copy Markdown
Member

Description

All bundle exec docscribe commands now run through the project's Ruby SDK instead of the system PATH Ruby. Previously, only the server startup path (ruby -e in startServerProcess()) used the SDK — the CLI fallback and gem check both ran bundle from the system PATH, potentially using a different Ruby version or gem set.

Changes

  • DefaultCommandExecutor — new environment: Map<String, String> constructor parameter, forwarded to GeneralCommandLine.withEnvironment(). Backward compatible (defaults to empty map).
  • buildSdkEnvironment() — new helper in DocscribeDaemon that resolves the Ruby SDK path via rubyCommand(), prepends SDK bin/ to PATH, and sets BUNDLE_GEMFILE.
  • performGemCheck() — applies SDK environment to ProcessBuilder so bundle exec docscribe --version uses the SDK Ruby.
  • fallback() — passes SDK environment to DefaultCommandExecutor so the CLI wrapper uses the SDK Ruby.
  • startServerProcess() — refactored to reuse buildSdkEnvironment() instead of inline env setup, reducing duplication.

Checklist

  • Tests pass (./gradlew test)
  • Formatting clean (./gradlew spotlessCheck)
  • Static analysis clean (./gradlew detekt)
  • Plugin verified (./gradlew verifyPlugin)
  • CHANGELOG updated

@unurgunite unurgunite self-assigned this Jul 5, 2026
@unurgunite unurgunite added the enhancement New feature or request label Jul 5, 2026
@unurgunite
unurgunite merged commit f0bdc1f into v0.1.5 Jul 5, 2026
1 check passed
@unurgunite
unurgunite deleted the feature/ruby-sdk-launch branch July 5, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant