Skip to content

Add symlink resolution and version extraction for Homebrew Poetry#342

Merged
karthiknadig merged 2 commits intomainfrom
persistent-silkworm
Feb 6, 2026
Merged

Add symlink resolution and version extraction for Homebrew Poetry#342
karthiknadig merged 2 commits intomainfrom
persistent-silkworm

Conversation

@karthiknadig
Copy link
Member

Fixes #137

Implement symlink resolution to retrieve the actual file path for Homebrew-installed Poetry and extract its version from the resolved path. This enhances compatibility with Homebrew's symlink structure.

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Test Coverage Report (Linux)

Metric Value
Current Coverage 58.2%
Base Branch Coverage 57.9%
Delta .3% ✅

Coverage increased! Great work!

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Performance Report (Linux) 🚀

Metric PR Baseline Delta Change
Server Startup 1ms 1ms 0ms 0%
Full Refresh 186ms 1002ms -816ms -80.0%

Legend
  • 🚀 Significant speedup (>100ms faster)
  • ✅ Faster than baseline
  • ➖ No significant change
  • 🔺 Slower than baseline (>100ms)
  • ⚠️ Significant slowdown (>500ms)

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Test Coverage Report (Windows)

Metric Value
Current Coverage 54.65%
Base Branch Coverage 54.49%
Delta 0.16% ✅

Coverage increased! Great work!

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Performance Report (Windows) ⚠️

Metric PR Baseline Delta Change
Server Startup 16ms 15ms 1ms 6.7%
Full Refresh 4642ms 2722ms 1920ms 70.5%

Legend
  • 🚀 Significant speedup (>100ms faster)
  • ✅ Faster than baseline
  • ➖ No significant change
  • 🔺 Slower than baseline (>100ms)
  • ⚠️ Significant slowdown (>500ms)

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Performance Report (macOS)

Metric PR Baseline Delta
Server Startup 1172ms 1068ms 104ms
Full Refresh 66845ms 64940ms 1905ms

Legend
  • 🚀 Significant speedup (>100ms faster)
  • ✅ Faster than baseline
  • ➖ No significant change
  • 🔺 Slower than baseline (>100ms)
  • ⚠️ Significant slowdown (>500ms)

@karthiknadig karthiknadig marked this pull request as ready for review February 6, 2026 05:27
@karthiknadig karthiknadig enabled auto-merge (squash) February 6, 2026 05:28
@vs-code-engineering vs-code-engineering bot added this to the February 2026 milestone Feb 6, 2026
@karthiknadig karthiknadig merged commit 3488eb1 into main Feb 6, 2026
30 checks passed
@karthiknadig karthiknadig deleted the persistent-silkworm branch February 6, 2026 05:37
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.

Detect poetry exes in /opt/homebrew/bin and resolve those symlinks to get to the actual install dir

2 participants